openstack/horizon
Code Issues Proposed changes

Merge "Revert "Making IP protocol field required""

Browse Source
This commit is contained in:
Zuul
2025-09-17 18:33:09 +00:00
committed by Gerrit Code Review
parent 3b6a6d8ea7 64163a6e00
commit 127319128b
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
openstack_dashboard/dashboards/project/security_groups/forms.py
View File

@@ -126,9 +126,8 @@ class AddRule(forms.SelfHandlingForm):
}))
ip_protocol = forms.IntegerField(
label=_('IP Protocol'), required=True,
help_text=_("Enter an integer value between -1 and 255 "
"(-1 means wild card)."),
label=_('IP Protocol'), required=False,
help_text=_("Enter an integer value between 0 and 255."),
validators=[utils_validators.validate_ip_protocol],
widget=forms.TextInput(attrs={
'class': 'switched',
@@ -284,6 +283,10 @@ class AddRule(forms.SelfHandlingForm):
self.fields['direction'].choices = [('ingress', _('Ingress')),
('egress', _('Egress'))]
self.fields['ip_protocol'].help_text = _(
"Enter an integer value between -1 and 255 "
"(-1 means wild card)."
)
self.fields['port_or_range'].choices = [
('port', _('Port')),

9
openstack_dashboard/dashboards/project/security_groups/tests.py
View File

@@ -461,8 +461,7 @@ class SecurityGroupsViewTests(test.TestCase):
'rule_menu': 'http',
'port_or_range': 'port',
'cidr': rule.ip_range['cidr'],
'remote': 'cidr',
'ip_protocol': 6}
'remote': 'cidr'}
res = self.client.post(self.edit_url, formData)
self.assertRedirectsNoFollow(res, self.detail_url)
@@ -545,8 +544,7 @@ class SecurityGroupsViewTests(test.TestCase):
'port_or_range': 'port',
'cidr': '0.0.0.0/0',
'security_group': sec_group.id,
'remote': 'sg',
'ip_protocol': 6}
'remote': 'sg'}
res = self.client.post(self.edit_url, formData)
self.assertRedirectsNoFollow(res, self.detail_url)
@@ -933,8 +931,7 @@ class SecurityGroupsViewTests(test.TestCase):
'port_or_range': 'range',
'rule_menu': 'all_tcp',
'cidr': rule.ip_range['cidr'],
'remote': 'cidr',
'ip_protocol': 6}
'remote': 'cidr'}
res = self.client.post(self.edit_url, formData)
self.assertRedirectsNoFollow(res, self.detail_url)