Skip to content

test(sagemaker): add HyperPod E2E tests#8793

Draft
aws-ajangg wants to merge 1 commit into
aws:masterfrom
aws-ajangg:hyperpod-e2e-tests
Draft

test(sagemaker): add HyperPod E2E tests#8793
aws-ajangg wants to merge 1 commit into
aws:masterfrom
aws-ajangg:hyperpod-e2e-tests

Conversation

@aws-ajangg

Copy link
Copy Markdown
Contributor

Problem

  • HyperPod SSH connection flows have no end-to-end test coverage against real AWS infrastructure. There's no automated way to verify that the full chain (authenticate → list clusters → kubectl → workspace connection) works correctly.

Solution

  • Add E2E tests for HyperPod SSH connections that exercise real AWS APIs (SageMaker, EKS, K8s).

3 tests covering critical user journeys:

  • Side panel → list spaces and verify status
  • Side panel → connect to running space → workspace connection established
  • Deeplink → connect to running space → workspace connection established

Tests skip gracefully when prerequisites (credentials, clusters, running spaces) are not met.

How to run

ada credentials update --account= --provider=isengard --role=Admin --once --profile=
eval "$(AWS_PROFILE= aws configure export-credentials --format env)"
AWS_DEFAULT_REGION= TEST_DIR="../../core/dist/src/testE2E/sagemaker" npm run testE2E -w packages/toolkit


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

E2E tests verify critical user journeys with real AWS infrastructure:
- Side panel → list spaces and verify status
- Side panel → connect to running space → workspace connection established
- Deeplink → connect to running space → workspace connection established

Tests skip gracefully when prerequisites (credentials, clusters) are not met.
@amazon-inspector-ohio

Copy link
Copy Markdown

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@amazon-inspector-ohio

Copy link
Copy Markdown

✅ I finished the code review, and didn't find any security or code quality issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant