Skip to content

[Test] Stabilize test_proxy - #7558

Merged
gmarciani merged 4 commits into
aws:release-3.16from
gmarciani:wip/mgiacomo/3160/text-proxy-0813-1
Aug 14, 2026
Merged

[Test] Stabilize test_proxy#7558
gmarciani merged 4 commits into
aws:release-3.16from
gmarciani:wip/mgiacomo/3160/text-proxy-0813-1

Conversation

@gmarciani

@gmarciani gmarciani commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Description of changes

Stabilize test_proxy.
1. use dynamic capacity reservation to reduce the risk of ICE.
2. reduce the number of compute nodes to the strictly required ones.
3. reduce the risk of dpkg locking failure by disabling boot-time apt jobs.

Tests

SUCCESS

test-suites:
    proxy:
      test_proxy.py::test_proxy:
        dimensions:
          - regions: [{{ c5_xlarge_CAPACITY_RESERVATION_8_INSTANCES_2_HOURS_NOPG_rhel9 }}]
            instances: ["c5.xlarge"]
            oss: ["rhel8", "rhel9"]
            schedulers: ["slurm"]
          - regions: [{{ c5_xlarge_CAPACITY_RESERVATION_20_INSTANCES_2_HOURS_NOPG_ubuntu2404 }}]
            instances: ["c5.xlarge"]
            oss: ["alinux2023", "ubuntu2204", "ubuntu2404", "rocky8", "rocky9"]
            schedulers: ["slurm"]

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

Fractional-GPU instances require the NVIDIA GRID/vGPU driver.
ParallelCluster installs the data-center driver instead, so it cannot bind to the vGPU
and cluster creation fails.
 1. use dynamic capacity reservation to reduce the risk of ICE.
 2. reduce the number of compute nodes to the strictly required ones.
 3. reduce the risk of dpkg locking failure by disabling boot-time apt jobs.
@gmarciani
gmarciani changed the base branch from develop to release-3.16 August 13, 2026 21:10
@gmarciani gmarciani changed the title Wip/mgiacomo/3160/text proxy 0813 1 [Test] Stabilize test_proxy Aug 13, 2026
@gmarciani gmarciani added skip-changelog-update Disables the check that enforces changelog updates in PRs 3.x Test labels Aug 13, 2026
@gmarciani
gmarciani changed the base branch from release-3.16 to integ-tests-3.16.0 August 13, 2026 21:55
@gmarciani
gmarciani marked this pull request as ready for review August 13, 2026 21:55
@gmarciani
gmarciani requested review from a team as code owners August 13, 2026 21:55
@gmarciani
gmarciani enabled auto-merge (rebase) August 14, 2026 00:42
@gmarciani
gmarciani changed the base branch from integ-tests-3.16.0 to release-3.16 August 14, 2026 13:12
Comment on lines +885 to +886
- regions: [{{ c5_xlarge_CAPACITY_RESERVATION_4_INSTANCES_2_HOURS_NOPG_OS_X86_6 }}]
instances: ["c5.xlarge"]

@hanwen-cluster hanwen-cluster Aug 14, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this change. As long as the availability zone is aligned, the test framework make capacity reservation automatically for smaller instance types like .xlarge
See the PR description here #7461

@gmarciani
gmarciani merged commit 466966b into aws:release-3.16 Aug 14, 2026
19 checks passed
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