<div dir="ltr"><div><div>I don't understand the details of this PR. Is there a dev out there who can look at this?<br><br></div>@suttner, Thank you for the PR. I'm not sure what testing needs to be done as part of the review. Would you be willing to write a test plan that a reviewer could use? We usually write them as pulp smash issues so they could become part of the nightly integration tests of Pulp. If so, you can write one here [0], and you can see some examples there too. I think having that will greatly speed up getting this accepted.<br><br>[0]: <a href="https://github.com/PulpQE/pulp-smash/issues">https://github.com/PulpQE/pulp-smash/issues</a><br><br></div><div>Thanks,<br></div>Brian<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 27, 2017 at 4:36 PM, Mihai Ibanescu <span dir="ltr"><<a href="mailto:mihai.ibanescu@gmail.com" target="_blank">mihai.ibanescu@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks for clarifying, I hadn't noticed the "handlers" portion of the path.</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 27, 2017 at 4:32 PM, Bernhard Suttner <span dir="ltr"><<a href="mailto:suttner@atix.de" target="_blank">suttner@atix.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:arial,helvetica,sans-serif;font-size:12pt;color:#000000">Hi Mihai,<br><br>the change is on the client side to use zypper instead of yum to install / uninstall / update packages and package groups. Zypper is the package manager on SUSE based linux distribution (pretty good as it is written in C++ and therefore really fast)<br><br>I guess, you are also planning to integrate dnf and use it instead of yum. Therefore, I think you will need to have a plugin structure to decide, which package manager to use on the various systems.<br><br>Best regards,<br>Bernhard<br><br><hr id="m_6578837952414356978m_8197940346335832272zwchr"><div><b>Von: </b>"Mihai Ibanescu" <<a href="mailto:mihai.ibanescu@gmail.com" target="_blank">mihai.ibanescu@gmail.com</a>><br><b>An: </b>"suttner" <<a href="mailto:suttner@atix.de" target="_blank">suttner@atix.de</a>><br><b>CC: </b>"Pulp-dev" <<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a>><br><b>Gesendet: </b>Montag, 27. März 2017 14:56:00<br><b>Betreff: </b>Re: [Pulp-dev] Use zypper on SUSE instead of yum (pull request)<br></div><div><div class="m_6578837952414356978h5"><br><div><div dir="ltr">Bernhard,<br><div>It may have been clear for others, but I need to clarify this for myself.</div><br><div>Is the intention of the patch to allow pulp to run on a SuSE system?</div><br><div>Or is it to make Pulp repositories that are properly understood by zypper?</div><br><div>The title of the PR seems to imply the latter, while the patch seems to implement the former.</div><br><div>Thank you!</div><div>Mihai</div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 27, 2017 at 3:12 AM, Bernhard Suttner <span dir="ltr"><<a href="mailto:suttner@atix.de" target="_blank">suttner@atix.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:arial,helvetica,sans-serif;font-size:12pt;color:#000000"><div>.... just spend some minutes and fixed the issues found by flake8. See pull request: <a href="https://github.com/pulp/pulp_rpm/pull/1041" target="_blank">https://github.com/pu<wbr>lp/pulp_rpm/pull/1041</a></div><br><br><hr id="m_6578837952414356978m_8197940346335832272m_7672071312691746658zwchr"><div style="color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><b>Von: </b>"Bernhard Suttner" <<a href="mailto:suttner@atix.de" target="_blank">suttner@atix.de</a>><br><b>An: </b><a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br><b>Gesendet: </b>Sonntag, 26. März 2017 16:29:13<br><b>Betreff: </b>[Pulp-dev] Use zypper on SUSE instead of yum (pull request)<div><div class="m_6578837952414356978m_8197940346335832272h5"><br><br>Hi, <br><br>I have send a pull request to pulp_rpm regarding using zypper instead of yum on SUSE based distributions. I know, that some (OK, a lot) of tests failed. <br><br>Would you merge this change if I would fix the issue? Just want to make sure, if there is anything else I should consider. Maybe there are other integrations planned like dnf and you want to re-structure the code in this part - e.g. with a configurable plugin structure. <br><br>Best regards, <br>Bernhard <br><br>ATIX Informationstechnologie und Consulting AG <br><br>______________________________<wbr>_________________<br>Pulp-dev mailing list<br><a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br><a href="https://www.redhat.com/mailman/listinfo/pulp-dev" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br></div></div></div><br></div></div><br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br></blockquote></div></div><br></div></div></div></div></div></blockquote></div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>