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:
[email protected]
