-
Notifications
You must be signed in to change notification settings - Fork 197
[main][Automation] Update versions #9172
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
[main][Automation] Update versions #9172
Conversation
These files are used for picking the starting (pre-upgrade) or ending (post-upgrade) agent versions in upgrade integration tests. The content is based on responses from https://www.elastic.co/api/product_versions and https://snapshots.elastic.co The current update is generated based on the following requirements: Package version: 9.2.0 ```json { "UpgradeToVersion": "9.2.0", "CurrentMajors": 1, "PreviousMajors": 1, "PreviousMinors": 2, "SnapshotBranches": [ "9.1", "9.0", "8.19", "7.17" ] } ```
Pinging @elastic/elastic-agent (Team:Elastic-Agent) |
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
1 similar comment
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
Pinging @elastic/elastic-agent (Team:Elastic-Agent) |
The version check for the rollback reason is pointing at >= 9.1.0-SNAPSHOT but the feature introduced with #8407 didn't make it in 9.1.0. Will change the version and retest. |
|
💔 Build Failed
Failed CI StepsHistory
|
@ycombinator 👋 do we have a way to debug why this fails? 🙂 |
it seems we have the same problem in 9.1 branch #9207 |
cc @michel-laterman @ycombinator for the FIPS upgrade test failure. |
Sorry I somehow missed the notifications on this PR. Looking into the https://buildkite.com/elastic/elastic-agent/builds/24588#01985b6a-6302-4b0b-86cc-86d846e00109/185-1131 and https://buildkite.com/elastic/elastic-agent/builds/24637#019879e4-b75a-4152-88e5-e759815de164 ASAP. |
CI is failing on this PR like so:
In both cases, the error seems to be that the test is timing out waiting for the |
I was able to manually create a |
Okay, progress, sort of. CI is now failing on a different error:
So now the issue seems to be the same as #9207 (comment). I've proposed a solution in that comment; let's continue the conversation on that PR so it's all in one place and then apply whatever we decided to do to both PRs. |
I did leave a reply here @ycombinator, and I understand this will require some tweaking on our side. In the meantime, we have PR #9048 that we’d ideally like to merge today to trigger the update‑versions automation. How about the following approach:
This way we can keep things unblocked while still moving towards the permanent fix. |
When you say, "invoke the automation", I assume you mean the automation that will create a new version of this PR here (#9172), correct?
++ to this approach, pending the one question above. |
yy I am referring to that. So I think we are aligned and we do agree |
closing this PR and it will be "re-created" shortly |
These files are used for picking the starting (pre-upgrade) or ending (post-upgrade) agent versions in upgrade integration tests.
The content is based on responses from https://www.elastic.co/api/product_versions and https://snapshots.elastic.co
The current update is generated based on the following requirements:
Package version: 9.2.0