Skip to content

veeam.py: ValueError: need more than 2 values to unpack #45

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
Closed
NavidSassan opened this issue Jan 18, 2022 · 4 comments
Closed

veeam.py: ValueError: need more than 2 values to unpack #45

NavidSassan opened this issue Jan 18, 2022 · 4 comments
Assignees
@markuslf

Comments

@NavidSassan
Copy link
Member

In GitLab by @Simon.Brenner on Jan 18, 2022, 14:36

with the new "libraries". the check-veeam plugin brings the error:

File "/usr/lib64/nagios/plugins/veeam-status", line 331, in 'module'
    main()
  File "/usr/lib64/nagios/plugins/veeam-status", line 143, in main
    success, result, token = lib.veeam2.get_token(args)
  File "/usr/lib64/nagios/plugins/lib/veeam2.py", line 43, in get_token
    timeout=args.TIMEOUT, insecure=True, extended=True)
ValueError: need more than 2 values to unpack

check command:
'/usr/lib64/nagios/plugins/veeam-status' '--critical' '90' '--password' '' '--timeout' '3' '--url' 'url' '--username' '' '--warning' '80'
@NavidSassan
Copy link
Member Author

In GitLab by @markuslf on Jan 20, 2022, 17:13

mentioned in commit aa08cea

@NavidSassan
Copy link
Member Author

In GitLab by @markuslf on Jan 20, 2022, 17:14

The error is fixed for now. Unfortunately, I can't test if the check really works because our Veeam trial license has expired.
The veeam-status checks and the veeam.py libs must be updated together.

@NavidSassan
Copy link
Member Author

In GitLab by @markuslf on Jan 21, 2022, 15:16

mentioned in commit 8e1b9b9

@NavidSassan
Copy link
Member Author

mentioned in commit 4538120

NavidSassan added a commit that referenced this issue Mar 1, 2022
@NavidSassan
Merge branch 'develop' into 'master'
4538120 
new release

Closes #30, #40, #41, #43, #44, #42, #46, #45, #47, #49, #50, #52, #51, #53, #54, #55, #56, #57, #59, #60, #61, and #62

See merge request linuxfabrik/lib!9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markuslf markuslf

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@markuslf @NavidSassan