| opendevreview | Merged openstack/glance master: Fix failing namespace list delete race https://review.opendev.org/c/openstack/glance/+/841970 | 01:01 |
|---|---|---|
| opendevreview | Abhishek Kekane proposed openstack/glance stable/yoga: Fix failing namespace list delete race https://review.opendev.org/c/openstack/glance/+/841935 | 06:16 |
| opendevreview | Mridula Joshi proposed openstack/glance master: Extending stores-detail API https://review.opendev.org/c/openstack/glance/+/842045 | 06:55 |
| opendevreview | Rajat Dhasmana proposed openstack/glance-specs master: Add new location APIs https://review.opendev.org/c/openstack/glance-specs/+/840882 | 08:01 |
| *** akekane_ is now known as abhishekk | 09:36 | |
| opendevreview | Abhishek Kekane proposed openstack/glance master: [WIP] Immediate caching of an image https://review.opendev.org/c/openstack/glance/+/842070 | 09:46 |
| whoami-rajat | rosmaita, hey, when you come around can you take a look at this patch, I had missed a few things in my previous gate job patch which are corrected here https://review.opendev.org/c/openstack/glance/+/841837 | 10:07 |
| rosmaita | whoami-rajat: ack | 12:45 |
| rosmaita | whoami-rajat: left a comment for you on https://review.opendev.org/c/openstack/glance/+/841548/2#message-28e1390b6cddd29795a179ee3a9fb680c76747af | 12:57 |
| rosmaita | whoami-rajat: abhishekk: dansmith: lmk what you think of https://review.opendev.org/c/openstack/devstack/+/841804 and https://review.opendev.org/c/openstack/glance/+/841805 | 13:17 |
| mrjoshi | dansmith, hey, can you please take a look at this patch https://review.opendev.org/c/openstack/glance-specs/+/835606 | 13:24 |
| rosmaita | whoami-rajat: none of the 5 glance-multistore-cinder-import jobs on https://review.opendev.org/c/openstack/glance/+/841805 had to trigger the extra wait ... how often was that happening? | 13:28 |
| dansmith | I hadn't noticed it, so I'm guessing not very often, unless it's a new behavior | 13:53 |
| dansmith | rosmaita: replied on your patch | 13:53 |
| dansmith | rosmaita: happy to "put my money where my mouth is" on my suggestions, just say the word | 13:53 |
| opendevreview | Pranali Deore proposed openstack/glance-tempest-plugin master: Refactor rbac tests structure https://review.opendev.org/c/openstack/glance-tempest-plugin/+/839605 | 14:09 |
| opendevreview | Pranali Deore proposed openstack/glance-tempest-plugin master: Refactor rbac metadef tests structure https://review.opendev.org/c/openstack/glance-tempest-plugin/+/842117 | 14:09 |
| rosmaita | dansmith: ty, will look later today (have many meetings in the way) | 14:09 |
| whoami-rajat | rosmaita, I'm also not sure why that job started failing suddenly and it | 14:16 |
| whoami-rajat | it's mainly failing on stable branches | 14:16 |
| dansmith | ah interesting | 14:16 |
| whoami-rajat | passing on master as far as I've seen | 14:16 |
| opendevreview | Rajat Dhasmana proposed openstack/glance master: DNM: Testing glance cinder against v2 removal https://review.opendev.org/c/openstack/glance/+/794356 | 14:19 |
| opendevreview | Merged openstack/glance master: Override GLANCE_USE_IMPORT_WORKFLOW in cinder jobs https://review.opendev.org/c/openstack/glance/+/841837 | 14:19 |
| rosmaita | dansmith: do you think 1 or 2 seconds would be better for polling? | 14:47 |
| dansmith | either, but I think we poll on one second elsewhere.. image show should be pretty lightweight (I'd hope) | 14:50 |
| rosmaita | ok, sounds good | 14:51 |
| abhishekk | dansmith, will try to cover PoC as per your suggestions on the patch | 14:52 |
| dansmith | abhishekk: ack, what happens if caching is interrupted today? | 14:54 |
| abhishekk | never tried it but I think it exits immediately | 14:56 |
| dansmith | right I think it does.. what I mean is.. do we write half an image to disk and then never notice? | 14:56 |
| dansmith | such that we dish out a bad image to clients? | 14:56 |
| abhishekk | most probably the later one | 14:57 |
| dansmith | ack :/ | 14:57 |
| abhishekk | will test it and let you know what exactly happens | 14:57 |
| abhishekk | it does not write half image to disk,it triggers the graceful shutdown and completes the caching part | 15:14 |
| abhishekk | this is with uwsgi enabled | 15:14 |
| abhishekk | I don't have standalone setup atm to test | 15:15 |
| abhishekk | dansmith, https://github.com/openstack/glance/blob/master/glance/common/wsgi_app.py#L91 | 15:33 |
| dansmith | okay I thought we weren't spawning the caching thread from the pool, but... ack | 15:33 |
| dansmith | I wonder about standalone yeah | 15:33 |
| abhishekk | even at this moment with uwsgi if you try to restart the glance service then it waits for task_pool to complete (which has no tasks) and then it kills the master but workers remains orphan | 15:34 |
| dansmith | but yeah, if we want to keep it the same, do not set daemon, send a sentinel to the queue | 15:34 |
| abhishekk | ack | 15:43 |
| opendevreview | Rajat Dhasmana proposed openstack/glance stable/yoga: Disable import workflow in glance cinder jobs https://review.opendev.org/c/openstack/glance/+/841747 | 15:57 |
| opendevreview | Rajat Dhasmana proposed openstack/glance stable/yoga: Disable import workflow in glance cinder jobs https://review.opendev.org/c/openstack/glance/+/841747 | 16:04 |
| *** dansmith_ is now known as dansmith | 16:55 | |
| *** jbernard_ is now known as jbernard | 17:05 | |
| * abhishekk signing out for he day | 17:33 | |
| opendevreview | Christian Rohmann proposed openstack/glance stable/xena: Fix set_property_atomic() boolean type casting https://review.opendev.org/c/openstack/glance/+/842162 | 18:04 |
| opendevreview | Christian Rohmann proposed openstack/glance stable/wallaby: Fix set_property_atomic() boolean type casting https://review.opendev.org/c/openstack/glance/+/842163 | 18:05 |
| opendevreview | Christian Rohmann proposed openstack/glance stable/victoria: Fix set_property_atomic() boolean type casting https://review.opendev.org/c/openstack/glance/+/842164 | 18:05 |
| *** melwitt_ is now known as melwitt | 18:08 | |
| opendevreview | Cyril Roelandt proposed openstack/glance master: doc: add aarch64 to list of architecture image properties https://review.opendev.org/c/openstack/glance/+/791453 | 19:15 |
| opendevreview | Brian Rosmaita proposed openstack/glance master: DNM: testing devstack lib/tempest import wait https://review.opendev.org/c/openstack/glance/+/841805 | 21:49 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Migrate CentOS Stream 8 job to CentOS Stream 9 https://review.opendev.org/c/openstack/glance/+/841034 | 23:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!