| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-25 | |||
| 06:08:21 | Kevin_Zheng | :) | |
| 06:08:45 | gmann | seems like 2 of us. let's wait couple of min | |
| 06:08:55 | alex_xu | o/ | |
| 06:09:09 | gmann | alex_xu: hi | |
| 06:09:17 | alex_xu | gmann: good afternoon | |
| 06:09:25 | gmann | good afternoon | |
| 06:10:32 | gmann | let's start | |
| 06:10:52 | gmann | #topic Priorities | |
| 06:11:11 | gmann | #link https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | |
| 06:11:13 | gmann | L57 | |
| 06:11:23 | openstackgerrit | Rajesh Tailor proposed openstack/nova master: Fix host validity check for live-migration https://review.openstack.org/401009 | |
| 06:11:32 | gmann | 1. Servers Ips non-unique network names | |
| 06:11:49 | gmann | this one is no progress. i did not get time to start this due to extensions merge work | |
| 06:12:06 | gmann | which mean this would not go in Rocky. and we need to carry this to Stein | |
| 06:13:15 | gmann | 2.Abort live migration in queued state: | |
| 06:13:25 | gmann | this is done, thanks Kevin_Zheng | |
| 06:13:35 | gmann | 3. Complex anti-affinity policies: | |
| 06:13:55 | gmann | this is also done, thanks yikun | |
| 06:14:13 | gmann | 4. Volume multiattach enhancements: | |
| 06:14:31 | gmann | this is on same state. and most probably will be stein one | |
| 06:15:30 | gmann | 5. API Extensions merge work | |
| 06:15:44 | gmann | this is partially done | |
| 06:16:14 | gmann | part-1 schema merge - Completed | |
| 06:16:15 | gmann | part-2 server_create merge - last patch on gate | |
| 06:16:52 | alex_xu | gmann: so we just done the server_create merge in Rocky, right? | |
| 06:17:11 | gmann | alex_xu: yes, last patch is on gate - | |
| 06:17:13 | gmann | #link https://review.openstack.org/#/c/583882/2 | |
| 06:17:19 | gmann | trying to get this in | |
| 06:17:45 | gmann | alex_xu: part-3 merging the view builder is left to merge. I have many of them up for review | |
| 06:18:05 | gmann | #link https://review.openstack.org/#/q/topic:bp/api-extensions-merge-rocky+status:open | |
| 06:18:22 | gmann | which will be in Stein i think. | |
| 06:18:31 | gmann | alex_xu: ? | |
| 06:18:52 | alex_xu | gmann: yea, just as mriedem said | |
| 06:19:13 | gmann | yeah, it is too late to merge them and gate also not so good since last week | |
| 06:19:41 | gmann | none of them are reviewed so i do not think they can make it | |
| 06:19:54 | gmann | alex_xu: today is FF or tomorrow ? | |
| 06:20:05 | alex_xu | tomorrow, our evening | |
| 06:20:13 | gmann | ok | |
| 06:20:50 | gmann | i have left for 3 patch to push to complete it but at least 6-7 total are left for review | |
| 06:21:31 | vishakha | gmann, Hi, i've been working on the bug https://bugs.launchpad.net/nova/+bug/1644457 but i found out that key_pair is a special case in which in_use remains always 0 as seen here https://github.com/openstack/nova/blob/master/nova/quota.py#L189 | |
| 06:21:32 | openstack | Launchpad bug 1644457 in OpenStack Compute (nova) "keypair quota error" [Medium,Confirmed] - Assigned to Vishakha Agarwal (vishakha.agarwal) | |
| 06:21:49 | gmann | 7 up for review + 5 remaining to push | |
| 06:22:11 | gmann | vishakha: ok, let's discuss that during bug topic | |
| 06:22:27 | gmann | alex_xu: so we make final to postponed view builder to stein? | |
| 06:23:00 | alex_xu | gmann: yea, I think so, there are a lot of view builder, right? | |
| 06:23:15 | alex_xu | #link https://review.openstack.org/#/q/topic:bp/handling-down-cell+(status:open+OR+status:merged) | |
| 06:23:24 | alex_xu | ^this one is looking for your view builder change | |
| 06:23:50 | gmann | yeah, that makes this series easy | |
| 06:23:59 | vishakha | gmann, ok | |
| 06:24:20 | gmann | alex_xu: total 12 patches to get in (7 up for review + 5 need to push) | |
| 06:24:47 | alex_xu | gmann: doesn't sound we can make it in one day | |
| 06:25:08 | gmann | alex_xu: yes, not possible. | |
| 06:25:41 | gmann | alex_xu: ok i will postponed them to stein. and will push them soon so that we can merge them early in stein | |
| 06:27:34 | gmann | and on same topic mriedem had query regarding moving the buildling of create_kwargs into helper method than in create() itself which makes it huge | |
| 06:27:56 | gmann | and deprecating the extensions policy which we already done in queens and i will remove them in stein | |
| 06:29:05 | gmann | moving the building of create_kwargs into helper can be discussed in stein as this is late to change them now in Rocky | |
| 06:30:15 | gmann | alex_xu: i am not sure we should open another specless BP for stein or just merge them as it is. anyways Rocky BP needs to be closed anyways. | |
| 06:30:34 | gmann | may be melwitt mriedem can suggest ^^ | |
| 06:32:26 | alex_xu | gmann: yea, that should be a question for them | |
| 06:32:32 | gmann | ok | |
| 06:32:54 | gmann | let's move next | |
| 06:32:55 | gmann | 6. Handling a down cell | |
| 06:33:08 | gmann | #link https://review.openstack.org/#/q/topic:bp/handling-down-cell+(status:open+OR+status:merged) | |
| 06:33:34 | gmann | actually i am not following this one. alex_xu you know if that is target for Rocky i mean can it make it by tomorrow ? | |
| 06:34:29 | alex_xu | gmann: sounds like target to Rocky, at least we have spec for rocky http://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/handling-down-cell.html | |
| 06:34:37 | gmann | yeah | |
| 06:34:41 | alex_xu | gmann: but looks like it starts very late | |
| 06:35:05 | gmann | i saw mriedem comment of doing service list change also in same microversion of server list - #link https://review.openstack.org/#/c/584829/ | |
| 06:35:45 | gmann | yea it is started late like me :) i also did start the extensions work too late due to QA things. | |
| 06:36:03 | gmann | anyways let's see how far it will go tomorrow | |
| 06:36:37 | gmann | that is all Rokcy items | |
| 06:37:00 | gmann | next may be after FF, we can keep eyes on API bugs and Stein specs for review | |
| 06:37:02 | gmann | #link https://review.openstack.org/#/q/project:openstack/nova-specs+status:open+message:%22apiimpact%22 | |
| 06:37:46 | gmann | that's all on priority, anything else to discuss otherwise we move to bug discussion | |
| 06:39:19 | gmann | seems no. let's move then | |
| 06:39:26 | gmann | #topic Bug Triage/Discussion | |
| 06:39:47 | gmann | #link https://etherpad.openstack.org/p/nova-api-weekly-bug-report | |
| 06:40:34 | gmann | curren total open bugs are 68. | |
| 06:40:44 | Kevin_Zheng | omg | |
| 06:41:08 | gmann | honestly saying i was getting less time for bugs due to extensions work but after FF i will give more time on this to burn it down to less | |
| 06:42:00 | gmann | 34 out of them are in-progress | |
| 06:42:20 | gmann | vishakha: you wanted to discuss some bug? | |
| 06:44:27 | vishakha | gmann, yes | |
| 06:44:30 | alex_xu | vishakha: can you reproduce it in master? the bug is reported in newton, and we change quota a lot recently, like we remove the commit, rollback stuff for cellv2 | |
| 06:44:47 | vishakha | gmann, Hi, i've been working on the bug https://bugs.launchpad.net/nova/+bug/1644457 but i found out that key_pair is a special case in which in_use remains always 0 as seen here https://github.com/openstack/nova/blob/master/nova/quota.py#L189 | |
| 06:44:48 | openstack | Launchpad bug 1644457 in OpenStack Compute (nova) "keypair quota error" [Medium,Confirmed] - Assigned to Vishakha Agarwal (vishakha.agarwal) | |
| 06:45:17 | gmann | alex_xu: it seems key_pairs is in special case for in_use always 0 | |
| 06:45:18 | gmann | #link https://github.com/openstack/nova/blob/master/nova/quota.py#L189 | |
| 06:45:39 | vishakha | gmann, I have a doubt when adding a key to instance. It should come in 'in_use' of quota_usage? | |
| 06:48:11 | gmann | vishakha: is it reproducible on master as alex_xu asked | |
| 06:48:42 | vishakha | gmann, yes it is | |
| 06:49:28 | vishakha | alex_xu, the 'in_use' for keypair is coming 0 always | |
| 06:51:40 | alex_xu | interesting... | |
| 06:52:55 | alex_xu | I thought it should call this https://github.com/openstack/nova/blob/master/nova/quota.py#L1244 | |
| 06:55:39 | alex_xu | vishakha: the `nova quota-show --detial` should show 0, but `nova quota-show --detail --user {user_id}` indeed should show the usage as my understand | |
| 06:56:26 | alex_xu | vishakha: I'm not sure whether https://github.com/openstack/nova/blob/master/nova/quota.py#L189 missing a check for the user_id isn't None | |
| 06:57:30 | gmann | yeah, db count id per user - https://github.com/openstack/nova/blob/e019be3724d949b1239c2cc3fbc00f1f69a3477c/nova/db/sqlalchemy/api.py#L3043 | |
| 06:57:48 | gmann | s/id/it | |
| 07:02:30 | vishakha | alex_xu, gmann : I have tried with --user parameter also, still showing 0 in_use | |
| 07:03:10 | gmann | alex_xu: i found some context of doing it intentionally or say as per old behaviour | |