| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-28 | |||
| 19:22:01 | mriedem | dansmith: well, compared to what? the alternative of making numa live migration miss the release to retrofit the sriov pci claim stuff into the actual MoveClaim? if so, then yes this dirty hackery is acceptable i guess | |
| 19:22:09 | mriedem | i have a feeling it's a hack that we'll have forever | |
| 19:22:12 | mriedem | but such is nova | |
| 19:22:31 | dansmith | yes, that's what I mean, and yes that's the shame I cast on artom earlier this morning about not cleaning it up after the release | |
| 19:22:38 | dansmith | we'll see if the shame works or not | |
| 19:23:04 | melwitt | cast shame 3 | |
| 19:23:18 | melwitt | highest level spell | |
| 19:23:30 | sean-k-mooney | one way or another we will converge them in U | |
| 19:23:31 | mriedem | artom is pretty good at saving throws though | |
| 19:23:43 | melwitt | lol | |
| 19:23:57 | sean-k-mooney | if artom does not work on it i will. | |
| 19:24:03 | dansmith | I'm going to pretend I understand all this dorkery and chuckle | |
| 19:24:20 | melwitt | I know, you're just too cool | |
| 19:24:44 | mriedem | https://en.wikipedia.org/wiki/Saving_throw god | |
| 19:24:45 | sean-k-mooney | what the DC on this charisma check | |
| 19:25:13 | mriedem | if it weren't for artom's high charisma he'd have been dead long ago | |
| 19:25:19 | dansmith | I just never played any of that stuff so I don't know | |
| 19:25:34 | melwitt | sean-k-mooney: I had to google that xD | |
| 19:25:46 | melwitt | lol mriedem | |
| 19:26:10 | melwitt | that was a good one | |
| 19:26:32 | sean-k-mooney | on of my friends runs a game and got me to sit in at teh weekend so that make 3 times i have played a dnd game now... not sure if that is a good or a bad thing | |
| 19:27:31 | melwitt | I'm sure it's good. I have never played, I'm video game only | |
| 19:27:44 | mriedem | i've never actually played a sit down board game but my brother had all that stuff and i rocked the ibm pc floppies for https://en.wikipedia.org/wiki/Pool_of_Radiance | |
| 19:28:21 | sean-k-mooney | most of my iteraction with D&D was baldur's gate on the pc | |
| 19:28:37 | melwitt | similar for me. I played https://en.wikipedia.org/wiki/Eye_of_the_Beholder_III:_Assault_on_Myth_Drannor | |
| 19:33:14 | dansmith | man ya'll'er dorks | |
| 19:33:39 | sean-k-mooney | say the ham radio nerd :) | |
| 19:34:09 | melwitt | shots fired | |
| 19:34:20 | dansmith | i, uh | |
| 19:34:22 | dansmith | well, | |
| 19:34:55 | dansmith | I guess I'll be proud of being a radio nerd and you all can be, uh, proud of your games :D | |
| 19:35:17 | sean-k-mooney | :) | |
| 19:36:11 | melwitt | lol | |
| 19:36:34 | dansmith | sean-k-mooney: do we need to recheck that LM test | |
| 19:36:36 | dansmith | ? | |
| 19:36:55 | dansmith | ah, you removed a bunch of other jobs | |
| 19:37:00 | dansmith | I thought the zuul report looked weird | |
| 19:37:14 | sean-k-mooney | it did but i still have an issue in v11 | |
| 19:37:26 | sean-k-mooney | im just goint to remove the sub group stuff | |
| 19:37:38 | sean-k-mooney | which is what i did in v10 | |
| 19:37:48 | dansmith | okay | |
| 19:38:23 | openstackgerrit | sean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor https://review.opendev.org/678887 | |
| 19:43:24 | sean-k-mooney | ok its actully running this time | |
| 19:43:26 | sean-k-mooney | http://zuul.openstack.org/stream/951e3e39a4014163aeb5c238c84a2840?logfile=console.log | |
| 19:44:00 | sean-k-mooney | ill keep an eye on it until dinner is done cooking | |
| 19:45:49 | melwitt | I'm gonna be out tomorrow and friday, fyi | |
| 20:38:28 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914 | |
| 20:38:28 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: use domain capabilities to get supported device models https://review.opendev.org/666915 | |
| 20:38:29 | openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 20:38:42 | sean-k-mooney | i have artoms series stacking. going for dinner backin 30 mins or so | |
| 20:39:08 | openstackgerrit | Merged openstack/nova master: Add power_on kwarg to ComputeDriver.spawn() method https://review.opendev.org/642590 | |
| 20:39:21 | openstackgerrit | Merged openstack/nova master: Add Destination.allow_cross_cell_move field https://review.opendev.org/614035 | |
| 20:41:34 | sean-k-mooney | also it looks like the legacy jobs just entrily ignore the stuff in the zuul.yaml | |
| 20:41:42 | sean-k-mooney | it failed again because it used qemu | |
| 20:59:20 | openstackgerrit | Merged openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.opendev.org/507486 | |
| 20:59:29 | openstackgerrit | Merged openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840 | |
| 20:59:50 | melwitt | \o/ | |
| 21:03:06 | mriedem | sean-k-mooney: yeah i wasn't sure if that was going to work b/c of the legacy job format, that's why i threw it into the run.yaml | |
| 21:03:24 | mriedem | sean-k-mooney: we could....push a dnm devstack patch that removes that LIBVIRT_TYPE=qemu default and add a depends-on from the nova patch | |
| 21:03:41 | mriedem | or simply changes the default in devstack to kvm and depends-on that | |
| 21:10:14 | sean-k-mooney | or you know hard code it in nova | |
| 21:11:13 | sean-k-mooney | it finished stacking while i was having dinner so ill test it now | |
| 21:20:08 | mriedem | yeah i guess we could hardcode it in nova in the dnm patch, that's the easiest | |
| 21:27:26 | fungi | any nova-stable-core (besides mriedem, thanks!!!) care to have a look at https://review.opendev.org/678098 ? | |
| 21:27:46 | mriedem | melwitt: ^ | |
| 21:27:50 | fungi | i've got an advisory staged and just want to set the date on it once i know that's approved, or likely to get approved | |
| 21:28:47 | fungi | i wouldn't normally bug folks, but it's been sitting there for a bit | |
| 21:37:50 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for validating instance and node https://review.opendev.org/656028 | |
| 21:37:50 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690 | |
| 21:37:51 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691 | |
| 21:37:51 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213 | |
| 21:43:32 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.77: Support Specifying AZ to unshelve https://review.opendev.org/665136 | |
| 21:51:13 | sean-k-mooney | ok i need to do it again but i think the latest patch works. | |
| 21:54:12 | mriedem | takashin: alex_xu: i think the 2.77 novaclient change is ready https://review.opendev.org/665136 | |
| 22:01:44 | sean-k-mooney | artom: dansmith: mriedem: http://paste.openstack.org/show/766650/ that is the vm xml before and after + the pci request before and after and the migration_context object | |
| 22:02:51 | sean-k-mooney | the flavor had cpu pinning enabeld. the hosts had non overlapping vcpu_pin_sets so the pinning had to be recalulated and the vm had an sriov-macvtap port | |
| 22:04:53 | sean-k-mooney | it look like cold migration back to the first host also works | |
| 22:10:48 | sean-k-mooney | the pci device in teh pci_device table are also updated correctly http://paste.openstack.org/show/766651/ | |
| 22:42:02 | melwitt | fungi: sorry about that. I see smcginnis has got it, thanks | |
| 22:43:12 | fungi | yep, thanks for following up! | |
| 22:43:26 | fungi | i'm planning to circulate the advisory tomorrow | |
| 22:43:57 | sean-k-mooney | melwitt: do we have that backported downstream yet | |
| 22:44:08 | sean-k-mooney | if not i can try and do that tomorow | |
| 22:44:15 | melwitt | sean-k-mooney: not that I'm aware of | |
| 22:44:26 | sean-k-mooney | we havent relesat with stien yet but it would be good to do before we go to rc | |
| 22:44:38 | melwitt | fungi: great, thank you for putting it together! | |
| 22:44:52 | sean-k-mooney | it doesnt actuly affect our product since we dont support linuxbridge | |
| 22:45:11 | sean-k-mooney | so its not as much of an issue downstream | |
| 22:45:45 | melwitt | sean-k-mooney: yeah. maybe could bring it up on the team call tomorrow, I'm not sure what would be the guidance about it | |
| 22:46:08 | sean-k-mooney | it was discussed downstream i just dont know if we did the backport yet | |
| 22:46:13 | melwitt | oh ok | |
| 22:46:20 | melwitt | I didn't do it so unless someone else did | |
| 22:47:04 | sean-k-mooney | it shoud be trivial enough to do. | |
| 22:47:12 | sean-k-mooney | anyway with that ill call it a night | |
| 22:47:13 | sean-k-mooney | o/ | |
| 22:53:39 | melwitt | o/ | |
| 23:17:50 | artom | sean-k-mooney, so that's a woot? | |
| 23:18:01 | artom | (If you're not gone) | |
| 23:27:35 | openstackgerrit | Merged openstack/nova master: [Trivial]Remove unused helper check_temp_folder https://review.opendev.org/678973 | |
| 23:51:13 | openstackgerrit | Merged openstack/os-vif stable/stein: only disable mac ageing for ovs hybrid plug https://review.opendev.org/678098 | |