-
Notifications
You must be signed in to change notification settings - Fork 2
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
Parameterize reindex job in GitLab with catalog to reindex #5336
Labels
+
[priority] High
debt
[type] A defect incurring continued engineering cost
demo
[process] To be demonstrated at the end of the sprint
demoed
[process] Successfully demonstrated to team
infra
[subject] Project infrastructure like CI/CD, build and deployment scripts
orange
[process] Done by the Azul team
Comments
Test this by having the operator use the new functionality (index a single catalog via env variable) to reindex a single sandbox catalog. |
achave11-ucsc
added a commit
that referenced
this issue
Feb 28, 2024
achave11-ucsc
added a commit
that referenced
this issue
Feb 28, 2024
achave11-ucsc
added a commit
that referenced
this issue
Feb 29, 2024
achave11-ucsc
added a commit
that referenced
this issue
Feb 29, 2024
achave11-ucsc
added a commit
that referenced
this issue
Feb 29, 2024
achave11-ucsc
added a commit
that referenced
this issue
Feb 29, 2024
achave11-ucsc
added a commit
that referenced
this issue
Mar 1, 2024
achave11-ucsc
added a commit
that referenced
this issue
Mar 1, 2024
For demo, reindex a small catalog in a main deployment. |
dsotirho-ucsc
pushed a commit
that referenced
this issue
Mar 5, 2024
dsotirho-ucsc
added a commit
that referenced
this issue
Mar 5, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
+
[priority] High
debt
[type] A defect incurring continued engineering cost
demo
[process] To be demonstrated at the end of the sprint
demoed
[process] Successfully demonstrated to team
infra
[subject] Project infrastructure like CI/CD, build and deployment scripts
orange
[process] Done by the Azul team
We frequently need to reindex a single catalog only. The only way to do this currently is to reindex locally by invoking the respective script with the
--catalogs
parameter.GitLab recently added the ability to pass variables to jobs when running them manually. I believe these variables are exposed to the job as an environment variable. It would be good to constrain what variables an operator can pass, but I am not sure that's possible.
Instead of threading an explicit argument through the job definition, the Makefile to the script, we should define a new environment variable that overrides the default of the
--catalogs
parameter. Sketch:The text was updated successfully, but these errors were encountered: