Skip to content

New best practice teams#91

Merged
getvictor merged 26 commits intomainfrom
new-teams-and-structure
Feb 3, 2026
Merged

New best practice teams#91
getvictor merged 26 commits intomainfrom
new-teams-and-structure

Conversation

@noahtalerman
Copy link
Member

@noahtalerman noahtalerman commented Jan 29, 2026

@noahtalerman noahtalerman marked this pull request as ready for review January 29, 2026 21:10
@noahtalerman
Copy link
Member Author

Hey @getvictor can you please review/merge this PR when you get the chance?

Copy link
Member

@getvictor getvictor left a comment

Choose a reason for hiding this comment

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

The CI check is failing. It runs against a test Fleet server on Render. Please make sure it is passing with these changes.

Please add the new secrets to the secrets in this repo:

Image

Comment on lines 3 to 5
- path: ../lib/macos.policies.yml
- path: ../lib/windows.policies.yml
- path: ../lib/linux.policies.yml
Copy link
Member

Choose a reason for hiding this comment

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

I don't see where these new files are at.

scripts:
- path: ../lib/macos/scripts/remove-zoom-artifacts.script.sh
- path: ../lib/macos/scripts/set-timezone.script.sh
- path: ../lib/collect-fleetd-logs.sh
Copy link
Member

Choose a reason for hiding this comment

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

I don't see where this new file is at.

@getvictor getvictor merged commit 965ab40 into main Feb 3, 2026
1 check passed
sgress454 added a commit to fleetdm/fleet that referenced this pull request Feb 3, 2026
Due to [recent changes to the fleet-gitops
repo](fleetdm/fleet-gitops#91), some tests need
to be updated to match the expected # of teams.
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.

2 participants