FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci(gcb): add --docker flag by devjgm · Pull Request #6122 · googleapis/google-cloud-cpp · GitHub

ci(gcb): add --docker flag - #6122

Merged
devjgm merged 4 commits into
googleapis:masterfrom
devjgm:gcb-docker
Mar 26, 2021
Merged

ci(gcb): add --docker flag#6122
devjgm merged 4 commits into
googleapis:masterfrom
devjgm:gcb-docker

Conversation

devjgm commented Mar 26, 2021
edited by coryan
Loading

Copy link
Copy Markdown
Contributor

This PR adds the -d | --docker flags to run the specified build using
the local docker. This let's users run our GCB builds locally in an
environment that's very similar (exact? pretty close) to the GCB
environment. This also makes debugging easier. Users can start a bash
shell in the specified docker container by passing the -s | --docker_shell
flag.

Examples:

build.sh cmake-install  # Runs build on GCB
build.sh cmake-install --local  # Runs build on your local machine
build.sh cmake-install --docker  # Runs build in docker
build.sh cmake-install --docker-shell  # Starts shell in docker

These docker builds create a "build-out" directory on the host system to
store all their output (similar to our current "cmake-out" directory,
but not cmake-specific). All their build output is stored in a
build-specific dir w/in build-out/.

Coming soon... gcloud ADC credentials will be available in the container so that integration tests work.


This change is 

This PR adds the `-d | --docker` flags to run the specified build using
the local docker. This let's users run our GCB builds locally in an
environment that's very similar (exact? pretty close) to the GCB
environment. This also makes debugging easier. Users can start a bash
shell in the specified docker container by passing the `-s | --docker_shell`
flag.

Examples:

```
build.sh cmake-install  # Runs build on GCB
build.sh cmake-install --local  # Runs build on your local machine
build.sh cmake-install --docker  # Runs build in docker
build.sh cmake-install --docker-shell  # Starts shell in docker
```

These docker builds create a "build-out" directory on the host system to
store all their output (similar to our current "cmake-out" directory,
but not cmake-specific). All their build output is stored in a
build-specific dir w/in build-out/.
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 26, 2021

codecov Bot commented Mar 26, 2021
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #6122 (fe94c8f) into master (1ee16f3) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6122   +/-   ##
=======================================
  Coverage   95.66%   95.67%           
=======================================
  Files        1173     1173           
  Lines      104842   104827   -15     
=======================================
- Hits       100301   100296    -5     
+ Misses       4541     4531   -10     
Impacted Files Coverage Δ
...anner/integration_tests/backup_integration_test.cc 21.05% <0.00%> (-0.93%) ⬇️
google/cloud/spanner/samples/samples.cc 81.14% <0.00%> (+0.76%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ee16f3...fe94c8f. Read the comment docs.

devjgm marked this pull request as ready for review March 26, 2021 19:25
devjgm requested a review from a team March 26, 2021 19:25
Comment thread ci/cloudbuild/build.sh Outdated
Comment thread ci/cloudbuild/build.sh Outdated
devjgm merged commit 6c90820 into googleapis:master Mar 26, 2021
devjgm deleted the gcb-docker branch March 26, 2021 20:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL