Get rid of need for host uuid

Bug #1637668 reported by Eric Fried
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pypowervm
Confirmed
Wishlist
Unassigned

Bug Description

We don't need host uuid for things like scsi mapper. We have to leave it in the signature for legacy reasons, but I could surely make it so you can just pass None.

Eric Fried (efried)
Changed in pypowervm:
importance: Undecided → Low
Revision history for this message
Eric Fried (efried) wrote :

This is ongoing, but extremely low priority.

Changed in pypowervm:
status: New → Confirmed
importance: Low → Wishlist
Revision history for this message
Eric Fried (efried) wrote :

Next up:
- Cache the sys UUID in the Adapter so we can use it from wherever without having to GET the ManagedSystem
- Identify all wrappers representing CHILDren of ManagedSystem and override their .get (and possibly other) methods to use ^ and therefore not need the parent_* kwargs
- Get rid of said kwargs in all references to said wrappers (here and in consuming code).

Changed in pypowervm:
assignee: Eric Fried (efried) → Eric Berglund (esberglu)
Revision history for this message
Eric Berglund (esberglu) wrote :

Some of the work was done in these commits. There may be more to do, I'm not sure since it's been so long since I've looked at this. Marking unassigned since I'm no longer on the team.

https://github.com/powervm/pypowervm/commit/2999b8128f85df3c969751c1e4a71dc3af741d2e
https://github.com/powervm/pypowervm/commit/651a01b630650083db78511b9297977a3fbe4876
https://github.com/powervm/pypowervm/commit/346d0c0df98b203fd0ee1ab4d0da14dff03225ce

Changed in pypowervm:
assignee: Eric Berglund (esberglu) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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