| opendevreview | Merged openstack/glance master: Reduce concurrency for tempest jobs https://review.opendev.org/c/openstack/glance/+/893855 | 08:53 |
|---|---|---|
| opendevreview | Merged openstack/glance master: db: Don't invoke and_() without arguments https://review.opendev.org/c/openstack/glance/+/889053 | 13:52 |
| croelandt | dansmith: just saw your comment on https://review.opendev.org/c/openstack/glance/+/893855 , did you also talk to Cinder? I think the whole thing stems from https://wiki.openstack.org/wiki/CinderBobcatMidCycleSummary#Session_Two:_R-6:_23_August_2023 and Cinder currently has 2 patches under review for reducing concurrency | 13:52 |
| dansmith | croelandt: yep, we disagree on the due diligence required to justify it | 13:55 |
| croelandt | OK, if this is not the right approach, this might need to be stopped before it spreads to all projects :) | 14:05 |
| dansmith | been trying dude ;) | 14:07 |
| croelandt | yeah, sorry :/ | 14:30 |
| croelandt | You talk about revisiting the numbers using opensearch | 14:31 |
| croelandt | do you actually have a way (or better, a script) to get stats on CI failures for a given period of time? | 14:31 |
| dansmith | its opensearch.. it's not awesome, but it's the best thing I know of | 14:40 |
| abishop | hi, I want to draw attention to the comment I left on an old (merged) patch, https://review.opendev.org/c/openstack/glance_store/+/852044 regarding the deprecation of glance_store's rados_connect_timeout option | 14:58 |
| croelandt | abishop: interesting | 15:51 |
| croelandt | so does Cinder set this option to a specific values for its Tempest tests? | 15:51 |
| croelandt | Because as far as I understand, Glance never made use of it | 15:51 |
| abishop | it's not a tempest issue, it's just whether we support the ability to configure the timeout | 15:56 |
| croelandt | yes but you're saying this in regards to the failures we have | 15:56 |
| croelandt | so your idea would be to reintroduce that parameter and then use it? | 15:57 |
| abishop | croelandt: basically yes, using cinder's technique of supplying the timeout value in the connect object | 16:43 |
| croelandt | Sounds good, I'll discuss that with the rest of the team on Monday | 16:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!