Bug 827269 - ksflatten ignoring nested includes
Summary: ksflatten ignoring nested includes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: pykickstart
Version: 17
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Chris Lumens
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 819672 838713 846835 (view as bug list)
Depends On:
Blocks: 819672
TreeView+ depends on / blocked
 
Reported: 2012-06-01 01:16 UTC by Joey Boggs
Modified: 2013-04-10 14:31 UTC (History)
5 users (show)

Fixed In Version: pykickstart-1.99.13-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-04-10 14:31:53 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
failed flatten (4.29 KB, text/plain)
2012-06-01 01:16 UTC, Joey Boggs
no flags Details
successful flatten (30.69 KB, text/plain)
2012-06-01 01:17 UTC, Joey Boggs
no flags Details
patch (2.95 KB, patch)
2012-06-13 23:27 UTC, Brian Lane
no flags Details | Diff

Description Joey Boggs 2012-06-01 01:16:39 UTC
Created attachment 588226 [details]
failed flatten

ksflatten seems to ignore nested %includes and places the section directly into the resulting kickstart.

1.99-4.1 from F16 works fine, 1.99-11 through 18 aren't working


# ovirt16 Node image recipe
%include common-install.ks
%include ovirt16-install.ks
%include repos.ks

%packages --excludedocs --nobase
%include common-pkgs.ks
%include ovirt16-pkgs.ks
%end

%post
%include common-post.ks
%include ovirt16-post.ks
%end

%post --nochroot
%include common-nochroot.ks
%end

%include image-minimizer.ks
%include common-manifest.ks

attaching resulting kickstart, not sure if something has changed in syntax or just a bug

Comment 1 Joey Boggs 2012-06-01 01:17:24 UTC
Created attachment 588227 [details]
successful flatten

Comment 2 Mike Burns 2012-06-13 21:51:06 UTC
Any chance someone can look at this?  This is strongly impacting oVirt-node.  It forces us to manually roll back to the F16 version of pykickstart and prevent ever updating it.

Thanks

Comment 3 Brian Lane 2012-06-13 23:27:35 UTC
Created attachment 591669 [details]
patch

Comment 4 Mike Burns 2012-06-14 13:31:58 UTC
*** Bug 819672 has been marked as a duplicate of this bug. ***

Comment 5 Chris Lumens 2012-06-18 17:18:12 UTC
I built 1.99.12 before lunch, but didn't push yet.  This'll be fixed in 1.99.13 instead.

Comment 6 Brian Lane 2012-07-09 23:33:30 UTC
*** Bug 838713 has been marked as a duplicate of this bug. ***

Comment 7 Mike Burns 2012-07-12 11:26:47 UTC
I see the build in F18, but can this be built for F17 as well?  This is a significant pain point for building ovirt-node and keeping the build machines relatively up to date.  

Thanks

Comment 8 Pavel Zhukov 2012-08-03 06:45:48 UTC
http://koji.fedoraproject.org/koji/taskinfo?taskID=4353353

This build works for me

Comment 9 Brian Lane 2012-08-09 16:31:45 UTC
*** Bug 846835 has been marked as a duplicate of this bug. ***

Comment 10 Mike Burns 2012-08-09 16:43:53 UTC
The F18/rawhide build works for me too, but it would be much appreciated if we could backport this fix into F17 as well.

Any chance of that happening?

Comment 11 Fedora Update System 2012-08-09 19:01:07 UTC
pykickstart-1.99.15-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/pykickstart-1.99.15-1.fc17

Comment 12 Fedora Update System 2012-08-10 22:32:36 UTC
Package pykickstart-1.99.15-1.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing pykickstart-1.99.15-1.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-11727/pykickstart-1.99.15-1.fc17
then log in and leave karma (feedback).

Comment 13 Netbulae 2012-08-28 11:39:22 UTC
Works for me, many thanks!!


Note You need to log in before you can comment on or make changes to this bug.