Thanks! Fixed the syntax error and pushed!
On 10/24/19 5:06 PM, Rick Elrod wrote:
> +1 after the syntax fix I mentioned inline.
>
> -re
>
> On 2019-10-24 15:56, Dusty Mabe wrote:
>> ---
>> roles/bodhi2/backend/files/new-updates-sync | 10 ++++++----
>> 1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/roles/bodhi2/backend/files/new-updates-sync
b/roles/bodhi2/backend/files/new-updates-sync
>> index d08c89301..9ad7628b8 100755
>> --- a/roles/bodhi2/backend/files/new-updates-sync
>> +++ b/roles/bodhi2/backend/files/new-updates-sync
>> @@ -25,8 +25,9 @@ RELEASES = {'f31': {'topic': 'fedora',
>> 'modules': ['fedora',
'fedora-secondary'],
>> 'repos': {'updates': {
>> 'from': 'f31-updates',
>> - 'ostrees': [{'ref':
'fedora/31/x86_64/updates/silverblue',
>> - 'dest': OSTREEDEST}],
>> + 'ostrees': [{'ref':
'fedora/31/%(arch)s/updates/silverblue',
>> + 'dest': OSTREEDEST,
>> + 'arches': ['x86_64',
'ppc64le', 'aarch64']},
>
> I think this is missing another ] at the end before the comma (to match
> with the [ after 'ostrees':
>
>> 'to': [{'arches': ['x86_64',
'armhfp', 'aarch64', 'source'],
>> 'dest': os.path.join(FEDORADEST,
'31', 'Everything')},
>> {'arches': ['ppc64le',
's390x'],
>> @@ -34,8 +35,9 @@ RELEASES = {'f31': {'topic': 'fedora',
>> ]},
>> 'updates-testing': {
>> 'from': 'f31-updates-testing',
>> - 'ostrees': [{'ref':
'fedora/31/x86_64/testing/silverblue',
>> - 'dest': OSTREEDEST}],
>> + 'ostrees': [{'ref':
'fedora/31/%(arch)s/testing/silverblue',
>> + 'dest': OSTREEDEST,
>> + 'arches': ['x86_64',
'ppc64le', 'aarch64']},
>
> Same here.
>
>> 'to': [{'arches': ['x86_64',
'aarch64', 'armhfp', 'source'],
>> 'dest': os.path.join(FEDORADEST,
'testing', '31', 'Everything')},
>> {'arches': ['ppc64le',
's390x'],
>>
> _______________________________________________
> infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
> To unsubscribe send an email to infrastructure-leave(a)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.fedora...
>