Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add audit doc #1193

Merged
merged 1 commit into from
Sep 14, 2016
Merged

Add audit doc #1193

merged 1 commit into from
Sep 14, 2016

Conversation

soltysh
Copy link
Contributor

@soltysh soltysh commented Sep 9, 2016

This is copy of #1168 which already have tech approval (so I'm applying the label based on that) , I'm recreating it so it's part of 1.4.

@devin-donnelly ptal

@janetkuo @sttts fyi


This change is Reviewable

@devin-donnelly
Copy link
Contributor

Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.


docs/admin/audit.md, line 12 [r1] (raw file):

## Audit in Kubernetes

If this is where we're introducing the context of Audit in Kubernetes, it would be a good place to give a one- or two-sentence description of what Audit does and why it exists.

Something like "Kubernetes Auditing makes sure that..." or "Kubernetes Audit provides..."


docs/admin/audit.md, line 50 [r1] (raw file):

- `audit-log-maxsize` - specifies maximum size in megabytes of the audit log file before it gets rotated. Defaults to 100MB

Audit logs are being appended if the file already existed or a new one will be

"If an audit log file already exists, Kubernetes appends new audit logs to that file. Otherwise, Kubernetes creates an audit log file at the location you specified in audit-log-path. If the audit log file exceeds the size you specify in audit-log-maxsize, Kubernetes will rename the current log file by appending the current timestamp on the file name (before the file extension) and create a new audit log file.


docs/admin/audit.md, line 54 [r1] (raw file):

renamed by putting the current timestamp at the end name of the file name (before
the file's extension)
Whenever a new logfile get created, old log files may be deleted. This policy is

Kubernetes may delete old log files when creating a new log file; you can configure how many files are retained and how old they can be by specifying the audit-log-maxbackup and audit-log-maxage options.


Comments from Reviewable

@soltysh
Copy link
Contributor Author

soltysh commented Sep 12, 2016

Comments addressed, ptal.

@devin-donnelly
Copy link
Contributor

:lgtm:


Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.


Comments from Reviewable

@devin-donnelly
Copy link
Contributor

Looks good, but there are some merge conflicts. I can't merge until they're resolved. @soltysh , would you mind taking a look?

@soltysh
Copy link
Contributor Author

soltysh commented Sep 14, 2016

@devin-donnelly rebased

@devin-donnelly devin-donnelly added this to the 1.4 milestone Sep 14, 2016
@devin-donnelly
Copy link
Contributor

Thanks. Merging.

@devin-donnelly devin-donnelly merged commit d581566 into kubernetes:release-1.4 Sep 14, 2016
@soltysh soltysh deleted the audit_doc_14 branch September 15, 2016 09:29
mikutas pushed a commit to mikutas/k8s-website that referenced this pull request Sep 22, 2022
* add linkerd uses iptables blog

Signed-off-by: Christian Mejlak <[email protected]>
Okabe-Junya pushed a commit to Okabe-Junya/website that referenced this pull request Dec 4, 2023
* [es] Localize cloud-native-tech.md
* Revert change from resolving conflicts

Signed-off-by: Rodolfo Martínez Vega <[email protected]>
Okabe-Junya pushed a commit to Okabe-Junya/website that referenced this pull request Dec 4, 2023
* [es] Localize cloud-native-tech.md
* Revert change from resolving conflicts

Signed-off-by: Rodolfo Martínez Vega <[email protected]>
Okabe-Junya pushed a commit to Okabe-Junya/website that referenced this pull request Dec 4, 2023
* [es] Localize cloud-native-tech.md
* Revert change from resolving conflicts

Signed-off-by: Rodolfo Martínez Vega <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants