-
Notifications
You must be signed in to change notification settings - Fork 171
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
(HTCONDOR-2153) issue_credentials() #2465
(HTCONDOR-2153) issue_credentials() #2465
Conversation
JaimeF OK'd this going in with only one of the methods tested, based on the way the code currently works. |
As is traditional, no release note for the version 2 bindings because they don't officially exist yet. |
This ended up being more code than I expected, so while it probably deserves its own ticket, it's instead part of the version 2 bindings parent ticket. |
@Todd-L-Miller New test is failing |
The test depends on the Python module |
…id technical reasons.
…ng on all platforms.
…ng on all platforms, maybe for real this time.
The test failures on Ubuntu 20 are caused by weird crypto nonsense on that platform:
...
related to using the auto-generated host cert. For now, I'll just disable the test on those two platforms. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Hook
condor_submit
's credential handling into the Python bindings.HTCondor Pull Request Checklist for internal reviewers
After the above