Revert "nfd-master: use only unbuffered chans in the nfd api-controller"

Fixes a bug where updates to NodeFeatureGroup objects were not happening
after the initial sync.

We need to use buffered channels for the "update all" channels as there
is the default: case in select that is sending the update all signal (to
the channel). An alternative would be to remove the default: case from
the select block but it is more efficient this way - just skip if the
reader side hasn't yet got the "update all" signal.

This reverts commit bf6ffad.

Author: marquiz

pkg/nfd-master/nfd-api-controller.go

@@ -66,9 +66,9 @@ func init() {
 func newNfdController(config *restclient.Config, nfdApiControllerOptions nfdApiControllerOptions) (*nfdController, error) {
 	c := &nfdController{
 		stopChan:                       make(chan struct{}),
-		updateAllNodesChan:             make(chan struct{}),
+		updateAllNodesChan:             make(chan struct{}, 1),
 		updateOneNodeChan:              make(chan string),
-		updateAllNodeFeatureGroupsChan: make(chan struct{}),
+		updateAllNodeFeatureGroupsChan: make(chan struct{}, 1),
 		updateNodeFeatureGroupChan:     make(chan string),
 	}