[docker] Fix Test-Node default activation#401
Merged
dean-amar merged 31 commits intoMar 17, 2026
Conversation
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
dean-amar
changed the title
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
dean-amar
requested review from
cendhu and
liran-funaro
and removed request for
cendhu and
liran-funaro
cendhu
reviewed
Mar 10, 2026
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
cendhu
reviewed
Mar 11, 2026
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
All database instances now share the same pre-generated TLS credentials. The `chown root:root /creds/*` command caused race conditions when multiple parallel test instances modified the same shared certificate files, leading to intermittent YugabyteDB startup hangs. Removed the unnecessary chown commands since Docker bind mounts handle file permissions automatically. Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
liran-funaro
approved these changes
Mar 17, 2026
Contributor
|
@dean-amar please rebase |
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
Signed-off-by: Dean Amar <Dean.Amar@ibm.com>
This was
linked to
issues
Apr 6, 2026
[sidecar testing] Start the sidecar using the private key and certificate from crypto material.
#319
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of change
Description
This PR configures TLS certificate paths from the config block for all services in the test node Docker image, enabling proper mTLS communication between components by default.
Related issues