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

Introduce build qualifier parameter #35155

Merged
merged 6 commits into from
Nov 2, 2018

Conversation

alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Nov 1, 2018

Continues work that begin in #34050

With this change we drop the snapshot and qualifier from the Version class of the build so this is represented as the version in server and we change the version properties to a string that can have qualifier and snapshot to be used throughout the build, this is also the version that is included in the build-tool properties.

The final change will be to remove qualifier from server's Version class.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@alpar-t alpar-t added the WIP label Nov 1, 2018
- VersionProperties.elasticsearch is now a string which can have qualifier
and snapshot too
- The Version class in the build no longer cares about snapshot and
qualifier.
@alpar-t alpar-t merged commit f227008 into elastic:master Nov 2, 2018
@alpar-t alpar-t deleted the version-qualifier-param branch November 2, 2018 03:27
@alpar-t
Copy link
Contributor Author

alpar-t commented Nov 2, 2018

Thanks for the review @nik9000

alpar-t added a commit that referenced this pull request Nov 2, 2018
* Introduce property to set version qualifier

- VersionProperties.elasticsearch is now a string which can have qualifier
and snapshot too
- The Version class in the build no longer cares about snapshot and
qualifier.
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v6.6.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants