Skip to content
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

ZOOKEEPER-4227: X509AuthFailureTest is failing consistently #1616

Closed
wants to merge 1 commit into from
Closed

ZOOKEEPER-4227: X509AuthFailureTest is failing consistently #1616

wants to merge 1 commit into from

Conversation

arshadmohammad
Copy link
Contributor

No description provided.

@arshadmohammad arshadmohammad mentioned this pull request Mar 8, 2021
Closed
@arshadmohammad arshadmohammad requested a review from ztzg March 9, 2021 17:26
@arshadmohammad
Copy link
Contributor Author

Tried to merge this change with #1599, but no response. lets review and merge this PR itself

@arshadmohammad
Copy link
Contributor Author

ping @eolivelli @ztzg

ztzg
ztzg approved these changes Mar 9, 2021
View reviewed changes
Copy link
Contributor

@ztzg ztzg left a comment

Choose a reason for hiding this comment

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

LGTM, and passes. (Though I hadn't experienced that failure.)

eolivelli
eolivelli approved these changes Mar 9, 2021
View reviewed changes
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

LGTM

I will merge tomorrow

@eolivelli eolivelli closed this in f52918a Mar 10, 2021
@arshadmohammad
Copy link
Contributor Author

arshadmohammad commented Mar 10, 2021 •
edited
Loading

This PR change is applicable to other branches as well.

@arshadmohammad
Copy link
Contributor Author

PR for branch-3.7 and 3.7.0 is #1635
PR for branch-3.6 is #1636

RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Aug 31, 2022
@arshadmohammad @RokLenarcic
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently
d10c5a2 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227
RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Aug 31, 2022
@arshadmohammad @RokLenarcic
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently
6825640 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227
RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Aug 31, 2022
@arshadmohammad @RokLenarcic
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently
4574d1c 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227
RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Sep 3, 2022
@arshadmohammad @RokLenarcic
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently
3036f45 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227
RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Sep 29, 2022
@arshadmohammad @RokLenarcic
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently
d226752 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227
anuragmadnawat1 pushed a commit to anuragmadnawat1/zookeeper that referenced this pull request Nov 2, 2022
@arshadmohammad @anurag-harness
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently
af98257 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227
@anuragmadnawat1 anuragmadnawat1 mentioned this pull request Nov 2, 2022
Merged
anuragmadnawat1 added a commit to anuragmadnawat1/zookeeper that referenced this pull request Nov 2, 2022
@anuragmadnawat1 @arshadmohammad
ZOOKEEPER-4227: X509AuthFailureTest is failing consistently (#118)
6f7a1f2 
Author: Mohammad Arshad <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1616 from arshadmohammad/ZOOKEEPER-4227

Co-authored-by: Mohammad Arshad <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eolivelli eolivelli eolivelli approved these changes

@ztzg ztzg ztzg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@arshadmohammad @ztzg @eolivelli