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

Obsolete pre 7.0 noarch package in rpm #39472

Merged
merged 2 commits into from
Feb 28, 2019

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Feb 27, 2019

This commit makes the rpm metadata indicate the pre 7.0 noarch packages
are obsoleted by this package. This fixes an issue where upgrading with
yum would cause an error thinking there was nothing to upgrade.

closes #39414

This commit makes the rpm metadata indicate the pre 7.0 noarch packages
are obsoleted by this package. This fixes an issue where upgrading with
yum would cause an error thinking there was nothing to upgrade.

closes elastic#39414
@rjernst rjernst added >bug :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts v7.0.0 v8.0.0 v7.2.0 labels Feb 27, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@rjernst rjernst merged commit 63413a8 into elastic:master Feb 28, 2019
@rjernst rjernst deleted the rpm_obsolete_noarch branch February 28, 2019 04:44
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Feb 28, 2019
This commit makes the rpm metadata indicate the pre 7.0 noarch packages
are obsoleted by this package. This fixes an issue where upgrading with
yum would cause an error thinking there was nothing to upgrade.

closes elastic#39414
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Feb 28, 2019
This commit makes the rpm metadata indicate the pre 7.0 noarch packages
are obsoleted by this package. This fixes an issue where upgrading with
yum would cause an error thinking there was nothing to upgrade.

closes elastic#39414
@@ -330,6 +331,8 @@ Closure commonRpmConfig(boolean oss) {
packageGroup 'Application/Internet'
requires '/bin/bash'

obsoletes packageName, '7.0.0', Flags.LESS
Copy link
Contributor

@alpar-t alpar-t Feb 28, 2019

Choose a reason for hiding this comment

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

@rjernst Could it be that we are missing or need to improve some upgrade test ?
It seems that lack of this should have caused an upgrade to fail.
Might be because there was no conflicts so both got installed ?

Copy link
Member Author

Choose a reason for hiding this comment

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

See #39414. The problem is with yum. We could and should probably also have packaging tests that use yum, but I don't think designing and building such tests should block 7.0. I created #39494 so we do not forget; thanks for making this point.

rjernst added a commit that referenced this pull request Feb 28, 2019
This commit makes the rpm metadata indicate the pre 7.0 noarch packages
are obsoleted by this package. This fixes an issue where upgrading with
yum would cause an error thinking there was nothing to upgrade.

closes #39414
rjernst added a commit that referenced this pull request Feb 28, 2019
This commit makes the rpm metadata indicate the pre 7.0 noarch packages
are obsoleted by this package. This fixes an issue where upgrading with
yum would cause an error thinking there was nothing to upgrade.

closes #39414
@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
>bug :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts Team:Delivery Meta label for Delivery team v7.0.0-rc1 v7.2.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

can't install elasticsearch-oss-7.0.0-beta1-x86_64.rpm over elasticsearch-oss-6.7.0.rpm
7 participants