Sync the release-4.17 branch with respect to main branch #200
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
this commit refactors the collection of resources in all namespaces to reduce code duplicacy. Signed-off-by: yati1998 <[email protected]>
This PR adds support of fetching the pod metadata using the kubernetes API. An additional pod-metadata.json file is created in the report with the dump of entire metadata for the pod. A much simpler debug log stating the image version being used is printed to the console as well. Signed-off-by: Niraj Yadav <[email protected]>
This PR updates the ImagePullPolicy for the must-gather pod to Always. This adds value to the development phase where one can just delete the pod and have an updated one with the latest image. Prior to this one had to use different tags for each test which was not ideal. Note: This would not increase pod deploy timinigs in prod env as we do not update the prod image that often and ImagePullPolicy of always first checks the SHA sum from registry against local cache. If the image is cached locally, it is not pulled again. Signed-off-by: Niraj Yadav <[email protected]>
refactor collection of resources in all namespace
Update ImagePullPolicy to Always
this commit collects new crds details which are newly created by ceph-csi-operator Signed-off-by: yati1998 <[email protected]>
Include pod metadata in must-gather output
add new crds created by ceph csi operation
currently client operator has started using csi operator so collecting the csi logs from a new host path hat csi-operator supports Signed-off-by: parth-gr <[email protected]>
Bug 2308101: logrotate: add logrotate functionality for csi operator
currently client operator has started using csi operator so collecting the csi logs from a new host path hat csi-operator supports Signed-off-by: parth-gr <[email protected]>
Bug 2308101: logrotate: add logrotate functionality for csi operator
Signed-off-by: Niraj Yadav <[email protected]>
Add missing PIDs while collecting ceph logs
This patch fixes the failing local builds and also cleans up the Dockerfile Signed-off-by: Niraj Yadav <[email protected]>
|
@yati1998: No Bugzilla bug is referenced in the title of this pull request. 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-sigs/prow repository. |
build: Fix local builds and cleanup Dockerfile
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: black-dragon74, yati1998 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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There are multiple PRs which was not backported to 4.17,
syncing them all here.