Skip to content

Comments

fix list_cassandra_nodes_client to use config file when running ds-collector -L#191

Merged
michaelsembwever merged 1 commit intomasterfrom
mck/fix-ds-collector-L-conf
Nov 4, 2025
Merged

fix list_cassandra_nodes_client to use config file when running ds-collector -L#191
michaelsembwever merged 1 commit intomasterfrom
mck/fix-ds-collector-L-conf

Conversation

@michaelsembwever
Copy link
Member

@PeterGilroy
Copy link

The nodetool command is now able to run successfully, however the command that is run now returns:

$ ssh automaton@10.10.10.10 -oStrictHostKeyChecking=no -A -o ConnectTimeout=2 -o ConnectionAttempts=1 -n -t '/tmp/datastax/ds-collector -L -f /tmp/datastax/collector.conf'
Pseudo-terminal will not be allocated because stdin is not a terminal.
read configuration from file: completed OK RC=0
10.10.10.10

as the script is taking 'read', 'configuration', 'from' etc. as separate node names this gives many name resolution errors:

$ ./ds-collector -f collector.conf -n 10.10.10.10 -T
read configuration from file: completed OK RC=0
version: master bb44f2f66aca0d9ac0d3542fb0cf1eb12e32fbd5

starting connection test to 10.10.10.10
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '10.10.10.10' (ED25519) to the list of known hosts.
Mon, 03 Nov 2025 11:51:05 +0000
running command date -R on 10.10.10.10: completed OK RC=0
Pseudo-terminal will not be allocated because stdin is not a terminal.
Pseudo-terminal will not be allocated because stdin is not a terminal.
ds-collector                                                                                                                                                   100%   46KB  93.3MB/s   00:00    
push ./ds-collector to host 10.10.10.10: completed OK RC=0
push ./ds-collector to host 10.10.10.10: completed OK RC=0
collector.conf                                                                                                                                                 100% 4807    16.0MB/s   00:00    
push collector.conf to host 10.10.10.10: completed OK RC=0
push collector.conf to host 10.10.10.10: completed OK RC=0
Pseudo-terminal will not be allocated because stdin is not a terminal.
Pseudo-terminal will not be allocated because stdin is not a terminal.
Pseudo-terminal will not be allocated because stdin is not a terminal.

starting connection test to read
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname read: Temporary failure in name resolution
running command date -R on read: completed NOTOK RC=255

starting connection test to configuration
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname configuration: Temporary failure in name resolution
running command date -R on configuration: completed NOTOK RC=255

starting connection test to from
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname from: Temporary failure in name resolution
running command date -R on from: completed NOTOK RC=255

starting connection test to file:
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname file:: Name or service not known
running command date -R on file:: completed NOTOK RC=255

starting connection test to completed
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname completed: Temporary failure in name resolution
running command date -R on completed: completed NOTOK RC=255

starting connection test to OK
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname ok: Temporary failure in name resolution
running command date -R on OK: completed NOTOK RC=255

starting connection test to RC=0
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname rc=0: Name or service not known
running command date -R on RC=0: completed NOTOK RC=255

starting connection test to 10.10.10.10
Pseudo-terminal will not be allocated because stdin is not a terminal.
Mon, 03 Nov 2025 11:51:22 +0000
running command date -R on 10.10.10.10: completed OK RC=0

So will need to add an if listNodesMode="false" to line 1056:

@PeterGilroy
Copy link

lgtm

@michaelsembwever michaelsembwever force-pushed the mck/fix-ds-collector-L-conf branch from b7ca64d to 4587f97 Compare November 4, 2025 10:45
@michaelsembwever michaelsembwever merged commit a42ce9a into master Nov 4, 2025
3 checks passed
@michaelsembwever michaelsembwever deleted the mck/fix-ds-collector-L-conf branch November 4, 2025 13:02
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