Summary: | Add a concept of dashboard parameters that invalidate others | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Julie Parent <jparent> | ||||
Component: | Tools / Tests | Assignee: | Julie Parent <jparent> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, dpranke, ojan, tony, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Julie Parent
2013-01-30 11:45:29 PST
Created attachment 185529 [details]
Patch
Comment on attachment 185529 [details]
Patch
I'm not sure I understand what it means for one parameter to invalidate another. Does that mean the parameters are mutually exclusive? I take it that, for example, if someone specifies testType=, you want group= to be reset to the default? So you can't actually specify both meaningfully? Is that the same thing as saying that one parameter overrides another?
There are parameters that depend on each other. So, for example, the list of builders available depends on the group. Imagine I have the flakiness dashboard loaded with the group set to ToT - chromium.org and the builder set to WebKit Linux: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20chromium.org&builder=WebKit%20Linux Then, I switch the group to ToT webkit.org. Webkit Linux is no longer a valid builder for that configuration. Currently, the dashboard will select a new default but the query string keeps the old one, so now there is a mismatch between what is selected and what is in the url (and is incorrect in other parts of the code too). Sometimes js errors hit due to these mismatches as well. Comment on attachment 185529 [details]
Patch
Okay. I understand what you're getting at, I think. I think the naming is a bit confusing, but I don't have any better ideas.
Comment on attachment 185529 [details] Patch Clearing flags on attachment: 185529 Committed r141355: <http://trac.webkit.org/changeset/141355> All reviewed patches have been landed. Closing bug. |