Draft: Move id_tokens to .publish_chart_package
What does this MR do?
Move id_tokens to .publish_chart_package
This might not be the right solution, see discussion in !4259 (comment 2489998900)..publish_chart_package runs the helm_push_oci.sh script
which uses Vault, so any job that includes .publish_chart_package
should have id_tokens set.
The publish_tagged_packages job failed in the pipeline for tag 8.11.2 for the patch release last week (see https://gitlab.com/gitlab-org/release/tasks/-/issues/19208#note_2490246140).
Related issues
Author checklist
For general guidance, please follow our Contributing guide.
Required
For anything in this list which will not be completed, please provide a reason in the MR discussion.
-
Merge Request Title and Description are up to date, accurate, and descriptive. -
MR targeting the appropriate branch. -
MR has a green pipeline. -
Documentation created/updated. -
Tests added/updated, and test plan for scenarios not covered by automated tests. -
Equivalent MR/issue for omnibus-gitlab opened.
Reviewers checklist
-
MR has a green pipeline on https://gitlab.com/gitlab-org/charts/gitlab. -
Consider downstream impact to the Operator, as per evaluating impact from changes to GitLab Chart.
Edited by Reuben Pereira