| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-26 | |||
| 23:05:34 | mriedem | to mention it's libvirt-only right now | |
| 23:06:49 | mriedem | these are ready to go https://review.openstack.org/#/q/topic:bp/sriov-pf-passthrough-neutron-port-vlan+status:open | |
| 23:06:53 | mriedem | +2s on all | |
| 23:07:35 | bauzas | mriedem: you have enough bandwidth for git rev'ing ocata-3 ? | |
| 23:07:45 | mriedem | bauzas: yeah | |
| 23:07:52 | mriedem | there is always the 'business center' | |
| 23:08:07 | bauzas | or do you want me to do a WIP and you just update it with the gerrit UI? | |
| 23:08:14 | mriedem | i've got it | |
| 23:08:18 | bauzas | okay | |
| 23:08:48 | bauzas | mriedem: dansmith: thanks for helping me on that busy day | |
| 23:09:11 | mriedem | no problema | |
| 23:09:16 | mriedem | er, de nada | |
| 23:11:40 | mriedem | 319379 is 38 in the gate? | |
| 23:11:41 | mriedem | crap | |
| 23:12:11 | mriedem | dansmith: so i guess i'm going to just be tagging o-3 without the cells v2 change, but don't think that's a major issue | |
| 23:12:32 | dansmith | mriedem: are you serious? | |
| 23:12:50 | mriedem | umm i guess not? | |
| 23:12:51 | mriedem | what breaks | |
| 23:13:02 | dansmith | my heart | |
| 23:13:04 | mriedem | ha | |
| 23:13:37 | dansmith | and the plan is for FFE? | |
| 23:13:44 | dansmith | I really really wanted it in o3 | |
| 23:13:48 | dansmith | it's in the gate, you know | |
| 23:13:49 | mriedem | i dont think FFE == o3 | |
| 23:13:58 | mriedem | at least that's not how i was thinking about it | |
| 23:14:01 | mriedem | they just land on the same day | |
| 23:14:22 | mriedem | let me ask | |
| 23:14:34 | dansmith | o3 is FF yes? I'm confused | |
| 23:15:15 | mriedem | meet me in -release | |
| 23:21:17 | mriedem | some comments in the grenade change https://review.openstack.org/#/c/424730/ | |
| 23:25:01 | sfinucan | mriedem: Are we still merging code? | |
| 23:26:00 | mriedem | sfinucan: get these https://review.openstack.org/#/q/topic:bp/sriov-pf-passthrough-neutron-port-vlan+status:open | |
| 23:26:21 | sfinucan | mriedem: That's exactly what I meant :) On it | |
| 23:28:56 | mriedem | dansmith: https://review.openstack.org/#/c/424730/ barfed | |
| 23:29:14 | dansmith | wow sure did | |
| 23:29:25 | mriedem | http://logs.openstack.org/30/424730/9/check/gate-grenade-dsvm-neutron-ubuntu-xenial/bd02e4d/logs/grenade.sh.txt.gz#_2017-01-26_23_08_37_014 | |
| 23:29:33 | mriedem | /opt/stack/new/grenade/projects/60_nova/resources.sh: line 145: syntax error near unexpected token `10' | |
| 23:29:39 | mriedem | sleep 10 | |
| 23:30:12 | dansmith | lol yeah | |
| 23:30:14 | dansmith | pythash | |
| 23:31:37 | openstackgerrit | Stephen Finucane proposed openstack/nova: libvirt: fix vCPU usage reporing for LXC/QEMU guests https://review.openstack.org/393254 | |
| 23:37:39 | openstackgerrit | Stephen Finucane proposed openstack/nova: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | |
| 23:39:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 23:39:42 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | |
| 23:39:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | |
| 23:39:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | |
| 23:39:59 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626 | |
| 23:40:06 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | |
| 23:40:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911 | |
| 23:40:20 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707 | |
| 23:40:31 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 23:40:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 23:41:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512 | |
| 23:41:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 23:41:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 23:43:57 | mriedem | claudiub: got the pci pass through one approved | |
| 23:47:45 | claudiub | mriedem: \o/ Time to party! :D | |
| 23:54:54 | sfinucan | mriedem: Anything else I need to look at before I log off for the night? | |
| 23:55:06 | mriedem | sfinucan: nope | |
| 23:55:16 | mriedem | sfinucan: everything else is in the gate or being watched by me and dan | |
| 23:55:28 | sfinucan | mriedem: Sweet. Cheers :) | |
| 23:55:32 | mriedem | thanks, later | |
| 23:55:44 | mlavalle | mriedem: you may want to take a look at this bug: https://bugs.launchpad.net/nova/+bug/1659215 | |
| 23:55:44 | openstack | Launchpad bug 1659215 in OpenStack Compute (nova) "Should not allow assign floating IPs that are already assigned to another port" [Undecided,In progress] - Assigned to Zhenyu Zheng (zhengzhenyu) | |
| 23:55:58 | mriedem | mlavalle: bugs are the last thing i want to look at right now | |
| 23:56:28 | mlavalle | mriedem: cool, I'll mark it invalid and you can take a look later | |
| 23:56:40 | mriedem | mlavalle: well, i'll forget, | |
| 23:56:50 | mriedem | but if it's a big deal, or controversial, or whatever, just start a ML thread about it | |
| 23:57:42 | mlavalle | mriedem: it's not. i'll take care of it. I just wanted a Nova team memeber to take a quick look. But I understand you are too busy now | |
| 23:58:12 | mlavalle | mriedem: unless I yell for help, you can forget about it :-) | |
| #openstack-nova - 2017-01-27 | |||
| 00:07:31 | openstackgerrit | Eric Brown proposed openstack/nova: Add RPC version aliases for Ocata https://review.openstack.org/425958 | |
| 00:07:33 | mriedem | looks like that grenade change is happy now | |
| 00:07:41 | mriedem | at least for single-node | |
| 00:17:39 | mriedem | dansmith: so the multinode grenade job passed with your change https://review.openstack.org/#/c/424730/10/projects/60_nova/resources.sh but i don't see this in the logs | |
| 00:17:39 | mriedem | echo "Expecting $expected computes" | |
| 00:19:37 | mriedem | oh hrm http://logs.openstack.org/30/424730/10/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/9a57ab4/logs/grenade.sh.txt.gz#_2017-01-26_23_58_11_661 | |
| 00:19:43 | mriedem | Expecting 0 computes | |
| 00:19:48 | mriedem | Found 2 computes | |
| 00:23:47 | openstackgerrit | Jaivish Kothari(janonymous) proposed openstack/os-vif: Fix broken Link https://review.openstack.org/425994 | |
| 00:24:21 | mriedem | http://logs.openstack.org/30/424730/10/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/9a57ab4/logs/grenade_save/grenade_db.ini | |
| 00:24:27 | mriedem | nova_compute_count = 0 | |
| 00:36:36 | openstackgerrit | Jaivish Kothari(janonymous) proposed openstack/os-vif: Removing Deprecated hacking Check https://review.openstack.org/425996 | |
| 00:46:52 | dansmith | mriedem: greeat | |
| 00:48:07 | mriedem | it's weird b/c it finds the 2 in the verify step | |
| 00:48:09 | mriedem | but not before it | |
| 00:48:23 | mriedem | but we're creating resources so we should have 2 computes | |
| 00:52:54 | alex_xu | mriedem: I'm already in vacation. But I think I have better to give you a update. The servers filter BP already merged the filter and sort part. For the policy rule change didn't merge yet, johnthetubaguy pointed out that changing a rule enforcement from a hard to soft should bump microversion. | |
| 00:53:03 | alex_xu | mriedem: and gmann works on it https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-whitelist-for-server-list-filter-sort-parameters | |
| 00:54:43 | alex_xu | mriedem: that probably needs two patch, one is about the rename of the policy rule, then change the hard to soft by microversion. I'm not sure whether it can get FFE. That should be your call | |
| 00:55:08 | alex_xu | mriedem: or you want something detail, I can write a email | |
| 00:55:50 | gmann | mriedem: as alex_xu said. i will try to push microversion patch (max over weekend) also and then you can take call on that | |
| 00:56:07 | alex_xu | gmann: thanks | |
| 00:57:13 | alex_xu | gmann: mriedem another thing is worth to track is this bug https://launchpad.net/bugs/1658571 | |
| 00:57:13 | openstack | Launchpad bug 1658571 in OpenStack Compute (nova) "Microversion 2.37 break 2.32 usage" [High,In progress] - Assigned to Artom Lifshitz (notartom) | |
| 00:57:38 | gmann | alex_xu: mriedem i updates the spec with latest discusion, if you guys can have a look - https://review.openstack.org/#/c/425533/ | |
| 00:59:07 | alex_xu | gmann: let me check now | |
| 00:59:16 | gmann | alex_xu: about your comment on https://review.openstack.org/#/c/415330/25/nova/policy.py | |
| 00:59:57 | alex_xu | gmann: yea | |
| 01:00:08 | gmann | alex_xu: that if condition is when policy file is updated right? for example say if old deployer has policy file(with old rule overridden) and do not change file then also it hit if condition ? | |
| 01:01:02 | mriedem | alex_xu: gmann: a ML thread on the sort/filters policy thing is probably best | |