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

Issue-598, Handling of bundled use only resources updation #605

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Conversation

worryg0d
Copy link
Collaborator

@worryg0d worryg0d commented Oct 25, 2023

When we use Instaclustr Console to create a cadence cluster with packaged solutions of Cassandra, Kafka, or OpenSearch it creates clusters that are only for bundled use. It means that a customer can't update bundled resource specifications in any way.

The operator does the same, but it doesn't create resources for bundled use only. So a customer can update those resource specifications.

This PR aims to fix the problem of packaged provisioning for Cadecene.

closes #598

@worryg0d worryg0d added the enhancement New feature or request label Oct 25, 2023
@worryg0d worryg0d self-assigned this Oct 25, 2023
@worryg0d worryg0d changed the title Issue-598, Handling of bundled use only resources Issue-598, Handling of bundled use only resources updation Oct 25, 2023
@@ -65,4 +67,6 @@ var (
ErrExposeServiceNotCreatedYet = errors.New("expose service is not created yet")
ErrExposeServiceEndpointsNotCreatedYet = errors.New("expose service endpoints is not created yet")
ErrOnlySingleConcurrentResizeAvailable = errors.New("only single concurrent resize is allowed")

ErrBundledUseOnlyResourceUpdateIsNotSupported = errors.New("updating of bundled use resource is not supported")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this new line is unnecessary, let's remove it. What are the other teammates opinions?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed it

@ribaraka ribaraka merged commit b5b1bb0 into main Oct 26, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Webhook for Bundled Use Only Cluster
5 participants