summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Merge "Remove fetching of networks that weren't created via nova-manage"Jenkins2012-01-301-9/+0
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Remove fetching of networks that weren't created via nova-manageBrad Hall2012-01-261-9/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For bug 922319: we want to make sure we only return networks created by nova-manage and not any arbitrary network that Quantum could have for a given tenant. Change-Id: Icd4a9c00d0ed4faf5e404060e8b7a658d9b83257
* | | | | | Merge "Remove session arguments from db.api"Jenkins2012-01-301-18/+12
|\ \ \ \ \ \
| * | | | | | Remove session arguments from db.apiMark Washenberger2012-01-271-18/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Possibly through copy-paste errors, some sqlalchemy session keyword args made it into the db api. Some aren't even passed in to the IMPL's equivalent function, and a db.api client has no sensible way of creating such a session to pass in. Therefore this patch removes them. Change-Id: I75237b542bdf4f75c1a07ae38bf485e0ff080339
* | | | | | | Merge "Refactor away the flags.DEFINE_* helpers"Jenkins2012-01-3079-1157/+1853
|\ \ \ \ \ \ \
| * | | | | | | Refactor away the flags.DEFINE_* helpersMark McLoughlin2012-01-2879-1157/+1853
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The next obvious step in porting to cfg is to define all options using cfg schemas directly rather than using the flags.DEFINE_* helpers. This is a large change, but it is almost entirely pure refactoring and does not result in any functional changes. The only change to note is that the default values for glance_host, glance_api_servers and default_publisher_id options are now using opt value interpolation i.e. -glance_host=_get_my_ip() +glance_host='$my_ip' -glance_api_servers=['%s:%d' % (FLAGS.glance_host, FLAGS.glance_port)] +glance_api_servers=['$glance_host:$glance_port'] -default_publisher_id=FLAGS.host +default_publisher_id='$host' Also note that the lower_bound check on the {report,periodic}_interval options are no more, but this has been true since cfg was first added. Change-Id: Ia58c8f0aaf61628bb55b1b8485118a2a9852ed17
* | | | | | | Merge "Raise 400 if bad kepair data is provided"Jenkins2012-01-304-1/+27
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | |
| * | | | | | Raise 400 if bad kepair data is providedBrian Waldon2012-01-284-1/+27
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 902395 Change-Id: If04a0bf432ad4828ac1fc68f6d33dff5f9f57e24
* | | | | | Merge "Add authorization checks to flavormanage extension"Jenkins2012-01-294-32/+11
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add authorization checks to flavormanage extensionBrian Waldon2012-01-254-32/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Relates to bp separate-nova-adminapi Change-Id: I69a851893b96ccbc2653716584bb3c3c0b0f3afa
* | | | | | Merge "Remove session arg from sm_backend_conf_update"Jenkins2012-01-271-1/+1
|\ \ \ \ \ \
| * | | | | | Remove session arg from sm_backend_conf_updateBrian Waldon2012-01-271-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 915108 Change-Id: Ic22bf5aa3d68ea59692ebe05d1b0b79757c70b52
* | | | | | Merge "fix `nova-manage image convert` exception"Jenkins2012-01-271-2/+1
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | fix `nova-manage image convert` exceptionPádraig Brady2012-01-271-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * bin/nova-manage (_convert_images): Don't convert the image id to a number as it's now a uuid. Change-Id: I4dfe5693d8f394fb9ee3f2c18c7e0b3f2524e931
* | | | | | Merge "Added validation of name when creating a new keypair"Jenkins2012-01-272-0/+29
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Added validation of name when creating a new keypairPhilip Knouff2012-01-272-0/+29
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug 900925 fixes bug 900927 Change-Id: Icef75af7dca299c566c366f41a71cdc49fa1af65
* / / / / Ignore case in policy role checks.Dan Prince2012-01-272-6/+14
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the default policy brain so that role checks ignore case. Fixes an issue where roles in keystone didn't exactly match the case of the role as specified in policy.json. Fixes LP Bug #922660. Change-Id: I05792755c9293e4dd80d642cb8eef6b0adda2ed4
* | | | Add a note explaining why unhandled exceptions shouldn't be returnedJohannes Erdfelt2012-01-261-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to users. For future developers so they don't re-introduce a bug by accident. Change-Id: I24f03adcc795db6252be988c076043bb74026733
* | | | uses the instance uuid in libvirt by introducing a new variable 'uuid' for ↵Christian Berendt2012-01-263-0/+18
| |/ / |/| | | | | | | | | | | | | | | | | | | | the used template instead of using a random uuid in libvirt fixes bug 917736 Change-Id: I1f8ba92878fb01cb25a7007839649e1ef7c4fcdc
* | | Merge "Fixing rebuilds on libvirt, seriously."Jenkins2012-01-261-9/+9
|\ \ \
| * | | Fixing rebuilds on libvirt, seriously.Naveed Massjouni2012-01-251-9/+9
| | | | | | | | | | | | | | | | | | | | bug: 867627 Change-Id: I497b18a56997e347995ad8869855127bd31cb974
* | | | Fixes bug 914418Monsyne Dragon2012-01-261-1/+1
| |_|/ |/| | | | | | | | | | | | | | | | | This turns out not to actually be a case of using the wrong exchange, rather it is that the exchange's 'durable' flag does not match the queue if FLAGS.durable is set to 'False'. Change-Id: I69ac7e84ac02a72dd204fa3aa27e73637594f509
* | | Merge changes If0c92dca,I75d8b616Jenkins2012-01-2613-10/+1008
|\ \ \ | | | | | | | | | | | | | | | | | | | | * changes: blueprint host-aggregates: OSAPI extensions blueprint host-aggregates: OSAPI/virt integration, via nova.compute.api
| * | | blueprint host-aggregates: OSAPI extensionsArmando Migliaccio2012-01-257-4/+619
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit introduces the OSAPI admin extensions for host aggregates. This is part of a series of commits that have started with change: https://review.openstack.org/#change,3035 Change-Id: If0c92dca7d6f5d794f84bdb2adc172c55dec6e74
| * | | blueprint host-aggregates: OSAPI/virt integration, via nova.compute.apiArmando Migliaccio2012-01-258-10/+393
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit introduces the first cut of integration between the OSAPI Admin extensions for host aggregates and the virt layer. This is part of a series of commits that have started with change: https://review.openstack.org/#change,3035 Change-Id: I75d8b616e3b8f8cef75d40d937e0dce9f29b16db
* | | | Merge "Remove LazySerializationMiddleware."Jenkins2012-01-266-57/+2
|\ \ \ \
| * | | | Remove LazySerializationMiddleware.Dan Prince2012-01-266-57/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes LazySerializationMiddleware and ExtensionMiddleware. They are no longer in use. Fixes LP Bug# 922152. Change-Id: Icd0b54d474d807125f522965de0d89741b14cf2f
* | | | | Merge "Bug #921730: plugins/xenserver/xenapi/etc/xapi.d/plugins/objectstore ↵Jenkins2012-01-263-261/+14
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | no longer in use"
| * | | | | Bug #921730: plugins/xenserver/xenapi/etc/xapi.d/plugins/objectstore no ↵Armando Migliaccio2012-01-263-261/+14
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | longer in use remove the plugin and update resources accordingly. Change-Id: Ib4184c9b81b40e6892886c71fd8de09ba55322af
* / / / / Adding live migration server actionsmasumotok2012-01-2612-59/+323
|/ / / / | | | | | | | | | | | | Change-Id: I5e1f5dddaf45d1c6eae8666647425bff748b639e
* | | | bug 921931: fix Quantum Manager VM launch race conditionDan Wendlandt2012-01-251-0/+1
| | | | | | | | | | | | | | | | Change-Id: I1f3a0f001477caa5f595c93c49f6c9e03a50149a
* | | | Fix authorization checks for simple_usage.showAnthony Young2012-01-255-29/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Normal users shouls be allowed to query their own usage info * Fixes bug 921327 * Address bcwaldon's comment about using a default {} in authorize * Remove is_admin references * Rebase and change expected auth failure response from 401 to 403 * Remove policy-related tests * Add back test_verify_show_cant_view_other_tenant, implemented with test policy Change-Id: Ib0ce46419b7aedad34de957bfe2e60b10c5af11c
* | | | Merge "Don't pass filter_properites to managers"Jenkins2012-01-262-1/+12
|\ \ \ \
| * | | | Don't pass filter_properites to managersChris Behrens2012-01-252-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 921789 distributed_scheduler is trying to pass filter_properties to compute manager but it's not serializable.. and we shouldn't pass it anyway. Change-Id: I17ce04b71dfa3cef77a6883ce3cd6fede95a538a
* | | | | Merge "Simplify somewhat complicated reduce() into sum()"Jenkins2012-01-261-4/+1
|\ \ \ \ \
| * | | | | Simplify somewhat complicated reduce() into sum()Johannes Erdfelt2012-01-251-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib9c742e391061de033f24b385b047b4a4316ead9
* | | | | | Ignore connection_type when no instances existBrian Waldon2012-01-251-4/+13
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In migration 74, we had required that the connection_type flag be set. That's annoying for new deployments, so bypass this check if there are no instances in the databse. Fixes bug 921294 Change-Id: I9b829e80ad7fa7ded3c7a471cb68c9b342d973bb
* | | | | Merge "Add HACKING compliance testing to run_test.sh"Jenkins2012-01-252-0/+207
|\ \ \ \ \
| * | | | | Add HACKING compliance testing to run_test.shJoe Gordon2012-01-242-0/+207
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tests so far: N101 TODO format N201 Except format N301 One import per line N302 import only modules N303 Invalid Import N304 Relative Import Change-Id: I33c021b842e7199b1f5f1f699ea17f7fa5f8ca49
* | | | | | Merge "Fix uptime calculation in simple_usage"Jenkins2012-01-252-1/+4
|\ \ \ \ \ \
| * | | | | | Fix uptime calculation in simple_usageAnthony Young2012-01-252-1/+4
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixes bug 921807 Change-Id: Ie6460adb95ef3b9cb0473023816c255aac707e99
* | | | | | Merge "Do not output admin_password in debug logs"Jenkins2012-01-251-1/+4
|\ \ \ \ \ \
| * | | | | | Do not output admin_password in debug logsThierry Carrez2012-01-241-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sanitize run_instance's admin_password argument from nova.rpc 'received' debug logging. Fixes bug 915025. Change-Id: I9004dee422a9e5411b8e440ab80030849d137dab
* | | | | | | Merge "rootwrap: Fix KillFilter matching"Jenkins2012-01-251-4/+5
|\ \ \ \ \ \ \
| * | | | | | | rootwrap: Fix KillFilter matchingAdam Gandelman2012-01-251-4/+5
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The match() method in the KillFilter is a bit buggy. Since most/many run through this logic in their way to being matched, it creates subtle issues elsewhere, specifically during linux_net bridge creation. This fixes two issues. 1. userargs gets elements directly popped off of it. If the KillFilter does not match the command its checking, the later filter that does is missing one argument. 2. Type error on pid when checking /proc, this was causing issues on instance tear down / kill -HUP'ing dnsmasq. Fixes bug 921784 Change-Id: I058ff2276e3154e8c1f6cc7077fa485db60e5827
* | | | | | | Merge "Fixes bug 921265 - i'nova-manage flavor create|list'"Jenkins2012-01-252-40/+24
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Fixes bug 921265 - i'nova-manage flavor create|list'Dean Troyer2012-01-252-40/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These problems were introduced in https://review.openstack.org/2918 * nova-manage needs the change from local_gb to root_gb/ephemeral_gb. * fixes 'nova-manage flavor ...' * fixes 'nova-manage service describe_resource ...' Change-Id: I63040359a425aac3ef7a0a84e6834b1dfe3a8790
* | | | | | | Merge "Fix bug 921715 - 'nova x509-create-cert' fails"Jenkins2012-01-251-4/+4
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Fix bug 921715 - 'nova x509-create-cert' failsDean Troyer2012-01-251-4/+4
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'CN' value passed to openssl was too long, shorten it. Change-Id: I9da6510bf23a8ad33b08f3373f09e0c5287fe051
* | | | | | Merge "Remove unused flags.Help*Flag"Jenkins2012-01-255-31/+0
|\ \ \ \ \ \