Currently luci-config uses cloud endpoints as a UI. It would be nice to have a proper UI for luci-config.
Currently luci-config uses API Explorer as a UI. It would be nice to have a proper UI for luci-config.
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/f52c4527495dc0c4fbb42def00117214784afc44 commit f52c4527495dc0c4fbb42def00117214784afc44 Author: cwpayton <cwpayton@google.com> Date: Thu Jun 08 00:16:03 2017 Will build features from here. BUG= 730832 Review-Url: https://codereview.chromium.org/2923973003 [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/.bowerrc [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/.gitignore [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/Makefile [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/README.md [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/bower.json [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/index.html [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/manifest.json [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/polymer.json [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/src/config-ui/config-ui.html [add] https://crrev.com/f52c4527495dc0c4fbb42def00117214784afc44/appengine/config_service/ui/test/config-ui/config-ui_test.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/bb34bc085032b88295f0215050c8735ea554b40b commit bb34bc085032b88295f0215050c8735ea554b40b Author: nodir <nodir@chromium.org> Date: Thu Jun 08 18:11:32 2017 config_service: optimize projects.get_metadata projects.get_metadata is used for all ACL checks for project config sets, thus it must be fast. Also it never changes. Cache it for 10m R=hinoka@chromium.org, sergeyberezin@chromium.org BUG= 730832 Review-Url: https://codereview.chromium.org/2932733002 [modify] https://crrev.com/bb34bc085032b88295f0215050c8735ea554b40b/appengine/config_service/projects.py
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/d3a810515f73c543eab2c1eb9db38a65f9d712e1 commit d3a810515f73c543eab2c1eb9db38a65f9d712e1 Author: nodir <nodir@chromium.org> Date: Thu Jun 08 18:27:04 2017 config_service: implement admin group Add "admin_group" to the acl config so that we can define a limited group that has admin privileges on luci-config, not other services. Also aggressively cache acl.cfg because it ~never changes R=hinoka@chromium.org BUG= 730832 Review-Url: https://codereview.chromium.org/2932473005 [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/components/components/config/proto/service_config.proto [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/components/components/config/proto/service_config_pb2.py [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/config_service/acl.py [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/config_service/acl_test.py [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/config_service/admin.py [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/config_service/api.py [modify] https://crrev.com/d3a810515f73c543eab2c1eb9db38a65f9d712e1/appengine/config_service/api_test.py
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/a24b2b14dc50bd7b711f2ca93ffb481becb15e89 commit a24b2b14dc50bd7b711f2ca93ffb481becb15e89 Author: nodir <nodir@chromium.org> Date: Mon Jun 12 20:53:13 2017 config_service: fetch repos and metadata concurrently optimize get_projects() by calling get_repos_async and get_metadata_async concurrently R=hinoka@chromium.org, sergeyberezin@chromium.org BUG= 730832 Review-Url: https://codereview.chromium.org/2931673003 [modify] https://crrev.com/a24b2b14dc50bd7b711f2ca93ffb481becb15e89/appengine/config_service/acl.py [modify] https://crrev.com/a24b2b14dc50bd7b711f2ca93ffb481becb15e89/appengine/config_service/acl_test.py [modify] https://crrev.com/a24b2b14dc50bd7b711f2ca93ffb481becb15e89/appengine/config_service/api.py [modify] https://crrev.com/a24b2b14dc50bd7b711f2ca93ffb481becb15e89/appengine/config_service/api_test.py [modify] https://crrev.com/a24b2b14dc50bd7b711f2ca93ffb481becb15e89/appengine/config_service/projects.py [modify] https://crrev.com/a24b2b14dc50bd7b711f2ca93ffb481becb15e89/appengine/config_service/projects_test.py
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/11c952f68eeb28691033eaf9d4057f9a190797ed commit 11c952f68eeb28691033eaf9d4057f9a190797ed Author: nodir <nodir@chromium.org> Date: Mon Jun 12 21:07:30 2017 config_service: use == for sentinel value I forgot to upload this change to https://codereview.chromium.org/2931673003 TBR=sergeyberezin@chromium.org BUG= 730832 Review-Url: https://codereview.chromium.org/2934783004 [modify] https://crrev.com/11c952f68eeb28691033eaf9d4057f9a190797ed/appengine/config_service/projects.py
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/2279363402929b7d5e45d5bb6591addab7824f2c commit 2279363402929b7d5e45d5bb6591addab7824f2c Author: cwpayton <cwpayton@google.com> Date: Tue Jun 13 17:47:57 2017 config_service: improve landing page config sets are now downloaded and searchable, but not clickable BUG= 730832 Review-Url: https://codereview.chromium.org/2935763003 [modify] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/app.yaml [modify] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/.gitignore [modify] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/bower.json [modify] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/index.html [add] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/src/config-ui/config-card.html [delete] https://crrev.com/d96c2be24ea3c769cce1cbe671af6786bb7ec1de/appengine/config_service/ui/src/config-ui/config-ui.html [add] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/src/config-ui/front-page.html [add] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/static/images/chromium.png [rename] https://crrev.com/2279363402929b7d5e45d5bb6591addab7824f2c/appengine/config_service/ui/test/config-ui/front-page_test.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/51155047e84961268eeffdfa2ec8a595ed4c21b4 commit 51155047e84961268eeffdfa2ec8a595ed4c21b4 Author: cwpayton <cwpayton@google.com> Date: Wed Jun 21 15:42:33 2017 config_service: integrated working config set page with routing The config-set-card elements on the front page now redirect to config-set pages that list the files of the config set. Currently the config-file-cards all link to gitiles. BUG= 730832 Review-Url: https://codereview.chromium.org/2953463002 [add] https://crrev.com/51155047e84961268eeffdfa2ec8a595ed4c21b4/appengine/config_service/ui/src/config-ui/config-file-card.html [add] https://crrev.com/51155047e84961268eeffdfa2ec8a595ed4c21b4/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/51155047e84961268eeffdfa2ec8a595ed4c21b4/appengine/config_service/ui/src/config-ui/config-ui.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/71c3c10aecb784a070f19d47eabb4ccfebc6c6c2 commit 71c3c10aecb784a070f19d47eabb4ccfebc6c6c2 Author: cwpayton <cwpayton@google.com> Date: Thu Jun 29 17:27:25 2017 config_service: Added google-signin-aware.html for Polymer 2.0. This is the original file that was pulled from Scarygami's 2.0-preview branch of the Google Web Component google-signin-aware. This CL is being uploaded so that my changes to this file can be viewed in another CL. The link to this particular file can be found here: https://github.com/Scarygami/google-signin/blob/599547e042ae4e5dec21051953754eec93fe26ca/google-signin-aware.html BUG= 730832 Review-Url: https://codereview.chromium.org/2963963003 [add] https://crrev.com/71c3c10aecb784a070f19d47eabb4ccfebc6c6c2/appengine/config_service/ui/common/third-party/google-signin-aware.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/160ef2c6cacca9617e67c94b758ff4303e3fe839 commit 160ef2c6cacca9617e67c94b758ff4303e3fe839 Author: cwpayton <cwpayton@google.com> Date: Fri Jun 30 00:15:39 2017 config_service: Add authentication to the UI. Users can now sign in to view all projects and services. This change involved creating a modified version of the google-signin-aware element that works with Polymer 2.0. This modified file is in a third party folder. Staging version can be found here: https://2941-71c3c10-tainted-cwpayton-dot-luci-config.appspot.com/newui BUG= 730832 Review-Url: https://codereview.chromium.org/2962703003 [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/app.yaml [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/handlers.py [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/README.md [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/bower.json [add] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/common/auth-signin.html [rename] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/common/third_party/google-signin/google-signin-aware.html [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/src/config-ui/config-ui.html [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/src/config-ui/front-page.html [modify] https://crrev.com/160ef2c6cacca9617e67c94b758ff4303e3fe839/appengine/config_service/ui/static/index.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/32c3d35dd963a5798200fbbdf18f9025317ec5e8 commit 32c3d35dd963a5798200fbbdf18f9025317ec5e8 Author: cwpayton <cwpayton@google.com> Date: Thu Jul 06 20:05:47 2017 config_service: Added force refresh to the config set page. Users can now force refresh the list of config files within a given config set if they would like to see their changes propogated immediately. BUG= 730832 Review-Url: https://codereview.chromium.org/2971033003 [modify] https://crrev.com/32c3d35dd963a5798200fbbdf18f9025317ec5e8/appengine/config_service/ui/bower.json [modify] https://crrev.com/32c3d35dd963a5798200fbbdf18f9025317ec5e8/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/32c3d35dd963a5798200fbbdf18f9025317ec5e8/appengine/config_service/ui/src/config-ui/config-ui.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/56121b33d24c5a9fb2a8aff8247cf52f749a6952 commit 56121b33d24c5a9fb2a8aff8247cf52f749a6952 Author: ayanaadylova <ayanaadylova@google.com> Date: Tue Jul 11 00:26:39 2017 config_service: add last import validation and tests In the front page user can see whether last import was successful or failed. In the config set page the appropriate error message is displayed if the last import failed. Added tests for front-page and config-set page. BUG= 730832 Review-Url: https://codereview.chromium.org/2959833002 [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/app.yaml [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/src/config-ui/config-file-card.html [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/src/config-ui/config-set-card.html [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/src/config-ui/config-ui.html [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/src/config-ui/front-page.html [add] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/static/elements.html [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/static/index.html [add] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/test/config-ui/config-set_test.html [modify] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/test/config-ui/front-page_test.html [add] https://crrev.com/56121b33d24c5a9fb2a8aff8247cf52f749a6952/appengine/config_service/ui/test/index.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/32ed948dea48cdeeb11ee7b4d8ab0465837f42c9 commit 32ed948dea48cdeeb11ee7b4d8ab0465837f42c9 Author: cwpayton <cwpayton@google.com> Date: Tue Jul 11 21:44:19 2017 config_service: Fixed bug regarding None entries in latest_revisions dict. Using the include_files option on config sets that had no latest revision number caused a 503 Internal Server Error because there was no check for None entries in the result from get_latest_revisions_async. This cl fixes that by adding a check. BUG= 730832 Review-Url: https://codereview.chromium.org/2981473002 [modify] https://crrev.com/32ed948dea48cdeeb11ee7b4d8ab0465837f42c9/appengine/config_service/api.py
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/4d627a522380d34a18e57b534fa38f4180e418c3 commit 4d627a522380d34a18e57b534fa38f4180e418c3 Author: cwpayton <cwpayton@google.com> Date: Thu Jul 13 03:37:10 2017 config_service: Changed lists from paper-items to paper-cards, fixed search box, added shadows and slight animation. BUG= 730832 Review-Url: https://codereview.chromium.org/2977763002 [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/bower.json [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/common/auth-signin.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/common/third_party/google-signin/google-signin-aware.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/src/config-ui/config-file-card.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/src/config-ui/config-set-card.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/src/config-ui/config-ui.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/src/config-ui/front-page.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/test/config-ui/config-set_test.html [modify] https://crrev.com/4d627a522380d34a18e57b534fa38f4180e418c3/appengine/config_service/ui/test/config-ui/front-page_test.html
Apparently Type=Launch makes the bug internal-only. Making this bug public again.
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/f16285c26e156183e6b896b6fa1966c4491fc70c commit f16285c26e156183e6b896b6fa1966c4491fc70c Author: ayanaadylova <ayanaadylova@google.com> Date: Wed Jul 19 21:16:13 2017 config_service: README update Updated README.md file inside ui folder. BUG= 730832 Review-Url: https://codereview.chromium.org/2985493002 [modify] https://crrev.com/f16285c26e156183e6b896b6fa1966c4491fc70c/appengine/config_service/ui/README.md
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/c14fbd22e221ec137f85c90d403299ccc85d03c1 commit c14fbd22e221ec137f85c90d403299ccc85d03c1 Author: ayanaadylova <ayanaadylova@google.com> Date: Thu Jul 20 04:07:05 2017 config_service: release the UI to the front page Added link to the APIs explorer in the header, removed /newui handler. BUG= 730832 Review-Url: https://codereview.chromium.org/2983103002 [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/handlers.py [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/ui/README.md [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/ui/common/auth-signin.html [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/ui/src/config-ui/config-ui.html [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/ui/src/config-ui/front-page.html [modify] https://crrev.com/c14fbd22e221ec137f85c90d403299ccc85d03c1/appengine/config_service/ui/static/index.html
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/421927b9bb0fa68f8b36394c5b12dc6aec3c0e2f commit 421927b9bb0fa68f8b36394c5b12dc6aec3c0e2f Author: ayanaadylova <ayanaadylova@google.com> Date: Fri Jul 21 21:28:27 2017 config_service: make UI adjustable for mobile Changed percents to pixels in the width fields of paper cards and made UI adjustable for mobile. BUG= 730832 Review-Url: https://codereview.chromium.org/2983203002 [modify] https://crrev.com/421927b9bb0fa68f8b36394c5b12dc6aec3c0e2f/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/421927b9bb0fa68f8b36394c5b12dc6aec3c0e2f/appengine/config_service/ui/src/config-ui/config-ui.html [modify] https://crrev.com/421927b9bb0fa68f8b36394c5b12dc6aec3c0e2f/appengine/config_service/ui/src/config-ui/front-page.html
Thanks Cameron and Ayana for the implementation and Sergey and Ryan for hosting!
Comment 1 by no...@chromium.org
, Jun 7 2017