Re: FBR: Fix new-updates-sync typo

2018-04-26 Thread Kevin Fenzi
+1

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


Re: FBR: Fix new-updates-sync typo

2018-04-26 Thread Mohan Boddu
LGTM +1

On Thu, Apr 26, 2018 at 6:57 PM Patrick Uiterwijk 
wrote:

> Hi,
>
> Can I get +1s for the following fix to my last version of the script?
>
> Patrick
>
>
> diff --git a/roles/bodhi2/backend/files/new-updates-sync
> b/roles/bodhi2/backend/files/new-updates-sync
> index b90c89384..7cfe15e5c 100755
> --- a/roles/bodhi2/backend/files/new-updates-sync
> +++ b/roles/bodhi2/backend/files/new-updates-sync
> @@ -170,7 +170,8 @@ def sync_ostree(dst, ref):
>  if src_commit == dst_commit:
>  logger.info('OSTree at %s, ref %s in sync', dst, ref)
>  else:
> -print('Syncing ostree ref %s -> %s' % (ref, src_commit,
> dst_commit))
> +print('Syncing ostree ref %s: %s -> %s'
> +  % (ref, src_commit, dst_commit))
>  logger.info('Syncing OSTree to %s, ref %s: %s -> %s',
>  dst, ref, src_commit, dst_commit)
>  cmd = ['ostree', 'pull-local', '--verbose', '--repo',
> ___
> infrastructure mailing list -- infrastructure@lists.fedoraproject.org
> To unsubscribe send an email to
> infrastructure-le...@lists.fedoraproject.org
>
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org


FBR: Fix new-updates-sync typo

2018-04-26 Thread Patrick Uiterwijk
Hi,

Can I get +1s for the following fix to my last version of the script?

Patrick


diff --git a/roles/bodhi2/backend/files/new-updates-sync
b/roles/bodhi2/backend/files/new-updates-sync
index b90c89384..7cfe15e5c 100755
--- a/roles/bodhi2/backend/files/new-updates-sync
+++ b/roles/bodhi2/backend/files/new-updates-sync
@@ -170,7 +170,8 @@ def sync_ostree(dst, ref):
 if src_commit == dst_commit:
 logger.info('OSTree at %s, ref %s in sync', dst, ref)
 else:
-print('Syncing ostree ref %s -> %s' % (ref, src_commit, dst_commit))
+print('Syncing ostree ref %s: %s -> %s'
+  % (ref, src_commit, dst_commit))
 logger.info('Syncing OSTree to %s, ref %s: %s -> %s',
 dst, ref, src_commit, dst_commit)
 cmd = ['ostree', 'pull-local', '--verbose', '--repo',
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org