Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

build: use trampoline_v2 for python samples #308

Closed
wants to merge 5 commits into from

Conversation

busunkim96
Copy link
Contributor

@busunkim96 busunkim96 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 1, 2021
@busunkim96 busunkim96 requested a review from a team as a code owner October 1, 2021 15:58
@product-auto-label product-auto-label bot added api: cloudasset Issues related to the googleapis/python-asset API. samples Issues that are directly related to samples. labels Oct 1, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 1, 2021
@busunkim96 busunkim96 changed the title chore: test samples kokoro build: use trampoline_v2 for python samples Oct 1, 2021
@busunkim96 busunkim96 added the owlbot:ignore instruct owl-bot to ignore a PR label Oct 1, 2021
@tseaver
Copy link
Contributor

tseaver commented Oct 1, 2021

@busunkim96 How do we keep owlbot from undoing these changes in a future PR?

@busunkim96 busunkim96 closed this Oct 1, 2021
busunkim96 added a commit to googleapis/synthtool that referenced this pull request Oct 5, 2021
…le (#1232)

* Use trampoline_v2 ([Docker CI Helper](https://github.com/GoogleCloudPlatform/docker-ci-helper)) for all python samples.
* Optionally allow samples to run with a Dockerfile specified in the library repository at `.kokoro/docker/samples`. On a successful test run, this Dockerfile is uploaded to `gcr.io/cloud-devrel-kokoro-resources/{repo-name}-samples-docker`. The option is enabled by passing `custom_samples_dockerfile=True` to `common.py_library` in `owlbot.py`.

    ```py
     templated_files = common.py_library(microgenerator=True, custom_samples_dockerfile=True)
    ```

See googleapis/python-asset#308 for an example of moving samples to to trampoline_v2.

See googleapis/python-recaptcha-enterprise#128 for an example of moving to trampoline_v2 + using a custom Dockerfile.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: cloudasset Issues related to the googleapis/python-asset API. cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing. owlbot:ignore instruct owl-bot to ignore a PR samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants