Merge pull request #30 from iilyak/implement-format-status

Refactor config:set to be able to pass sensitive flag

Refactor config:set to be able to pass sensitive flag

The sensitive flag wouldn't log the value.

Merge pull request #29 from apache/notify_on_reload

Raise notify events on reload

Raise notify events on reload

When config is reloaded from disk, raise notification events
for values that have changed or are deleted.

Merge pull request #28 from cloudant/keep_features_on_config_restart

Keep features on config process restart.

Keep features on config process restart.

We used to store a list of enabled features in ets table owned by `config`
process. This meant that on every `config` process restart we loose the
list. Therefore welcome endpoint start to return empty list of features.

Use application environment to keep the list.

fixes #1930

Merge pull request #27 from apache/fix-regex-in-key

Fix regex patterns in keys

Update .travis.yml to match release versions used in main repo

Fix regex patterns in keys

Make sure to escape regex patterns in config keys

Merge branch 'COUCHDB-1380-fix-nasty-error' of github.com:cloudant/couchdb-config

Fix nasty 500 error on POST to /_config

Change the variable name from _Else to Else

Fix nasty 500 error on POST to /_config

Merge branch 'remove-R16B03-1-build' of github.com:cloudant/couchdb-config

Remove R16B03-1 build flavor

fix tests for #1602

config: tighten config validation

Merge remote-tracking branch 'cloudant/support-not-persist-variants'

Fix #19

Support not persist variants for helper functions

This adds support for specifying Persist argument for the following
helper functions:

 * set_boolean
 * set_integer
 * set_float

Merge branch 'use-callbacks-for-behavior'

Fixes #18

Use callback directive for config_listener behaviour

This knocks out a few dialyzer errors such as:

`Callback info about the config_listener behaviour is not available`

It is also more descriptive as it specifies types and argument names for each
callback.

Merge remote-tracking branch 'cloudant/issue-703'

Fixes #703

Add longer timeouts for operations which could write to disk

It turns out that 5 seconds is often not enough in a severly throttled test
environment, and simple operations like config:set and config:delete raise
timeout errors.

Increase default 5 second timeout to half a minute. This should hopefully
handle even heavily throttled IO environments.

Fixed #703

Replace couch_log calls with mocks

Depending on the order of test suite invocations its possible that the
config suite will end up with -DTEST compiled versions from couch_log's
eunit test suite. These have syncrhonous calls where the normal versions
are all gen_server:cast/2 calls. This breaks config's test suite when
since couch_log is not started.

Rather than go through the hoops of getting couch_log started (which
causes a circular dependency) I've just mocked out the calls so that
they don't cause issues.

Merge branch 'fix-eunit-tests'

Make sure to stop the correct applications

We need to make sure and stop all applications that we started.

Let test_util figure out our deps

Dependencies are already listed in config.app.src so we should just let
the test_util figure out what else needs starting.

Features API and implementation.

Implement ability to enable, disable and query feature flags.

Features are identified as atoms. Usage intent is for various components in the
system to enable features, then the HTTP API will expose those to the user.
For example, features could indicate the presence of an optional component, a
plugin or a new mode of operation.

The API has 3 functions:

 * `config:features/0` : Return a sorted list of feature flags

 * `config:enable_feature/1` : Enables a feature. Feature argument could be a
   list, a binary or an atom.

 * `config:disable_feature/1` : Disables a feature.

Implementation is a thin wrapper around setting and deleting keys from the
'[features]' config section. This means, users can also set their own features
there via the .ini config files. Features set via the API are not persistent,
so applications will have to set them every time they initialize.

COUCHDB-3180

Merge remote branch 'cloudant:fix-return-value'

This closes #13

Signed-off-by: ILYA Khlopotov <iilyak@ca.ibm.com>

handle_event expected to return {ok, State}

Fix return value of handle_event callback.

COUCHDB-3102

Merge remote branch 'cloudant:3102-fixup-for-config_subscription'

This closes #12

Signed-off-by: ILYA Khlopotov <iilyak@ca.ibm.com>

Add config_listener_mon:start_link/2

This a fixup commit for f09a2eb7d

COUCHDB-3102

Merge remote branch 'cloudant:3102-restart-monitor'

This closes #11

Signed-off-by: ILYA Khlopotov <iilyak@ca.ibm.com>

Make sure we don't subscribe multiple times

Check there is no subscription from given Pid in subscribe_for_changes

COUCHDB-3102

Return `{ok, Pid}` from config_listener_mon:subscribe/2

Add config:subscribe_for_changes/1

Add a new gen event handler which sends plain events to the Subscriber.

COUCHDB-3102

Fix compilation warning

Consider only needed handlers in n_handlers

When testing. Make sure we don't count other config_event handlers
present in the system.

Merge branch '3096-fix-config-listener-accumulation'

Fix config listener event handler registration

We rely on `gen_event:add_sup_handler/3` to remove handlers when the
process that registered for events exits. On master this was changed so
that config becomes the process that's monitored by gen_event. As such
any handler that is registered (say, for when an index is opened) adds a
handler to the config_event gen_event process. Since the config process
never exits these handlers are never removed.

