Commit Graph

324 Commits

Author SHA1 Message Date
OpenDev Sysadmins 798ebab490 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:50:54 +00:00
AlexMuresan fc62f89fc3 Raise proper exception in case of vif plug errors
The Nova Compute manager expects a VirtualInterfacePlugException
exception to be raised when a vif plug fails.

This particularly affects the service initialization step. If we're
raising an exception other than the above mentioned one, the exception
won't be cought, in which case the Nova Compute service will stop.

This change ensures that we're raising the expected exception if vif
plugging fails.

(cherry picked from commit 6adc5cf4d4)

Change-Id: Ibaa405577d4fa06f27fdd9440aca49be0c620af5
Closes-bug: #1736392
2018-04-03 11:53:38 +00:00
Jenkins 1641b53fc2 Merge "Add live migration timeout for clustered instances" into stable/ocata 2017-06-20 08:40:31 +00:00
Lucian Petrut 0c8037d166 Add live migration timeout for clustered instances
os-win now allows setting a timeout when migrating clustered
instances. When that timeout is hit, pending migrations will be
canceled.

For the moment, we're only going to use this timeout value when
live migrating clustered instances.

The default timeout is 300s, adjustable through a config option.

Conflicts:
	hyperv/nova/conf.py

NOTE(claudiub): the config options have been refactored on master.

Change-Id: I21a7a8edd763f0203b58dc58fb30b7a495ec04cd
(cherry picked from commit 067f602612)
2017-06-19 22:57:33 +00:00
OpenStack Proposal Bot 08242dba3b Updated from global requirements
Change-Id: I6696c3ba158e6357178f41a4f48c54cf9dbe7985
2017-06-19 06:17:28 +00:00
Jenkins b5566ac4c9 Merge "Fix inconsistent hostnames" into stable/ocata 2017-06-14 14:43:57 +00:00
Lucian Petrut 2391aa47e4 Fix inconsistent hostnames
This change ensures that the hostname is used consistently throughout
the Hyper-V driver.

This fixes a VM port binding issue in case of failovers.

Conflicts:
        hyperv/nova/cluster/clusterops.py

Change-Id: Ic1c9644721dad451fb68131bdc5f9b3397d5e267
Closes-Bug: #1694238
(cherry picked from commit 2205dc79e0)
2017-06-12 09:05:09 +00:00
Alin Balutoiu 8d174c7140 Fixes timeout waiting for vif plugging callback for instance
Currently, plug_vifs is called when powering on the instance.
The call to wait_for_plug_events will give a timeout waiting
for the plug event when using neutron-ovs-agent.

This patch addresses this issue by moving the call to plug_vifs
from power_on to after the create_instance call, which is prior
to waiting for plug events. This change modifies the behaviour
of power_on only during the instance spawn, the other calls
remaining the same.

Change-Id: I13b7caf84434a4b9b6f2795cddc7401874c46647
Closes-Bug: #1684250
(cherry picked from commit ae76684e07)
2017-06-12 08:54:33 +00:00
Jenkins c11cbd0ad4 Merge "[Trivial] Hyper-V: accept Glance vhdx images" into stable/ocata 2017-06-09 12:44:53 +00:00
Claudiu Belu 6dd7b10592 tox: use nova stable/ocata
nova stable/ocata should be used for the ocata branch, otherwise
the unit tests are failing.

Change-Id: If7495d769f8588252f21bfd2b16da75059ca3129
2017-06-08 12:04:14 +03:00
Lucian Petrut 93dc6f791b [Trivial] Hyper-V: accept Glance vhdx images
Although the Hyper-V driver is used with VHDX images most of the
time, it rejects Glance images marked as VHDX.

Note that for legacy reasons, people usually just mark them as vhd
at he moment, which is misleading and unsafe.

This change ensures that such images are handled properly.

Closes-Bug: #1693470

Change-Id: I8cf4339cdf17721c257cf7aa5597594aa2d1c53f
(cherry picked from commit 387c53cccc)
2017-05-29 10:00:08 +00:00
Jenkins 7f925601e6 Merge "hyperv: Fixes Generation 2 VMs volume boot order" into stable/ocata 2017-05-02 10:54:36 +00:00
Jenkins 3c70042cb3 Merge "Improve cluster event listener" into stable/ocata 2017-05-02 10:42:05 +00:00
Claudiu Belu 75972a942e hyperv: Fixes Generation 2 VMs volume boot order
The patch that introduced setting boot order to the nova
Hyper-V driver [1] was using a method that no longer existed
(Generation 2 VMs only). That method was renamed by the patch
that refactored volumeops (to use os-brick) [2].

[1] d68c04299aa2c04aea16e881d93076236cc64d7b
[2] 758a32f7cef6c675b35c04dd8d276c918be188dd

Change-Id: Ib374199a972ed1ab4342e08c68d1713051a76517
Closes-Bug: #1687570
2017-05-02 10:26:25 +00:00
Jenkins cef16d8db8 Merge "Ensure instance files are deleted when using multiple CSVs" into stable/ocata 2017-04-25 16:04:03 +00:00
Lucian Petrut 47488df1bf Ensure instance files are deleted when using multiple CSVs
When using multiple CSVs, we rely on the instance itself when
locating its files.

The issue is that we're fetching the instance files location after
destroying the instance, for which reason we're leaking instance
files.

Closes-Bug: #1633556

Change-Id: Ic09ad9e8346a69b74d6f930ee0756f511d0c3ac6
(cherry picked from commit 90a08935ed)
2017-04-25 00:11:34 -07:00
Lucian Petrut 58043ab9be Fix live migration log message formatting
At the moment, whenever we live migrate an instance, we'll get a
trace in the logs because of a log formatting exception.

This change fixes this trivial issue.

Change-Id: Ie8119bfe90690fd4a13e03162963b25696f3d565
(cherry picked from commit 820fa8ca5e)
2017-04-25 06:55:00 +00:00
Lucian Petrut 1d9e7bb65b Improve cluster event listener
At the moment, we handle each failover event sequentially, also
waiting a fixed amount between them.

This change ensures we do this concurrently and that we don't wait
between events.

Change-Id: I225bb63050c7f044d1c7562bcdb2640da4321273
(cherry picked from commit 9f0ea777a7)
2017-04-25 06:44:14 +00:00
OpenStack Proposal Bot b5ef934c00 Updated from global requirements
Change-Id: Ib015d3ba2cbd4b2bf0ef282c2041eb91e38760fb
2017-03-09 18:12:33 +00:00
OpenStack Proposal Bot daeae74879 Updated from global requirements
Change-Id: Ifcabe61ae57f698eb0f9871c8a238978faa4e7f2
2017-02-20 22:25:48 +00:00
Claudiu Belu 65c1d85fbe Updates .gitreview to point to stable/ocata
Change-Id: I65b4eaffaf429376d13a1f278bb89b9109a68da3
2017-02-20 07:23:35 -08:00
Jenkins 16a5b57ba9 Merge "hyperv: make sure to plug OVS VIFs after resize/migrate" 2017-02-17 15:05:59 +00:00
Claudiu Belu a333a3d389 pdk: replaces missing raised exception
nova.exception.MissingParameter has been removed, which is causing
the unit tests to fail.

Replaces the raised MissingParameter exception with InstanceUnacceptable.

Change-Id: I09a498a3eb898fdb908e134450adef8aeb3863b5
2017-02-17 12:24:08 +02:00
Claudiu Belu 5eb0db9b36 hyperv: make sure to plug OVS VIFs after resize/migrate
Since the network_info wasn't getting passed to vmops.power_on
after a resize/cold migrate confirmation, the OVS VIFs weren't
getting plugged.

(cherry-picked from commit 568a682e5efa6e4fd49b69e02cbd80f1d8a96c63)

Change-Id: I9097debeb68cb25dec27e7b3e979cf51d951c250
Closes-Bug: #1657424
2017-02-16 15:11:11 +02:00
Jenkins 8226eb34b1 Merge "Add automatic host shutdown instance action option" 2017-02-14 11:27:46 +00:00
OpenStack Proposal Bot fc4e80c7a2 Updated from global requirements
Change-Id: Iab6545e3a22b666ce261b7f3a864bdd8d7c6a81c
2017-02-10 05:47:42 +00:00
Lucian Petrut e834135f93 Add automatic host shutdown instance action option
At the moment, the instances will be automatically saved whenever
the host is shut off. This adds a disk overhead which may not be
desired.

For this reason, this patch makes it configurable. By turning on
'instance_automatic_shutdown', instances will be turned off instead
of being saved when the host is turned off.

Note that existing VMs will not be affected by this config option.
For this reason, whenever an admin will change it, he has to make
sure that the existing VMs are updated.

Instance disk overhead will be calculated based on the assumption
that all the instances are configured the same way.

Depends-On: I68490a4e257eef88bdd07f3ec14d345a2df79a4a

Change-Id: I46c1497fe1863b09c5f41ddd0cbd6a23e9e4828c
2017-02-09 14:51:32 +02:00
Jenkins da73f59389 Merge "refactors vmops' create_instance" 2017-02-07 12:31:04 +00:00
Jenkins 571d36e2e9 Merge "refactors cold migration / resize" 2017-02-07 12:30:54 +00:00
Claudiu Belu 6eb9fdc9e3 refactors vmops' create_instance
Splits the create_instance method into two, separating the
"updateable" resources from the method. This new method will be
called from migrationops in the following patch, when performing a resize.

Change-Id: I94277e8f1a58ecc1529f3cf92deac3596f5b65d9
2017-02-06 20:24:46 +02:00
Adelina Tuvenie d3d59920b9 refactors cold migration / resize
Simplifies the cold resize / migration process by making the
destination node pull the VM files from the source instead of
vice-versa. Doing so, the logic is simplified greatly, the
destination node only has to copy the files, regardless if it
is on shared storage or not.

