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

[8.x](backport #3142) [Asset Inventory][Azure] Do not add --install-servers flag for versions < 9.0 #3148

Closed
wants to merge 1 commit into from

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Mar 26, 2025

Summary of your changes

Fixes an issue where releases earlier than 9.0 also got an --install-servers flag. Using the same code as regular Azure install script.

Related Issues

Fixes https://github.com/elastic/security-team/issues/11500


This is an automatic backport of pull request #3142 done by [Mergify](https://mergify.com).

…ns < 9.0 (#3142)

do not add --install-servers flag for versions < 9.0

(cherry picked from commit cc1ed67)

# Conflicts:
#	deploy/asset-inventory-arm/install-agent.sh
@mergify mergify bot requested a review from a team as a code owner March 26, 2025 11:36
Copy link
Author

mergify bot commented Mar 26, 2025

Cherry-pick of cc1ed67 has failed:

On branch mergify/bp/8.x/pr-3142
Your branch is up to date with 'origin/8.x'.

You are currently cherry-picking commit cc1ed67b.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   deploy/asset-inventory-arm/install-agent.sh

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Copy link
Author

mergify bot commented Mar 31, 2025

This pull request has not been merged yet. Could you please review and merge it @kubasobon? 🙏

@kubasobon kubasobon closed this Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant