| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-06-08 | |||
| 19:41:39 | sean-k-mooney | so it will report back sooner | |
| 19:41:51 | sean-k-mooney | normaly i just use check with the other disabeld | |
| 19:42:11 | sean-k-mooney | the goal is the same jsut run the one job i care about and not waste ci resouces | |
| 19:42:33 | dansmith | ack, I will leave it in check because it's easier | |
| 19:42:58 | dansmith | gah, conflict in .zuul | |
| 19:43:04 | sean-k-mooney | but ya once you do that all you need to do is reference it by its new name and you should be good to go | |
| 19:43:22 | sean-k-mooney | ya that is the downside to commenting | |
| 19:44:08 | sean-k-mooney | by the way i normally comment out all template bar - check-requirements | |
| 19:44:46 | sean-k-mooney | but again that is not needed i just normally do that while im getting the job working to disable pep8 and the tox jobs | |
| 19:44:53 | openstackgerrit | Dan Smith proposed openstack/nova master: Plumb image import functionality through our glance module https://review.opendev.org/731550 | |
| 19:44:54 | openstackgerrit | Dan Smith proposed openstack/nova master: Make libvirt able to trigger a backend image copy when needed https://review.opendev.org/656998 | |
| 19:44:55 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Add libvirt RBD configuration doc https://review.opendev.org/731577 | |
| 19:44:55 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 19:45:06 | dansmith | sean-k-mooney: ^ | |
| 19:45:34 | dansmith | sean-k-mooney: thanks for your help. it's so much easier to just hint at you doing it, but I have to do it myself *sometimes* :) | |
| 19:45:56 | sean-k-mooney | you proably dont need the required-projects: nova but it wont break anything | |
| 19:46:05 | sean-k-mooney | you will get that form devstack via inheritance | |
| 19:46:12 | dansmith | I copied that from another job | |
| 19:46:19 | sean-k-mooney | ya its fine | |
| 19:46:25 | dansmith | blindly without knowing why.. seems silly in the nova tree | |
| 19:46:29 | sean-k-mooney | its already required by tempest or devstack | |
| 19:46:33 | dansmith | ack | |
| 19:47:43 | sean-k-mooney | so what that does is it adds nova to LIBS_FROM_GIT | |
| 19:47:58 | sean-k-mooney | which for libs has the sideffect of make depens-on work | |
| 19:48:19 | sean-k-mooney | but its also used for zuul to know which poject to allow depens on to work with | |
| 19:48:24 | sean-k-mooney | in general that is | |
| 19:48:53 | sean-k-mooney | so if you want depens on to work say with ooo then somewhere in the job inheritance you would need a reqiured projects | |
| 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 | |