Monday, 2020-07-13

*** rosmaita has left #openstack-glance03:05
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-glance04:33
*** bhagyashris|away is now known as bhagyashris04:41
abhishekkdansmith, you said  something about fix in oslo.policy, is that not helping?04:44
*** Luzi has joined #openstack-glance05:37
*** udesale has joined #openstack-glance05:38
*** rcernin has quit IRC06:02
*** rcernin has joined #openstack-glance06:11
*** m75abrams has joined #openstack-glance06:25
*** tosky has joined #openstack-glance06:39
*** amoralej|off is now known as amoralej07:04
*** rcernin has quit IRC08:06
*** baojg has quit IRC09:21
*** k_mouza has joined #openstack-glance09:21
*** baojg has joined #openstack-glance09:26
*** rcernin has joined #openstack-glance09:53
*** rcernin has quit IRC10:03
*** rcernin has joined #openstack-glance10:07
*** rcernin has quit IRC10:40
openstackgerritErno Kuvaja proposed openstack/glance master: Don't use Stevedore 3.0.0 which breaks gate  https://review.opendev.org/74066511:02
jokkeabhishekk: ^^11:02
*** baojg has quit IRC11:14
*** baojg has joined #openstack-glance11:15
*** bhagyashris is now known as bhagyashris|afk11:33
*** tkajinam has quit IRC11:37
openstackgerritErno Kuvaja proposed openstack/glance master: Don't use Stevedore 3.0.0 which breaks gate  https://review.opendev.org/74066511:52
*** amoralej is now known as amoralej|lunch12:03
*** udesale_ has joined #openstack-glance12:09
*** udesale has quit IRC12:12
*** bhagyashris|afk is now known as bhagyashris12:35
*** jv_ has joined #openstack-glance12:57
*** jv_ has quit IRC12:57
*** jv_ has joined #openstack-glance12:57
*** jv_ has quit IRC12:59
*** jv_ has joined #openstack-glance12:59
*** rosmaita has joined #openstack-glance13:03
*** amoralej|lunch is now known as amoralej13:04
*** zombieJulia is now known as TheJulia13:06
jokkesmcginnis: what happened to the no releasing at Fridays rule? Ref. the stevedore stuff13:26
*** baojg has quit IRC13:26
*** udesale_ has quit IRC13:27
*** irclogbot_3 has quit IRC13:27
*** udesale_ has joined #openstack-glance13:28
*** irclogbot_3 has joined #openstack-glance13:28
smcginnisjokke: It's mostly been relaxed since it has the two step verification process to go through of the releases jobs and then the requirements update jobs. But none of those test something like docs builds, so this was a bit of an odd one.13:29
jokkesmcginnis: well, that's kind of the reason why releasing friday is bad idea, cause everything will never get tested and then shit breaks over weekend ;)13:30
smcginnisYeah, this was a problem for sure. It was a small release with only two commits, but turns out they were significant enough to cause problems.13:32
abhishekksmcginnis, https://ethercalc.openstack.org/kq4gwn9gqzck13:33
smcginnisabhishekk: Nice!13:33
abhishekkthanks to you :D13:34
dansmithabhishekk: I don't think it's merged and released, but it's not a *fix* in oslo.policy, it's just removing a copy.copy() that they don't need, but the failure is still something on the glance side13:46
abhishekkdansmith, ack13:52
jokkesmcginnis: yeah I saw the work in flight for stevedore so I went with !=3.0.0 instead of <13:53
jokkeStill wondering if there is something we should od differently or if this was just something that was overlooked in stevedore side13:54
*** bhagyashris is now known as bhagyashris|afk13:56
smcginnisjokke: I think this was just an odd case that was overlooked. Not likely to happen again.13:57
jokkekk, I'll focus my time to other things then13:58
*** Luzi has quit IRC14:00
abhishekkdansmith, looks like this class is preventing it "ImmutableProperties"14:01
dansmithabhishekk: yeah I briefly looked last week, but didn't come up with any reason why14:11
dansmithabhishekk: maybe it copies the object and then tries to set all the properties on the new object, which fails?14:11
abhishekkdansmith, I suspect that might be the case14:12
dansmithprobably means those immutable things need __deepcopy__ handlers to make sure they're properly copy-able14:12
abhishekkat the moment I am not able to do anything as my keyboard has stopped working and I am using virtual keyboard for time being14:13
*** m75abrams has quit IRC15:06
*** udesale_ has quit IRC15:07
* abhishekk dinner break, back in 45 mins15:38
*** gyee has joined #openstack-glance15:41
*** amoralej is now known as amoralej|off16:22
*** k_mouza has quit IRC16:33
openstackgerritMerged openstack/glance master: Don't use Stevedore 3.0.0 which breaks gate  https://review.opendev.org/74066516:38
jokke \\o \o/ o// o/716:53
*** johnthetubaguy has quit IRC17:02
*** johnthetubaguy has joined #openstack-glance17:03
* abhishekk signing out for the day18:05
dansmithabhishekk: okay, but you're going to miss my celebration of a finally-passing nova-glance-ceph-multistore-copy-on-demand test run  in a few minutes :)18:06
*** m75abrams has joined #openstack-glance18:08
openstackgerritMerged openstack/glance master: Drop collections.abc compat handling  https://review.opendev.org/74032618:23
openstackgerritMerged openstack/glance master: Refactor common auth token code in images test  https://review.opendev.org/73870118:23
openstackgerritMerged openstack/glance master: Add a functional test for non-owned image copying  https://review.opendev.org/73870218:23
jokkedansmith: how is it looking?19:26
jokkedoes it work?19:26
jokkesmcginnis: rosmaita: if either of you have time to look through https://review.opendev.org/#/c/738672/ would be amazing as looks like the gate is finally behaving for a change19:29
rosmaitaok19:29
dansmithjokke: yep, one cinder test failed because they're creating a non-raw image to test with, which won't work on ceph in my config, but otherwise it all worked19:33
rosmaitajokke: approved (i left some comments for you though)19:36
jokkerosmaita: thanks!19:36
rosmaitanp19:36
rosmaitakeep on removing code!19:37
jokkedansmith: that sounds good19:37
jokkerosmaita: the next one in that chain is ~-3800 lines or something nice like that ;)19:37
jokkehttps://review.opendev.org/#/c/738673/4 <- this one :P19:43
jokkesmcginnis: good news, the patch rosmaita just +W'd stacked with the stevedore 3.1.0 requiremnt patch and the glance doc job passed so the issue got indeed fixed19:47
smcginnis\o/19:47
*** johnthetubaguy has quit IRC20:46
*** johnthetubaguy has joined #openstack-glance20:47
openstackgerritMerged openstack/glance master: Removal of 'enable_v2_api'  https://review.opendev.org/73867220:56
openstackgerritErno Kuvaja proposed openstack/python-glanceclient master: Pass Global Request ID on with session client  https://review.opendev.org/74082121:37
openstackgerritMerged openstack/glance_store master: Stop to use the __future__ module.  https://review.opendev.org/73284521:50
openstackgerritErno Kuvaja proposed openstack/python-glanceclient master: Pass Global Request ID on with session client  https://review.opendev.org/74082121:55
openstackgerritMerged openstack/python-glanceclient master: Stop to use the __future__ module.  https://review.opendev.org/73292022:08
*** k_mouza has joined #openstack-glance22:12
*** tosky has quit IRC22:42
*** tkajinam has joined #openstack-glance22:54
*** rcernin has joined #openstack-glance23:04
*** rcernin has quit IRC23:05
*** rcernin has joined #openstack-glance23:08
*** k_mouza has quit IRC23:17
*** hoonetorg has quit IRC23:24
*** k_mouza has joined #openstack-glance23:30
*** k_mouza has quit IRC23:34
*** hoonetorg has joined #openstack-glance23:39
*** k_mouza has joined #openstack-glance23:40
*** k_mouza has quit IRC23:45
*** k_mouza has joined #openstack-glance23:48
*** k_mouza has quit IRC23:53
*** k_mouza has joined #openstack-glance23:57

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!