667375 - Improvements for 'location statuses' page

Status: VERIFIED FIXED

(Webtools :: Bouncer, defect)

Description

[Nick Thomas [:nthomas] (UTC+12)]

For a selected product a very long list of locations is returned, the 'DISABLED' ones first, then the 'ok' ones. 

'Quick' enhancement: reverse the sort so 'ok' comes first
Better enhancement: default to not showing disabled locations

Of the 'ok' locations the active flag for the mirror is ignored, so the list can be misleading.

Enhancement: hide lines where mirror is disabled, or also mark them DISABLED

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: guess for hiding disabled mirrors

Comment 2

[Fred Wenzel [:wenzel]]

Anthony, will you take a look?

Comment 3

[Anthony Ricaud (:rik)]

Proposed fix with https://github.com/fwenzel/tuxedo/pull/2

Comment 4

[Fred Wenzel [:wenzel]]

r+

It just looks like this can't be automatically merged (says github), so make sure you iron out any merge conflicts before landing.

Comment 5

[Anthony Ricaud (:rik)]

Merged this.

Still needs a push.

Comment 6

[Nick Thomas [:nthomas] (UTC+12)]

Could we schedule a push for this soon ? It would have been very helpful to have live during the relaese today.

Comment 7

[Anthony Ricaud (:rik)]

CCing jabba and justdave who helped with the last push of Bouncer to see if they can help.

Comment 8

[Nick Thomas [:nthomas] (UTC+12)]

Deployed in bug 716988. Looks good.