Skip to content

add zypper arguments to snapshot description #315

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.

Sign up for GitHub

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

Open
kmotavalli wants to merge 10 commits into
base: master
Choose a base branch
from
Open

add zypper arguments to snapshot description #315

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
c6f8ab5
add zypper arguments to snapshot description
kmotavalli Jan 6, 2017
38f1673
parse configuration for extended zypper output
kmotavalli Jan 11, 2017
13a1439
update zypp-plugin.conf with zypper cmdline opts
kmotavalli Jan 11, 2017
ed2c422
eliminate trailing whitespace in description
kmotavalli Jan 11, 2017
e1e0bac
fix wording in zypper-extended-description comment
kmotavalli Jan 11, 2017
c58c282
fix typo in zypp-plugin comment
kmotavalli Jan 12, 2017
1243596
don't use an array for zypper_extended_description
kmotavalli Jan 12, 2017
d952d2c
use boolean True instead of the string "true"
kmotavalli Jan 12, 2017
5015c03
use booleans
kmotavalli Jan 12, 2017
02130c5
use integers instead of a string for string len
kmotavalli Jan 12, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion scripts/zypp-plugin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,12 @@ def match_solvables(self, names):
if found and important:
return True, True
return found, important

def zypper_arguments(self):
if basename(readlink("/proc/%d/exe" % getppid())) == "zypper":
return " ".join(open("/proc/%s/cmdline" % getppid()).read().split('\x00')[1:])
else:
return ""


def PLUGINBEGIN(self, headers, body):
Expand All @@ -153,7 +159,7 @@ def PLUGINBEGIN(self, headers, body):

logging.debug("headers: %s" % headers)

self.description = "zypp(%s)" % basename(readlink("/proc/%d/exe" % getppid()))
self.description = "zypp(%s) %s" % (basename(readlink("/proc/%d/exe" % getppid())), self.zypper_arguments())

@aschnell aschnell Jan 11, 2017

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFIAS the description has a trailing whitespace if it's not from zypper. That's not nice.

self.userdata = self.get_userdata(headers)

self.ack()
Expand Down