| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-09 | |||
| 16:32:34 | stephenfin | lyarwood: Have you got this the right way round? https://review.openstack.org/#/c/530744/ | |
| 16:33:47 | stephenfin | Wait - O comes before P | |
| 16:34:27 | lyarwood | :D | |
| 16:37:56 | mriedem | tovin07: done | |
| 16:38:09 | edleafe | mriedem: thx. Found a bug in my own code, so I'm working on fixing it | |
| 16:39:12 | cdent | functional tests love you more | |
| 16:42:18 | mriedem | efried: powervm ci is going to use SEA rather than OVS for networking right? | |
| 16:42:35 | efried | For the immediate future, yes. esberglu ^ | |
| 16:42:52 | mriedem | so what networking is this running against? https://review.openstack.org/#/c/422512/ | |
| 16:42:55 | mriedem | if SEA isn't yet there | |
| 16:43:35 | efried | I forget that answer. esberglu is on the way... | |
| 16:43:51 | mriedem | Configured mechanism driver names: ['openvswitch', 'linuxbridge'] | |
| 16:43:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Detach volumes on _rebuild_default_impl failure https://review.openstack.org/442105 | |
| 16:43:58 | mriedem | so maybe just the defaults for ML2? | |
| 16:45:17 | lyarwood | stephenfin / mdbooth; if you have any review time left today https://review.openstack.org/#/c/460243/ has been respun to include the fix from https://review.openstack.org/#/c/531233/ | |
| 16:46:38 | esberglu | mriedem: That's just running the CI we've been using since last release with no networking implementation | |
| 16:46:55 | esberglu | About to merge a patch that fixes SEA CI | |
| 16:48:18 | efried | esberglu IOW, neither network patch is yet running a CI that tests networking, but soon the SEA patch will run CI with SEA networking? | |
| 16:48:40 | efried | esberglu And... if we were to run the OVS patch once that is done, it would what, fail? | |
| 16:49:11 | esberglu | efried: No it would pass, but it would be testing SEA networking | |
| 16:49:24 | efried | esberglu How, without the SEA patch? | |
| 16:49:41 | esberglu | efried: PowerVM is running SEA CI on all nova changes as of yesterday | |
| 16:49:54 | efried | esberglu How, without the SEA patch? | |
| 16:50:07 | esberglu | It is applying the SEA patch | |
| 16:50:10 | efried | ahh | |
| 16:50:39 | esberglu | Hence the issue applying the SEA patch to the OVS and SEA changes, which we just fixed | |
| 16:50:43 | efried | that's what we were *just* working on, yeah :) | |
| 16:50:56 | efried | pieces come together. | |
| 16:51:04 | efried | mriedem Does this answer your question? | |
| 16:52:28 | mriedem | huh | |
| 16:52:29 | mriedem | ok | |
| 16:52:52 | mriedem | typically applying patches to the CI results for code that's not yet merged is a bit a of a no-no | |
| 16:53:34 | openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | |
| 16:54:19 | efried | mriedem We have a chicken/egg here | |
| 16:55:25 | mriedem | well, that egg wasn't necessary in pike was it? | |
| 16:55:34 | esberglu | mriedem: No we did this in pike as well | |
| 16:55:40 | mriedem | or is the chicken the os-vif stuff added with the ovs change, which the SEA change relies on? | |
| 16:55:43 | mriedem | oh fun | |
| 16:57:55 | efried | mriedem Yeah, the SEA change is on top of the OVS change. Technically we would probably be able to wait to switch on the patching-and-SEA-networking part of the CI until after the OVS patch merges, but for the sake of getting both patches in the review pipeline as quickly as possible... | |
| 17:03:22 | gibi | mriedem: have you planned to join to the notification subteam meeting? | |
| 17:04:17 | stephenfin | claudiub: Want to look at https://review.openstack.org/#/c/447505/ so I can sign off on it? | |
| 17:05:15 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | |
| 17:07:02 | gibi | mriedem: as nobody joined, I've just closed the meeting, we can discuss thing separatly if neede | |
| 17:08:06 | mriedem | gibi: sorry missed your ping | |
| 17:08:36 | stephenfin | lyarwood: Looked, but my head is gone. I'll hit it tomorrow morning | |
| 17:08:52 | lyarwood | stephenfin: no issues | |
| 17:18:59 | claudiub | stephenfin: done. just fyi. the patch merged in oslotest, and it is included in oslotest 3.2.0. right now, I'm still waiting for the global-requirements bump for oslotest, and then for the OpenStack Bot requirements update to be proposed to nova and be merged, Then finally, that patch is mergeable. :) | |
| 17:19:45 | efried | bauzas You still working on series https://review.openstack.org/#/c/528832/ locally? I see "Done" comments from ~8h ago but no new patch set. | |
| 17:20:09 | openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | |
| 17:27:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: test to make sure volume_use_multipath is properly used https://review.openstack.org/532270 | |
| 17:27:53 | bauzas | efried: yup, I'm in there | |
| 17:28:15 | efried | k. | |
| 17:29:19 | bauzas | dansmith: around ? I have a thought | |
| 17:29:45 | bauzas | dansmith: I'm just building a dictionary of assigned mdevs but looking up the guests by init_host | |
| 17:30:05 | bauzas | dansmith: when I'm assigning a mdev to a guest, I'm also amending that dict | |
| 17:30:29 | efried | bauzas Re+W https://review.openstack.org/#/c/466051/ after rebase? | |
| 17:30:46 | bauzas | dansmith: but for example, say we migrate the instance (or just destroy it), I should call the method recreating the dict | |
| 17:31:22 | bauzas | dansmith: or we could have a mdev said to be assigned while the guest is destroyed | |
| 17:32:04 | bauzas | dansmith: do you think it would be better to call that method by each method (like migrate, rebuild and destroy) or just use a periodic method ? | |
| 17:32:26 | dansmith | bauzas: um, you're not persisting that dict are you? | |
| 17:32:33 | bauzas | if you don't understand my concern, no worries, I'll upload the series soon | |
| 17:33:02 | bauzas | dansmith: by persisting, you mean what ? using an attribute ? | |
| 17:33:19 | dansmith | I mean... persisting it.. I'm not sure what you'd need to update periodically, | |
| 17:33:31 | dansmith | but maybe just upload and we can talk about it concretely | |
| 17:33:34 | bauzas | nevermind, I'll upload | |
| 17:33:40 | bauzas | sure, just working on the tests now | |
| 17:34:41 | bauzas | efried: +Wd | |
| 17:34:45 | efried | thx | |
| 17:35:02 | openstackgerrit | Merged openstack/nova master: Add an additional description for 'token_ttl' https://review.openstack.org/532007 | |
| 17:35:30 | dansmith | mriedem: did you check that this is actually using multiple cells? https://review.openstack.org/#/c/517273/39/nova/tests/functional/compute/test_migration_list.py | |
| 17:38:48 | mriedem | dansmith: i think he copied that from your instance list tests, but it doesn't look like it explicitly creates multiple cells | |
| 17:39:22 | mriedem | https://github.com/openstack/nova/blob/master/nova/tests/functional/compute/test_instance_list.py#L25 | |
| 17:39:24 | mriedem | he needs that | |
| 17:39:27 | mriedem | good point | |
| 17:39:29 | dansmith | but I set | |
| 17:39:32 | dansmith | yeah :) | |
| 17:39:39 | mriedem | removed my +2 | |
| 17:39:54 | mriedem | lay down the -1 | |
| 17:40:05 | dansmith | I done did | |
| 17:44:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: add tests to check multipath in iscsi/fc volume connectors https://review.openstack.org/532272 | |
| 17:47:23 | mriedem | dansmith: per https://review.openstack.org/#/c/517273/39/nova/db/sqlalchemy/api.py@4410 did you want to hold up https://review.openstack.org/#/c/514904/ ? | |
| 17:47:49 | dansmith | no I just wanted to make the comment to sound like I'm paying attention | |
| 17:47:54 | mriedem | heh, ok | |
| 19:15:20 | openstackgerrit | Mark Goddard proposed openstack/nova master: WIP: Send traits to ironic on server boot https://review.openstack.org/508116 | |
| 19:15:20 | openstackgerrit | Mark Goddard proposed openstack/nova master: Add get_traits() method to ComputeDriver https://review.openstack.org/532287 | |
| 19:15:21 | openstackgerrit | Mark Goddard proposed openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288 | |
| 19:15:21 | openstackgerrit | Mark Goddard proposed openstack/nova master: Add has_any_traits() to provider tree https://review.openstack.org/532289 | |
| 19:15:22 | openstackgerrit | Mark Goddard proposed openstack/nova master: Call get_traits() in the resource tracker https://review.openstack.org/532290 | |
| 19:31:44 | efried | mgoddard Let me know if you want to chat about ^ | |
| 20:13:14 | mriedem | wait wait wait, | |
| 20:13:18 | mriedem | johnthetubaguy has a minion? | |
| 20:15:23 | openstackgerrit | Merged openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type https://review.openstack.org/514763 | |
| 20:18:37 | rybridges | Hey Guys! Have a really quick question. Related to this spec: https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/quiesced-image-snapshots-with-qemu-guest-agent.html | |
| 20:19:17 | rybridges | Is that QEMU guest agent that is mentioned something that a user would install on their own instance? Or is that something that we (as operators) would install on the hypervisor itself? | |
| 20:20:29 | mriedem | rybridges: it's in the image | |
| 20:20:44 | mriedem | ‘hw_qemu_guest_agent=yes’ property and ‘hw_require_fsfreeze=yes’ property is set on the image metadata, and QEMU Guest Agent is installed and enabled in the instance | |
| 20:20:54 | mriedem | s/in the instance/in the image/ | |
| 20:21:12 | mriedem | the image metadata is just telling nova that the image has the guest agent and requires fsfreeze | |
| 20:21:14 | mriedem | prior to snapshot | |
| 20:24:09 | rybridges | ok so the part i am confused about is "QEMU Guest Agent is installed and enabled in the instance" | |