BZ#759149 System templates with repo names that have spaces fail to push to cloud provider

Chris Alfonso calfonso at redhat.com
Mon Dec 5 20:41:53 UTC 2011


bump
On 01/12/11 10:03 -0500, Chris Alfonso wrote:
>On 01/12/11 09:43 -0500, Chris Alfonso wrote:
>>When a repository has a client certificate associated with it and the repo name has a space in it,
>>the scp file copy fails.  I've converted the spaces to underscores for the file and directory structure.
>>---
>>oz/RedHat.py |    4 ++--
>>1 files changed, 2 insertions(+), 2 deletions(-)
>>
>>diff --git a/oz/RedHat.py b/oz/RedHat.py
>>index a2fd845..48a4483 100644
>>--- a/oz/RedHat.py
>>+++ b/oz/RedHat.py
>>@@ -962,7 +962,7 @@ class RedHatCDYumGuest(RedHatCDGuest):
>>            # Create the local copies of any needed SSL files
>>            def _create_certfiles(repo, cert, fileext, propname):
>>                certdict[propname] = {}
>>-                filename = "%s-%s" % (repo.name, fileext)
>>+                filename = "%s-%s" % (repo.name.replace(" ", "_"), fileext)
>>                localname = os.path.join(self.icicle_tmp, filename)
>>                certdict[propname]["localname"] = localname
>>                f = open(localname, 'w')
>>@@ -1013,7 +1013,7 @@ class RedHatCDYumGuest(RedHatCDGuest):
>>            if not host and not guest:
>>                raise oz.OzException.OzException("Could not reach repository %s from the host or the guest, aborting" % (repo.url))
>>
>>-            filename = repo.name + ".repo"
>>+            filename = repo.name.replace(" ", "_") + ".repo"
>>            localname = os.path.join(self.icicle_tmp, filename)
>>            f = open(localname, 'w')
>>            f.write("[%s]\n" % repo.name)
>>-- 
>>1.7.6.4
>>
>
>I did it again.. This patch applies to
>
>https://github.com/imcleod/oz.git


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/aeolus-devel/attachments/20111205/1a5ec441/attachment.sig>


More information about the aeolus-devel mailing list