Update kubeadm.md

[cortopy]

Kubeadm no longer deletes files in /etc/cni/net.d

What type of PR is this?

/kind documentation

What this PR does / why we need it:

I can see there was a long discussion about how kubeadm used to delete files in /etc/cni/net.d. Even the docs this commit modifies are mentioned there.

However, kubeadm merged this PR to solve the issue and it no longer deletes files in /etc/cni/net.d

In fact, when reseting a kubeadm cluster, there is this piece of information in the terminal:

The reset process does not clean CNI configuration. To do so, you must remove /etc/cni/net.d

Which issue(s) this PR fixes:

None

Special notes for your reviewer:

Does this PR introduce a user-facing change?

None

[openshift-ci-robot]

Hi @cortopy. Thanks for your PR.

I'm waiting for a cri-o member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-ci-robot]

Hi @cortopy. Thanks for your PR.

I'm waiting for a cri-o member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[cortopy]

/assign @umohnani8

[codecov]

Codecov Report

Merging #3538 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3538   +/-   ##
=======================================
  Coverage   43.97%   43.97%           
=======================================
  Files         100      100           
  Lines        7723     7723           
=======================================
  Hits         3396     3396           
  Misses       4030     4030           
  Partials      297      297           

[saschagrunert]

/ok-to-test

[saschagrunert]

LGTM. Thanks @cortopy 🙏

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cortopy, saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [saschagrunert]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cortopy, saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [saschagrunert]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[saschagrunert]

/retest

[haircommander]

However, kubeadm merged this PR to solve the issue and it no longer deletes files in /etc/cni/net.d

that's awesome to hear, the clearing of the directory always confused me and was a source of headaches for new users

thank you very much for the patch @cortopy

/lgtm

[haircommander]

/retest

[haircommander]

Apr 08 13:21:43 ip-172-18-15-16.ec2.internal crio[15608]: time="2020-04-08 13:21:43.169589297Z" level=warning msg="Failed to open /etc/passwd: open /var/lib/containers/storage/overlay/87939d719a64e1ea148acacc182b941beaa6098389cd1617017c21aaf57352af/merged/etc/passwd: no such file or directory"
Apr 08 13:21:43 ip-172-18-15-16.ec2.internal crio[15608]: time="2020-04-08 13:21:43.169931486Z" level=warning msg="Failed to open /etc/group: open /var/lib/containers/storage/overlay/87939d719a64e1ea148acacc182b941beaa6098389cd1617017c21aaf57352af/merged/etc/group: no such file or directory"

this is the only error I see that could be related to the failure here

unrelated to this PR though,
/retest

[openshift-ci-robot]

@cortopy: The following test failed, say /retest to rerun all failed tests:

Test name	Commit	Details	Rerun command
ci/openshift-jenkins/e2e_features_fedora	eda8c6e	link	/test e2e_features_fedora

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[haircommander]

happened again...