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

Fix message queue sample #693

Merged
merged 1 commit into from
Apr 2, 2024
Merged

Fix message queue sample #693

merged 1 commit into from
Apr 2, 2024

Conversation

ianstanton
Copy link
Member

The message queue sample yaml fails to apply with the following error:

2024-04-02T13:05:54.645148Z ERROR reconciling object:reconcile:reconcile:reconcile_cnpg: controller::cloudnativepg::cnpg:1160: Error patching cluster: ApiError: admission webhook "vcluster.cnpg.io" denied the request: Cluster.postgresql.cnpg.io "sample-message-queue" is invalid: spec.resources.requests.memory: Invalid value: "512Mi": Memory request is lower than PostgreSQL `shared_buffers` value: Invalid (ErrorResponse { status: "Failure", message: "admission webhook \"vcluster.cnpg.io\" denied the request: Cluster.postgresql.cnpg.io \"sample-message-queue\" is invalid: spec.resources.requests.memory: Invalid value: \"512Mi\": Memory request is lower than PostgreSQL `shared_buffers` value", reason: "Invalid", code: 422 }) object.ref=CoreDB.v1alpha1.coredb.io/sample-message-queue.default object.reason=reconciler requested retry trace_id=00000000000000000000000000000000 instance_name=sample-message-queue

This PR resolves the issue.

Signed-off-by: Ian Stanton <ian@tembo.io>
@ChuckHend
Copy link
Collaborator

Just a reminder that these yamls are not equivalent to a Stack spec. We can remove this MQ yaml, and the other yamls associated with Stacks to reduce confusion.

@ianstanton ianstanton merged commit d4c8b8a into main Apr 2, 2024
9 checks passed
@ianstanton ianstanton deleted the fix-mq-sample branch April 2, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants