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(sealos): e2e test for sealos #25

Merged
merged 1 commit into from
Oct 3, 2024
Merged

test(sealos): e2e test for sealos #25

merged 1 commit into from
Oct 3, 2024

Conversation

cuisongliu
Copy link
Owner

@cuisongliu cuisongliu commented Oct 3, 2024

This pull request introduces new jobs in the GitHub Actions workflows and includes several other improvements and fixes. The most important changes include adding new jobs to the CI workflows, updating commands to use the repository owner dynamically, and modifying the Makefile to streamline the testing process.

CI Workflow Enhancements:

  • .github/workflows/push.yml: Added a new job (job1) to build and push images using sealos. This job includes steps for setting up Go, verifying sealos, building images, and pushing them to the repository.
  • .github/workflows/test.yml: Added a new job (job2) similar to job1 in push.yml, which includes steps for checking out the code, setting up Go, verifying sealos, and building images.

Command Updates:

Makefile Improvements:

  • Makefile: Simplified the test and e2e targets by removing unnecessary dependencies on manifests and generate. [1] [2]

Code Fixes:

  • gen/gen.go: Added a condition to handle the "latest" version by setting it to "0.0.0" to avoid issues with version replacement in the Chart.yaml file.

@cuisongliu cuisongliu force-pushed the e2e-sealos branch 3 times, most recently from 5830526 to fa2eb90 Compare October 3, 2024 02:49
@cuisongliu cuisongliu merged commit 6f3a279 into main Oct 3, 2024
6 checks passed
@cuisongliu cuisongliu deleted the e2e-sealos branch October 3, 2024 02:56
@github-actions github-actions bot mentioned this pull request Oct 3, 2024
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