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.

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

[Bug]: sq-poller timeout type mismatch with https transport #956

Closed
gioccher opened this issue May 28, 2024 · 0 comments · Fixed by #955
Closed

[Bug]: sq-poller timeout type mismatch with https transport #956

gioccher opened this issue May 28, 2024 · 0 comments · Fixed by #955
Labels
bug Something isn't working

Comments

@gioccher
Copy link

Suzieq version

0.23.0

Install Type

container

Python version

3.8

Impacted component

sq-poller

Steps to Reproduce

use https transport

snippet from inventory.yml

devices:
- name: arista
  transport: https
  devtype: eos
  port: 443

Expected Behavior

[WORKER 0]: 2024-05-28 20:31:59,207 - suzieq.poller.worker.nodes.node - WARNING - eos supplied for 10.1.2.3:443
[WORKER 0]: 2024-05-28 20:31:59,447 - suzieq.poller.worker.nodes.node - INFO - 10.1.2.3:443 exec: ['show version', 'show hostname']
[WORKER 0]: 2024-05-28 20:31:59,697 - suzieq.poller.worker.inventory.inventory - INFO - Added node device.example.net:443 of type eos

Observed Behavior

[WORKER 0]: 2024-05-28 20:23:02,758 - suzieq.poller.worker.nodes.node - WARNING - eos supplied for 10.1.2.3:443
[WORKER 0]: 2024-05-28 20:23:02,758 - suzieq.poller.worker.nodes.node - ERROR - Unable to connect to 10.1.2.3:443, error: timeout parameter cannot be of <class 'int'> type, please use 'timeout=ClientTimeout(...)'
[WORKER 0]: 2024-05-28 20:23:02,758 - suzieq.poller.worker.nodes.node - INFO - Connection to 10.1.2.3:443 will be retried from 2024-05-28 20:24:02.758817

Screenshots

Additional Context

@gioccher gioccher added the bug Something isn't working label May 28, 2024
@gioccher gioccher changed the title [Bug]: sq-poller timeout type mismatch [Bug]: sq-poller timeout type mismatch with https transport May 28, 2024
@claudious96 claudious96 linked a pull request May 29, 2024 that will close this issue
5 tasks
@ddutt ddutt closed this as completed in #955 Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant