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

feat: add service_account_email for export subscriptions #1927

Merged
merged 3 commits into from
Jun 10, 2024
Merged

Conversation

gcf-owl-bot[bot]
Copy link
Contributor

@gcf-owl-bot gcf-owl-bot bot commented May 30, 2024

  • Regenerate this pull request now.

PiperOrigin-RevId: 638641437

Source-Link: googleapis/googleapis@a3016a8

Source-Link: https://github.com/googleapis/googleapis-gen/commit/2001c330aee62a9d4e8a9852d8e5bf2662818ad4
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMjAwMWMzMzBhZWU2MmE5ZDRlOGE5ODUyZDhlNWJmMjY2MjgxOGFkNCJ9

@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 May 30, 2024
@product-auto-label product-auto-label bot added the size: xl Pull request size is extra large. label May 30, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 30, 2024
@product-auto-label product-auto-label bot added the api: pubsub Issues related to the googleapis/nodejs-pubsub API. label May 30, 2024
@gcf-owl-bot gcf-owl-bot bot marked this pull request as ready for review May 30, 2024 15:58
@gcf-owl-bot gcf-owl-bot bot requested review from a team as code owners May 30, 2024 15:58
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: xl Pull request size is extra large. labels May 30, 2024
@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label May 30, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 30, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 30, 2024
@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 May 30, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 30, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 30, 2024
Copy link
Contributor Author

gcf-owl-bot bot commented May 31, 2024

Owl bot is regenerating pull request 1927...
Build name: N2U2NzU2NTQtM2ZkMC00Y2VhLTlkNjUtNGJjZDZmMmVmNDg4

yoshi-approver
yoshi-approver previously approved these changes May 31, 2024
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 31, 2024
@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 May 31, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 31, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 31, 2024
feywind
feywind previously approved these changes May 31, 2024
PiperOrigin-RevId: 638641437

Source-Link: googleapis/googleapis@a3016a8

Source-Link: googleapis/googleapis-gen@2001c33
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMjAwMWMzMzBhZWU2MmE5ZDRlOGE5ODUyZDhlNWJmMjY2MjgxOGFkNCJ9
@gcf-merge-on-green gcf-merge-on-green bot dismissed stale reviews from feywind and yoshi-approver May 31, 2024 19:58

This review does not reference the most recent commit, and you are using the secure version of merge-on-green. Please re-review the most recent commit.

@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: xl Pull request size is extra large. labels May 31, 2024
@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 May 31, 2024
yoshi-approver
yoshi-approver previously approved these changes May 31, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 31, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 31, 2024
@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 May 31, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 31, 2024
@gcf-merge-on-green gcf-merge-on-green bot dismissed yoshi-approver’s stale review May 31, 2024 20:06

This review does not reference the most recent commit, and you are using the secure version of merge-on-green. Please re-review the most recent commit.

@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 31, 2024
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Jun 1, 2024
@feywind feywind merged commit c532854 into main Jun 10, 2024
18 checks passed
@feywind feywind deleted the owl-bot-copy branch June 10, 2024 20:17
sofisl added a commit to googleapis/repo-automation-bots that referenced this pull request Jun 11, 2024
)

I noticed in googleapis/nodejs-pubsub#1927 that
merge-on-green kept dismissing auto-approve's latest review due to
post-processor updates, other commits, etc. Ideally, those commits would
retrigger auto-approve to re-review the PR. But there seems to be a race
condition where sometimes merge-on-green beats auto-approve to
re-approving and dismisses the PR. This fix would allow auto-approve to
listen to reviews dismissed, so that it always has a chance to re-review
the most recent commit when an `automerge: exact` label has been placed.

As part of this work, the pull_request_review.dismissed event needs to
access the pr number through
[`pull_request.number`](https://docs.github.com/en/webhooks/webhook-events-and-payloads#pull_request_review),
so I had to update this for all the events and make sure the test
fixtures had the appropriate number there too.

Fixes #5096
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the googleapis/nodejs-pubsub API. owl-bot-copy size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants