Skip to content

Publish container image for v0.1.0 release #85

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

Closed
achandrasekar opened this issue May 22, 2025 · 1 comment · Fixed by #97
Closed

Publish container image for v0.1.0 release #85

achandrasekar opened this issue May 22, 2025 · 1 comment · Fixed by #97
Assignees
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Milestone

Comments

@achandrasekar
Copy link
Contributor

We need a publicly accessible container image when we ship v0.1.0 release.

@achandrasekar achandrasekar added this to the v0.1.0 milestone May 22, 2025
@achandrasekar achandrasekar added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label May 22, 2025
@achandrasekar
Copy link
Contributor Author

/assign @wangchen615

achandrasekar pushed a commit that referenced this issue May 29, 2025
This PR implements automated Docker image publishing (to
quay.io/inference-perf) for both PR merges and releases.

In detail, the changes include:

- A new workflow (in .github/workflows/docker‑build.yml) that triggers
on PR merges (to main) and builds (and pushes) a Docker image (tagged
with the PR number and “latest” for the main branch).
- An update to the release workflow (in .github/workflows/release.yml)
so that (on a version tag) a Docker image is also built and pushed
(tagged with the release version and “latest”).
- Both workflows use GitHub Actions’ caching (via “cache‑from” and
“cache‑to”) for faster builds.

**Required Secrets:**
QUAY_USERNAME (your quay.io username)
QUAY_PASSWORD (your quay.io password or token)

**Testing:**
[x] Verified that the Docker image builds successfully (and is amd64).
[x] Tested locally (using Podman).
[x] (Note: The quay.io repository (quay.io/inference‑perf) must be
created before the first push.)

Pull this container with the following Docker command:
```
docker pull quay.io/inference-perf/inference-perf:latest
```

Closes #85

---------

Co-authored-by: Copilot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants