pr feedback

Author: aphralG

internal/resource/resource_service.go

@@ -194,12 +194,12 @@ func (r *ResourceService) ApplyConfig(ctx context.Context, instanceID string) (*
 		}
 	}
 
-	configContext, parseErr := r.ParseConfig(ctx, instance)
-	if parseErr != nil || configContext == nil {
+	nginxConfigContext, parseErr := r.nginxConfigParser.Parse(ctx, instance)
+	if parseErr != nil || nginxConfigContext == nil {
 		return nil, fmt.Errorf("failed to parse config %w", parseErr)
 	}
 
-	datasource.UpdateNginxInstanceRuntime(instance, configContext)
+	datasource.UpdateNginxInstanceRuntime(instance, nginxConfigContext)
 
 	slog.DebugContext(ctx, "Updated Instance Runtime after parsing config", "instance", instance.GetInstanceRuntime())
 
@@ -213,22 +213,14 @@ func (r *ResourceService) ApplyConfig(ctx context.Context, instanceID string) (*
 		return nil, fmt.Errorf("failed to reload NGINX %w", reloadErr)
 	}
 
-	return configContext, nil
+	return nginxConfigContext, nil
 }
 
 func (r *ResourceService) ParseConfig(ctx context.Context, instance *mpi.Instance) (*model.NginxConfigContext, error) {
 	slog.DebugContext(ctx, "Parsing NGINX instance config", "instance_id", instance.GetInstanceMeta().GetInstanceId())
 
 	nginxConfigContext, parseErr := r.nginxConfigParser.Parse(ctx, instance)
 	if parseErr != nil {
-		slog.WarnContext(
-			ctx,
-			"Unable to parse NGINX instance config",
-			"config_path", instance.GetInstanceRuntime().GetConfigPath(),
-			"instance_id", instance.GetInstanceMeta().GetInstanceId(),
-			"error", parseErr,
-		)
-
 		return nil, parseErr
 	}
 

internal/watcher/watcher_plugin.go

@@ -158,7 +158,7 @@ func (*Watcher) Subscriptions() []string {
 }
 
 func (w *Watcher) handleConfigApplyRequest(ctx context.Context, msg *bus.Message) {
-	slog.Info("Received ConfigApplyRequest event")
+	slog.Info("Watcher plugin received ConfigApplyRequest event")
 	managementPlaneRequest, ok := msg.Data.(*mpi.ManagementPlaneRequest)
 	if !ok {
 		slog.ErrorContext(ctx, "Unable to cast message payload to *mpi.ManagementPlaneRequest",