| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-28 | |||
| 18:16:38 | sean-k-mooney | the base gate jobs hardcode it to qemu but we can just override it | |
| 18:16:50 | dansmith | ack | |
| 18:21:43 | mriedem_away | overriding it is what i did in the run.yaml | |
| 18:22:07 | sean-k-mooney | ya but it didnt work | |
| 18:22:08 | lbragstad | johnthetubaguy any chance you'd be willing to take a peek at https://review.opendev.org/#/c/644615/ ? | |
| 18:22:09 | sean-k-mooney | https://github.com/openstack/devstack/blob/master/.zuul.yaml#L223 | |
| 18:22:12 | sean-k-mooney | im just going to set this | |
| 18:22:14 | mriedem | i removed it from the run_test script b/c it was only setting it on the primary and not the subnode compute as well, but we shouldn't need to do that since it's in local.conf | |
| 18:22:27 | mriedem | sean-k-mooney: yeah, that's what i did in the run.yaml | |
| 18:22:47 | mriedem | sean-k-mooney: https://review.opendev.org/#/c/678887/8/playbooks/legacy/nova-live-migration/run.yaml | |
| 18:22:59 | sean-k-mooney | i didnt think you coudl do it there | |
| 18:23:10 | sean-k-mooney | or rather i hav enever tried too | |
| 18:23:18 | sean-k-mooney | oh ya | |
| 18:23:19 | mriedem | it's the legacy way | |
| 18:23:24 | sean-k-mooney | so it ended up being there twice | |
| 18:23:33 | sean-k-mooney | both with qemu and then with kvm | |
| 18:23:39 | sean-k-mooney | i dont know why qemu | |
| 18:23:42 | sean-k-mooney | won | |
| 18:23:44 | sean-k-mooney | but it did | |
| 18:24:28 | mriedem | yeah, so i guess just overriding in zuul.yaml might work | |
| 18:24:37 | sean-k-mooney | ill set it in all the places | |
| 18:24:43 | sean-k-mooney | one of them will work :) | |
| 18:24:53 | mriedem | i'd say while doing that, also comment out the jobs we don't care about running | |
| 18:25:09 | sean-k-mooney | ya no need to was gate time | |
| 18:25:16 | mriedem | could also just do post-config|$NOVA_CPU_CONF | |
| 18:25:40 | sean-k-mooney | yep | |
| 18:25:46 | mriedem | but i don't really trust that LIBVIRT_TYPE variable | |
| 18:25:53 | sean-k-mooney | i can do that too | |
| 18:26:01 | mriedem | dansmith: so is it worth me looking at https://review.opendev.org/#/c/635669/45 right now? | |
| 18:26:13 | dansmith | yes | |
| 18:32:50 | openstackgerrit | sean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor https://review.opendev.org/678887 | |
| 18:33:13 | sean-k-mooney | that is proably over kill but it should for the config to what we want it to have | |
| 18:50:13 | openstackgerrit | sean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor https://review.opendev.org/678887 | |
| 18:52:19 | openstackgerrit | sean mooney proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor https://review.opendev.org/678887 | |
| 19:11:53 | mriedem | dansmith: i've disposed of my soiled diaper | |
| 19:12:53 | dansmith | mriedem: I was looking for confirmation that the hacks to avoid disrupting the sriov stuff would fly with you, which I assume from the summary is ... yes? | |
| 19:13:09 | mriedem | it's definitely better than it was | |
| 19:13:25 | mriedem | being explicit i mean | |
| 19:13:51 | dansmith | yes, but... is that a yes? :) | |
| 19:14:20 | sean-k-mooney | ill redeploy with this version either later today or tomorow and report what i find. | |
| 19:16:06 | dansmith | sean-k-mooney: thanks | |
| 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: use domain capabilities to get supported device models https://review.opendev.org/666915 | |
| 20:38:28 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914 | |
| 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 | |