yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #68366
[Bug 1587263] Re: A yesno filter should consider string
The reported issue cannot be reproduced in the master branch (queens).
The volume table filter now uses the server-side filtering which means
the cinder API does filtering, so I guess it works. This is one of the
right approaches.
As discussed in the proposed review, we agreed that it is not a good
idea to add a workaround in the yesno filter itself. Instead, we should
fix a problem in the API layer of Cinder (which is best), the API
wrapper in horizon or the JS REST API layer.
>From the above, yesno filter should not be touched, so Won't Fix.
** Changed in: horizon
Milestone: next => None
** Changed in: horizon
Assignee: Hiroyuki Eguchi (h-eguchi) => (unassigned)
** Changed in: horizon
Importance: Wishlist => Undecided
** Changed in: horizon
Status: In Progress => Won't Fix
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1587263
Title:
A yesno filter should consider string
Status in OpenStack Dashboard (Horizon):
Invalid
Bug description:
A yesno filter should consider string
For instance, a bootable item in cinder volume
can be string "true" or string "false".
So, a yesno filter returns "Yes" if value of bootable is string
"false".
There is a possibility of same case in other item.
By all rights, we should change the type from string to boolean.
However, It would be better for us to consider a string data type in yesno filter.
To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1587263/+subscriptions
References