| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-06-08 | |||
| 19:49:07 | sean-k-mooney | so that he zuul cloner can prepare it properly | |
| 19:49:43 | dansmith | ah | |
| 19:49:49 | dansmith | sounds like one of those things it's easy to forget until after a few hours of tearing out hair wondering why | |
| 19:49:55 | sean-k-mooney | you normally dont need to know that | |
| 19:50:04 | sean-k-mooney | yep exactly | |
| 19:50:50 | sean-k-mooney | for os-vif for example if you dont add it it will use the latest version allowed by upper-constraints.txt instead of master | |
| 19:51:05 | sean-k-mooney | which is normaly the correct behavior unless you are creating an os-vif job | |
| 19:52:09 | dansmith | we should just squash all the projects back into one tree obviously.. this would be so much easier ;) | |
| 19:52:42 | sean-k-mooney | hehe it works for the kernel but i like gerrit not exploding | |
| 19:52:57 | sean-k-mooney | although to be fair im not sure if would have an issue with that | |
| 19:53:12 | dansmith | well, I'm joking of course | |
| 19:53:14 | sean-k-mooney | i know that gitea would cry whoever | |
| 19:53:17 | dansmith | but sometimes ... it'd be easier | |
| 19:53:27 | sean-k-mooney | sometimes yes | |
| 19:53:54 | sean-k-mooney | i know that neutron-lib can be a pain for the neutron folks in that regard | |
| 19:54:04 | dansmith | aight, well, time to go do something else while that runs so I can see how close it is | |
| 19:54:06 | dansmith | thanks again | |
| 19:54:09 | sean-k-mooney | its partly why tempest lib is now dead | |
| 19:54:18 | sean-k-mooney | no worries o/ | |
| 19:54:48 | sean-k-mooney | dansmith: you have a syntax error | |
| 19:55:03 | dansmith | noo | |
| 19:55:05 | sean-k-mooney | oh i see | |
| 19:55:27 | sean-k-mooney | you commented out the definition of dsvm-irrelevant-files | |
| 19:55:40 | sean-k-mooney | but you are still using it on line 371 | |
| 19:56:43 | dansmith | I thought that was defined somewhere magic.. where did I nuke it? | |
| 19:57:01 | sean-k-mooney | actully that is not the issue its defined at the top of the file i think | |
| 19:57:15 | sean-k-mooney | ya line 16 | |
| 19:57:20 | dansmith | okay I thought something like that | |
| 19:57:36 | sean-k-mooney | the & is the defintion and * is a deference or alis | |
| 19:57:42 | dansmith | did something fail or you just eye-parsed it and thought it was broken? | |
| 19:58:21 | sean-k-mooney | oh you nuked a different one | |
| 19:58:33 | sean-k-mooney | policies-irrelevant-files | |
| 19:58:37 | sean-k-mooney | on line 390 | |
| 19:59:14 | sean-k-mooney | commented | |
| 19:59:21 | dansmith | oh that wasn't in yours I guess so I just block-removed | |
| 19:59:59 | sean-k-mooney | ya just comment out the jobs in gate and you shoudl be fine | |
| 20:00:35 | sean-k-mooney | it would be nice if there was a cleaner way to do this but it is what it is | |
| 20:00:54 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 20:01:05 | dansmith | thanks | |
| 20:01:38 | sean-k-mooney | no worries that has bit me in the past too | |
| 20:02:45 | sean-k-mooney | looking at https://zuul.openstack.org/status its queue so i think it passed syntax checking | |
| 20:03:01 | sean-k-mooney | zuul is really quick to comment abck if there is a sysntax error | |
| 20:03:26 | sean-k-mooney | although it was a bit delayed that time | |
| 21:19:39 | sean-k-mooney | dansmith: looks like your job jsut started running https://zuul.openstack.org/stream/98f29ccd122c4ff080aca6cd71be7d52?logfile=console.log so it will be a while but i proably will work at this point assuming the glance config options were correct | |
| 21:22:39 | dansmith | sweet | |
| #openstack-nova - 2020-06-09 | |||
| 00:34:18 | openstackgerrit | melanie witt proposed openstack/nova master: Raise InstanceMappingNotFound if StaleDataError is encountered https://review.opendev.org/734400 | |
| 01:59:25 | alex_xu | gibi: do we have priorities for this release? | |
| 06:10:34 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 06:13:23 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 08:49:23 | openstackgerrit | Hervé Beraud proposed openstack/nova master: Use unittest.mock instead of mock https://review.opendev.org/734447 | |
| 08:49:23 | openstackgerrit | Hervé Beraud proposed openstack/nova master: Use unittest.mock instead of mock https://review.opendev.org/734447 | |
| 09:36:47 | gibi | alex_xu: we did not talk about priorities. If you feel it is needed then start a thread on the ML about it and let see what list we can put together | |
| 09:37:10 | gibi | alex_xu: I myself would prioritize the whathever thing will end up in a runway slot | |
| 09:39:07 | brinzhang0 | Luckily we have runway slot to track^ | |
| 09:39:44 | gibi | brinzhang0: yeah, I polished the etherpad for victoria yesterday | |
| 10:51:29 | sean-k-mooney | gibi: i think it would be nice to prioities some of the feature that were close but did not quite make it and see if we can get them merged early (before m1) so that they dont drag on | |
| 10:51:40 | sean-k-mooney | gibi: im thinking of the provider.yaml file | |
| 10:52:39 | sean-k-mooney | im sure there are other features that were similarly close. | |
| 10:55:03 | lvdombrkr | hello guys, any updates on this? https://review.opendev.org/#/c/733982/ | |
| 10:56:13 | sean-k-mooney | lvdombrkr: it looks like the review feed back has not been adressed | |
| 10:56:50 | sean-k-mooney | so untill that is done its not going to make any more progress | |
| 10:57:37 | lvdombrkr | sean-k-mooney: thanks for answer. who can/should proceed this? | |
| 10:57:40 | sean-k-mooney | it was also only upladed 2 days ago so its a relitivly new patch an di suspect frickler has not had time to look at the comments form yesterday yet | |
| 10:58:17 | lvdombrkr | sean-k-mooney: ok, i see, thank you for detailed answer | |
| 10:58:20 | sean-k-mooney | lvdombrkr: well frickler is the authour but if your are interested in it reach out to them and ask if you can help | |
| 11:00:32 | gibi | sean-k-mooney: I encurage the authors of such patch to put them in the runway queue. If there are priority feature and there are patches in the runway then we could easily get confliting priorities. Still as I said to alex_xu let's start an ML thread about it if you feel that project priorities helps | |
| 11:02:28 | sean-k-mooney | gibi: yep i know that is definetly the way to go, im not sure if project prioritys are needed but they can sometimes be helpful. unified limits would be high on my personal list of project prioites but i have not really looked at the rest of what is proposed to intuit what else shoudl be on that list | |
| 11:04:17 | gibi | I did not have a list either | |
| 11:10:28 | frickler | lvdombrkr: I don't have much time currently to work on the inject fix, if you or someone else wants to take over, feel free to do so. seems this is getting ever more complicated from the simple py2/3 adaption I assumed initially :-/ | |
| 11:12:25 | sean-k-mooney | frickler: lvdombrkr i think the main feedback is keep the data as text strings and only convert to bytes if and when needed to do so. althoug i just read the top level coments and didnt actully review the patch | |
| 11:36:20 | lvdombrkr | frickler sean-k-mooney : ok i see guys, unfrortinetly i will be not able contribute by myself because im not too confident on this.. | |
| 14:41:09 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 15:40:46 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/stein: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732696 | |
| 15:50:47 | stephenfin | melwitt: Could I get your eyes on https://review.opendev.org/#/q/topic:bug/1550919+status:open today, please? | |
| 16:04:44 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708 | |
| 16:17:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732717 | |
| 16:17:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: libvirt: Fix misleading debug msg "Instance is running" https://review.opendev.org/732716 | |
| 16:17:00 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,Fix released] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood) | |
| 16:17:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Add functional test for bug 1550919 https://review.opendev.org/732715 | |
| 16:37:52 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 17:48:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Don't allow "reserving" file-backed memory https://review.opendev.org/734658 | |
| 17:49:00 | stephenfin | sean-k-mooney: ^ | |
| 17:54:27 | sean-k-mooney | stephenfin: oh you have a fix altreay i asumed you were just going to file the bug | |
| 17:55:35 | sean-k-mooney | stephenfin: the code does not do what the title says | |
| 17:56:00 | sean-k-mooney | we still allow you to reserve the memory | |
| 18:02:09 | sean-k-mooney | dansmith: https://review.opendev.org/#/c/734184/ seames to work | |
| 18:02:16 | sean-k-mooney | dansmith: did you enable any test that use it | |
| 18:07:24 | sean-k-mooney | there is a bunch of stuff like this https://zuul.opendev.org/t/openstack/build/fe828e4e683841f5a079bb9342c48493/log/controller/logs/screen-n-cpu.txt#34772 | |
| 18:07:37 | sean-k-mooney | asking the Image API where its store is | |
| 18:07:54 | sean-k-mooney | then creating snapshot(45941487007e4ecb876ca326b1245c50) on rbd image(c6f91c2a-085f-469e-a5f7-96652d96e9da_disk) | |
| 18:10:04 | sean-k-mooney | i gues this rbd image import is what your looking to see https://zuul.opendev.org/t/openstack/build/fe828e4e683841f5a079bb9342c48493/log/controller/logs/screen-n-cpu.txt#12725 | |
| 18:11:16 | sean-k-mooney | in all the other cases it seams to just create a snapshot of the already imported image | |
| 18:12:14 | sean-k-mooney | dansmith: so that shows the its working properly right ^ | |
| 18:13:04 | sean-k-mooney | we are now creatinging thin image snapshots on the RBD pool instead of copying the image over and over again | |
| 18:32:25 | melwitt | stephenfin: will do | |
| 18:33:32 | melwitt | elod: would you mind taking a look at these too please https://review.opendev.org/#/q/topic:bug/1550919+status:open | |
| 19:09:50 | dansmith | sean-k-mooney: I just got back. I think I'll need to further tweak it to make sure it uploads to one store and does the copy to the other, unless the default store thing magically made that happen | |
| 19:09:52 | dansmith | will check in a biot | |
| 19:35:57 | elod | melwitt: started to look at them but was a bit distracted, will do tomorrow (my) morning | |
| 19:36:25 | melwitt | thank you :) | |