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

[updatecli] update elastic stack version for testing 9.0.0-e70a10ac-SNAPSHOT #3893

Merged
merged 10 commits into from
Sep 19, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 11, 2024

Bump elastic-stack to latest snapshot version

Update snapshot.yml

1 file(s) updated with "$1=9.0.0-e70a10ac-SNAPSHOT": * dev-tools/integration/.env

GitHub Action workflow link
Updatecli logo

Created automatically by Updatecli

Options:

Most of Updatecli configuration is done via its manifest(s).

  • If you close this pull request, Updatecli will automatically reopen it, the next time it runs.
  • If you close this pull request and delete the base branch, Updatecli will automatically recreate it, erasing all previous commits made.

Feel free to report any issues at github.com/updatecli/updatecli.
If you find this tool useful, do not hesitate to star our GitHub repository as a sign of appreciation, and/or to tell us directly on our chat!

@github-actions github-actions bot added automation dependency backport-skip Skip notification from the automated backport with mergify labels Sep 11, 2024
Copy link
Contributor

mergify bot commented Sep 11, 2024

backport-8.x has been added to help with the transition to the new branch 8.x.

@mergify mergify bot added the backport-8.x Automated backport to the 8.x branch with mergify label Sep 11, 2024
@v1v v1v removed the backport-8.x Automated backport to the 8.x branch with mergify label Sep 12, 2024
Made with ❤️️ by updatecli
@github-actions github-actions bot force-pushed the updatecli_main_bump-elastic-stack-snapshot-main branch from 91bd508 to 7d0a881 Compare September 12, 2024 15:06
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.16.0-2156c99d-SNAPSHOT [updatecli] update elastic stack version for testing 8.16.0-bff94f3c-SNAPSHOT Sep 12, 2024
@michel-laterman
Copy link
Contributor

@jlind23 this pr updates the 8.16.x snapshot on main; it's going to fail until we have 9.x images for main.
Also do we need to do anything to activate similar prs for the 8.x branch?

@ycombinator
Copy link
Contributor

ycombinator commented Sep 12, 2024

do we need to do anything to activate similar prs for the 8.x branch?

I suspect this has something to do with

uses: elastic/oblt-actions/elastic/active-branches@v1

At the moment this step appears to be generating ["7.17", "8.15", "main"] as its output (example).

@v1v might be able to help figure out why 8.x is not being included in the list.

@v1v
Copy link
Member

v1v commented Sep 13, 2024

@v1v might be able to help figure out why 8.x is not being included in the list.

Because the first snapshot in the Unified Release was not published when the GitHub workflow ran.

Those files are generated and use https://artifacts-api.elastic.co/v1/versions as the source of truth. It will not be shown if that entrypoint does not contain those details.

That's not the case anymore since https://storage.googleapis.com/artifacts-api/snapshots/branches.json now contains the new branch.

image

and I triggered this GH workflow and now things are back to normal

image

#3905 is the one targeting 8.x

@ycombinator
Copy link
Contributor

buildkite test this

Made with ❤️️ by updatecli
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.16.0-bff94f3c-SNAPSHOT [updatecli] update elastic stack version for testing 9.0.0-5fdca544-SNAPSHOT Sep 13, 2024
@jlind23
Copy link
Contributor

jlind23 commented Sep 16, 2024

buildkite test this

Made with ❤️️ by updatecli
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 9.0.0-5fdca544-SNAPSHOT [updatecli] update elastic stack version for testing 9.0.0-d56464e5-SNAPSHOT Sep 16, 2024
@michel-laterman
Copy link
Contributor

michel-laterman commented Sep 16, 2024

The current error is Error response from daemon: manifest for docker.elastic.co/elasticsearch/elasticsearch:9.0.0-d56464e5-SNAPSHOT-amd64 not found: manifest unknown: manifest unknown

