Commit Graph

274 Commits

Author SHA1 Message Date
OpenDev Sysadmins e0b2865c18 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:51:42 +00:00
Zuul 6a69a7b22d Merge "Change how we retrieve host wwpns from block_device_mapping" 2018-02-09 11:00:34 +00:00
Andreas Scheuring 45b9e903ac Change how we retrieve host wwpns from block_device_mapping
Nova commit [1] has changed the format of the block_device_mapping.
We need to adapt our way how we get the list of host wwpns.

[1] 0258cecaca

Change-Id: I852aeeb57c891c8bd1458c83feae05f58046eb8a
2018-02-09 11:36:46 +01:00
Andreas Scheuring 538030a03d Fail if instance is connected to a VLAN network
VLAN networking is not supported by nova-dpm. This patch
introduces a new exception that is thrown if somebody tries
to connect a DPM instance to a VLAN network.

Change-Id: I00e10d33040fed62a446b8a1df8eb9ae534a7a43
2018-01-24 10:35:07 +01:00
Andreas Scheuring 07055fe19e Throw specific InvalidVIFTypeError on attach_nic
When a invalid vif_type was given, we used to throw a generic
exception. This patch introduces a new specific exception
InvalidVIFTypeError.

Change-Id: I225e1ccb6fead59b26a011b0ad02b28983751f87
2018-01-24 10:07:31 +01:00
Andreas Scheuring d0b5290a8f Add attach_nics method to PartitionInstance
We now can add all NICs with a single command!
This is also usefull once we might introduce ssc support which
requires additional verifications cross all nics.

Change-Id: Ic78686b662c97d67d6376e8b8592aa20887026f8
2018-01-23 15:53:30 +01:00
Andreas Scheuring ae261d2405 Fix confusing between block_device_mapping and _info
Patches [1][2] confused block_device_info (bdi) with
block_device_mapping (bdm). We always need the bdm. Nova_dpm's spawn
method now transforms the passed in bdi into a bdm.

[1] 112c4cda58
[2] 255d62a20b

Change-Id: Id3ec0b1861bedab76db42233d49752a277892c77
2018-01-19 17:23:09 +01:00
Andreas Scheuring bc04ba1279 prep_for_spawn: remove unused params and make private
This patch modifies the method prep_for_spawn
* Making it private -> _prep_for_spawn
* Removing unused parameters

Change-Id: Ie2e2259badef350082b150b7989a93e85c09e9db
2018-01-19 09:57:11 +01:00
Andreas Scheuring f05c21544a Add unittest for _get_partition_instance
Change-Id: Id652de00ae66a6e347665a8160a2d996580deb04
2018-01-19 09:04:19 +01:00
Andreas Scheuring a8d040c276 Remove redundant image check
In prep_for_spawn we checked if the an image was provided
during launch. If so we aborted. This check is redundant, as we already
check this in _get_partition_instance.

The Unittests are still correct!

Change-Id: Idfee4d6dd80043c9a468b42c7b33dc7669a2529d
2018-01-19 09:04:07 +01:00
Andreas Scheuring 255d62a20b set_boot_properties does NOT take the bd anymore
The block_device is now gathered from block_device_info which
is now available as class variable.

Change-Id: Icebdd6ed76a1384e11019f79e0c9d0f20162a005
2018-01-19 07:32:21 +00:00
Andreas Scheuring 112c4cda58 Pass in block_device_info into PartitionInstance
The partitionInstance constructor now also takes the
block_device_info as optional kwarg.

Change-Id: Ibbfcceb671ea069e65729aed88aa2cd90e20b0d5
2018-01-19 07:32:13 +00:00
Zuul 6c83181d89 Merge "Add context to PartitionInstance" 2018-01-19 07:27:38 +00:00
Andreas Scheuring 6d78828226 Add context to PartitionInstance
For some actions (e.g. downloading an image) we need the context
to be present in PartitionIstance. This patch introduces
the context property in this class. In addition context is set
during launch.

It is not required during other operations for now.

Change-Id: I01b7bee02e79c9b141731a88f7afb77bed7ae1cc
2018-01-19 07:01:23 +00:00
OpenStack Proposal Bot 48a1671ae4 Updated from global requirements
Change-Id: Ie8dde1037e0e83309ba2d4d13691a5e3a3990105
2018-01-18 03:12:09 +00:00
Andreas Scheuring 2548d56f95 Create nic string for os-specific-params in PartitionInst class
Till now the String was generated and set from the driver class.
This patch moves both actions into the PartitionInstance class.

This allows us overwriting this method once we support another
partition type (e.g. ssc).

Change-Id: If535cbdcdf878f5e9e2f91d0f0d1128021efcbbc
2018-01-16 13:15:33 +01:00
Andreas Scheuring d4a3e6a58b Change how we set the boot-os-specific-params for a nic
For each nic we need to add some data to the partitions
boot-os-specific-parameters property. Till now we were accumulating
everything to a single string which set the property once. The new
approach is to update the property for each nic.

This leads to more read/write accesses on this partition property. But
on the other hand we're gaining flexibility. E.g. someone might want to
add a NIC after spawn. Or some other data should be added. Those
scenarios were not supported in the past.

This is also helpful to move this code into the partitioninstance
class. In the future this action might not be required for all type of
partitions (e.g. not for ssc).

Change-Id: I0262f2660b3e89b455bb1b1ece016342c8347c80
2018-01-16 13:14:19 +01:00
Andreas Scheuring 7b038bc10a Extract new method get_nic_properties_dict
We extract the pieces that generate the nic dict from the attach_nic
method into a separate method. This allows overwriting the creation
of the dict once we support for example ssc.

Change-Id: I17642d78b62eb71b0d89bec1fb6cd552b11f6fd8
2018-01-16 10:32:17 +01:00
Andreas Scheuring 443a1bd547 Transition to DPMVIF object
The previous patch has introduced the DPMVIF object. We now make
use of it in our code.

Change-Id: Ifbfcb391ec619127fd7de7a3aa88f4f5868ed28c
2018-01-16 10:12:53 +01:00
Andreas Scheuring 3e57711742 Introduce DPMVIF object
The DPMVIF object represents a VIF (virtual interface for networking).
Nova provides us a large dictionary. This object provides a better method
to access properties of a vif.

Change-Id: I1e6c408c244330d9e59b988aecacce82e2c07b8e
2018-01-16 10:03:25 +01:00
Andreas Scheuring eba768acca Move creation of boot properties into vm class
We build up all the properties required for boot now in the
vm class. The driver does not need to deal with gathering
information for boot anymore.

Change-Id: Iad15a7a404a3467d417f27e739a3e4dc7233af6f
2018-01-16 10:03:08 +01:00
Andreas Scheuring 8aef196ad4 Introduce Blockdevice object
This object should simplify the usage of the block_device dictionary
provided by nova. Attributes can now easily be accesed via object
properties and method, instead of doing complex operations on the dict.

This patch does not yet exploit this object. This is done in a follow
up patch.

Change-Id: I4b88761c86dc8996fb490cf8680ceaf1480d445d
2018-01-16 10:03:01 +01:00
Andreas Scheuring bce9106265 Set_boot_properties retrieves boot-hba-uri on its own
The set_boot_hba uri method can determine the boot-hba-uri on its own
as this information is available in the object instance.

Change-Id: I39f15db7865457a7e91452e7a024bc4944a5317b
2018-01-15 16:01:20 +01:00
Andreas Scheuring a9c2c15fdd Retrieve boot_hba uri from hba object
In some cases we need the hba object to retrieve the wwpn of
the hba used for boot, in some the element-uri. We had 2 different
ways of gathering this information.

This patch transforms the get_boot_hba_uri method into a more generic
get_boot_hba method. From this object it's easy to retrieve the
required properties 'element-uri' and 'wwpn'.

Change-Id: If518a915408caf5c59d4425b8aa2e0673acbcc19
2018-01-15 16:00:35 +01:00
Andreas Scheuring a0c7b2bf6e Retrieve PartitionInstance via getter method
In some methods we directly created an instance of the PartitionInstnace object.
The instantiation of this object is now moved into a separate method. This is
even more usefull when we introduce ssc support in the future. In that case
we could simply introduce a SSCPartitionInstance class. The newly created method
would be the only place where the decission between ssc and pure dpm is done.

Change-Id: I06e6d8ee84b7f308f2cdbf120c08e46afd4dd8be
2018-01-15 15:58:16 +01:00
OpenStack Proposal Bot 0e24d6b6cd Updated from global requirements
Change-Id: I337ff468e9d5546d781b041b1aa68a26fb525d31
2017-11-29 09:08:10 +00:00
OpenStack Proposal Bot dc83645894 Updated from global requirements
Change-Id: Id71fab970b0eb42c421dc40b339184657566b478
2017-11-16 11:16:28 +00:00
Sreeram Vancheeswaran 1ba30f87af Fix the breaking change to ComputeDriver.spawn in nova-dpm
Following updates are made to the signature of spawn method and its
corresponding references in test_driver.py.
	- add "allocations" kw arg as per [1]
	- remove "flavor" kw arg  as per [2]

[1] http://lists.openstack.org/pipermail/openstack-dev/2017-October/123738.htm
[2] 8749bda96e

Closes-Bug: 1724744

Change-Id: I6fd6974a0b32bbcc520a2fdd2c948367f3e8ae5f
Signed-off-by: Sreeram Vancheeswaran <sreeram.vancheeswaran@in.ibm.com>
2017-11-06 19:27:25 +05:30
Vu Cong Tuan a2ab076fd2 Remove SCREEN_LOGDIR from devstack
SCREEN_LOGDIR has already been deprecated [1]
[1] https://review.openstack.org/#/c/499186

Change-Id: If010be03207fb13414d5f7199676bd7b92f358f8
2017-10-05 16:52:14 +07:00
Sreeram Vancheeswaran bc6a8828b6 Get release notes back on nova-dpm RTD
Release Notes section was returning empty for nova-dpm in RTD.
This patch fixes the same.

Closes-Bug: 1718103

Change-Id: Iac1832331b50f1b971c686be77d83a60f1bd53cd
Signed-off-by: Sreeram Vancheeswaran <sreeram.vancheeswaran@in.ibm.com>
2017-09-18 22:57:27 -07:00
OpenStack Proposal Bot 121f34f9ce Updated from global requirements
Change-Id: I7773be565335e89f6ba15d94b48fccf0254eefb1
2017-09-16 23:18:40 +00:00
Jenkins 802dc7f80a Merge "Remove loop_status_update function" 2017-09-11 19:30:23 +00:00
Prabhat Ranjan 0b23d509e0 Remove loop_status_update function
Remove loop_status_update function and use
wait_for_status which is provided by zhmcclient

Closes-Bug: #1672979

Change-Id: I9b0d4edbfd9ccf44bae38afceff6200501511807
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-09-08 13:47:54 +05:30
Jenkins c3e60f32c8 Merge "Update the documentation link for doc migration" 2017-08-30 13:14:31 +00:00
Jenkins 2b513d6859 Merge "Docs: Getting started page and storage chapter" 2017-08-28 07:48:10 +00:00
Jenkins 4a5a13c5dc Merge "rhel7 image docs: Add installation of growpart" 2017-08-28 07:47:13 +00:00
OpenStack Proposal Bot 8e0e94a95a Updated from global requirements
Change-Id: I4d4b348a977d5ab78e8ac2a5fcbff5f943507f24
2017-08-18 11:36:39 +00:00
Jenkins 2b53cd3f33 Merge "If HMC is up and CPC is down" 2017-08-04 07:08:55 +00:00
Jenkins 25d9590853 Merge "PowerOn of a partition in paused state not working" 2017-08-04 07:08:49 +00:00
Prabhat Ranjan 8749bda96e No SQL parameter establish error
It seems nova removed 'instance.instance_type_id'
by which we were getting flavor.
Now instance has 'instance.flavor'.
Direct access of flavor.

