| *** yolanda_ has joined #zuul | 01:23 | |
| *** yolanda__ has quit IRC | 01:24 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack-infra/zuul-jobs master: Attempt to copy the coverage report even if job fails https://review.openstack.org/582690 | 02:55 |
|---|---|---|
| *** sshnaidm|off has quit IRC | 06:33 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Invalidate az cache on bad request https://review.openstack.org/582746 | 07:12 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Invalidate az cache on bad request https://review.openstack.org/582746 | 07:14 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Invalidate az cache on bad request https://review.openstack.org/582746 | 07:16 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Invalidate az cache on bad request https://review.openstack.org/582746 | 07:22 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Invalidate az cache on bad request https://review.openstack.org/582746 | 08:29 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Invalidate az cache on bad request https://review.openstack.org/582746 | 08:38 |
| tobiash | corvus: regarding re-enqueue performance during reconfiguration I see jobs that take around 700-800ms in _createJobGraph | 09:54 |
| tobiash | corvus: it looks like collecting job variants takes 3ms and I have a hunch that the rest is taken during applying the colleted variants | 09:58 |
| tobiash | corvus: the job in question has an inheritance path length of 8 which seems a bit excessive but the first two are from our base jobs and the last is from the final matching pipeline variant | 10:02 |
| tobiash | corvus: so I think that shows that we should to look closer what's that time consuming in applyVariant | 10:03 |
| tobiash | corvus: that project has around 10 of these jobs in their job graph and typically more than 10 current queue items | 10:04 |
| tobiash | corvus: so that sums up to a relatively long stall of the service during reconfiguration | 10:05 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Add debug message to job freezing https://review.openstack.org/582750 | 10:06 |
| tobiash | corvus: did further analysis on that now and I think it's the decryption of the secrets which takes so long. Those jobs have 9 secrets where some of them are bigger secrets with a total of 25 pkcs1-oaep blocks | 11:35 |
| tobiash | corvus: so maybe it makes sense to cache the decrypted secrets in the layout | 11:37 |
| tristanC | mordred: after vuejs, i gave react a try to use native patternfly library, here is mockup of the webui: https://github.com/TristanCacqueray/zuul-react | 12:07 |
| *** sshnaidm|off has joined #zuul | 12:26 | |
| *** sshnaidm|off is now known as sshnaidm | 12:50 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Build container images using pbrx https://review.openstack.org/582732 | 13:05 |
| tobiash | corvus: now I crafted a similar unit test with 8 inheritance layers and 9 copies of the fake secret | 13:12 |
| tobiash | corvus: caching of the decrypted secrets in the layout takes the applyVariant loop down from 190ms to 4ms on my laptop | 13:12 |
| tobiash | so that seems to be a huge win | 13:12 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Cache decryption results https://review.openstack.org/582794 | 14:42 |
| tobiash | corvus: this gives a massive performance boost when dealing with many secrets ^ | 14:44 |
| *** jimi|ansible has quit IRC | 15:14 | |
| *** sambetts_ has quit IRC | 16:51 | |
| *** sambetts_ has joined #zuul | 16:55 | |
| *** rcarrill1 has joined #zuul | 17:10 | |
| *** rcarrillocruz has quit IRC | 17:13 | |
| openstackgerrit | Merged openstack-infra/zuul master: zuul-web: config_errors endpoint: return 404 when tenant not found https://review.openstack.org/582281 | 17:23 |
| openstackgerrit | Merged openstack-infra/zuul master: zuul-web: jobs list endpoint: return 404 when tenant not found https://review.openstack.org/582200 | 17:26 |
| openstackgerrit | Merged openstack-infra/zuul master: client: show subcommand propose a more meaningful help message https://review.openstack.org/582321 | 17:28 |
| openstackgerrit | Merged openstack-infra/zuul master: zuul-web: jobs list endpoint: Add test and fix tenant not found 500 error https://review.openstack.org/581810 | 17:32 |
| openstackgerrit | Merged openstack-infra/zuul master: gerrit: rstrip baseurl trailing / https://review.openstack.org/581945 | 17:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role for installing docker and configuring registry mirror https://review.openstack.org/580730 | 18:06 |
| openstackgerrit | Merged openstack-infra/zuul master: sql: use a declarative base model https://review.openstack.org/575275 | 18:08 |
| mordred | tobiash: that's a nice simple patch. :) | 18:10 |
| tobiash | mordred: yes and that took more time than one might expect by looking at it ;) | 18:23 |
| mordred | tobiash: the best patches always do | 19:12 |
| *** goern has quit IRC | 19:37 | |
| tristanC | mordred: i have some time this week, may i work on the multi-tenant ui bugfix or are you about to push a change? | 22:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: configure-pypi-mirrors: add initial role https://review.openstack.org/581165 | 22:58 |
| tristanC | tobiash: what do you think about https://review.openstack.org/580547 , should i start fixing the tests? | 23:02 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: status: add pool column to nodepool list output https://review.openstack.org/579046 | 23:07 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: builder: do not configure provider that doesn't manage images https://review.openstack.org/578642 | 23:10 |
| *** elyezer has joined #zuul | 23:31 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!