Isilon / isilon_data_insights_connector

Connector to fetch stats from OneFS and push them to InfluxDB/Grafana
MIT License
76 stars 36 forks source link

unexpected keyword argument server_hostname #74

Closed jjprachyl closed 3 years ago

jjprachyl commented 5 years ago

getting this error when attempting to start isi data insights py

using sdk 7.2 unexpected keyword argument ‘server_hostname’

How do i get past this? i can’t get it to start

tenortim commented 5 years ago

Hi @jjprachyl, could you please list the details of how you are starting the collector and the full error message (I presume there's a stack trace?).

jjprachyl commented 5 years ago

Starting with the command “./isi_data_insights_d.py start”

it asks for credentials and validate cert (n)

then “Failed to configure SDK for cluster x.x.x.x using sdk isi_sdk_7_2. Error: init() got an unexpected keyword argument ‘server_hostname’

tenortim commented 5 years ago

Hi @jjprachyl , I'm completely unable to reproduce the issue. "server_hostname" isn't in the source code for the data insights collector, nor is it in the SDK that I am aware of. Could you provide details on how you are trying to set this up? What OS/distro are you installing on? Did you follow the instructions from README.md, and if so, which way did you try to set this up? Would you be willing to upload a diff of your config file against the example config file provided (please obsfuscate any sensitive information such as passwords).

jjprachyl commented 5 years ago

Wow, don’t know what I did. I started over and ran into a lot of the issues in the issues section. Re-ran the requirements with sudo after upgrading setuptools. The instructions could be updated from Keith’s 2017 blog post. That would be helpful. I’m going to start over again and try to document my process to get an error free run through. Thanks for taking a look tenortim!

tenortim commented 5 years ago

Thanks @jjprachyl, yes, updating the instructions is high on my list of updates that need to be made. I'm also looking into provide a dockerized version to simplify this.

tenortim commented 3 years ago

Closing this old issue. Please re-open or open a new one if there are problems.