[GitHub] lburgazzoli commented on a change in pull request #442: Add field 'camel-version' to Integration CRD

[GitBox <gi...@apache.org>]

lburgazzoli commented on a change in pull request #442: Add field 'camel-version' to Integration CRD
URL: https://github.com/apache/camel-k/pull/442#discussion_r257269073
 
 

 ##########
 File path: pkg/trait/gc.go
 ##########
 @@ -109,7 +113,12 @@ func getOldGenerationResources(e *Environment) ([]unstructured.Unstructured, err
 		return nil, err
 	}
 
-	selector, err := labels.Parse(fmt.Sprintf("camel.apache.org/integration=%s,camel.apache.org/generation,camel.apache.org/generation notin (%d)", e.Integration.Name, e.Integration.GetGeneration()))
+	selectors := []string{
+		fmt.Sprintf("camel.apache.org/integration=%s", e.Integration.Name),
+		fmt.Sprintf("camel.apache.org/generation,camel.apache.org/generation notin (%d)", e.Integration.GetGeneration()),
+	}
 
 Review comment:
   @astefanutti I had to split the line to fix lint findings, can you check it it is still ok ?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services