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

AmazonMQBroker RabbitMQ encryptionOptions is not supported for RabbitMQ brokers. #1643

Closed
eloo opened this issue Jan 20, 2023 · 2 comments · Fixed by #1900
Closed

AmazonMQBroker RabbitMQ encryptionOptions is not supported for RabbitMQ brokers. #1643

eloo opened this issue Jan 20, 2023 · 2 comments · Fixed by #1900
Labels
bug Something isn't working

Comments

@eloo
Copy link

eloo commented Jan 20, 2023

What happened?

We had applied an update to our rabbitmq broker and then the state was going to synced false and ready false.

The error is encryptionOptions is not supported for RabbitMQ brokers.
Which is also documented here: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-amazonmq-broker-encryptionoptions.html

Looks like the encryption option needs a condition here in the late init if its a activeMQ or rabbitMQ
https://github.com/crossplane-contrib/provider-aws/blob/master/pkg/controller/mq/broker/setup.go#L177

How can we reproduce it?

Create rabbitmq and try to update it

What environment did it happen in?

Crossplane version:
crossplanecontrib/provider-aws:v0.36.0
crossplane/crossplane:v1.10.1

@eloo eloo added the bug Something isn't working label Jan 20, 2023
@github-actions
Copy link

github-actions bot commented Aug 6, 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 Aug 6, 2023
@eloo
Copy link
Author

eloo commented Aug 6, 2023

/fresh

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.

1 participant