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

Weird XO Error Message When VM Started and All Servers Disabled #6316

Closed
evoelise opened this issue Jul 4, 2022 · 2 comments · Fixed by #6408
Closed

Weird XO Error Message When VM Started and All Servers Disabled #6316

evoelise opened this issue Jul 4, 2022 · 2 comments · Fixed by #6408
Assignees

Comments

@evoelise
Copy link

evoelise commented Jul 4, 2022

XO from the sources

When an attempt to start a VM is done when all servers in the pool are disabled a weird error message is shown.

At first just a single letter is shown "g" when the VM is started, when the error is clicked to view in the error log then the following is shown - with the use name as the error.

Screenshot 2022-07-02 213826

It appears this happens as the actual error has no message:

vm.start
{
  "id": "122162af-de31-8a27-b78c-1**********f",
  "bypassMacAddressesCheck": false,
  "force": false
}
{
  "errors": [
    "XCPNG02: HOST_DISABLED(OpaqueRef:a1e9a8f3-0a79-4824-b29f-d**********0)",
    "XCPNG01: HOST_DISABLED(OpaqueRef:9ae9d4e6-3928-44c0-ad64-0**********e)",
    "XCPNG03: HOST_DISABLED(OpaqueRef:484dc396-2d90-497e-a35a-1**********c)"
  ],
  "message": "",
  "name": "Error",
  "stack": "Error: 
    at Xapi._startVm (file:///opt/xo/xo-builds/xen-orchestra-202206111352/packages/xo-server/src/xapi/index.mjs:880:15)
    at Xapi.startVm (file:///opt/xo/xo-builds/xen-orchestra-202206111352/packages/xo-server/src/xapi/index.mjs:916:7)
    at Api.callApiMethod (file:///opt/xo/xo-builds/xen-orchestra-202206111352/packages/xo-server/src/xo-mixins/api.mjs:310:20)"
}

@olivierlambert
Copy link
Member

Hi,

  1. It's normal since a host in maintenance mode will refuse to start any new VM
  2. Indeed, the display is maybe problematic in that case. Pinging @Rajaa-BARHTAOUI to take a look.

@julien-f
Copy link
Member

julien-f commented Sep 2, 2022

I believe simply passing an error message here should be enough:

throw new AggregateError(

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

Successfully merging a pull request may close this issue.

5 participants