-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Implement REPLACE operation for EnvoyFilter patch #27426
Implement REPLACE operation for EnvoyFilter patch #27426
Conversation
/retest |
Add a "REPLACE" operation for EnvoyFilter patch. This operation only supports Network filter and HTTP filter. This is the implementation of issue istio#27425 Signed-off-by: huabing zhao <zhaohuabing@gmail.com>
01d5ba5
to
70fd431
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this.
pilot/pkg/networking/core/v1alpha3/envoyfilter/listener_patch_test.go
Outdated
Show resolved
Hide resolved
Add debug info when no matching filter found. Add negative test. Change test case name to indicate http filter. Signed-off-by: huabing zhao <zhaohuabing@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
- 27425 | ||
releaseNotes: | ||
- | | ||
**Added** "REPLACE" operation for EnvoyFilter. "REPLACE" opration can replace contents of a named filter with new contents. It is only valid for HTTP_FILTER and NETWORK_FILTER. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
**Added** "REPLACE" operation for EnvoyFilter. "REPLACE" opration can replace contents of a named filter with new contents. It is only valid for HTTP_FILTER and NETWORK_FILTER. | |
**Added** "REPLACE" operation for EnvoyFilter. "REPLACE" operation can replace contents of a named filter with new contents. It is only valid for HTTP_FILTER and NETWORK_FILTER. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the typo, how could I fix it? This has already been merged.
You can send another PR I guess. Sorry didn't see it had been merged.
Thanks for the PR!
…On Mon, Sep 21, 2020 at 6:38 PM zhaohuabing ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In releasenotes/notes/27425.yaml
<#27426 (comment)>:
> @@ -0,0 +1,8 @@
+apiVersion: release-notes/v2
+kind: bug-fix
+area: traffic-management
+issue:
+ - 27425
+releaseNotes:
+ - |
+ **Added** "REPLACE" operation for EnvoyFilter. "REPLACE" opration can replace contents of a named filter with new contents. It is only valid for HTTP_FILTER and NETWORK_FILTER.
Sorry for the typo, how could I fix it? This has already been merged.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#27426 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEYGXJZ3WRDGFTH7R73DMLSG752DANCNFSM4RTBD4NA>
.
|
Add a "REPLACE" operation for EnvoyFilter patch. This operation only supports Network filter and HTTP filter.
This is the implementation of issue #27425
Signed-off-by: huabing zhao zhaohuabing@gmail.com
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[*] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.