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(installer): Properly support DD_AGENT_MINOR_VERSION when it's not in the minor.patch format #30754

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

BaptisteFoy
Copy link
Contributor

@BaptisteFoy BaptisteFoy commented Nov 5, 2024

What does this PR do?

Supports setting DD_AGENT_MINOR_VERSION=58

Motivation

Describe how to test/QA your changes

Possible Drawbacks / Trade-offs

Additional Notes

@github-actions github-actions bot added short review PR is simple enough to be reviewed quickly team/windows-agent labels Nov 5, 2024
@BaptisteFoy BaptisteFoy added changelog/no-changelog qa/done QA done before merge and regressions are covered by tests team/fleet-automation and removed team/windows-agent labels Nov 5, 2024
@agent-platform-auto-pr
Copy link
Contributor

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=48230721 --os-family=ubuntu

Note: This applies to commit 140d378

Copy link

Regression Detector

@BaptisteFoy BaptisteFoy marked this pull request as ready for review November 5, 2024 11:57
@BaptisteFoy BaptisteFoy requested review from a team as code owners November 5, 2024 11:57
Copy link
Contributor

@julien-lebot julien-lebot left a comment

Choose a reason for hiding this comment

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

Very cool 🔥

@BaptisteFoy
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Nov 5, 2024

🚂 MergeQueue: pull request added to the queue

The median merge time in main is 22m.

Use /merge -c to cancel this operation!

@dd-mergequeue dd-mergequeue bot merged commit 59d8056 into main Nov 5, 2024
257 of 258 checks passed
@dd-mergequeue dd-mergequeue bot deleted the baptiste.foy/FA/fix-minor-version branch November 5, 2024 13:30
@github-actions github-actions bot added this to the 7.61.0 milestone Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog qa/done QA done before merge and regressions are covered by tests short review PR is simple enough to be reviewed quickly team/fleet-automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants