Bug 16192 - Install of docker-registry-0.9.1-1.mga5 post-install scipt fails
Summary: Install of docker-registry-0.9.1-1.mga5 post-install scipt fails
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: x86_64 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: advisory MGA5-64-OK
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2015-06-24 01:36 CEST by Scott Karns
Modified: 2015-09-18 17:58 CEST (History)
3 users (show)

See Also:
Source RPM: docker-registry
CVE:
Status comment:


Attachments

Description Scott Karns 2015-06-24 01:36:24 CEST
When docker-registry-.9.1-1.mga5 was installed to Mageia 5 system, the following script error was encountered:

/var/tmp/rpm-tmp.RSsvVk: line 1: fg: no job control
warning: %post(docker-registry-0.9.1-1.mga5.noarch) scriptlet failed, exit status 1
ERROR: 'script' failed for docker-registry-0.9.1-1.mga5
Comment 1 David Walser 2015-06-24 20:04:54 CEST
That's because it's using the wrong scriplets, it's using Fedora ones.

It should use the Mageia ones, as documented here:
https://wiki.mageia.org/en/System_Service_policy

Assignee: bugsquad => bruno

Comment 2 Bruno Cornec 2015-06-25 01:02:19 CEST
Cauldron updated

Status: NEW => ASSIGNED

Comment 3 Bruno Cornec 2015-06-25 01:09:49 CEST
Update made to 5 and advisory proposed

Assignee: bruno => qa-bugs

Comment 4 Samuel Verschelde 2015-06-25 10:09:53 CEST
(In reply to Bruno Cornec from comment #3)
> Update made to 5 and advisory proposed

I don't see the advisory nor the list or RPMs :)
Comment 5 Rémi Verschelde 2015-06-25 14:38:36 CEST
Actually the advisory was uploaded directly to SVN. Please post it also in the bug report next time Bruno, so that QA testers know what they are supposed to test.

Here it is:

Advisory
========
Updated docker-registry package fixes bad macros usage

Fix macro usage by moving fedora ones to mageia ones

SRPM:
=====
docker-registry-0.9.1-1.1.mga5

RPMS:
=====
docker-registry-0.9.1-1.1.mga5.noarch
Rémi Verschelde 2015-06-25 14:38:45 CEST

Whiteboard: (none) => advisory

Comment 6 Samuel Verschelde 2015-06-25 14:43:52 CEST
(In reply to Rémi Verschelde from comment #5)
> Updated docker-registry package fixes bad macros usage

This is good for a commit message, but for an advisory it should tell what the issue was in the first place (a post-installation script that failed and what consequences).
David Walser 2015-06-30 18:51:21 CEST

Source RPM: (none) => docker-registry

Comment 7 Samuel Verschelde 2015-07-27 14:22:19 CEST
Mageia 5 64.

Summary: seems fine, but there are warnings.

I can confirm that the update package no longer gives errors during installation and removal. And I looked at the spec file and the only change concerns those scripts.

I haven't tested the program in itself, but unless something was broken during the build, there's no reason a regression would have been introduced by the changes.

Note that users will still see a failure during the update because the uninstall scriplet of the old package fails.

docker-registry service starts fine. Logs contain warnings though, altough I don't think they need blocking the update. If there's something to fix, though, then it would be great to fix it in the same update to avoid to have twice the amount of download for users.

Logs:

juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 WARNING: LRU cache disabled!
juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 WARNING: LRU cache disabled!
juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 DEBUG: Will return docker-registry.drivers.file.Storage
juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 DEBUG: Will return docker-registry.drivers.file.Storage
juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 WARNING: Cache storage disabled!
juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 WARNING: LRU cache disabled!
juil. 27 14:16:50 localhost gunicorn[11381]: 27/Jul/2015:14:16:50 +0000 DEBUG: Will return docker-registry.drivers.file.Storage
juil. 27 14:16:51 localhost gunicorn[11381]: 27/Jul/2015:14:16:51 +0000 WARNING: Cache storage disabled!
juil. 27 14:16:51 localhost gunicorn[11381]: 27/Jul/2015:14:16:51 +0000 WARNING: LRU cache disabled!
juil. 27 14:16:51 localhost gunicorn[11381]: 27/Jul/2015:14:16:51 +0000 DEBUG: Will return docker-registry.drivers.file.Storage

Bruno, can you answer that so that we can reach a decision?

Whiteboard: advisory => advisory MGA4-64-OK

Samuel Verschelde 2015-07-27 14:22:40 CEST

Whiteboard: advisory MGA4-64-OK => advisory MGA5-64-OK

Samuel Verschelde 2015-07-27 14:22:50 CEST

Whiteboard: advisory MGA5-64-OK => advisory MGA5-64-OK?

Lewis Smith 2015-08-27 20:39:40 CEST

CC: (none) => lewyssmith
Whiteboard: advisory MGA5-64-OK? => advisory MGA5-64-OK? Feedback

Lewis Smith 2015-08-27 20:41:08 CEST

Whiteboard: advisory MGA5-64-OK? Feedback => advisory MGA5-64-OK? feedback

Bruno Cornec 2015-09-15 22:18:56 CEST

CC: (none) => bruno

Comment 8 Bruno Cornec 2015-09-15 22:22:08 CEST
Sorry, missed your comment Samuel.

As said in this post:
https://github.com/docker/docker-registry/issues/595

These are just warnings, not errors. I think we're good to upload.
David Walser 2015-09-17 20:38:12 CEST

Whiteboard: advisory MGA5-64-OK? feedback => advisory MGA5-64-OK

Comment 9 claire robinson 2015-09-18 16:45:09 CEST
Validating from Stormi's comment 7. Advisory amended as per comment 5.

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs

Comment 10 Mageia Robot 2015-09-18 17:58:15 CEST
An update for this issue has been pushed to Mageia Updates repository.

http://advisories.mageia.org/MGAA-2015-0128.html

Status: ASSIGNED => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.