You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Sanjay Tripathi <sa...@citrix.com> on 2013/12/09 12:42:37 UTC

Review Request 16128: CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16128/
-----------------------------------------------------------

Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh Chodapuneedi.


Bugs: CLOUUDSTACK-4973
    https://issues.apache.org/jira/browse/CLOUUDSTACK-4973


Repository: cloudstack-git


Description
-------

CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

    While deploying a VM, user passes the "keyboard" parameter to specify the
    default language for that VM but in the consoleView, the default language
    selected is en-us irrespective of the default language of the VM.


Diffs
-----

  server/src/com/cloud/servlet/ConsoleProxyClientParam.java e420110 
  server/src/com/cloud/servlet/ConsoleProxyServlet.java b3415db 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyAjaxHandler.java 6cadeca 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientBase.java 07a1b6f 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientParam.java 8de4955 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyHttpHandlerHelper.java 297e711 

Diff: https://reviews.apache.org/r/16128/diff/


Testing
-------

Verified the fix locally by deploying a VM with keyboard=uk and keyboard=jp.


Thanks,

Sanjay Tripathi


Re: Review Request 16128: CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16128/#review30911
-----------------------------------------------------------


Commit 526043b715dc279ad58cb37c28d81b177eb2aa44 in branch refs/heads/4.3 from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=526043b ]

CLOUDSTACK-4973: Specified keyboard language is not showing as default in consoleView passed during deployVM.


- ASF Subversion and Git Services


On Dec. 20, 2013, 10:42 a.m., Sanjay Tripathi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16128/
> -----------------------------------------------------------
> 
> (Updated Dec. 20, 2013, 10:42 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh Chodapuneedi.
> 
> 
> Bugs: CLOUUDSTACK-4973
>     https://issues.apache.org/jira/browse/CLOUUDSTACK-4973
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.
> 
>     While deploying a VM, user passes the "keyboard" parameter to specify the
>     default language for that VM but in the consoleView, the default language
>     selected is en-us irrespective of the default language of the VM.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/servlet/ConsoleProxyClientParam.java e420110 
>   server/src/com/cloud/servlet/ConsoleProxyServlet.java b3415db 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyAjaxHandler.java 6cadeca 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientBase.java 07a1b6f 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientParam.java 8de4955 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyHttpHandlerHelper.java 297e711 
>   systemvm/js/ajaxviewer.js 9643204 
> 
> Diff: https://reviews.apache.org/r/16128/diff/
> 
> 
> Testing
> -------
> 
> Verified the fix locally by deploying a VM with keyboard=uk and keyboard=jp.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>


Re: Review Request 16128: CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16128/#review30915
-----------------------------------------------------------


Commit f6b063de7994c65d307d39fe51229c671ad91f16 in branch refs/heads/master from Sanjay Tripathi
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=f6b063d ]

CLOUDSTACK-4973: Specified keyboard language is not showing as default in consoleView passed during deployVM.


- ASF Subversion and Git Services


On Dec. 20, 2013, 10:42 a.m., Sanjay Tripathi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16128/
> -----------------------------------------------------------
> 
> (Updated Dec. 20, 2013, 10:42 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh Chodapuneedi.
> 
> 
> Bugs: CLOUUDSTACK-4973
>     https://issues.apache.org/jira/browse/CLOUUDSTACK-4973
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.
> 
>     While deploying a VM, user passes the "keyboard" parameter to specify the
>     default language for that VM but in the consoleView, the default language
>     selected is en-us irrespective of the default language of the VM.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/servlet/ConsoleProxyClientParam.java e420110 
>   server/src/com/cloud/servlet/ConsoleProxyServlet.java b3415db 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyAjaxHandler.java 6cadeca 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientBase.java 07a1b6f 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientParam.java 8de4955 
>   services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyHttpHandlerHelper.java 297e711 
>   systemvm/js/ajaxviewer.js 9643204 
> 
> Diff: https://reviews.apache.org/r/16128/diff/
> 
> 
> Testing
> -------
> 
> Verified the fix locally by deploying a VM with keyboard=uk and keyboard=jp.
> 
> 
> Thanks,
> 
> Sanjay Tripathi
> 
>


Re: Review Request 16128: CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

Posted by Sanjay Tripathi <sa...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16128/
-----------------------------------------------------------

(Updated Dec. 20, 2013, 10:42 a.m.)


Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh Chodapuneedi.


Bugs: CLOUUDSTACK-4973
    https://issues.apache.org/jira/browse/CLOUUDSTACK-4973


Repository: cloudstack-git


Description
-------

CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

    While deploying a VM, user passes the "keyboard" parameter to specify the
    default language for that VM but in the consoleView, the default language
    selected is en-us irrespective of the default language of the VM.


Diffs (updated)
-----

  server/src/com/cloud/servlet/ConsoleProxyClientParam.java e420110 
  server/src/com/cloud/servlet/ConsoleProxyServlet.java b3415db 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyAjaxHandler.java 6cadeca 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientBase.java 07a1b6f 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientParam.java 8de4955 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyHttpHandlerHelper.java 297e711 
  systemvm/js/ajaxviewer.js 9643204 

Diff: https://reviews.apache.org/r/16128/diff/


Testing
-------

Verified the fix locally by deploying a VM with keyboard=uk and keyboard=jp.


Thanks,

Sanjay Tripathi


Re: Review Request 16128: CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

Posted by Sanjay Tripathi <sa...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16128/
-----------------------------------------------------------

(Updated Dec. 19, 2013, 10:39 a.m.)


Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh Chodapuneedi.


Changes
-------

Rebased with latest 4.3 code.


Bugs: CLOUUDSTACK-4973
    https://issues.apache.org/jira/browse/CLOUUDSTACK-4973


Repository: cloudstack-git


Description
-------

CLOUDSTACK-4973: Specified locale keyboard language is not showing as default in consoleView passed during deployVM.

    While deploying a VM, user passes the "keyboard" parameter to specify the
    default language for that VM but in the consoleView, the default language
    selected is en-us irrespective of the default language of the VM.


Diffs (updated)
-----

  server/src/com/cloud/servlet/ConsoleProxyClientParam.java e420110 
  server/src/com/cloud/servlet/ConsoleProxyServlet.java b3415db 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyAjaxHandler.java 6cadeca 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientBase.java 07a1b6f 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyClientParam.java 8de4955 
  services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyHttpHandlerHelper.java 297e711 

Diff: https://reviews.apache.org/r/16128/diff/


Testing
-------

Verified the fix locally by deploying a VM with keyboard=uk and keyboard=jp.


Thanks,

Sanjay Tripathi