| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-06-08 | |||
| 19:39:01 | dansmith | sean-k-mooney: so what about the bit of parenting this from the existing job and replacing the existing thing under check to reference this... that look okay too? | |
| 19:39:50 | sean-k-mooney | yes although when im testing i normally disable the other check jobs untill i get it working | |
| 19:40:05 | dansmith | by just deleting them? | |
| 19:40:35 | sean-k-mooney | https://review.opendev.org/#/c/681771/2/.zuul.yaml | |
| 19:40:48 | sean-k-mooney | i normally coment them out but ya deleting works too | |
| 19:40:49 | dansmith | ack | |
| 19:40:52 | dansmith | smart :) | |
| 19:41:19 | sean-k-mooney | actully thats slight a bad example since i enabled noop and use the experimental queue to test | |
| 19:41:26 | sean-k-mooney | the expermiental queue is faster | |
| 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 | |