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

Backport PR #16995 to 8.17: Use centralized version qualifier #17002

Closed
wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link
Contributor

Backport PR #16995 to 8.17 branch, original message:


Release notes

[rn:skip]

What does this PR do?

To avoid manual invocations specifying the version qualifier for prereleases in this commit we leverage a centralized version of truth for the version qualifier.

We also honor the DRA_BRANCH BK var for the generated step names (the functionality was already there, it's just a cosmetic improvement).

Why is it important/What is the impact to the user?

Removes the need to specify VERSION_QUALIFIER when building DRA (staging) artifacts entirely.

How to test this PR locally

Can only be tested in BK. Links to be provided:

NOTE This is raised against 9.0 because it's the immediate need, and should be forward ported to main, as well as backported to all 8.x branches.

To avoid manual invocations specifying the version qualifier for
prereleases in this commit we leverage a centralized version of truth
for the version qualifier.

We also honor the `DRA_BRANCH` BK var for the generated step names
(the functionality was already there, it's just a cosmetic improvement).

(cherry picked from commit ac56d99)
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@elasticmachine
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

@dliappis
Copy link
Contributor

closing backport due to #16995 (comment)

@dliappis dliappis closed this Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants