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

Update audit configuration description #3269

Merged
merged 1 commit into from
Nov 29, 2016

Conversation

soltysh
Copy link

@soltysh soltysh commented Nov 22, 2016

With k8s 1.4 we switched to using upstream audit mechanics, which gave us a few additional options. I've updated the description with those new options.

@ahardin-rh ptal

@@ -689,7 +689,8 @@ Each audit log contains two entries:
.. The source IP of the request
.. The HTTP method being invoked
.. The original user invoking the operation
.. The impersonated user for the operation
.. The impersonated user for the operation (`self` meaning himself here)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@soltysh Is it accurate to say that self means the user's self?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's basically himself. When you're not using --as flag you're acting as yourself, this is when self will show up here, which is most of the time. When you do specify --as flag for any command this will be the username which you specified as the flag value. I hope that makes sense :)

@ahardin-rh
Copy link
Contributor

@soltysh Thank you! I can do some minor style edits in a follow-up PR.

@ahardin-rh ahardin-rh merged commit ab2a675 into openshift:master Nov 29, 2016
@ahardin-rh
Copy link
Contributor

[rev_history]
|xref:../install_config/master_node_configuration.adoc#install-config-master-node-configuration[Master and Node Configuration]
|Updated the xref:../install_config/master_node_configuration.adoc#master-node-config-audit-config[Audit Configuration] section description and added Audit Configuration Parameters.
%

@soltysh soltysh deleted the update_audit branch November 30, 2016 08:09
@ahardin-rh ahardin-rh modified the milestones: Next Release, Staging, Published 01/26/2017 Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants