Skip to content

Conversation

kezhuw
Copy link
Member

@kezhuw kezhuw commented Mar 31, 2025

A fixup to #2224, see #2224 (comment).

Copy link
Member

@ctubbsii ctubbsii left a comment

Choose a reason for hiding this comment

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

Nice catch! I missed this one. I did a quick check to see if there were any others with "x and couldn't find any.

@kezhuw
Copy link
Member Author

kezhuw commented Mar 31, 2025

Jenkins displayed as failure due to no changes to java sources.

The first run of cppunit failed due to flaky test. https://github.com/apache/zookeeper/actions/runs/14178293310/job/39718358388#step:6:1521

I am going to merge this as it is simple and master ci fails constantly.

cc @ctubbsii @tisonkun @anmolnar

@kezhuw kezhuw merged commit b86ccf1 into apache:master Mar 31, 2025
15 of 16 checks passed
@kezhuw kezhuw deleted the ZOOKEEPER-3766-hotfix-fix-TestSASLAuth-failure branch April 6, 2025 09:57
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.

2 participants