Skip to content
Snippets Groups Projects

CI artifact report keyword for cluster applications

Merged Thong Kuah requested to merge cluster-applications-artifact into master
All threads resolved!

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

Availability and Testing

Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Just one point to discuss @tkuah!

  • Tiger Watson assigned to @tkuah and unassigned @tigerwnz

    assigned to @tkuah and unassigned @tigerwnz

  • Lucas Charles mentioned in merge request !28331 (merged)

    mentioned in merge request !28331 (merged)

  • Thong Kuah marked the checklist item Documentation (if required) as completed

    marked the checklist item Documentation (if required) as completed

  • Thong Kuah marked the checklist item Code review guidelines as completed

    marked the checklist item Code review guidelines as completed

  • Thong Kuah resolved all threads

    resolved all threads

  • Thong Kuah added 474 commits

    added 474 commits

    Compare with previous version

  • Author Maintainer

    @tigerwnz I have resolved that thread now. Can you please take another look ?

  • Thong Kuah assigned to @tigerwnz and unassigned @tkuah

    assigned to @tigerwnz and unassigned @tkuah

  • Thong Kuah changed the description

    changed the description

  • Thong Kuah changed title from Cluster applications CI artifact report to CI artifact report keyword for cluster applications

    changed title from Cluster applications CI artifact report to CI artifact report keyword for cluster applications

  • Tiger Watson approved this merge request

    approved this merge request

  • Thanks @tkuah, LGTM!

  • Tiger Watson assigned to @tkuah and unassigned @tigerwnz

    assigned to @tkuah and unassigned @tigerwnz

  • mentioned in issue #214028 (closed)

    • Author Maintainer
      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

  • Thong Kuah marked as a Work In Progress

    marked as a Work In Progress

  • 🤖 GitLab Bot 🤖 changed milestone to %13.0

    changed milestone to %13.0

  • Thong Kuah added 3172 commits

    added 3172 commits

    Compare with previous version

  • Thong Kuah mentioned in merge request !29870 (merged)

    mentioned in merge request !29870 (merged)

  • Thong Kuah unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Thong Kuah resolved all threads

    resolved all threads

  • Thong Kuah added 1490 commits

    added 1490 commits

    Compare with previous version

  • Thong Kuah changed the description

    changed the description

  • Author Maintainer

    @proglottis Can you please help review as maintainer ?

  • Thong Kuah assigned to @proglottis and unassigned @tkuah

    assigned to @proglottis and unassigned @tkuah

  • James Fargher approved this merge request

    approved this merge request

  • James Fargher enabled an automatic merge when the pipeline for 9919ca76 succeeds

    enabled an automatic merge when the pipeline for 9919ca76 succeeds

  • merged

  • James Fargher mentioned in commit c76d3e2e

    mentioned in commit c76d3e2e

  • 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.


    :robot: This comment is generated automatically using the Release Tools project.

  • 🤖 GitLab Bot 🤖 changed the description

    changed the description

  • Please register or sign in to reply
    Loading