Skip to content

Commit 93b73e9

Browse files
jjblack604jjblack604
committed
Bug Fix, Default SG Rule Custom SG
Currently the Default setting for CustomSG Rule is set to True, this means all new SGs inherit these rules, with no way for user to override this behavior. Closes Bug: #2054629 Change-Id: Icef7c91654dbced76a5492437e723c4e2a0f3102
1 parent 2eea916 commit 93b73e9

2 files changed

Lines changed: 17 additions & 17 deletions

File tree

‎openstackclient/network/v2/default_security_group_rule.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ def get_parser(self, prog_name):
142142
parser.add_argument(
143143
'--for-custom-sg',
144144
action='store_true',
145-
default=True,
145+
default=False,
146146
help=_(
147147
"Set this default security group rule to be used in all "
148148
"custom security groups created manually by users"

‎openstackclient/tests/unit/network/v2/test_default_security_group_rule.py‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ def _setup_default_security_group_rule(self, attrs=None):
7272
'remote_ip_prefix': '0.0.0.0/0',
7373
'location': 'MUNCHMUNCHMUNCH',
7474
'used_in_default_sg': False,
75-
'used_in_non_default_sg': True,
75+
'used_in_non_default_sg': False,
7676
}
7777
attrs = attrs or {}
7878
# Overwrite default attributes.
@@ -217,7 +217,7 @@ def test_create_default_rule(self):
217217
'protocol': self._default_sg_rule.protocol,
218218
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
219219
'used_in_default_sg': False,
220-
'used_in_non_default_sg': True,
220+
'used_in_non_default_sg': False,
221221
}
222222
)
223223
self.assertEqual(self.expected_columns, columns)
@@ -251,7 +251,7 @@ def test_create_protocol_any(self):
251251
'protocol': self._default_sg_rule.protocol,
252252
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
253253
'used_in_default_sg': False,
254-
'used_in_non_default_sg': True,
254+
'used_in_non_default_sg': False,
255255
}
256256
)
257257
self.assertEqual(self.expected_columns, columns)
@@ -287,7 +287,7 @@ def test_create_remote_address_group(self):
287287
'protocol': self._default_sg_rule.protocol,
288288
'remote_address_group_id': self._default_sg_rule.remote_address_group_id,
289289
'used_in_default_sg': False,
290-
'used_in_non_default_sg': True,
290+
'used_in_non_default_sg': False,
291291
}
292292
)
293293
self.assertEqual(self.expected_columns, columns)
@@ -334,7 +334,7 @@ def test_create_remote_group(self):
334334
'protocol': self._default_sg_rule.protocol,
335335
'remote_group_id': 'remote-group-id',
336336
'used_in_default_sg': False,
337-
'used_in_non_default_sg': True,
337+
'used_in_non_default_sg': False,
338338
}
339339
)
340340
self.assertEqual(self.expected_columns, columns)
@@ -366,7 +366,7 @@ def test_create_source_group(self):
366366
'protocol': self._default_sg_rule.protocol,
367367
'remote_group_id': 'remote-group-id',
368368
'used_in_default_sg': False,
369-
'used_in_non_default_sg': True,
369+
'used_in_non_default_sg': False,
370370
}
371371
)
372372
self.assertEqual(self.expected_columns, columns)
@@ -400,7 +400,7 @@ def test_create_source_ip(self):
400400
'protocol': self._default_sg_rule.protocol,
401401
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
402402
'used_in_default_sg': False,
403-
'used_in_non_default_sg': True,
403+
'used_in_non_default_sg': False,
404404
}
405405
)
406406
self.assertEqual(self.expected_columns, columns)
@@ -434,7 +434,7 @@ def test_create_remote_ip(self):
434434
'protocol': self._default_sg_rule.protocol,
435435
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
436436
'used_in_default_sg': False,
437-
'used_in_non_default_sg': True,
437+
'used_in_non_default_sg': False,
438438
}
439439
)
440440
self.assertEqual(self.expected_columns, columns)
@@ -585,7 +585,7 @@ def test_create_icmp_type(self):
585585
'protocol': self._default_sg_rule.protocol,
586586
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
587587
'used_in_default_sg': False,
588-
'used_in_non_default_sg': True,
588+
'used_in_non_default_sg': False,
589589
}
590590
)
591591
self.assertEqual(self.expected_columns, columns)
@@ -623,7 +623,7 @@ def test_create_icmp_type_zero(self):
623623
'protocol': self._default_sg_rule.protocol,
624624
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
625625
'used_in_default_sg': False,
626-
'used_in_non_default_sg': True,
626+
'used_in_non_default_sg': False,
627627
}
628628
)
629629
self.assertEqual(self.expected_columns, columns)
@@ -661,7 +661,7 @@ def test_create_icmp_type_greater_than_zero(self):
661661
'protocol': self._default_sg_rule.protocol,
662662
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
663663
'used_in_default_sg': False,
664-
'used_in_non_default_sg': True,
664+
'used_in_non_default_sg': False,
665665
}
666666
)
667667
self.assertEqual(self.expected_columns, columns)
@@ -698,7 +698,7 @@ def test_create_icmp_type_negative_value(self):
698698
'protocol': self._default_sg_rule.protocol,
699699
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
700700
'used_in_default_sg': False,
701-
'used_in_non_default_sg': True,
701+
'used_in_non_default_sg': False,
702702
}
703703
)
704704
self.assertEqual(self.expected_columns, columns)
@@ -741,7 +741,7 @@ def test_create_ipv6_icmp_type_code(self):
741741
'protocol': self._default_sg_rule.protocol,
742742
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
743743
'used_in_default_sg': False,
744-
'used_in_non_default_sg': True,
744+
'used_in_non_default_sg': False,
745745
}
746746
)
747747
self.assertEqual(self.expected_columns, columns)
@@ -780,7 +780,7 @@ def test_create_icmpv6_type(self):
780780
'protocol': self._default_sg_rule.protocol,
781781
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
782782
'used_in_default_sg': False,
783-
'used_in_non_default_sg': True,
783+
'used_in_non_default_sg': False,
784784
}
785785
)
786786
self.assertEqual(self.expected_columns, columns)
@@ -811,7 +811,7 @@ def test_create_with_description(self):
811811
'protocol': self._default_sg_rule.protocol,
812812
'remote_ip_prefix': self._default_sg_rule.remote_ip_prefix,
813813
'used_in_default_sg': False,
814-
'used_in_non_default_sg': True,
814+
'used_in_non_default_sg': False,
815815
}
816816
)
817817
self.assertEqual(self.expected_columns, columns)
@@ -831,7 +831,7 @@ class TestDeleteDefaultSecurityGroupRule(TestDefaultSecurityGroupRule):
831831
'remote_ip_prefix': '0.0.0.0/0',
832832
'location': 'MUNCHMUNCHMUNCH',
833833
'used_in_default_sg': False,
834-
'used_in_non_default_sg': True,
834+
'used_in_non_default_sg': False,
835835
}
836836
_default_sg_rules = list(
837837
sdk_fakes.generate_fake_resources(

0 commit comments

Comments
 (0)