Re: [PATCH] bodhi update run: Attempt to get better Atomic images media name during updates run

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Thu, Jun 21, 2018 at 11:51 PM, Dusty Mabe <dusty@xxxxxxxxxxxxx> wrote:


On 06/21/2018 10:49 AM, Sinny Kumari wrote:
> From past few weeks, we have started producing Atomic composes
> media during Fedora 28 bodhi updates and updates-testing run.
> Media name of AH ISO are perfectly fine. But for cloud images 
> media name[1] looks something like
> Fedora-AtomicHost-28_Update-20180620.1235.x86_64.qcow2 for qcow2.
> It will be nice to have media name like
> Fedora-AtomicHost-28-20180620.0.x86_64.qcow2 which
> is what we get during regular atomic twoweek compose run [2].
>
> This patch (available in email attachment) attempts to fix the media name.
> Taken inspiration from releng issue [3] and ansible commit [4]. I am not
> sure how to test it locally, any pointers will be useful.

Thanks Sinny!

Since we are already only building images for >= 28 then I don't think we
need the if/endif statements in your patch.

Yeah , we don't now but added it because we may need in future when F29
is released. I have updated patch (available in attachment) with removing if condition.


I would say to try to work with randy/lsedlar to test this in stage but
since nothing is yet depending on these images I'm not opposed to just
merging this and seeing if it works in prod.

Randy is on PTO currently, will check with lsedlar if he can test this patch
on stage.
 



--
From e69edba3a3c8da48b3102440a462c57e9919dbce Mon Sep 17 00:00:00 2001
From: Sinny Kumari <sinny@xxxxxxxxxx>
Date: Thu, 21 Jun 2018 19:55:57 +0530
Subject: [PATCH] Attempt to get better Atomic images media name during update
 run

Atomic cloud images produced during updates and updates-testing
run contains name something like
Fedora-AtomicHost-28_Update-20180620.1235.x86_64.qcow2 . It
will be nice to have media name
like Fedora-AtomicHost-28-20180620.0.x86_64.qcow2 which we
get during regular atomic twoweek compose run

Signed-off-by: Sinny Kumari <sinny@xxxxxxxxxx>
---
 roles/bodhi2/backend/templates/pungi.rpm.conf.j2 | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/roles/bodhi2/backend/templates/pungi.rpm.conf.j2 b/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
index 631e07af7..eec0a1615 100644
--- a/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
+++ b/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
@@ -198,6 +198,7 @@ image_build = {
     '^AtomicHost$': [
         {
             'image-build': {
+                'version': '!VERSION_FROM_VERSION_DATE_RESPIN',
                 'format': [('qcow2', 'qcow2'), ('raw-xz', 'raw.xz')],
                 'name': 'Fedora-AtomicHost',
                 'kickstart': 'fedora-atomic.ks',
@@ -218,6 +219,7 @@ image_build = {
         },
         {
             'image-build': {
+                'version': '!VERSION_FROM_VERSION_DATE_RESPIN',
                 'format': [('vagrant-libvirt','vagrant-libvirt.box'), ('vagrant-virtualbox','vagrant-virtualbox.box')],
                 'name': 'Fedora-AtomicHost-Vagrant',
                 'kickstart': 'fedora-atomic-vagrant.ks',
-- 
2.14.4

_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx/message/A2ABHNJW7GIL4QNHPO2A2DFJPBU2CB6J/

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux