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

removing weave from the nodes #4570

Merged
merged 1 commit into from
May 31, 2023
Merged

removing weave from the nodes #4570

merged 1 commit into from
May 31, 2023

Conversation

camilamacedo86
Copy link
Contributor

@camilamacedo86 camilamacedo86 commented May 31, 2023

What this PR does / why we need it:

Note all files has been removed from the node:

$ curl -fsSL https://staging.kurl.sh/version/v2023.05.30-0-8d6f16d6/f0a3e1c/tasks.sh | sudo bash -s weave-to-flannel-primary cert-key=837a1a60ef6b88be8cf70ede1edf5b701e7e6f06cbe0e13f75c227028701d006
⚙  Running tasks with the argument(s): weave-to-flannel-primary cert-key=837a1a60ef6b88be8cf70ede1edf5b701e7e6f06cbe0e13f75c227028701d006
W0531 16:27:12.994985   59229 common.go:84] your configuration file uses a deprecated API spec: "kubeadm.k8s.io/v1beta2". Please use 'kubeadm config migrate --old-config old.yaml --new-config new.yaml', which will write the new, similar spec using a newer API version.
[preflight] Reading configuration from the cluster...
[preflight] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -o yaml'
[control-plane] Using manifest folder "/etc/kubernetes/manifests"
[control-plane] Creating static Pod manifest for "kube-apiserver"
[control-plane] Creating static Pod manifest for "kube-controller-manager"
[control-plane] Creating static Pod manifest for "kube-scheduler"
✔ Successfully updated camila-ubuntu-2004-lts-task-node-a to use Flannel
camila@camila-ubuntu-2004-lts-task-node-a:~$ ls -la /var/lib/weave
total 40
drwxr-xr-x  2 root root  4096 May 31 15:41 .
drwxr-xr-x 48 root root  4096 May 31 15:43 ..
-rw-r-----  1 root root 65536 May 31 16:21 weave-netdata.db

Therefore, this PR perform the same changes for master: #4551

Which issue(s) this PR fixes:

Fixes # [sc-77764]

Special notes for your reviewer:

  • Create a master and 3 nodes to migrate from weave to flannel
  • Then, check the files in the node.

Steps to reproduce

Does this PR introduce a user-facing change?

Fixes weave uninstall by removing interfaces first and began to remove lib/weave from nodes when the task is used to update them in the upgrade process. 

Does this PR require documentation?

@camilamacedo86 camilamacedo86 requested a review from a team as a code owner May 31, 2023 16:41
@camilamacedo86 camilamacedo86 added type::bug Something isn't working bug::normal labels May 31, 2023
@camilamacedo86 camilamacedo86 merged commit 16a9e42 into main May 31, 2023
@camilamacedo86 camilamacedo86 deleted the weave-node-task branch May 31, 2023 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug::normal type::bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants