Update operator version parsing

Summary: there were a lot of spammy `Failed to parse current Node Kernel version` messages for people. there is some additional cleanup of the version string that we do for our checks in `px deploy`, which should be done here as well.

Test Plan: skaffold ran operator

Reviewers: vihang, philkuz, zasgar

Reviewed By: vihang
Signed-off-by: Michelle Nguyen <michellenguyen@pixielabs.ai>

Differential Revision: https://phab.corp.pixielabs.ai/D9907

GitOrigin-RevId: ca3ee637c7c1a6d1f1f3099a6b430e401e84eaaf

src/operator/controllers/node_watcher.go

@@ -20,6 +20,7 @@ package controllers
 
 import (
 	"context"
+	"strings"
 	"time"
 
 	"github.com/blang/semver"
@@ -46,7 +47,18 @@ var (
 )
 
 func nodeIsCompatible(node *v1.Node) bool {
-	currentSemVer, err := semver.Make(node.Status.NodeInfo.KernelVersion)
+	version := node.Status.NodeInfo.KernelVersion
+	// We don't actually care about pre-release tags, so drop them since they sometimes cause parse error.
+	sp := strings.Split(version, "-")
+	if len(sp) == 0 {
+		return true
+	}
+	version = sp[0]
+	version = strings.TrimPrefix(version, "v")
+	// Minor version can sometime contain a "+", we remove it so it parses properly with semver.
+	version = strings.TrimSuffix(version, "+")
+
+	currentSemVer, err := semver.Make(version)
 	if err != nil {
 		log.WithError(err).Error("Failed to parse current Node Kernel version")
 		return true