[Bug 980313] New: /usr/lib/zypp/plugins/commit/susemanager should be moved to susemanager related package and not be part of base salt-minion
http://bugzilla.suse.com/show_bug.cgi?id=980313 Bug ID: 980313 Summary: /usr/lib/zypp/plugins/commit/susemanager should be moved to susemanager related package and not be part of base salt-minion Classification: openSUSE Product: openSUSE Distribution Version: Leap 42.1 Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Other Assignee: bnc-team-screening@forge.provo.novell.com Reporter: mrueckert@suse.com QA Contact: qa-bugs@suse.de Found By: --- Blocker: --- seen with salt-minion-2015.8.8-68.1.x86_64 from systemsmanagement:saltstack the salt setup on the machine was turned off atm and this noticably slows down all zypp operations. if this plugin is mostly for susemanager it shouldnt be part of the normal salt-minion package. if it is meant to be used with normal salt, i would recommend renaming it. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=980313
Bo Maryniuk
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c2
Andrej Semen
http://bugzilla.suse.com/show_bug.cgi?id=980313
Marcus Rückert
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c7
--- Comment #7 from Bernhard Wiedemann
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c8
Duncan Mac-Vicar
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c9
--- Comment #9 from Marcus Rückert
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c10
Duncan Mac-Vicar
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c11
--- Comment #11 from Duncan Mac-Vicar
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c12
Bo Maryniuk
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c13
Bo Maryniuk
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c14
Klaus Kämpf
This SR should be reverted: https://build.opensuse.org/request/show/430686
This SR is for 2016.3 and thus not breaking (released) SUSE Manager. Splitting SUSE Manager specifics off of salt is the "right thing to do" going forward. Putting it into Salt was a shortcut we took when releasing SUSE Manager 3. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c15
--- Comment #15 from Duncan Mac-Vicar
http://bugzilla.suse.com/show_bug.cgi?id=980313
Bo Maryniuk
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c17
--- Comment #17 from Duncan Mac-Vicar
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c18
--- Comment #18 from Klaus Kämpf
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c19
--- Comment #19 from Bo Maryniuk
A /usr/lib/zypp/plugins/commit/*susemanager* plugin must not be part of Salt. In this respect, I agree with the original reporter.
However, a generic /usr/lib/zypp/plugins/commit/*salt_notifier*, together with a respective 'yum' notifier (for Fedora/RHEL, 'apt' will follow later) might be part of Salt
You say "must not be part of Salt", then you say "might be part of Salt", which already is anyway. My point is, this is not something one just takes and changes on his own alone, but must be *team* discussion prior that and clear knowledge, reflecting in the RFC we *agreed*: https://github.com/SUSE/susemanager-rfc/blob/master/text/00003-salt-package-... — please note, this section is about Zypper + Salt. The name is probably an issue, but I find it a low priority, because the content of the plugin won't change. Additionally, this part nor affects overall package functionality neither is directly user-land tooling whether generic users must use "susemanager" script directly, hence this makes the task even lower priority. However, abruptly splitting without looking through the consequences and verification by support team is what I find certainly wrong. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c20
--- Comment #20 from Klaus Kämpf
You say "must not be part of Salt", then you say "might be part of Salt", which already is anyway.
Bo, please read carefully. A *susemanager* plugins must not be part of Salt, a *salt_notifier* might.
My point is, this is not something one just takes and changes on his own alone, but must be *team* discussion prior that and clear knowledge, reflecting in the RFC we *agreed*:
I totally agree when it comes to upgrading Salt **in SUSE Manager** (that is systemsmanagement:saltstack:products) to 2016.3 However, systemsmanagement:saltstack is for openSUSE. Nobody took care of this project. 2016.3 is already ~6 months old and wasn't submitted to :saltstack yet. Since darix and tampakrap took the initiative to push the upgrade (and agreed to do the testing and fixing if needed) to openSUSE, they have my full support. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=980313
Bo Maryniuk
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c23
--- Comment #23 from Swamp Workflow Management
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c26
--- Comment #26 from Swamp Workflow Management
http://bugzilla.suse.com/show_bug.cgi?id=980313
http://bugzilla.suse.com/show_bug.cgi?id=980313#c27
--- Comment #27 from Swamp Workflow Management
https://bugzilla.suse.com/show_bug.cgi?id=980313
https://bugzilla.suse.com/show_bug.cgi?id=980313#c30
--- Comment #30 from Swamp Workflow Management
https://bugzilla.suse.com/show_bug.cgi?id=980313
https://bugzilla.suse.com/show_bug.cgi?id=980313#c32
--- Comment #32 from Swamp Workflow Management
https://bugzilla.suse.com/show_bug.cgi?id=980313
https://bugzilla.suse.com/show_bug.cgi?id=980313#c38
--- Comment #38 from Swamp Workflow Management
https://bugzilla.suse.com/show_bug.cgi?id=980313
https://bugzilla.suse.com/show_bug.cgi?id=980313#c39
--- Comment #39 from Swamp Workflow Management
participants (2)
-
bugzilla_noreply@novell.com
-
bugzilla_noreply@suse.com