-
Notifications
You must be signed in to change notification settings - Fork 26
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 SupportBundle collection functionalities #140
Conversation
1616468
to
cbaaa8e
Compare
fc0d758
to
574ce22
Compare
/theia-test-e2e |
574ce22
to
1477caa
Compare
982bc6b
to
6d31169
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM, some nits.
Signed-off-by: Shawn Wang <wshaoquan@vmware.com>
Signed-off-by: Shawn Wang <wshaoquan@vmware.com>
Signed-off-by: Shawn Wang <wshaoquan@vmware.com>
6d31169
to
15a2f86
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, check this late due to the sick leave. LGTM, thanks!
@wsquan171 no additional comments from me. I think we can merge this PR. |
This PR adds support bundle collection functionalities to Theia manager.
A new API endpoint
apis/system.theia.antrea.io/v1alpha1/supportbundles
is added to trigger / retrieve log bundle collection. The following logs will be collected as part of the operation:/var/log/antrea/flow-aggregator/
)/var/log/clickhouse-server/
)/var/log/grafana/
)The following file structure is to be expected after successful collection:
Logging configuration of ClickHouse and Grafana are also added to enable better configuration of log level and detention control.
To trigger log collection, or retrieve a log bundle job that finished earlier, simply run:
theia supportbundle
Theia CLI optional arg
--since
currently only supports filtering logs collected from Theia manager and Flow aggregator. Support of other components will be added in a future change.Close #62