Bug 1189432

Summary:	distro picker options in Provision tab are not updated when system model attributes are changed
Product:	[Retired] Beaker	Reporter:	Dan Callaghan <dcallagh>
Component:	web UI	Assignee:	Anwesha Chatterjee <achatter>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Anwesha Chatterjee <achatter>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	19	CC:	achatter, dcallagh, ebaak, rjoost
Target Milestone:	25.0	Keywords:	EasyFix, Patch, Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-03-19 04:18:34 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 Dan Callaghan 2015-02-05 10:28:18 UTC

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

Steps to Reproduce:
1. Create a new system.
2. On the Essentials tab, add some supported architectures.
3. Go to the Provision tab and try to provision a distro.

Actual results:
No distro families are offered, because the tab still thinks that the system supports no arches.
If you refresh the page, the distro picker will then offer some choices.

Expected results:
Distro picker should be re-rendered when the arches change, so that it offers choices straight away without refreshing the page.

Comment 1 Dan Callaghan 2017-01-12 03:02:56 UTC

Fix here is to make the distro picker widget listen to the system model's arches and re-render itself. I *think* it should just be an extra this.listenTo() call in the initialize() for the distro picker view.

Comment 2 Anwesha Chatterjee 2017-10-26 00:49:53 UTC

https://gerrit.beaker-project.org/#/c/5889/

Comment 3 Anwesha Chatterjee 2017-11-01 02:11:29 UTC

Dan, with the way you were reproducing the bug, turns out that the issue is not with a change related to system arches, but with the change in lab controller. So the fix here is to add a this.listenTo() for change:lab_controller_id

Comment 4 Anwesha Chatterjee 2017-11-01 05:37:54 UTC

Turns out in the end that the issue was the way system.kid passed in distro_picker_options caused the widget to not pick up on changes (since it used this variable to display osmajor options)

Comment 6 Anwesha Chatterjee 2018-01-10 03:19:48 UTC

Verified by following these steps:

* Created a system
* Added an associated labcontroller
* Checked provision tab does not offer any families as no supported architectures are assigned
* add supported arch in essentials tab
* Check that provision tab now offers distro families

Comment 7 Dan Callaghan 2018-02-06 00:36:00 UTC

This bug was fixed on the release-24 branch, to appear in Beaker 24.6, but we are not planning any more maintenance releases for the 24.x series. The next release will be Beaker 25.0. Updating the milestone accordingly.

Comment 8 Roman Joost 2018-03-19 04:18:34 UTC

Beaker 25.0 has been released.

Release notes are available upstream: https://beaker-project.org/docs/whats-new/release-25.html