Skip to content

[IAM] Relax login nodes tag condition in public policies to prevent cluster creation/update failure#7490

Merged
gmarciani merged 3 commits into
aws:developfrom
gmarciani:wip/mgiacomo/3160/fix-policies-0716-1
Jul 17, 2026
Merged

[IAM] Relax login nodes tag condition in public policies to prevent cluster creation/update failure#7490
gmarciani merged 3 commits into
aws:developfrom
gmarciani:wip/mgiacomo/3160/fix-policies-0716-1

Conversation

@gmarciani

@gmarciani gmarciani commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Description of changes

Context

ParallelCluster relies on an IAM policy that has become too restrictive following a recent AWS service-side change to the CloudFormation ELBv2 Listener resource. When a cluster hits this issue Clusters that include login nodes fail to create or update when deployed through the ParallelCluster API or the CloudFormation Custom Resource.

Changes

  • Fix cluster creation and update failures on clusters with login nodes, caused by the IAM policy requiring the parallelcluster:cluster-name tag to be the only tag instead of just being present.
  • Add a login-nodes pool to the sample cluster deployed by the 1-click custom-resource template. This increases feature coverage so the 1-click path also provisions login nodes.

Notes

This change is necessary and sufficient to resolve the impact described in #7491. We'll consider a broader relaxation later for better future-proofing.

Tests

  • Manually validate deploying a cluster with login nodes through custom resource.
  • SUCCESS
test-suites:
  pcluster_api:
    test_api.py::test_login_nodes:
      dimensions:
        - regions: ["us-east-1"]
          instances: ["c5.xlarge"]
          oss: ["alinux2023"]
          schedulers: ["slurm"]
  custom_resource:
    # Create a cluster via the pcluster CFN custom resource
    test_cluster_custom_resource.py::test_cluster_create:
      dimensions:
        - regions: ["us-east-1"]
          instances: ["c5.xlarge"]
          oss: ["alinux2023"]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@gmarciani
gmarciani force-pushed the wip/mgiacomo/3160/fix-policies-0716-1 branch from 147a778 to 2011d29 Compare July 16, 2026 20:44
@gmarciani gmarciani added the 3.x label Jul 16, 2026
@gmarciani
gmarciani marked this pull request as ready for review July 16, 2026 20:55
@gmarciani
gmarciani requested review from a team as code owners July 16, 2026 20:55
@gmarciani gmarciani changed the title Wip/mgiacomo/3160/fix policies 0716 1 [IAM] Relax login nodes tag condition in public policies. to prevent cluster creation/update failure Jul 16, 2026
@gmarciani
gmarciani force-pushed the wip/mgiacomo/3160/fix-policies-0716-1 branch from 2011d29 to aa01404 Compare July 16, 2026 21:25
@gmarciani gmarciani changed the title [IAM] Relax login nodes tag condition in public policies. to prevent cluster creation/update failure [IAM] Relax login nodes tag condition in public policies to prevent cluster creation/update failure Jul 16, 2026
Add a login-nodes pool to the sample cluster deployed by the 1-click custom-resource template. This increases feature coverage so the 1-click path also provisions login nodes.
… nodes,

caused by the IAM policy requiring the `parallelcluster:cluster-name` tag
to be the only tag instead of just being present.
@gmarciani
gmarciani force-pushed the wip/mgiacomo/3160/fix-policies-0716-1 branch from aa01404 to 183f1ca Compare July 16, 2026 21:40
Comment thread cloudformation/custom_resource/cluster-1-click.yaml Outdated
…e use to deploy cluster with CFN custom resource, to prevent intermittent deployment failures.
@gmarciani
gmarciani enabled auto-merge (rebase) July 17, 2026 21:00
@gmarciani
gmarciani merged commit 2466fc2 into aws:develop Jul 17, 2026
19 checks passed
@gmarciani
gmarciani deleted the wip/mgiacomo/3160/fix-policies-0716-1 branch July 17, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants