Currently Milo requires swarming tasks to have allow_milo:1, which is a hack. This bug is about removing this requirement
I think a proposal is to use the "luci_project" swarming tag as the ACL mechanism.
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/dd5690205f17fb9b744d1466368093e9c07efef6 commit dd5690205f17fb9b744d1466368093e9c07efef6 Author: hinoka <hinoka@google.com> Date: Fri Apr 14 00:14:17 2017 Milo: ACLs for internal swarm jobs This uses the luci_project:<project name> tag for ACLing swarming builds. BUG= 697640 Review-Url: https://codereview.chromium.org/2818563002 [add] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/frontend/expectations/swarming.build-build-internal.html [modify] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/frontend/main_test.go [modify] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/build.go [modify] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/build_test.go [modify] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/buildinfo.go [add] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/expectations/build-internal.json [modify] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/html_data.go [add] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/testdata/build-internal [add] https://crrev.com/dd5690205f17fb9b744d1466368093e9c07efef6/milo/appengine/swarming/testdata/build-internal.swarm
This has been rolled
Comment 1 by hinoka@chromium.org
, Mar 1 2017