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

Merged
merged 3 commits into from
Dec 18, 2024

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/testify@v1.9.0/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
@ycombinator
Copy link
Contributor Author

@michel-laterman Thanks for attempting to fix the test in this PR but it appears to be still failing. Would you mind taking another look when you get a chance?

@michel-laterman
Copy link
Contributor

The failure is now:

{"log.level":"info","@timestamp":"2024-12-02T22:22:40.097Z","log.origin":{"function":"github.com/elastic/elastic-agent/internal/pkg/agent/cmd.waitForFleetServer.func1","file.name":"cmd/enroll_cmd.go","file.line":826},"message":"Fleet Server - Error - failed version compatibility check with elasticsearch (Agent: 8.18.0-SNAPSHOT, Elasticsearch: 8.17.0): unsupported version","ecs.version":"1.6.0"}
--
  | Error: fleet-server failed: timed out waiting for Fleet Server to start after 2m0s

Which is expected as far as failure-cases go, this is because the .env file is still pointing to 8.17.x on the 8.x branch:

ELASTICSEARCH_VERSION=8.17.0-8ac6e7f5-SNAPSHOT

@ycombinator
Copy link
Contributor Author

this is because the .env file is still pointing to 8.17.x on the 8.x branch:

ELASTICSEARCH_VERSION=8.17.0-8ac6e7f5-SNAPSHOT

Thanks @michel-laterman. @v1v Looks like https://github.com/elastic/fleet-server/actions/runs/12182478886 isn't running for the 8.x branch (which is now targeting 8.18.0). That would explain the above situation. Any ideas why this might be?

@v1v
Copy link
Member

v1v commented Dec 6, 2024

@v1v Looks like elastic/fleet-server/actions/runs/12182478886 isn't running for the 8.x branch (which is now targeting 8.18.0). That would explain the above situation. Any ideas why this might be?

https://storage.googleapis.com/artifacts-api/snapshots/branches.json was not reporting 8.x but 8.15, that's now fixed.

https://github.com/elastic/fleet-server/actions/runs/12196799407 ran and #4184 has been created with the change from 8.17 to 8.18

@ycombinator
Copy link
Contributor Author

Thanks @v1v. Once #4184 is merged, I will rebase this PR on 8.x.

@ycombinator ycombinator force-pushed the revert-4133-skip-TestAgentInstallSuite branch from 3d34487 to 6d9aeb4 Compare December 13, 2024 22:47
Copy link

@ycombinator ycombinator enabled auto-merge (squash) December 17, 2024 23:28
@ycombinator ycombinator merged commit 212652c into 8.x Dec 18, 2024
8 checks passed
@ycombinator ycombinator deleted the revert-4133-skip-TestAgentInstallSuite branch December 18, 2024 17:54
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.

4 participants