Home_greyopenFATE - openSUSE feature tracking > #304889
Dashboard | Search | Sign up | Login

Please login or register to be able to edit or vote this feature.

needs_relogin/needs_reboot as a generic flag

Feature state

openSUSE-11.1
Done

Description

There are situations, where relogin or reboot are needed to activate update:

  • package updates
  • proxy settings

Now this situation is not done in a consistent way (silently ignoring problem, note in help, abuse of License entry, pop-up...).

Providing a common way for setting these flags (via package provides, YaST calls, sysconfig comments) (and maybe comment what will happen ignoring it) would help in user experience (user is notified about need of relogin/reboot in a consistent way and only once) and administration (automatic relogin or reboot during maintenance window).

Discussion


icons/user_comment.png S. V. wrote: (9 years ago)

needs_reboot is properly handled by our tools. needs_relogin does not exist.

icons/user_comment.png J. S. wrote: (9 years ago)

We handle reboot only for package management not for anything else (the question is whether it is relevant somewhere else), thus we will need to have a more generic solution.

Martin, would you, please, have a look and suggest a generic handler?

icons/user_comment.png S. B. wrote: (9 years ago)

needs_relogin would be useful in these situations:

  • New package, which provides file in /etc/profile.d was installed (or /etc/profile.d was singnificantly modified by the update). Example: ladspa
  • Packages, that cause that /etc/profile will evaluate environment. Example: Install any KDE application in GNOME only system. /opt/kde3 is not in PATH until next login
  • Patches, that affect login related programs. Example: security update of ssh-agent or gnome-keyring
  • YaST settings, that cause environment variable changes. Example: System proxy settings
icons/user_comment.png D. M. wrote: (9 years ago)

Stefan, how is restart and reboot currently handled in packagekit, what would we need for relogin?

icons/user_comment.png D. M. wrote: (9 years ago)

(note, ZYpp parts are already done, applications missing )

icons/user_comment.png S. H. wrote: (9 years ago)

PackageKit has three different types for reboot/restart:
-Restart System
-Restart Application
-Restart Session

icons/user_comment.png D. M. wrote: (9 years ago)

Stefan Haas will adapt code from PackageKit to support this flag.

icons/user_comment.png D. M. wrote: (9 years ago)

Thomas, does the KDE applet handle the relogin needed flag from PackageKit?

icons/user_comment.png T. G. wrote: (9 years ago)

kde4-kupdateapplet listens to PackageKit's restart signal since version 0.8.4.

icons/user_comment.png D. M. wrote: (9 years ago)

Ladislav, is are the current flags in YaST handled in pkg-bindings?

icons/user_comment.png L. S. wrote: (9 years ago)

I just have committed the support for Patch::reloginSuggested() flag in yast SVN. It will be in yast2-pkg-bindings-2.17.8.

I guess we also need some support in YCP... Jiri?

icons/user_comment.png J. S. wrote: (9 years ago)

Thomas/Stefan: which message does the applet show? I think we should try to show the same one in YaST and zypper.

(BTW, zypper should be adapted as well -> Jano)

icons/user_comment.png T. G. wrote: (9 years ago)

kupdateapplet shows "Please reboot your system as soon as possible." or "Please relogin into your system as soon as possible.". If PackageKit provides details, these details are shown too.
Please let me know if you prefer other strings.

icons/user_comment.png J. S. wrote: (9 years ago)

Thanks.

So I'm going to use this one "At least one of the updates installed requires
restart of the session.
Relogin into your system as soon as possible."

I removed the "Please" intentionally, as such wording should not be used (I assume it can be found is the text style guide) and added kind of explanation sentence (similar is used by YaST in similar situations).

icons/user_comment.png T. G. wrote: (9 years ago)

I've removed the "please", too. Thanks for your hint.

icons/user_comment.png S. B. wrote: (9 years ago)

Please tell me, how could I set this flag in the spec file. Actually, scriptlets are the best place to do it, as there is possible to filter unneeded restarts.

Example: If krb5-client package is installed for the first time, relogin is required, but in case of update from new enough version, relogin is not needed, because there is no change in /etc/profile.d/krb5.sh.

icons/user_comment.png J. S. wrote: (9 years ago)

I have no idea if this flag can be used for packages. My understanding is that it is part of patch description.

icons/user_comment.png S. B. wrote: (9 years ago)

Relogin is needed especially for newly installed packages. "As a generic flag" in the FATE was intended as: Generic solution usable in both yast and packages. Packages need most probably a command line utility to set this flag in YaST.

icons/user_comment.png S. V. wrote: (9 years ago)

I'm afraid we have to stick with per-patch flag for openSUSE 11.1. I suggest to have another Fate open for per-package.

icons/user_comment.png J. S. wrote: (9 years ago)

YaST part done in yast2-online-update-2.17.5

Last change: 9 years ago
Voting
Score: 0
  • Negative: 0
  • Neutral: 0
  • Positive: 0
Feature Export
Application-xmlXML   Text-x-logPlaintext   PrinterPrint