diff options
| author | Akihiro Motoki <amotoki@gmail.com> | 2017-08-17 08:43:44 +0000 |
|---|---|---|
| committer | Akihiro Motoki <amotoki@gmail.com> | 2017-08-21 10:30:59 +0000 |
| commit | c6c24eda59f8203f112125a4d1c073158576dd6b (patch) | |
| tree | 58abe3a3047f93698c5bfa7ece9a487116a8d39b /openstack_dashboard/dashboards/project/security_groups | |
| parent | 8db04ad33aa70672993f22267ebce018fcf18197 (diff) | |
| download | horizon-c6c24eda59f8203f112125a4d1c073158576dd6b.tar.gz | |
Make icmp_type/code optional
In Neutron API icmp_type/code are not required fields,
so they should be optional.
Previously we support Nova API which requires icmp_type
and icmp_code but we dropped Nova security group API support in Pike
and we no longer need to care it.
Change-Id: Ie460e763928b4f50f84c38d88556df632078ba42
Closes-Bug: #1712025
Diffstat (limited to 'openstack_dashboard/dashboards/project/security_groups')
| -rw-r--r-- | openstack_dashboard/dashboards/project/security_groups/forms.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/openstack_dashboard/dashboards/project/security_groups/forms.py b/openstack_dashboard/dashboards/project/security_groups/forms.py index 7fd6c8095..4d85aa941 100644 --- a/openstack_dashboard/dashboards/project/security_groups/forms.py +++ b/openstack_dashboard/dashboards/project/security_groups/forms.py @@ -175,6 +175,7 @@ class AddRule(forms.SelfHandlingForm): utils_validators.validate_port_range]) icmp_type = forms.IntegerField(label=_("Type"), + required=False, help_text=_("Enter a value for ICMP type " "in the range (-1: 255)"), widget=forms.TextInput(attrs={ @@ -186,6 +187,7 @@ class AddRule(forms.SelfHandlingForm): validate_icmp_type_range]) icmp_code = forms.IntegerField(label=_("Code"), + required=False, help_text=_("Enter a value for ICMP code " "in the range (-1: 255)"), widget=forms.TextInput(attrs={ |