[PATCH] include variant in the name for .treeinfo when it exists
by Dennis Gilmore
if there is a variant set include that in the name, otherwise
all the variants end up having the same name set.
Signed-off-by: Dennis Gilmore <dennis(a)ausil.us>
---
src/pylorax/treeinfo.py | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/pylorax/treeinfo.py b/src/pylorax/treeinfo.py
index 6f3f620..ac9806c 100644
--- a/src/pylorax/treeinfo.py
+++ b/src/pylorax/treeinfo.py
@@ -33,11 +33,16 @@ class TreeInfo(object):
self.c = ConfigParser.ConfigParser()
+ if variant:
+ name = "%s-%s-%s" % (name, variant, version)
+ else:
+ name = "%s-%s" % (name, version)
+
section = "general"
data = {"timestamp": time.time(),
"family": product,
"version": version,
- "name": "%s-%s" % (product, version),
+ "name": name,
"variant": variant or "",
"arch": basearch,
"packagedir": packagedir}
--
2.3.0
9 years, 2 months
[PATCH] Use the LUKS device for swap in fstab (#1196200)
by Vratislav Podzimek
If we create an encrypted swap we need to use the LUKS device in fstab for such
swap not the partition/LV.
Signed-off-by: Vratislav Podzimek <vpodzime(a)redhat.com>
---
pyanaconda/kickstart.py | 27 +++++++++++++++++++++++----
1 file changed, 23 insertions(+), 4 deletions(-)
diff --git a/pyanaconda/kickstart.py b/pyanaconda/kickstart.py
index 756eb81..3656352 100644
--- a/pyanaconda/kickstart.py
+++ b/pyanaconda/kickstart.py
@@ -933,6 +933,7 @@ class LogVolData(commands.logvol.F21_LogVolData):
raise KickstartValueError(formatErrorMsg(self.lineno,
msg=_("The \"%s\" file system type is not supported.") % ty))
+ add_fstab_swap = None
# If we were given a pre-existing LV to create a filesystem on, we need
# to verify it and its VG exists and then schedule a new format action
# to take place there. Also, we only support a subset of all the
@@ -956,7 +957,7 @@ class LogVolData(commands.logvol.F21_LogVolData):
devicetree.registerAction(ActionCreateFormat(device, fmt))
if ty == "swap":
- storage.addFstabSwap(device)
+ add_fstab_swap = device
else:
# If a previous device has claimed this mount point, delete the
# old one.
@@ -1011,7 +1012,7 @@ class LogVolData(commands.logvol.F21_LogVolData):
storage.createDevice(request)
if ty == "swap":
- storage.addFstabSwap(request)
+ add_fstab_swap = request
if self.encrypted:
if self.passphrase and not storage.encryptionPassphrase:
@@ -1038,8 +1039,16 @@ class LogVolData(commands.logvol.F21_LogVolData):
luksdev = LUKSDevice("luks%d" % storage.nextID,
fmt=luksformat,
parents=request)
+ if ty == "swap":
+ # swap is on the LUKS device not on the LUKS' parent device,
+ # override the info here
+ add_fstab_swap = luksdev
+
storage.createDevice(luksdev)
+ if add_fstab_swap:
+ storage.addFstabSwap(add_fstab_swap)
+
class Logging(commands.logging.FC6_Logging):
def execute(self, *args):
if logger.loglevel == DEFAULT_LEVEL:
@@ -1267,6 +1276,7 @@ class PartitionData(commands.partition.F18_PartData):
kwargs["primary"] = self.primOnly
+ add_fstab_swap = None
# If we were given a pre-existing partition to create a filesystem on,
# we need to verify it exists and then schedule a new format action to
# take place there. Also, we only support a subset of all the options
@@ -1289,7 +1299,7 @@ class PartitionData(commands.partition.F18_PartData):
devicetree.registerAction(ActionCreateFormat(device, kwargs["fmt"]))
if ty == "swap":
- storage.addFstabSwap(device)
+ add_fstab_swap = device
# tmpfs mounts are not disks and don't occupy a disk partition,
# so handle them here
elif self.fstype == "tmpfs":
@@ -1315,7 +1325,7 @@ class PartitionData(commands.partition.F18_PartData):
storage.createDevice(request)
if ty == "swap":
- storage.addFstabSwap(request)
+ add_fstab_swap = request
if self.encrypted:
if self.passphrase and not storage.encryptionPassphrase:
@@ -1342,8 +1352,17 @@ class PartitionData(commands.partition.F18_PartData):
luksdev = LUKSDevice("luks%d" % storage.nextID,
fmt=luksformat,
parents=request)
+
+ if ty == "swap":
+ # swap is on the LUKS device not on the LUKS' parent device,
+ # override the info here
+ add_fstab_swap = luksdev
+
storage.createDevice(luksdev)
+ if add_fstab_swap:
+ storage.addFstabSwap(add_fstab_swap)
+
class Raid(commands.raid.F20_Raid):
def execute(self, storage, ksdata, instClass):
for r in self.raidList:
--
2.1.0
9 years, 2 months
Please take a look at https://github.com/mulkieran/blivet (tasks branch)
by Anne Mulhern
The patches from Feb 27 are the relevant ones.
The first six are cleanup.
The remaining, until commit cf724750c simply put identified tasks into separate classes
with appropriate interfaces and then use those interfaces where appropriate, but
keeping behavior entirely consistent.
The rest experiment slightly with using the interface to refine some of the properties of
filesystem devices.
The number of patches is large and I don't want to inundate the mailing list, so I'm trying
this.
- mulhern
9 years, 2 months
[PATCH rhel6-branch] On interactive kickstart installs, still get the password from the ks file.
by Chris Lumens
...unless the user enters a new password in the UI. In that case, use whatever
they enter.
Resolves: rhbz#995531
---
iw/account_gui.py | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/iw/account_gui.py b/iw/account_gui.py
index 7d5d292..2da1b5e 100644
--- a/iw/account_gui.py
+++ b/iw/account_gui.py
@@ -62,6 +62,9 @@ class AccountWindow (InstallWindow):
if not self.rootPassword['isCrypted']:
self.pw.set_text(self.rootPassword['password'])
self.confirm.set_text(self.rootPassword['password'])
+ else:
+ self.pw.set_text("xxxxxxxx")
+ self.confirm.set_text("xxxxxxxx")
# pressing Enter in confirm == clicking Next
vbox = self.xml.get_widget("account_box")
@@ -95,6 +98,10 @@ class AccountWindow (InstallWindow):
self.capslock.set_text("")
def getNext (self):
+ # check if we already have an encrypted password from kickstart
+ if self.rootPassword["isCrypted"] and self.pw.get_text() == "xxxxxxxx":
+ return None
+
pw = self.pw.get_text()
confirm = self.confirm.get_text()
--
2.2.2
9 years, 2 months