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

Removing notificationConfiguration from s3 bucket config doesn't remove it from the bucket. #1140

Closed
empath-nirvana opened this issue Feb 10, 2022 · 4 comments · Fixed by #1974
Labels
bug Something isn't working

Comments

@empath-nirvana
Copy link

What happened?

I created a bucket with a notifications queue and wanted to remove the queue -- removing the notificationConfiguration field did not remove it from the bucket. Adding notificationConfiguration: {} Caused crossplane to fill in the current bucket configuration.

Is there a way to remove the notification configuration through the crossplane provider?

How can we reproduce it?

What environment did it happen in?

Crossplane version:
1.6.1 of crossplane -- 1.21.1 of the AWS provider

@empath-nirvana empath-nirvana added the bug Something isn't working label Feb 10, 2022
@p3rshin
Copy link

p3rshin commented May 5, 2022

Can confirm that Issue persists with v0.26.0

@github-actions
Copy link

github-actions bot commented Sep 9, 2023

Crossplane does not currently have enough maintainers to address every issue and pull request. This issue has been automatically marked as stale because it has had no activity in the last 90 days. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.

@github-actions github-actions bot added the stale label Sep 9, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 23, 2023
@zonybob
Copy link
Contributor

zonybob commented Oct 24, 2023

/fresh

Observing this in v0.44.2. Cannot update a topic or queue ARN nor can I delete the notification configuration.

@MisterMX MisterMX reopened this Oct 25, 2023
@MisterMX MisterMX removed the stale label Oct 25, 2023
@zonybob
Copy link
Contributor

zonybob commented Oct 25, 2023

I have a branch where I have fixed an associated issue (allowing TopicARN updates). I may also try to resolve this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants