906 Commits

Author SHA1 Message Date
Jenkins
39b6e00eed Merge "Fixes the output of 'os-evacuate' command." 2013-03-01 23:51:43 +00:00
Jenkins
413bff3e11 Merge "Accept 201 status code on POST" 2013-03-01 23:51:09 +00:00
Jenkins
29a652c60d Merge "Fix how tests were failing due to missing attributes." 2013-03-01 23:50:42 +00:00
Jenkins
4e2c829135 Merge "A minimum of Python3 fixes so that installation works without errors/warnings." 2013-03-01 23:50:39 +00:00
Jenkins
58a780c95a Merge "Allows admins to reset-network of an instance" 2013-03-01 23:48:41 +00:00
Jenkins
929e6d8913 Merge "Added limit to image-list in a preparatory step toward addressing bug 1001345." 2013-03-01 23:01:14 +00:00
Vasyl Khomenko
9ab600fb80 Make ip_protocol parameter in security groups rules case insensitive
Added convert ip_protocol to uppercase.
Added tests

Fixes: bug #1052725

Change-Id: I966ddad0800556b1cc848003d07d4a897ce1b9c1
2013-02-26 04:15:59 -05:00
Alex Glikson
feaff79919 Fixes the output of 'os-evacuate' command.
Change-Id: I5f02e66f316acdd10df42bec3f6628a8efe5fd97
Fixes: bug #1132790
2013-02-25 06:35:27 +02:00
Rami Vaknin
aac5767894 Update the docstring of cloudpipe-configure command
Change-Id: Iab820200664618769f11b2d21a3731ef6b81a877
2013-02-23 16:24:13 +02:00
Gordon Chung
e043bd1560 Accept 201 status code on POST
On POST requests, keystone should return '201 Created' according to
api docs.

Related to but not dependent on:
https://bugs.launchpad.net/keystone/+bug/1131119

Change-Id: I04c398ad7946f5ddcfd2059e4f8d97608e5d0ed3
2013-02-22 14:31:03 -05:00
Joshua Harlow
8be810b59c Fix how tests were failing due to missing attributes.
Fixes: bug #1131845

For unknown reasons all tests continued to pass even after
this change was introduced, besides that being a very odd thing
to have happen it is better to fix said attributes for those
that are running the tests individually in there own CI pipelines.

Change-Id: Iee6acc1d4a528afb840459ca47d754ec7bae8f32
2013-02-22 10:20:40 -08:00
Davanum Srinivas
34c8a6bb7a Missing import for gnomekeyring
follow on to previous attempt to fix to bug 1116302,
looks like we missed an import

Fixes LP#1116302

Change-Id: If56e3cedaa63a594907bb851a2701bd64806ed85
2013-02-21 11:41:02 -05:00
Thomas Schreiber
68e6af73ba A minimum of Python3 fixes so that installation works without errors/warnings.
Fixes bug: 1130937
Change-Id: I740652fcd5804fc1c120fc409afdf4693c8e5781
2013-02-20 23:17:49 +01:00
Sulochan Acharya
25ae8f0770 Allows admins to reset-network of an instance
Adding some functionality to allow admins to call on
resetNetwork action. Fixes bug #1130766

Change-Id: I3d0945b115d565bd2c54a5526ba51036e941a28f
2013-02-20 10:55:28 -06:00
Dan Smith
5c8caf06e9 Add support for os-attach-interfaces
This is dependent on nova change:

 I4f8f677af58afcb928379e5cf859388d1da45d51

Related to blueprint network-adapter-hotplug

Change-Id: Ieef603e85c6557cbfd2fe4ae7109e6ca235ba51d
2013-02-19 16:02:51 -05:00
Anita Kuno
8811ced007 Added limit to image-list in a preparatory step toward addressing bug 1001345.
Currently novaclient doesn't use the limit or marker params.
As a step to addressing bug 1001345 which requires pagination,
this patch introduces the use of limit as an option
passed to the image-list function.

Change-Id: Ia32f9e923b4eb9bcdde3b7bc1722c59d7791d104
2013-02-18 20:14:53 +00:00
Vasyl Khomenko
f35635be47 Extend test coverage (shell, fping)
Added unit tests for shell code, that wasn't covered before.

Implements: bp/python-novaclient-unittests
Change-Id: I76e3df05d53e6c08e224feb30eb90ae819d8cdce
2013-02-18 06:48:50 -05:00
Jenkins
85785309fa Merge "Support showing extra fields in server list" 2013-02-15 23:43:22 +00:00
Jenkins
23d45611b9 Merge "Avoid doing a deep copy on the availability zone manager" 2013-02-15 23:41:49 +00:00
Jenkins
76bacaa645 Merge "Allow extensions to provide a name when discovered on the python path." 2013-02-15 23:41:42 +00:00
Jenkins
28653e8550 Merge "Expand and improve baremetal API." 2013-02-15 23:40:34 +00:00
Jenkins
7ca20df4fa Merge "Fix nova availability-zone-list for admin users" 2013-02-15 23:40:31 +00:00
Jenkins
60fd879f9b Merge "Make availability_zone in aggregate_create optional" 2013-02-15 23:40:25 +00:00
Davanum Srinivas
132465231a Issue when gnomekeyring is present but not the current backend
Issue was identified in the review for the previous
patch for bug 1116302. Took this chance to rename _IOError
to a better name (KeyringIOError)

Change-Id: I321353d519eaebea27617702f92ecafe2052eb8e
2013-02-15 10:11:51 -05:00
David Scannell
430ef7071e Avoid doing a deep copy on the availability zone manager
When novaclient is in debug mode then the HTTPClient will
configure its logger with a StreamHandler that holds an
instance of thread.lock, which cannot be copied. As a result
running novaclient with the --debug flag will cause errors
and mask real issues being debugged.

Fixes bug 1123561

Change-Id: Idf19d62ff3e5b02b029f9089f403a697164231ac
2013-02-14 15:56:57 -05:00
David Scannell
c0e85a84b0 Allow extensions to provide a name when discovered on the python path.
Using novaclient with some extensions via python code, you might have an
invocation like this:

    extensions = shell.OpenStackComputeShell()._discover_extensions("1.1")
    novaclient = Client("1.1", user, apikey, project, authurl, extensions=extensions,
                        endpoint_type=shell.DEFAULT_NOVA_ENDPOINT_TYPE,
                        service_type=shell.DEFAULT_NOVA_SERVICE_TYPE)

If you have an extension like 'myextension.py' in the v1_1/contrib directory, you'll
end up with a very sensible attribute on the resulting novaclient object, i.e.

    novaclient.myextension

If you have a package distributed in the package myextension_python_novaclient_ext,
then it'll automatically be picked up as an extension (awesome!) but the name is not
as intuitive.

    novaclient.myextension_python_novaclient_ext

This patch simply changes this to allow the Extension to provide a name for itself.
The possibility of collisions exists, but is not really any more significant than
before (where you might have different versions of the same package installed in the
system or heck, even a bizarrely named 'myextension_python_novaclient_ext.py' in the
contrib/ directory).

Fixes bug 1058366

Change-Id: Ie68463ffd7a939744e035b20fd50a7dc8da605de
2013-02-14 13:22:25 -05:00
Davanum Srinivas
d1d4f33aca Fix IOError with gnomekeyring.find_network_password_sync
find_network_password_sync throws a gnomekeyring.IOError
when a non-root user tries to run nova client
from a ssh console. If we don't catch this exception nova client
throws the traceback (shown in the bug report) and stops.
If we catch this exception (just like we catch ValueError),
and return None, Nova client executes just fine.

Fixes LP# 1116302

Change-Id: If6937b3f8eafb1dc55224b2ca2bd0f93ae07f8c6
2013-02-14 11:58:08 -05:00
Chris Jones
a926216808 Expand and improve baremetal API.
This adds an additional command to list the interfaces associated with a
baremetal node.

It also fixes some docstrings and renames the existing interface
commands to be more consistent with the node commands.

Change-Id: Ia6ae383d76adb1c9d632bf69ec22438f1412c66f
2013-02-13 09:14:49 +00:00
Jenkins
67b3db2bbc Merge "Corrects 2nd argument type for logging" 2013-02-13 02:03:55 +00:00
Joe Gordon
c97879a3f2 Fix nova availability-zone-list for admin users
Don't need to convert returned timestamp

Fix bug 1123381

Change-Id: I1dbc82475b44b8d321fffb512083fc7a3b13153d
2013-02-12 17:30:28 -08:00
Joe Gordon
9f0cf6a547 Make availability_zone in aggregate_create optional
Aggregates do not require an availability zone any more, but in order to
keep the current API, allow for availability zone to be set to None (the new
default value)

Fixes bug 1123468

Change-Id: I216c4fc808a91b0a5f602ee02ae1bca46adb73f4
2013-02-12 23:00:30 +00:00
Victoria Martínez de la Cruz
2b81789f1a Corrects 2nd argument type for logging
Fixes a regression introduced in commit
c73afa9fd1df14bff9186c1e73ac8f3593ef81db.

This completely breaks Horizon in last
DevStack version.

