Skip to content

Add jmxPasswordFilePath option - #190

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

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

Conversation

@michaelsembwever

Copy link
Copy Markdown
Contributor

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

@Scharlotten Scharlotten left a comment

Copy link
Copy Markdown
Contributor

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