-
Notifications
You must be signed in to change notification settings - Fork 17
feat: support required extensions #78
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
Conversation
85c3690 to
0c388d6
Compare
0c388d6 to
c994428
Compare
Signed-off-by: Matteo <[email protected]>
c994428 to
c75a070
Compare
Signed-off-by: Niccolò Fei <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see we are not adding all the extensions defined as dependencies when we create the Database CR in the automated tests. CNPG doesn't create the extensions with the CASCADE option so all the required extensions need to be explicitly created beforehand.
Co-authored-by: Gabriele Fedi <[email protected]> Signed-off-by: Niccolò Fei <[email protected]>
This will be added as part of #92, given that in #92 we are adding an adhoc field in the |
initial support for extensions dependencies
introduce a new field in the metadata spec to define a list of required extensions
supposing a metadata file like
the command
generate-testing-valuescommand will generate the proper extensions likedagger \ call \ -m ./dagger/maintenance/ \ generate-testing-values --target pgrouting --extension-image ghcr.io/cloudnative-pg/pgrouting:4.0.0-18-bookworm export --path=pgrouting/values.yamlcloses #18