SNAT option for neutron

[Simon]

When setting a gateway for a router, there is an option to enable/disable SNAT for the router. Currently, this option isn't available on horizon.. Here is a simple patch for your consideration.

[Simon (xchenum)]

Patch here:

http://paste.openstack.org/show/47934/

[Akihiro Motoki (amotoki)]

Have you signed CLA?
If so, I hope you propose the patch to gerrit?

On Fri, Oct 4, 2013 at 5:03 AM, Simon <email address hidden> wrote:
> Patch here:
>
> http://paste.openstack.org/show/47934/
>
> --
> You received this bug notification because you are subscribed to
> OpenStack Dashboard (Horizon).
> Matching subscriptions: horizon-bugs
> https://bugs.launchpad.net/bugs/1234912
>
> Title:
> SNAT option for neutron
>
> Status in OpenStack Dashboard (Horizon):
> New
>
> Bug description:
> When setting a gateway for a router, there is an option to
> enable/disable SNAT for the router. Currently, this option isn't
> available on horizon.. Here is a simple patch for your consideration.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/horizon/+bug/1234912/+subscriptions

--
Akihiro MOTOKI <email address hidden>

[Simon (xchenum)]

I think I have signed both personal and enterprise agreement in my previous job..

Here is a new patch that adds some display info:
http://paste.openstack.org/show/47935/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/49933

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/54276

[Rob Cresswell (robcresswell-deactivatedaccount)]

Patch had been abandoned for over a year. Removed Assignee and updated.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/205423

[OpenStack Infra (hudson-openstack)]

Change abandoned by Rob Cresswell (<email address hidden>) on branch: master
Review: https://review.openstack.org/205423
Reason: Due to inactivity and merge conflict

[Ivan Kolodyazhny (e0ne)]

Looks like it's already implemented in master. Please, re-open bug if it's still actual