Skip to content

Conversation

@yhrn
Copy link

@yhrn yhrn commented Apr 30, 2023

All the other dep bumps are a direct result of bumping vcluster. Except for the default chart version all code changes were forced by breaking changes in the dependencies.

The vendoring makes this PR huge but I don't see any other way of doing this. I have tested that with this change creating a k8s vCluster v0.15.0 (Kubernetes v1.26.1) works when using Helm v3.11.1.

All the other dep bumps are a direct result of bumping vcluster.
Except for the default chart version all code changes were forced by
breaking changes in the dependencies.
@yhrn
Copy link
Author

yhrn commented Apr 30, 2023

If this PR is accepted it would be great if it would be followed by a new release.

@FabianKramm
Copy link
Member

@yhrn thanks a lot for the PR! Sorry for letting this hang so long, LGTM!

@FabianKramm FabianKramm merged commit f3139cc into loft-sh:main May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants