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

patch: updated all of the versions - ALL OF THEM! #306

Merged
merged 1 commit into from
Sep 6, 2023
Merged

Conversation

drew-viles
Copy link
Contributor

@drew-viles drew-viles commented Sep 4, 2023

  • New unikorn release with a note on deprecating and removing cilium taints from clusteropenstack helm application as this is now provided via the cilium operator as of 1.14.0 - CNI: keep cni config on shutdown; taint instead, queue deletions cilium/cilium#23486
    We can't do this yet as it would break any apps pre kubernetes-cluster-1.4.0
  • Updates a bunch of other packages with a new preview application bundle for both clusters and control plane
  • Some minor doc corrections

A note worth making on this is I cannot for the life of me get a current cluster spun up in kind. It's failing to spin up vcluster when using control-plane bundle 1.1.0 - the error is annoyingly only exec /coredns: operation not permitted which is in practice useless. I've compared the vcluster I've got with some that are running in eckmgt and it's exactly the same.

So, I can't test an upgrade on this so if you fancy giving a quick crack at the upgrade I'd be greatly appreciative.
That all being said I have tested running the new version that's been created and all is working just nicely.

@drew-viles drew-viles force-pushed the CLDNTV-266 branch 4 times, most recently from 800ee78 to bbb1aa3 Compare September 5, 2023 11:08
* New unikorn release with a note on deprecating and removing cilium taints from `clusteropenstack` helm application as this is now provided via the cilium operator as of 1.14.0 - cilium/cilium#23486
  We can't do this yet as it would break any apps pre kubernetes-cluster-1.4.0
* Updates a bunch of other packages with a nuew preview application bundle for both clusters and control plane
* Some minor doc corrections
@yankcrime
Copy link
Contributor

I've hit the same vcluster error and came across loft-sh/vcluster#1141 - upgrading to a newer version of vcluster itself suggests that'll fix the problem. Was this not the case in your testing?

@drew-viles
Copy link
Contributor Author

So I've been going on the approach of wanting to upgrade from what's deployed in eckmgt to the new version. A newer version of vcluster is working on my new 1.2.0 which is vcluster 0.15.7 (original is 0.15.0.

I'm happy to bump it in the old one on my dev cluster but 0.15.0 is working on eckmgt so a bit confused.

At least I can test a basic upgrade of everything else though so let me check that over

@yankcrime
Copy link
Contributor

There'll be some difference in what kubeadm does versus how kind (or K3s itself in my case) does things, since this suggested fix did the trick for me without having to upgrade.

@drew-viles
Copy link
Contributor Author

I've managed to now test the upgrade and new cluster rollout and all is working just fine. I had a couple resource running that remained accessible the entire time and all looks good.
Happy days.

@drew-viles drew-viles merged commit 6c8516f into main Sep 6, 2023
2 checks passed
@drew-viles drew-viles deleted the CLDNTV-266 branch September 6, 2023 13:51
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.

None yet

2 participants