Skip to content
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

fixes #7815 - fixing package upload feature with new sub-man #15

Merged
merged 1 commit into from Oct 9, 2014

Conversation

jlsherrill
Copy link
Member

No description provided.

uep = connection.UEPConnection(cert_file=ConsumerIdentity.certpath(),
key_file=ConsumerIdentity.keypath())
mgr = certmgr.CertManager(uep=uep)
return mgr
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be cleaner to put the imports within get_manager(), but is that kosher?

@jlsherrill
Copy link
Member Author

Tested on

subscription-manager-1.13.5-1.el6.x86_64.rpm & subscription-manager-1.9.1-1.el6.x86_64.rpm

@mmoll
Copy link

mmoll commented Oct 8, 2014

tested, works for me. 👍

@ehelms
Copy link
Member

ehelms commented Oct 9, 2014

ACK

jlsherrill added a commit that referenced this pull request Oct 9, 2014
fixes #7815 - fixing package upload feature with new sub-man
@jlsherrill jlsherrill merged commit b7f1df8 into Katello:master Oct 9, 2014
@jlsherrill jlsherrill deleted the 7815 branch October 9, 2014 17:26
zjhuntin pushed a commit to zjhuntin/katello-agent that referenced this pull request Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants