Skip to content

refresh certs for tls and ocsp tests#23931

Closed
wbpcode wants to merge 4 commits intoenvoyproxy:release/v1.22from
wbpcode:dev-fix-certs-release-1.22
Closed

refresh certs for tls and ocsp tests#23931
wbpcode wants to merge 4 commits intoenvoyproxy:release/v1.22from
wbpcode:dev-fix-certs-release-1.22

Conversation

@wbpcode
Copy link
Copy Markdown
Member

@wbpcode wbpcode commented Nov 10, 2022

Signed-off-by: wbpcode wangbaiping@corp.netease.com

Commit Message: refresh certs for tls and ocsp tests
Additional Description:

refresh certs for tls and ocsp tests

Risk Level: low. test only.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

wbpcode added 2 commits November 10, 2022 14:36
Signed-off-by: wbpcode <wangbaiping@corp.netease.com>
@wbpcode
Copy link
Copy Markdown
Member Author

wbpcode commented Nov 10, 2022

cc @phlax here for complete CI running.

@phlax
Copy link
Copy Markdown
Member

phlax commented Nov 10, 2022

i think you will need at least #23220 to get that far in CI

Signed-off-by: Xie Zhihao <zhihao.xie@intel.com>
@wbpcode
Copy link
Copy Markdown
Member Author

wbpcode commented Nov 10, 2022

i think you will need at least #23220 to get that far in CI

I cherry picked it. But a seperated backport should be necessary for it.

@phlax
Copy link
Copy Markdown
Member

phlax commented Nov 10, 2022

I cherry picked it. But a seperated backport should be necessary for it.

all of the required backports to fix the branch are on #23842 (with the exception of fix here)

once we can run the branch i will rebase the commits from that PR

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>
@phlax phlax self-assigned this Nov 11, 2022
@wbpcode
Copy link
Copy Markdown
Member Author

wbpcode commented Nov 12, 2022

cc @phlax feel free to close this PR when you complete the related pick. Thanks.

@phlax
Copy link
Copy Markdown
Member

phlax commented Nov 15, 2022

thanks @wbpcode for help!

@phlax phlax closed this Nov 15, 2022
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.

3 participants