Commit Graph

173 Commits

Author SHA1 Message Date
OpenDev Sysadmins 6203dbb61e 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:58 +00:00
Arun Mani 6c2a535a22 Fix flavorsync issue during nova-powervc startup
During nova-powervc service startup, if the flavor sync
fails for some reasons, the code is blowing up causing
the service to get restarted abruptly. This fix is
to handle the same.

Change-Id: I3ce98153d0d3c0e184fd5ae44d7f0e54c19cebe2
Closes-Bug: #1819873
Signed-off-by: Arun Mani <arun.mani@in.ibm.com>
2019-03-13 05:58:19 -05:00
Punith Kenchappa 4685d5fbc7 Custom properties are missing in the image details section.
This is due to the changes made to handle the
community change where V1 glance API got removed
with Queens release of openstack. And, with V2
the response format has changed returning fewer
properties.This bug is to fix the issue related
to missing  custom properties and add them back
to be displayed to the user.

Change-Id: I116ad05b00de6e94cf6c507dbcffa607b916fc42
Closes-Bug: #1814739
2019-02-07 12:16:46 +05:30
Punith Kenchappa 9980f8ee64 Replace glance V1 API calls to use V2 API
With the Queens release of Openstack, the community removed the
support for v1 glance API. Thus, we need to fix the corresponding
implementation in the PowerVC Driver using glance v1 calls to v2 calls.
As part of these changes, we've removed all the v1 glance client calls
to v2, and also observed that the response format has changed as well
and these changes should address all of that.

Change-Id: Ic76bad59ba3da7178acaeeb093113358e50c6169
Closes-Bug: #1783096
2018-07-25 10:45:40 +05:30
arunmani 9a6a26a644 Fix build failure
Change-Id: I81df5c3fd085f71215a572e4d6b620afe50daeb5
Closes-Bug: #1754589
2018-03-09 04:29:38 -05:00
arunmani 7f8aabe246 Fix issue related to keystone version check
Update the keystone version check logic to use version
attribute to detect the keystone version. The current
check breaks the driver sync logic if the attribute
isn't present in the keystone client object.

Change-Id: I271f39b320aec080e84c47b41b53d32be6879af2
Closes-Bug: #1754589
2018-03-09 03:38:47 -05:00
arunmani 9ed13036de Fix minor issues with the driver sync logic
Update the if check in nova sync code to return instance
flavor from powervc, if the call to get the same from db
failed for some reasons. Also, update the keystone version
check in the common package.

Change-Id: Iafc489ad723ef6732bddda029f1976204deef37d
Closes-Bug: #1742377
2018-01-10 02:11:44 -05:00
Zuul 782def6a98 Merge "Add mac address for PORT_UPDATE_FIELDS" 2018-01-04 07:04:20 +00:00
Zuo ZongMing f9694e7180 Add mac address for PORT_UPDATE_FIELDS
Synchronize openstack and powervc-driver port mac_address

Change-Id: If961b8995cf32a9607ff45695268c0bf471b6671
2017-12-15 15:05:55 +08:00
Zuo ZongMing 0f39087e97 Fixed neutron misspelling
Change-Id: Ic5440bb86448b5846e70f4dd802c6ed384773b05
2017-12-15 11:10:53 +08:00
Arun Mani 50d1123691 Revert "Modify incorrect method to translate error message"
This reverts commit 161721311a.

Change-Id: Ia57e7dc952d36e9f4d8c5158b8fae3e44a586cbc
2017-11-13 05:05:26 +00:00
guanzuoyu 161721311a Modify incorrect method to translate error message
When process of powervc-drver connecting keystone failed, it would
give the error message in factory.py, but message objects do not
support str(), so it needs to be modified by using unicode.

Change-Id: I45bbb488300a5fe5c4e4f71265c3abacd775f41d
Closes-Bug: #1730356
2017-11-06 19:21:54 +08:00
arunmani 3f0a4a88c4 Increase timeout to handle multiple heat stack deploys
When multiple heat stack deployments are submitted, there
is mismanagement of IP assigned and heat stack fails with
duplicate IP report. The time to sleep to avoid race condition
in such cases seems too less, and it was found that increasing
the same works better in such case.

Change-Id: I5cd6ac008f9160e35a3e70c831f8f151d0825458
Closes-Bug: #1724302
2017-10-26 04:36:02 -04:00
arunmani 0738af993d Include project_id in QUOTAS.reserve call
Add project_id kwarg as part of the call to deduct quotas
during unregister volume. This will make sure the
deduction happens for the correct project to which
the volume actually belongs to in ICM.

Change-Id: Ib0596aef1b2fde4da0c8164141d39dae2ece4c94
Closes-Bug: #1712093
2017-08-22 02:35:46 -04:00
arunmani b04397a642 Test review bug fix
Closes-Bug: #1671156

Change-Id: Icf43b6a34a48405666edda2451e45adb79057c71
2017-03-10 03:49:26 -05:00
Jenkins 9871dee5e1 Merge "Test demo" 2017-03-07 07:29:00 +00:00
Jerry Cai 2e22ee2bab Test demo
test demo

Change-Id: I9b5217541b17cb34a10aeecca54bb072d0744f11
Closes-Bug: #1670591
2017-03-07 15:25:55 +08:00
Jerry Cai 06a4141f8c Add support for scheduler hints
Add support for scheduler hints.

Change-Id: I31a0f362eb8b9f83972d53d011900d1a038ab65c
Closes-Bug: #1666145
2017-02-20 16:44:07 +08:00
Jerry Cai 95fd19527b Detach sync problem fix
Detach sync problem fix

Closes-Bug: #1647939
Change-Id: I91f31a6bc9730dcbc373406da52bc34349d0de15
2016-12-07 14:51:16 +08:00
Jerry Cai e5babcc287 sync driver problem for 1.3.x
sync driver problem for 1.3.x 

Closes-bug: #1647588
Change-Id: I93c8e808d8ab7f542bf9d1ac9517ebb6e804aafc
2016-12-06 16:32:52 +08:00
Jerry Cai 27b8e9db04 Add support to pvc 131 and 132
Add support to pvc 131 and 132

Change-Id: I752f53f9d8abebab0de86890edd70b17d30b0297
Closes-Bug: #1643787
2016-11-25 09:43:52 +08:00
Jerry Cai 444dfcca9a boot instance with server name case insensitive
boot instance with server name case insensitive, it should use
display_name which is case sensitive rather than hostname.

Change-Id: I742d83236ea3268fc64629e48c08f02562c7b11a
Closes-Bug: #1628404
2016-09-28 16:18:41 +08:00
Jerry Cai 90586b97e2 Remove setting first scg id to boot parameter
Remove setting first scg id to boot parameter, 
or this will fail in mult-scg environment.

Change-Id: I4e7349a2f5b6cbb6f59b629841442c9b8efe2fe4
Closes-Bug: #1608395
2016-08-01 15:14:25 +08:00
Jerry Cai 49d1509c4e Fix for volume description(master)
Fix for volume description is moved from property "display_description"
to "description" for master Edit

Change-Id: I566fb3c18823552becc267be5f0dd31d52694df5
Closes-Bug: #1545973
2016-02-16 16:23:19 +08:00
Jerry Cai 0ae7803b61 Change default spawn defer placement to True
By default we use the host and available zone that ICM chooses, now I
will change it to the design that "Ignore the host that ICM selected,
defer it to powervc scheduler".

Change-Id: Ieaa53ebc425452ac5bad73441a32862373e456b9
Closes-Bug: #1538452
2016-01-27 17:22:20 +08:00
Qing Wu Wang 1408dfd931 ip locked causes boot failed
Nova booting a vm and neutron creating new port would race the port.
If the neutron creates the port before booting vm,
the booting process would be failed as ip/port locked.

Change-Id: Ib5dbbb0b2fbcca17961cddcf983d70779a4dfecd
Closes-Bug: #1505952
2015-10-19 02:46:31 -05:00
Jerry Cai 6478bd1764 Git configuration verify for moving stackforge to openstack repository
This is a test checkin to verify git process success that move the
repository from stackforge to openstack.

