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

Use fleet-server binary version for cloude2e images #3314

Merged

Conversation

michel-laterman
Copy link
Contributor

What is the problem this PR solves?

Bumping fleet-server versions causes the cloude2e test to fail

How does this PR solve the problem?

Use the fleet-server binary version with snapshot flag when building the cloude2e test image instead of the stack version so fleet-server builds will not fail on version bumps.

Use the fleet-server binary version with snapshot flag when building the
cloude2e test image instead of the stack version so fleet-server builds
will not fail on version bumps.
@michel-laterman michel-laterman added the Team:Fleet Label for the Fleet team label Feb 23, 2024
@michel-laterman michel-laterman requested a review from a team as a code owner February 23, 2024 18:29
@michel-laterman michel-laterman added backport-v8.12.0 Automated backport with mergify backport-v8.13.0 Automated backport with mergify labels Feb 23, 2024
@@ -286,7 +286,7 @@ release-manager-release: ## - Builds a snapshot release. The Go version defined
## get-version : Get the Fleet server version
.PHONY: get-version
get-version:
@echo $(DEFAULT_VERSION)
@echo $(VERSION)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

$VERSION differs from $DEFAULT_VERSION as it may include snapshot or other version indicators.
The only other target (besides cloude2e) that uses get-version is the buildkite dra target, which should filter out the snapshot indicator

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@michel-laterman michel-laterman enabled auto-merge (squash) February 23, 2024 18:46
Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@michel-laterman michel-laterman merged commit 8f7dcc9 into elastic:main Feb 23, 2024
8 checks passed
@michel-laterman michel-laterman deleted the fix-cloude2e-version-selection branch February 23, 2024 21:20
mergify bot pushed a commit that referenced this pull request Feb 23, 2024
Use the fleet-server binary version with snapshot flag when building the
cloude2e test image instead of the stack version so fleet-server builds
will not fail on version bumps.

(cherry picked from commit 8f7dcc9)
mergify bot pushed a commit that referenced this pull request Feb 23, 2024
Use the fleet-server binary version with snapshot flag when building the
cloude2e test image instead of the stack version so fleet-server builds
will not fail on version bumps.

(cherry picked from commit 8f7dcc9)
michel-laterman added a commit that referenced this pull request Feb 23, 2024
Use the fleet-server binary version with snapshot flag when building the
cloude2e test image instead of the stack version so fleet-server builds
will not fail on version bumps.

(cherry picked from commit 8f7dcc9)

Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com>
michel-laterman added a commit that referenced this pull request Feb 23, 2024
Use the fleet-server binary version with snapshot flag when building the
cloude2e test image instead of the stack version so fleet-server builds
will not fail on version bumps.

(cherry picked from commit 8f7dcc9)

Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com>
michel-laterman added a commit to michel-laterman/fleet-server that referenced this pull request Mar 4, 2024
Use the fleet-server binary version with snapshot flag when building the
cloude2e test image instead of the stack version so fleet-server builds
will not fail on version bumps.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v8.12.0 Automated backport with mergify backport-v8.13.0 Automated backport with mergify Team:Fleet Label for the Fleet team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants