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

feat: stop requiring namespace object permissions #4753

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

mnencia
Copy link
Member

@mnencia mnencia commented Jun 2, 2024

Remove the namespace deletion check at the start of the reconciliation
cycle and the permissions to read namespace definitions.

Closes #4752

@mnencia mnencia requested a review from a team as a code owner June 2, 2024 07:58
@github-actions github-actions bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.21 release-1.22 release-1.23 labels Jun 2, 2024
Copy link
Contributor

github-actions bot commented Jun 2, 2024

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@mnencia mnencia force-pushed the dev/no-namespace branch 2 times, most recently from 3e5f824 to e4e84a4 Compare June 2, 2024 07:59
@mnencia
Copy link
Member Author

mnencia commented Jun 2, 2024

This PR requires #4735 to be merged first.

@mnencia
Copy link
Member Author

mnencia commented Jun 2, 2024

This PR supersedes #4737 and closes #4736

Remove the namespace deletion check at the start of the reconciliation
cycle and the permissions to read namespace definitions.

Closes #4752

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
@mnencia mnencia changed the title Stop requiring namespace object permissions feat: stop requiring namespace object permissions Jun 3, 2024
@mnencia
Copy link
Member Author

mnencia commented Jun 3, 2024

/test tl=4 l=local d=main

Copy link
Contributor

github-actions bot commented Jun 3, 2024

@mnencia, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/9353473336

@leonardoce
Copy link
Contributor

/ok-to-merge tests are fine

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Jun 4, 2024
@leonardoce leonardoce merged commit 6b88a08 into main Jun 4, 2024
31 of 32 checks passed
@leonardoce leonardoce deleted the dev/no-namespace branch June 4, 2024 07:09
cnpg-bot pushed a commit that referenced this pull request Jun 4, 2024
Remove the namespace deletion check at the start of the reconciliation
cycle and the permissions to read namespace definitions.

Closes #4752

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 6b88a08)
cnpg-bot pushed a commit that referenced this pull request Jun 4, 2024
Remove the namespace deletion check at the start of the reconciliation
cycle and the permissions to read namespace definitions.

Closes #4752

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 6b88a08)
cnpg-bot pushed a commit that referenced this pull request Jun 4, 2024
Remove the namespace deletion check at the start of the reconciliation
cycle and the permissions to read namespace definitions.

Closes #4752

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 6b88a08)
dougkirkley pushed a commit to dougkirkley/cloudnative-pg that referenced this pull request Jun 11, 2024
Remove the namespace deletion check at the start of the reconciliation
cycle and the permissions to read namespace definitions.

Closes cloudnative-pg#4752

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Douglass Kirkley <dkirkley@eitccorp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-requested ◀️ This pull request should be backported to all supported releases ok to merge 👌 This PR can be merged release-1.21 release-1.22 release-1.23
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: Stop requiring namespace object permissions
4 participants