Skip to content

Feat/master/database management#123

Merged
anestos merged 8 commits into
masterfrom
feat/master/database-management
Jun 1, 2026
Merged

Feat/master/database management#123
anestos merged 8 commits into
masterfrom
feat/master/database-management

Conversation

@anestos
Copy link
Copy Markdown
Collaborator

@anestos anestos commented May 21, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds optional database schema management to the idsvr Helm chart via a pre-install / pre-upgrade hook Job, with accompanying values and documentation to configure JDBC connectivity (inline or via Secret).

Changes:

  • Introduces curity.dbSchemaManagementJob.* values to configure an opt-in schema management Job.
  • Adds a new Helm template that runs idsvr -I as a pre-install,pre-upgrade hook.
  • Documents the new feature and how to provide JDBC connection details securely.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
idsvr/values.yaml Adds configuration block for the new dbSchemaManagementJob feature.
idsvr/templates/database-schema-management.yaml New hook Job template to run idsvr -I with JDBC env vars and Secret support.
idsvr/README.md Documents the new values and usage examples for schema management.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread idsvr/templates/database-schema-management.yaml Outdated
Comment thread idsvr/templates/database-schema-management.yaml
Comment thread idsvr/README.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@anestos anestos requested a review from pjoshi2023 May 21, 2026 13:24
@anestos anestos merged commit d9b39fb into master Jun 1, 2026
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.

5 participants