Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Test] In test_fsx_lustre_dra, remove unnecessary update of DRA1 that may cause cluster update failure. #6649

Open
wants to merge 3 commits into
base: integ-tests-3.12.0
Choose a base branch
from

Conversation

gmarciani
Copy link
Contributor

@gmarciani gmarciani commented Jan 31, 2025

Description of changes

[Test] In test_fsx_lustre_dra, remove unnecessary update of DRA1 that may cause cluster update failure.
In fact, there is a known FSx limitation that can cause FSx timeout when multiple DRAs are updated concurrently.

Tests

  • test_fsx_lustre_dra succeeded

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 Jan 31, 2025
@gmarciani gmarciani requested review from a team as code owners January 31, 2025 15:17
… may cause cluster update failure.

In fact, there is a known FSx limitation that can cause FSx timeout when multiple DRAs are updated concurrently.
@gmarciani gmarciani force-pushed the wip/mgiacomo/3120/test-dra-fix-0131-1 branch from 21f95e0 to 474bde4 Compare January 31, 2025 15:20
gmarciani and others added 2 commits January 31, 2025 10:32
Error output on Python <= 3.11:
```
error: argument --node-type: invalid choice: 'invalid' (choose from 'HeadNode', 'ComputeNode', 'LoginNode')
```

Error output on Python 3.12:
```
error: argument --node-type: invalid choice: 'invalid' (choose from HeadNode, ComputeNode, LoginNode
```

Therefore, this commit uses regex to add more flexibility to the tests

Signed-off-by: Hanwen <hanwenli@amazon.com>
@gmarciani gmarciani closed this Feb 8, 2025
auto-merge was automatically disabled February 8, 2025 05:24

Pull request was closed

@gmarciani gmarciani reopened this Feb 8, 2025
@gmarciani gmarciani enabled auto-merge (rebase) February 8, 2025 05:24
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.

3 participants