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

Problems with version 5.3 #67

Open
shibumi opened this issue Feb 1, 2019 · 3 comments · May be fixed by #68
Open

Problems with version 5.3 #67

shibumi opened this issue Feb 1, 2019 · 3 comments · May be fixed by #68

Comments

@shibumi
Copy link

shibumi commented Feb 1, 2019

Hi,
I get with the newest proxmox version the following errors:

Traceback (most recent call last):
  File "/root/proxmox-dokuwiki-exporter.py", line 27, in <module>
    for node in proxmox.nodes.get():
  File "/root/.local/lib/python3.5/site-packages/proxmoxer/core.py", line 84, in get
    return self(args)._request("GET", params=params)
  File "/root/.local/lib/python3.5/site-packages/proxmoxer/core.py", line 79, in _request
    resp.content))
proxmoxer.core.ResourceException: 500 Internal Server Error: 

As message In the server error I get a full graph of my cluster (here redacted). what the hell?!

The line that triggers this error is this one here: for node in proxmox.nodes.get():

@pskrzyns
Copy link

pskrzyns commented Feb 5, 2019

@shibumi new version of pvesh has a few changes:

  • by default it returns ACII text instead of json (you need to add --format-output json to make it work as before)
  • if request is correct it doesn't return response code which is exptected by proxmoxer (resulting with 500 internal error)

#59 should fix it

@pskrzyns pskrzyns linked a pull request Feb 5, 2019 that will close this issue
@shibumi
Copy link
Author

shibumi commented Feb 5, 2019

@pskrzyns i use proxmoxer as library for my own script. No idea where I should include this --format-output json..

@pskrzyns
Copy link

pskrzyns commented Feb 6, 2019

@shibumi if u don't want to wait for merge of #68 and new lib version you can:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants