Commit Graph

236 Commits

Author SHA1 Message Date
OpenDev Sysadmins 6e4c497bac 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
Sreeram Vancheeswaran 9d3a1ec291 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-19 06:01:08 +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
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
Jenkins 8c7c087574 Merge "Instance delete always stops partition" 2017-07-14 10:12:23 +00:00
Prabhat Ranjan 4c23e0b0bc Instance delete always stops partition
Instance delete always stops partition even if partition ia already stoped.
Work around is when it try to stop the partition which
is already stopped, then catch exception and pass it.

Closes-Bug: #1701506

Change-Id: I8ddb84de307bc58fa68136c503408a2d65ef629a
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-07-14 14:29:20 +05:30
Jenkins 9eceb53519 Merge "Removal of max_instances config parameter" 2017-07-14 08:42:25 +00:00
sreeteja b51945a05f Removal of max_instances config parameter
There is a NumInstancesFilter which can be used to filter the number of
instances that are supposed to be launched on the host this replaces the
work that is planned for max_instances config parameter to perform.
It is removed as the main functionality of this parameter is not yet
implemented in the code.

Change-Id: Ie545a5f74f7b63ea56812577c201fcb443014998
2017-07-12 14:15:56 +05:30
Prabhat Ranjan dcf6bb6954 Several unit test cases are breaking with zhmcclient==0.14.0
Closes-Bug: #1703578

Change-Id: Ied1981d0df1e85422693547c32fd212b749a2387
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-07-11 18:23:14 +05:30
Jenkins 21916e7095 Merge "Remove usage of parameter enforce_type" 2017-07-06 07:10:17 +00:00
Jenkins efdcc182a5 Merge "Min value of max_memory is been added" 2017-07-04 11:33:03 +00:00
sreeteja 37e2ca4215 Min value of max_memory is been added
512mb is the minimum memory that is required to launch a partition and
rounding off starts only from value greater that 512mb this decides the
min value needs to be set to the config parameter max_memory.

closes-Bug: 1689267

Change-Id: I493fec40b412feb5303ab7ca9ee2f78c9b9220db
2017-07-04 11:19:04 +00:00
OpenStack Proposal Bot e92e284fa3 Updated from global requirements
Change-Id: I1acfb289c9b2b6d0ae6ffa691d81a984c1e16d1b
2017-06-27 12:16:53 +00:00
Luong Anh Tuan bb24705cea Replace assertTrue(isinstance()) with assertIsInstance()
Some of tests use different method of assertTrue(isinstance(A, B)) or
assertEqual(type(A), B). The correct way is to use assertIsInstance(A,
B) provided by testtools.

Change-Id: I3e3917db4f62cc713e0edcef81f2d4d8bb867067
Closes-Bug: #1268480
2017-06-05 16:41:30 +07:00
Vu Cong Tuan e74ca0b7e9 Remove usage of parameter enforce_type
Oslo.config deprecated parameter enforce_type and change its default
value to True in Ifa552de0a994e40388cbc9f7dbaa55700ca276b0. Remove the
usage of it to avoid DeprecationWarning: "Using the 'enforce_type'
argument is deprecated in version '4.0' and will be removed in version
'5.0': The argument enforce_type has changed its default value to True
and then will be removed completely."

Change-Id: I807ee7b4cc622d8833802c7812f4ee03fc2d40a7
Closes-Bug: #1694616
2017-06-02 16:44:43 +07:00
Jenkins 5c5b2e0af6 Merge "Added default value to the max_instances parameter of dpm.py" 2017-05-30 08:42:07 +00:00
sreeteja dbe45cf03f Added default value to the max_instances parameter of dpm.py
The default value is set to -1, which means that there is no artificial
upper bound for the number of instances on this CPC. The number of
instances is only limited by the number of (free) partitions on this
CPC.
closes-Bug: 1683318

Change-Id: I80f8b36fa21cef313e0b5a821a26877afd1af980
2017-05-29 11:20:40 +00:00
Jenkins 18b7600994 Merge "Updated from global requirements" 2017-05-17 10:58:18 +00:00
Jenkins 959e00e459 Merge "fix up release notes build" 2017-05-17 07:01:55 +00:00
Doug Hellmann 5cbac25ce5 fix up release notes build
Created a place holder for release notes files in the right location
in the source tree and include them in the documentation build with
a history file.

Related-Bug: #1661319
Change-Id: I451ee6d0876bc987d2d066fdd95efca2bda9261e
2017-05-17 12:11:45 +05:30
OpenStack Proposal Bot 51a2ee97f5 Updated from global requirements
Change-Id: Ief6e2f2b57bf4394c13aca66cc44d4be34ae4152
2017-05-17 03:53:56 +00:00
Jenkins cdb4adf1a8 Merge "Cleanup spec folder" 2017-05-15 13:40:06 +00:00
Prabhat Ranjan 9b2dccbed8 Multi storage adapter support broken
The initator-target map, we're not necessarily picking
the entry for our boot adapter, but just the first one
that is returned by get_partition_wwpns. But in fact
we must not take the first one, but we must take the
wwpn of our selected boot adapter as initiator wwpns
(and then lookup the target wwpns for it).

Closes-Bug: #1662511

Change-Id: Ib5890c9e696e6bfae5d2915f87b4baf151b2ed9a
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-05-08 16:08:21 +05:30
Juergen Leopold ff1b77f71a Handles exception in init_host() if CPC is not found
This code writes an error log in the nova log
if the CPC is not found in the init_host() method
of the DPMDriver. This will help to debug problems.
The error message contains information about the
HMC host, HMC userid and CPC UUID that were used.

Closes-Bug: #1659037

