<div dir="ltr">Hey Jirka!<div><br></div><div>I wasn't aware of the other options available.  Basically yes, you now just need to upload a tarball of the templates to swift.  You can see in the client:</div><div><br></div><div><div>-        tarball.tarball_extract_to_swift_container(</div><div>-            swift_client, <a href="http://tmp_tarball.name">tmp_tarball.name</a>, container_name)</div><div>+        _upload_file(swift_client, container_name,</div><div>+                     constants.TEMPLATES_TARBALL_NAME, <a href="http://tmp_tarball.name">tmp_tarball.name</a>)</div></div><div><br></div><div>Other than that it should be the same.  I'm not sure what files the UI wants to look at in swift, but certainly some are still there.  Basically any file that exists</div><div>in swift is not overwritten by the contents of the tar file.  So if a file exists in swift it takes priority.</div><div><br></div><div>I'll try to catch you on irc but I know our timezones are quite different.</div><div><br></div><div>Thanks for looking into it!</div><div><br></div><div>   Ian</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 8, 2018 at 10:46 AM Jiri Tomasek <<a href="mailto:jtomasek@redhat.com" target="_blank">jtomasek@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello, thanks for bringing this up.<div><br></div><div>I am going to try to test this patch with TripleO UI tomorrow. Without properly looking at the patch, questions I would like to get answers for are:</div><div><br></div><div>How is this going to affect ways to create/update deployment plan? Currently user is able to create deployment plan by:</div><div>- not providing any files - creating deployment plan from default files in /usr/share/openstack-tripleo-heat-templates</div><div>- providing a tarball</div><div>- providing a local directory of files to create plan from</div><div>- providing a git repository link</div><div><br></div><div>These changes will have an impact on certain TripleO UI operations where (in rare cases) we reach directly for a swift object</div><div><br></div><div>IIUC it seems we are deciding to consider deployment plan as a black box packed in a tarball, which I quite like, we'll need to provide a standard way how to provide custom files to the plan.</div><div><br></div><div>How is this going to affect CLI vs GUI workflow as currently CLI creates the plan as part of the deploy command, rather than GUI starts its workflow by selecting/creating deployment plan and whole configuration of the plan is performed on the deployment plan. Then the deployment plan gets deployed. We are aiming to introduce CLI commands to consolidate the behaviour of both clients to what GUI workflow is currently.</div><div><br></div><div>I am going to try to find answers to these questions and identify potential problems in next couple of days.</div><div><br></div><div>-- Jirka</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 7, 2018 at 5:34 PM Dan Prince <<a href="mailto:dprince@redhat.com" target="_blank">dprince@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for taking this on Ian! I'm fully on board with the effort. I<br>
like the consolidation and performance improvements. Storing t-h-t<br>
templates in Swift worked okay 3-4 years ago. Now that we have more<br>
templates, many of which need .j2 rendering the storage there has<br>
become quite a bottleneck.<br>
<br>
Additionally, since we'd be sending commands to Heat via local<br>
filesystem template storage we could consider using softlinks again<br>
within t-h-t which should help with refactoring and deprecation<br>
efforts.<br>
<br>
Dan<br>
On Wed, Aug 1, 2018 at 7:35 PM Ian Main <<a href="mailto:imain@redhat.com" target="_blank">imain@redhat.com</a>> wrote:<br>
><br>
><br>
> Hey folks!<br>
><br>
> So I've been working on some patches to speed up plan operations in TripleO.  This was originally driven by the UI needing to be able to perform a 'plan upload' in something less than several minutes. :)<br>
><br>
> <a href="https://review.openstack.org/#/c/581153/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/581153/</a><br>
> <a href="https://review.openstack.org/#/c/581141/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/581141/</a><br>
><br>
> I have a functioning set of patches, and it actually cuts over 2 minutes off the overcloud deployment time.<br>
><br>
> Without patch:<br>
> + openstack overcloud plan create --templates /home/stack/tripleo-heat-templates/ overcloud<br>
> Creating Swift container to store the plan<br>
> Creating plan from template files in: /home/stack/tripleo-heat-templates/<br>
> Plan created.<br>
> real    3m3.415s<br>
><br>
> With patch:<br>
> + openstack overcloud plan create --templates /home/stack/tripleo-heat-templates/ overcloud<br>
> Creating Swift container to store the plan<br>
> Creating plan from template files in: /home/stack/tripleo-heat-templates/<br>
> Plan created.<br>
> real    0m44.694s<br>
><br>
> This is on VMs.  On real hardware it now takes something like 15-20 seconds to do the plan upload which is much more manageable from the UI standpoint.<br>
><br>
> Some things about what this patch does:<br>
><br>
> - It makes use of process-templates.py (written for the undercloud) to process the jinjafied templates.  This reduces replication with the existing version in the code base and is very fast as it's all done on local disk.<br>
> - It stores the bulk of the templates as a tarball in swift.  Any individual files in swift take precedence over the contents of the tarball so it should be backwards compatible.  This is a great speed up as we're not accessing a lot of individual files in swift.<br>
><br>
> There's still some work to do; cleaning up and fixing the unit tests, testing upgrades etc.  I just wanted to get some feedback on the general idea and hopefully some reviews and/or help - especially with the unit test stuff.<br>
><br>
> Thanks everyone!<br>
><br>
>     Ian<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>
<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>
</blockquote></div>
__________________________________________________________________________<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>
</blockquote></div>