Skip to content

resource_monitors: remove well_known_names#16761

Merged
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
daixiang0:resource
Jun 2, 2021
Merged

resource_monitors: remove well_known_names#16761
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
daixiang0:resource

Conversation

@daixiang0
Copy link
Copy Markdown
Member

@daixiang0 daixiang0 commented Jun 2, 2021

Signed-off-by: Long Dai long0dai@foxmail.com

Commit Message:

Remove well_known_names in resource_monitors field, part of #7238

Additional Description:
Risk Level: LOW
Testing: functional, integration
Docs Changes: NONE
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@daixiang0 daixiang0 requested a review from htuch as a code owner June 2, 2021 02:57
Signed-off-by: Long Dai <long0dai@foxmail.com>
@daixiang0
Copy link
Copy Markdown
Member Author

please have a look @mattklein123

Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit 9a1387c into envoyproxy:main Jun 2, 2021
@daixiang0 daixiang0 deleted the resource branch June 3, 2021 00:52
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: Long Dai <long0dai@foxmail.com>
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.

2 participants