audit-log: Add controller config for audit logging #8210

Merged
merged 2 commits into from Dec 13, 2017

Conversation

Projects
None yet
3 participants
Member

babbageclunk commented Dec 13, 2017

Description of change

Plumb the audit logging configuration options into controller config.

QA steps

Bootstrap a controller without auditing enabled (the default) - /var/log/juju/audit.log isn't created. Kill the controller and bootstrap again with config settings:

auditing-enabled=true
audit-log-max-size=100M
audit-log-max-backups=5
audit-log-capture-args=true

See that the audit log file is created and it contains audit log entries.
Check that the options above are shown in the output of juju controller-config.

Documentation changes

These controller config options will need to be documented.

Member

babbageclunk commented Dec 13, 2017

$$merge$$

Contributor

jujubot commented Dec 13, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju

@jujubot jujubot merged commit 82d9eb6 into juju:2.3 Dec 13, 2017

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details

@babbageclunk babbageclunk deleted the babbageclunk:audit-config branch Dec 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment