Bug 1435958 - Reports - pods per ready status - nonexistent pods presented
Summary: Reports - pods per ready status - nonexistent pods presented
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.9.0
Assignee: Nimrod Shneor
QA Contact: Gilad Shefer
URL:
Whiteboard: container:pod:report
Depends On:
Blocks: 1484895
TreeView+ depends on / blocked
 
Reported: 2017-03-26 08:47 UTC by Gilad Shefer
Modified: 2018-06-06 11:49 UTC (History)
10 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1484895 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:13:02 UTC
Category: ---
Cloudforms Team: Container Management
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
PodsPerReadyStatusBug (95.30 KB, image/png)
2017-03-26 08:47 UTC, Gilad Shefer
no flags Details

Description Gilad Shefer 2017-03-26 08:47:53 UTC
Created attachment 1266498 [details]
PodsPerReadyStatusBug

Description of problem:
Reports - pods per ready status - unexistent pods presented


Version-Release number of selected component (if applicable):
5.8

How reproducible:
100%

Steps to Reproduce:
1. GoTo: Could Intel --> Reports --accordion--> Reports -> All Reports --> Configuration Management --> Containers --> Pods per Ready Status
2. Queue
3. Once done -> open report

Actual results:
Some pods that doesn't exists (i.e. Doesn't appear in OCP (the output of 'oc get pods --all-namespaces'))

Expected results:
All the pods presented in the report exists and appears in OCP

Additional info:

Comment 2 Mooli Tayer 2017-03-26 11:02:25 UTC
Gilad we have an on going issue with objects not deleted from the inventory after removal from OCP, so I would like to understand if this is reports related.

Can you reproduce and check if the items in the report appear in the inventory as well?

Comment 3 Barak 2017-03-27 09:19:10 UTC
Loic can you please clarify what "Pods per Ready Status" report means ?

It has a True/Fals value ...

Comment 4 Ari Zellner 2017-03-28 09:29:52 UTC
Barak, "Pods per Ready Status" shows each pod and its 'ready' condition out of its `container_conditions`. 

I think this happens because the report is showing archived pods. its not filtering out the archived pods out of `ContainerGroup.all`(should be `ContainerGroup.where(ems_id => nil)`.

Do we want to filter these out to 'live' pods only? i think we should.

Comment 5 Federico Simoncelli 2017-05-22 21:18:42 UTC
(In reply to Ari Zellner from comment #4)
> Do we want to filter these out to 'live' pods only? i think we should.

Yes we should. We had a similar discussion in bug 1416486.
Maybe it's possible to achieve this with a filter on "Deleted On".

Comment 6 Nimrod Shneor 2017-06-28 16:23:24 UTC
Merged https://github.com/ManageIQ/manageiq-ui-classic/pull/1605

Comment 7 Nimrod Shneor 2017-06-29 04:34:16 UTC
Ugh wrong bug.

Comment 8 Loic Avenel 2017-07-03 11:43:22 UTC
(In reply to Barak from comment #3)
> Loic can you please clarify what "Pods per Ready Status" report means ?
> 
> It has a True/Fals value ...

Replied by Ari

Comment 9 Federico Simoncelli 2017-08-02 12:44:13 UTC
(In reply to Nimrod Shneor from comment #6)
> Merged https://github.com/ManageIQ/manageiq-ui-classic/pull/1605

Can you share the correct link? Thanks.

Comment 10 Nimrod Shneor 2017-08-02 13:10:45 UTC
PR: https://github.com/ManageIQ/manageiq/pull/15405


Note You need to log in before you can comment on or make changes to this bug.