Merge " Bug Fix, Default SG Rule Custom SG"

Zuul · openstack-gerrit · commit 6373c91a77ca · 2024-03-19T14:44:00.000Z
diff --git a/openstackclient/network/v2/default_security_group_rule.py b/openstackclient/network/v2/default_security_group_rule.py
@@ -142,7 +142,7 @@ def get_parser(self, prog_name):
         parser.add_argument(
             '--for-custom-sg',
             action='store_true',
-            default=True,
+            default=False,
             help=_(
                 "Set this default security group rule to be used in all "
                 "custom security groups created manually by users"
diff --git a/openstackclient/tests/unit/network/v2/test_default_security_group_rule.py b/openstackclient/tests/unit/network/v2/test_default_security_group_rule.py
@@ -72,7 +72,7 @@ def _setup_default_security_group_rule(self, attrs=None):
             'remote_ip_prefix': '0.0.0.0/0',
             'location': 'MUNCHMUNCHMUNCH',
             'used_in_default_sg': False,
-            'used_in_non_default_sg': True,
+            'used_in_non_default_sg': False,
         }
         attrs = attrs or {}
         # Overwrite default attributes.
@@ -217,7 +217,7 @@ def test_create_default_rule(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -251,7 +251,7 @@ def test_create_protocol_any(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -287,7 +287,7 @@ def test_create_remote_address_group(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_address_group_id': self._default_sg_rule.remote_address_group_id,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -334,7 +334,7 @@ def test_create_remote_group(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_group_id': 'remote-group-id',
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -366,7 +366,7 @@ def test_create_source_group(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_group_id': 'remote-group-id',
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -400,7 +400,7 @@ def test_create_source_ip(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -434,7 +434,7 @@ def test_create_remote_ip(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -585,7 +585,7 @@ def test_create_icmp_type(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -623,7 +623,7 @@ def test_create_icmp_type_zero(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -661,7 +661,7 @@ def test_create_icmp_type_greater_than_zero(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -698,7 +698,7 @@ def test_create_icmp_type_negative_value(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -741,7 +741,7 @@ def test_create_ipv6_icmp_type_code(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -780,7 +780,7 @@ def test_create_icmpv6_type(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -811,7 +811,7 @@ def test_create_with_description(self):
                 'protocol': self._default_sg_rule.protocol,
                 'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
                 'used_in_default_sg': False,
-                'used_in_non_default_sg': True,
+                'used_in_non_default_sg': False,
             }
         )
         self.assertEqual(self.expected_columns, columns)
@@ -831,7 +831,7 @@ class TestDeleteDefaultSecurityGroupRule(TestDefaultSecurityGroupRule):
         'remote_ip_prefix': '0.0.0.0/0',
         'location': 'MUNCHMUNCHMUNCH',
         'used_in_default_sg': False,
-        'used_in_non_default_sg': True,
+        'used_in_non_default_sg': False,
     }
     _default_sg_rules = list(
         sdk_fakes.generate_fake_resources(

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ def _setup_default_security_group_rule(self, attrs=None):`
`72`	`72`	`'remote_ip_prefix': '0.0.0.0/0',`
`73`	`73`	`'location': 'MUNCHMUNCHMUNCH',`
`74`	`74`	`'used_in_default_sg': False,`
`75`		`- 'used_in_non_default_sg': True,`
	`75`	`+ 'used_in_non_default_sg': False,`
`76`	`76`	`}`
`77`	`77`	`attrs = attrs or {}`
`78`	`78`	`# Overwrite default attributes.`
`@@ -217,7 +217,7 @@ def test_create_default_rule(self):`
`217`	`217`	`'protocol': self._default_sg_rule.protocol,`
`218`	`218`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`219`	`219`	`'used_in_default_sg': False,`
`220`		`- 'used_in_non_default_sg': True,`
	`220`	`+ 'used_in_non_default_sg': False,`
`221`	`221`	`}`
`222`	`222`	`)`
`223`	`223`	`self.assertEqual(self.expected_columns, columns)`
`@@ -251,7 +251,7 @@ def test_create_protocol_any(self):`
`251`	`251`	`'protocol': self._default_sg_rule.protocol,`
`252`	`252`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`253`	`253`	`'used_in_default_sg': False,`
`254`		`- 'used_in_non_default_sg': True,`
	`254`	`+ 'used_in_non_default_sg': False,`
`255`	`255`	`}`
`256`	`256`	`)`
`257`	`257`	`self.assertEqual(self.expected_columns, columns)`
`@@ -287,7 +287,7 @@ def test_create_remote_address_group(self):`
`287`	`287`	`'protocol': self._default_sg_rule.protocol,`
`288`	`288`	`'remote_address_group_id': self._default_sg_rule.remote_address_group_id,`
`289`	`289`	`'used_in_default_sg': False,`
`290`		`- 'used_in_non_default_sg': True,`
	`290`	`+ 'used_in_non_default_sg': False,`
`291`	`291`	`}`
`292`	`292`	`)`
`293`	`293`	`self.assertEqual(self.expected_columns, columns)`
`@@ -334,7 +334,7 @@ def test_create_remote_group(self):`
`334`	`334`	`'protocol': self._default_sg_rule.protocol,`
`335`	`335`	`'remote_group_id': 'remote-group-id',`
`336`	`336`	`'used_in_default_sg': False,`
`337`		`- 'used_in_non_default_sg': True,`
	`337`	`+ 'used_in_non_default_sg': False,`
`338`	`338`	`}`
`339`	`339`	`)`
`340`	`340`	`self.assertEqual(self.expected_columns, columns)`
`@@ -366,7 +366,7 @@ def test_create_source_group(self):`
`366`	`366`	`'protocol': self._default_sg_rule.protocol,`
`367`	`367`	`'remote_group_id': 'remote-group-id',`
`368`	`368`	`'used_in_default_sg': False,`
`369`		`- 'used_in_non_default_sg': True,`
	`369`	`+ 'used_in_non_default_sg': False,`
`370`	`370`	`}`
`371`	`371`	`)`
`372`	`372`	`self.assertEqual(self.expected_columns, columns)`
`@@ -400,7 +400,7 @@ def test_create_source_ip(self):`
`400`	`400`	`'protocol': self._default_sg_rule.protocol,`
`401`	`401`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`402`	`402`	`'used_in_default_sg': False,`
`403`		`- 'used_in_non_default_sg': True,`
	`403`	`+ 'used_in_non_default_sg': False,`
`404`	`404`	`}`
`405`	`405`	`)`
`406`	`406`	`self.assertEqual(self.expected_columns, columns)`
`@@ -434,7 +434,7 @@ def test_create_remote_ip(self):`
`434`	`434`	`'protocol': self._default_sg_rule.protocol,`
`435`	`435`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`436`	`436`	`'used_in_default_sg': False,`
`437`		`- 'used_in_non_default_sg': True,`
	`437`	`+ 'used_in_non_default_sg': False,`
`438`	`438`	`}`
`439`	`439`	`)`
`440`	`440`	`self.assertEqual(self.expected_columns, columns)`
`@@ -585,7 +585,7 @@ def test_create_icmp_type(self):`
`585`	`585`	`'protocol': self._default_sg_rule.protocol,`
`586`	`586`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`587`	`587`	`'used_in_default_sg': False,`
`588`		`- 'used_in_non_default_sg': True,`
	`588`	`+ 'used_in_non_default_sg': False,`
`589`	`589`	`}`
`590`	`590`	`)`
`591`	`591`	`self.assertEqual(self.expected_columns, columns)`
`@@ -623,7 +623,7 @@ def test_create_icmp_type_zero(self):`
`623`	`623`	`'protocol': self._default_sg_rule.protocol,`
`624`	`624`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`625`	`625`	`'used_in_default_sg': False,`
`626`		`- 'used_in_non_default_sg': True,`
	`626`	`+ 'used_in_non_default_sg': False,`
`627`	`627`	`}`
`628`	`628`	`)`
`629`	`629`	`self.assertEqual(self.expected_columns, columns)`
`@@ -661,7 +661,7 @@ def test_create_icmp_type_greater_than_zero(self):`
`661`	`661`	`'protocol': self._default_sg_rule.protocol,`
`662`	`662`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`663`	`663`	`'used_in_default_sg': False,`
`664`		`- 'used_in_non_default_sg': True,`
	`664`	`+ 'used_in_non_default_sg': False,`
`665`	`665`	`}`
`666`	`666`	`)`
`667`	`667`	`self.assertEqual(self.expected_columns, columns)`
`@@ -698,7 +698,7 @@ def test_create_icmp_type_negative_value(self):`
`698`	`698`	`'protocol': self._default_sg_rule.protocol,`
`699`	`699`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`700`	`700`	`'used_in_default_sg': False,`
`701`		`- 'used_in_non_default_sg': True,`
	`701`	`+ 'used_in_non_default_sg': False,`
`702`	`702`	`}`
`703`	`703`	`)`
`704`	`704`	`self.assertEqual(self.expected_columns, columns)`
`@@ -741,7 +741,7 @@ def test_create_ipv6_icmp_type_code(self):`
`741`	`741`	`'protocol': self._default_sg_rule.protocol,`
`742`	`742`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`743`	`743`	`'used_in_default_sg': False,`
`744`		`- 'used_in_non_default_sg': True,`
	`744`	`+ 'used_in_non_default_sg': False,`
`745`	`745`	`}`
`746`	`746`	`)`
`747`	`747`	`self.assertEqual(self.expected_columns, columns)`
`@@ -780,7 +780,7 @@ def test_create_icmpv6_type(self):`
`780`	`780`	`'protocol': self._default_sg_rule.protocol,`
`781`	`781`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`782`	`782`	`'used_in_default_sg': False,`
`783`		`- 'used_in_non_default_sg': True,`
	`783`	`+ 'used_in_non_default_sg': False,`
`784`	`784`	`}`
`785`	`785`	`)`
`786`	`786`	`self.assertEqual(self.expected_columns, columns)`
`@@ -811,7 +811,7 @@ def test_create_with_description(self):`
`811`	`811`	`'protocol': self._default_sg_rule.protocol,`
`812`	`812`	`'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,`
`813`	`813`	`'used_in_default_sg': False,`
`814`		`- 'used_in_non_default_sg': True,`
	`814`	`+ 'used_in_non_default_sg': False,`
`815`	`815`	`}`
`816`	`816`	`)`
`817`	`817`	`self.assertEqual(self.expected_columns, columns)`
`@@ -831,7 +831,7 @@ class TestDeleteDefaultSecurityGroupRule(TestDefaultSecurityGroupRule):`
`831`	`831`	`'remote_ip_prefix': '0.0.0.0/0',`
`832`	`832`	`'location': 'MUNCHMUNCHMUNCH',`
`833`	`833`	`'used_in_default_sg': False,`
`834`		`- 'used_in_non_default_sg': True,`
	`834`	`+ 'used_in_non_default_sg': False,`
`835`	`835`	`}`
`836`	`836`	`_default_sg_rules = list(`
`837`	`837`	`sdk_fakes.generate_fake_resources(`