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 #16258 - Skip importer ssl verify for puppet module #6268

Merged
merged 1 commit into from Aug 26, 2016

Conversation

johnpmitsch
Copy link
Contributor

ContentViewPuppetModules were being checked for verify_ssl_on_sync?
but this is not a field for CVPMs

@parthaa
Copy link
Contributor

parthaa commented Aug 23, 2016

Can ya add a quick test for this ? -> APJ pending test

@zjhuntin
Copy link
Member

Tested and this works great!

@jlsherrill
Copy link
Member

hah, i actually fixed this here: #6234

but you added the test before i did :)

APJ

ContentViewPuppetModules were being checked for verify_ssl_on_sync?
but this is not a field for CVPMs
@johnpmitsch
Copy link
Contributor Author

[test]

@johnpmitsch
Copy link
Contributor Author

[test]

2 similar comments
@jlsherrill
Copy link
Member

[test]

@johnpmitsch
Copy link
Contributor Author

[test]

@johnpmitsch johnpmitsch merged commit a527f5d into Katello:master Aug 26, 2016
@johnpmitsch johnpmitsch deleted the cvpe_capsule_16258 branch August 26, 2016 20:02
kgaikwad pushed a commit to kgaikwad/katello that referenced this pull request Aug 30, 2016
ContentViewPuppetModules were being checked for verify_ssl_on_sync?
but this is not a field for CVPMs
jlsherrill pushed a commit to jlsherrill/katello that referenced this pull request Aug 30, 2016
ContentViewPuppetModules were being checked for verify_ssl_on_sync?
but this is not a field for CVPMs
jlsherrill pushed a commit to jlsherrill/katello that referenced this pull request Sep 27, 2016
ContentViewPuppetModules were being checked for verify_ssl_on_sync?
but this is not a field for CVPMs
Conflicts:
	app/models/katello/glue/pulp/repo.rb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants