zVM Cloud Connector

check MDISK method is incorrect

Bug #1769796 reported by jichenjc
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zVM Cloud Connector
Fix Committed
Low
nafeiyang

Bug Description

line 161 we probably should use xxx.startswith('MDISK') to avoid some edge condition

154 def create_disks(self, userid, disk_list):
155 zvmutils.check_guest_exist(userid)
156
157 user_direct = self._smutclient.get_user_direct(userid)
158
159 exist_disks = []
160 for ent in user_direct:
161 if 'MDISK' in ent:
162 md_vdev = ent.split()[1].strip()
163 exist_disks.append(md_vdev)
164

jichenjc (jichenjc)
Changed in python-zvm-sdk:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → nafeiyang (nafeiy)
Revision history for this message
nafeiyang (nafeiy) wrote :

Fix commited
https://review.gerrithub.io/#/c/mfcloud/python-zvm-sdk/+/410389/

Changed in python-zvm-sdk:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.