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 #20089 - Remove deleted puppet modules #6843
Conversation
Issues: #20089 |
app/models/katello/glue/pulp/repo.rb
Outdated
@@ -207,6 +207,14 @@ def yum_importer_values(capsule) | |||
config.merge(importer_ssl_options(capsule)) | |||
end | |||
|
|||
def puppet_importer_values(capsule) | |||
config = { | |||
:feed => self.importer_feed_url(capsule), |
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.
Use 2 spaces for indentation in a hash, relative to the start of the line where the left curly brace is.
a32150f
to
7447302
Compare
[test] |
@jlsherrill : Thanks! Test Cases Passed 👍 |
app/models/katello/glue/pulp/repo.rb
Outdated
@@ -157,8 +157,8 @@ def generate_importer(capsule = SmartProxy.default_capsule!) | |||
when Repository::FILE_TYPE | |||
Runcible::Models::IsoImporter.new(importer_ssl_options(capsule).merge(:feed => importer_feed_url(capsule))) | |||
when Repository::PUPPET_TYPE | |||
options = {:feed => importer_feed_url(capsule)} | |||
Runcible::Models::PuppetImporter.new(importer_ssl_options(capsule).merge(options)) | |||
#options = {:feed => importer_feed_url(capsule)} |
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.
I think you meant to delete this line?
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.
Removed the comment.
@@ -125,7 +125,7 @@ <h6 translate> | |||
</p> | |||
</div> | |||
|
|||
<div class="checkbox" ng-show="repository.content_type === 'yum'"> | |||
<div class="checkbox" ng-show="repository.content_type === 'yum'|| repository.content_type === 'puppet'"> |
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.
add a space between the word 'yum' and || please :)
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.
Modified the spacing.
minor issues and nitpicks, but otherwise seems to work fine! |
[test] |
ACK thanks @sjha4 !! |
Steps to reproduce:
Changes to fix:
Using remove_missing flag on the pulp end to clone behavior as is on the yum repos.