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

Use centralized version qualifier (#16995) #17005

Closed
wants to merge 1 commit into from

Conversation

dliappis
Copy link
Contributor

Forward port PR #16995 to main 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).
@dliappis dliappis added the ci label Jan 31, 2025
@dliappis dliappis self-assigned this Jan 31, 2025
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

cc @dliappis

@dliappis
Copy link
Contributor Author

closing this forward port 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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants