-
Notifications
You must be signed in to change notification settings - Fork 482
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add gitops
app config fields, API access, activities, tests
#26282
base: main
Are you sure you want to change the base?
Conversation
7366ae2
to
d8072eb
Compare
81ba1d6
to
5360783
Compare
04e7b46
to
49b33f0
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #26282 +/- ##
==========================================
+ Coverage 63.72% 63.85% +0.13%
==========================================
Files 1655 1656 +1
Lines 158544 159054 +510
Branches 4144 4125 -19
==========================================
+ Hits 101032 101566 +534
+ Misses 49587 49549 -38
- Partials 7925 7939 +14
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
gitops
app config fields, API access, activities, tests
s.Do("PATCH", "/api/latest/fleet/config", json.RawMessage(`{ | ||
"gitops": { "gitops_mode_enabled": true, "repository_url": "a.b.cc" } | ||
}`), http.StatusOK) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should just add a quick s.lastActivityOfTypeMatches
to make sure the activity shows up when you enable gitops ui mode
For #26230
gitops
settings to app configChecklist for submitter