The koji 1.22.0 update added handling for the '--extra-boot-args' kickstart option. UNfortunately, this support wasn't working right and causes all our images specifying such options to fail.
The good news is that the fix is just a one-liner:
- cmd.extend(['--extra-boot-args', - '--append="%s"' % b_append]) + cmd.extend(['--extra-boot-args', '"%s"' % b_append])
I've built a new rpm with this patch applied and would like to do a freeze break to apply it to the 'builders' ansible group. (This applies to kojid).
See: https://pagure.io/koji/pull-request/2452 https://pagure.io/fedora-infrastructure/issue/9271
+1s?
kevin
The koji 1.22.0 update added handling for the '--extra-boot-args' kickstart option. UNfortunately, this support wasn't working right and causes all our images specifying such options to fail.
The good news is that the fix is just a one-liner:
cmd.extend(['--extra-boot-args','--append=\"%s\"' % b_append])
cmd.extend(['--extra-boot-args', '\"%s\"' % b_append])I've built a new rpm with this patch applied and would like to do a freeze break to apply it to the 'builders' ansible group. (This applies to kojid).
Seems fine +1
See: https://pagure.io/koji/pull-request/2452 https://pagure.io/fedora-infrastructure/issue/9271
+1s?
kevin _______________________________________________ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedorapro...
On Thu, Aug 27, 2020 at 2:36 PM kevin kevin@scrye.com wrote:
The koji 1.22.0 update added handling for the '--extra-boot-args' kickstart option. UNfortunately, this support wasn't working right and causes all our images specifying such options to fail.
The good news is that the fix is just a one-liner:
cmd.extend(['--extra-boot-args','--append=\"%s\"' % b_append])
cmd.extend(['--extra-boot-args', '\"%s\"' % b_append])
LGTM +1
I've built a new rpm with this patch applied and would like to do a freeze break to apply it to the 'builders' ansible group. (This applies to kojid).
See: https://pagure.io/koji/pull-request/2452 https://pagure.io/fedora-infrastructure/issue/9271
+1s?
kevin _______________________________________________ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedorapro...
+1
On Thu, 27 Aug 2020 at 16:40, Mohan Boddu mboddu@bhujji.com wrote:
On Thu, Aug 27, 2020 at 2:36 PM kevin kevin@scrye.com wrote:
The koji 1.22.0 update added handling for the '--extra-boot-args' kickstart option. UNfortunately, this support wasn't working right and causes all our images specifying such options to fail.
The good news is that the fix is just a one-liner:
cmd.extend(['--extra-boot-args','--append=\"%s\"' % b_append])
cmd.extend(['--extra-boot-args', '\"%s\"' % b_append])LGTM +1
I've built a new rpm with this patch applied and would like to do a freeze break to apply it to the 'builders' ansible group. (This applies to kojid).
See: https://pagure.io/koji/pull-request/2452 https://pagure.io/fedora-infrastructure/issue/9271
+1s?
kevin _______________________________________________ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to
infrastructure-leave@lists.fedoraproject.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives:
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedorapro... _______________________________________________ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedorapro...
On Thu, Aug 27, 2020 at 2:37 PM kevin kevin@scrye.com wrote:
The koji 1.22.0 update added handling for the '--extra-boot-args' kickstart option. UNfortunately, this support wasn't working right and causes all our images specifying such options to fail.
The good news is that the fix is just a one-liner:
cmd.extend(['--extra-boot-args','--append=\"%s\"' % b_append])
cmd.extend(['--extra-boot-args', '\"%s\"' % b_append])I've built a new rpm with this patch applied and would like to do a freeze break to apply it to the 'builders' ansible group. (This applies to kojid).
See: https://pagure.io/koji/pull-request/2452 https://pagure.io/fedora-infrastructure/issue/9271
+1s?
LGTM!
+1
infrastructure@lists.fedoraproject.org
-
kevin -
Mohan Boddu -
Neal Gompa -
Peter Robinson -
Stephen John Smoogen