Skip to content

Try to enable pipeline test #21816

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.

Sign up for GitHub

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

Jump to bottom
Merged
chenrujun merged 65 commits into from
May 25, 2021
Merged

Try to enable pipeline test #21816

chenrujun merged 65 commits into from
May 25, 2021

Conversation

@zhichengliu12581
Copy link
Contributor

@zhichengliu12581 zhichengliu12581 commented May 25, 2021

Try to enable pipeline test which disabled in #20765

@zhichengliu12581
add key vault dynamic refresh
814d82f
@zhichengliu12581
add static function and update parameter to static
b264a59
@zhichengliu12581
merge master
7f9e831
@zhichengliu12581
add parameter and fix error
8cf927e
@zhichengliu12581
update sample and add tests
fa9c2bd
@zhichengliu12581
merge master
294de13
@zhichengliu12581
implement the delete method in order to add test
78af221
@zhichengliu12581
update README.md
3eedf43
@zhichengliu12581
Extraction method and change test
fd2df3b
@zhichengliu12581
try to fix pipeline error
7cfa907
@zhichengliu12581
remove error context
6d6f6be
@zhichengliu12581
merge master
b799c08
@zhichengliu12581
update with comment
e108259
@zhichengliu12581
revert changes cause change function to refresh certificates
7dbd980
@zhichengliu12581
revert KeyVaultKeyStore
ebb3e70
@zhichengliu12581
Store side load certificates and portal certificates separately
d47a635
@zhichengliu12581
create class to store parameter and provide function to make paramete…
5a018f3 
…r private
@zhichengliu12581
enable over trust manager in server side for refresh certificates
6e93adb
@zhichengliu12581
merge master
ce6aa3a
@zhichengliu12581
add Annotation
bad43a8
@zhichengliu12581
revert unimportant changes for easy review
3fea835
@zhichengliu12581
nothing changed, just format code
1fd57e5
@zhichengliu12581
add properties to enable refresh when have untrusted certificates
f8ab509
@zhichengliu12581
nothing changed, just format code
ac9fcce
@zhichengliu12581
Merge branch 'master' of https://github.com/Azure/azure-sdk-for-java …
ee9a5a4 
…into add-key-vault-dynamic-refresh-parameter
@zhichengliu12581
add test and Override delete function
c379cc9
@zhichengliu12581
Add configuration to cancel the test
7be8c8a
@zhichengliu12581
add tests for certificateInfo
b71aca4
@zhichengliu12581
update names and optimization test with comments
54463fa
@zhichengliu12581
create class to store side load certificatees
0498b85
@zhichengliu12581
Merge branch 'master' of https://github.com/Azure/azure-sdk-for-java …
ea3967f 
…into add-key-vault-dynamic-refresh-parameter
@zhichengliu12581
add temp test class just for find out why error in pipeline
80dc6fa
@zhichengliu12581
Merge branch 'master' of https://github.com/Azure/azure-sdk-for-java …
e584999 
…into add-key-vault-dynamic-refresh-parameter
@zhichengliu12581
Thoroughly modify the test to see if it is an error caused by the code
e8f189c
@zhichengliu12581
try to find way to fix error in pipeline
a84d627
@zhichengliu12581
another try to fix pipeline error by change the way to create SSLContext
e25aa8b
@zhichengliu12581
Update ServerSocketTempTest.java
1a4e0ab
@zhichengliu12581
Merge branch 'master' of https://github.com/Azure/azure-sdk-for-java …
6f9cbd5 
…into add-key-vault-dynamic-refresh-parameter
@zhichengliu12581
revert change cause master have change the getAlias and add log
cbe3c23
@zhichengliu12581
final test to find error
7d8e5b3
@zhichengliu12581
revert change to let pipeline success
d9ffd21
@zhichengliu12581
fix readme error
18f364b
@zhichengliu12581
try to find way to enable pipeline test
0440652
@ghost ghost added KeyVault azure-spring All azure-spring related issues labels May 25, 2021
@chenrujun
Copy link

chenrujun commented May 25, 2021

/azp run java - keyvault - tests

@azure-pipelines
Copy link

azure-pipelines bot commented May 25, 2021

Azure Pipelines successfully started running 1 pipeline(s).

chenrujun
chenrujun approved these changes May 25, 2021
View reviewed changes
Copy link

@chenrujun chenrujun left a comment

Choose a reason for hiding this comment

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

LGTM.

@chenrujun chenrujun merged commit 648912c into Azure:master May 25, 2021
@chenrujun chenrujun mentioned this pull request May 25, 2021
Closed
@zhichengliu12581 zhichengliu12581 mentioned this pull request May 25, 2021
Merged
@zhichengliu12581 zhichengliu12581 deleted the try-to-enable-pipeline-test branch August 24, 2021 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@backwind1233 backwind1233 Awaiting requested review from backwind1233

@g2vinay g2vinay Awaiting requested review from g2vinay

@samvaity samvaity Awaiting requested review from samvaity

@saragluna saragluna Awaiting requested review from saragluna

@stliu stliu Awaiting requested review from stliu

@vcolin7 vcolin7 Awaiting requested review from vcolin7

@yiliuTo yiliuTo Awaiting requested review from yiliuTo

1 more reviewer

@chenrujun chenrujun chenrujun approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

azure-spring All azure-spring related issues KeyVault

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zhichengliu12581 @chenrujun