Skip to content

Comments

Add jmxPasswordFilePath option#190

Merged
michaelsembwever merged 1 commit intomasterfrom
mck/Scharlotten/jmxfilepathfix
Oct 20, 2025
Merged

Add jmxPasswordFilePath option#190
michaelsembwever merged 1 commit intomasterfrom
mck/Scharlotten/jmxfilepathfix

Conversation

@michaelsembwever
Copy link
Member

updates to #183

Updated input parameters and added jmxPasswordFilePath variable to input files and updated nodetoolCredentials logic in the ds-collector bash script added the option to use the file

Copy link
Contributor

@Scharlotten Scharlotten left a comment

Choose a reason for hiding this comment

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

Thanks - the PR is only complete with ds-collector update as well - approving it

When provided extracts and sets the jmxPassword from jmxPasswordFilePath
@michaelsembwever michaelsembwever force-pushed the mck/Scharlotten/jmxfilepathfix branch from 4c65fd5 to 03bbd2f Compare October 20, 2025 08:30
@michaelsembwever michaelsembwever merged commit 87b1072 into master Oct 20, 2025
0 of 3 checks passed
@michaelsembwever michaelsembwever deleted the mck/Scharlotten/jmxfilepathfix branch October 20, 2025 08:31
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