Closes-Bug: #1707628

Change-Id: I128d8adb0754c5b5313fc3e5ed7c521afa0df508
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-08-01 19:04:59 +05:30
OpenStack Proposal Bot 4af5c380ec Updated from global requirements
Change-Id: I076aa4eb844d8062473268304e6491e1d84ed014
2017-07-28 12:55:04 +00:00
Andreas Scheuring 62bda6e9a6 Update local.conf
With the current local.conf it is not possible to use floating-ips
to access an instance.

This patch extends and refactors the local.conf, that devstack creates
2 networks. A public network (used for floating ip) - public, and a
private network where the instances get attached to - provider.

In addition the ovs is configured to be able to deal with both networks.
However one manual step is required. The bridge br-provider needs to be
created manually before stack.sh is run.

Change-Id: I776552c41161d53d33ad9c6c750f47ef6d5ccce6
2017-07-27 14:59:29 +00:00
Sreeram Vancheeswaran 1acca0b2a1 Remove openstackdocstheme
As per [1] removed the openstackdocstheme

[1] -http://lists.openstack.org/pipermail/openstack-dev/2017-July/119566.html

Change-Id: I4f643ff91f44b6b1f264318dbe52f6e5296df878
Signed-off-by: Sreeram Vancheeswaran <sreeram.vancheeswaran@in.ibm.com>
2017-07-27 11:17:13 +05:30
Prabhat Ranjan 4e94168fd3 "tox -e cover" command failing with error
ImportError: No module named wsgi_intercept

Closes-Bug: #1706522

Change-Id: Id4c00c537b02873c5431d5367a1094e445d27433
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-07-26 13:51:22 +05:30
Prabhat Ranjan fb4abd53d3 PowerOn of a partition in paused state not working
When we do inband shutdown of partition using
command 'shutdown -P now'
Then partition is not going to 'stopped' state.
It is going to 'PAUSED' state.

In DPM there is no such implementation so that we
can start partition from PAUSED state. So workaround
is we need to stop the partition then start.

This patch also includes the cases when partition is
in "starting/stopping" state.

Closes-Bug: #1678002

Change-Id: I1898b85a8441c3175db0a6de6d620bca19af69ec
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-07-25 15:29:30 +05:30
Prabhat Ranjan 2c17a2f354 cpc_object_id is breaking in unit test
After new version of os-dpm-1.1.0 cpc_object_id
is breaking in unit test

Change-Id: Ic17ff50549756728c1213a946d98b0fe2203e813
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-07-25 14:25:21 +05:30
Hangdong Zhang 9f87d9ed04 Update the documentation link for doc migration
Change-Id: I268b5dc4b1a079e6ca26a439541e7a2ea74456af
2017-07-21 14:05:27 +08:00
Jenkins c69f6d63d3 Merge "Modified reserve_host_memory config parameter" 2017-07-20 07:23:13 +00:00
sreeteja 443f0ae228 Modified reserve_host_memory config parameter
This is in connection with the minimum value assigned to the max-memory
parameter in local.conf under dpm section.The default value of
reserve_host_memory is 512Mb by which it blocks 512Mb of memory on the
host this means to launch an instance on a host 1024 is the minimum
memory that is needed as 512Mb is being reserved inorder to mitigate
this issue the reserved memory is made 0.

Change-Id: I05ee4275cd5d80b1edb614d94fe0b038551bc3cc
2017-07-20 09:03:22 +02:00
Prabhat Ranjan 472aa01199 If HMC is up and CPC is down
If HMC is up and CPC is down then the 'ifl-processor' and
'storage-customer' values are 0.

Closes-Bug: #1684102

Change-Id: If7c10ca66c4e47c873cc9c7277a715dee3040ad5
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-07-14 16:41:26 +05:30