[PATCH] new-updates-sync: sync multiarch silverblue ostree content
--- 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']}, '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']}, 'to': [{'arches': ['x86_64', 'aarch64', 'armhfp', 'source'], 'dest': os.path.join(FEDORADEST, 'testing', '31', 'Everything')}, {'arches': ['ppc64le', 's390x'],
+1 LGTM
On Thu, Oct 24, 2019 at 3:57 PM Dusty Mabe dusty@dustymabe.com 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']}, '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']}, 'to': [{'arches': ['x86_64', 'aarch64', 'armhfp', 'source'], 'dest': os.path.join(FEDORADEST, 'testing', '31', 'Everything')}, {'arches': ['ppc64le', 's390x'],-- 2.20.1 _______________________________________________ infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@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.fedorapro...
+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'],
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@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@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.fedorapro...
infrastructure@lists.fedoraproject.org
-
Dusty Mabe -
Mohan Boddu -
Rick Elrod