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(config): Configurable bootstrap of ownership types #9308

Merged

Conversation

skrydal
Copy link
Collaborator

@skrydal skrydal commented Nov 27, 2023

Changes logic of ownership types bootstrap process:

  • Now it runs with every start of the GMS (instead of only during upgrade)
  • It is possible to provide file path to custom ownership types instead of hard-coded ones

In case the new environmental variable (BOOTSTRAP_OWNERSHIP_TYPES_FILE) is not set, the old one is used. In such case the only change of behavior of the app will be the fact that the process will be run with every restart of the GMS instead of only running during the upgrade (similarly to how policies are set).

Change similar to the #8812

@github-actions github-actions bot added the devops PR or Issue related to DataHub backend & deployment label Nov 27, 2023
Copy link
Collaborator

@jjoyce0510 jjoyce0510 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@jjoyce0510 jjoyce0510 merged commit 03be68c into datahub-project:master Nov 28, 2023
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops PR or Issue related to DataHub backend & deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants