qubes-dom0-update: report when there are no updates available #1475

andrewdavidwong · Dec 1, 2015

Currently, when sys-firewall is based on fedora-23-minimal, there is no feedback indicating whether qubes-dom0-updates simply found no updates or whether it completely failed for some other reason:

[user@dom0 ~]$ sudo qubes-dom0-update
Using sys-firewall as UpdateVM to download updates for Dom0; this may take some time...
Running command on VM: 'sys-firewall'...
Checking for dom0 updates...
Yum command has been deprecated, use dnf instead.
See 'man dnf' and 'man yum2dnf' for more information.

[user@dom0 ~]$

(Or maybe qubes-dom0-update really is failing here because it's trying to use yum instead of dnf, in which case this ticket is really about that.)

My experience suggests that the update script works, although it complains about this. But I agree that this is somewhat more than just a cosmetic issue, as it makes some confusion.

v6ak · Dec 4, 2015

My experience suggests that the update script works, although it complains about this. But I agree that this is somewhat more than just a cosmetic issue, as it makes some confusion.

This (text about deprecated yum) is quite hard to solve, because yum in F23 unconditionally prints that text, but dnf have some missing features/bugs making it hardly useful for dom0 updates:
https://bugzilla.redhat.com/show_bug.cgi?id=1279001
https://bugzilla.redhat.com/show_bug.cgi?id=1227014

marmarek · Dec 4, 2015

This (text about deprecated yum) is quite hard to solve, because yum in F23 unconditionally prints that text, but dnf have some missing features/bugs making it hardly useful for dom0 updates:
https://bugzilla.redhat.com/show_bug.cgi?id=1279001
https://bugzilla.redhat.com/show_bug.cgi?id=1227014

IMHO, the text about yum/dnf is not the problem. You can just leave it there. The problem, rather, is that there is no text after that to tell the user "No updates found." Instead, the user is just left to wonder whether there was some error preventing the system from even checking for updates in the first place (or downloading them, or installing them, etc.).

andrewdavidwong · Dec 4, 2015

IMHO, the text about yum/dnf is not the problem. You can just leave it there. The problem, rather, is that there is no text after that to tell the user "No updates found." Instead, the user is just left to wonder whether there was some error preventing the system from even checking for updates in the first place (or downloading them, or installing them, etc.).

Ah, I see. Will look into that.

marmarek · Dec 4, 2015

Ah, I see. Will look into that.

Maybe some hack like the following would be good enough:

… | grep -vE '^(Yum command has been deprecated, use dnf instead\.|See \'man dnf\' and \'man yum2dnf\' for more information\.)$'

v6ak · Dec 5, 2015

Maybe some hack like the following would be good enough:

… | grep -vE '^(Yum command has been deprecated, use dnf instead\.|See \'man dnf\' and \'man yum2dnf\' for more information\.)$'

On Sat, Dec 05, 2015 at 03:16:01AM -0800, Vít Šesták wrote:

Maybe some hack like the following would be good enough:

… | grep -vE '^(Yum command has been deprecated, use dnf instead\.|See \'man dnf\' and \'man yum2dnf\' for more information\.)$'

Do you know how to do it, without merging stdout and stderr?

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

marmarek · Dec 5, 2015

On Sat, Dec 05, 2015 at 03:16:01AM -0800, Vít Šesták wrote:

Maybe some hack like the following would be good enough:

… | grep -vE '^(Yum command has been deprecated, use dnf instead\.|See \'man dnf\' and \'man yum2dnf\' for more information\.)$'

Do you know how to do it, without merging stdout and stderr?

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Hmm, without merging stderr+stdout, it is not so easy, but still doable: https://gist.github.com/v6ak/58097ef190157d573646

But I understand it is not so simple hack now.

v6ak · Dec 5, 2015

Hmm, without merging stderr+stdout, it is not so easy, but still doable: https://gist.github.com/v6ak/58097ef190157d573646

But I understand it is not so simple hack now.

Automated announcement from builder-github

The package python2-dnf-plugins-qubes-hooks-3.1.9-1.fc21 has been pushed to the r3.1 testing repository for the Fedora fc21 template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r3.1-current-testing

QubesOS/qubes-issues

Join GitHub today

qubes-dom0-update: report when there are no updates available #1475

Comments

andrewdavidwong commented Dec 1, 2015

This comment has been minimized.

v6ak commented Dec 4, 2015

This comment has been minimized.

marmarek commented Dec 4, 2015

This comment has been minimized.

andrewdavidwong commented Dec 4, 2015

marmarek added bug C: core P: minor labels Dec 4, 2015

marmarek added this to the Release 3.0 updates milestone Dec 4, 2015

This comment has been minimized.

marmarek commented Dec 4, 2015

This comment has been minimized.

v6ak commented Dec 5, 2015

This comment has been minimized.

marmarek commented Dec 5, 2015

This comment has been minimized.

v6ak commented Dec 5, 2015

marmarek closed this in marmarek/old-qubes-core-agent-linux@c46c1e4 Dec 26, 2015

This comment has been minimized.

marmarek commented Dec 26, 2015

marmarek added the r3.1-fc21-cur-test label Dec 26, 2015

This comment has been minimized.

marmarek commented Dec 26, 2015

marmarek added the r3.1-fc22-cur-test label Dec 26, 2015

This comment has been minimized.

marmarek commented Dec 26, 2015

marmarek added the r3.1-fc23-cur-test label Dec 26, 2015

This comment has been minimized.

marmarek commented Dec 26, 2015

marmarek added the r3.1-jessie-cur-test label Dec 26, 2015

This comment has been minimized.

marmarek commented Dec 26, 2015

marmarek added the r3.1-stretch-cur-test label Dec 26, 2015

This comment has been minimized.

marmarek commented Dec 26, 2015

marmarek added the r3.1-wheezy-cur-test label Dec 26, 2015

marmarek referenced this issue Jan 7, 2016

This comment has been minimized.

marmarek commented Jan 12, 2016

marmarek added r3.1-fc21-stable and removed r3.1-fc21-cur-test labels Jan 12, 2016

This comment has been minimized.

marmarek commented Jan 12, 2016

marmarek added r3.1-fc22-stable and removed r3.1-fc22-cur-test labels Jan 12, 2016

This comment has been minimized.

marmarek commented Jan 12, 2016

marmarek added r3.1-fc23-stable and removed r3.1-fc23-cur-test labels Jan 12, 2016

This comment has been minimized.

marmarek commented Jan 13, 2016

marmarek added r3.1-jessie-stable and removed r3.1-jessie-cur-test labels Jan 13, 2016

This comment has been minimized.

marmarek commented Jan 13, 2016

marmarek added r3.1-wheezy-stable and removed r3.1-wheezy-cur-test labels Jan 13, 2016

marmarek added a commit to QubesOS/qubes-core-agent-linux that referenced this issue Feb 29, 2016

This comment has been minimized.

marmarek commented Feb 29, 2016

marmarek added the r3.0-jessie-testing label Feb 29, 2016

This comment has been minimized.

marmarek commented Feb 29, 2016

marmarek added the r3.0-wheezy-testing label Feb 29, 2016

This comment has been minimized.

marmarek commented Feb 29, 2016

marmarek added the r3.0-fc20-testing label Feb 29, 2016

This comment has been minimized.

marmarek commented Feb 29, 2016

marmarek added the r3.0-fc21-testing label Feb 29, 2016

This comment has been minimized.

marmarek commented Feb 29, 2016

marmarek added the r3.0-fc22-testing label Feb 29, 2016

This comment has been minimized.

marmarek commented Feb 29, 2016

marmarek added the r3.0-fc23-testing label Feb 29, 2016

This comment has been minimized.

marmarek commented Mar 10, 2016

marmarek added r3.0-fc20-stable and removed r3.0-fc20-testing labels Mar 10, 2016

This comment has been minimized.

marmarek commented Mar 10, 2016

marmarek closed this in `marmarek/old-qubes-core-agent-linux@c46c1e4` Dec 26, 2015