I can recreate this failure locally; but it works if I remove the amd64 suffix (which is also done in #3904)

@v1v
Copy link
Member

v1v commented Sep 16, 2024

The current error is Error response from daemon: manifest for docker.elastic.co/elasticsearch/elasticsearch:9.0.0-d56464e5-SNAPSHOT-amd64 not found: manifest unknown: manifest unknown

That's something to be reported to the Release team,

$ curl -s https://artifacts-api.elastic.co/v1/versions/8.15.1-SNAPSHOT/builds/latest | jq .build.build_id
"8.15.1-67a5416d"
$ docker pull docker.elastic.co/elasticsearch/elasticsearch:8.15.1-67a5416d-SNAPSHOT-amd64
...
$ curl -s https://artifacts-api.elastic.co/v1/versions/8.16-SNAPSHOT/builds/latest | jq .build.build_id
"8.16.0-874a7f05"

$ docker pull docker.elastic.co/elasticsearch/elasticsearch:8.16.0-874a7f05-SNAPSHOT-amd64 
Error response from daemon: manifest for docker.elastic.co/elasticsearch/elasticsearch:8.16.0-874a7f05-SNAPSHOT-amd64 not found: manifest unknown: manifest unknown

$ curl -s https://artifacts-api.elastic.co/v1/versions/8.15-SNAPSHOT/builds/latest | jq .build.build_id
"8.15.2-7f2cd899"

$ docker pull docker.elastic.co/elasticsearch/elasticsearch:8.15.2-7f2cd899-SNAPSHOT-amd64
Error response from daemon: manifest for docker.elastic.co/elasticsearch/elasticsearch:8.15.2-7f2cd899-SNAPSHOT-amd64 not found: manifest unknown: manifest unknown

for some reason 8.x, main, 8.15 are not producing those docker images anymore.

While 8.15.1 is the latest version with support for -amd64

@jlind23
Copy link
Contributor

jlind23 commented Sep 16, 2024

@v1v the double dots in your command looks weird to me:
8..16

@jlind23
Copy link
Contributor

jlind23 commented Sep 16, 2024

Maybe @rdner would know why we are not producing those 🤷🏼

@v1v
Copy link
Member

v1v commented Sep 16, 2024

@v1v the double dots in your command looks weird to me:

A typo in my end, i reviewed those commands and updated the comment with the output.

Maybe @rdner would know why we are not producing those 🤷🏼

Those are docker images related to elasticsearch

BTW, I asked in the internal slack channel about this behaviour and you are in the loop, in case it helps

@jlind23
Copy link
Contributor

jlind23 commented Sep 16, 2024

Those are docker images related to elasticsearch

🤦🏼 sorry missed it.

@michel-laterman
Copy link
Contributor

buildkite test this

@michel-laterman michel-laterman force-pushed the updatecli_main_bump-elastic-stack-snapshot-main branch from 7173f19 to 8647683 Compare September 17, 2024 18:11
@michel-laterman michel-laterman force-pushed the updatecli_main_bump-elastic-stack-snapshot-main branch from 8647683 to ac8f20d Compare September 17, 2024 18:29
@michel-laterman
Copy link
Contributor

cloud e2e tests are failing:


╷
--
  | │ Error: failed creating deployment
  | │
  | │   with ec_deployment.deployment,
  | │   on main.tf line 46, in resource "ec_deployment" "deployment":
  | │   46: resource "ec_deployment" "deployment" {
  | │
  | │ api error: 1 error occurred:
  | │ 	* deployments.identity.invalid_acs: Failed to calculate ACS for [kibana] resource [8ee3719c35104aac97a823ec1ebc0bd1]
  | │
  | │
  | ╵
  | ╷
  | │ Error: failed creating deployment
  | │
  | │   with ec_deployment.deployment,
  | │   on main.tf line 46, in resource "ec_deployment" "deployment":
  | │   46: resource "ec_deployment" "deployment" {
  | │
  | │ set "request_id" to "oe500xbo896pai5r7b7mfk2q385zx0bwnqk0sqh7jl7w6ylyay4uydmfp900cge9" to recreate the deployment resources
  | ╵

@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 9.0.0-d56464e5-SNAPSHOT [updatecli] update elastic stack version for testing 9.0.0-71a571da-SNAPSHOT Sep 18, 2024
@michel-laterman
Copy link
Contributor

buildkite test this

2 similar comments
@michel-laterman
Copy link
Contributor

buildkite test this

@michel-laterman
Copy link
Contributor

buildkite test this

Made with ❤️️ by updatecli
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 9.0.0-71a571da-SNAPSHOT [updatecli] update elastic stack version for testing 9.0.0-e70a10ac-SNAPSHOT Sep 19, 2024
@@ -150,6 +150,7 @@ steps:

- label: ":gcloud: Cloud e2e Test"
key: "cloud-e2e-test"
skip: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Tracked with #3920

@michel-laterman
Copy link
Contributor

buildkite test this

Copy link

@michel-laterman michel-laterman merged commit efc1b7f into main Sep 19, 2024
8 checks passed
@michel-laterman michel-laterman deleted the updatecli_main_bump-elastic-stack-snapshot-main branch September 19, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation backport-skip Skip notification from the automated backport with mergify dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants