Skip to content

UNCLEANTLY Revert " notifier: adding basic oncall notifications (#298…#30044

Merged
alyssawilk merged 1 commit intoenvoyproxy:mainfrom
alyssawilk:rollback2
Oct 9, 2023
Merged

UNCLEANTLY Revert " notifier: adding basic oncall notifications (#298…#30044
alyssawilk merged 1 commit intoenvoyproxy:mainfrom
alyssawilk:rollback2

Conversation

@alyssawilk
Copy link
Copy Markdown
Contributor

…20)"

This reverts commit c495291.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

…yproxy#29820)"

This reverts commit c495291.

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Oct 9, 2023
@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 @mattklein123

🐱

Caused by: #30044 was opened by alyssawilk.

see: more, trace.

Copy link
Copy Markdown
Contributor

@jmarantz jmarantz left a comment

Choose a reason for hiding this comment

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

blindly approving with no idea of context.

@phlax
Copy link
Copy Markdown
Member

phlax commented Oct 9, 2023

as commented offline - i think this is not needed - i believe the ci issue should be fixed (i dont think it ran again yet)

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

please dont land w/out confirming its not fixed!

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @alyssawilk

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Oct 9, 2023
@alyssawilk alyssawilk merged commit e419b26 into envoyproxy:main Oct 9, 2023
@alyssawilk alyssawilk deleted the rollback2 branch March 19, 2024 19:46
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.

4 participants