Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

refs #8345 - explicitly install katello package #142

Merged
merged 1 commit into from
Dec 11, 2014

Conversation

stbenjam
Copy link
Contributor

@stbenjam
Copy link
Contributor Author

stbenjam commented Dec 4, 2014

[test]

@ehelms
Copy link
Member

ehelms commented Dec 4, 2014

Looks like this needs a rebase.

@stbenjam
Copy link
Contributor Author

stbenjam commented Dec 8, 2014

@ehelms Rebased

@@ -11,7 +11,7 @@
#
# $pulp_master:: whether the capsule should be identified as a pulp master server
#
# $pulp_admin_password:: password for the Pulp admin user. It should be left blank so that a random password is generated
# $pulp_admin_password:: passowrd for the Pulp admin user.It should be left blank so that random password is generated
Copy link
Member

Choose a reason for hiding this comment

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

This seems like a strange change to be here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed, good catch, and it was also my fault.

This PR should have gone to puppet-capsule:
#145

But the Librarian update here is valid.

@ehelms
Copy link
Member

ehelms commented Dec 9, 2014

APJ

@stbenjam
Copy link
Contributor Author

stbenjam commented Dec 9, 2014

[test]

1 similar comment
@stbenjam
Copy link
Contributor Author

[test]

stbenjam added a commit that referenced this pull request Dec 11, 2014
refs #8345 - explicitly install katello package
@stbenjam stbenjam merged commit 71e8fe3 into Katello:master Dec 11, 2014
@stbenjam stbenjam deleted the 8345 branch March 26, 2015 13:51
zjhuntin pushed a commit to zjhuntin/katello-installer that referenced this pull request Oct 30, 2018
Fixes #5135 BZ 1085541; removed duplicate output.
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants