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
Bug 1959650: Gather SDI-related MachineConfigs #422
Bug 1959650: Gather SDI-related MachineConfigs #422
Conversation
Skipping CI for Draft Pull Request. |
bc59d90
to
9e0eae0
Compare
@natiiix: This pull request references Bugzilla bug 1959650, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
Requesting review from QA contact: In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@natiiix: This pull request references Bugzilla bug 1959650, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Also added error dump to log for debugging.
9e0eae0
to
3eaf728
Compare
/retest |
unfortunately I don't have a SAP cluster so I can't really test it, but I went through the code and run the extensive unittest for this change and everything works as expected. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: 0sewa0, natiiix The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
7 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@natiiix: All pull requests linked via external trackers have merged: Bugzilla bug 1959650 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This PR implements an enhancement gathering two MachineConfigs related to the SAP Data Intelligence workload. This enhancement will not be backported past v4.8.
Description from Jira:
Implementation note: Most SAP-related enhancements first check for the
datahubs
resource to check if the cluster is running an SDI payload and collect nothing if the resource is not present on the cluster. In the case of this enhancement, this check is intentionally omitted because there should be no way for any of the gathering conditions to be met on a non-SDI cluster and the MachineConfigs are a part of core configuration, which could be present even if thedatahubs
resource was missing.Categories
Sample archive
docs/insights-archive-sample/config/machineconfigs/75-worker-sap-data-intelligence.json
docs/insights-archive-sample/config/machineconfigs/99-sdi-generated-containerruntime.json
Documentation
docs/gathered-data.md
-SAPMachineConfigs
sectionUnit Tests
pkg/gather/clusterconfig/sap_machine_configs_test.go
Privacy
Yes. There are no sensitive data in the newly collected information.
After a discussion with a SAP team representative, it was determined that the selected MachineConfigs should not contain any sensitive information under normal circumstances.
Changelog
No.
References
Jira Task: https://issues.redhat.com/browse/CCXDEV-4829
Bugzilla Bug: https://bugzilla.redhat.com/show_bug.cgi?id=1959650