Bug 1745726

Summary:	[RFE] - Add ability to include facts on list of consumers
Product:	[Community] Candlepin	Reporter:	Tramaine Darby <tdarby>
Component:	candlepin	Assignee:	candlepin-bugs
Status:	CLOSED WONTFIX	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	2.7	CC:	csnyder, ffranz, nmoumoul, redakkan
Target Milestone:	---	Keywords:	FutureFeature
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2022-02-14 15:07:50 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:

Description Tramaine Darby 2019-08-26 18:22:43 UTC

Description of problem: In the OpenShift Cluster Manager application - there is logic to cleanup consumers (delete consumers for which the clusters have been deleted or archived). we have the cluster ID as a fact in the consumer in order to be able to make the relation. so I take the list of consumers belonging to an org, and use the cluster ID in facts to compare which ones still exist. however, since the list doesn't include facts, I have to query each consumer we got just in order to have the facts and be able to compare. it's an important bottleneck in the job today.

this is the exact place: https://gitlab.cee.redhat.com/service/uhc-account-manager/blob/develop/pkg/services/reconcile/reconcile_organization.go#L65

note that's listing consumers for an owner, in /owners/ID/consumers

Desired change: Add an ?includeFacts=true flag on the call to /owners/ID/consumers to return the facts on each consumer returned in the list.

Comment 2 Tramaine Darby 2022-02-14 14:38:34 UTC

We believe this use case is no longer an issue, and do not need this change.

Comment 3 Nikos Moumoulidis 2022-02-14 15:07:50 UTC

Closing as per comment #2