Change-Id: I4606cefc9943c0a1a3a0db0c496dcce5c468407f
Signed-off-by: Juergen Leopold <leopoldj@de.ibm.com>
2017-05-05 15:09:02 +05:30
Jenkins e525fb3c54 Merge "get_available_nodes refresh handling is missing" 2017-05-03 10:58:48 +00:00
Prabhat Ranjan f331120a63 get_available_nodes refresh handling is missing
Closes-Bug: #1669302

Change-Id: Ibc820b4b63e147e5ca2c4642d0536c3a1dffa649
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-05-03 14:45:53 +05:30
Prabhat Ranjan 3a9086eca4 Remove requests.packages.urllib3.disable_warnings()
Remove requests.packages.urllib3.disable_warnings() from
unit testcase because it is not affacting unit testcase.
Means no warning log without
requests.packages.urllib3.disable_warnings().

But keep as it is in main code because we are using
it at one place only.
https://github.com/openstack/nova-dpm/blob/stable/ocata/nova_dpm/virt/dpm/client_proxy.py#L41

Closes-Bug: #1669299

Change-Id: I2eefc01743c2549ebe8cdc32eb95d03a05f7bc0f
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-05-02 14:17:11 +05:30
Jenkins ca84a77306 Merge "image-tools: Precise dev-bus-id generation" 2017-05-02 07:16:47 +00:00
Prabhat Ranjan 0c66a91884 MaxMemoryExceededError uses max_processors value
See 3bb1595d1f/nova_dpm/virt/dpm/utils.py (L38-L44)
Especially the line: config_mem=CONF.dpm.max_processors,
That should be "max_memory".

Closes-Bug: #1687051

Change-Id: I3e631550b29647234404638e201f40f362311d89
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-05-02 11:47:29 +05:30
Jenkins 163ee6f3b7 Merge "Update release docs for milestone releases" 2017-04-28 15:00:53 +00:00
Jenkins e7a4106b58 Merge "Boot from image fails without a clear error message" 2017-04-28 15:00:37 +00:00
Prabhat Ranjan 3bb1595d1f Define proper exception instead of ValidationError
In nova_dpm.virt.dpm.utils.py need to write proper
exceptions instead of ValidationError or Exceptions.

for example
https://github.com/openstack/nova-dpm/blob/master/nova_dpm/virt/dpm/utils.py#L42

Closes-Bug: #1669359

Change-Id: If41458cf46db5dcd93cbcd69e28a8bad19bcb9ac
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-04-26 14:42:33 +05:30
Prabhat Ranjan c1d3edfb43 Change file name from test_utils.py to test_data.py
Our file name convention for unit test is as follow
Add prefix "test_" to the name of the file in
nova_dpm.virt.dpm

we have utils.py file in nova_dpm.virt.dpm and we have
file test_utils.py in unit test also. But test_utils.py
does not contain any unit testcase of utils.py. It contains
something else. Basically it contains some comman data for
other testcase.

Change-Id: I99b9927848a62a6737692aa64e2796b3b766f915
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-04-26 11:38:15 +05:30
Prabhat Ranjan fde3d2625a Change fake session from comman file test_utils.py
Change fake session for
nova_dpm.tests.unit.virt.test_driver.dpmDPMdriverInitHostTestCase

It is very confusing when we use comman fake session available
in test_utils.py. So use individual fake session in each file.

Change-Id: I578362ff1cbfdd2f0207270fa84c40cb06bea3ea
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-04-25 14:36:32 +05:30
Jenkins 95e15b2936 Merge "guest tools: Fix layer2 configuration" 2017-04-24 10:48:57 +00:00
Prabhat Ranjan ff1e8ed060 Boot from image fails without a clear error message
Booting an instance form image fails somewhere with
a magic error message.But as we do not support the boot
from image case. We should check that at first
when the spawn method is entered or even the
pre_for_spawn method is entered and abort with
an appropriate error message!

Closes-Bug: #1663550

Change-Id: I577b28d034322b109b81c978cb50a299be140510
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-04-24 15:21:06 +05:30
Jenkins fc57d88e5e Merge "spawn is not checking if volume is a FC volume" 2017-04-24 09:23:08 +00:00
Andreas Scheuring 335150cc7b guest tools: Fix layer2 configuration
The guest image tools do not properly configure a network interface
in the image for layer2. When hipersockets is used, setting the MAC
fails with "Operation not supported". No communication with the
dhcp on the network is possible.

OSA was not affected as layer2 is the default for OSA if nothing is
provided (for hipersockets it's layer3).

Change-Id: I203a8d4e62afb10666e9b131164e77ab4f85d890
Closes-Bug: #1685737
2017-04-24 11:17:12 +02:00
Prabhat Ranjan cc01b3bafb spawn is not checking if volume is a FC volume
On instance spawn we only indirectly check
if the requested volume is a FC volume via [1].
This will either fail with an "KeyError" or a
"NoneType Error" depending on how the
block_device_info object looks like for non
FC volumes.

[1] https://github.com/openstack/nova-dpm/blob/1.0.0/nova_dpm/virt/dpm/driver.py#L374

Closes-Bug: #1668343

Change-Id: I6eefda4127402d43ad9407190626220722ddee58
Signed-off-by: Prabhat Ranjan <pranjank@in.ibm.com>
2017-04-24 14:20:08 +05:30
Andreas Scheuring 0b086b4b66 image-tools: Precise dev-bus-id generation
Adding an additional comment on how the device-bus-id is generated.

Change-Id: Ief9c57866578dba13ce0741dd43448c35eae4610
Closes-Bug: #1682155
2017-04-20 17:08:43 +02:00