Skip to content

Commit

Permalink
misc: add checking while append new vm
Browse files Browse the repository at this point in the history
This patch will add checking cpu affinity while user click to add new vm.
When I was following client's findings up I found that if I click to add
a new post-launched vm for step 3.Configure settings for scenario and launch
scripts, it failed to show error messages. The current version will check cpu
affinity and serial port for post-launched and hv when creating a new vm, it
wont verify when adding new post-launched & pre-launched vms, it will fail to
save scenario configuration file without any explanation. I've rebuilt and run
configurator, confirmed the checking procedure works.

Signed-off-by: dongpingx <[email protected]>
Tracked-On: projectacrn#8601
Reviewed-by: Junjie Mao [email protected]
  • Loading branch information
dongpingx authored and acrnsi-robot committed Jun 17, 2024
1 parent 5c9e1c0 commit 7739f0e
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,12 @@ export default {
}
maxVMID++;
this.scenario.vm.push(configurator.newVM(maxVMID, load_order))
//checking while adding new vm
let scenarioXMLData = this.scenarioToXML(this.scenario)
let all_errors = configurator.pythonObject.validateScenario(this.board.content, scenarioXMLData)
this.errors = this.translateErrors(all_errors, this.scenario)
this.switchTab(maxVMID)
},
deleteVM() {
Expand Down

0 comments on commit 7739f0e

Please sign in to comment.