For fleetctl gitops, when MDM configs are not explicitly defined in gitops yml file, they are now set to default values.

[getvictor]

For fleetctl gitops, when MDM configs are not explicitly defined in gitops yml file, they are now set to default values.
#17209

Gitops role can now read org config/settings. This is used to determine whether license is Premium.
Doc changes for permission access: #17238

Checklist for submitter

If some of the following don't apply, delete the relevant line.

• Changes file added for user-visible changes in changes/ or orbit/changes/.
  See Changes files for more information.
• Documented any permissions changes (docs/Using Fleet/manage-access.md)
• Added/updated tests
• Manual QA for all new/changed functionality

[getvictor]

@noahtalerman I'm adding the ability for gitops user to read fleet config, so that it can know if Fleet Premium is used. The config access is not documented, so it should not require doc changes.

[noahtalerman]

I'm adding the ability for gitops user to read fleet config, so that it can know if Fleet Premium is used.

@getvictor makes sense.

The config access is not documented, so it should not require doc changes.

By config do you mean info included in the GET /config endpoint?

If so, I think we document this in several rows in the permissions table here:

There might be more rows I'm missing.

[getvictor]

I'm adding the ability for gitops user to read fleet config, so that it can know if Fleet Premium is used.

@getvictor makes sense.

The config access is not documented, so it should not require doc changes.

By config do you mean info included in the GET /config endpoint?

If so, I think we document this in several rows in the permissions table here:

There might be more rows I'm missing.

You're right, I saw looking for config and not settings. I'll update docs in a separate PR.

[codecov]

Codecov Report

Attention: Patch coverage is 92.94118% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 65.51%. Comparing base (977c376) to head (9713d69).
Report is 2 commits behind head on main.

Files	Patch %	Lines
cmd/fleetctl/gitops.go	25.00%	4 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #17223      +/-   ##
==========================================
+ Coverage   65.50%   65.51%   +0.01%     
==========================================
  Files        1192     1192              
  Lines      107474   107551      +77     
  Branches     2569     2569              
==========================================
+ Hits        70399    70463      +64     
- Misses      31702    31710       +8     
- Partials     5373     5378       +5     

Flag	Coverage Δ
backend	66.51% <92.94%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lucasmrod]

Really cool integration tests!