Skip to content
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

feat: create automatic install policies for fleet-maintained apps #24298

Merged
merged 7 commits into from
Dec 4, 2024

Conversation

jahzielv
Copy link
Contributor

@jahzielv jahzielv commented Dec 2, 2024

Related issue: #22077

Checklist for submitter

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

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    See Changes files for more information.
  • Input data is properly validated, SELECT * is avoided, SQL injection is prevented (using placeholders for values in statements)
  • Added/updated tests
  • Manual QA for all new/changed functionality

ghernandez345 and others added 5 commits November 20, 2024 11:41
relates to #23136

This is the UI for the creating of policies when adding fleet maintained
software. This includes only the creating of the policy and there will
be another PR for viewing more information on the software titles
details page.


**new install type options. this determines if a policy should be
created or not.**

![image](https://github.com/user-attachments/assets/20538c66-bc1c-4903-aa70-83d24da97617)


there are also some new icons for the software titles.

- [] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
- [x] Manual QA for all new/changed functionality
relates to #23137, #23136

implements to the rest of the UI for automatically creating fleet
policies when adding a fleet maintained app. Also includes the API
changes needed for this which include changing the `GET
/software/titles` and `GET /software/titles/:id` endpoints to include
the `automatic_install_policies` data.

UI added includes:

**Adding tag for automatic install software titles**


![image](https://github.com/user-attachments/assets/a7f17350-58f2-44bc-8ea0-477c633b394a)

**Adding modal to show the policies associated with that software
title**


![image](https://github.com/user-attachments/assets/eb08f3e0-0dcd-44d7-915c-b08b7434f615)

- [x] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
- [x] Added/updated tests
- [x] Manual QA for all new/changed functionality

---------

Co-authored-by: Jahziel Villasana-Espinoza <jahziel@fleetdm.com>
# Checklist for submitter

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

<!-- Note that API documentation changes are now addressed by the
product design team. -->

- [x] Added/updated tests
- [x] Manual QA for all new/changed functionality
@jahzielv jahzielv requested review from a team as code owners December 2, 2024 23:13
Copy link

codecov bot commented Dec 2, 2024

Codecov Report

Attention: Patch coverage is 62.69841% with 47 lines in your changes missing coverage. Please review.

Project coverage is 63.43%. Comparing base (b482223) to head (ddf5e1d).
Report is 20 commits behind head on main.

Files with missing lines Patch % Lines
ee/server/service/maintained_apps.go 35.29% 10 Missing and 1 partial ⚠️
server/datastore/mysql/policies.go 77.50% 6 Missing and 3 partials ⚠️
...er/DataTable/SoftwareNameCell/SoftwareNameCell.tsx 12.50% 7 Missing ⚠️
...Titles/SoftwareTable/SoftwareTitlesTableConfig.tsx 0.00% 6 Missing ⚠️
frontend/components/icons/AutomaticSelfService.tsx 20.00% 4 Missing ⚠️
server/datastore/mysql/software_installers.go 50.00% 2 Missing and 1 partial ⚠️
server/datastore/mysql/software_titles.go 62.50% 2 Missing and 1 partial ⚠️
server/service/maintained_apps.go 50.00% 2 Missing ⚠️
frontend/components/icons/User.tsx 50.00% 1 Missing ⚠️
server/datastore/mysql/maintained_apps.go 95.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #24298   +/-   ##
=======================================
  Coverage   63.43%   63.43%           
=======================================
  Files        1584     1586    +2     
  Lines      150498   150587   +89     
  Branches     3771     3831   +60     
=======================================
+ Hits        95465    95531   +66     
- Misses      47415    47435   +20     
- Partials     7618     7621    +3     
Flag Coverage Δ
backend 64.31% <71.00%> (+0.01%) ⬆️
frontend 52.62% <30.76%> (-0.04%) ⬇️

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.

gillespi314
gillespi314 previously approved these changes Dec 3, 2024
@jahzielv jahzielv merged commit f0e3a57 into main Dec 4, 2024
27 checks passed
@jahzielv jahzielv deleted the feat-create-policies-from-fleet-apps branch December 4, 2024 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants