← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 2023362] Re: "IP Protocol" should be manfatory field when user creates security group for "Other protocol"

 

Reviewed:  https://review.opendev.org/c/openstack/horizon/+/948291
Committed: https://opendev.org/openstack/horizon/commit/9445725a9e56956ea06d4338fc4cccd9bb89ad94
Submitter: "Zuul (22348)"
Branch:    master

commit 9445725a9e56956ea06d4338fc4cccd9bb89ad94
Author: Dmitriy Chubinidze <dcu995@xxxxxxxxx>
Date:   Sun Apr 27 07:50:55 2025 +0000

    Making IP protocol field required
    
    While adding rule to security group and choosing
    "Other Protocol" field, we should make "IP Protocol"
     field mandatory to prevent getting error.
    
    Change-Id: I56da00c6a000c019699edda12b405b6ddc389001
    Closes-Bug: #2023362


** Changed in: horizon
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/2023362

Title:
  "IP Protocol" should be manfatory field when user creates security
  group for "Other protocol"

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  Repro steps:

  1. Create new Security Group
  2. Create new rule for that security group:
       Rule: Other ProtocolDanger: There was an error submitting the form. Please try again.
       Description: Empty
       Direction: Ingress
       IP Protocol: Leave Empty
       Remote CIDR
       CIDR: 0.0.0.0/0 or ::/0

  Error popup is show with the message:
  "Danger: There was an error submitting the form. Please try again."

  If user would specify -1 as a IP protocol, rule is being created.

  
  Expected behaviour: 
  "IP Protocol" field should be marked as mandatory.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/2023362/+subscriptions



References