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

combine osquery + mdm information for connected_to_fleet #20230

Merged
merged 4 commits into from
Jul 9, 2024

Conversation

roperzh
Copy link
Member

@roperzh roperzh commented Jul 5, 2024

for #20057

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.
  • Added/updated tests
  • Manual QA for all new/changed functionality

for #20057

this still leaves edge cases like a host that is reset to factory
settings and doesn't install `fleetd` again, but this problem was
already there even before we introduced `connected_to_fleet`
@roperzh roperzh marked this pull request as ready for review July 8, 2024 13:46
@roperzh roperzh requested a review from a team as a code owner July 8, 2024 13:46
@@ -0,0 +1 @@
* Improved the accuracy of the heuristic used to deterimine if a host is connected to Fleet via MDM by using osquery data for hosts that didn't send a Checkout message.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Improved the accuracy of the heuristic used to deterimine if a host is connected to Fleet via MDM by using osquery data for hosts that didn't send a Checkout message.
* Improved the accuracy of the heuristic used to determine if a host is connected to Fleet via MDM by using osquery data for hosts that didn't send a Checkout message.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you! these go through chatgpt (😨) before they land the release notes so I'll leave it just so I don't lose your ✅ and have to bother you again + fight CI

@roperzh roperzh merged commit cc13d3c into main Jul 9, 2024
19 checks passed
@roperzh roperzh deleted the 20057-connected-tweaks branch July 9, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants