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/03/01 12:11:25 UTC

[GitHub] [camel-k] astefanutti opened a new issue #2077: Upgrade to Service Binding operator v0.5.0

astefanutti opened a new issue #2077:
URL: https://github.com/apache/camel-k/issues/2077


   v0.5.0 has just been released. #2076 prepares the ground by upgrading to an intermediate version, but v0.5.0 introduces breaking changes in the ServiceBinding API that should be integrated. See:
   
   https://github.com/redhat-developer/service-binding-operator/releases/tag/v0.5.0
   
   


----------------------------------------------------------------
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 issue #2077: Upgrade to Service Binding operator v0.5.0

Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-787901875


   /cc @johnpoth 


----------------------------------------------------------------
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] arthurdm commented on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
arthurdm commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790738278


   Thanks for link @astefanutti / @johnpoth!
   
   Yes - are you noted, we're about to remove the `CustomServiceProjection` from the spec.  In order to leverage the new `ClusterApplicationResourceMapping` artifact my recommendation would be to expose the needed elements (volumeMount, env, etc) in the `.spec` of `Integration` - so that we don't get into a reconciler loop between the Service Binding Operator changing the generated `Deployment` that is owned by the Camel K Operator.


----------------------------------------------------------------
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] johnpoth commented on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
johnpoth commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-787914944


   That was fast! I can take a look


----------------------------------------------------------------
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 issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790747377


   I've created #2096, as an umbrella issue to make Integration _PodSpecable_.


----------------------------------------------------------------
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 closed issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
astefanutti closed issue #2077:
URL: https://github.com/apache/camel-k/issues/2077


   


----------------------------------------------------------------
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] arthurdm commented on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
arthurdm commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-789048500


   I am interested in learning more about the connection and usage between Camel K and Service Binding, as I work on the [Service Binding Spec](https://k8s-service-bindings.github.io/spec/).   Any details that could be shared?


----------------------------------------------------------------
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] johnpoth commented on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
johnpoth commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790802730


   In the case of ServiceBinding, the integration [isn't needed](https://github.com/k8s-service-bindings/spec/commit/20daf4e312ca502a301cb552624680152d4fe319) to be PodSpecable


----------------------------------------------------------------
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 issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790414974


   @arthurdm you can find some details in https://github.com/apache/camel-k/blob/master/proposals/service-binding/service-binding.adoc.


----------------------------------------------------------------
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 issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-791453914


   Fixes with #2094.


----------------------------------------------------------------
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] johnpoth commented on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
johnpoth commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790643699


   @arthurdm I think one relevant part of the specification that is relevant here is the use of [custom service projection](https://github.com/k8s-service-bindings/spec#custom-projection) where camel-k handles the mounting of the binding information (in the form of a `secret`) into the `Deployment`.  I am mentioning this as it's scheduled to be [removed](https://github.com/k8s-service-bindings/spec/pull/143) if my understanding is correct.
   
   One of the benefit of using this approach was that the `Deployment` did not have to be created in an incomplete state: waiting for the binding information and the resulting `Pods` failing. 
   


----------------------------------------------------------------
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 issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790806341


   > In the case of ServiceBinding, the integration [isn't needed](https://github.com/k8s-service-bindings/spec/commit/20daf4e312ca502a301cb552624680152d4fe319) to be PodSpecable
   
   It seems that was the case an hour ago 😉.


----------------------------------------------------------------
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] johnpoth commented on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
johnpoth commented on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790502394


   There is also a [meeting](https://github.com/redhat-developer/service-binding-operator#community-discussion-contribution-and-support)  where use cases can be discussed. Next one is today at 1:00 PM UTC I think


----------------------------------------------------------------
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] arthurdm edited a comment on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
arthurdm edited a comment on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790738278


   Thanks for link @astefanutti / @johnpoth!
   
   Yes - are you noted, we're about to remove the `CustomServiceProjection` from the spec.  In order to leverage the new `ClusterApplicationResourceMapping` artifact my recommendation would be to expose the needed elements (volumeMount, env, etc) in the `.spec` of `Integration` - so that we don't get into a reconciler loop when the Service Binding Operator changes the generated `Deployment` that is owned by the Camel K Operator.


----------------------------------------------------------------
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] johnpoth edited a comment on issue #2077: Upgrade to Service Binding operator to v0.5.0

Posted by GitBox <gi...@apache.org>.
johnpoth edited a comment on issue #2077:
URL: https://github.com/apache/camel-k/issues/2077#issuecomment-790643699


   @arthurdm I think one part of the specification that is relevant here is the use of [custom service projection](https://github.com/k8s-service-bindings/spec#custom-projection) where camel-k handles the mounting of the binding information (in the form of a `secret`) into the `Deployment`.  I am mentioning this as it's scheduled to be [removed](https://github.com/k8s-service-bindings/spec/pull/143) if my understanding is correct.
   
   One of the benefit of using this approach was that the `Deployment` did not have to be created in an incomplete state: waiting for the binding information and the resulting `Pods` failing. 
   
   Leveraging [application-resource-mapping](https://github.com/k8s-service-bindings/spec#application-resource-mapping) might let us achieve the same behavior.
   


----------------------------------------------------------------
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