You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/01/21 15:24:38 UTC

[GitHub] [camel-k] jon-ruckwood opened a new pull request #1933: Fix helm chart to be compatible with Go 1.14

jon-ruckwood opened a new pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933


   Additionally, add 'cluster' to fix type error.
   
   See https://github.com/helm/helm/issues/7711#issuecomment-593113347
   
   Fixes #1664
   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [camel-k] astefanutti commented on pull request #1933: Fix helm chart so it is compatible with Go 1.14

Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933#issuecomment-764798383


   Thanks!


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [camel-k] jon-ruckwood commented on pull request #1933: Fix helm chart to be compatible with Go 1.14

Posted by GitBox <gi...@apache.org>.
jon-ruckwood commented on pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933#issuecomment-764720610


   Lint output:
   
   ```sh
   $ helm lint  ./helm/camel-k                                                                             
   engine.go:159: [INFO] Missing required value: Field "platform.build.registry.address" is required when not running on OpenShift (set "platform.cluster=OpenShift" if you're using OpenShift instead)!
   ==> Linting ./helm/camel-k
   
   1 chart(s) linted, 0 chart(s) failed
   ```
   Install locally:
   
   ```sh
   $ helm install --generate-name --set platform.build.registry.address=http://localhost:5000/ ./helm/camel-k
   NAME: camel-k-1611242562
   LAST DEPLOYED: Thu Jan 21 15:22:44 2021
   NAMESPACE: sandbox
   STATUS: deployed
   REVISION: 1
   TEST SUITE: None
   NOTES:
   Connect the world with Camel K on Kubernetes!
   ```


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [camel-k] astefanutti commented on pull request #1933: Fix helm chart so it is compatible with Go 1.14

Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933#issuecomment-764798383


   Thanks!


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [camel-k] astefanutti merged pull request #1933: Fix helm chart so it is compatible with Go 1.14

Posted by GitBox <gi...@apache.org>.
astefanutti merged pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [camel-k] astefanutti merged pull request #1933: Fix helm chart so it is compatible with Go 1.14

Posted by GitBox <gi...@apache.org>.
astefanutti merged pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [camel-k] jon-ruckwood commented on pull request #1933: Fix helm chart to be compatible with Go 1.14

Posted by GitBox <gi...@apache.org>.
jon-ruckwood commented on pull request #1933:
URL: https://github.com/apache/camel-k/pull/1933#issuecomment-764720610


   Lint output:
   
   ```sh
   $ helm lint  ./helm/camel-k                                                                             
   engine.go:159: [INFO] Missing required value: Field "platform.build.registry.address" is required when not running on OpenShift (set "platform.cluster=OpenShift" if you're using OpenShift instead)!
   ==> Linting ./helm/camel-k
   
   1 chart(s) linted, 0 chart(s) failed
   ```
   Install locally:
   
   ```sh
   $ helm install --generate-name --set platform.build.registry.address=http://localhost:5000/ ./helm/camel-k
   NAME: camel-k-1611242562
   LAST DEPLOYED: Thu Jan 21 15:22:44 2021
   NAMESPACE: sandbox
   STATUS: deployed
   REVISION: 1
   TEST SUITE: None
   NOTES:
   Connect the world with Camel K on Kubernetes!
   ```


----------------------------------------------------------------
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:
users@infra.apache.org