Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

Failed to deploy docker postgresql resource using the built-in resource definition #1970

Closed
Finenyaco opened this issue Jan 19, 2024 · 5 comments
Assignees
Labels
bug-qa Something isn't working, and have not yet hit a real release
Milestone

Comments

@Finenyaco
Copy link

Describe the bug

Failed to deploy docker postgresql using the built-in resource definition postgresql.

Steps to reproduce

  1. Setup Walrus with docker mode.
  2. Create a posgresql resource using the built-in resource definition postgresql.
  3. Check the deployment result.

Result

  1. Failed to deploy.
image
  1. The engine version is incorrect.
image

Expected behavior

Deploy successfully.

Environment

  • Walrus version: main 98e5445
  • UI version: dev-3fcbabe

Additional context
Add any other context about the problem here.

@Finenyaco Finenyaco added the bug-qa Something isn't working, and have not yet hit a real release label Jan 19, 2024
@gitlawr gitlawr added this to the v0.5.0 milestone Jan 19, 2024
@Finenyaco
Copy link
Author

Environment

  • Walrus version: main 98e5445
  • UI version: dev-3fcbabe

Test Result:

Similar issues exist in other fields. For example, in the container definition, the 'Accept Changed' attribute in Kubernetes was originally modifiable, but now it is not.

image

@shanewxy
Copy link
Contributor

Environment

  • Walrus version: main 98e5445
  • UI version: dev-3fcbabe

Test Result:

Similar issues exist in other fields. For example, in the container definition, the 'Accept Changed' attribute in Kubernetes was originally modifiable, but now it is not.

image

Fixed engine version.

But as for 'Accept Changed', it is a readonly attribute in docker-containerservice, so that it will result in readonly in merged resource definition.

@Finenyaco
Copy link
Author

Environment

  • Walrus version: main 66a2214
  • UI version: dev-7449733

Test Result:

  • postgresql: pass
image
  • containservice: But 'Accept Changed' can be set to true now. It is supposed to be immutable according to the design.
image

@thxCode
Copy link
Member

thxCode commented Jan 20, 2024

we are going to process merging read-only attribute in resource definition, which means the built-in containerservice is not allowed Accept-Changed. #1979

@Finenyaco
Copy link
Author

Environment

  • Walrus version: main 3806cbe
  • UI version: v0.5.0-rc2

Test Result: Verified containerservice resource definition: pass

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug-qa Something isn't working, and have not yet hit a real release
Projects
None yet
Development

No branches or pull requests

4 participants