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 Create Environment agent version for Serverless #2547

Merged
merged 6 commits into from
Oct 10, 2024

Conversation

opauloh
Copy link
Contributor

@opauloh opauloh commented Sep 20, 2024

Summary of your changes

This fixes #2546, by using the version inputted by the user in the Create Environment parameters instead of using the version from the env.ELK_VERSION.

While it is true that the version doesn't matter on Serverless in terms of Kibana and Elasticsearch, it does matter for deploying agents as the snapshots might be no longer available.

@opauloh opauloh requested a review from a team as a code owner September 20, 2024 20:05
Copy link

mergify bot commented Sep 20, 2024

This pull request does not have a backport label. Could you fix it @opauloh? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit
    NOTE: backport-v8.x has been added to help with the transition to the new branch 8.x.

@eyalkraft
Copy link
Contributor

eyalkraft commented Sep 26, 2024

Thanks Paulo! I'll try to deploy an env from this brach to validate

@mergify mergify bot mentioned this pull request Sep 26, 2024
5 tasks
Copy link
Contributor

@eyalkraft eyalkraft left a comment

Choose a reason for hiding this comment

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

Seems to work!

@opauloh
Copy link
Contributor Author

opauloh commented Oct 8, 2024

@gurevichdmitry it seems I can't merge this PR even after it was approved, could you merge it?

@eyalkraft eyalkraft enabled auto-merge (squash) October 10, 2024 02:49
@eyalkraft eyalkraft merged commit beb0270 into elastic:main Oct 10, 2024
11 checks passed
kubasobon pushed a commit to kubasobon/cloudbeat that referenced this pull request Oct 29, 2024
fix: using the input version for serverless

Co-authored-by: Dmitry Gurevich <[email protected]>
Co-authored-by: eyalkraft <[email protected]>
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.

Create Environment failing for serverless
3 participants