Skip to content
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

Syncing latest changes from master for rook #691

Closed
wants to merge 11 commits into from

Conversation

df-build-team
Copy link

PR containing the latest commits from master branch

BlaineEXE and others added 11 commits July 22, 2024 14:43
Add CephObjectStore spec.hosting.advertiseEndpoint configuration. This
provides a clear documented default for which endpoint Rook "advertises"
to dependent resources like CephObjectStores, OBCs, and COSI
Buckets/Accesses and allows users to override the default behavior if
desired.

The current default is to round-robin an endpoint from
spec.hosting.dnsNames, which has proven to be troublesome for some
users' object store configurations. This change provides much-needed
disambiguation for users.

This may be a breaking change for some existing spec.hosting.dnsNames
users. This is unexpected but is documented.

Signed-off-by: Blaine Gardner <[email protected]>
Fixes: rook#14518

A recent change seems to have broken the permissions of the auto-assign
action.

This tries to fix this by making permissions more specific.

Signed-off-by: Michael Adam <[email protected]>
ci: fix the permissions of the assign action
Fixes: rook#14518

The previous attempt to fix the auto-assign action

has introduced an indentation error in the workflow yaml file.

This should finally fix it, hopefully.

Signed-off-by: Michael Adam <[email protected]>
With the creation of the 1.15 branch, add the branch
to the mergify rules. Also remove the 1.10 backport
rules since no longer needed.

Signed-off-by: Travis Nielsen <[email protected]>
ci: Add 1.15 branch to mergify rules
ci: fix syntax of auto-assign action
object: add hosting.advertiseEndpoint config
add generated csv changes

Signed-off-by: Ceph Jenkins <[email protected]>
Syncing latest changes from upstream master for rook
@df-build-team df-build-team requested a review from a team August 1, 2024 17:46
Copy link

openshift-ci bot commented Aug 1, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: df-build-team

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 10, 2024
@travisn
Copy link

travisn commented Aug 21, 2024

Superseded by #706, and resync from master --> release-4.17 is no longer needed anyway

@travisn travisn closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants