Bug 1635305
| Summary: | Subscriptions details page does not handle Org switch correctly. | ||
|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Perry Gagne <pgagne> |
| Component: | Subscription Management | Assignee: | Jeremy Lenz <jlenz> |
| Status: | CLOSED ERRATA | QA Contact: | Perry Gagne <pgagne> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 6.4 | CC: | egolov, jlenz, walden |
| Target Milestone: | 6.6.0 | Keywords: | Regression, Triaged, UserExperience |
| Target Release: | Unused | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | tfm-rubygem-katello-3.12.0.15-1 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2019-10-22 19:51:08 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: | |||
|
Description
Perry Gagne
2018-10-02 14:55:49 UTC
Created redmine issue https://projects.theforeman.org/issues/25113 from this bug Upstream bug assigned to afeferku The PR [1] is based on some subscription page refactoring that was done upstream. It will likely be much easier and safer to apply this to a future release such as 6.6. Marking as untriaged to get another look at triaging this for a future release since there is a workaround for now (reload the page). [1] https://github.com/Katello/katello/pull/8022/ Upstream bug assigned to jomitsch Upstream bug assigned to jomitsch Upstream bug assigned to jomitsch Upstream bug assigned to jomitsch The attached PR addresses Flow 2 (the 'id undefined' errors): https://github.com/Katello/katello/pull/8227 Flow 1 happens because the page doesn't check for permission to view the subscription details, because we don't have easy access to that permission data. Per the discussion on the PR, Flow 1 will be addressed as part of a more comprehensive look at how Katello deals with permissions. (In reply to Jeremy from comment #13) > The attached PR addresses Flow 2 (the 'id undefined' errors): > > https://github.com/Katello/katello/pull/8227 > > Flow 1 happens because the page doesn't check for permission to view the > subscription details, because we don't have easy access to that permission > data. > Per the discussion on the PR, Flow 1 will be addressed as part of a more > comprehensive look at how Katello deals with permissions. Flow #1 will be fixed by https://projects.theforeman.org/issues/27450 which will prevent users from accessing pages they don't have permission to view. Organization switching in general will also be helped by https://projects.theforeman.org/issues/27451, which suggests that we rewrite the organization switcher in foreman in react so that it stores the organization in the foreman redux store. Once we do that we can fix katello to use that store and we'll only have one single source of truth for what organization is selected. Both of these efforts are too large for 6.6 so look for them (hopefully) in 6.7 TL;DR: only flow 2 is fixed as part of this BZ. Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/25113 has been resolved. Verified fix in 6.6 snap 17. Verified that Flow 2 described above now redirects the user to the "Select an organization" page as expected. As for flow 1 I filed bz 1745736. This should be resolved. Please let me know if any additional info is needed. 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-2019:3172 |