Solves the migration issue when the compute nodes have different
instances_path config option set, as the destination node will
copy the files to its configured instances_path folder.

Co-Authored-By: Claudiu Belu <cbelu@cloudbasesolutions.com>

Change-Id: Ia44850675e8cb8eb21017e1c4321802574dfe6a0
2017-02-03 14:31:05 +02:00
Jenkins ddc422da86 Merge "Pass-in empty array uuid filter to server list" 2017-02-01 12:08:04 +00:00
Jenkins d57d92c970 Merge "Hyper-V PCI Passthrough" 2017-01-30 12:01:55 +00:00
Claudiu Belu 79c4b36315 Pass-in empty array uuid filter to server list
If there are no VMs running on the compute node, the given instance_uuids
will be an empty array. This should be added as a filter, otherwise, the
entire list of instances will be returned.

Change-Id: I9a77eeb410f59feda5ee84429eae8895ae3f3c26
Closes-Bug: #1660293
2017-01-30 13:09:52 +02:00
Iulia Toader 702ec6a842 Hyper-V PCI Passthrough
Discrete Device Assignment is a new feature in Windows Server 2016,
offering users the possibility of taking some of the PCI Express
devices in their systems and pass them through directly to a guest VM.

Co-Authored-By: Claudiu Belu <cbelu@cloudbasesolutions.com>

Depends-On: I8e7782d3e1e9f8e92406604f05504a7754ffa3c2

Implements: blueprint hyper-v-pci-passthrough

Change-Id: Ica395a982da1ffa4ead41601124d192a838ea1ad
2017-01-30 10:58:37 +02:00
Claudiu Belu 7635722514 fixes wrong usage of vmops post_start_vifs
The post_start_vifs method has been renamed to plug_vifs in a
previous patch, but no all the references have been updated.

Change-Id: Ic70197e4a186202a5f50fd6548fc354f07bc7076
2017-01-26 12:46:32 +02:00
Jenkins 11b1bdbd5e Merge "resync and cleanup" 2017-01-23 14:45:40 +00:00
Claudiu Belu 588b84cade resync and cleanup
Solve some minor differences between nova and compute-hyperv,
and cleanup some unit tests.

Change-Id: Id2971cdf582d9681d6e40d27940e0a145a325d91
2017-01-23 14:13:05 +02:00
Jenkins 34f44412ad Merge "Updated from global requirements" 2017-01-23 10:20:05 +00:00
Jenkins 57bb553927 Merge "resync: hyper-v: wait for neutron vif plug events" 2017-01-23 10:19:54 +00:00
Jenkins 59adc9faed Merge "resync: Adds Hyper-V OVS ViF driver" 2017-01-23 10:19:49 +00:00
Jenkins 7b11b18967 Merge "resync: Add support for setting boot order in Hyper-V" 2017-01-23 10:19:25 +00:00
Jenkins 1cf71c8314 Merge "resync: Hyper-V: Adds vNUMA implementation" 2017-01-23 10:17:38 +00:00
Claudiu Belu c2e8cc605c resync: hyper-v: wait for neutron vif plug events
(cherry-picked from commit 00fdf284085866c77590255059ae74b672dd5a85)

Change-Id: Icba6f86a5ec1d303da6dd37f8d0acefd29570b1d
2017-01-20 15:18:30 +02:00
Claudiu Belu da4c9ad33d resync: Adds Hyper-V OVS ViF driver
(cherry-picked from commit 07b6580a1648a860eefb5a949cb443c2a335a89a)

Change-Id: I24dc8b80ae686a5945ab67175ba8f6ca9b5a099f
2017-01-20 15:15:12 +02:00
Claudiu Belu 4ac4ef8759 resync: Add support for setting boot order in Hyper-V
(cherry-picked from commit d68c04299aa2c04aea16e881d93076236cc64d7b)

Change-Id: Ifbb57e064062f1ae9926875c7d4e727bf0f258f9
2017-01-20 13:39:00 +02:00
Claudiu Belu 1ea0ea8b1b resync: Hyper-V: Adds vNUMA implementation
(cherry-picked from commit 2195e4d68486ed70e55d0b5f038b13bd35e3271c)

Change-Id: Ic4e03e641356ea54206ce8fdaecb2837824d2747
2017-01-20 13:36:24 +02:00
OpenStack Proposal Bot 64effbaebc Updated from global requirements
Change-Id: I6d26f32bcda48682e8f1689d017b50a523bd34a4
2017-01-20 00:10:53 +00:00
OpenStack Proposal Bot fb3a3cfec3 Updated from global requirements
Change-Id: I28db975508699bb4780fa22baab507acce46e2fa
2017-01-12 15:03:26 +00:00
Claudiu Belu a32c6b404a HyperVClusterDriver: reclaim failovered instances on start
The driver will now detect and reclaim any failovered instances
that appeared on the host while the nova-compute service was down.

Closes-Bug: #1646809

Change-Id: Id7b053f9debcb869efb4440cfaf6a3b66b067b8c
2017-01-11 14:33:19 +00:00