| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-13 | |||
| 08:02:48 | bauzas | did we cut the cord ? | |
| 08:02:58 | bauzas | I see a lot of merge conflicts as expected | |
| 08:03:15 | kashyap | aspiers: For reference, see this: https://opendev.org/openstack/nova/commit/e155baefb0 | |
| 08:03:28 | gibi | bauzas: the bw series is approved. I see vpmem in also approved | |
| 08:03:34 | bauzas | gibi: actually, nevermind, I'll read the nova meeting logs | |
| 08:03:50 | bauzas | gibi: the cpu-resources too, but it got conflicts | |
| 08:04:07 | gibi | bauzas: yepp | |
| 08:07:18 | bauzas | okay, so tl;dr: let's continue trying to merge stuff that was previously approved | |
| 08:07:43 | bauzas | gibi: how is your series ? still in the weeds ? | |
| 08:09:38 | gibi | bauzas: one patch merged during the night, rest is in check queue after recheck | |
| 08:11:01 | bauzas | ack | |
| 08:13:47 | kashyap | aspiers: Sent the doc fix to Glance: https://review.opendev.org/681990 useful-image-properties: Update note on machine types | |
| 08:33:40 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: Update flavor-related metadata on resize https://review.opendev.org/681004 | |
| 08:38:55 | luyao | stephenfin: are you around, I see you string the vpmems and cpu resources, they merge conflict now | |
| 08:39:58 | luyao | stephenfin: I'll rebase to master branch | |
| 08:41:52 | luyao | stephenfin: this is the last patch , right ? https://review.opendev.org/#/c/674895/41 | |
| 08:55:46 | openstackgerrit | Luyao Zhong proposed openstack/nova master: db: Add resources column in instance_extra table https://review.opendev.org/678447 | |
| 08:55:46 | openstackgerrit | Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResourceList objs https://review.opendev.org/678448 | |
| 08:55:47 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449 | |
| 08:55:47 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450 | |
| 08:55:48 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452 | |
| 08:55:48 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453 | |
| 08:55:49 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454 | |
| 08:55:49 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455 | |
| 08:55:50 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 08:55:50 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 08:55:51 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793 | |
| 08:55:51 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693 | |
| 08:55:52 | openstackgerrit | Luyao Zhong proposed openstack/nova master: objects: Add 'InstanceNUMATopology.cpu_pinning' property https://review.opendev.org/680106 | |
| 08:55:52 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Validate CPU config options against running instances https://review.opendev.org/680107 | |
| 08:55:53 | openstackgerrit | Luyao Zhong proposed openstack/nova master: trivial: Use sane indent https://review.opendev.org/680229 | |
| 08:55:53 | openstackgerrit | Luyao Zhong proposed openstack/nova master: objects: Add 'NUMACell.pcpuset' field https://review.opendev.org/680108 | |
| 08:55:54 | openstackgerrit | Luyao Zhong proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800 | |
| 08:55:54 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571 | |
| 08:59:32 | bauzas | luyao: AFAICS, looks like stephenfin rebased his series on top of yours to prevent more merge conflicts | |
| 08:59:36 | bauzas | stephenfin: correct ? | |
| 09:01:51 | luyao | bauzas: I think so, I handled the merge conflict due to my first two patches | |
| 09:05:59 | bauzas | cool | |
| 09:24:29 | aspiers | kashyap: thanks, please see also https://review.opendev.org/#/c/681866/ | |
| 09:25:56 | kashyap | aspiers: Ah, it's the other thing you mentioned; metadata props. Nice | |
| 09:30:24 | kashyap | aspiers: +1, FWIW. Wauw, given the number of values/props, could spot no typos :D | |
| 09:41:53 | aspiers | kashyap: why is ARM singled out in these docs when we have other defaults for other archs? | |
| 09:43:00 | kashyap | aspiers: I don't know who wrote it originally. But I agree, we should be complete and specify the defaults for other arches, too? | |
| 09:43:12 | aspiers | That was my thought | |
| 09:43:44 | stephenfin | luyao: Correct. Looks like you have it handled. Thanks :) | |
| 09:43:45 | kashyap | Mention it in the review, please. | |
| 09:43:47 | aspiers | kashyap: We probably also need comments in the nova code "# if you change this, don't forget to change useful-image-properties.rst in glance" | |
| 09:44:09 | kashyap | aspiers: Yes, good reminder, to keep things in sync | |
| 09:44:23 | aspiers | kashyap: I'll do that now | |
| 09:47:12 | aspiers | kashyap: hw_machine_type is not the only thing needing a reminder, what else? | |
| 09:49:57 | aspiers | hw_*_bus is one | |
| 09:55:36 | kashyap | aspiers: Yeah, hw_*_bus and machine type are the only things that come to mind | |
| 10:01:01 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add reminder to update corresponding glance docs https://review.opendev.org/682012 | |
| 10:01:02 | aspiers | kashyap: ^^^ | |
| 10:04:38 | kashyap | aspiers: Already has it open | |
| 10:10:00 | kashyap | aspiers: Reviewed; only thing is - do you want to spell out the name of the document? | |
| 10:10:22 | aspiers | kashyap: I removed that from the first version, because it could change, and anyway there are multiple files | |
| 10:10:38 | aspiers | there's the metadefs JSON too | |
| 10:10:39 | kashyap | aspiers: Okay, thought so. | |
| 10:11:44 | kashyap | aspiers: And on casing, is my guess correct? | |
| 10:22:33 | aspiers | kashyap: replied | |
| 10:31:29 | kashyap | aspiers: So I have to "unlearn" the capitalization everywhere except at the start of sentence. | |
| 10:31:35 | aspiers | kashyap: IIUC | |
| 10:32:06 | kashyap | Okido; regardless, the change is ready for merge :-) | |
| 10:40:39 | openstackgerrit | Takashi Kajinami proposed openstack/nova master: Add an option to pass extra parameters for libvirt migration uri https://review.opendev.org/682019 | |
| 10:40:52 | stephenfin | gibi: I'd say hold on rechecking the mox patches | |
| 10:41:59 | stephenfin | We discussed it in the meeting last night and decided to wait until U to merge those now since the gate is busted | |
| 10:43:24 | openstackgerrit | Takashi Kajinami proposed openstack/nova master: Add an option to pass extra parameters for libvirt migration uri https://review.opendev.org/682019 | |
| 10:49:07 | openstackgerrit | Takashi Kajinami proposed openstack/nova master: Add an option to pass extra parameters for libvirt migration uri https://review.opendev.org/682019 | |
| 10:55:06 | openstackgerrit | Merged openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972 | |
| 10:58:45 | tkajinam | ping stephenfin | |
| 11:11:55 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a misuse of assertGreaterEqual https://review.opendev.org/682025 | |
| 11:14:31 | takashin | Hi, core reviewers. Would you review this patch https://review.opendev.org/#/c/682025/ related to a gate failure? | |
| 11:17:19 | cdent | good find takashin | |
| 12:06:31 | sean-k-mooney | donnyd: i have had no issue with any of the FN jobs that have been run in the last 24 hours. they all completed fine. the reson we are rechcking them is to pick up different versions fo the code. i am using a feature of how zuul works to mean i dont need to keep rebasing the zuul job to test diffent parts of the code via an in repo depends on | |
| 12:14:44 | donnyd | sean-k-mooney: that is great news | |
| 12:14:56 | donnyd | Let me know if there is anything else I can do | |
| 12:16:21 | sean-k-mooney | donnyd: at this point we are just waiting on zuul. also my brain is mostly fried after the last few days so im not going to be doing much more until next week | |
| 12:17:24 | gibi | stephenfin: ack | |
| 12:19:36 | sean-k-mooney | donnyd: my next ci task is to create the nodepool lable for nested-virt-bionic and update my ovs-dpdk job https://review.opendev.org/#/c/656580/ then work with limestone and/or vexxhost to expand the multi-numa lable to them and try and enable the whitebox tempest pluging in the nfv job we have been Running against FN | |
| 12:21:09 | sean-k-mooney | donnyd: the white box tempest pluging, combined with writing some new standard test will allow me to compress the many fo the temport jobs down into 1 job | |
| 12:21:42 | sean-k-mooney | donnyd: which im hoping to run as a perodic job intially and eventuly move to check after its stableised. | |
| 12:22:09 | sean-k-mooney | donnyd: so for now relax, you have done more then enough to help with this effort | |
| 12:23:04 | donnyd | I'm very glad I could be helpful. | |
| 12:26:34 | sean-k-mooney | donnyd: for what its worth this is the set of ci topic we talked about last PTG https://etherpad.openstack.org/p/nova-ptg-train-ci | |
| 12:28:28 | sean-k-mooney | donnyd: line 36 multi numa guests and basically everytin form line 31 to 45 is what you have allowed us to start testing | |
| 12:30:43 | donnyd | Well if fn can do anything else, never be afraid to ask | |
| 12:32:22 | sean-k-mooney | efried: speaking of that etherpad line 29 https://etherpad.openstack.org/p/nova-ptg-train-ci can i anounce/propose the deprectation fo libvirt with virt-type uml | |
| 12:33:35 | sean-k-mooney | efried: there are other virt types that also shoudl be marked experimenal or deprecated but too slince they have no testing. lxc is almost workign again but there are defiently gaps | |
| 12:34:01 | sean-k-mooney | so i kind of want to hold off on that until Ussuri | |
| 12:35:28 | sean-k-mooney | efried: mriedem has an experimal lxc job and we have patches to fixed some of the short comings with cloud init https://review.opendev.org/#/c/667976/ | |
| 12:42:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Follow up for I220fa02ee916728e241503084b14984bab4b0c3b https://review.opendev.org/680810 | |
| 12:59:49 | donnyd | sean-k-mooney: I'm pretty interested in lxc functionality due to my resources not being as much as some others out there | |
| 13:00:34 | sean-k-mooney | ya it would be nice to get it working fully again | |
| 13:00:58 | sean-k-mooney | i was able to boot a contianer and inject keys with that cloud init patch | |
| 13:01:08 | donnyd | And knocking down targets from the ptg is a good thing too | |
| 13:02:14 | sean-k-mooney | the next gap for lxc that i hit while testing it is we are not correctly resizing the filesystme in the image to match the route disk on the flaovr | |
| 13:02:23 | sean-k-mooney | and you cant do it inside the contianer | |
| 13:02:43 | sean-k-mooney | so we would have to fix that before i would be comfortable useing it in production | |
| 13:09:57 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Use os-brick locking for volume attach and detach https://review.opendev.org/614190 | |