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

not all container images are being tagged in quay.io #641

Closed
jacobweinstock opened this issue Sep 12, 2022 · 0 comments · Fixed by #645
Closed

not all container images are being tagged in quay.io #641

jacobweinstock opened this issue Sep 12, 2022 · 0 comments · Fixed by #645
Assignees

Comments

@jacobweinstock
Copy link
Member

jacobweinstock commented Sep 12, 2022

This repo creates multiple container images: tink server, tink controller, tink cli, tink worker. Currently, in code, we tag all of these different binaries together. It appears that only the tink server container image has the associated tag of v0.7.0 in quay.io.

Expected Behaviour

I would like for all container images pushed to quay.io to be tagged.

Current Behaviour

Possible Solution

It appears that our GitHub action for tags current just supports a single image. https://github.com/tinkerbell/tink/blob/main/.github/workflows/tags.yaml We'll probably want to modify this to accommodate all the container images we create.

Steps to Reproduce (for bugs)

Context

Your Environment

  • Operating System and version (e.g. Linux, Windows, MacOS):

  • How are you running Tinkerbell? Using Vagrant & VirtualBox, Vagrant & Libvirt, on Packet using Terraform, or give details:

  • Link to your project or a code example to reproduce issue:

@mergify mergify bot closed this as completed in #645 Oct 4, 2022
mergify bot added a commit that referenced this issue Oct 4, 2022
## Description


Tags were not being pushed to quay.io for services other than quay.io/tinkerbell/tink, this tags tink-controller, tink-worker, and tink-cli with tags as well.

## Why is this needed



Fixes: #641 

## How Has This Been Tested?





## How are existing users impacted? What migration steps/scripts do we need?





## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants