| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-28 | |||
| 15:15:21 | melwitt | ok, thanks. sorry didn't know you already knew | |
| 15:15:43 | dansmith | yeah, I opened the tab when mriedem pinged us yesterday | |
| 15:18:00 | melwitt | k | |
| 15:22:27 | mriedem | artom: did you ever follow up with getting something on the starlingx weekly meeting about testing the numa LM series? | |
| 15:30:33 | mriedem | just realized that we have 41 approved blueprints for train with only 12 complete and 2 weeks to FF | |
| 15:31:57 | mriedem | probably time to start deferring things that we know aren't going to land | |
| 15:34:25 | donnyd | so yesterdays test of mounting _base on a shared location didn't go well | |
| 15:35:46 | donnyd | for some unknown reason qemu-img info reports back that the cached image could not be found | |
| 16:00:21 | mriedem | stephenfin: can we effectively mark this complete? https://blueprints.launchpad.net/nova/+spec/remove-cells-v1 | |
| 16:00:29 | mriedem | there are only 2 open changes https://review.opendev.org/#/q/topic:bp/remove-cells-v1+(status:open) | |
| 16:00:54 | openstackgerrit | Merged openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 16:00:58 | stephenfin | Yup, it's complete. Those two are very tricky post-removal cleanup patches | |
| 16:01:05 | stephenfin | I'll come back to them when I can | |
| 16:03:41 | cdent | cells v1 is gone? wooooooo! | |
| 16:03:42 | mriedem | melwitt: would be sweet if we can get https://review.opendev.org/#/c/640898/ approved by the end of the week - it's 2 weeks to FF and it'd be nice to land that series in time for train | |
| 16:04:46 | melwitt | mriedem: indeed, I'm gonna update it after this last call I'm on. thanks for reminding | |
| 16:09:01 | dansmith | mriedem: are you pretty confident on this archive multi cells thing? I've skimmed the logic and tests and just can't really get myself to go deep on it | |
| 16:09:40 | dansmith | seems pretty low risk even if there is something wrong in here, so I'm tempted to just +W based on my skimming | |
| 16:09:52 | dansmith | s/seems/should be/ | |
| 16:12:32 | openstackgerrit | Merged openstack/nova master: scheduler: Flatten 'ResourceRequest.from_extra_specs', 'from_image_props' https://review.opendev.org/674894 | |
| 16:12:41 | openstackgerrit | Merged openstack/nova master: Remove 'hw:cpu_policy', 'hw:mem_page_size' extra specs from API samples https://review.opendev.org/675338 | |
| 16:12:57 | mriedem | dansmith: bombs away | |
| 16:13:05 | mriedem | and yeah i've gone over it several times in detail now | |
| 16:13:09 | mriedem | probably annoying detail for melwitt | |
| 16:13:21 | mriedem | the nova-next change on top also shows it working as expected | |
| 16:15:11 | dansmith | oh sweet I didn't even notice | |
| 16:16:21 | dansmith | got those as well | |
| 16:19:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add online migration for legacy NUMA objects https://review.opendev.org/537414 | |
| 16:19:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure online migrations have a unique name https://review.opendev.org/679096 | |
| 16:19:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Rename 'nova.tests.unit.test_nova_manage' https://review.opendev.org/679095 | |
| 16:19:37 | stephenfin | mriedem: You'll probably be interested in the last of those ^ | |
| 16:21:29 | dansmith | mriedem: 2019-08-28 16:12:53.222764 | primary | 2019-08-28 16:12:53.222 | Missing value auth-url required for auth plugin password | |
| 16:21:37 | dansmith | mriedem: on the grenade lm job | |
| 16:21:49 | dansmith | mriedem: the regular one failed with node_failure | |
| 16:29:48 | melwitt | thanks dansmith++ | |
| 16:33:40 | artom | mriedem, dansmith, was on calls in the car all morning, catching up on scrollback | |
| 16:33:49 | artom | mriedem, crap, forgot about that starlingx meeting thing :( | |
| 16:33:57 | artom | Thanks for reminding me | |
| 16:35:38 | artom | (And right now I need to drive daughter to school, so I need to run again) | |
| 16:35:56 | artom | Though I'll try to respond to dansmith on that initial claims patch before I do so | |
| 16:36:09 | dansmith | artom: we should try to get that test patch running again first | |
| 16:36:13 | dansmith | I'm guessing we | |
| 16:36:19 | dansmith | just need to source openrc first somewhere | |
| 16:38:41 | artom | dansmith, yeah, I need to look at that too - sean-k-mooney seemed to also have ideas | |
| 16:38:55 | dansmith | gI think I got it | |
| 16:39:10 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor https://review.opendev.org/678887 | |
| 16:39:28 | dansmith | pulled a couple lines from the ceph hook which use osc | |
| 16:39:52 | dansmith | artom: I think his ideas were about making it suitable for all time, and that's not the point of this :) | |
| 16:40:24 | artom | dansmith, heh, indeed, quick and dirty all the way :D | |
| 16:40:26 | sean-k-mooney | well i was just suggestin using the way i know works | |
| 16:40:39 | sean-k-mooney | but also yes they are the less hacky way of doing it | |
| 16:40:46 | sean-k-mooney | if you way works thats good enough for this | |
| 16:41:51 | dansmith | I wish we could control which job runs on a patch.. like if the commit message starts with "DNM" then zuul can -2 it and only run tests the commit message asks for or something | |
| 16:46:23 | artom | dansmith, https://review.opendev.org/#/c/635669/45 there, hope that answers is | |
| 16:46:25 | artom | *it | |
| 16:49:14 | dansmith | artom: yeah, although I'm not sure why we need the call out if the current ordering will work | |
| 16:49:24 | dansmith | especially if we plan to remove it from the sriov path and have it in here in the long term | |
| 16:49:26 | mriedem | dansmith: we could also make that DNM change run faster but commenting out all other jobs except the ones we want to run | |
| 16:49:40 | dansmith | mriedem said he would go through the hackery in there and see if he could stomach it | |
| 16:49:43 | dansmith | post-haircut Iguess | |
| 16:50:05 | mriedem_away | i got distracted with the DNM debugging earlier | |
| 16:50:13 | dansmith | that's fine, | |
| 16:50:16 | dansmith | I want to see that test run anyway | |
| 16:51:43 | artom | dansmith, I don't like depending on an ordering that's so far away, but maybe we could call it out in a comment in the compute manager, and lest _test_pci() do its thing | |
| 16:51:48 | artom | *let | |
| 16:52:11 | dansmith | artom: can you just remove the other accounting and rely on this one? | |
| 16:52:30 | dansmith | I mean, I don't want to go all into refactoring it, just curious | |
| 16:52:41 | artom | dansmith, that would be a larger effort to make SRIOV live migration use the move claim | |
| 16:52:43 | dansmith | and yes, not relying on ordering is good, but.. | |
| 16:52:50 | artom | Which should get done, but probably not as part of this series | |
| 16:53:11 | dansmith | I'm not sure I understand why it's any different | |
| 16:53:26 | dansmith | because SRIOV migration is not a different operation, so I would expect you'd still hit all this code and be fine | |
| 16:53:29 | dansmith | but mkay | |
| 16:53:49 | artom | I guess I'm just being overly cautious, and don't want to start a potential rabbit hole at this stage? | |
| 16:53:57 | dansmith | yeah for sure | |
| 16:54:09 | dansmith | just... you said it would work with the current ordering | |
| 16:54:19 | artom | Oh, you mean that? yeah, we could do that | |
| 16:54:26 | dansmith | no, | |
| 16:54:49 | dansmith | I'm taking that to its logical (to me) conclusion | |
| 16:54:50 | dansmith | anyway, | |
| 16:54:51 | dansmith | this is why we really should have been working on this set at the beginning of a cycle and not the end | |
| 16:55:06 | dansmith | so it's frustrating to have to bake in debt like this because of the timeline, | |
| 16:55:09 | artom | Yeah | |
| 16:55:16 | dansmith | especially if you're going to be busy again early next cycle such that it never gets cleaned up | |
| 16:55:51 | artom | I mean, there's no guarantee that we would have stumbled on this earlier, but it's a true thing as a matter of principle, so I can't disagree | |
| 16:56:03 | artom | Anyways, I gotta run for first school day | |
| 16:56:11 | artom | Well, half-day | |
| 16:57:56 | openstackgerrit | Merged openstack/nova master: Add nova.compute.utils.delete_image https://review.opendev.org/637605 | |
| 16:58:06 | openstackgerrit | Merged openstack/nova master: Refactor ComputeManager.remove_volume_connection https://review.opendev.org/642183 | |
| 17:06:29 | spatel | sean-k-mooney: hey!! afternoon | |
| 17:15:02 | donnyd | Do people connect cdrom devices for reasons other than config-drive in most cases? | |
| 17:21:43 | dansmith | mriedem_away: numa flavor got created after my change | |
| 17:25:05 | jroll | so I'm doing GPU passthrough with a few different devices. one of them apparently exposes some SRIOV PFs, so it's presenting as `type-PF` rather than `type-PCI`, and so passthrough isn't working with a `pci_passthrough` property on the flavor. do I need to like, figure out how to do SRIOV and go that route, or is there a way to tell nova to maek it just a regular PCI device? | |
| 17:25:24 | dansmith | mriedem_away: and...failed to boot an instance with it | |
| 18:12:06 | sean-k-mooney | dansmith: Failed to start libvirt guest: libvirtError: Requested operation is not valid: cpu affinity is not supported | |
| 18:12:42 | sean-k-mooney | <domain type='qemu'> | |
| 18:13:02 | sean-k-mooney | setting the virt_type to kvm did not work | |
| 18:13:57 | sean-k-mooney | yep its set to qemu | |
| 18:13:58 | sean-k-mooney | https://d8076f0c34babfa2d8fa-c3c714f53a28264bde7e35c9b01061ab.ssl.cf2.rackcdn.com/678887/8/check/nova-live-migration/21801ca/logs/etc/nova/nova-cpu.conf.txt.gz | |
| 18:14:06 | sean-k-mooney | which is why the job failed | |