fix: use RWMutex lock while concurrent read/write (#3462)

Signed-off-by: prateekpandey14 <prateek.pandey@nirmata.com>

pkg/webhooks/generation.go

@@ -207,8 +207,10 @@ func (ws *WebhookServer) updateAnnotationInGR(gr *kyverno.GenerateRequest, logge
 	if len(grAnnotations) == 0 {
 		grAnnotations = make(map[string]string)
 	}
+	ws.Lock()
 	grAnnotations["generate.kyverno.io/updation-time"] = time.Now().String()
 	gr.SetAnnotations(grAnnotations)
+	ws.Unlock()
 	_, err := ws.kyvernoClient.KyvernoV1().GenerateRequests(config.KyvernoNamespace).Update(contextdefault.TODO(), gr, metav1.UpdateOptions{})
 	if err != nil {
 		logger.Error(err, "failed to update generate request for the resource", "generate request", gr.Name)

pkg/webhooks/server.go

@@ -7,6 +7,7 @@ import (
 	"fmt"
 	"io/ioutil"
 	"net/http"
+	"sync"
 	"time"
 
 	"github.com/go-logr/logr"
@@ -124,6 +125,8 @@ type WebhookServer struct {
 	grController *generate.Controller
 
 	promConfig *metrics.PromConfig
+
+	sync.RWMutex
 }
 
 // NewWebhookServer creates new instance of WebhookServer accordingly to given configuration