Yes, after some extermination, that is the way I'll set it in the next
patch update.

One other thing, for the multi-select box, it seems the current library
that foreman uses is a little buggy around updating the multiselect box and
refreshing it.
After updating the library to v0.9.8 from  v0.9.3, everything starting
working straight away.
Any objections to updating the library again to 0.9.8?

Mark


On Wed, Jul 17, 2013 at 10:47 AM, Dominic Cleal <notifications@github.com>wrote:

> In lib/fog_extensions/aws/server.rb:
>
> > @@ -13,6 +16,10 @@ def dns
> >           dns_name || private_dns_name
> >        end
> >
> > +      def vm_ip_address
> > +        managed_ip == 'private' ? :private_ip_address : :public_ip_address
>
> This should be a method as vm_ip_address needs to the return the actual IP
> value.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/theforeman/foreman/pull/778/files#r5238013>
> .
>