euca2ools parsing of config file is severely limited
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
euca2ools |
Fix Released
|
Undecided
|
Unassigned | ||
euca2ools (Ubuntu) |
Fix Released
|
Low
|
Dustin Kirkland | ||
Lucid |
Fix Released
|
Low
|
Dustin Kirkland |
Bug Description
Binary package hint: euca2ools
The euca2ools reading of user or system config file is very limited.
The reason for this is that the config file is bash syntax (expected to be sourced by bash in .bash_profile or such), but the parsing is a limited parsing done with python and string replacements (see euca2ools/
~ line 360
| if read_config:
| lines = eucarc.readlines()
| comment = re.compile('^#')
| for line in lines:
| line = line.strip(
| line = line.replace('\'', '')
| line = line.strip()
| line = line.replace(
| if not comment.
| parts = line.split('=', 1)
| if len(parts) == 2:
| self.environ[
The following easily demonstrates the issue:
$ mkdir euca && cd euca
$ sudo euca_conf --get-credentials mycreds.zip
$ unzip mycreds.zip
$ ln -s $PWD/eucarc ~/.eucarc
$ euca-bundle-image --image mycreds.zip
Invalid cert
$ egrep "(EUCA_
EUCA_KEY_
export EC2_CERT=
If bash is the syntax for the config file, then it makes most to use bash to parse the config file, rather than limited python as above.
ProblemType: Bug
Architecture: amd64
Date: Tue Feb 23 12:50:57 2010
DistroRelease: Ubuntu 10.04
InstallationMedia: Error: [Errno 13] Permission denied: '/var/log/
Package: euca2ools 1.2-0ubuntu1 [modified: usr/share/
PackageArchitec
ProcEnviron:
LANG=en_US.UTF-8
SHELL=/bin/bash
ProcVersionSign
SourcePackage: euca2ools
Uname: Linux 2.6.32-14-server x86_64
Related branches
- Eucalyptus Maintainers: Pending requested
-
Diff: 1182 lines (+160/-154)43 files modifiedbin/euca-add-group (+3/-3)
bin/euca-add-keypair (+3/-3)
bin/euca-allocate-address (+3/-3)
bin/euca-associate-address (+3/-3)
bin/euca-attach-volume (+3/-3)
bin/euca-authorize (+4/-2)
bin/euca-bundle-image (+3/-3)
bin/euca-bundle-vol (+3/-3)
bin/euca-confirm-product-instance (+3/-3)
bin/euca-create-snapshot (+3/-3)
bin/euca-create-volume (+3/-2)
bin/euca-delete-bundle (+3/-3)
bin/euca-delete-group (+3/-3)
bin/euca-delete-keypair (+3/-3)
bin/euca-delete-snapshot (+3/-3)
bin/euca-delete-volume (+3/-3)
bin/euca-deregister (+3/-3)
bin/euca-describe-addresses (+3/-3)
bin/euca-describe-availability-zones (+3/-3)
bin/euca-describe-groups (+3/-3)
bin/euca-describe-image-attribute (+3/-3)
bin/euca-describe-images (+3/-2)
bin/euca-describe-instances (+3/-3)
bin/euca-describe-keypairs (+3/-3)
bin/euca-describe-regions (+3/-3)
bin/euca-describe-snapshots (+3/-3)
bin/euca-describe-volumes (+3/-3)
bin/euca-detach-volume (+3/-3)
bin/euca-disassociate-address (+3/-3)
bin/euca-download-bundle (+3/-3)
bin/euca-get-console-output (+3/-3)
bin/euca-modify-image-attribute (+3/-3)
bin/euca-reboot-instances (+3/-3)
bin/euca-register (+3/-3)
bin/euca-release-address (+3/-3)
bin/euca-reset-image-attribute (+3/-3)
bin/euca-revoke (+3/-3)
bin/euca-run-instances (+3/-3)
bin/euca-terminate-instances (+3/-3)
bin/euca-unbundle (+3/-3)
bin/euca-upload-bundle (+3/-3)
bin/euca-version (+3/-3)
euca2ools/euca2ools/__init__.py (+33/-31)
Changed in euca2ools (Ubuntu Lucid): | |
milestone: | none → ubuntu-10.04-beta-1 |
tags: | added: patch |
Changed in euca2ools (Ubuntu Lucid): | |
status: | New → Confirmed |
milestone: | ubuntu-10.04-beta-1 → none |
Changed in euca2ools (Ubuntu Lucid): | |
assignee: | nobody → Dustin Kirkland (kirkland) |
status: | Confirmed → In Progress |
tags: | added: patch-accepted-upstream |
affects: | eucalyptus → euca2ools |
Changed in euca2ools: | |
status: | Fix Committed → Fix Released |
Attaching suggested fix.