[PATCH aggregator] BZ #644448 - Instance Details: please include operating system version

[Jan Provazník]

From: Jan Provaznik

https://bugzilla.redhat.com/show_bug.cgi?id=644448
---
 src/app/views/instance/show.haml |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/app/views/instance/show.haml b/src/app/views/instance/show.haml
index fd69b8c..c751a2e 100644
--- a/src/app/views/instance/show.haml
+++ b/src/app/views/instance/show.haml
@@ -17,6 +17,9 @@
       = label_tag :operating_system, 'Operating system'
       %span= @instance.template.xml.platform
     %li
+      = label_tag :operating_system_version, 'Operating system version'
+      %span= @instance.template.xml.platform_version
+    %li
       = label_tag :provider, 'Provider'
       %span= @instance.cloud_account.provider.name if @instance.cloud_account
     %li

[Jason Guiditta]

On Tue, 2010-10-26 at 15:43 +0200, [e-mail] wrote:
> From: Jan Provaznik
> https://bugzilla.redhat.com/show_bug.cgi?id=644448
>
> src/app/views/instance/show.haml |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/src/app/views/instance/show.haml b/src/app/views/instance/show.haml
> index fd69b8c..c751a2e 100644
> --- a/src/app/views/instance/show.haml
> +++ b/src/app/views/instance/show.haml
> @@ -17,6 +17,9 @@
>        = label_tag :operating_system, 'Operating system'
>        %span= @instance.template.xml.platform
>      %li
>  = label_tag :operating_system_version, 'Operating system version'
>  %span= @instance.template.xml.platform_version
> %li

I think this would probably be better just combined on the 'Operating
system' line above.

-j
>    = label_tag :provider, 'Provider'
>    %span= @instance.cloud_account.provider.name if @instance.cloud_account
>  %li

[Jan Provazník]

From: Jan Provaznik

https://bugzilla.redhat.com/show_bug.cgi?id=644448
---
 src/app/views/instance/show.haml |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/app/views/instance/show.haml b/src/app/views/instance/show.haml
index fd69b8c..cb34dde 100644
--- a/src/app/views/instance/show.haml
+++ b/src/app/views/instance/show.haml
@@ -15,7 +15,7 @@
       %span= @instance.private_addresses
     %li
       = label_tag :operating_system, 'Operating system'
-      %span= @instance.template.xml.platform
+      %span= "#{@instance.template.xml.platform} #{@instance.template.xml.platform_version}"
     %li
       = label_tag :provider, 'Provider'
       %span= @instance.cloud_account.provider.name if @instance.cloud_account

[Jason Guiditta]

On Wed, 2010-10-27 at 09:21 +0200, [e-mail] wrote:
> From: Jan Provaznik
> https://bugzilla.redhat.com/show_bug.cgi?id=644448
>
> src/app/views/instance/show.haml |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

ACK, pushed.