Refs #8710 - install katello-debug by default #33
Conversation
|
Hello, anyone? @iNecas @jlsherrill Thanks! |
|
Context: I've created foreman-debug and katello-debug subpackages. We want to install them on the Server as well as on the Capsules. Katello-debug has dependency of foreman-debug (it is technically an "extension" to it). |
|
Yea this is fine, but the package has to exist before we merge it, it's not in the nightly repos yet. I imagine because the Pipeline is red? http://ci.theforeman.org/view/Katello%20Pipeline/ |
| @@ -5,6 +5,10 @@ | |||
| package{ ['rubygem-smart_proxy_pulp']: | |||
| ensure => installed, | |||
| } | |||
|
|
|||
| package{ ['katello-debug']: | |||
| ensure => installed, | |||
There was a problem hiding this comment.
Actually why is this in the if block for pulp? We should always install katello-debug, or should we?
|
What @stbenjam sais |
lzap
force-pushed
the
katello-debug-install
branch
from
5336dfb
to
3252a92
Compare
|
Right fixed. |
|
Looks like the packages are there now, so ACK |
|
@lzap You can merge your own PR's, it's how it's done in the Katello GitHub org, but I'll merge this one |
Refs #8710 - install katello-debug by default
Belongs to Katello/katello#4970