add unmarshal error to EnvoyPatchPolicy status

[zhaohuabing]

What type of PR is this?

This PR add unmarshal error to EnvoyPatchPolicy status to make our lives a bit easier :-)

Before:

  - lastTransitionTime: "2023-09-01T02:48:06Z"
    message: unable to unmarshal xds resource {"name":"teg-e2e-test/teg/https","virtual_hosts":[{"name":"teg-e2e-test/teg/https/www_example_com","domains":["www.example.com"],"routes":[{"name":"httproute/teg-e2e-test/http-oidc-authn-policy/rule/0/match/0/www_example_com","match":{"prefix":"/"},"route":{"cluster":"httproute/teg-e2e-test/http-oidc-authn-policy/rule/0"}}],"typed_per_filter_config":{"envoy.filters.http.dynamic_forward_proxy":{"allow_headers":"keep-alive,user-agent,cache-control,content-type,content-transfer-encoding,custom-header-1,x-accept-content-transfer-encoding,x-accept-response-streaming,x-user-agent,x-grpc-web,grpc-timeout","allow_methods":"GET,
      PUT, DELETE, POST, OPTIONS","allow_origin_string_match":[{"prefix":"*"}],"expose_headers":"custom-header-1,grpc-status,grpc-message"}}}],"ignore_port_in_host_matching":true}
    reason: Invalid
    status: "False"
    type: Programmed

After:

  - lastTransitionTime: "2023-09-01T02:56:50Z"
    message: 'unable to unmarshal xds resource {"name":"teg-e2e-test/teg/https","virtual_hosts":[{"name":"teg-e2e-test/teg/https/www_example_com","domains":["www.example.com"],"routes":[{"name":"httproute/teg-e2e-test/http-oidc-authn-policy/rule/0/match/0/www_example_com","match":{"prefix":"/"},"route":{"cluster":"httproute/teg-e2e-test/http-oidc-authn-policy/rule/0"}}],"typed_per_filter_config":{"envoy.filters.http.dynamic_forward_proxy":{"allow_headers":"keep-alive,user-agent,cache-control,content-type,content-transfer-encoding,custom-header-1,x-accept-content-transfer-encoding,x-accept-response-streaming,x-user-agent,x-grpc-web,grpc-timeout","allow_methods":"GET,
      PUT, DELETE, POST, OPTIONS","allow_origin_string_match":[{"prefix":"*"}],"expose_headers":"custom-header-1,grpc-status,grpc-message"}}}],"ignore_port_in_host_matching":true},
      err:proto: (line 1:397): missing "@type" field'
    reason: Invalid
    status: "False"
    type: Programmed

[Xunzhuo]

Plz resolve the CI errors first : )

[codecov]

Codecov Report

Merging #1859 (e6e0496) into main (a324513) will decrease coverage by 0.02%.
The diff coverage is 12.50%.

@@            Coverage Diff             @@
##             main    #1859      +/-   ##
==========================================
- Coverage   65.24%   65.22%   -0.02%     
==========================================
  Files          86       86              
  Lines       12480    12480              
==========================================
- Hits         8142     8140       -2     
- Misses       3821     3823       +2     
  Partials      517      517              

Files Changed	Coverage Δ
internal/xds/translator/jsonpatch.go	30.10% <12.50%> (ø)

... and 3 files with indirect coverage changes

[zhaohuabing]

@zirain @Xunzhuo is codecov/patch a must for a PR to get merged?

[zirain]

@zirain @Xunzhuo is codecov/patch a must for a PR to get merged?

it's not a must