<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 24, 2014 at 12:32 PM, Nikhil Manchanda <span dir="ltr"><<a href="mailto:nikhil@manchanda.me" target="_blank">nikhil@manchanda.me</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=""><br>
> On Wed, Jul 23, 2014 at 7:28 AM, Denis Makogon <<a href="mailto:dmakogon@mirantis.com">dmakogon@mirantis.com</a>> wrote:<br>
</div>> [...]<br>
<div class="">>><br>
>> Add Neutron-based configuration for DevStack to let folks try it<br>
<br>
</div>This makes sense to tackle; now that the neutron integration pieces have<br>
merged in Trove (yahoo!).<br>
<br>
However, it looks like the changes you propose in your DevStack patchset<br>
[1] have been copied directly from the trove-integration scripts at<br>
[2]. I have two primary concerns with this:<br>
<br>
a. Most of these values are only required for the trove functional<br>
tests to pass -- they aren't required for a user install of trove with<br>
Neutron. For such values, the trove-integration scripts seem like a<br>
better place for this configuration.<br>
<br>
b. Since the trove functional tests run based on the trove-integration<br>
scripts, what this means is that if this change is merged, this<br>
configuration code will be run twice, once in devstack, and once again<br>
as part of the test-init script from trove-integration.<br>
<br>
[1] <a href="https://review.openstack.org/#/c/108966" target="_blank">https://review.openstack.org/#/c/108966</a><br>
[2] <a href="https://github.com/openstack/trove-integration/blob/master/scripts/redstack#L406-427" target="_blank">https://github.com/openstack/trove-integration/blob/master/scripts/redstack#L406-427</a><br>
<div class=""><br>
<br>
<br>
>> Implementing/providing new type of testing job that will test on a regular<br>
>> basis all Trove tests with enabled Neutron to verify that all our networking<br>
>> preparations for instance are fine.<br>
>><br>
>> The last thing is the most interesting. And i’d like to discuss it with all<br>
>> of you, folks.<br>
>> So, i’ve wrote initial job template taking into account specific<br>
>> configuration required by DevStack and Trove-integration, see [4], and i’d<br>
>> like to receive all possible feedbacks as soon as possible.<br>
><br>
<br>
</div>So it looks like the test job you propose [3] is based on a current<br>
experimental job template: gate-trove-functional-dsvm-{datastore}<br>
[4]. Since pretty much most of it is an exact copy (except for the<br>
NEUTRON_ENABLED bit) I'd suggest working that in as a parameter to the<br>
current job template instead of duplicating the exact same code as part<br>
of another job.<br>
<br></blockquote><div>Nikhil, i already did lots of refactoring inside trove.yaml (see patchset [1] and its dependent patchset). The same thing i'm going to do <br>I know that there's lots of duplications, just wanted to describe complete template.<br>
<br></div><div>The actual question is - Is given template is correct? Would it work with trove-integration and with pure devstack in the nearest future?<br><br></div><div>P.S. I've got only basic knowledge about jenkins jobs inside infra.<br>
</div><div><br>[1] <a href="https://review.openstack.org/#/c/100601/">https://review.openstack.org/#/c/100601/</a><br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

[3] <a href="https://gist.github.com/denismakogon/76d9bd3181781097c39b" target="_blank">https://gist.github.com/denismakogon/76d9bd3181781097c39b</a><br>
[4] <a href="https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/trove.yaml#L30-63" target="_blank">https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/trove.yaml#L30-63</a><br>

<br>
<br>
Thanks,<br>
Nikhil<br>
<div class=""><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>