Project

General

Profile

Bug #37958

Hide VNC option if VM has existing VNC device

Added by Brandon Schneider 9 months ago. Updated 9 months ago.

Status:
Done
Priority:
No priority
Assignee:
Vaibhav Chauhan
Category:
GUI (new)
Target version:
11.2-BETA2
Seen in:
Master - FreeNAS Nightlies
Severity:
Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

Currently, we assign the VNC device to slot 29 in the middleware plugin, so if there are two (or more) VNC devices they clash as both cannot occupy the same port. If the VM already has a VNC device, the web UI should not allow them to add another one. Supporting multiple VNC devices is not allowed at this time.

Screenshot_VM-VNC.png (135 KB) Screenshot_VM-VNC.png Aaron St. John, 07/20/2018 07:27 AM
22103

History

#1 Updated by Erin Clark 9 months ago

  • Assignee changed from Erin Clark to Vaibhav Chauhan

#2 Updated by Dru Lavigne 9 months ago

  • Target version changed from Backlog to 11.2-BETA2

#3 Updated by Vaibhav Chauhan 9 months ago

  • Status changed from Unscreened to In Progress

in review:https://github.com/freenas/webui/pull/1021

#4 Updated by Dru Lavigne 9 months ago

  • Subject changed from VNC device is allowed to be added more then once to Hide VNC option if VM has existing VNC device

#5 Updated by Erin Clark 9 months ago

  • Status changed from In Progress to Ready for Testing

#6 Updated by Aaron St. John 9 months ago

22103

Tested on latest nightlies (FreeNAS-11.2-MASTER-201807200857).

Method of testing:
1. Created a FreeBSD VM with the latest nightlies FreeNAS .iso
2. During the process I made sure the Enable VNC option was checked.
3. After creation I clicked the Add plus symbol and found that the VNC type was not in the drop down list. See attached screenshot.

#7 Updated by Aaron St. John 9 months ago

  • Status changed from Ready for Testing to Passed Testing

#8 Updated by Dru Lavigne 9 months ago

  • Needs QA changed from Yes to No
  • Needs Merging changed from Yes to No

#9 Updated by Timothy Moore II 9 months ago

  • Needs Doc changed from Yes to No

Docs PR: [angulargui branch] https://github.com/freenas/freenas-docs/pull/255 . Investigation shows this change is not reflected in the legacy UI, so no master branch PR is necessary.

#10 Updated by Dru Lavigne 9 months ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF