Add an alert to warn about `gitops` keyword deprecation
What does this MR do and why?
This MR adds an alert on the Kubernetes clusters page to warn our users about the deprecation of the gitops
keyword for the agent configuration.
We create a list of all agent configuration files that are present in the current project. Then, we request the raw files one by one and parse the response searching for the gitops
keyword. We stop requesting files once the gitops
keyword is found.
This way we can show the alert only to the users who are affected by the deprecation.
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
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
How to set up and validate locally
- Visit Operate -> Kubernetes clusters page by clicking Connect a cluster button.
- Create a could of agents
- Add configuration files to the agents by creating the file under
.gitlab/agents/<your-agent-name>/config.yaml
- Use
gitops
keyword in the yaml file, for examplegitops: manifest_projects: - id: <your-project-path>
- Visit Operate -> Kubernetes clusters page again and verify the alert.
Related to #440179 (closed)
Merge request reports
Activity
changed milestone to %16.10
assigned to @anna_vovchenko
changed milestone to %16.9
added 1 commit
- 0ce9b9f0 - Add alert about the gitops keyword deprecation
Reviewer roulette
Category Reviewer Maintainer frontend @slashmanov
(UTC+4, 2 hours ahead of author)
@shampton
(UTC-7, 9 hours behind 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 UserBundle size analysis [beta]
This compares changes in bundle size for entry points between the commits 29987dc9 and 9591085b
Special assetsEntrypoint / Name Size before Size after Diff Diff in percent average 4.13 MB 4.13 MB - 0.0 % mainChunk 3.13 MB 3.13 MB - 0.0 %
Please look at the full report for more details
Read more about how this report works.
Generated by
DangerEdited by Ghost UserE2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for 9591085bexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Plan | 53 | 0 | 0 | 0 | 53 | ✅ | | Govern | 65 | 0 | 1 | 0 | 66 | ✅ | | Create | 53 | 0 | 12 | 0 | 65 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Data Stores | 31 | 0 | 1 | 0 | 32 | ✅ | | Verify | 31 | 0 | 0 | 0 | 31 | ✅ | | Package | 14 | 0 | 2 | 0 | 16 | ✅ | | Monitor | 7 | 0 | 0 | 0 | 7 | ✅ | | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 261 | 0 | 17 | 0 | 278 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-review-qa:
test report for 9591085bexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Create | 2 | 6 | 3 | 0 | 11 | ❌ | | Monitor | 0 | 4 | 0 | 0 | 4 | ❌ | | Govern | 0 | 3 | 0 | 0 | 3 | ❌ | | Plan | 0 | 3 | 1 | 0 | 4 | ❌ | | Data Stores | 0 | 2 | 0 | 0 | 2 | ❌ | | Package | 0 | 0 | 1 | 0 | 1 | ➖ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 2 | 18 | 5 | 0 | 25 | ❌ | +-------------+--------+--------+---------+-------+-------+--------+
Edited by Ghost Usermentioned in merge request !143649 (merged)
- Resolved by Anna Vovchenko
@lorenzvanherwaarden, could you please do the initial frontend review?
added UX workflowready for review labels and removed workflowready for development label
requested review from @lorenzvanherwaarden
- Resolved by Anna Vovchenko
requested review from @ipelaez1
- Resolved by Anna Vovchenko
- Resolved by Anna Vovchenko
- Resolved by Andrew Fontaine