| Summary: | Inventory Refresh failing for Container Provider. | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Josh Carter <jocarter> | ||||||||
| Component: | Providers | Assignee: | Mooli Tayer <mtayer> | ||||||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Einat Pacifici <epacific> | ||||||||
| Severity: | high | Docs Contact: | |||||||||
| Priority: | high | ||||||||||
| Version: | 5.6.0 | CC: | bazulay, cpelland, fsimonce, jfrey, jhardy, jocarter, mtayer, obarenbo, yzamir | ||||||||
| Target Milestone: | GA | Keywords: | TestOnly | ||||||||
| Target Release: | 5.8.0 | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | 5.8.0.0 | Doc Type: | If docs needed, set a value | ||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | |||||||||||
| : | 1388983 1388984 (view as bug list) | Environment: | |||||||||
| Last Closed: | 2017-06-12 16:43:26 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: | |||||||||
| Bug Depends On: | |||||||||||
| Bug Blocks: | 1388983, 1388984 | ||||||||||
| Attachments: |
|
||||||||||
|
Comment 11
CFME Bot
2016-10-25 18:00:55 UTC
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/58578628b1cadc001c747a653e08df4ad1365672 commit 58578628b1cadc001c747a653e08df4ad1365672 Author: Mooli Tayer <mtayer> AuthorDate: Sun Oct 16 01:10:35 2016 +0300 Commit: Mooli Tayer <mtayer> CommitDate: Tue Oct 25 12:09:45 2016 +0300 Handle nil container state https://bugzilla.redhat.com/show_bug.cgi?id=1383498 .../providers/kubernetes/container_manager/refresh_parser.rb | 4 ++-- .../providers/kubernetes/container_manager/refresh_parser_spec.rb | 7 +++++++ 2 files changed, 9 insertions(+), 2 deletions(-) Created attachment 1218428 [details]
Proposed Patch (hot fix)
The gzipped tar archive has a single file app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb
On a Test appliance please make a backup of the existing file app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb and replace it with the enclosed file.
Created attachment 1218430 [details]
Proposed Patch (hot fix)
The gzipped tar archive has a single file app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb
On a Test appliance please make a backup of the existing file app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb and replace it with the enclosed file.
Created attachment 1218431 [details]
Proposed Patch (hot fix)
The gzipped tar archive has a single file app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb
On a Test appliance please make a backup of the existing file app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb and replace it with the enclosed file.
Mooli, Yaacov, this BZ has a long history. Can you please provide details on how to reproduce? Einat It would be best if we had a way to mock kubernetes data. for now you can verify this by following: https://gist.github.com/moolitayer/e5e7d0cf910a05fa5a7f400f0a115f2f Mooli,
What's the expected result for the above?
I ran the queries and the final query returned:
{:type=>"ManageIQ::Providers::Kubernetes::ContainerManager::Container", :ems_ref=>"0_registry_openshift/origin-docker-registry:v1.4.0-alpha.1", :name=>"registry", :restart_count=>2, :backing_ref=>"docker://77fe0206d0f9b651cc501ddc0318816189d6aca7d8013066446130b18bf8cb69", :container_image=>{:name=>"openshift/origin-docker-registry", :tag=>"v1.4.0-alpha.1", :digest=>nil, :image_ref=>"docker://sha256:c9e453bd2e055aad9283f2fd6d51e309ef871fc6c27bc6060e366bf066b33bed", :registered_on=>2017-04-24 17:46:24 UTC, :container_image_registry=>nil}, :last_state=>:terminated, :last_reason=>nil, :last_started_at=>nil, :last_finished_at=>nil, :last_exit_code=>nil, :last_signal=>nil, :last_message=>nil}
Is this the expected result?
Yes, When run without the fix it raises an exception Verified. Ran script as explained in comment 26. |