Skip to content

[Test] Tolerate intermittent dcvsessionlauncher crash on g5g instances on AL23 associated to known issue in DCV#7404

Merged
gmarciani merged 1 commit into
aws:developfrom
gmarciani:wip/mgiacomo/3160/test-dcv-0522-1
May 22, 2026
Merged

[Test] Tolerate intermittent dcvsessionlauncher crash on g5g instances on AL23 associated to known issue in DCV#7404
gmarciani merged 1 commit into
aws:developfrom
gmarciani:wip/mgiacomo/3160/test-dcv-0522-1

Conversation

@gmarciani
Copy link
Copy Markdown
Contributor

@gmarciani gmarciani commented May 22, 2026

Description of changes

In DCV tests, tolerate harmless crash associated to known bug in DCV.
When this crash creates a problem for pcluster, the test will capture the impact with another soft assertion that capture the dcv connection failure.

We are already tolerating this crash, but the regex was not able to match the crash signature on AL2023.

Tests

SUCCESS test_dcv_configuration on UBU22, Ubu24, AL23. Verified that the test capotures and tolerate the crash:

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

@gmarciani gmarciani added skip-changelog-update Disables the check that enforces changelog updates in PRs 3.x Test labels May 22, 2026
@gmarciani gmarciani marked this pull request as ready for review May 22, 2026 14:04
@gmarciani gmarciani requested review from a team as code owners May 22, 2026 14:04
@gmarciani gmarciani changed the title [Test] Fix dcv pattern matching crash. [Test] Tolerate intermittent dcvsessionlauncher crash on g5g instances on AL23 associated to known issue in DCV May 22, 2026
@gmarciani gmarciani force-pushed the wip/mgiacomo/3160/test-dcv-0522-1 branch from 39e33b8 to c00804a Compare May 22, 2026 14:06
@gmarciani gmarciani merged commit 1d5043b into aws:develop May 22, 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