Bug 1443740

Summary:	Simplify Container and Container Definition Models
Product:	Red Hat CloudForms Management Engine	Reporter:	Federico Simoncelli <fsimonce>
Component:	Providers	Assignee:	Ari Zellner <azellner>
Status:	CLOSED ERRATA	QA Contact:	juwatts
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.7.0	CC:	fsimonce, jfrey, jhardy, obarenbo, simaishi, snaim
Target Milestone:	GA
Target Release:	5.9.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	container
Fixed In Version:	5.9.0.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-03-01 13:11:36 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	Container Management	Target Upstream Version:
Embargoed:

Description Federico Simoncelli 2017-04-19 21:15:02 UTC

Description of problem:
In an early design the Container Definition table was supposed to be used for Replication Controllers information as well. For that reason it is decoupled from the Container table.

It is worth investigating if we can simplify the Container and Container Definition models (1-to-1) by consolidating the data into a single table.

Comment 2 Dave Johnson 2017-07-14 02:50:35 UTC

Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set it to Low/Low.

Comment 3 Beni Paskin-Cherniavsky 2017-09-24 15:07:03 UTC

Done by Ari.  PRs involved (perhaps incomplete):

https://github.com/ManageIQ/manageiq-schema/pull/24
https://github.com/ManageIQ/manageiq/pull/15393
https://github.com/ManageIQ/manageiq-providers-kubernetes/pull/42
https://github.com/ManageIQ/manageiq-providers-kubernetes/pull/76
https://github.com/ManageIQ/manageiq-providers-openshift/pull/40
https://github.com/ManageIQ/manageiq-ui-classic/pull/1760
https://github.com/ManageIQ/manageiq-automation_engine/pull/55
https://github.com/ManageIQ/manageiq/pull/15721

QE: this was just a refactoring, no new functionality.
Old functionality this could have broken if buggy, would be nice to verify:
After refresh UI shows partial info for pod containers that have spec but no status, and full info for running containers.
I *think* a simple way to get pod stuck without containerStatuses is specify non-existing image.  Another is request > quota so the pod is not scheduled.

Comment 4 juwatts 2017-12-01 19:49:44 UTC

CFME Version:
5.9.0.11.20171127204214_e316988

Verification:
Verified via regression testing of Inventory Refresh

Comment 7 errata-xmlrpc 2018-03-01 13:11:36 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:0380