Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Kevin Fenzi
+1 seems reasonable. kevin signature.asc Description: OpenPGP digital signature ___ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org Fedora Code of

Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Mohan Boddu
Hopefully this is the last change: --- a/roles/bodhi2/backend/files/new-updates-sync +++ b/roles/bodhi2/backend/files/new-updates-sync @@ -18,11 +18,13 @@ RAWHIDESOURCE = '/mnt/koji/compose/rawhide/' FEDORADEST = '/pub/fedora/linux/updates/' FEDORAMODDEST = '/pub/fedora/linux/modular/updates/'

Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Dusty Mabe
LGTM On 08/27/2018 04:07 PM, Mohan Boddu wrote: > Dusty, how about this: > > diff --git a/roles/bodhi2/backend/files/new-updates-sync > b/roles/bodhi2/backend/files/new-updates-sync > index 9ddc591..194e2bd 100755 > --- a/roles/bodhi2/backend/files/new-updates-sync > +++

Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Mohan Boddu
Dusty, how about this: diff --git a/roles/bodhi2/backend/files/new-updates-sync b/roles/bodhi2/backend/files/new-updates-sync index 9ddc591..194e2bd 100755 --- a/roles/bodhi2/backend/files/new-updates-sync +++ b/roles/bodhi2/backend/files/new-updates-sync @@ -14,6 +14,7 @@ logger =

Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Dusty Mabe
mostly looks good a few nits inline On 08/27/2018 03:57 PM, Mohan Boddu wrote: > diff --git a/roles/bodhi2/backend/files/new-updates-sync > b/roles/bodhi2/backend/files/new-updates-sync > index 9ddc591..4537f0b 100755 > --- a/roles/bodhi2/backend/files/new-updates-sync > +++

Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Dusty Mabe
I think you talked to patrick about this and I'll trust him on whether this change will actually work or not. As far as the configs it looks ok to me except for one small typo (missing end brace): diff --git a/roles/bodhi2/backend/files/new-updates-sync

[Fedocal] Reminder meeting : Infra Office Hours

2018-08-27 Thread cverna
Dear all, You are kindly invited to the meeting: Infra Office Hours on 2018-08-28 from 18:00:00 to 19:00:00 UTC At fedora-ad...@irc.freenode.net The meeting will be about: Weekly hour dedicated to answer questions or help people with fixing tickets or implementing features. Source:

Re: [PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Kevin Fenzi
Seems odd to me to compose something in rawhide, then sync it here... but ok, it should work, I just think it will end up being confusing. :) kevin signature.asc Description: OpenPGP digital signature ___ infrastructure mailing list --

[PATCH] rawhide ostree sync as part of new-updates-sync script

2018-08-27 Thread Mohan Boddu
diff --git a/roles/bodhi2/backend/files/new-updates-sync b/roles/bodhi2/backend/files/new-updates-sync index 3bcfbea..1cb22c2 100755 --- a/roles/bodhi2/backend/files/new-updates-sync +++ b/roles/bodhi2/backend/files/new-updates-sync @@ -20,7 +20,23 @@ FEDORAALTDEST =

Re: [PATCH] robosig: rename atomic workstation to silverblue

2018-08-27 Thread Dusty Mabe
On 08/27/2018 11:39 AM, Mohan Boddu wrote: > LGTMĀ +1 > pushed ___ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org Fedora Code of Conduct:

Re: [PATCH] robosig: rename atomic workstation to silverblue

2018-08-27 Thread Mohan Boddu
LGTM +1 On Sun, Aug 26, 2018 at 10:50 PM Dusty Mabe wrote: > --- > roles/robosignatory/files/robosignatory.production.py | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/roles/robosignatory/files/robosignatory.production.py >