-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
systemd: Show package updates status on front page #8822
systemd: Show package updates status on front page #8822
Conversation
Screenshot from my system: Or for security updates: Clicking on the link leads to "Software Updates". If cockpit-packagekit is not installed, it's not a link, just text. If the system is not registered: Again, clicking on it leads to /subscriptions (if installed, otherwise it's just a normal text). While the updates are loading, there is a spinner icon with "Checking For Updates…". It usually appears < 1 s as this does not have to load any details about the updates. This is also the reason why it does not see the individual Red Hat security update severity classification, but I think this is a worthwhile tradeoff for a massively faster result. |
I tested it on my system, and the Server page said "Bug Fix Updates Available", but when I went to the Software Updates page, it showed me a bunch security updates. I do get and "Oops", so something might be janky. |
The registration notification works as expected! |
3b3a099
to
dafd726
Compare
@andreasn: You are on Fedora? cockpit-packagekit has a rather expensive workaround for https://bugs.freedesktop.org/show_bug.cgi?id=101070 . The current system page doesn't currently load all the details from all the updates, so it can't figure this out (it's also just a heuristics by parsing changelogs for CVEs). This should work fine on RHEL. I don't have known oopses here, can you please post the log of it? |
TypeError: self.refresh_os_updates_state is not a function[Learn More] system.js:3911:14 |
dafd726
to
1f270d5
Compare
@andreasn: I could reproduce your bug at last. subscription-manager is broken in Fedora 27, I just filed https://bugzilla.redhat.com/show_bug.cgi?id=1555384 . Once I fixed that locally, I got the crash as well, stupid typo :-) Thanks for finding! Fix pushed. |
Several tests fail on unexpected SELinux violations from rhsmd (example 1, example 2):
These need to be reported downstream and naughty-ed. |
rhsm is broken on our fedora-27 image due to a missing dependency, I filed that as https://bugzilla.redhat.com/show_bug.cgi?id=1556738. But I cannot reproduce the various SELinux violations manually, with either a busctl, or directly starting /usr/libexec/rhsmd or even with logging in on cockpit and have that do the d-bus call. I do get the violations with running testAbrtReport locally, but that's not yet helpful for downstream. At least after sitting after that test cases' failure, the violations happen again straight after logging in. So that test triggers some change on the system. Update: This is due to the test calling |
355bf53
to
db20fed
Compare
I reported the rhsmd SELinux violations downstream, and added a naughty override. I also removed some obsolete SELinux disablings in the tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Awesome work!
Ugh, on Windows 8 (with Edge) we get page oopses like mad due to |
db20fed
to
38adb29
Compare
I fixed the Promise oopses with IE in #8838, and included this here to already confirm with CI (tests run fine locally now, and I verified that the subscription/updates status works in IE). Setting to blocked until that PR lands. |
38adb29
to
10fb3c5
Compare
It is not necessary, we want to validate this behaviour in the default system configuration (with SELinux enabled).
https://bugzilla.redhat.com/show_bug.cgi?id=1278287 got fixed in Fedora 23 three years ago already. Let's make sure we test it under the default system configuration with SELinux enabled.
Show a summary indicator on the system front page whether the host has available package updates. https://bugzilla.redhat.com/show_bug.cgi?id=1495543 Fixes cockpit-project#7758 Closes cockpit-project#8822
10fb3c5
to
bdec9cb
Compare
Show a summary indicator on the system front page whether the host has available package updates. https://bugzilla.redhat.com/show_bug.cgi?id=1495543 Fixes #7758 Closes #8822
With dnf we don't (currently) get a valid package update severity. In these cases, stop showing "bug fix" for them, and just show "Updates available" without further qualification. Do keep the bug icon for them though. Follow-up for PR cockpit-project#8822 Closes cockpit-project#8867
Show a summary indicator on the system front page whether the host has
available package updates.
https://bugzilla.redhat.com/show_bug.cgi?id=1495543
Fixes #7758
TypeError: self.refresh_os_updates_state is not a function
Promise
polyfill for IE (PR lib: Add missing Promise polyfill for IE #8838)follow-ups: