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

feat(cloud-native): add cnSqlDbSchema property to helm chart #97

Open
wants to merge 4 commits into
base: 4.5
Choose a base branch
from

Conversation

misba7
Copy link
Contributor

@misba7 misba7 commented Jan 27, 2025

Closes #94

@misba7 misba7 requested review from moabu and iromli as code owners January 27, 2025 20:35
@misba7 misba7 self-assigned this Jan 27, 2025
@misba7 misba7 removed the request for review from moabu January 27, 2025 20:36
Copy link
Contributor

@iromli iromli left a comment

Choose a reason for hiding this comment

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

Missing changes:

  1. attribute in config sub-chart
  2. docs generated by helm-docs

Signed-off-by: Amro Misbah <[email protected]>
@iromli
Copy link
Contributor

iromli commented Jan 30, 2025

@misba7 let me know once we have PR for #96 so I can merge and close sub and parent issues.

@misba7 misba7 requested a review from ossdhaval as a code owner January 31, 2025 08:02
@@ -429,6 +429,23 @@ This documentation demonstrates how to upgrade a Kubernetes setup of Gluu >=4.2
cnSqldbUserPassword: <postgres-user-password> # adjust according to your setup
```
1. Postgresql database schema defaults to empty string.
To override the default schema in a Gluu version `>=4.5.6`, add the following to `values.yaml`:
Copy link
Contributor

Choose a reason for hiding this comment

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

The version should be less than or equal to 4.5.6

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.

feat(cloud-native): add attribute for SQL database schema in Helm chart
2 participants