SubnetManager should use the main context

[manuelbuil]

Description

For some unknown reason, subnetManager was using a different context than the main one, thus the cancel() method to clean up flannel resources will never work in this case.

This PR fixes that and subnetManager will use the main context

Todos

• Tests
• Documentation
• Release note

Release Note

None required

[thomasferrandiz]

I just noticed that this context is wrong as well:

flannel/pkg/subnet/kube/kube.go

Line 612 in f5b88d9

node, err := ksm.client.CoreV1().Nodes().Get(context.TODO(), ksm.nodeName, metav1.GetOptions{})

Context.TODO is only for dev

[manuelbuil]

I just noticed that this context is wrong as well:

flannel/pkg/subnet/kube/kube.go

Line 612 in f5b88d9

node, err := ksm.client.CoreV1().Nodes().Get(context.TODO(), ksm.nodeName, metav1.GetOptions{})

Context.TODO is only for dev

Should it be Context.Background? I wonder if we could do something to pipe the main context

[thomasferrandiz]

I think we can get the main context because GetStoredMacAddress is called in RegisterNetwork where it is available.

[manuelbuil]

I think we can get the main context because GetStoredMacAddress is called in RegisterNetwork where it is available.

done!