Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Plan Wizard - VM Import selection state #240

Closed
priley86 opened this issue Apr 23, 2018 · 2 comments · Fixed by #516
Closed

Plan Wizard - VM Import selection state #240

priley86 opened this issue Apr 23, 2018 · 2 comments · Fixed by #516
Assignees
@mturley
Labels
enhancement Sprint 11

Comments

@priley86
Copy link
Member

  • If user selects "discover" option, deselect all VMs by default.
  • If user select "CSV Import option, after the import, select all valid VMs by default.

cc: @vconzola @serenamarie125 @AparnaKarve
@priley86 priley86 mentioned this issue Apr 23, 2018
Merged
@mturley mturley assigned mturley and unassigned AparnaKarve Jul 25, 2018
@mturley mturley mentioned this issue Jul 25, 2018
Closed
@mturley
Copy link
Contributor

mturley commented Jul 25, 2018 •
edited
Loading

For my own convenience, example CSV from Aparna:

Name,Host,UID,Provider
cfmetest67,esxi1,421fcbfc-3b4c-46d6-c7f4-da8a27e0243b,vCenter
demo-infoblox,esxi1,421fd6d6-1788-77f8-f53a-6db47ce65175,vCenter
namwrncld037,esxi1,421f6d94-5f9c-5f80-90ca-08ba516c45ee,vCenter
namwrncld055,esxi1,421f2b21-9dff-488d-f47e-0a7c7707997d,vCenter
nate-ocp-test,esxi1,421f5d50-5746-5f22-8d06-7c55f49e41b1,vCenter

example CSV with some valid, some conflicted, and some invalid VMs (when used with brett-test mapping in nightlydb):

Name
test-vm-kkulkarn
windows-server-2008
windows-server-2012
something-invalid
Wingtips
another-invalid-vm

This was referenced Jul 25, 2018
Closed
Merged
@mturley
Copy link
Contributor

mturley commented Jul 25, 2018

PR #516 fixes this.

AparnaKarve added a commit that referenced this issue Jul 26, 2018
@AparnaKarve
Merge pull request #516 from mturley/vm-import-state
1f69175 
[#240] Plan Wizard: Select valid VMs by default on CSV import, but not in discovery mode
simaishi pushed a commit that referenced this issue Jul 31, 2018
@AparnaKarve @simaishi
Merge pull request #516 from mturley/vm-import-state
b1d626e 
[#240] Plan Wizard: Select valid VMs by default on CSV import, but not in discovery mode
(cherry picked from commit 1f69175)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1610393
@priley86 priley86 mentioned this issue Aug 9, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mturley mturley

Labels
enhancement Sprint 11
Projects
None yet
Milestone
No milestone
5 participants
@mturley @AparnaKarve @priley86 @serenamarie125 @vconzola