Make new UI be the default UI

Make new UI be the default UI

BUG=631047
Committed: https://github.com/luci/luci-py/commit/1b4140a3a933e208e075d2b61cb150acc6a1f9f5

[kjlubick, 2016-10-10 19:33:44]

kjlubick@google.com changed reviewers:
+ maruel@chromium.org

[M-A Ruel, 2016-10-11 16:19:35]

lgtm

Once it's rolled out, the next change would be to setup forwarders and remove
the templates. I'm fine with doing it more incrementally.

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
File appengine/swarming/handlers_frontend.py (right):

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_frontend.py:827:
('/<page:(bot|botlist|task|tasklist|())>', UIHandler),
() is not needed

[kjlubick, 2016-10-11 16:58:12]

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
File appengine/swarming/handlers_frontend.py (right):

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_frontend.py:827:
('/<page:(bot|botlist|task|tasklist|())>', UIHandler),
On 2016/10/11 at 16:19:35, M-A Ruel wrote:
> () is not needed

I'm pretty sure it is, otherwise it doesn't match empty string (i.e. index)

Unless you meant 
    ('/<page:(bot|botlist|task|tasklist|)>', UIHandler),

?

[M-A Ruel, 2016-10-11 16:59:32]

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
File appengine/swarming/handlers_frontend.py (right):

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_frontend.py:827:
('/<page:(bot|botlist|task|tasklist|())>', UIHandler),
On 2016/10/11 16:58:12, kjlubick wrote:
> On 2016/10/11 at 16:19:35, M-A Ruel wrote:
> > () is not needed
> 
> I'm pretty sure it is, otherwise it doesn't match empty string (i.e. index)
> 
> Unless you meant 
>     ('/<page:(bot|botlist|task|tasklist|)>', UIHandler),

yes this is what I meant.

[kjlubick, 2016-10-11 17:09:06]

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
File appengine/swarming/handlers_frontend.py (right):

https://codereview.chromium.org/2410573002/diff/1/appengine/swarming/handlers...
appengine/swarming/handlers_frontend.py:827:
('/<page:(bot|botlist|task|tasklist|())>', UIHandler),
On 2016/10/11 at 16:59:31, M-A Ruel wrote:
> On 2016/10/11 16:58:12, kjlubick wrote:
> > On 2016/10/11 at 16:19:35, M-A Ruel wrote:
> > > () is not needed
> > 
> > I'm pretty sure it is, otherwise it doesn't match empty string (i.e. index)
> > 
> > Unless you meant 
> >     ('/<page:(bot|botlist|task|tasklist|)>', UIHandler),
> 
> yes this is what I meant.

Done.

[kjlubick, 2016-10-13 16:51:00]

The CQ bit was checked by kjlubick@google.com

[kjlubick, 2016-10-13 16:51:01]

The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2410573002/#ps20001
(title: "simplify regex")

[commit-bot: I haz the power, 2016-10-13 16:51:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-13 16:54:44]

Description was changed from

==========
Make new UI be the default UI

BUG=631047
==========

to

==========
Make new UI be the default UI

BUG=631047

Committed:
https://github.com/luci/luci-py/commit/1b4140a3a933e208e075d2b61cb150acc6a1f9f5
==========

[commit-bot: I haz the power, 2016-10-13 16:54:46]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/1b4140a3a933e208e075d2b61cb150acc6a1f9f5