Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating the read me to include more details about the cqlsh_host con… #107

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updating the read me to include more details about the cqlsh_host con… #107

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions ds-collector/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,6 +204,16 @@ sed -i 's/^#?skipS3=.*/skipS3=\"true\"/'
sed -i 's/^#?keepArtifact=.*/keepArtifact=\"true\"/'
```

* If the script fails when attempting to run the `cqlsh` command due to an incorrect host address, an additional configuration item can be uncommented and used to supply a suitable host value. The default behaviour of the setting is that a node will use the `hostname` command to provide the address to use as it is run on the node. Problems occur if this hostname does not resolve to the IP address that the node is listening for connections on. An alternative command can be used instead, which is supplied via configuration.
Copy link
Member

Choose a reason for hiding this comment

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

we need to make mention that it is recommended and normal to first configure cqlHost.
that this is only needed when the hostname cannot be a standard ip/hostname on all nodes, and not $(hostname)

Copy link
Member

Choose a reason for hiding this comment

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

I still curious if this comment/info is better here or here: https://github.com/datastax/diagnostic-collection/blob/master/ds-collector/collector.conf.in#L139


An example of a replacement when running on AWS to use the local private IP is:

```
cqlsh_host="$(curl http://169.254.169.254/latest/meta-data/local-ipv4)"
Copy link
Member

Choose a reason for hiding this comment

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

have we tested this works in the configuration file? if it does, this it would make more sense that the line https://github.com/datastax/diagnostic-collection/blob/master/ds-collector/collector.conf.in#L139
was uncommented like

cqlsh_host="$(hostname)"

```

This will ensure that each node uses it's own internal IP address as the `cqlsh` host address.


What is Collected
=================
Expand Down