{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/37996","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/37996/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/37996/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/37996/events","html_url":"https://github.com/saltstack/salt/issues/37996","id":192620774,"number":37996,"title":"influxdb_user.present does not pass client_args","user":{"login":"stefan-as","id":18169125,"avatar_url":"https://avatars.githubusercontent.com/u/18169125?v=3","gravatar_id":"","url":"https://api.github.com/users/stefan-as","html_url":"https://github.com/stefan-as","followers_url":"https://api.github.com/users/stefan-as/followers","following_url":"https://api.github.com/users/stefan-as/following{/other_user}","gists_url":"https://api.github.com/users/stefan-as/gists{/gist_id}","starred_url":"https://api.github.com/users/stefan-as/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stefan-as/subscriptions","organizations_url":"https://api.github.com/users/stefan-as/orgs","repos_url":"https://api.github.com/users/stefan-as/repos","events_url":"https://api.github.com/users/stefan-as/events{/privacy}","received_events_url":"https://api.github.com/users/stefan-as/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-30T16:57:43Z","updated_at":"2016-12-01T17:25:15Z","closed_at":null,"body":"### Description of Issue/Question\r\n\r\nThere are several calls to influxdb execution modules missing the **client_args, what results in failing state runs in cases where additional client arguments are needed to connect to the database.\r\n\r\nhttps://github.com/saltstack/salt/blob/2016.11/salt/states/influxdb_user.py#L24\r\nhttps://github.com/saltstack/salt/blob/2016.11/salt/states/influxdb_user.py#L81\r\n[...]\r\n\r\n### Setup\r\n\r\n```\r\ninfluxdb_user.present:\r\n    - name: sensu\r\n    - password: foobar\r\n    - host: ip6-localhost\r\n```\r\n\r\n### Steps to Reproduce Issue\r\n\r\nsalt-call state.apply\r\n(spot the host value in traceback)\r\n\r\n```\r\n----------\r\n          ID: sensu_influxdb\r\n    Function: influxdb_user.present\r\n        Name: sensu\r\n      Result: False\r\n     Comment: An exception occurred in this state: Traceback (most recent call last):\r\n                File \"/usr/lib/python2.7/dist-packages/salt/state.py\", line 1744, in call\r\n                  **cdata['kwargs'])\r\n                File \"/usr/lib/python2.7/dist-packages/salt/loader.py\", line 1702, in wrapper\r\n                  return f(*args, **kwargs)\r\n                File \"/usr/lib/python2.7/dist-packages/salt/states/influxdb_user.py\", line 72, in present\r\n                  changes = _changes(name, admin)\r\n                File \"/usr/lib/python2.7/dist-packages/salt/states/influxdb_user.py\", line 24, in _changes\r\n                  existing_user = __salt__['influxdb.user_info'](name)\r\n                File \"/usr/lib/python2.7/dist-packages/salt/modules/influx.py\", line 198, in user_info\r\n                  matching_users = (user for user in list_users(**client_args)\r\n                File \"/usr/lib/python2.7/dist-packages/salt/modules/influx.py\", line 163, in list_users\r\n                  return client.get_list_users()\r\n                File \"/usr/lib/python2.7/dist-packages/influxdb/client.py\", line 633, in get_list_users\r\n                  return list(self.query(\"SHOW USERS\").get_points())\r\n                File \"/usr/lib/python2.7/dist-packages/influxdb/client.py\", line 325, in query\r\n                  expected_response_code=expected_response_code\r\n                File \"/usr/lib/python2.7/dist-packages/influxdb/client.py\", line 241, in request\r\n                  raise e\r\n              ConnectionError: HTTPConnectionPool(host='localhost', port=8086): Max retries exceeded with url: /query?q=SHOW+USERS (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x7f45d18b8350>: Failed to establish a new connection: [Errno 111] Connection refused',))\r\n     Started: 14:10:02.779676\r\n    Duration: 9098882.04 ms\r\n     Changes:   \r\n----------\r\n```\r\n\r\n### Versions Report\r\n\r\n```\r\nSalt Version:\r\n           Salt: 2016.11.0\r\n \r\nDependency Versions:\r\n           cffi: Not Installed\r\n       cherrypy: Not Installed\r\n       dateutil: 2.4.2\r\n          gitdb: 0.6.4\r\n      gitpython: 1.0.1\r\n          ioflo: Not Installed\r\n         Jinja2: 2.8\r\n        libgit2: Not Installed\r\n        libnacl: Not Installed\r\n       M2Crypto: 0.21.1\r\n           Mako: 1.0.3\r\n   msgpack-pure: Not Installed\r\n msgpack-python: 0.4.6\r\n   mysql-python: Not Installed\r\n      pycparser: Not Installed\r\n       pycrypto: 2.6.1\r\n         pygit2: Not Installed\r\n         Python: 2.7.12 (default, Nov 19 2016, 06:48:10)\r\n   python-gnupg: Not Installed\r\n         PyYAML: 3.11\r\n          PyZMQ: 15.2.0\r\n           RAET: Not Installed\r\n          smmap: 0.9.0\r\n        timelib: Not Installed\r\n        Tornado: 4.2.1\r\n            ZMQ: 4.1.4\r\n \r\nSystem Versions:\r\n           dist: Ubuntu 16.04 xenial\r\n        machine: x86_64\r\n        release: 4.4.0-47-generic\r\n         system: Linux\r\n        version: Ubuntu 16.04 xenial\r\n```"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/264236725","html_url":"https://github.com/saltstack/salt/issues/37996#issuecomment-264236725","issue_url":"https://api.github.com/repos/saltstack/salt/issues/37996","id":264236725,"user":{"login":"Ch3LL","id":6886822,"avatar_url":"https://avatars.githubusercontent.com/u/6886822?v=3","gravatar_id":"","url":"https://api.github.com/users/Ch3LL","html_url":"https://github.com/Ch3LL","followers_url":"https://api.github.com/users/Ch3LL/followers","following_url":"https://api.github.com/users/Ch3LL/following{/other_user}","gists_url":"https://api.github.com/users/Ch3LL/gists{/gist_id}","starred_url":"https://api.github.com/users/Ch3LL/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ch3LL/subscriptions","organizations_url":"https://api.github.com/users/Ch3LL/orgs","repos_url":"https://api.github.com/users/Ch3LL/repos","events_url":"https://api.github.com/users/Ch3LL/events{/privacy}","received_events_url":"https://api.github.com/users/Ch3LL/received_events","type":"User","site_admin":false},"created_at":"2016-12-01T17:25:15Z","updated_at":"2016-12-01T17:25:15Z","body":"@stefan-as looks like this was added here: 932fc681  and is new in 2016.11.0.\r\n\r\nLooks to be an easy enough fix. If you want to take a stab please feel free. Also is it just that `influxdb_user` state module that is effected by this?"}}
{"actor":{"display_login":"Ch3LL"}}