<div dir="ltr"><div>Many thanks, Jeremy.</div><div>Ignazio<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">Il giorno gio 10 gen 2019 alle ore 13:45 Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2019-01-10 09:28:45 +0100 (+0100), Ignazio Cassano wrote:<br>
> Hello Doug, sorry but I am not so expert of gerrit and how community<br>
> process for patching works.<br>
<br>
The Code and Documentation volume of the OpenStack Contributor Guide<br>
has chapters on the Git and Gerrit workflows our community uses:<br>
<br>
<a href="https://docs.openstack.org/contributors/code-and-documentation/" rel="noreferrer" target="_blank">https://docs.openstack.org/contributors/code-and-documentation/</a><br>
<br>
> I saw the <a href="https://review.openstack.org/#/c/577477/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/577477/</a> page but I cannot<br>
> understand if those patches are approved and backported on stable queens.<br>
> Please, help me to understand....<br>
<br>
Typically, we propose backports under a common Change-Id to the<br>
master branch change. Here you can see that backports to<br>
stable/rocky and stable/queens were proposed Monday by Bharat Kunwar:<br>
<br>
<a href="https://review.openstack.org/#/q/Ife5558f1db4e581b64cc4a8ffead151f7b405702" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/Ife5558f1db4e581b64cc4a8ffead151f7b405702</a><br>
<br>
The stable/queens backport is well on its way to approval; it's passing<br>
CI jobs (the Verified +1 from Zuul) and already has one of the<br>
customary two stable branch core reviews (the Code-Review +2 vote<br>
from Spyros Trigazis), so I expect it's well on its way to approval.<br>
<br>
> For example: I cloned the stable/queens magnum branch, the file<br>
> magnum/drivers/common/templates/kubernetes/fragments/configure-kubernetes-master.sh<br>
> is different from the same file I downloaded from cherry-picks, so<br>
> I presume the patch is not merged in the branch yet.<br>
<br>
The stable/queens backport looks like it still needs some work, as<br>
evidenced by the Verified -1 vote from Zuul. It's currently failing<br>
CI jobs openstack-tox-pep8 (coding style validation) and<br>
magnum-functional-k8s (a Kubernetes functional testsuite for<br>
Magnum). The names of those jobs in the Gerrit webUI lead to<br>
detailed build logs, which can be used to identify and iterate on<br>
solutions to get them passing for that change.<br>
<br>
> I presume the link you sent me (<br>
> <a href="https://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes" rel="noreferrer" target="_blank">https://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes</a><br>
> ) is for developers....that's right ?<br>
<br>
It's for anyone in the community who wants to help. "Developer" is<br>
just a reference to someone performing an activity, not a<br>
qualification.<br>
<br>
> Thanks ans sorry for my poor skill<br>
[...]<br>
<br>
Please don't apologize. Skills are just something we learn, nobody<br>
is born knowing any of this.<br>
-- <br>
Jeremy Stanley<br>
</blockquote></div>