Add CI variables for Google Cloud Artifact Registry integration
What does this MR do and why?
Related to GAR Integration: Add predefined CI variables (#425153 - closed). This change introduces three new CI environment variables which are only present if the Google Cloud Artifact Registry project integration (!141127 (merged)) is enabled.
This change does not include any documentation yet. We're going to introduce documentation for this integration later on, as part of #425158 (closed).
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
How to set up and validate locally
-
First we need to setup the Google Cloud Artifact Registry project integration (that this change depends on) as explained in !141127 (merged):
-
Simulate a SaaS instance in the local GDK. Note that this is an EE feature.
-
Enable the feature flag in the rails console:
Feature.enable(:gcp_artifact_registry)
-
Visit
Project > Settings > Integrations
-
Choose to configure
Google Cloud Platform Artifact Registry
integration -
Fill the form with random strings (no need for proper values for test purposes) and enable the integration.
-
The new integration should appear in the list of active integrations.
-
-
Now we can setup the actual test for this change:
- Make sure your GDK is setup correctly for Runner;
- Create or edit the
.gitlab-ci.yml
file on your test project so that it includes a job like this:test: script: - env | grep GOOGLE_ARTIFACT_REGISTRY
- Merge the change and inspect the job result. Should look something like the screenshot shared above (edit: I've renamed the vars since the screenshot was taken).
Merge request reports
Activity
changed milestone to %16.9
assigned to @jdrpereira
1 Warning featureaddition and featureenhancement merge requests normally have a documentation change. Consider adding a documentation update or confirming the documentation plan with the Technical Writer counterpart.
For more information, see:
- The Handbook page on merge request types.
- The definition of done documentation.
1 Message CHANGELOG missing: If this merge request needs a changelog entry, add the
Changelog
trailer to the commit message you want to add to the changelog.If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
Reviewer roulette
Category Reviewer Maintainer backend @bauerdominic
(UTC+1, 1 hour ahead of author)
@maddievn
(UTC+2, 2 hours ahead of author)
groupimport and integrate (backend) @jnutt
(UTC+0, same timezone as author)
Maintainer review is optional for groupimport and integrate (backend) ~"Verify" Reviewer review is optional for ~"Verify" @drew
(UTC+0, same timezone as author)
Please check reviewer's status!
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by Ghost Userremoved automation:ml label
added workflowin dev label and removed workflowblocked label
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for fffe81c6expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Plan | 4 | 0 | 0 | 0 | 4 | ✅ | | Data Stores | 2 | 0 | 0 | 0 | 2 | ✅ | | Package | 21 | 0 | 2 | 0 | 23 | ✅ | | Create | 8 | 0 | 3 | 0 | 11 | ✅ | | Monitor | 4 | 0 | 0 | 0 | 4 | ✅ | | Govern | 3 | 0 | 0 | 0 | 3 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 42 | 0 | 5 | 0 | 47 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for fffe81c6expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Package | 227 | 0 | 17 | 18 | 244 | ✅ | | Create | 145 | 3 | 24 | 4 | 172 | ❌ | | Govern | 6 | 0 | 0 | 0 | 6 | ✅ | | Data Stores | 4 | 0 | 0 | 0 | 4 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Plan | 8 | 0 | 0 | 0 | 8 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 398 | 3 | 41 | 22 | 442 | ❌ | +-------------+--------+--------+---------+-------+-------+--------+
Edited by Ghost Useradded 352 commits
-
ce45d1ae...5fe8dea3 - 351 commits from branch
master
- 1e7f84ce - Add CI variables for Google Cloud Artifact Registry integration
-
ce45d1ae...5fe8dea3 - 351 commits from branch
requested review from @dmeshcharakou
- Resolved by Furkan Ayhan
@dmeshcharakou, can you review this?