openstackgerrit | Ghanshyam Mann proposed openstack/glance master: DNM: testing bionic node https://review.openstack.org/642313 | 02:37 |
---|---|---|
*** MattMan_ has quit IRC | 03:23 | |
*** MattMan_ has joined #openstack-glance | 03:24 | |
*** udesale has joined #openstack-glance | 04:03 | |
*** Sravan has joined #openstack-glance | 04:31 | |
*** Sravan has quit IRC | 05:36 | |
*** Sravan has joined #openstack-glance | 06:07 | |
*** Sravan has quit IRC | 06:11 | |
*** abhishekk has joined #openstack-glance | 06:24 | |
*** Sravan has joined #openstack-glance | 06:59 | |
*** pcaruana has joined #openstack-glance | 07:00 | |
*** rcernin has quit IRC | 07:03 | |
*** Sravan has quit IRC | 07:04 | |
*** mosulica has joined #openstack-glance | 07:32 | |
*** belmoreira has joined #openstack-glance | 07:40 | |
*** belmoreira has quit IRC | 07:42 | |
*** belmoreira has joined #openstack-glance | 07:42 | |
*** Sravan has joined #openstack-glance | 07:53 | |
*** Sravan has quit IRC | 07:57 | |
*** tkajinam has quit IRC | 08:17 | |
*** awalende has joined #openstack-glance | 09:23 | |
openstackgerrit | Lucian Petrut proposed openstack/glance master: Allow glance tests to run on Windows https://review.openstack.org/634007 | 09:40 |
*** mosulica has quit IRC | 09:44 | |
*** mosulica has joined #openstack-glance | 09:48 | |
*** priteau has joined #openstack-glance | 10:03 | |
*** Sravan has joined #openstack-glance | 10:04 | |
*** Sravan has quit IRC | 10:09 | |
*** abhishekk has quit IRC | 10:33 | |
*** udesale has quit IRC | 10:34 | |
*** edmondsw has joined #openstack-glance | 11:47 | |
*** rosmaita has joined #openstack-glance | 11:57 | |
*** markvoelker has quit IRC | 12:14 | |
rosmaita | jokke_: abhishek: smcginnis: around? | 12:33 |
jokke_ | yup | 12:48 |
rosmaita | need your opinion on https://review.openstack.org/#/c/620234/ -- see Tim Burke's comment | 12:50 |
rosmaita | jokke_: ^^ | 12:51 |
*** jdillaman has joined #openstack-glance | 12:59 | |
jokke_ | I do understand his point and if those util changes in flight fixes the problem it makes it redundant on this level. And I would love if we had time and resources to dig into the rabbit hole and find the very rootcause for this. yet we have neither and this clearly is critical issue that needs to be solved for the Stein release | 13:00 |
rosmaita | well, that other patch is for the other direction (from swift to glance) | 13:02 |
rosmaita | the patch he mentions on the review | 13:02 |
jokke_ | both of them? | 13:03 |
rosmaita | i don't think the other utility patches come into play, since they are for the Cooperative reader | 13:03 |
jokke_ | there is the one as well oh glance I just +2'd | 13:03 |
jokke_ | oh it was that | 13:03 |
*** Sravan has joined #openstack-glance | 13:05 | |
jokke_ | If we need to apply bandaid to have more stable service, at this point of the cycle so be it | 13:06 |
rosmaita | that's my feeling, too | 13:07 |
jokke_ | Perhaps we should mention the concerns of our FD behavior in reno as reminder for ourselves to keep eye on it | 13:07 |
jokke_ | ? | 13:07 |
rosmaita | so my question is do we stick with the looking for a 0 read, or take tim's suggestion and just check for an empty string response? | 13:08 |
*** Sravan has quit IRC | 13:09 | |
jokke_ | I dislike both 0 sized I/O requests and modifying results for bandaid ;) | 13:10 |
rosmaita | well, i don't think we have an option about modifying results | 13:11 |
jokke_ | computationally cathing the zero read and faking it is much lighter operation than comparing every single result and checking if we happen to have empty string there | 13:12 |
rosmaita | i wasn't thinking of that, but you are right ... my concern was just to keep the band-aid small so that it doesn't cover too much | 13:13 |
smcginnis | I do kind of like Tim's suggestion. That looks like it _should_ be the right way to handle things. | 13:15 |
smcginnis | But clearly something still missing from this picture somewhere. | 13:15 |
rosmaita | i'm wondering if it's in the swiftclient somewhere ... we hand our reader class off to the client to do the put ... but i am only saying that because i don't see where it's happening on our code. we've got stuff wrapped in other stuff wrapped in other stuff, so i could be missing something | 13:17 |
jokke_ | So my point is that Tim's proposal catches a lot and replaces the result with empty byte | 13:18 |
jokke_ | like correctly behaving fd that actually would give empty byte insteado f empty string | 13:19 |
openstackgerrit | Brian Rosmaita proposed openstack/glance_store master: Prevent unicode object error from zero-byte read https://review.openstack.org/620234 | 13:55 |
*** priteau has quit IRC | 14:44 | |
*** awalende has quit IRC | 14:49 | |
*** FlorianFa has joined #openstack-glance | 14:50 | |
*** FlorianFa has quit IRC | 14:52 | |
*** FlorianFa has joined #openstack-glance | 14:52 | |
*** udesale has joined #openstack-glance | 14:56 | |
*** priteau has joined #openstack-glance | 14:59 | |
openstackgerrit | Merged openstack/glance master: Clarify the Glance/metadefs relationship https://review.openstack.org/608769 | 15:00 |
*** asettle has joined #openstack-glance | 15:28 | |
*** KeithMnemonic has joined #openstack-glance | 15:38 | |
KeithMnemonic | is there someone here who can answer a question about https://docs.openstack.org/image-guide/share-images.html#sharing-an-image it seems this does not work when the image was created as a result of doing an instance snapshot from a volume backed instance. in that case you also need to transfer the cinder volume. https://docs.openstack.org/cinder/latest/cli/cli-manage-volumes.html just wondering if this | 15:40 |
KeithMnemonic | was supposed to be documented somewhere or is there some spec that would also do the volume transfer when you do the image transfer | 15:40 |
asettle | jokke_ ^^ | 15:41 |
*** mosulica has quit IRC | 15:49 | |
*** mosulica has joined #openstack-glance | 15:56 | |
openstackgerrit | Brian Rosmaita proposed openstack/glance_store master: DNM: swift backend functional test https://review.openstack.org/642505 | 16:04 |
*** pcaruana has quit IRC | 16:23 | |
*** pcaruana has joined #openstack-glance | 16:23 | |
jokke_ | KeithMnemonic & asettle: Afaik we do not transfer the ownership of any resources upon share | 16:31 |
KeithMnemonic | jokke: thanks so maybe a doc bug to add a note about images that came from volume backed instance snapshots? | 16:34 |
openstackgerrit | Merged openstack/glance master: Add an oslo.policy.enforcer entrypoint https://review.openstack.org/637985 | 16:42 |
*** nsmeds has left #openstack-glance | 16:45 | |
*** gyee has joined #openstack-glance | 16:46 | |
*** udesale has quit IRC | 16:52 | |
*** priteau has quit IRC | 16:54 | |
*** jdillaman has quit IRC | 17:01 | |
jokke_ | I have bit hard time trying to understand how it's supposed to work. It will fail one way or other if it's volume backed. Either you can't share it or you can't use it yourself anymore | 17:05 |
jokke_ | but maybe I'm just thick and missing the point why sharing it was good idea at the first place | 17:10 |
*** priteau has joined #openstack-glance | 17:12 | |
*** Sravan has joined #openstack-glance | 17:19 | |
*** mosulica has quit IRC | 17:21 | |
*** jdillaman has joined #openstack-glance | 17:56 | |
*** mosulica has joined #openstack-glance | 18:02 | |
openstackgerrit | Brian Rosmaita proposed openstack/glance_store master: DNM: swift backend functional test https://review.openstack.org/642505 | 18:23 |
openstackgerrit | Brian Rosmaita proposed openstack/glance_store master: DNM: swift backend functional test https://review.openstack.org/642505 | 18:29 |
*** mosulica has quit IRC | 18:40 | |
*** priteau has quit IRC | 18:55 | |
openstackgerrit | Brian Rosmaita proposed openstack/glance_store master: DNM: swift backend functional test https://review.openstack.org/642505 | 19:19 |
*** Sravan has quit IRC | 19:50 | |
KeithMnemonic | jokke_ a customer thought it worked regardless of how the image was created | 20:20 |
*** Sravan has joined #openstack-glance | 20:20 | |
*** Sravan has quit IRC | 20:25 | |
*** itlinux has joined #openstack-glance | 20:30 | |
*** itlinux has quit IRC | 20:34 | |
*** itlinux has joined #openstack-glance | 20:56 | |
*** Sravan has joined #openstack-glance | 21:28 | |
*** pcaruana has quit IRC | 21:45 | |
*** Sravan has quit IRC | 21:48 | |
*** Sravan has joined #openstack-glance | 21:48 | |
*** itlinux has quit IRC | 21:55 | |
*** Sravan has quit IRC | 21:57 | |
*** Sravan has joined #openstack-glance | 22:02 | |
*** Sravan has quit IRC | 22:05 | |
*** NostawRm has joined #openstack-glance | 22:06 | |
*** Sravan has joined #openstack-glance | 22:08 | |
*** Sravan has quit IRC | 22:18 | |
*** Sravan has joined #openstack-glance | 22:25 | |
*** Sravan has quit IRC | 22:26 | |
*** Sravan has joined #openstack-glance | 22:39 | |
*** Sravan has quit IRC | 22:43 | |
*** threestrands has joined #openstack-glance | 22:50 | |
*** Sravan has joined #openstack-glance | 22:54 | |
*** tkajinam has joined #openstack-glance | 22:56 | |
*** Sravan has quit IRC | 22:57 | |
*** Sravan has joined #openstack-glance | 22:59 | |
*** Sravan has quit IRC | 23:05 | |
*** Sravan has joined #openstack-glance | 23:08 | |
*** Sravan has quit IRC | 23:44 | |
*** threestrands has quit IRC | 23:45 | |
*** Sravan has joined #openstack-glance | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!