<div dir="ltr"><div>Hi, Luigi!</div><div><br></div><div>Thanks for this short spec :)</div><div>Changes looking good for me, and I think Sahara-team can help with pushing this changes to sahara-tests repo.</div><div><br></div><div>But I have a question: why we need to use detached branch for it? And then we have the next problem "Temporarily exclude a specific branch from the CI". Maybe force push can be easier and faster?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 21, 2016 at 4:07 AM, Luigi Toscano <span dir="ltr"><<a href="mailto:ltoscano@redhat.com" target="_blank">ltoscano@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi,<br>
<br>
as discussed in the last (two) Sahara meetings, I'm working on moving the Tempest API tests from the Tempest repository to the new sahara-tests repository, which contains only (non-tempest) scenario tests and it's branchless as well.<br>
The move is a natural consequence of the Tempest focus on the "core six" (removing the burden of additional reviews from core Tempest), and of the existence of the Tempest Plugin interface.<br>
<br>
So far, nothing new. The difference from that past is that I'd like to keep the history of the imported code.<br>
<br>
I've the feeling that there is a bit of resistance to the idea of importing the history, but talking around it seems that it is feasible. It requires few additional steps, which I'm going to describe here for the posterity and to gather feedback, and which I'm going to execute in the next days if there are no blocking concerns.<br>
<br>
<br>
== Extract tempest/api/data_processing from Tempest and filter it<br>
Easy with git-split (<a href="https://github.com/ajdruff/git-splits" rel="noreferrer" target="_blank">https://github.com/ajdruff/git-splits</a>, thanks Evgeny Sikachev) and a bit of cleanup (removal of the first empty commit with `git rebase -i --root`). This code should then be merged in a detached branch of sahara-tests (created with `git checkout --orphan <branch>`).<br>
-> See a preview here: <a href="https://github.com/ltoscano-rh/sahara-tests/commits/tempest-sahara-api" rel="noreferrer" target="_blank">https://github.com/ltoscano-rh/sahara-tests/commits/tempest-sahara-api</a><br>
<br>
<br>
== Push the sanitized history to a detached branch of sahara-tests<br>
It's really two substeps:<br>
<br>
= Temporarily exclude a specific branch from the CI<br>
Change to openstack-infra/project-config and openstack/sahara-ci-config<br>
-> Requires reviews from infra and Sahara cores<br>
<br>
= Merge the content of the imported, detached branch to sahara-tests<br>
Ninja-approval of all the commits of the imported branch (38 commits). Without tests and use of tools like gertty it can be fast.<br>
-> It will require the help of a (patient) Sahara core, but I think I could volunteer some of them :)<br>
<br>
<br>
== Merge the imported branch into sahara-tests master<br>
Again, two substeps:<br>
<br>
= Change the ACLs for sahara-tests to allow a merge commit<br>
-> Requires review from infra (project-config, gerrit/acls)<br>
<br>
= Push the merge commit of the imported branch to sahara-tests master to gerrit<br>
Preview available here: <a href="https://github.com/ltoscano-rh/sahara-tests/commits/master" rel="noreferrer" target="_blank">https://github.com/ltoscano-rh/sahara-tests/commits/master</a><br>
-> Requires normal review from Sahara cores.<br>
<br>
<br>
== Adapt the imported code to use the Tempest Plugin interface<br>
Preview available here: <a href="https://github.com/ltoscano-rh/sahara-tests/commits/tempestapi-intree" rel="noreferrer" target="_blank">https://github.com/ltoscano-rh/sahara-tests/commits/tempestapi-intree</a><br>
-> Requires normal review from Sahara cores.<br>
<br>
<br>
== Undo all the temporarily hacks introduced before<br>
-> Requires normal review from Sahara cores.<br>
<br>
<br>
I hope that it could be useful.<br>
<span><font color="#888888"><br>
--<br>
Luigi<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">----- </span><br style="color:rgb(136,136,136);font-size:12.8px"><div style="color:rgb(136,136,136)"><div dir="ltr"><div style="font-size:12.8px"><span style="font-family:arial;font-size:small">Best Regards,</span><br></div><div style="font-size:12.8px"><span style="font-family:arial;font-size:small"><br></span></div><div><font face="arial" size="2">Evgeny Sikachev</font><br style="font-family:arial;font-size:small"><div style="font-size:small;font-family:arial">QA Engineer</div><div style="font-size:small;font-family:arial">Mirantis, Inc</div></div></div></div></div></div>
</div></div>