feature: yanet2 init commit

[borislitv]

No description provided.

[github-actions]

🤖 AI Code Review

Changed files: api/v2alpha1/groupversion_info.go,api/v2alpha1/yanet_types.go,api/v2alpha1/yanet_webhook.go,api/v2alpha1/yanet_webhook_test.go,api/v2alpha1/yanetconfig_types.go,api/v2alpha1/yanetconfig_webhook.go,api/v2alpha1/yanetconfig_webhook_test.go,api/v2alpha1/zz_generated.deepcopy.go,internal/controller/yanet_reconciler_v2.go,internal/helpers/merge_v2.go,internal/helpers/merge_v2_test.go,internal/helpers/nodes_v2.go,internal/manifests/builder_v2.go,internal/manifests/controlplane_v2.go,internal/manifests/dataplane_v2.go,api/v1alpha1/yanet_types.go,api/v1alpha1/yanetconfig_types.go,api/v1alpha1/zz_generated.deepcopy.go,cmd/main.go,internal/controller/yanet_controller.go

This automated review checks for:

• 🔒 Race conditions and concurrency issues
• 🔗 Potential deadlocks
• ⚠️ Error handling patterns
• 🚀 Goroutine lifecycle management
• 📋 Context usage
• 🎯 Nil safety

🔍 Static Analysis Results

📊 Staticcheck

✅ No issues found

🔒 Security Analysis (gosec)

✅ No security issues found

🔧 Go Vet

✅ No issues found

🏁 Race Detector

✅ No race conditions detected

🚀 Concurrency Patterns Analysis

Goroutines found:

