| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-27 | |||
| 15:26:41 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 16:17:12 | openstackgerrit | Dan Smith proposed openstack/nova master: Ignore deleted services in minimum version calculation https://review.openstack.org/438578 | |
| 16:17:24 | dansmith | mriedem: I think we want that ^ back to mitaka | |
| 16:18:13 | mriedem | ok i'll hold https://review.openstack.org/#/c/438570/ up for that | |
| 16:18:56 | dansmith | cool | |
| 16:19:50 | openstack | Launchpad bug 1667667 in OpenStack Compute (nova) "Rebuilding instance ignores image's property hw_disk_bus" [Low,Confirmed] | |
| 16:19:50 | akijak | hi, I've found a bug in nova concerning rebuilding and 'hw_disk_bus' image property (https://bugs.launchpad.net/nova/+bug/1667667), I'm looking for someone who could give me some directions how to fix that :) can you help? | |
| 16:21:21 | mriedem | dansmith: +2, just need another taker | |
| 16:24:26 | sdague | mriedem: where are these patches? | |
| 16:24:44 | sfinucan | macsz: Any thoughts on https://review.openstack.org/#/c/429042/? | |
| 16:28:08 | mriedem | sdague: https://review.openstack.org/438578 | |
| 16:28:16 | sdague | mriedem: yeh, I found it | |
| 16:33:10 | ildikov | johnthetubaguy: sorry, I passed out, but yes it is, the link was added to one the earlier comments of the patch | |
| 16:33:19 | ildikov | johnthetubaguy: and thanks for approving :) | |
| 16:34:38 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 16:36:11 | efried | mriedem mikal: Per request, split out a spawn/delete change set that's a mere 362LOC. Managed to make it pass jenkins *and* our CI. https://review.openstack.org/#/c/438119/ | |
| 16:39:42 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: ironic inventory reporting https://review.openstack.org/437602 | |
| 16:41:16 | macsz | sfinucan: hmm merging them together sounds like a good idea, let me check the hyper-v code, let's see if it behaves the same as the libvirt's counterpart | |
| 16:51:54 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | |
| 16:55:14 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 16:57:30 | mriedem | sdague: you know how you renamed gate-tempest-dsvm-nnet-ubuntu-xenial ? | |
| 16:57:41 | sdague | mriedem: yep | |
| 16:57:46 | sdague | what now? | |
| 16:57:50 | mriedem | something was broken in the branch filtering | |
| 16:58:02 | mriedem | we haven't been running nova-net since ocata unless it's in the cells v1 job | |
| 16:58:09 | mriedem | so i'm guessing gate-tempest-dsvm-nnet-ubuntu-xenial is running neutron | |
| 16:58:11 | mriedem | else it would be failing | |
| 16:58:20 | mriedem | it should be running on pike or ocata though | |
| 16:58:23 | mriedem | *shouldn't | |
| 16:59:11 | sdague | mriedem: yep, seems to be running neutron | |
| 16:59:19 | sdague | where was the branch filtering for that? | |
| 16:59:43 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 16:59:51 | mriedem | i assume layout.yaml | |
| 16:59:54 | mriedem | looking | |
| 17:01:44 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 17:01:44 | mriedem | L2051 | |
| 17:02:07 | sdague | mriedem: ok, pushing a change | |
| 17:02:15 | sdague | https://review.openstack.org/438621 ? | |
| 17:04:44 | mriedem | sdague: see my comments in https://review.openstack.org/#/c/437137/ | |
| 17:04:47 | mriedem | in the project def too | |
| 17:05:10 | mriedem | the nnet job should be explicit about using nova-network since d-g defaults to using neutron if not specified since ocata | |
| 17:06:12 | sdague | mriedem: there is no explicit nnet | |
| 17:06:27 | mriedem | the nnet job is explicitly nova-network right? | |
| 17:06:33 | mriedem | hence the rename | |
| 17:06:44 | mriedem | you should set "export DEVSTACK_GATE_NEUTRON=0" in those jobs | |
| 17:07:16 | mriedem | because of https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate-wrap.sh#L206 | |
| 17:07:37 | sdague | ah, I didn't realize that happened | |
| 17:07:48 | mriedem | yeah, part of getting nova-net out of ocata jobs | |
| 17:08:00 | mriedem | clarkb and i tag teamed like the hart brothers | |
| 17:10:29 | sdague | how about now? | |
| 17:10:52 | mriedem | you missed one | |
| 17:12:31 | sdague | gah, unsaved buffer | |
| 17:16:59 | openstackgerrit | JeffreyGuan proposed openstack/python-novaclient master: Grammar typo in the comments for function set_meta. https://review.openstack.org/438631 | |
| 17:18:26 | mriedem | dansmith: backports are up https://review.openstack.org/#/q/Ic96a5eb3728f97a3c35d2c5121e6fdcd4fd1c70b,n,z | |
| 17:21:47 | mriedem | sdague: +1 now | |
| 17:25:57 | openstackgerrit | John L. Villalovos proposed openstack/nova master: Use flake8-import-order https://review.openstack.org/435575 | |
| 17:30:40 | dansmith | mriedem: awesome thanks | |
| 17:39:51 | dansmith | mriedem: he found another couple things, but they're newton and beyond so we should be good on mitaka once those merge | |
| 17:40:07 | mriedem | word to your mother | |
| 17:40:20 | mriedem | reminds me i was going to write a spec about exposing the service version in the REST API | |
| 17:42:41 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Backlog spec on preemptible instances https://review.openstack.org/438640 | |
| 17:57:01 | dansmith | mriedem: actually I think we already fixed the other things he hit in newton, but I'll confirm with him tomorrow when he's back | |
| 17:59:56 | openstack | bug 1644248 in OpenStack Compute (nova) ocata "Nova incorrectly tracks live migration progress" [High,In progress] https://launchpad.net/bugs/1644248 - Assigned to Matt Riedemann (mriedem) | |
| 17:59:56 | cfriesen | mriedem: here's the stable/newton release note for bug 1644248 | |
| 18:00:11 | cfriesen | mriedem: https://review.openstack.org/438650 | |
| 18:00:23 | mriedem | ok | |
| 18:09:55 | openstackgerrit | Maciej Szankin proposed openstack/nova master: Remove invalid tests-py3 whitelist item https://review.openstack.org/438660 | |
| 18:11:15 | openstackgerrit | Maciej Szankin proposed openstack/nova master: Remove invalid tests-py3 whitelist item https://review.openstack.org/438660 | |
| 18:12:43 | macsz | johnthetubaguy: https://review.openstack.org/#/c/438660/ | |
| 18:14:06 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/438309 | |
| 18:18:08 | sfinucan | mriedem: RE: the cliff vs. click thing - sorry I missed that discussion (we had gifts to get) | |
| 18:18:10 | openstackgerrit | Maciej Szankin proposed openstack/nova master: Tests: remove .testrepository/times.dbm in tox.ini https://review.openstack.org/435111 | |
| 18:18:14 | sfinucan | but I've comments left on https://review.openstack.org/#/c/433603/2 | |
| 18:18:55 | sfinucan | tl;dr: IMO, cliff is broken by design and no one should be using it. In addition, click offers enough cool things to justify its inclusion in g-r | |
| 18:20:00 | sfinucan | I'd be happy to try talk the openstackclient team around, but without "code on the table" I doubt I'd get far :) | |
| 18:20:35 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: remove *_topic config opts https://review.openstack.org/433924 | |
| 18:21:26 | sdague | sfinucan: well, feel free to have the conversation with the openstack client team, but definitely personally -1 on replacing one different CLI system with another one, instead of standardizing into what everyone else in the community is using | |
| 18:21:57 | dansmith | me too | |
| 18:22:35 | openstackgerrit | John L. Villalovos proposed openstack/nova master: Use flake8-import-order https://review.openstack.org/435575 | |
| 18:24:01 | sfinucan | sdague: I do think they're standardizing on a bad choice but that's understandable | |
| 18:25:25 | sdague | sfinucan: well, there is a ton of experience in the community, cliff was adopted into openstack as an openstack project, so am very much not interested in running off in a different direction | |
| 18:27:20 | sfinucan | sdague: Yup, standardizing on things is good. It's just that I think cliff is a poor choice to standardize on | |
| 18:29:00 | sdague | sfinucan: ok, well get in a time machine and go back to 2013 and have that argument :) | |
| 18:29:20 | sfinucan | :) | |
| 18:51:47 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Reserve NUMA nodes with PCI devices attached https://review.openstack.org/364468 | |
| 18:55:19 | openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: Add support for showing flavor details (v2.43) https://review.openstack.org/435141 | |
| 19:02:14 | mriedem | cfriesen: i added the co-authored-by on your newton reno | |
| 19:02:21 | mriedem | +2 now | |
| 19:08:25 | cfriesen | mriedem: oops, missed that. thanks for the tweak | |
| 19:33:01 | openstackgerrit | Sujitha proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vmops.GetConsoleOutputTestCase https://review.openstack.org/438678 | |
| 19:35:04 | openstackgerrit | Sujitha proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vmops[1] https://review.openstack.org/438678 | |
| 20:21:15 | mriedem | sdague: did you ever find a patch to remove LocalFS? | |
| 20:21:26 | mriedem | i couldn't find one of yours like that | |
| 20:21:39 | sdague | I might have abanoned it, let me go look | |
| 20:22:26 | sdague | https://review.openstack.org/#/c/324720/3 - just restored and recheck | |
| 20:46:25 | mriedem | melwitt: on https://review.openstack.org/#/c/431749/ - was there anything you wanted to add to that given the discussion at the PTG about adding project/user IDs to the placement API for handling quota when a cell is down? | |
| 21:04:31 | mriedem | sdague: just checking, but did you agree to take a stab at a spec for storing limits in keystone? | |
| 21:05:11 | sdague | mriedem: yeh, I need to go look at the keystone spec | |
| 21:05:25 | mriedem | thanks | |
| 21:05:25 | mriedem | ok, just updating some notes | |