Skip to content

Commit 41babb3

Browse files
committed
Preserve arbitrary orders to break dependency on fixes
1 parent bd271bd commit 41babb3

File tree

2 files changed

+13
-20
lines changed

2 files changed

+13
-20
lines changed

src/confcom/azext_confcom/lib/arm_to_aci_policy_spec.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,10 +75,6 @@ def arm_container_env_to_aci_policy_spec_env(
7575

7676
for env_var in [
7777
*process_env_vars_from_template(parameters, {}, container_properties, approve_wildcards),
78-
*config.OPENGCS_ENV_RULES,
79-
*config.FABRIC_ENV_RULES,
80-
*config.MANAGED_IDENTITY_ENV_RULES,
81-
*config.ENABLE_RESTART_ENV_RULE,
8278
]:
8379
yield AciContainerPropertyEnvVariable(**env_var)
8480

src/confcom/azext_confcom/security_policy.py

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -628,22 +628,19 @@ def load_policy_from_arm_template_str(
628628

629629
aci_policies = []
630630

631-
fragments = sorted(
632-
[
633-
AciFragmentSpec(
634-
feed=fragment["feed"],
635-
issuer=fragment["issuer"],
636-
includes=fragment["includes"],
637-
minimum_svn=infrastructure_svn or fragment["minimum_svn"],
638-
path=fragment.get("path"),
639-
)
640-
for fragment in [
641-
*(included_fragments or []),
642-
*(config.DEFAULT_REGO_FRAGMENTS if not exclude_default_fragments else []),
643-
]
644-
],
645-
key=lambda fragment: fragment.feed,
646-
)
631+
fragments = [
632+
AciFragmentSpec(
633+
feed=fragment["feed"],
634+
issuer=fragment["issuer"],
635+
includes=fragment["includes"],
636+
minimum_svn=infrastructure_svn or fragment["minimum_svn"],
637+
path=fragment.get("path"),
638+
)
639+
for fragment in [
640+
*(config.DEFAULT_REGO_FRAGMENTS if not exclude_default_fragments else []),
641+
*(included_fragments or []),
642+
]
643+
]
647644

648645
try:
649646
for policy_spec in arm_to_aci_policy_spec(

0 commit comments

Comments
 (0)