Syncing latest changes from upstream main for ceph-csi-operator #52
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
controller-runtime accepts the pointer to the memory which is reserved by application and runtime deepcopies the object in it's cache to the memory provided by application. this commit removes extra copy of driver spec which isn't required. Signed-off-by: Leela Venkaiah G <lgangava@ibm.com>
Signed-off-by: Leela Venkaiah G <lgangava@ibm.com>
Signed-off-by: Leela Venkaiah G <lgangava@ibm.com>
added a quick start guide for the minimal installation Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
csiaddons required new RBAC in the next release to create/update the csiaddonsnode object based on the owner deployment/daemonset names of the pods its running with. Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
remove extra deep copy of object from cache
doc: add quick-start guide
csiaddons: add new RBAC
provide ability to adjust deployment strategy
Links were broken due to using a nonexistent devel branch. Signed-off-by: Michael Adam <obnox@samba.org>
I the operator design document, custom resource definition(CRD) seems to have been used as a synonym for custom resource(CR) which is usually not correct. This change makes the wording a little clearer by using CRD only when technically correct and using CR when appropriate. Signed-off-by: Michael Adam <obnox@samba.org> Update docs/design/operator.md Co-authored-by: nb-ohad <ohad.mitrani@noobaa.com>
docs: clarify the operator design doc a bit
Fix broken links in the PR template
df-build-team
force-pushed
the
sync_us--main
branch
4 times, most recently
from
e4d74d2 to
f0a6ce9
Compare
added 2 commits
November 11, 2024 03:03
…-main Signed-off-by: Ceph Jenkins <ceph-jenkins@redhat.com>
add generated bundle changes Signed-off-by: Ceph Jenkins <ceph-jenkins@redhat.com>
df-build-team
force-pushed
the
sync_us--main
branch
from
f0a6ce9 to
cb33e40
Compare
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: df-build-team, Madhu-1 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.
PR containing the latest commits from upstream main branch