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

Error on vm.set from Jobs tab in XO web GUI #5973

Closed
just-333 opened this issue Oct 30, 2021 · 8 comments
Closed

Error on vm.set from Jobs tab in XO web GUI #5973

just-333 opened this issue Oct 30, 2021 · 8 comments

Comments

@just-333
Copy link

vm.set
{
  "id": [
    "fd85ef9f-617c-0caf-4f33-5a7b732b41d3"
  ],
  "memoryMax": 2
}
{
  "code": 10,
  "data": {
    "errors": [
      {
        "code": null,
        "reason": "type",
        "message": "must be string, but is array",
        "property": "@.id"
      }
    ]
  },
  "message": "invalid parameters",
  "name": "XoError",
  "stack": "XoError: invalid parameters
    at Module.invalidParameters (/opt/xo/xo-builds/xen-orchestra-202110301132/packages/xo-common/src/api-errors.js:21:32)
    at Object.call (file:///opt/xo/xo-builds/xen-orchestra-202110301132/packages/xo-server/src/xo-mixins/api.mjs:73:18)
    at Api.callApiMethod (file:///opt/xo/xo-builds/xen-orchestra-202110301132/packages/xo-server/src/xo-mixins/api.mjs:300:19)"
}

How it looks on Premium version of XOA (trial)

vm.set
{
  "id": "fd85ef9f-617c-0caf-4f33-5a7b732b41d3",
  "memoryMax": 2
}
{
@just-333
Copy link
Author

faulty version:
xo-server 5.83.0

XOA premium trial version:
xo-server 5.82.3

@julien-f
Copy link
Member

As the error message says: you passed an array for the id param but a single string is expected.

@just-333
Copy link
Author

just-333 commented Oct 30, 2021

This is created from the web gui tab "Jobs". Thats where I'm getting the error from

jobs-error

@just-333 just-333 changed the title Error on vm.set Error on vm.set from Jobs tab in XO web GUI Oct 30, 2021
pdonias added a commit that referenced this issue Mar 25, 2022
This reverts commit e2e4539.

See #5983
See #5973
See zammad#5844

This change broke other methods' params.
pdonias added a commit that referenced this issue Mar 25, 2022
This reverts commit e2e4539.

See #5983
See #5973
See zammad#5844

This change broke other methods' params.
pdonias added a commit that referenced this issue Mar 29, 2022
@pdonias
Copy link
Member

pdonias commented Apr 14, 2022

Fixed by #5944
See #5983

@pdonias pdonias closed this as completed Apr 14, 2022
@julien-f
Copy link
Member

@pdonias Because your reverted #5944 to fix #5983, shouldn't this issue still be open?

@olivierlambert
Copy link
Member

Re-asking @pdonias before closing :D

@pdonias
Copy link
Member

pdonias commented Sep 14, 2023

@julien-f is investigating a bug that is most likely related to this, so let's wait :)

@julien-f
Copy link
Member

Closing in favor of #7033

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

No branches or pull requests

4 participants