Change-Id: Ib754c6ea325534399a34ff76a290475ac652fea9
Fixes: bug #1122958
2013-02-12 12:05:41 -03:00
Mitsuhiko Yamazaki
0933c64251 Add format options to 'nova coverage-report'.
This adds --html and --xml options to 'nova coverage-report'
to enable selection of coverage report format.
If specifying none of these, text-formatted report will be created.

This also adds --combine option to 'nova coverage-start' to make sure
that 'nova coverage-report --html' works fine.

Fixes bug 1114766

Change-Id: I9fee26bd5c45cac35f425ac7abbced4e2f3ff4df
2013-02-12 10:22:37 +09:00
Jenkins
0db2ff24c6 Merge "Fix run_tests.sh --coverage" 2013-02-11 20:25:52 +00:00
Jenkins
320e2ffc00 Merge "Mask permissions on private key files" 2013-02-11 17:33:20 +00:00
Sascha Peilicke
c73afa9fd1 Update to requests >= 0.8
The requests module dropped all configuration with the 1.0.0 release.
There's no danger_mode and no 'verbose'' mode. The former
shouldn't be necessary anymore and the latter can be done by setting
a different log handler for the request.logging root logger.

Change-Id: Iec169ef6e39097814cdbf1b777bc0590236692ba
2013-02-11 12:49:28 +01:00
Zane Bitter
0b4590cb24 Mask permissions on private key files
When using "nova x509-create-cert", the private key should be written to
a file with the permissions 0400, not (world-readable) 0644, in line
with how ssh private keys are treated.

bug 1112605

Change-Id: I0b20378efba38fa58f4ad9a33cd15b3432ebb8a2
Signed-off-by: Zane Bitter <zbitter@redhat.com>
2013-02-11 12:22:37 +01:00
Kieran Spear
2eeab72aee Fix run_tests.sh --coverage
Looks like a couple of typos crept in here preventing coverage
generation.

Change-Id: I08a61a67dadc5dcd334f9f288d8ee6719d469ca1
2013-02-11 11:22:47 +11:00
Kieran Spear
e800196860 Support showing extra fields in server list
Adds a --fields argument that sets the fields to display.
ID is always displayed.

Fixes bug #1076473

Change-Id: If3462e6a490ea16da4834a7f40f96b111c9e8227
2013-02-11 11:19:42 +11:00
Jenkins
1ea7e65d85 Merge "management_url not set by authenticate method" 2013-02-10 02:35:42 +00:00
Jenkins
dae03378c7 Merge "Update .coveragerc" 2013-02-10 02:35:40 +00:00
Sean McCully
17b251d8b0 management_url not set by authenticate method
Fixes Bug 1109243.
If management_url not set, raise UnAuthorized exception,
otherwise url chaining methods fail and cause NoneType
exceptions.

Change-Id: I19e87a5dcfcf93b4fa1d423bd99de352679fa16d
2013-02-08 15:25:25 -06:00
Alessio Ababilov
3724528db7 Update .coveragerc
Set up proper source and omit options.

Change-Id: I966256a3a3bd37b438e98e7bcf1402482c42b567
Implements: blueprint update-coveragerc
2013-02-06 16:47:06 +02:00
Jenkins
e389e8a6b4 Merge "Fix default format of 'nova coverage-report'" 2013-02-05 18:58:06 +00:00
Zhi Yan Liu
4516efa8db Show the summary or details of all availability zones of a region
Adding "availability-zone-list" sub-command allow user list all
availability zones and its status in a region.
* summary list for normal user: list availability zones summary whitin a
flat view.
* details list for administrator user: list availability zones details
within a tree view, include zones, hosts and components.

Implement one workitem for bp:show-availability-zone

Change-Id: Id87fe470c7e0f6fbfb9465551f63717724b5fc18
2013-02-05 20:31:50 +08:00
Jenkins
ef6aaa88f8 Merge "Live migration with an auto selection of dest." 2013-02-05 04:32:31 +00:00
Jenkins
f905b6bd5b Merge "Add help about the id 'auto' for flavor-create" 2013-02-05 04:32:28 +00:00
Joe Gordon
96630b8248 Upgrade to pep8 1.3.3
Also expand scope of what is covered by pep8 test

Change-Id: Ifc8924914b5a0d625bc8df6442ee85eb21459cde
2013-02-01 12:24:19 -08:00
Anita Kuno
3ca976d4a0 Fixed 7 pep8 errors.
Change-Id: I2d509ed383fd8d6cef91dd0c58a51eafc6995bda
2013-02-01 15:35:21 +00:00
Svetlana Shturm
7a95186047 Live migration with an auto selection of dest.
Change-Id: I2e0880b0e5b0565e138ad7f72d88ccd7e431e79f
Blueprint: live-migration-scheduling
DocImpact:
2013-02-01 13:32:43 +00:00