-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: add test script for cli server #39
Conversation
id: push-worker | ||
uses: docker/build-push-action@v5 | ||
with: | ||
context: . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add load: true
ref :https://docs.docker.com/build/ci/github-actions/export-docker/
.github/workflows/cli_test.yaml
Outdated
docker ps | ||
- name: Run test script | ||
run: | | ||
sh -C tests/example_test.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the top part is similar to https://github.com/kubeagi/core-library/blob/main/.github/workflows/image_build_test.yml, so I guess you could just modify that file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I simply cp those into the test action yaml for sure. Could we merge all tests together or leave it here as we might need other tests that would be better executed seperately?
7f8de0b
to
9019056
Compare
.github/workflows/cli_test.yaml
Outdated
PYTHON_INDEX_URL=${{ env.PYTHON_INDEX_URL }} | ||
- name: Run Docker Container | ||
run: | | ||
docker run -d --name temp-container test-image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
temp-container -> cli-container
4accb05
to
8334d51
Compare
What type of PR is this?
/kind feature
What this PR does / why we need it
Which issue(s) this PR fixes
Fixes #38
Special notes for your reviewer