Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Disable OwlBot PRs. #10696

Merged
merged 4 commits into from
Apr 15, 2024
Merged

chore: Disable OwlBot PRs. #10696

merged 4 commits into from
Apr 15, 2024

Conversation

blakeli0
Copy link
Contributor

Why

Now that we set up daily hermetic build generation which would creates one PR for googleapis proto changes for all services, and it has been working consistently for the last ~4 weeks, we can safely disable OwlBot PRs.

How

OwlBot is installed org wide and we can not disable it for a single repo. The bot uses the existence of .OwlBot.yaml to determine if a PR needs to be created. Hence, in order to disable OwlBot PRs, all .OwlBot.yaml files are renamed to .Owlbot-hermetic.yaml, all references to .OwlBot.yaml are also updated.

Next

Once this PR is merged, the hermetic build generation will start to fail due to the assumption in hermetic build scripts that each library must contains a .OwlBot.yaml file. A follow up PR will be created in sdk-platform-java to address it.

@blakeli0 blakeli0 requested a review from a team as a code owner April 13, 2024 03:28
@JoeWang1127
Copy link
Contributor

JoeWang1127 commented Apr 15, 2024

Do you use an command to rename the files? Could you add the command in the description?

@blakeli0
Copy link
Contributor Author

Do you use an command to rename the files? Could you add the command in the description?

Yes, this is the command I used
find . -name ".OwlBot.yaml" -type f -exec bash -c 'mv "$0" "${0%.OwlBot.yaml}.OwlBot-hermetic.yaml"' {} \

@blakeli0 blakeli0 merged commit 8cac6d6 into main Apr 15, 2024
32 of 33 checks passed
@blakeli0 blakeli0 deleted the disable-owlbot branch April 15, 2024 21:00
blakeli0 added a commit to googleapis/sdk-platform-java that referenced this pull request Apr 16, 2024
…ml. (#2642)

This is follow up of
googleapis/google-cloud-java#10696. The hermetic
build generation process would fail without this PR due to unable to
find OwlBot.yaml files.
This PR changes all the references of `OwlBot.yaml` to
`OwlBot-hermetic.yaml`, removes `OwlBot.yaml` from a few places that
don't need to reference it in the first places.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants