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

fix: re-enroll devices that are removed from ABM and then added back (#23757) #23835

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

jahzielv
Copy link
Contributor

Cherry pick for #23757

…23757)

> Related issue: #23200

# 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] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
See [Changes
files](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/Committing-Changes.md#changes-files)
for more information.
- [x] Input data is properly validated, `SELECT *` is avoided, SQL
injection is prevented (using placeholders for values in statements)
- [x] Added/updated tests
- [x] Manual QA for all new/changed functionality
Copy link

codecov bot commented Nov 14, 2024

Codecov Report

Attention: Patch coverage is 76.31579% with 9 lines in your changes missing coverage. Please review.

Project coverage is 63.16%. Comparing base (f385a8d) to head (d420c29).
Report is 2 commits behind head on rc-minor-fleet-v4.60.0.

Files with missing lines Patch % Lines
server/datastore/mysql/hosts.go 80.64% 4 Missing and 2 partials ⚠️
server/mdm/apple/apple_mdm.go 57.14% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@                    Coverage Diff                     @@
##           rc-minor-fleet-v4.60.0   #23835      +/-   ##
==========================================================
- Coverage                   63.17%   63.16%   -0.01%     
==========================================================
  Files                        1559     1559              
  Lines                      148191   148227      +36     
  Branches                     3722     3722              
==========================================================
+ Hits                        93617    93629      +12     
- Misses                      47180    47197      +17     
- Partials                     7394     7401       +7     
Flag Coverage Δ
backend 64.00% <76.31%> (-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.

@jahzielv jahzielv merged commit 13ca79f into rc-minor-fleet-v4.60.0 Nov 15, 2024
21 checks passed
@jahzielv jahzielv deleted the 23200-ade-enroll-cherry-pick-1 branch November 15, 2024 19:38
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.

2 participants