CI artifact report keyword for cluster applications
What does this MR do?
First part of #35672 (closed)
Add CI keyword for a cluster applications artifact report
The idea is that we will allow CI/CD managed applications to report back the list of installed applications back to GitLab CI. The format supported is Helm v2 list
with json
output format. Parsing of the artifact and changing Managed-Cluster-Applications.gitlab-ci.yml
to use the keyword will be done in subsequent MRs.
Related issue: #35672 (closed)
This is GL specific and used only by internal GitLab code so no updates to https://docs.gitlab.com/ee/ci/yaml/ is required
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
Documentation (if required) -
Code review guidelines -
Merge request performance guidelines -
Style guides - [-] Database guides
- [-] Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process.
Merge request reports
Activity
changed milestone to %12.10
added 626 commits
-
a11c498b...35b18fe2 - 624 commits from branch
master
- f7a923e9 - New CI keyword for cluster applications artifact
- ade3fa73 - Amend CI template to use new report keyword
-
a11c498b...35b18fe2 - 624 commits from branch
Reviewer roulette
Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category (e.g. frontend or backend), and by a maintainer in all other categories.
To spread load more evenly across eligible reviewers, Danger has randomly picked a candidate for each review slot. Feel free to override these selections if you think someone else would be better-suited, or the chosen person is unavailable.
To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines.
Once you've decided who will review this merge request, mention them as you normally would! Danger does not (yet?) automatically notify them for you.
Category Reviewer Maintainer backend Alex Buijs ( @alexbuijs
)James Fargher ( @proglottis
)Generated by
DangerEdited by 🤖 GitLab Bot 🤖marked the checklist item Changelog entry as completed
marked the checklist item Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. as completed
@tigerwnz Can you please give this an initial review ? I have yet to spin up another MR to parse and update the DB state
Edited by Thong Kuahmentioned in issue #35672 (closed)
- Resolved by Thong Kuah
Just one point to discuss @tkuah!
mentioned in merge request !28331 (merged)
marked the checklist item Documentation (if required) as completed
marked the checklist item Code review guidelines as completed
added 474 commits
-
8c21267c...75317d7c - 473 commits from branch
master
- 04ffeda2 - New CI keyword for cluster applications artifact
-
8c21267c...75317d7c - 473 commits from branch
@tigerwnz I have resolved that thread now. Can you please take another look ?
Thanks @tkuah, LGTM!
mentioned in issue #214028 (closed)
- Resolved by Thong Kuah
This is probably not going to be merged in for %12.10. I am still proving out the concept of updating the cluster with the uploaded artifact (in a new MR). Will upload the new MR for discussion as might be useful to get extra eyes on it
changed milestone to %13.0
added missed:12.10 label
mentioned in issue #32358 (closed)
added 3172 commits
-
04ffeda2...949faa28 - 3171 commits from branch
master
- 0b1eeda2 - New CI keyword for cluster applications artifact
-
04ffeda2...949faa28 - 3171 commits from branch
mentioned in merge request !29870 (merged)
added 1490 commits
-
0b1eeda2...3442487e - 1489 commits from branch
master
- 68f9eebd - New CI keyword for cluster applications artifact
-
0b1eeda2...3442487e - 1489 commits from branch
@proglottis Can you please help review as maintainer ?
assigned to @proglottis and unassigned @tkuah
enabled an automatic merge when the pipeline for 9919ca76 succeeds
mentioned in commit c76d3e2e
added workflowstaging label
added workflowcanary label and removed workflowstaging label
added workflowproduction label and removed workflowcanary label
This merge request has been deployed to the pre.gitlab.com environment, and will be included in the upcoming self-managed GitLab 13.0.0 release.
This comment is generated automatically using the Release Tools project.added published label
added groupenvironments label and removed groupconfigure [DEPRECATED] label