-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: added "exec" authorization provider #193
base: master
Are you sure you want to change the base?
Conversation
5315def
to
c21e8eb
Compare
788f156
to
ddf51bd
Compare
Hi @cgbaker - what is the latest on this? I see changes have been pushed to the branch recently. |
I don't know. I have moved to a new company, so I will not be continuing
this work.
…On Wed, Jan 16, 2019, 05:51 doriordan ***@***.*** wrote:
Hi @cgbaker <https://github.com/cgbaker> - what is the latest on this? I
see changes have been pushed to the branch recently.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#193 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABmPT8utz5Yc5HVOTdcvW7hG7LCNPUw8ks5vDwQ6gaJpZM4V7p8M>
.
|
This is Anthony from Galactic Fog. We are pretty heavy skuber users, and we are going to continue working on this. For now we are re-upping this feature to finish polishing it off for the other public cloud providers, so you may see a few more pull requests. After that we plan to look at what we can do to finish getting the retries done at the level that David suggested. That's an issue we don't have as we handle it above the skuber level, but we would prefer not to use a forked build the way we do currently. We would still suggest either leaving the pull out here for people that need this, or accepting it as it shouldn't break anything. We will provide an update in 2 weeks. |
Thanks for the updates @cgbaker and @anthonyskipper. |
* Created a test for reproducing bug * Fixed Job status classes
…icFog/skuber into 173-external-auth-support
This repo seems to have been abandoned. Thanks, |
This is a WIP pull request for comments. Please see the current comments in #173 for context and for arguments why this may not be ready to merge.