./internal/controller/suite_test.go:108:	go func() {
./internal/controller/yanet_reconciler_test.go:131:		go func(hostNum int) {
./api/v1alpha1/groupversion_info.go:31:	// SchemeBuilder is used to add go types to the GroupVersionKind scheme
./api/v2alpha1/groupversion_info.go:31:	// SchemeBuilder is used to add go types to the GroupVersionKind scheme

Mutex usage:

./internal/controller/yanet_controller.go:44:	lock           sync.Mutex
./internal/controller/yanet_reconciler_test.go:90:				lock:           sync.Mutex{},
./internal/controller/yanet_reconciler_test.go:119:		lock:           sync.Mutex{},
./api/v1alpha1/yanetconfig_types.go:58:	Lock   sync.Mutex      `json:"-"`
./api/v2alpha1/yanetconfig_types.go:75:	Lock   sync.Mutex      `json:"-"`

Context usage:

./internal/controller/suite_test.go:59:	ctx, cancel = context.WithCancel(context.TODO())
./internal/controller/yanet_controller_integration_test.go:49:		ctx := context.Background()
./internal/controller/yanet_controller_integration_test.go:402:		ctx := context.Background()
./internal/controller/node_deletion_test.go:130:			ctx := context.Background()
./internal/manifests/dataplane_test.go:78:	dep := DeploymentForDataplane(context.Background(), yanet, config, nodes)
./internal/manifests/dataplane_test.go:143:	dep := DeploymentForDataplane(context.Background(), yanet, config, nodes)
./internal/manifests/dataplane_test.go:166:	dep := DeploymentForDataplane(context.Background(), yanet, config, nodes)
./internal/manifests/controlplane_test.go:83:	dep := DeploymentForControlplane(context.Background(), yanet, config, nodes)
./internal/manifests/controlplane_test.go:219:	dep := DeploymentForControlplane(context.Background(), yanet, config, nodes)
./internal/manifests/controlplane_test.go:243:	dep := DeploymentForControlplane(context.Background(), yanet, config, nodes)
./internal/manifests/controlplane_test.go:277:	dep := DeploymentForControlplane(context.Background(), yanet, config, nodes)
./internal/manifests/helpers_test.go:251:	ctx := context.Background()
./internal/manifests/announcer_test.go:82:	dep := DeploymentForAnnouncer(context.Background(), yanet, config, nodes)
./internal/manifests/announcer_test.go:180:	dep := DeploymentForAnnouncer(context.Background(), yanet, config, nodes)
./internal/manifests/announcer_test.go:246:	dep := DeploymentForAnnouncer(context.Background(), yanet, config, nodes)
./internal/manifests/bird_test.go:83:	dep := DeploymentForBird(context.Background(), yanet, config, nodes)
./internal/manifests/bird_test.go:185:	dep := DeploymentForBird(context.Background(), yanet, config, nodes)
./internal/manifests/bird_test.go:257:	dep := DeploymentForBird(context.Background(), yanet, config, nodes)
./internal/manifests/bird_test.go:282:	dep := DeploymentForBird(context.Background(), yanet, config, nodes)
./internal/helpers/helpers_test.go:33:	ctx := context.Background()
./internal/helpers/helpers_test.go:228:	ctx := context.Background()
./internal/helpers/helpers_test.go:398:	ctx := context.Background()
./internal/helpers/helpers_test.go:542:	ctx := context.Background()
./api/v1alpha1/yanetconfig_webhook_test.go:129:			ctx := context.Background()
./api/v1alpha1/yanetconfig_webhook_test.go:205:			ctx := context.Background()
./api/v1alpha1/yanetconfig_webhook_test.go:230:	ctx := context.Background()
./api/v1alpha1/yanet_webhook_test.go:104:			ctx := context.Background()
./api/v1alpha1/yanet_webhook_test.go:222:			ctx := context.Background()
./api/v1alpha1/yanet_webhook_test.go:248:	ctx := context.Background()
./api/v2alpha1/yanetconfig_webhook_test.go:87:			_, err := tt.yanetConfig.ValidateCreate(context.Background(), tt.yanetConfig)
./api/v2alpha1/yanet_webhook_test.go:117:			_, err := tt.yanet.ValidateCreate(context.Background(), tt.yanet)

Defer patterns:

./internal/controller/yanet_reconciler.go:50:	defer r.lock.Unlock()
./internal/controller/yanet_reconciler_test.go:132:			defer wg.Done()

⚠️ Error Handling Patterns

Potential unchecked errors:

./internal/controller/yanet_controller_integration_test.go:425:			_, err := configReconciler.Reconcile(ctx, reconcile.Request{
./internal/controller/yanet_controller_integration_test.go:444:			_, err = configReconciler.Reconcile(ctx, reconcile.Request{
./internal/controller/node_deletion_test.go:131:			_, err := r.handleNodeDeletion(ctx, tt.nodeName)
./internal/helpers/http_getters_test.go:75:			_, err := HttpGet(server.URL)
./internal/helpers/http_getters_test.go:89:	_, err := HttpGet("http://invalid-host-that-does-not-exist-12345.local")
./internal/helpers/http_getters_test.go:106:	_, err := HttpGet(server.URL)
./api/v1alpha1/yanetconfig_webhook_test.go:206:			_, err := tt.new.ValidateUpdate(ctx, tt.old, tt.new)
./api/v1alpha1/yanetconfig_webhook_test.go:231:	_, err := config.ValidateDelete(ctx, config)
./api/v1alpha1/yanet_webhook_test.go:105:			_, err := tt.yanet.ValidateCreate(ctx, tt.yanet)
./api/v1alpha1/yanet_webhook_test.go:223:			_, err := tt.new.ValidateUpdate(ctx, tt.old, tt.new)
./api/v1alpha1/yanet_webhook_test.go:249:	_, err := yanet.ValidateDelete(ctx, yanet)
./api/v2alpha1/yanetconfig_webhook_test.go:87:			_, err := tt.yanetConfig.ValidateCreate(context.Background(), tt.yanetConfig)
./api/v2alpha1/yanet_webhook_test.go:117:			_, err := tt.yanet.ValidateCreate(context.Background(), tt.yanet)

Nil checks:

./cmd/main.go:127:	if err != nil {
./cmd/main.go:151:	if err = (&yanetv1alpha1.Yanet{}).SetupWebhookWithManager(mgr); err != nil {
./cmd/main.go:155:	if err = (&yanetv1alpha1.YanetConfig{}).SetupWebhookWithManager(mgr); err != nil {
./cmd/main.go:161:	if err := mgr.AddHealthzCheck("healthz", healthz.Ping); err != nil {
./cmd/main.go:165:	if err := mgr.AddReadyzCheck("readyz", healthz.Ping); err != nil {
./cmd/main.go:171:	if err := mgr.Start(ctrl.SetupSignalHandler()); err != nil {
./internal/controller/yanet_reconciler.go:80:			if err := r.Update(ctx, yanet); err != nil {
./internal/controller/yanet_reconciler.go:91:		if err := r.Update(ctx, yanet); err != nil {
./internal/controller/yanet_reconciler.go:101:	if err != nil {
./internal/controller/yanet_reconciler.go:117:		if setErr := ctrl.SetControllerReference(yanet, dep, r.Scheme); setErr != nil {
./internal/controller/yanet_reconciler.go:127:		if err != nil && errors.IsNotFound(err) {
./internal/controller/yanet_reconciler.go:138:			if err != nil {
./internal/controller/yanet_reconciler.go:161:		} else if err != nil {
./internal/controller/yanet_reconciler.go:190:			if err != nil {
./internal/controller/yanet_reconciler.go:225:	if err != nil {
./internal/controller/yanet_reconciler.go:254:		if err != nil {
./internal/controller/yanetconfig_controller.go:59:	if err != nil {
./internal/controller/yanet_controller_integration_test.go:154:			if err == nil {
./internal/controller/yanet_controller_integration_test.go:161:			if err == nil {
./internal/controller/yanet_controller_integration_test.go:168:			if err == nil {
./internal/controller/yanet_controller_integration_test.go:211:				if err != nil {
./internal/controller/yanet_controller_integration_test.go:278:				if err != nil {
./internal/controller/yanet_controller_integration_test.go:310:				if err != nil {
./internal/controller/yanet_controller_integration_test.go:366:				if err != nil {
./internal/controller/yanet_controller_integration_test.go:379:				if err != nil {
./internal/controller/yanet_controller.go:86:	if errV2 == nil {
./internal/controller/yanet_controller.go:95:		if reconcileErr != nil {
./internal/controller/yanet_controller.go:107:	if err != nil {
./internal/controller/yanet_controller.go:122:		if reconcileErr != nil {
./internal/controller/yanet_controller.go:134:	if err != nil {

---

Automated analysis using static analysis tools and pattern matching