Automatically dispatch services endpoint to module. |
||
Issue descriptionCurrently, LogDog requires users to manually specicify that "/prpc/logdog.Services/..." pRPC calls get dispatched to the services endpoint as an optimization. Fix this, so that the optimization is no longer necessary and that all such calls automatically get dispatched to the services endpoint.
,
Mar 13 2017
,
Mar 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/154ad1259b6a3be20a5a105da533f45d8c066998 commit 154ad1259b6a3be20a5a105da533f45d8c066998 Author: Dan Jacques <dnj@chromium.org> Date: Tue Mar 14 02:35:00 2017 Remove explicit service reference. As of this CL: https://codereview.chromium.org/2743243004 LogDog application will automatically route all service endpoint calls to the "services" module, removing the need to explicitly perform this action. Remote the explicit routing parameter from the bootstrap. BUG= chromium:700934 TEST=unit Change-Id: Ic5d2f6f13a515a64bb2a88249ff08b6135fec3f8 Reviewed-on: https://chromium-review.googlesource.com/454102 Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Daniel Jacques <dnj@chromium.org> [modify] https://crrev.com/154ad1259b6a3be20a5a105da533f45d8c066998/scripts/slave/logdog_bootstrap.py [modify] https://crrev.com/154ad1259b6a3be20a5a105da533f45d8c066998/scripts/slave/unittests/logdog_bootstrap_test.py
,
Mar 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/154ad1259b6a3be20a5a105da533f45d8c066998 commit 154ad1259b6a3be20a5a105da533f45d8c066998 Author: Dan Jacques <dnj@chromium.org> Date: Tue Mar 14 02:35:00 2017 Remove explicit service reference. As of this CL: https://codereview.chromium.org/2743243004 LogDog application will automatically route all service endpoint calls to the "services" module, removing the need to explicitly perform this action. Remote the explicit routing parameter from the bootstrap. BUG= chromium:700934 TEST=unit Change-Id: Ic5d2f6f13a515a64bb2a88249ff08b6135fec3f8 Reviewed-on: https://chromium-review.googlesource.com/454102 Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Daniel Jacques <dnj@chromium.org> [modify] https://crrev.com/154ad1259b6a3be20a5a105da533f45d8c066998/scripts/slave/logdog_bootstrap.py [modify] https://crrev.com/154ad1259b6a3be20a5a105da533f45d8c066998/scripts/slave/unittests/logdog_bootstrap_test.py |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Mar 13 2017