| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-26 | |||
| 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 | |
| 01:01:05 | mriedem | i'm leaving soon for dinner | |
| 01:01:54 | alex_xu | gmann: it only happened when the policy file changed. the olso.policy will check timestamp of policy file, then it will reload the file. Then you will pass that if condition | |
| 01:02:03 | alex_xu | mriedem: got it | |
| 01:02:36 | mriedem | for https://launchpad.net/bugs/1658571 we said we'd fix it after FF as a bug fix, but that was before i was reminded that we only have 1 week until rc1 | |
| 01:02:36 | openstack | Launchpad bug 1658571 in OpenStack Compute (nova) "Microversion 2.37 break 2.32 usage" [High,In progress] - Assigned to Artom Lifshitz (notartom) | |
| 01:03:01 | gmann | mriedem: +1 | |
| 01:03:10 | alex_xu | mriedem: 1 week from now? | |
| 01:03:15 | mriedem | alex_xu: yeah | |
| 01:03:25 | alex_xu | ok, got it | |
| 01:03:29 | mriedem | so, that one might end up waiting until pike | |
| 01:03:34 | mriedem | since it was already broken in newton | |
| 01:03:40 | mriedem | we might just have to document it for ocata | |
| 01:03:56 | alex_xu | ok | |
| 01:04:22 | mriedem | i've tagged the bug with ocata-rc-potential though so we don't forget about it | |
| 01:04:30 | alex_xu | cool | |
| 01:04:36 | gmann | it looks pretty close if we can get in Ocata only but yea let's not hurry on that may be | |
| 01:04:48 | alex_xu | gmann: +1 | |
| 01:05:02 | gmann | alex_xu: but for old deployer timestamp will not be changed right? | |
| 01:05:25 | gmann | alex_xu: where we can see those warning so that we can verify the same | |
| 01:05:59 | alex_xu | gmann: emm...they should get a warning after upgrade, due to 'saved_file_rules' is empty | |
| 01:07:00 | gmann | alex_xu: ah yea, thats is nice point | |
| 01:16:32 | artom | mriedem, is one week really too little? | |
| 01:16:51 | artom | I've addressed gmann's and alex_xu's feedback :) | |
| 01:17:14 | gmann | artom: +1, ll try to check today. m sure that is very close | |
| 01:17:52 | artom | The patch itself is fairly straightforward, if discussion is needed I think it's more about the strategy | |
| 01:18:04 | mriedem | artom: w/o looking at the change i can't really say | |
| 01:18:12 | mriedem | artom: we should have a spec for the thing too since it's a new microversion right? | |
| 01:18:14 | artom | My understanding is that a new microversion is the prefered approach | |