The end result of all of this is that on a busy cluster the config_event
process will end up with millions of handlers consuming many gigabytes
of RAM.

This change creates a monitor process for every event handler. This
monitors the process wanting to listen for config changes and exits when
the requesting process exits. This means that we maintain our pure
callback API improvement while correctly removing handlers.

COUCHDB-3096

Clean up config_tests

This cleans up the config_tests both stylistically as well as removes
some race conditions around message passing from the config handler.

This also reformats and changes a lot of the tests so that we're using
consistent patterns through out the file. Unfortunately foreach and
foreachx are terrible constructs and require using the `?_test/1` macro
which is a bit annoying but ended up being the least worst approach I
could find.

COUCHDB-3096

Consult default.d/local.d for ini files

COUCHDB-3089

Merge branch 'enable-read-concurrency'

Enable read_concurrency for config ets table

Merge remote-tracking branch 'github/pr/7'

Don't set real log levels in tests

Setting the actual "log" "level" in tests triggers extraneous logging,
which is undesirable to have interleaved into test output.

Remove unnecessary application dependencies

This removes all but the minimum applications necessary to pass the tests.

Previously, references to some extra applications such as lager would cause
test failures in downstream builds which omitted those applications.

COUCHDB-2866

Update handlers counter in tests

Somehow, but here we depend on existed and registered config
changes listeners. This change respects recent couch_log update
which subscribes to config changes and eventually bumps this counter.

Fix typo in tests: s/beahiour/behaviour/

Update Travis config

- Clone faster
- Remove unsupported R14B04
- Use containers
- Use new better way to run specific app tests

Add tests for to_integer and to_float

Fix to_float(Int)

Fix to_integer and to_float

Integrate with Travis CI

Add LICENSE file

allow atoms too (chttpd, bind_address)

Merge remote-tracking branch 'iilyak/2561-make-config-API-consistent'

This closes #2

COUCHDB-2561

Enforce type verification for config:set/get

Check the type of given default value to make sure it is supported.
Raise error(badarg) from set/get on type missmatch.
Add tests for the feature

COUCHDB-2561

Callback driven API for config_listener behaviour

This implementaion replaces a mix of message driven and callback driven
API for config_listener. In particular it replaces gen_event_EXIT
message with a call to Module:handle_config_stop(Pid, Reason, State).
This fixes the problem of using config:listen_for_changes in supervisor
context where there is no way to handle arbitrary messages.

COUCHDB-2561

Change setup/teardown logic and enable some tests

COUCHDB-2561

Remove tests for dropped register functionality

COUCHDB-2561

start couch_stats last to ensure we load couch_log stats

start couch_stats for test suite

Use couch_eunit include paths

Disable problematic tests

The couch_httpd port is now 5986

Be explicit about starting and stopping deps

Switch to using config instead of couch_config

Use updated couch_eunit.hrl path macros

Update include paths

Move files out of test/couchdb into top level test/ folder

Port 083-config-no-files.t etap test suite to eunit

Merged into couch_config_tests suite.

Port 082-config-register.t etap test suite to eunit

Merged into couch_config_tests suite.

Port 081-config-override.t etap test suite to eunit

Merged into couch_config_tests suite.
Setup fixtures.

Port 080-config-get-set.t etap test suite to eunit

Merge remote-tracking branch 'upstream/master' into 1843-feature-bigcouch

Conflicts:
src/config.erl

Change API to function per level

Switch to couch_log

Let rebar build the module list

Build with rebar

Fix up copyright headers

Fix core etap tests

Add a config:reload/0 and HTTP trigger

Theoretically this should prevent all of those annoying test suite
failures when a test fails in with a temporary config set and fails to
undo its changes.

This works by storing the list of INI files in the config server and on
command will clear its ets table and re-read data from disk thus
clearing its cache of non-persisted values. Obviously this isn't
something that should be relied on in production settings.

Rewrite the boot procedure for the couch app

We had a lot of historical cruft lying around from before people knew
hot OTP worked. This does some cleanup to make things a bit more tidy
through this section but we still have a long ways to go.

Allow literal dot in config section names

BugzID: 15952

New build system for config

Remove Cloudant build system remnants

Merge pull request #5 from cloudant/25731-type-safety

Type-safe config:get and :set methods

Guard get/set/delete to require list arguments

BugzID: 25731

Type-safe config:get and :set methods

BugzID: 25731

set module version to 1

Merge pull request #4 from cloudant/20326-config-reasons

Add a "Reason" parameter to config:set and config:delete

Add a "Reason" parameter to config:set and config:delete

This is intended to support better auditing of config changes. Existing code
and tools will be handled gracefully.

BugzID: 20326

Merge pull request #3 from cloudant/15952-allow-dots

Allow literal dot in config section names

Allow literal dot in config section names

BugzID: 15952

Merge pull request #2 from cloudant/log-config-changes

Log all config changes

Log all config changes

Merge pull request #1 from cloudant/13179-config-refactor

Allow multiple listeners per callback module

Allow multiple listeners per callback module

Useful for couch_rep_httpc.

BugzID: 13179

Check that default config files exist

Avoid trying to load the default config files if they don't exist.