-
Notifications
You must be signed in to change notification settings - Fork 13
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
chore: enable kres #17
Conversation
fe23db1
to
1fb298c
Compare
To all parties involved, I would like to make a strong case to have this tool added to the official extensions repo. While standard Talos users can continue to use this tool the original way designed, having this be an extension simplifies implementation as extensions can be baked into the image so that now all vSphere VMs can enable this right from clone/deploy. The simplest and easiest way to do that I believe is using Image Factory. The biggest reason honestly to add this as an official extension is for Omni I think. Being able to create an ova from within Omni with the vmtoolsd extension already added would be great. An even bigger reason is for upgradability. While now Omni connectivity settings can be added to custom Talos builds with custom extensions, not unifying these two repos breaks Omni's design of managing and upgrading these VMs as Omni does not allow for incorporation of non official extensions into images built from within it's interface. At least I believe so; who knows, could be wrong. So again, I hope my reasons sound well.. reasonable. I actually do have a commit burning a hole in my fork of the extensions repo begging for a pull request but I digress and will wait for the maintainers of this repo to come to a decision. |
Hi @JinxCappa, I agree having Note however that this repository is a community effort, and none of the maintainers are affiliated with Sidero. They just offered to host the repository under `siderolabs/'. So my advice would be to just create that PR, parallel to what we are doing here. |
I appreciate the response and am looking forward to submitting my PR. As a question to @frezbo, as this repo stands now the releases are not tagged with 'v{semver}' but just the plain version instead. With the addition of kres to this repo, will this be changed to match the release format of siderolabs repos? I ask as this will affect pulling the source to build the extension as well as the renovate regex comment in the PR. View the below links for reference: If so, I don't have an issue submitting my PR to extensions but note that it will have to be modified if this PR gets finalized and reformats release tags. |
Yes, it would starting with |
Enable kres and update repo branch settings. Fixes: siderolabs#14 Signed-off-by: Noel Georgi <[email protected]>
@frezbo thanks! Container generated using the modified Makefile works as DaemonSet as well as Talos System Extension. Judging by the new github workflow code, we would still need to manually create tags, right? |
yes, just make sure it has the |
if you could +1 (approval), I can then get it merged |
/m |
Enable kres and update repo branch settings.
Notable changes:
Makefile
,Dockerfile
, Github Actions workflow etc are autogenerated, run amake rekres
to update things..kres.yaml
, additional options can be added and amake rekres
should update all..kres.yaml
)Pull requests require status checks and approval.
Usually a PR can be merged by adding a
/m
comment. We do this in the org repos to avoid merge commits that has no trusted GPG signature (GitHub merge commits doesn;t do aff
merge and adds their own GPG signature)I have disabled the GPG signature to be enforced for a commit.
Fixes: #14