#295 Koji API expects white-space separated list.
Merged 9 months ago by jkaluza. Opened 9 months ago by jkaluza.
jkaluza/freshmaker lb-arches  into  master

file modified
+1 -1

@@ -322,7 +322,7 @@ 

          arches = [conf.manifest_v2_arch_map.get(arch, arch) for arch in arches]

  

          # Finally, return the list, joined.

-         return ','.join(arches)

+         return ' '.join(arches)

  

      @region.cache_on_arguments()

      def _get_additional_data_from_distgit(self, repository, branch, commit):

file modified
+1 -1

@@ -1695,7 +1695,7 @@ 

              ]

          }

          result = image._get_architectures_from_registry("foo", self.build)

-         self.assertEqual(result, 'x86_64,s390x,ppc64le')

+         self.assertEqual(result, 'x86_64 s390x ppc64le')

          requests.get.assert_called_once_with(

              'http://blue-pulp-smocker01.sledmat.com:8888/v2/devtools/'

              'rust-toolset-7-rhel7/manifests/'

The Koji client accepts also ',' as separator, but for API, we need to use white-space.

rebased onto 12dc294

9 months ago

Pull-Request has been merged by jkaluza

9 months ago