Bug 1391850 - Compute-Clouds-Instances throwing unexpected error
Summary: Compute-Clouds-Instances throwing unexpected error
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: All
OS: Linux
high
medium
Target Milestone: GA
: 5.8.0
Assignee: Jason Frey
QA Contact: Jiri Stefanisin
URL:
Whiteboard: cloud:vm:ui:error
Depends On:
Blocks: 1428126 1428129
TreeView+ depends on / blocked
 
Reported: 2016-11-04 09:00 UTC by Krishna Babu K
Modified: 2020-12-14 07:50 UTC (History)
10 users (show)

Fixed In Version: 5.8.0.4
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1428126 1428129 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:06:08 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Screen shot reference for the reported issue. (118.74 KB, image/png)
2016-11-04 09:00 UTC, Krishna Babu K
no flags Details
production-log for unexpected error on compute-clouds-Instances (134.28 KB, text/plain)
2017-03-24 15:06 UTC, Krishna Babu K
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1375684 0 high CLOSED error loading instance page 2021-02-22 00:41:40 UTC

Internal Links: 1375684

Description Krishna Babu K 2016-11-04 09:00:03 UTC
Created attachment 1217327 [details]
Screen shot reference for the reported issue.

Description of problem:
While navigating to Compute-Clouds-Instances page throwing unexpected error

Version-Release number of selected component (if applicable):
CFME Version 5.7.0.9-beta2.1.20161101182054_eb0afaa 
Browser Version firefox-49.0-2.fc24.x86_64
Environment https://10.65.207.162/

How reproducible:
always

Steps to Reproduce:
1. Log into cfme
2. Navigate to Compute-Clouds-Instances
3. 'All instances by provider' results in unexpected error

Actual results:
unexpected error

Expected results:
Instances by provider should present

Additional info:
screen shot attached.

Comment 5 Dave Johnson 2016-11-11 05:32:18 UTC
Is this a dupe of bug 1375684 ?

Comment 7 Jason Frey 2016-11-21 20:31:54 UTC
The "Multiple Parents Found" thing is not something I can fix as a blocker.  It's an issue we've been looking at for a long time and cannot find a way to reproduce (there is another BZ ticket for this same problem).  We thought that maybe the out-of-band refresh was the issue and that was corrected, but it did not fix the problem.

There are other errors in this ticket that should probably be addressed, but possibly should be made into separate BZs (not sure if either of those should or should not be blockers)

Specifically, 

- Error caught: [NoMethodError] undefined method `split' for nil:NilClass
- MIQ(ManageIQ::Providers::OpenshiftEnterprise::Container Manager::EventCatcher::Runner#start_event_monitor) EMS [cmTeamMaster 01.rhq.lab.eng.bos.redhat.com] as [] Event Monitor 
Thread aborted because [undefined method `rindex' for nil:NilClass]
[----] E, [2016-11-06T23:03:53.498450 #17530:890b9f8] ERROR -- : [NoMethodError]: undefined method `rindex' for nil:NilClass  Method:[rescue in block in start_event_monitor]

Comment 10 CFME Bot 2017-03-01 18:41:26 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/af9e051cb6d0d5b90b0e5448cc8ca15e5557011d

commit af9e051cb6d0d5b90b0e5448cc8ca15e5557011d
Author:     Jason Frey <jfrey>
AuthorDate: Fri Feb 24 01:50:55 2017 -0500
Commit:     Jason Frey <jfrey>
CommitDate: Fri Feb 24 17:07:22 2017 -0500

    Fix "Multiple Parents Found" issue when moving a relationship.
    
    Originally, `.parent=` was written in terms of `add_children`.  However,
    `add_children` supports duplicating nodes in the tree.  This is reasonable
    because you may have a tree where multiple entries make sense, and for
    those things that have multiple entries, you wouldn't call `.parent`, you
    would probably call `.parents` and it is expected that calling `.parent`
    would blow up with the "Multiple Parents Found" error.  However, the
    expectation of calling `.parent=` is that it will become the new, sole,
    parent, and so implementing it via add_children is wrong because it
    creates an oddity in that you can call `.parent=` and then not be able to
    call `.parent` afterwards.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1379464
    https://bugzilla.redhat.com/show_bug.cgi?id=1391850
    https://bugzilla.redhat.com/show_bug.cgi?id=1406431

 app/models/account.rb                         |  4 +-
 app/models/mixins/relationship_mixin.rb       | 21 +++++-----
 lib/extensions/ar_miq_set.rb                  |  3 +-
 spec/models/mixins/relationship_mixin_spec.rb | 56 +++++++++++++++++++++++++++
 4 files changed, 71 insertions(+), 13 deletions(-)

Comment 13 Krishna Babu K 2017-03-24 15:06:52 UTC
Created attachment 1266135 [details]
production-log for unexpected error on compute-clouds-Instances

Tested on Version 5.8.0.7.20170321164727_1c97ccd 
Reported issue is still reproducible, attaching production log for reference.
Browser Version firefox-52.0-4.fc25.x86_64
Environment https://10.65.207.123/


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