Bug 1212577

Summary:	Editing host or computer profile associated with VmWare resource fails with undefined method
Product:	Red Hat Satellite	Reporter:	Bryan Kearney <bkearney>
Component:	Provisioning	Assignee:	Shlomi Zadok <szadok>
Status:	CLOSED NEXTRELEASE	QA Contact:	Katello QA List <katello-qa-list>
Severity:	high	Docs Contact:
Priority:	high
Version:	6.0.7	CC:	andrew.schofield, aupadhye, bbuckingham, bkearney, chrobert, cwelton, ddevra, jswensso, katello-qa-list, lzap, nshaik, ohadlevy, pmoravec, szadok
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	All
OS:	Linux
URL:	http://projects.theforeman.org/issues/9617
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1188435	Environment:
Last Closed:	2015-06-30 16:44:43 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1188435
Bug Blocks:

Comment 2 Bryan Kearney 2015-04-16 18:07:13 UTC

Upstream bug assigned to szadok

Comment 3 Bryan Kearney 2015-04-16 18:07:17 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/9617 has been closed
-------------
Shlomi Zadok
Applied in changeset commit:d76ca3043ca1bac1229667e342da149e3cd6f304.

Comment 4 Pavel Moravec 2015-05-29 14:00:55 UTC

(In reply to Bryan Kearney from comment #3)
> Moving to POST since upstream bug http://projects.theforeman.org/issues/9617
> has been closed
> -------------
> Shlomi Zadok
> Applied in changeset commit:d76ca3043ca1bac1229667e342da149e3cd6f304.

My understanding is this commit just wraps the error but does not resolve the underlying problem with VMs in folders. Where fog library commits:

https://github.com/fog/fog/commit/bb9a4e8ef723b5855b9a5088a5080deda42aafa1
https://github.com/fog/fog/commit/6f205bbe0cb9dd2fab3c6b0d371b9933aba41507

are supposed to really fix it.

Is there an option to have those commits in Sat6.1? Either by rebasing ruby193-rubygem-fog or to backporting those two fixes to current ruby193-rubygem-fog?

Comment 5 Shlomi Zadok 2015-06-15 11:20:58 UTC

It would be great to have those commits in (greater upgrade fog gem to 1.28+)
However, the proposed solution is also for cases when there is no cluster (whether because it does not exist, the user has no permission, or it is in folders (which the two commits fix)

Comment 6 Bryan Kearney 2015-06-30 16:44:43 UTC

This bug is resolved in the Satellite 6.1 code base in bug https://bugzilla.redhat.com/show_bug.cgi?id=1188435. We are not planning to backport this to 6.0, and are therefore closing out this bug. Please feel free to contact bkearney and dcaplan with any issues.