Those suggestions sound good. Here are the changes I will make in the UI:

- Change icon to use a ‘recycle’ symbol, to avoid confusion w/ reboot VM
- Rename action(s) to reflect the name of the API calls.
- Update the confirm dialog to say, “This action will restore the VM to a fresh 
root disk. After this action, the current root disk will be attached to the VM.”

-Brian
 
On Mar 20, 2014, at 11:46 AM, Mike Tutkowski <mike.tutkow...@solidfire.com> 
wrote:

> I think with the addition of detailed confirmation info that that is a good
> solution.
> 
> Thanks!
> 
> 
> On Thu, Mar 20, 2014 at 12:44 PM, Jessica Wang <jessica.w...@citrix.com>wrote:
> 
>> And also provide detailed info in confirmation dialog when clicking the
>> icon that fires restoreVirtualMachine API.
>> 
>> e.g. "This action will restore the VM to a fresh new root disk.
>> After this action, the current root disk wll be destroyed and a new root
>> disk will be attached to the VM."
>> 
>> 
>> Harikrishna,
>> do you any suggestion for the text in confirmation dialog?
>> 
>> 
>> -----Original Message-----
>> From: Jessica Wang [mailto:jessica.w...@citrix.com]
>> Sent: Thursday, March 20, 2014 11:32 AM
>> To: Mike Tutkowski
>> Cc: Brian Federle; Harikrishna Patnala; Alena Prokharchyk;
>> dev@cloudstack.apache.org; Stephen Turner
>> Subject: RE: Resetting a VM is broken?
>> 
>> How about we just match text with API name?
>> 
>> i. Label the icon that fires recoverVirtualMachine API as "Recover".
>> ii. Label the icon that fires restoreVirtualMachine API as "Restore".
>> 
>> 
>> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
>> Sent: Thursday, March 20, 2014 11:29 AM
>> To: Jessica Wang
>> Cc: Brian Federle; Harikrishna Patnala; Alena Prokharchyk;
>> dev@cloudstack.apache.org; Stephen Turner
>> Subject: Re: Resetting a VM is broken?
>> 
>> OK, thanks for the history there.
>> 
>> What about the text we're using? Are we OK with Reset VM, for example?
>> 
>> Thanks!
>> 
>> On Thu, Mar 20, 2014 at 12:26 PM, Jessica Wang <jessica.w...@citrix.com
>> <mailto:jessica.w...@citrix.com>> wrote:
>> Mike,
>> 
>> Right.
>> 
>> The history is recoverVirtualMachine API was created several years ago,
>> restoreVirtualMachine didn't exist yet at that time.
>> So, the icon that fires recoverVirtualMachine API was labeled as "Restore"
>> at that time.
>> 
>> restoreVirtualMachine API came along several years later.
>> Then, since label "Restore" has already taken, the icon that fires
>> restoreVirtualMachine API was therefore labeled as "Reset".
>> 
>> The result is indeed weird.
>> We'll fix the label to match API name.
>> 
>> Jessica
>> 
>> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com<mailto:
>> mike.tutkow...@solidfire.com>]
>> Sent: Thursday, March 20, 2014 11:03 AM
>> To: Jessica Wang
>> Cc: Brian Federle; Harikrishna Patnala; Alena Prokharchyk;
>> dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>; Stephen
>> Turner
>> 
>> Subject: Re: Resetting a VM is broken?
>> 
>> Thanks, Jessica
>> 
>> The even weirder part, though, is that the text Restore maps to the
>> recoverVirtualMachine API and the text Reset VM maps to the
>> restoreVirtualMachine.
>> 
>> On Thu, Mar 20, 2014 at 11:58 AM, Jessica Wang <jessica.w...@citrix.com
>> <mailto:jessica.w...@citrix.com>> wrote:
>>> [Mike Tutkowski] Unless I'm reading this incorrectly, it looks like
>>> label.action.restore.instance (Restore Instance) maps to
>> recoverVirtualMachine
>>> and
>>> label.resetVM (Reset VM) maps to restoreVirtualMachine
>> Yes, this is confusing.
>> I'll file an UI bug to match icon label with API name.
>> i.e. The icon that fires recoverVirtualMachine API will be labeled as
>> "Recover"
>> and the icon that fires restoreVirtualMachine API will be labeled as
>> "Restore"
>> 
>> 
>> -----Original Message-----
>> From: Brian Federle
>> Sent: Thursday, March 20, 2014 10:29 AM
>> To: Alena Prokharchyk
>> Cc: dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>; Brian
>> Federle; Jessica Wang; Mike Tutkowski; Harikrishna Patnala
>> Subject: Re: Resetting a VM is broken?
>>>> the Reset Vm option is presented as a button unlike the icons
>> I think this was because all the other icons we tried to represent this
>> with were too confusing to users - they were clicking on 'reset VM'
>> thinking it was 'reboot VM' - then as a last minute fix the decision was to
>> just to show a text label.
>> 
>> Any ideas from anyone on what would be a good icon representation?
>> 
>> -Brian
>> 
>> On Mar 19, 2014, at 3:02 PM, Alena Prokharchyk <
>> alena.prokharc...@citrix.com<mailto:alena.prokharc...@citrix.com>> wrote:
>> 
>>> 1) Didn't notice "the elephant" - that the Reset Vm option is presented
>> as
>>> a button unlike the icons for other options. UI folks - Brian/Jessica-
>> why
>>> we decided to represent this option differently in the UI?
>>> 
>>> 2) Mike, I've looked at the code and the history. We've always allocated
>> a
>>> new volume during the vm restore call. Hari, can you please confirm that
>>> this feature you wrote, was designed to work this way
>>> 
>>> 
>>> -Alena.
>>> 
>>> 
>>> On 3/19/14, 2:41 PM, "Mike Tutkowski" <mike.tutkow...@solidfire.com
>> <mailto:mike.tutkow...@solidfire.com>> wrote:
>>> 
>>>> To clarify: Reboot VM seemed to keep the VM running and just reboot the
>>>> OS.
>>>> 
>>>> Reset VM seemed to shut the VM down and then re-start it (with the same
>>>> root disk for the OS).
>>>> 
>>>> 
>>>> On Wed, Mar 19, 2014 at 3:40 PM, Mike Tutkowski <
>>>> mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>>
>> wrote:
>>>> 
>>>>> I could be wrong, but I thought as recently as 4.3 that Reset VM
>>>>> essentially stopped and then re-started the VM with the same root disk.
>>>>> 
>>>>> It seemed to differ from Reboot VM only in the sense that the VM was
>>>>> never
>>>>> technically stopped (the OS inside the VM was just rebooted).
>>>>> 
>>>>> 
>>>>> On Wed, Mar 19, 2014 at 3:38 PM, Mike Tutkowski <
>>>>> mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>>
>> wrote:
>>>>> 
>>>>>> I'm using XenServer 6.1 hosts (two of them) in this case.
>>>>>> 
>>>>>> 
>>>>>> On Wed, Mar 19, 2014 at 3:37 PM, Mike Tutkowski <
>>>>>> mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>>
>> wrote:
>>>>>> 
>>>>>>> Yeah, stopping and restarting works fine...as does, I believe,
>>>>>>> rebooting
>>>>>>> the VM.
>>>>>>> 
>>>>>>> Right below Reboot VM, there is a Reset VM option.
>>>>>>> 
>>>>>>> If that means "re-install," then we probably should clean up the old
>>>>>>> root disk.
>>>>>>> 
>>>>>>> 
>>>>>>> On Wed, Mar 19, 2014 at 3:30 PM, Alena Prokharchyk <
>>>>>>> alena.prokharc...@citrix.com<mailto:alena.prokharc...@citrix.com>>
>> wrote:
>>>>>>> 
>>>>>>>> don¹t see ³Reset Vm² option in UI; do you mean ³Reboot instance²?
>>>>>>>> I¹ve
>>>>>>>> just tried that, as well as calling stop/startVM from the UI. The Vm
>>>>>>>> boots
>>>>>>>> up with its original disk.
>>>>>>>> 
>>>>>>>> I¹m having Xen/Nfs, not sure it matters
>>>>>>>> 
>>>>>>>> -Alena.
>>>>>>>> 
>>>>>>>> On 3/19/14, 2:19 PM, "Mike Tutkowski" <mike.tutkow...@solidfire.com
>> <mailto:mike.tutkow...@solidfire.com>>
>>>>>>>> wrote:
>>>>>>>> 
>>>>>>>>> I didn't look into what the GUI calls, but I invoked the Reset VM
>>>>>>>> option
>>>>>>>>> in
>>>>>>>>> the GUI.
>>>>>>>>> 
>>>>>>>>> Thanks
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> On Wed, Mar 19, 2014 at 3:17 PM, Alena Prokharchyk <
>>>>>>>>> alena.prokharc...@citrix.com<mailto:alena.prokharc...@citrix.com>>
>> wrote:
>>>>>>>>> 
>>>>>>>>>> Mike, just to confirm - did you call
>>>>>>>>>> stopVirtualMachine/startVirtualMachine apis?
>>>>>>>>>> 
>>>>>>>>>> -alena.
>>>>>>>>>> 
>>>>>>>>>> On 3/19/14, 1:55 PM, "Mike Tutkowski"
>>>>>>>> <mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>>
>>>>>>>>>> wrote:
>>>>>>>>>> 
>>>>>>>>>>> Hi,
>>>>>>>>>>> 
>>>>>>>>>>> I noticed today while running through some test cases for 4.4
>>>>>>>> that
>>>>>>>>>>> resetting a VM does not work as expected.
>>>>>>>>>>> 
>>>>>>>>>>> Instead of the typical stop and re-start behavior where the VM is
>>>>>>>>>> booted
>>>>>>>>>>> back up using the same root disk, the VM gets a new root disk
>>>>>>>> when
>>>>>>>> it
>>>>>>>>>> is
>>>>>>>>>>> booted back up.
>>>>>>>>>>> 
>>>>>>>>>>> Can anyone confirm this finding for me with his or her setup?
>>>>>>>>>>> 
>>>>>>>>>>> Thanks!
>>>>>>>>>>> 
>>>>>>>>>>> --
>>>>>>>>>>> *Mike Tutkowski*
>>>>>>>>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>>>>>>>>> e: mike.tutkow...@solidfire.com<mailto:
>> mike.tutkow...@solidfire.com>
>>>>>>>>>>> o: 303.746.7302<tel:303.746.7302>
>>>>>>>>>>> Advancing the way the world uses the
>>>>>>>>>>> cloud<http://solidfire.com/solution/overview/?video=play>
>>>>>>>>>>> *(tm)*
>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> --
>>>>>>>>> *Mike Tutkowski*
>>>>>>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>>>>>>> e: mike.tutkow...@solidfire.com<mailto:
>> mike.tutkow...@solidfire.com>
>>>>>>>>> o: 303.746.7302<tel:303.746.7302>
>>>>>>>>> Advancing the way the world uses the
>>>>>>>>> cloud<http://solidfire.com/solution/overview/?video=play>
>>>>>>>>> *(tm)*
>>>>>>>> 
>>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> --
>>>>>>> *Mike Tutkowski*
>>>>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>>>>> e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>
>>>>>>> o: 303.746.7302<tel:303.746.7302>
>>>>>>> Advancing the way the world uses the
>>>>>>> cloud<http://solidfire.com/solution/overview/?video=play>
>>>>>>> *(tm)*
>>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> --
>>>>>> *Mike Tutkowski*
>>>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>>>> e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>
>>>>>> o: 303.746.7302<tel:303.746.7302>
>>>>>> Advancing the way the world uses the
>>>>>> cloud<http://solidfire.com/solution/overview/?video=play>
>>>>>> *(tm)*
>>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> *Mike Tutkowski*
>>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>>> e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>
>>>>> o: 303.746.7302<tel:303.746.7302>
>>>>> Advancing the way the world uses the
>>>>> cloud<http://solidfire.com/solution/overview/?video=play>
>>>>> *(tm)*
>>>>> 
>>>> 
>>>> 
>>>> 
>>>> --
>>>> *Mike Tutkowski*
>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>> e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>
>>>> o: 303.746.7302<tel:303.746.7302>
>>>> Advancing the way the world uses the
>>>> cloud<http://solidfire.com/solution/overview/?video=play>
>>>> *(tm)*
>>> 
>> 
>> 
>> 
>> --
>> Mike Tutkowski
>> Senior CloudStack Developer, SolidFire Inc.
>> e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>
>> o: 303.746.7302<tel:303.746.7302>
>> Advancing the way the world uses the cloud<
>> http://solidfire.com/solution/overview/?video=play>(tm)
>> 
>> 
>> 
>> --
>> Mike Tutkowski
>> Senior CloudStack Developer, SolidFire Inc.
>> e: mike.tutkow...@solidfire.com<mailto:mike.tutkow...@solidfire.com>
>> o: 303.746.7302
>> Advancing the way the world uses the cloud<
>> http://solidfire.com/solution/overview/?video=play>(tm)
>> 
> 
> 
> 
> -- 
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud<http://solidfire.com/solution/overview/?video=play>
> *(tm)*

Reply via email to