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

Stable/release GitHub action does not have version annotations #1085

Open
nderjung opened this issue Dec 11, 2023 · 9 comments
Open

Stable/release GitHub action does not have version annotations #1085

nderjung opened this issue Dec 11, 2023 · 9 comments
Assignees
Labels
kind/enhancement New feature or request priority/medium Issue which is important, but KraftKit is usable without it also.

Comments

@nderjung
Copy link
Member

Feature request summary

See: https://github.com/unikraft/kraftkit/actions/runs/7164563288/job/19505330651#step:8:933

Describe alternatives

No response

Related architectures

None

Related platforms

None

Additional context

No response

@nderjung nderjung added the kind/enhancement New feature or request label Dec 11, 2023
@craciunoiuc craciunoiuc added the priority/medium Issue which is important, but KraftKit is usable without it also. label Feb 29, 2024
@craciunoiuc
Copy link
Member

Became more important now as we do releases more often.

@camnwalter
Copy link
Contributor

Hi, I'm a computer science student at the University of Texas at Austin, and my partner and I would like to contribute and solve this issue. We are interested in working on this if it is still open. Thank you.

Can you explain it a bit more? The linked action has expired.

@nderjung
Copy link
Member Author

nderjung commented Apr 8, 2024

@craciunoiuc was this fixed?

@craciunoiuc
Copy link
Member

It's not fixed, I just manually rebuild after this finishes.

@camnwalter you can take a look if you want.

Basically the github-action:v0 image that is built here[1][2] contains a version of kraftkit with the version ~dirty. This makes the update-available prompt pop up everywhere in the CI which si a bit annoying.

To build the image correctly I use this line from @nderjung: DOCKER=docker IMAGE_TAG=blahblahblah make buildenv-myself && KRAFTKIT_VERSION=blahblahblah DOCKER=docker make buildenv-github-action

[1] https://github.com/unikraft/kraftkit/blob/staging/.github/workflows/github-action.yaml
[2] https://github.com/unikraft/kraftkit/blob/staging/buildenvs/github-action.Dockerfile

@camnwalter
Copy link
Contributor

It's not fixed, I just manually rebuild after this finishes.

@camnwalter you can take a look if you want.

Basically the github-action:v0 image that is built here[1][2] contains a version of kraftkit with the version ~dirty. This makes the update-available prompt pop up everywhere in the CI which si a bit annoying.

To build the image correctly I use this line from @nderjung: DOCKER=docker IMAGE_TAG=blahblahblah make buildenv-myself && KRAFTKIT_VERSION=blahblahblah DOCKER=docker make buildenv-github-action

[1] https://github.com/unikraft/kraftkit/blob/staging/.github/workflows/github-action.yaml [2] https://github.com/unikraft/kraftkit/blob/staging/buildenvs/github-action.Dockerfile

Did you fix this in 7867daa?

@craciunoiuc
Copy link
Member

I think so, but I'm not 100% sure

@nderjung
Copy link
Member Author

GHA still reports -dirty

@craciunoiuc
Copy link
Member

@camnwalter still up for grabs 😢

@camnwalter
Copy link
Contributor

Ok, I'll try looking into it

@craciunoiuc craciunoiuc self-assigned this Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request priority/medium Issue which is important, but KraftKit is usable without it also.
Projects
Status: Ready
Status: 🧊 Icebox
Development

No branches or pull requests

3 participants