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 #7553: Update ConsumerIdentity location. #12

Merged
merged 1 commit into from Sep 24, 2014
Merged

Fixes #7553: Update ConsumerIdentity location. #12

merged 1 commit into from Sep 24, 2014

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Sep 24, 2014

Subscription manager library moved the location of ConsumerIdentity
that katello-agent relies on. Instead of being at:

subscription_manager.certlib

The ConsumerIdentity class looks like it is located at:

subscription_manager.identity

This fix should work for all versions of subscription-manager, not just
the latest.

Subscription manager library moved the location of ConsumerIdentity
that katello-agent relies on. Instead of being at:

subscription_manager.certlib

The ConsumerIdentity class looks like it is located at:

subscription_manager.identity

This fix should work for all versions of subscription-manager, not just
the latest.
@jlsherrill
Copy link
Member

This would break compatibility with older versions. We'll have to do something like:

try:
  import new_module
except ImportError:
  import old_module

@jlsherrill
Copy link
Member

Actually nevermind, it shouldn't break compatibility

@jlsherrill
Copy link
Member

ACK, tested, worked well

ehelms added a commit that referenced this pull request Sep 24, 2014
Fixes #7553: Update ConsumerIdentity location.
@ehelms ehelms merged commit ec02808 into Katello:master Sep 24, 2014
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
2 participants