Change-Id: Ie79f79ce6428c517f20d8602ef8d1016ea9796fa
Closes-Bug: #1507455
2015-10-19 14:06:30 +08:00
Jerry Cai 8b6a7a3544 Check host status that disable the host service if host is down
Check host status that disable the host service if host is down

Change-Id: I241262e3bf055821cde96a52bfbefa3018cb7b27
Closes-Bug: #1498369
2015-09-23 15:29:56 +08:00
Jerry Cai d0be0b27d7 List all resources across different service
List all resources across different service by adding "all-tenant"
when call restAPI

Change-Id: Ie29e0bac6fa4c3fe4b771489a722b6712edc3a28
Closes-Bug: #1496781
2015-09-17 18:05:06 +08:00
Jerry Cai 8ff2c0d8e7 Cinder duplicated volume deletion due to potential time window issue
Cinder duplicated volume deletion, which will cause multiple quota
deduction.

Change-Id: I5e970cdb682abac51599b4da93b325fc0707c8f5
Closes-Bug: #1494561
2015-09-11 11:20:28 +08:00
Jerry Cai 276880ece8 Catch quota reserve exception to avoid function break
Catch quota reserve exception to avoid function break,
and catch get_info() excption to avoid startup break.

Change-Id: If2f8f535caeef81264f323df73927f0cfb4ecfb2
Closes-bug: #1493726
2015-09-09 17:38:59 +08:00
Jerry Cai 50d97fd983 quotas are not updated from powervc side
quotas are not updated when the volumes/instances are synced from
powervc side, this fix help to sync the quota during sync resources
from powervc side, increase quota when insert to local resource,
deduct quota when delete local resources.

Change-Id: Ic5a77e7d154b247d253db19d59ee66a7aa473995
Closes-Bug: 1493232
2015-09-09 11:14:28 +08:00
Jerry Cai 2c5f580f48 Volume sync and create/attach/detach implement for multiattach function
This bug tracks the implementation for volume sync and create/attach/detach
for multiattach function. Host status update also fixed along with this bug.

Change-Id: I45a776775473818c75388c3554664982849c4fae
Closes-Bug: #1491723
2015-09-03 16:16:38 +08:00
Jerry Cai d96ff4ab09 [Fix]Quotas are not properly updated initially for on-boarded...
fix a problem for previous check-in

Change-Id: I06c6f173618ccc5de3601aff0ca4b4f7b3a9e784
Closes-Bug: #1472923
2015-07-09 15:15:36 +08:00
Jerry Cai cffe6e26ba Quotas are not properly updated initially for on-boarded VMs from PowerVC
Quotas are not properly updated initially for on-boarded VMs from PowerVC,
when using powervc driver, need to commit quota after insert a pvc instance
when sync from powervc side.

Change-Id: I29916631129f9c3c1bb25b7fda947668328c57fd
Closes-Bug: #1472845
2015-07-09 10:49:32 +08:00
Jerry Cai d80d863743 Cinder duplicated volume sync due to potential time window issue
From PBC, the customer encounter below error:

1. Create a volume in ICM
2. Update PowerVC UUID in cinder volume metadata, in the mean time,
PowerVC volume created, end event comes.
3. Check if PowerVC UUID set in volume metadata, but there is
Time window issue here, sometimes db update not reflect
immediately, so no PVC UUID found , and create a duplicated
volume by mistake.

Change-Id: I4e5546034dbf3ff50c2db588e948818431a665c6
Closes-Bug: #1472126
2015-07-07 15:38:50 +08:00
Jerry Cai b8cc4d3065 Fix neutron V2 No valid authentication is available problem
neutron v2 client need admin context, or it will cause the failure

Change-Id: I3a7d1eee60dd513756755f1da29c131e543cebb0
Closes-Bug: #1469625
2015-06-29 16:03:43 +08:00
Jerry Cai 6d39c193f7 instance flavor need to map to 'cpus' instead of 'vcpus'
instance flavor need to map to 'cpus' instead of 'vcpus'

Change-Id: I4fa60bf77979fbf995a278ff1d4ad883637a3254
Closes-Bug: #1464960
2015-06-15 14:12:57 +08:00
Jerry Cai 73362092e6 Update nova driver due to API changed
Update nova driver due to API changed

