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

add a command to list kernel tags and pull kernels built by little-vm-helper-images #184

Merged
merged 6 commits into from
Apr 18, 2024

Conversation

mtardy
Copy link
Member

@mtardy mtardy commented Apr 11, 2024

I could also replace the lvh images pull implementation so that it doesn't require a docker daemon (can also remove the dependency).

This is to prepare the addition of two new commands: get and catalog
that won't use the flag and thus we can no longer set the flag as
persistent on the kernels function.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy mtardy added the enhancement New feature or request label Apr 11, 2024
@mtardy mtardy requested a review from kkourt April 11, 2024 11:49
@mtardy mtardy requested a review from joestringer April 11, 2024 12:37
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm excited to see this! I didn't look too closely at the code yet, but I did wonder about one thing about the usability with being able to just copy+paste a container image URL to specify a kernel to pull, see the thread below.

README.md Show resolved Hide resolved
cmd/lvh/kernels/pull.go Show resolved Hide resolved
vendor/modules.txt Outdated Show resolved Hide resolved
cmd/lvh/kernels/build.go Show resolved Hide resolved
This command allows to list the built kernels by LVHI.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
This one uses the google/go-containerregistry (or crane) library that
can perform actions without a docker daemon running.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a couple of minor nits where I would have made different decisions but if you feel strongly about them then that feedback doesn't need to be applied.

cmd/lvh/main.go Show resolved Hide resolved
cmd/lvh/kernels/pull.go Show resolved Hide resolved
cmd/lvh/kernels/pull.go Outdated Show resolved Hide resolved
Allow the user to provide the repo and provide examples for the CI
repository, rather than hardcoding lvh org OCI repos and CI flag.

Signed-off-by: Joe Stringer <joe@isovalent.com>
Copy link
Collaborator

@kkourt kkourt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@kkourt kkourt merged commit 74f265a into main Apr 18, 2024
2 checks passed
@kkourt kkourt deleted the pr/mtardy/add-get-kernel branch April 18, 2024 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants