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/08/25 13:36:33 UTC

[GitHub] [camel] utamas opened a new pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

utamas opened a new pull request #5987:
URL: https://github.com/apache/camel/pull/5987


   <!-- Uncomment and fill this section if your PR is not trivial
   - [ ] Make sure there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
   - [ ] Each commit in the pull request should have a meaningful subject line and body.
   - [ ] If you're unsure, you can format the pull request title like `[CAMEL-XXX] Fixes bug in camel-file component`, where you replace `CAMEL-XXX` with the appropriate JIRA issue.
   - [ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
   - [ ] Run `mvn clean install -Psourcecheck` in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
   Below are the contribution guidelines:
   https://github.com/apache/camel/blob/main/CONTRIBUTING.md
   -->
   


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-919913138


   Let's see what the others think about this. We released 3.11.2 in the last week, so there will be some time to wait for 3.11.3 too eventually. Thanks for reporting.


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] orpiske commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
orpiske commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-919913936


   > If it's impossible to be used we could make an exception or at least relaxing the service account key required parameter. @davsclaus @orpiske @lburgazzoli and all the Camel committers what do you think?
   
   From what I could gather of the problem, I think relaxing the parameter might be a reasonable compromise. 


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-906094117


   Ah yeah didnt spot it was not against main branch, I am moving it to main


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-906093380


   I don't think this should go in 3.11.x @davsclaus 


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] iliuta commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
iliuta commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-919912188


   Thanks @oscerd for the quick answer.
   I try to keep the libraries we're using up to date. I think we could wait a little longer for the 3.12, as end of month is in two weeks for now. 
   But, as I said, it looks to me like a quite annoying regression, as it used to work. If possible, relaxing account key parameter in order to have it working with Workload Identity would be really great.


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] iliuta commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
iliuta commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-919905636


   Hi, I ran into the same issue while upgrading from 3.8.0 to 3.11.2. (It works well with 3.8.0.)
   Our Kubernetes pods refused to work, because of the "PERMISSION_DENIED: The request is missing a valid API key" error.
   Digging for the issue, I've noticed the mandatory serviceAccountKey parameter which obviously we don't have because we're using Workload Identity (the recommended way to go).
   
   Any ideas when the 3.12.x version will be released?
   Or maybe there will be a backport for 3.11.x? For me it looks like a regression ...
   
   Thanks a lot for the work for Camel! Great library!


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-919908007


   Hello @iliuta 
   3.12.0 should be released by the end of this month/beginning of October.
   
   There won't be a backport to 3.11.x because it's a new feature and also because it's an LTS release. 
   
   If it's impossible to be used we could make an exception or at least relaxing the service account key required parameter. @davsclaus @orpiske @lburgazzoli and all the Camel committers what do you 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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #5987:
URL: https://github.com/apache/camel/pull/5987#issuecomment-906095527


   Done so this is on main only


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus merged pull request #5987: CAMEL-16883: Enable authentication in GKE via workload identity.

Posted by GitBox <gi...@apache.org>.
davsclaus merged pull request #5987:
URL: https://github.com/apache/camel/pull/5987


   


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

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org