| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| | |_|_|/ / /
|/| | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fixes LP Bug #969545
Change-Id: I60d4300aa04597e2d8b0eea31ab0303b0b3c48f9
|
| |\ \ \ \ \ \ |
|
| | | |_|_|_|/
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fixes lp984219
Change-Id: I06b113355acbc954052734ecb33a281649ec5876
|
| |\ \ \ \ \ \ |
|
| | | |/ / / /
| |/| | | |
| | | | | |
| | | | | | |
Change-Id: Ied2595b56da7e0a5b17d17282c2aa8dc066ef9f3
|
| |\ \ \ \ \ \ |
|
| | | |/ / / /
| |/| | | |
| | | | | |
| | | | | | |
Change-Id: I80926766c1cc8d41a9fe35a625018022e732140a
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
|
| | | |/ / /
| |/| | |
| | | | |
| | | | | |
Change-Id: I0a30c61db57d4704eb001f1d96127acc41878bfe
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: I8ab738f3042ccf451d769c62b9fee2d20fa23953
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: I3456b4c98aa6527d10452e8f578a30e95a65db8e
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The size of `_create_instance` is making it increasingly difficult to
work with. This is part of a series of patches aims to break it up into
smaller, more digestible pieces.
Change-Id: Ic668f0414937b255b1b1e1392c204307665822aa
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
They were separated only to make it easier to stubout, but we want
more test coverage, not less. So merge them together and fix the
tests while we're at it.
Change-Id: I18333017868b91fb71b8d3dce831463a1cd5bf81
|
| | |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Full instance objects are passed to all methods now, so it no longer
needs to handle three different variants of argument. Also, make sure
most code uses _get_vm_opaque_ref instead of calling VMHelper.lookup
directly so it checks that a vm_ref was actually returned.
Change-Id: Icc8b819e1d4b8bf342867d09e61fe05e6eab9e70
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The only unrescue test that happens right now is that it fails if not
already in rescue. This test fills out to test unrescue works as
intended. Also, remove unnecessary stub from rescue test.
Change-Id: I4451afe5a7d2ada8dfaefad804345b66ac895094
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
For debugging purposes, it's useful to see what event type is being sent
and not just where it's being sent
Change-Id: I809a15a76204c79d37ed68b649cd2aea80ad3db2
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
VMOps is only used by connection.py, but it's not always obvious
which methods are intended to be public and which are used
internally for implementation reasons. Prefix any internal methods
with _ to make that clearer.
Change-Id: I50244c18233d6d7463057cf5df42d5a5a3a6fab9
|
| |/ /
| |
| |
| |
| |
| | |
clarify its purpose.
Change-Id: I7bcfbd1a0545f992b1deae54e4deb956ae33e15d
|
| |/
|
|
|
|
|
| |
This keeps all of the XenAPI related code together in one directory,
like libvirt.
Change-Id: I1de51eb8bcead0695e9f8ede466416e0257d0148
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: Ifb199d644ce47c5a222f1f1bfc22a14a42f75686
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Nova should not depend on python-novaclient. Remove it.
Note: In order to get unit tests to pass I had to add in
a temporary dependency on httplib2. This is due to the fact
that pip installing glance does not properly setup all
the dependencies. See LP Bug #975483 for details.
Fixes LP Bug #978992.
Change-Id: I50c00139f9dc303223601230fd962abac977cd5e
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
fix bug #971640
Change-Id: I48efc5babdd9b233342a33c87c461aabf5f5915b
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add instance logging coverage to some methods in
nova/compute/manager.py
Change-Id: Ib1979d2bcc316cc5642c84b15af3b8ce89c3311e
|
| |\ \ \ \ |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: I1eac99495dc2ed1b23ce12377aa93534ef4eb527
|
| |\ \ \ \ \ |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
tl;dr - this is a cleaner fix for bug #949038
It seems clear to me that all of the DB APIs should not explicitly pass
read_deleted='no' or read_deleted='yes' to model_query without good
reason.
We want to allow callers to specify read_deleted via the context and
that only works if we don't explicitly pass it to model_query().
If we don't explicitly specify it to model_query(), we use the value
from the context which defaults to 'no'.
Given all that, there is no need to support read_deleted to any of the
DB API calls because they should support specifying the flag via the
context. There should also be no need to pass read_deleted='no' because
that is the default.
Really, the only place there should be any mention of read_deleted is
where we want read_deleted='yes' behaviour e.g.
- In tests, where we want to check the operational_state of an
aggregate after it has been deleted
- Where we want to support undeleting an aggregate or aggreate host
Change-Id: I916a8d189a33d7f30838cccb26531a024066ef96
|
| |\ \ \ \ \ \ |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
fixes bug 971552
If the string format operation in NovaException raises an exception be sure to
log the kwargs at log level error
Change-Id: Idfc54b8c05256bd7c849c86dc45f0b493727fe58
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | | |
baremetal_allow_project_net_traffic"
|
| | | |_|_|_|/ /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
baremetal_injected_network_template
baremetal_uri
baremetal_allow_project_net_traffic
Change-Id: I9766924cc7621e14fc52088a558b95cfd5d7ce8d
|
| |\ \ \ \ \ \ \ |
|
| | | |_|_|/ / /
| |/| | | | |
| | | | | | |
| | | | | | | |
Change-Id: I5ce1b4b7032f6f82dfaa7a8f60139e148a250db8
|
| |\ \ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* this is a separate commit to make backporting easier
Change-Id: I10ff04567809a2cdfbea8882f3f2c19571a60a40
|
| |\| | | | | | | |
|
| | | |_|_|_|/ /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* Makes sure the uuid is returned as id if it exists
* Simplifies db get for manager.get_networks
* Removes direct db access from manager which was breaking test
* Updates tests to verify the new logic
* Makes sure Remote NotFounds are turned into 404s
(The RemoteError blocks can be removed once
https://review.openstack.org/5749 lands)
* Fixes bug 977712
* Fixes bug 977723
Change-Id: I6aa815960782c7ae5165aeebd83bdaaa62c19b04
|
| |\ \ \ \ \ \ \
| |/ / / / / /
|/| | | | | | |
|
| | | |/ / / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
(nova): TRACE: File "/usr/lib64/python2.6/site-packages/nova/utils.py",
line 860, in __init__
(nova): TRACE: self.tname = "-%x-%x" % (ident & 0xffffffff, gident &
0xffffffff)
(nova): TRACE: TypeError: unsupported operand type(s) for &: 'NoneType'
and 'int'
Fixes: bug 971629
Change-Id: I3d34816c36a11cd6fc2dd9c889c93c5872133a2c
|
| |\ \ \ \ \ \ |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Remove dead code from nova/wsgi.py in Server around
the irrelevant start_tcp method.
Change-Id: I3222d90f6208bb3ff38d79f56365b22e551f2029
|
| |\ \ \ \ \ \ \
| |/ / / / / /
|/| | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
This change uses json to serialize an exception so that it can be sent
through RPC calls to be reconstructed on the other side. The traceback is added
to the exception message. If recreating the exception fails for whatever reason
then a RemoteError is created containing all of the exception information.
Adds flag 'allowed_rpc_exception_modules' to prevent dangerous modules from
being accessed and allowing arbitrary code to be run.
Fixes bug 920705
Fixes bug 940500
Change-Id: Ife3b64b19fe8abbc730184d4ee7d9fcabfd29db3
|
| |\ \ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fixes some "its" and "it's" mistakes, and unifies the tense of
some documentation.
Change-Id: I9b83aba535b554e80b2cf1e048bb3a4e5cac0e5e
|