Syncing latest changes from main for kubernetes-csi-addons #85
Conversation
When kubernetes-csi-addons are included in the ODF product, we need to document the review/backport guidelines and list the maintainers. This repository is currently part of the responsibilities of the Ceph-CSI team. Signed-off-by: Niels de Vos <ndevos@redhat.com> (cherry picked from commit 6b32ad6)
Dependabot does not need to report available updates for vendored dependencies in the downstream repository. Updates to dependencies are synced from the upstream repository when needed. There is also the "Upstream First" requirement, which we follow closely. See-also: https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/configuration-options-for-dependency-updates#open-pull-requests-limit Signed-off-by: Niels de Vos <ndevos@redhat.com> (cherry picked from commit 6e4f4ba)
As ODF-Operator is used for deploying the CSI-Addons components, the CSI-Addons operator does not need to be visible in the OperatorHub. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2042997 Signed-off-by: Niels de Vos <ndevos@redhat.com> (cherry picked from commit 15e9b1a)
There is no upstream icon for the CSI-Addons project yet. All operators are required to have an icon before they can be published. Until there is an icon in upstream, use the standard Red Hat icon for product builds. Signed-off-by: Niels de Vos <ndevos@redhat.com>
Sync upstream main branch to downstream
This commits add instructions to resolve conflicts if they arise while syncing upstream changes. Signed-off-by: Rakshith R <rar@redhat.com>
[DOWNSTREAM-ONLY]: document process for resolving conflicts
Sync rhs:main with csi-addons:main NetworkFence: Explicitly mentioned Fenced and Unfenced state
sync the upstream main branch
Sync the upstream main branch
sync downstream with upstream
Yug left the company, so removing him from the team. Signed-off-by: Niels de Vos <ndevos@redhat.com>
[DOWNSTREAM-ONLY] remove Yug from OWNERS
Sync the upstream changes from `csi-addons/kubernetes-csi-addons:main` into the `main` branch
Sync upstream main branch
Depandabot automatic updating for GitHub Actions is not wanted. Changes should flow in from the upstream repository instead. See-also: 945da36 Signed-off-by: Niels de Vos <ndevos@redhat.com>
[DOWNSTREAM-ONLY] ci: disable Dependabot PR creation
sync downstream main with upstream main
sync downstream main with upstream main
sync downstream main with upstream main branch
sync downstream main with upstream main branch
sync downstream main with upstream main branch
sync downstream main with upstream main branch
Sync the upstream changes from `csi-addons/kubernetes-csi-addons:main` into the `main` branch
Sync the upstream changes from csi-addons/kubernetes-csi-addons:main into the main branch
sync downstream main with upstream main
[DOWNSTREAM-ONLY] ci: disable Dependabot PR creation
|
The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the red-hat-storage org. You can then trigger verification by writing
|
|
@Nikhil-Ladha it looks like the owner files is not valid in this repo |
These are the downstream-only changes that are part of the /lgtm |
| ## Backports | ||
|
|
||
| All changes in this repository are *backports* from the [upstream | ||
| project][https://github.com/ceph/ceph-csi]. There should be no functional |
There was a problem hiding this comment.
we should not have cephcsi link here
There was a problem hiding this comment.
these things need to be corrected in the main branch.
There was a problem hiding this comment.
Well, there is no point in requesting changes here. These are just automated sync PRs that fetch commits and merge to release branch.
We can suggest and add commits to these or update the main branch and backport from there to avoid these type of issues.
| All changes in this repository are *backports* from the [upstream | ||
| project][https://github.com/ceph/ceph-csi]. There should be no functional | ||
| changes (only process/CI/building/..) in this repository compared to the | ||
| upstream project. Fixes for any of the release branches should first land in |
There was a problem hiding this comment.
| upstream project. Fixes for any of the release branches should first land in | |
| upstream project. Fixes for any of the release branches should first land in |
|
/lgtm |
Addressed with #86 |
|
Codespell is expected to be addressed in upstream csi-addons#465 , needs backporting to the |
|
/approve All required changes have been posted for the |
|
@nixpanic: Overrode contexts on behalf of nixpanic: codespell 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. |
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: df-build-team, Madhu-1, nixpanic 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 |
|
/override test-lint |
|
@nixpanic: Overrode contexts on behalf of nixpanic: test-lint 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. |
Reported-in: #85 Signed-off-by: Niels de Vos <ndevos@ibm.com>
Reported-in: red-hat-storage#85 Signed-off-by: Niels de Vos <ndevos@ibm.com>
Reported-in: #85 Signed-off-by: Niels de Vos <ndevos@ibm.com>
PR containing the latest commits from main branch