Skip to content

Conversation

@ericywl
Copy link
Contributor

@ericywl ericywl commented Jan 16, 2026

The EC API that we use to get available stack versions was recently changed to remove previously release SNAPSHOTs. This causes some OS smoke tests to fail since we use SNAPSHOTs for all of the tests and some of them are missing. This PR fixes the issue by falling back to released versions instead of SNAPSHOTs if they are not available.

@ericywl ericywl self-assigned this Jan 16, 2026
@ericywl ericywl requested a review from a team as a code owner January 16, 2026 08:44
@github-actions
Copy link
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify
Copy link
Contributor

mergify bot commented Jan 16, 2026

This pull request does not have a backport label. Could you fix it @ericywl? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-7.17 is the label to automatically backport to the 7.17 branch.
  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit.
  • backport-9./d is the label to automatically backport to the 9./d branch. /d is the digit.
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

cc @ericywl

@ericywl ericywl added the backport-skip Skip notification from the automated backport with mergify label Jan 16, 2026
# terraform_init

local REGION=$(echo var.region | terraform console | tr -d '"')
# local REGION=$(echo var.region | terraform console | tr -d '"')
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we plan to use this terraform code again? If not, shouldn't we remove it?

Suggested change
# local REGION=$(echo var.region | terraform console | tr -d '"')

The terraform code we are initializing here could be removed (at least partially)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip Skip notification from the automated backport with mergify

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants