| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-29 | |||
| 13:35:26 | mriedem | lpetrut: ah very possible. this doesn't grab the instance first, but if it did, the _get_instance method in the compute API would set the cell mapping in the context | |
| 13:36:13 | mriedem | lpetrut: can you report a bug? | |
| 13:36:37 | gibi | mriedem: ahh I see why you didn't added a FakeDriver variant. | |
| 13:36:40 | lpetrut | mriedem: sure. the issue is that at this point, the instance id is unknown as it's not passed in the api call. | |
| 13:36:59 | mriedem | lpetrut: yeah, which sucks, so we'll likely have to iterate the cells looking for the bdm | |
| 13:37:12 | mriedem | and once found, target the context for the rest of the calls | |
| 13:37:31 | mriedem | we wouldn't have hit this in CI because only certain drivers, like NFS, test this API | |
| 13:37:42 | mriedem | there is an NFS job, but it's non-voting and in the experimental queue i think | |
| 13:38:09 | mriedem | gibi: right i'd have to create a new test class to use a new virt driver | |
| 13:38:16 | openstackgerrit | Viktor Varga proposed openstack/nova master: Ensure image metadata when unshelving BFV instance https://review.openstack.org/498729 | |
| 13:38:18 | mriedem | gibi: i can see what mocking out the driver method instead looks like | |
| 13:38:38 | lpetrut | mriedem: yep, that one's failing because of this: http://logs.openstack.org/21/498321/5/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/b8bca96/logs/screen-c-vol.txt.gz?level=TRACE | |
| 13:39:21 | mriedem | yeah - open a bug and we'll get a fix and backport for pike | |
| 13:39:35 | mriedem | probably not the pike GA at this point though, RC2 was last week | |
| 13:39:48 | lpetrut | mriedem: sure. thanks | |
| 13:40:20 | mriedem | lpetrut: btw, does hyperv have a backend that supports this API? or just got this from a customer? | |
| 13:40:46 | lpetrut | yeah, we have the Cinder SMB driver which uses it | |
| 13:41:00 | gibi | mriedem: if you mock.patch.object the FakeDriver object in only in the compute manager that you need to behave differently then you can even drop the if from your fake method. But these are just small gains I agree | |
| 13:41:18 | mriedem | lpetrut: ok, should get the docs updated then https://developer.openstack.org/api-ref/compute/#assisted-volume-snapshots-os-assisted-volume-snapshots | |
| 13:41:26 | mriedem | because that says only the libvirt driver supports this | |
| 13:41:44 | lpetrut | good point, thanks for mentioning it | |
| 13:42:10 | mriedem | lpetrut: actually i only see the libvirt driver implementing the "volume_snapshot_create" method in tree | |
| 13:42:20 | mriedem | are you sure you don't have something in the compute hyperv repo that's not in nova? | |
| 13:43:19 | lpetrut | mriedem: actually, you're right. that didn't make it upstream | |
| 13:43:31 | mriedem | the bdm not found bug is still valid | |
| 13:43:39 | mriedem | for libvirt + nfs | |
| 13:43:58 | lpetrut | yep, I'll point to the NFS CI results | |
| 13:49:42 | gibi | mriedem: OK. I'm convinced. The manager method your patch mocks just basically calls the driver methdon I suggested to mock, so we are not loosing real coverage with your mock | |
| 13:50:27 | gibi | mriedem: so I'm +1 | |
| 13:51:40 | mriedem | heh, and i just changed the test to mock the driver | |
| 13:52:29 | mriedem | and it's passing, so i'll just leave it | |
| 13:53:28 | gibi | mriedem: ohh, OK | |
| 13:53:36 | gibi | mriedem: I like it both ways now | |
| 13:54:20 | mriedem | jaypipes: is the derek carr on that k8s rescheduler email the same as the quarterback for the raiders? | |
| 13:54:25 | mriedem | because if so, that would be impressive | |
| 13:54:35 | mriedem | oh also a red hat employee | |
| 14:01:36 | openstackgerrit | Dan Smith proposed openstack/nova-specs master: WIP: Add migration-allocations spec https://review.openstack.org/498510 | |
| 14:07:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add user index page https://review.openstack.org/498817 | |
| 14:07:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add configuration index page https://review.openstack.org/498818 | |
| 14:07:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages https://review.openstack.org/498819 | |
| 14:07:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add contents pages https://review.openstack.org/498820 | |
| 14:07:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout https://review.openstack.org/498821 | |
| 14:08:19 | sean-k-mooney | beagles: hi o/ did you get an answer to your os-vif question? | |
| 14:09:37 | beagles | sean-k-mooney, no I ended up filing a bug to avoid falling through cracks https://bugs.launchpad.net/os-vif/+bug/1713590 | |
| 14:09:37 | openstack | Launchpad bug 1713590 in os-vif "Plugging VFs no longer works without a readable phys_switch_id" [Undecided,New] | |
| 14:10:37 | beagles | sean-k-mooney, I'm not 100% on the code path so there might be something else going on or the configuration of the test system might be missing something. From the information I had, this seemed a valid issue | |
| 14:11:30 | artom | claudiub|2, hey, back, took longer than planned, sorry | |
| 14:11:47 | claudiub|2 | artom: no problem. :) | |
| 14:12:51 | claudiub|2 | artom: i have a question regarding the test_device_tagging tempest test | |
| 14:12:56 | artom | claudiub|2, sure | |
| 14:13:02 | claudiub|2 | artom: what kind of image are you using when testing it? | |
| 14:13:10 | claudiub|2 | artom: i assume cirros, right? | |
| 14:13:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove deprecated call to sphinx.util.compat https://review.openstack.org/498824 | |
| 14:15:57 | sean-k-mooney | beagles: os vif does not handel pluging vf normally | |
| 14:16:17 | sean-k-mooney | beagles: its should only do this if you have ovs with hardware offlod support. | |
| 14:16:36 | artom | claudiub|2, it uses a boot volume, created from whatever's image defined in the configuration | |
| 14:17:00 | artom | claudiub|2, https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_device_tagging.py#L102 | |
| 14:17:01 | sean-k-mooney | beagles: in you ml2_conf.ini i would guess you have the openvswitch mech driver listeted before the sriovnic agent correct? | |
| 14:17:02 | claudiub|2 | artom: i know. :) | |
| 14:17:12 | claudiub|2 | artom: but what do *you* use to test it with? :D | |
| 14:17:23 | artom | claudiub|2, doh, hah! Yeah, I used cirros | |
| 14:17:34 | claudiub|2 | hm interesting | |
| 14:17:34 | andreaf | zul: around? I have a change in Tempest that if merged would break some nova-lxd integration tests from the in-tree tempest plugin | |
| 14:17:47 | artom | claudiub|2, what's up? | |
| 14:17:51 | beagles | sean-k-mooney, interesting ... tbh I don't know, the test system is likely gone at the moment. Where there was a patch that introduced this change to tripleo a short while ago, I think that's a safe bet | |
| 14:18:21 | andreaf | zul: but I don't see those tests running anywhere in nova-lxd gate, so I was wondering if I can just change tempest and propose a patch to fix nova-lxd afterward? | |
| 14:18:31 | sean-k-mooney | beagles: basically is you want sriov and ovs in the same deployment and all your sriov host dont supprot hardware offload you have to list sriov first. | |
| 14:19:07 | claudiub|2 | artom: i'm also using a cirros, but only eth0 is configured in it, even if all the interfaces are attached. the problem is that the interface which gets to be eth0 is random, and making the VM un-ssh-able | |
| 14:19:35 | claudiub|2 | artom: as the floating IP is associated with just one of the many private IPs the VM has. | |
| 14:20:08 | beagles | sean-k-mooney,k. I'll check on that | |
| 14:20:16 | sean-k-mooney | beagles: that said we can probaly harden that could a more and retrun a specific exction rather then causing an io error by reading a file that does not exist | |
| 14:20:22 | claudiub|2 | artom: was wondering if you had an issue like this | |
| 14:20:43 | artom | claudiub|2, I haven't run it manually in a long time | |
| 14:20:53 | artom | claudiub|2, how long has this been going on? | |
| 14:21:18 | claudiub|2 | artom: i've started testing the pike release, so i've only seen it nowadays | |
| 14:21:56 | artom | claudiub|2, interesting. https://review.openstack.org/#/c/463837/ merged recently, and it changes which floating ip client gets used | |
| 14:22:16 | artom | But it passed the gate, so in and of itself it's not the root cause | |
| 14:23:02 | claudiub|2 | hm, interesting | |
| 14:24:12 | artom | claudiub|2, there was also https://review.openstack.org/#/c/471116/ | |
| 14:24:25 | artom | But again, passed the gate, so by itself it's not the issue | |
| 14:24:37 | zul | andreaf: check with jamespage | |
| 14:24:56 | artom | claudiub|2, I don't suppose you're running with nova-network by omission? | |
| 14:25:14 | claudiub|2 | artom: i don't think i've ever used nova-network. :) | |
| 14:25:25 | artom | claudiub|2, good. | |
| 14:25:36 | andreaf | zul: cool thanks - jamespage: this is the tempest patch https://review.openstack.org/#/c/492577/ | |
| 14:26:00 | andreaf | jamespage: and this is nova-lxd code that would be affected http://git.openstack.org/cgit/openstack/nova-lxd/tree/nova_lxd_tempest_plugin/tests/api/compute/servers/test_create_server.py | |
| 14:26:14 | artom | claudiub|2, I'm sorry I can't be of more help, but I feel like you'll have to dig into tempest to figure out why it's doing that | |
| 14:26:46 | artom | The test itself doesn't assign any floating IPs, it just sets validatable=True | |
| 14:26:51 | claudiub|2 | artom: yeah, will do. thanks for the input though. :) | |
| 14:26:58 | artom | And then tempest does its thing behind the scenes | |
| 14:29:44 | artom | claudiub|2, it may have something to do with the tenant network in your deployment: https://github.com/openstack/tempest/blob/master/tempest/common/compute.py#L206 | |
| 14:31:03 | stephenfin | mriedem, jaypipes: Fancy taking a look at these? https://review.openstack.org/#/c/498817/ and successors | |
| 14:31:26 | stephenfin | They fix some broken links that dhellmann pointed out and should be backported to Pike | |
| 14:31:37 | mriedem | stephenfin: seems like something sdague would have an opinion about | |
| 14:31:40 | mriedem | regarding layout | |
| 14:31:50 | stephenfin | Indeed. Any idea when he's back? | |
| 14:31:57 | mriedem | this week | |
| 14:32:00 | mriedem | he was here yesterday | |
| 14:32:11 | stephenfin | Oh, then I can wait for him too | |
| 14:32:23 | stephenfin | (fwiw, I'm mostly just moving his stuff about) | |