Refs #8710 - created katello-debug sub-package

[lzap]

With review of katello-debug plugin script, did a cleanup.

@jsherrill @ehelms can you guys give me assistance in changing composes,
building and pushing the subpackage change out?

I also want to make a change to the installer to install katello-debug
package on Capsules. What is the best way to do that @iNecas?

Thanks

[stbenjam]

For now, you'll want to add it to puppet-capsule... but in the future we need to have a katello-capsule package that requires this, had a restart script, and anything else capsule-specific.

[stbenjam]

Due to the installer break out, {/etc,/var/log}/capsule-installer and {/etc,/var/log}/sam-installer as well

[lzap]

Oh good catch, amended.

[lzap]

Oh I figured out where the compose files are :-) Nice and handy.

http://koji.katello.org/koji/taskinfo?taskID=218610

[lzap]

I was thinking about puppet-capsule and started actually the file, but I figured out it is unnecessary. Foreman-debug helper functions ignores files/dirs which does not exist. Therefore you can run katello-debug both on Server and Capsule and both return reasonable tarballs. I can split the file into two, but I think that is little bit over-complicated.

[stbenjam]

Ah, I meant to use puppet-capsule to install the package on the capsule because there's no dedicated package yet, unless you want to add one here to the katello.spec.

You could also add it to capsule-installer, although I'd prefer it to be separate.

[jlsherrill]

if this is just for the capsule, any reason to not call it 'capsule-debug'?

[stbenjam]

I can think of a few reasons why the prefix is better, it comes up in a search for rpm -qa | grep katello, it will automatically work with katello-remove, etc.

[stbenjam]

And I believe it's for both, one debug package for capsule and katello.

[lzap]

Yeah, it works for both. You run the same script on Server and Capusle and it gives reasonable results for both. Splitting is an overkill IMHO.

[jlsherrill]

Ah i see, l LGTM

[jlsherrill]

[test]

[ehelms]

Do we still need these?

[jlsherrill]

they can be removed

[lzap]

Both 31-32?

[lzap]

Amended the three lines.

[lzap]

Hello?

[lzap]

Please when merging this also merge this one: https://github.com/Katello/puppet-capsule/pull/33/files

Downstream BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1134109

Thanks

[jlsherrill]

[test]

[jlsherrill]

ACK from me, @stbenjam ?

[stbenjam]

Yea ACK

[jlsherrill]

Thanks @lzap