Change-Id: I661ce2fcb8915634aae4af69b04ee59a10507c33
Closes-Bug: #1460934
2015-06-02 14:41:08 +08:00
terryyao a16ac6df90 Exclude non-bootable storage template from image-topology
PowerVC 1.2.3 added complex boot/data volume concepts and it's image
accessible storage connective group's volume type API will return all
volume types for boot and data volume, we expose the PowerVC storage
templates in image's image-topology property and that property need only
include the bootable storage templates based on PowerVC 1.2.3 introduced
boot/data volume concept as we just expose the volume only for boot
volume at this time. The fix is just to exclude the SCG accessible
storage template via it's
['extra_specs']['capabilities:volume_backend_name'] property, and only
include the the storage template with such property value in SCG's
boot_providers list which means that volume type is a bootable volume
types. If SCG does not have such boot_providers property, then we just
ignore such checking.

Close-Bug: #1460563
Change-Id: Ifa608a514ec70b831d04802b728dff14864578ab
2015-06-01 16:32:46 +08:00
Jenkins 714bc0d9d8 Merge "Block device mapping object mismatches with the latest community API" 2015-05-28 05:45:38 +00:00
terryyao 4f2035dfe2 fix the neutron-powervc utility broken issue
The neutron-powervc is a debug utility for powervc-driver's neutron rpc
service. It's broken with neutron kilo level code

Close-Bug: #1457752
Change-Id: I57101bcdccaa470fea01c8a4abacc1f694abca9b
2015-05-22 14:24:28 +08:00
Jerry Cai 8825cfb4db Block device mapping object mismatches with the latest community API
Block device mapping object mismatches with the latest community API,
which causes the sync problem.

Change-Id: I1e5059b1fb6f98e73b5fabbeb839a3281f8f29b5
Closes-Bug: #1455331
2015-05-15 14:29:02 +08:00
Jerry Cai d43d07acbc Using volume v2 client instead of v1 to get local volume
There is something wrong with volume v1 api to get volume detail info,
need to use volume v2 client instead of v1 to get local volume.

Change-Id: I76d067d2b1ee13c363dd5c22332d99d277c83f47
Closes-Bug: #1453358
2015-05-09 15:10:24 +08:00
Jerry Cai 6e2e506a7e Tempest failed due to openstack interface changed
Tempest failed due to openstack interface changed, fix it as replace all
unused driver method parameters with "*args, **kwargs"

Change-Id: I25c7e0f0eee7c8d16ac4de4e5918fa690544e33c
Closes-Bugs: #1451717
2015-05-05 17:05:40 +08:00
Jerry Cai 9708bca3fb vm update failed due to nova api removed
vm update failed due to nova api removed, need to use
instance object API instead

Change-Id: I5bdac5ec6b6af3340d3af3aff0cd4ae4859bd442
Closes-Bug: #1451131
2015-05-03 13:17:15 +08:00
Jerry Cai ead42c5d1b Glance service lacks v1 endpoint that fails the glance-powervc sync
From the latest ICM, versions are stripped from image service,
only ip:port left, in this case only "CURRENT" status client can be
retrieved, that is to say, for image service only v2.3 url returned
but v1.1 is lost, this will fail glance sync progress as both of
v2.3 and v1.1 are necessary, add this workaround to fix the problem.

Change-Id: I2723ae80654a2daffe7e3f7af0dbf73843bbfea2
Closes-Bug: #1450974
2015-05-02 10:42:23 +08:00
Jenkins 96d9f55201 Merge "Fix cinder detach interface change mismatch problem" 2015-04-29 03:34:30 +00:00
Jerry Cai b0d3faeae4 Fix cinder detach interface change mismatch problem
Fix cinder detach interface change mismatch problem

Change-Id: I97a2319df08c90b55a9d16011e7a4da747fbc8a1
Closes-Bug: #1449826
2015-04-29 11:15:45 +08:00
Jerry Cai e58fed6bf1 Empty remove list will cause failure on glance sync
Empty remove list will cause failure on glance sync

Change-Id: I2433cd66b18239d1e12b6d2691327f4c534db1ec
Closes-Bug: #1446095
2015-04-20 16:35:52 +08:00