Fix retry bug in watcher.go

When I ported the WaitForAllAssetsReady implementation from python to golang,
I forgot a "continue" in the retry loop when I got rid of the try/catch, which
leads to a null ptr crash right after.

Original change that introduced the bug:
https://chromium-review.googlesource.com/c/enterprise/cel/+/1388185

Bug itself:
https://crrev.com/c/1388185/10/go/cel/deploy/watcher.go#88
https://crrev.com/c/1388185/10/test/infra/gcp/runtime_config.py#b100

Change-Id: I7499054e134e81fa4e30fbb66785c65fd2de6572
Reviewed-on: https://chromium-review.googlesource.com/c/1427379
Reviewed-by: Fei Ling <feiling@chromium.org>
1 file changed
tree: 5ffbfd7998d8d232e259ec03ab191865c48d203b
  1. build/
  2. docs/
  3. examples/
  4. go/
  5. infra/
  6. resources/
  7. schema/
  8. scripts/
  9. test/
  10. .gitignore
  11. .gitmodules
  12. .style.yapf
  13. .vpython
  14. build.py
  15. CODE_OF_CONDUCT.md
  16. codereview.settings
  17. CONTRIBUTING.md
  18. Gopkg.lock
  19. Gopkg.toml
  20. LICENSE
  21. navbar.md
  22. OWNERS
  23. PRESUBMIT.py
  24. README.md
  25. run_tests.py
  26. test.py
  27. VERSION
README.md

Chrome Enterprise Lab

Chrome Enterprise Lab is an inaccurately named set of tools for building enterprise labs quickly and easily. The labs so built can be used for system level end-to-end testing of Google Chrome/Chromium.

Have a peek at the Design document.

Also have a peek at the Code of Conduct.

Most of the code is in Go. See Guide to code to get a head start on the code in this repository.

Building

Prerequisites

  • Go: Download Go from the Go download page. This project requires Go 1.9.4 or later. The build will fail with prior versions.

  • Depot Tools: Used for managing the checkout and the contributor workflow.

  • Protocol Buffers Compiler: Protocol buffers are used extensively for wranging all the data that needs to be shuttled around. Can be installed automatically via ./build.py deps --install.

  • Dep : Used for Go depedency management. This can be installed automatically by running ./build.py deps --install from the root of the source tree.

  • Go support for Protocol Buffers : This can be installed automatically by running ./build.py deps --install from the root of the source tree.

Get The Source

There are two ways to get the source. One is to use managed deps, and the other is to use plain go get. The latter workflow doesn't quite work yet due to this repository not being integrated with go.chromium.org. So this page only mentions the managed dependency workflow.

  1. Clone this repository:

    Assumes that $GOPATH is a single path and not a : delimited list.

    mkdir -p ${GOPATH}/src/chromium.googlesource.com/enterprise
    cd ${GOPATH}/src/chromium.googlesource.com/enterprise
    git clone https://chromium.googlesource.com/enterprise/cel
    cd cel
    
  2. Get the dependencies:

    python build.py deps --install
    

Build It

  1. Use the build script:

    python build.py build
    
  2. Also make sure the tests pass.

    python build.py test
    

Contributing

See CONTRIBUTING.