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

Revert "Skip tests that depend on 8.18.0-SNAPSHOT artifacts being available" #4152

Open
wants to merge 3 commits into
base: 8.x
Choose a base branch
from

Conversation

ycombinator
Copy link
Contributor

Reverts #4133 as it's now possible to create 8.18.0-SNAPSHOT deployments in the ESS production CFT region.

@ycombinator ycombinator requested a review from a team as a code owner November 25, 2024 18:03
@ycombinator ycombinator added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Nov 25, 2024
@ycombinator ycombinator requested review from pierrehilbert and michel-laterman and removed request for kaanyalti and pchila November 25, 2024 18:03
pierrehilbert
pierrehilbert previously approved these changes Nov 26, 2024
@pierrehilbert
Copy link
Contributor

/test

@ycombinator
Copy link
Contributor Author

E2E tests are failing like so:

=== RUN   TestAgentInstallSuite
--
  | agent_install_test.go:94:
  | Error Trace:	/opt/buildkite-agent/builds/bk-agent-prod-gcp-1732865060834845856/elastic/fleet-server/testing/e2e/agent_install_test.go:143
  | /opt/buildkite-agent/builds/bk-agent-prod-gcp-1732865060834845856/elastic/fleet-server/testing/e2e/agent_install_test.go:94
  | /opt/buildkite-agent/go/pkg/mod/github.com/stretchr/[email protected]/suite/suite.go:157
  | /opt/buildkite-agent/builds/bk-agent-prod-gcp-1732865060834845856/elastic/fleet-server/testing/e2e/agent_install_test.go:61
  | Error:      	Should be true
  | Test:       	TestAgentInstallSuite
  | Messages:   	unable to find package download for fileName = elastic-agent-8.18.0-SNAPSHOT-linux-x86_64.tar.gz

I'm not really sure why the package is not available for download, given that we have https://snapshots.elastic.co/8.18.0-4aa4b16c/summary-8.18.0-SNAPSHOT.html. @michel-laterman any thoughts?

@michel-laterman
Copy link
Contributor

What the downloadAgent step does is do GET https://artifacts-api.elastic.co/v1/search/$VERSION-SNAPSHOT
then search the returned json body's packages object for the file.

I can see the package name if I manually do the request.

Maybe we should switch the download to GET https://artifacts-api.elastic.co/v1/downloads/elastic-agent-package/elastic-agent-8.18.0-SNAPSHOT-linux-x86_64.tar.gz directly?

@michel-laterman michel-laterman dismissed stale reviews from pierrehilbert and themself via bb5ad56 December 2, 2024 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants