Skip to content
This repository has been archived by the owner on Sep 16, 2023. It is now read-only.

docs: clarify that project number is used as the canonical project identifier for job and job template names #247

Merged
merged 3 commits into from Oct 8, 2021

Conversation

gcf-owl-bot[bot]
Copy link
Contributor

@gcf-owl-bot gcf-owl-bot bot commented Oct 5, 2021

docs: fix broken link and code formatting
feat: add support for allow_missing param on DELETE request
fix!: remove Encryption settings that were published due to a sync issue
BREAKING CHANGE: requests specifying Encryption settings are rejected by the server

  • Regenerate this pull request now.

PiperOrigin-RevId: 401033248

Source-Link: googleapis/googleapis@ef9130f

Source-Link: https://github.com/googleapis/googleapis-gen/commit/9ebec3346a64690c063eb72f1b1c1d6d279177ce
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOWViZWMzMzQ2YTY0NjkwYzA2M2ViNzJmMWIxYzFkNmQyNzkxNzdjZSJ9

…entifier for job and job template names docs: fix broken link and code formatting feat: add support for allow_missing param on DELETE request fix!: remove Encryption settings that were published due to a sync issue BREAKING CHANGE: requests specifying Encryption settings are rejected by the server

PiperOrigin-RevId: 401033248

Source-Link: googleapis/googleapis@ef9130f

Source-Link: googleapis/googleapis-gen@9ebec33
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOWViZWMzMzQ2YTY0NjkwYzA2M2ViNzJmMWIxYzFkNmQyNzkxNzdjZSJ9
@gcf-owl-bot gcf-owl-bot bot requested a review from a team as a code owner October 5, 2021 19:48
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Oct 5, 2021
@product-auto-label product-auto-label bot added the api: transcoder Issues related to the googleapis/java-video-transcoder API. label Oct 5, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 5, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 5, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 5, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Oct 5, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 5, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 5, 2021
@Neenu1995 Neenu1995 changed the title docs: clarify that project number is used as the canonical project identifier for job and job template names docs: fix broken link and code formatting feat: add support for allow_missing param on DELETE request fix!: remove Encryption settings that wer... docs: clarify that project number is used as the canonical project identifier for job and job template names Oct 5, 2021
@Neenu1995 Neenu1995 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 5, 2021
@Neenu1995
Copy link
Contributor

verify the breaking change before merging.

@chanseokoh
Copy link
Contributor

verify the breaking change before merging.

Looks like the clirr errors are all about Encryption. The PR description says

fix!: remove Encryption settings that were published due to a sync issue

So I think it was an accident that these were published, and this PR is correcting them.

@chanseokoh
Copy link
Contributor

@Neenu1995 does the above make sense?

@Neenu1995
Copy link
Contributor

Looking at the discussion in the CL, looks like this doesn't break any customers. Also, java is pre GA for this library. So we can take on the change as a minor version bump.

@Neenu1995 Neenu1995 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 8, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Oct 8, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 8, 2021
@chanseokoh chanseokoh added the automerge Merge the pull request once unit tests and other checks pass. label Oct 8, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 8, 2021
@chanseokoh chanseokoh merged commit 27b43b5 into main Oct 8, 2021
@chanseokoh chanseokoh deleted the owl-bot-9dcc8174-76a4-4605-a3cf-5e82378c4fd3 branch October 8, 2021 19:02
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: transcoder Issues related to the googleapis/java-video-transcoder API. cla: yes This human has signed the Contributor License Agreement. owl-bot-copy
Projects
None yet
3 participants