johnpoth commented on a change in pull request #2328:
URL: https://github.com/apache/camel-k/pull/2328#discussion_r642320476



##########
File path: pkg/apis/camel/v1/common_types.go
##########
@@ -93,8 +93,9 @@ type MavenSpec struct {
        // A JKS formatted keystore is automatically created to store the CA 
certificate(s),
        // and configured to be used as a trusted certificate(s) by the Maven 
commands.
        // Note that the root CA certificates are also imported into the 
created keystore.
-       CASecret *corev1.SecretKeySelector `json:"caSecret,omitempty"`
-       Timeout  *metav1.Duration          `json:"timeout,omitempty"`
+       CASecret     *corev1.SecretKeySelector `json:"caSecret,omitempty"`
+       Timeout      *metav1.Duration          `json:"timeout,omitempty"`
+       Repositories []string                  `json:"repositories,omitempty"`

Review comment:
       Makes sense, looks like using the 
[Repostory](https://github.com/apache/camel-k/blob/main/pkg/util/maven/maven_types.go#L29)
 creates an import cycle due to Maven Log depending on camel v1, will look into 
it...




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to