Bug 554717
Summary: | RFE: Better groups exclusion support | |||
---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Alexander Todorov <atodorov> | |
Component: | pykickstart | Assignee: | Chris Lumens <clumens> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | |
Severity: | medium | Docs Contact: | ||
Priority: | low | |||
Version: | rawhide | CC: | clumens | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | All | |||
OS: | Linux | |||
Whiteboard: | ||||
Fixed In Version: | pykickstart-1.68-1 | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 555311 (view as bug list) | Environment: | ||
Last Closed: | 2010-07-01 20:57:54 UTC | Type: | --- | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 555311, 558516 |
Description
Alexander Todorov
2010-01-12 13:31:23 UTC
Has this actually been tested in an install? I don't see how just adding the excluded group to newExcludedSet is going to help. That'll mean the group name gets into self.excludedList, which then gets passed on to YumBackend.deselectPackage. I don't think that's doing anything magic to iterate over the packages in a group, or knows to deselect the group as opposed to a package. It almost seems like we need an extra list here of groups to exclude, then call the appropriate thing in anaconda. Of course, I could just not be following the patch correctly. (In reply to comment #1) > Has this actually been tested in an install? No. What I meant is that we've tested that pykickstart with this patch will generate a proper ks.cfg file. I've started a test job with manually created ks.cfg file to see how anaconda will interpret it. It's likely that anaconda will require another fix or modification of this patch. Can you attach a test script and example of what a "proper ks.cfg file" looks like? I want to test an alternate patch here and make sure we get the same results. A test script is the following SNAKE template: from pykickstart.version import FC6 from installdefaults import InstallKs # minimal install, procedural style def ks(**context): '''Package set / Everything''' ks=InstallKs() if ks.version <= FC6: ks.packages.add(['@everything']) else: ks.packages.add(['*']) ks.packages.add(['-@conflicts']) return ks The result should contain (along with other default values): %packages * -@conflicts Okay, fixing this to work in anaconda too makes things a whole lot more complicated, but I think I've managed to make sense of it all. This should now work like you want both in pykickstart's __str__ methods and in anaconda. Chris, can you post your patch or an updates.img for RHEL5.5 so I can give it a try? This works OK for QE. snake.test server now generates a ks.cfg with %packages * -@conflicts |