tadayosi commented on a change in pull request #2421:
URL: https://github.com/apache/camel-k/pull/2421#discussion_r655222156



##########
File path: pkg/trait/deployer.go
##########
@@ -96,6 +96,8 @@ func (t *deployerTrait) Apply(e *Environment) error {
                                        } else if 
isIncompatibleServerError(err) {
                                                t.L.Info("Fallback to 
client-side apply to patch resources")
                                                hasServerSideApply = false
+                                       } else {
+                                               t.L.Info("Failed with 
server-side apply due to: %v", err.Error())

Review comment:
       Yes, that makes sense. I just tried it earlier but found one test from 
container_test.go failed so I naively thought CSA might be necessary for local 
testing without looking into it deeply. Let me work on it with this pull req. 
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:
us...@infra.apache.org


Reply via email to