commit | 5111a653a00cea76ce42b76f63e6efdf00aafe89 | [log] [tgz] |
---|---|---|
author | Congbin Guo <guocb@chromium.org> | Tue Apr 13 20:03:29 2021 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Apr 13 20:03:29 2021 |
tree | 2737ebb66465b41141063d5f63c879be3e4844ce | |
parent | a003f9e2d6882808174d5ac3543b1051f43873a0 [diff] |
Revert "caching-backend: conf-creator: wait until the node is configured in UFS" This reverts commit da77f51f650bfbac7229647057c8e2717d46dcc7. Reason for revert: This CL blocks the normal dasemonset rolling out since PODs will not be in Running state if not configured in UFS. Original change's description: > caching-backend: conf-creator: wait until the node is configured in UFS > > As the init container of caching backends, right now conf-creator > creates some non-operational conf files for Nginx and keepalived when > the node is not configured in UFS yet. After the UFS configuration done, > we have to restart the POD. > > With this patch, it will not exit until the node is configured. > > BUG=None > TEST=Built and ran locally with fake 'NODE_IP' and 'NODE_NAME' to > verify the behavior is as expected, i.e. > > Change-Id: I51568b68096b96b4fe5dbec5117fd7a4bf4fc4fa > Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2808700 > Commit-Queue: Congbin Guo <guocb@chromium.org> > Auto-Submit: Congbin Guo <guocb@chromium.org> > Reviewed-by: Allen Li <ayatane@chromium.org> > Cr-Commit-Position: refs/heads/master@{#39958} Bug: None Change-Id: Ie16000b4c9964e8ee09d5a8ba87f60bfec69a52b No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/2811978 Auto-Submit: Congbin Guo <guocb@chromium.org> Reviewed-by: Sanika Kulkarni <sanikak@chromium.org> Reviewed-by: Allen Li <ayatane@chromium.org> Commit-Queue: Allen Li <ayatane@chromium.org> Cr-Commit-Position: refs/heads/master@{#40127}
Testing Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path
modifications.from infra.path_hacks.common import <stg>
is actually getting <stg>
from build/scripts/common.git push origin <updated hash>:deployed
git push
commandIf you've added a new module, run your tests with test.py:
*_test.py
files to this directory.Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>
.
Tests still not getting picked up by test.py? Double-check to make sure you have __init__.py
files in each directory of your module so Python recognizes it as a package.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style, except functions use lowercase_with_underscores
. Use yapf (git cl format
) to autoformat new code.