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

upgrade(fleet): Support fleet policies for the injector #29833

Merged
merged 16 commits into from
Nov 6, 2024

Conversation

BaptisteFoy
Copy link
Contributor

@BaptisteFoy BaptisteFoy commented Oct 7, 2024

What does this PR do?

Supports the APM Injector for Fleet policies

Motivation

Describe how to test/QA your changes

This was tested end to end on a VM with:

  • This PR
  • Latest agent
  • Specific build of the injector
  • Staging backend of Fleet Automation

Possible Drawbacks / Trade-offs

Additional Notes

@BaptisteFoy BaptisteFoy added changelog/no-changelog qa/done QA done before merge and regressions are covered by tests team/fleet-automation labels Oct 7, 2024
@BaptisteFoy BaptisteFoy force-pushed the baptiste.foy/FA/injector-config branch from 1885cff to a75b662 Compare October 7, 2024 09:04
@pr-commenter
Copy link

pr-commenter bot commented Oct 7, 2024

Test changes on VM

Use this command from test-infra-definitions to manually test this PR changes on a VM:

inv create-vm --pipeline-id=48256524 --os-family=ubuntu

Note: This applies to commit 14c13a3

Base automatically changed from baptiste.foy/FA/package-agnostic-cdn to main October 7, 2024 10:46
Copy link

cit-pr-commenter bot commented Oct 7, 2024

Regression Detector

@BaptisteFoy BaptisteFoy force-pushed the baptiste.foy/FA/injector-config branch from 2c7de0d to 705340e Compare October 10, 2024 13:32
@BaptisteFoy BaptisteFoy force-pushed the baptiste.foy/FA/injector-config branch from 15d1fad to 1f53081 Compare November 5, 2024 12:09
@github-actions github-actions bot added the medium review PR review might take time label Nov 5, 2024
@BaptisteFoy BaptisteFoy marked this pull request as ready for review November 5, 2024 16:23
@BaptisteFoy BaptisteFoy requested review from a team as code owners November 5, 2024 16:23
@BaptisteFoy
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Nov 6, 2024

🚂 MergeQueue: pull request added to the queue

The median merge time in main is 23m.

Use /merge -c to cancel this operation!

@dd-mergequeue dd-mergequeue bot merged commit 2bb38f7 into main Nov 6, 2024
224 checks passed
@dd-mergequeue dd-mergequeue bot deleted the baptiste.foy/FA/injector-config branch November 6, 2024 16:38
@github-actions github-actions bot added this to the 7.61.0 milestone Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog medium review PR review might take time qa/done QA done before merge and regressions are covered by tests team/fleet-automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants