Skip to content

dep: update com_github_google_perfetto: v22.1 (current: v21.0)#19259

Merged
rojkov merged 1 commit intoenvoyproxy:mainfrom
Nordix:upgrade-com_github_google_perfetto
Dec 13, 2021
Merged

dep: update com_github_google_perfetto: v22.1 (current: v21.0)#19259
rojkov merged 1 commit intoenvoyproxy:mainfrom
Nordix:upgrade-com_github_google_perfetto

Conversation

@kfaseela
Copy link
Copy Markdown
Contributor

Signed-off-by: Faseela K faseela.k@est.tech

Fixes #19216

@repokitteh-read-only repokitteh-read-only Bot added the deps Approval required for changes to Envoy's external dependencies label Dec 13, 2021
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @moderation

🐱

Caused by: #19259 was opened by kfaseela.

see: more, trace.

@kfaseela kfaseela marked this pull request as draft December 13, 2021 11:29
Signed-off-by: Faseela K <faseela.k@est.tech>
@kfaseela kfaseela force-pushed the upgrade-com_github_google_perfetto branch from 8d5dbfa to e6fbf24 Compare December 13, 2021 13:11
@kfaseela
Copy link
Copy Markdown
Contributor Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #19259 (comment) was created by @kfaseela.

see: more, trace.

@kfaseela kfaseela marked this pull request as ready for review December 13, 2021 17:39
@moderation
Copy link
Copy Markdown
Contributor

@htuch @rojkov we should add an owner for Perfetto to https://github.com/envoyproxy/envoy/blob/main/CODEOWNERS

This change look OK? If so I'll approve the deps

@rojkov rojkov enabled auto-merge (squash) December 13, 2021 19:20
Copy link
Copy Markdown
Member

@rojkov rojkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!
Yes, the change looks good to me.

@rojkov rojkov merged commit ae6192d into envoyproxy:main Dec 13, 2021
@rojkov
Copy link
Copy Markdown
Member

rojkov commented Dec 13, 2021

Heh, I accidentally enabled auto-merge in my github mobile app and the PR got happily merged despite the deps label.

@phlax
Copy link
Copy Markdown
Member

phlax commented Dec 13, 2021

the PR got happily merged despite the deps label.

i thought we had fixed it to not do that

@rojkov
Copy link
Copy Markdown
Member

rojkov commented Dec 13, 2021

I think it's not the machinery's fault: the check is not marked as required. I should have been more cautious with the app: it didn't even give me a chance to edit the commit message.

@phlax
Copy link
Copy Markdown
Member

phlax commented Dec 13, 2021

in this case i thought/hoped i had fixed this specific issue

see here #18814 (comment)

@kfaseela kfaseela deleted the upgrade-com_github_google_perfetto branch January 13, 2022 16:23
joshperry pushed a commit to joshperry/envoy that referenced this pull request Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Newer release available com_github_google_perfetto: v22.1 (current: v21.0)

4 participants