Skip to content

[Test] Extend test_slurm_accounting to use a mixed-case ClusterName#7344

Merged
gmarciani merged 1 commit intoaws:developfrom
gmarciani:wip/mgiacomo/3160/test-accounting-0417-1
Apr 17, 2026
Merged

[Test] Extend test_slurm_accounting to use a mixed-case ClusterName#7344
gmarciani merged 1 commit intoaws:developfrom
gmarciani:wip/mgiacomo/3160/test-accounting-0417-1

Conversation

@gmarciani
Copy link
Copy Markdown
Contributor

@gmarciani gmarciani commented Apr 17, 2026

Description of changes

Extend test_slurm_accounting to use a mixed-case ClusterName via CustomSlurmSettings, protecting against regressions fixed in aws/aws-parallelcluster-cookbook#3164 and also reported in aws/aws-parallelcluster-cookbook#3159

Tests

SUCCEEDED end2end test:

test-suites:
  schedulers:
    test_slurm_accounting.py::test_slurm_accounting:
      dimensions:
        - regions: ["us-east-1"]
          instances: ["c5.xlarge"]
          oss: ["alinux2023"]
          schedulers: ["slurm"]

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

@gmarciani gmarciani requested review from a team as code owners April 17, 2026 16:03
@gmarciani gmarciani added skip-changelog-update Disables the check that enforces changelog updates in PRs 3.x Test labels Apr 17, 2026
@gmarciani gmarciani force-pushed the wip/mgiacomo/3160/test-accounting-0417-1 branch 2 times, most recently from ca3d720 to 84f2d17 Compare April 17, 2026 16:08
@gmarciani gmarciani force-pushed the wip/mgiacomo/3160/test-accounting-0417-1 branch from 84f2d17 to b88d15c Compare April 17, 2026 16:40
@gmarciani gmarciani merged commit 643aa3c into aws:develop Apr 17, 2026
24 checks passed
@gmarciani gmarciani deleted the wip/mgiacomo/3160/test-accounting-0417-1 branch April 17, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x skip-changelog-update Disables the check that enforces changelog updates in PRs Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants