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@xxxxxxxxxxxxxxxxxxxxxxx
> To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxxxxxxxxxx
>
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx
