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

HAMi supports e2e #775

Merged
merged 1 commit into from
Jan 9, 2025
Merged

HAMi supports e2e #775

merged 1 commit into from
Jan 9, 2025

Conversation

Rei1010
Copy link
Collaborator

@Rei1010 Rei1010 commented Jan 3, 2025

What type of PR is this?
/kind feature

What this PR does / why we need it:
Implement initial E2E test suite setup

Which issue(s) this PR fixes:
Fixes #476

Special notes for your reviewer:
Workflow: Build image and Helm chart, upload them to artifacts, run e2e by downloading artifacts

Does this PR introduce a user-facing change?:

@Rei1010 Rei1010 changed the title Implement initial E2E test suite setup [Draft] Implement initial E2E test suite setup Jan 3, 2025
@Rei1010 Rei1010 changed the title [Draft] Implement initial E2E test suite setup Implement initial E2E test suite setup Jan 6, 2025
Signed-off-by: wen.rui <wen.rui@daocloud.io>
@chaunceyjiang
Copy link
Contributor

other LGTM

@Rei1010 Rei1010 requested a review from chaunceyjiang January 7, 2025 09:44
@wawa0210 wawa0210 added the kind/feature new function label Jan 8, 2025
@wawa0210 wawa0210 changed the title Implement initial E2E test suite setup HAMi supports e2e Jan 8, 2025
@lengrongfu
Copy link
Member

首先总体来说e2e设计的很好,通过在e2e中引入matrix可以适配不同的芯片;但是我有三个优化建议,可以作为后续优化方向:

@lengrongfu
Copy link
Member

/lgtm

@Rei1010
Copy link
Collaborator Author

Rei1010 commented Jan 8, 2025

首先总体来说e2e设计的很好,通过在e2e中引入matrix可以适配不同的芯片;但是我有三个优化建议,可以作为后续优化方向:

Add a todo list for this #788

@archlitchi
Copy link
Collaborator

/lgtm

@archlitchi archlitchi merged commit 594ef64 into Project-HAMi:master Jan 9, 2025
13 checks passed
haitwang-cloud pushed a commit to haitwang-cloud/HAMi that referenced this pull request Jan 10, 2025
Signed-off-by: wen.rui <wen.rui@daocloud.io>
Signed-off-by: haitwang-cloud <haitao_wht@outlook.com>
haitwang-cloud pushed a commit to haitwang-cloud/HAMi that referenced this pull request Jan 10, 2025
…and processing

fix: improve UUID parsing and error handling in ExtractMigTemplatesFromUUID function
refactor: update test cases for ExtractMigTemplatesFromUUID with comprehensive scenarios
chore: clean up imports and enhance test readability across various test files

Signed-off-by: haitwang-cloud <haitao_wht@outlook.com>

fix: nvidia-device-plugin no version info (Project-HAMi#779)

Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
Signed-off-by: haitwang-cloud <haitao_wht@outlook.com>

Implement initial E2E test suite setup (Project-HAMi#775)

Signed-off-by: wen.rui <wen.rui@daocloud.io>
Signed-off-by: haitwang-cloud <haitao_wht@outlook.com>

Draft E2E test design (Project-HAMi#633)

Signed-off-by: Rei1010 <wen.rui@daocloud.io>
Signed-off-by: wen.rui <wen.rui@daocloud.io>
Signed-off-by: haitwang-cloud <haitao_wht@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature new function
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HAMi SANDBOX PROJECT ONBOARDING task list
5 participants