Bug #8371
closed[Deployment] Upgrading Rails packages on CentOS uninstalls them
Description
When yum upgrades a package, it runs the postinst script for the newly installed version, then runs the cleanup script for the previously installed version. Our cleanup script always removes the package, so upgrading a package effectively uninstalls it. The cleanup script needs to recognize the "upgrade from previous version" call, and not clean up anything in that case.
Updated by Brett Smith over 8 years ago
- Status changed from New to In Progress
http://ftp.rpm.org/max-rpm/s1-rpm-inside-scripts.html#S2-RPM-INSIDE-ERASE-TIME-SCRIPTS is the cite that explains why this is the correct change.
Updated by Nico César over 8 years ago
review @ acab2f7c338f245baa7e5d9dca57476d12d863bf
I don't see a clear reason for the previous " [ "$1" = "1" ] || [ "$1" = "2" ] " . but whatever, LGTM now.
Updated by Brett Smith over 8 years ago
Nico Cesar wrote:
review @ acab2f7c338f245baa7e5d9dca57476d12d863bf
I don't see a clear reason for the previous " [ "$1" = "1" ] || [ "$1" = "2" ] " . but whatever, LGTM now.
I think it was just copied over as boilerplate from the postinst (where it makes more sense). Thanks.
Updated by Brett Smith over 8 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados-dev|commit:623af8e46e921f59abeeee9e7ae8ec4aef890b5b.
Updated by Brett Smith over 8 years ago
- Status changed from Resolved to New
- Target version changed from 2016-03-02 sprint to Arvados Future Sprints
Still an issue as of the 2016-03-30 packages.
Updated by Brett Smith over 8 years ago
Brett Smith wrote:
Still an issue as of the 2016-03-30 packages.
Actually, this might be mistaken. It might be that the current packages are fine, but the postrm from previously-installed (and buggy) packages runs as you upgrade. If that's the case, there's really no way around this other than to go through, or somehow inhibit the buggy postrm from running.
Updated by Tom Morris almost 8 years ago
- Assigned To changed from Brett Smith to Tom Morris
Updated by Tom Morris over 7 years ago
- Assigned To changed from Tom Morris to Javier Bértoli
Javier - can you please review the current state of this and either close or update? Thanks!
Updated by Javier Bértoli over 7 years ago
- Target version changed from Arvados Future Sprints to 2017-04-12 sprint
Updated by Javier Bértoli over 7 years ago
- Status changed from New to Closed
Tried on arvagrant, with CentOS 7, deploying first version
20161229114527-997479d1408139e96ecdb42a60b4f727f814f6c9
and then upgrading to version
20170123133526-6441ff0411fc0316c0825e8647150b1e079256ab.
All tests passes, nothing seems to be broken and/or missing, so I guess this is solved now.