Skip to content

build: update .OwlBot.yaml for google-cloud-org-policy#13516

Merged
parthea merged 3 commits into
mainfrom
update-owlbot-yaml-orgpolicy
Feb 11, 2025
Merged

build: update .OwlBot.yaml for google-cloud-org-policy#13516
parthea merged 3 commits into
mainfrom
update-owlbot-yaml-orgpolicy

Conversation

@parthea

@parthea parthea commented Feb 11, 2025

Copy link
Copy Markdown
Contributor

This PR fixes an issue in the .OwlBot.yaml for google-cloud-org-policy where there is an erroneous folder $2 in /owl-bot-staging/google-cloud-org-policy/google/cloud/orgpolicy/v1

To reproduce the issue run the following commands in a clone of https://github.com/googleapis/googleapis

bazelisk build //google/cloud/orgpolicy/v1:orgpolicy-v1-py
bazelisk build //google/cloud/orgpolicy/v2:orgpolicy-v2-py

Then in this repository run the following command to copy code from bazel-bin

docker run --rm --user $(id -u):$(id -g)   -v $(pwd):/repo   -v <path to your bazel-bin>:/bazel-bin   gcr.io/cloud-devrel-public-resources/owlbot-cli:latest copy-bazel-bin   --source-dir /bazel-bin --dest /repo   --config-file=/packages/google-cloud-org-policy/.OwlBot.yaml

and then run the post processor with

docker run --user $(id -u):$(id -g) --rm -v $(pwd):/repo -w /repo gcr.io/cloud-devrel-public-resources/owlbot-python-mono-repo:latest

Repeat the steps again with the updated .OwlBot.yaml file and see that the erroneous folder $2 is no longer present

@parthea parthea marked this pull request as ready for review February 11, 2025 16:41
@parthea parthea requested a review from a team February 11, 2025 16:41
@parthea parthea changed the title build: update .OwlBot.yaml for oogle-cloud-org-policy build: update .OwlBot.yaml for google-cloud-org-policy Feb 11, 2025
@parthea parthea enabled auto-merge (squash) February 11, 2025 17:01
@parthea parthea merged commit 17fd214 into main Feb 11, 2025
@parthea parthea deleted the update-owlbot-yaml-orgpolicy branch February 11, 2025 19:10
@parthea parthea mentioned this pull request Feb 11, 2025
Merged
parthea added a commit that referenced this pull request Feb 11, 2025
@parthea
chore: delete erroneous $2 directory (#13518)
2c92938 
This issue was fixed in
#13516 but
#13499 did not
include this fix. This PR deletes the erroneous folder `$2` added in
#13499
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ohmayr ohmayr ohmayr approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@parthea @ohmayr