Bug 1254102
| Summary: | Discovery rules page "undefined method `name' for nil:NilClass" in case hostgroup was deleted | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Alexander Braverman <abraverm> | ||||
| Component: | Discovery Plugin | Assignee: | Lukas Zapletal <lzap> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Stephen Benjamin <stbenjam> | ||||
| Severity: | high | Docs Contact: | |||||
| Priority: | high | ||||||
| Version: | 6.1.0 | CC: | abalakht, bbuckingham, bkearney, lzap, mmccune, pmutha, sghai, stbenjam, szadok, xdmoon | ||||
| Target Milestone: | Unspecified | Keywords: | Triaged | ||||
| Target Release: | Unused | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| URL: | http://projects.theforeman.org/issues/11574 | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2016-07-27 09:17:52 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: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 1338516 | ||||||
| Attachments: |
|
||||||
Upstream seems to solve this issue at https://github.com/theforeman/foreman_discovery/blob/develop/app/views/discovery_rules/index.html.erb#L19 Can this be cherry-picked? Which commit sha is that? Can you please prepare cherry pick request for that? Reproduced. If we delete Hostgroup after associating it to Discovery Rule. Upstream patch: https://github.com/theforeman/foreman_discovery/commit/7b7b2acde1c51a33ca09a2b56b1f635c84d09c53 The above commit only masks the fact a rule does not have a hostgroup. Upstream patch done: https://github.com/theforeman/foreman_discovery/pull/212 Cherry pick the UI patch as well ^^^ Upstream bug assigned to lzap Moving to POST since upstream bug http://projects.theforeman.org/issues/11574 has been closed ------------- Anonymous Applied in changeset commit:foreman_discovery|82d93bfb7748482ca32ab293d8c0b47c38c4c018. Created hostgroup Foo, with discovery rule Bar. Tried to delete hostgroup Foo, it's no longer possible: "Cannot delete record because dependent discovery rules exist" 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/RHBA-2016:1501 |
Created attachment 1063664 [details] Foreman debug Description of problem: Opening 'Discovery Rules' page results with an error: undefined method `name' for nil:NilClass Version-Release number of selected component (if applicable): 6.1.1 How reproducible: Open 'Discovery Rules' Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info: Satellite was upgraded from 6.1.0 to 6.1.1