| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-11 | |||
| 06:50:22 | alex_xu | hah | |
| 06:50:36 | gmann | alex_xu: how about you ? | |
| 06:50:37 | alex_xu | I only review one, and not finish yet | |
| 06:50:42 | alex_xu | #link https://review.openstack.org/#/c/486850/6 | |
| 06:50:53 | alex_xu | thinking of we can make this move on | |
| 06:51:17 | gmann | ohk. | |
| 06:51:22 | alex_xu | Kevin_Zheng: it is your patch, I'm still thinking we should use environ variable to build a correct url instead of strip the url | |
| 06:51:30 | alex_xu | strip the url is too hacky | |
| 06:52:08 | Kevin_Zheng | already forgot details about that one :) | |
| 06:52:33 | alex_xu | hah | |
| 06:53:16 | alex_xu | use the bug report example. the req.environ['PATH_INFO'] can get '/v2.0', and the req.environ['SCRIPT_NAME'] can get '/compute'. | |
| 06:55:16 | alex_xu | we should be able to figure which part of url is base url | |
| 06:56:42 | alex_xu | Kevin_Zheng: it is ok we can continue the discussion in the patch, since you need remember the detail | |
| 06:57:00 | Kevin_Zheng | yeah, I have to read it again | |
| 06:57:08 | gmann | yea, i think we can recall on patch and continue | |
| 06:57:43 | gmann | i will do more in-progress review before next office hour | |
| 06:58:19 | gmann | 3 min left | |
| 06:58:29 | gmann | anything on bug discussion? | |
| 06:59:00 | gmann | let's move to open in case of anything else | |
| 06:59:03 | gmann | #topic Open Discussion | |
| 06:59:28 | gmann | any more things to bring up ? | |
| 06:59:59 | Kevin_Zheng | nope | |
| 07:00:15 | gmann | ok let's close the meeting then . | |
| 07:00:28 | gmann | thanks everyone for joining. good to see more people in API discussion :) | |
| 07:00:40 | gmann | #endmeeting | |
| 07:00:42 | openstack | Meeting ended Wed Jul 11 07:00:40 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 07:00:43 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-11-06.00.html | |
| 07:00:44 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-11-06.00.txt | |
| 07:00:45 | openstack | Log: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-11-06.00.log.html | |
| 07:02:31 | alex_xu | gmann: yea, good see a lot of people again :) thanks for leading the meeting | |
| 07:03:41 | gmann | alex_xu: np! | |
| 07:29:23 | gmann | alex_xu: scheduler would not come into pic in case of single host right ? i mean anti-affinity will always pass when only single host. | |
| 07:31:56 | gmann | i mean schd will return same single host always | |
| 07:37:32 | openstackgerrit | huanhongda proposed openstack/nova-specs master: Add .idea folder to .gitignore https://review.openstack.org/581611 | |
| 07:40:39 | gmann | alex_xu: currently i can boot 2 server on same host in case of anti-affinity policy on my single node env | |
| 07:42:04 | alex_xu | gmann: that is bug? | |
| 07:43:11 | gmann | alex_xu: donot know, i feel that should fail like it does when i use different_host filter on my single node env | |
| 07:43:56 | alex_xu | gmann: I also think so | |
| 07:46:49 | gmann | bauzas: ^^ | |
| 07:50:34 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534 | |
| 07:58:31 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nit in afc7650e64753ab7687ae2c4f2714d4bb78a4e5a https://review.openstack.org/581616 | |
| 08:16:43 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542 | |
| 08:26:12 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562 | |
| 08:59:10 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261 | |
| 09:03:06 | openstack | Launchpad bug 1746863 in OpenStack Compute (nova) "scheduler affinity doesn't work with multiple cells" [High,In progress] - Assigned to melanie witt (melwitt) | |
| 09:03:06 | gibi | gmann: I think you see this https://bugs.launchpad.net/nova/+bug/1746863 | |
| 09:05:37 | gmann | gibi: but i have only single compute node no multi cell. | |
| 09:06:06 | gibi | gmann: do you have superconductor? | |
| 09:06:52 | gibi | gmann: for example devstack by default starts with super conductor and cell1 conductor | |
| 09:10:50 | gmann | gibi: yeah its superconductor | |
| 09:12:59 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559034 | |
| 09:13:09 | openstackgerrit | huanhongda proposed openstack/nova-specs master: Return "deleted" time of instance when vm_state is SOFT-DELETED https://review.openstack.org/581638 | |
| 09:13:22 | lyarwood | sahid: ^ can you take a look at that rebase, I can't seem to run tox against stable/ocata on F28 at the moment. | |
| 09:13:44 | gmann | gibi: thanks. | |
| 09:16:47 | gibi | gmann: I have a tempest modification that reproduces the problem https://review.openstack.org/#/c/581349 | |
| 09:20:18 | gibi | gmann: regardin that change I acutally wondering how can I test in tempest that booting in an affinity group goes to NoValid host due to a) the compute having a memeber from the group is full b) the compute is unavailable | |
| 09:20:53 | gibi | gmann: I think I cannot make a compute full or disable in tempest without breaking otuer parallel tempest cases | |
| 09:21:05 | gibi | s/otuer/other/ | |
| 09:25:17 | gmann | gibi: yeah, it is difficult to generate that scenario as we cannot disable compute service or full | |
| 09:25:44 | gmann | i was thinking with help of quota things but that would generate the scenario you want | |
| 09:31:05 | gibi | gmann: but quota is not per compute so it wont remove a single compute from scheduling | |
| 09:31:38 | gmann | yeah | |
| 09:31:46 | gibi | gmann: another way would be to create a flavor that is bigger than the remaining space on the compute the affinity group members are | |
| 09:32:15 | gmann | but how we would know how big flavor needs to be | |
| 09:32:21 | gibi | gmann: are the multi node computes has fixed size or it depends on the node they are running? | |
| 09:32:52 | gmann | gibi: it depends on node they are running. | |
| 09:33:01 | gmann | gibi: and it can be any size on non-gate env | |
| 09:33:29 | gibi | gmann: than I would have to ask nova/placement about the available resources to create a big enough flavor | |
| 09:35:43 | gibi | gmann: ahh it is not good | |
| 09:36:30 | gibi | gmann: I would need a flavor that requests more than available on the group's compute but still fits on the other compute (in multinode) | |
| 09:37:03 | gmann | gibi: another idea is to keep booting the server with same_host hint on anti-affinity compute till No-host | |
| 09:37:39 | gmann | but it can be very long scenario tests and need exact enabled_filter on setup | |
| 09:38:08 | gibi | gmann: and that could potentially interfere with test running in paralell | |
| 09:39:02 | gmann | yeah. not good | |
| 09:42:38 | gibi | gmann: OK, I conclude that this cannot be covered in tempest. So we need to cover that in nova functional | |
| 09:43:24 | gmann | gibi: yeah, functional tests is the way | |
| 09:43:39 | gmann | gibi: but for this bug, your patch is ok right? -https://bugs.launchpad.net/nova/+bug/1770434 | |
| 09:43:40 | openstack | Launchpad bug 1746863 in OpenStack Compute (nova) "duplicate for #1770434 scheduler affinity doesn't work with multiple cells" [High,In progress] - Assigned to melanie witt (melwitt) | |
| 09:44:16 | gmann | gibi: i mean doing multiple request not single | |
| 09:44:44 | gibi | gmann: yeah, that seems doable. I will push a new PS soon fixing the TODO and mriedem's comments | |
| 09:45:11 | gibi | gmann: it won't cover affinity + NoValid host case but that I will describe in the commit message | |
| 09:45:16 | gmann | gibi: yeah, because at least covering that scenario is good | |
| 09:45:23 | gmann | yeah | |
| 09:45:38 | gibi | gmann: thanks | |
| 09:45:50 | gmann | gibi: i will check that tomorrow. added in my list | |
| 09:46:00 | gibi | gmann: thank you | |
| 09:46:04 | gmann | time to go home, have good lunch | |
| 09:46:12 | gibi | have a good afternoon | |
| 09:53:15 | yikun | gibi: hi | |
| 09:53:47 | yikun | could you take a look on | |
| 09:53:52 | yikun | https://review.openstack.org/#/c/563401/31/nova/tests/unit/compute/test_compute_utils.py@1179 | |
| 09:55:23 | yikun | should we change {"max_server_per_host": "3"} to {"max_server_per_host": 3} keep same use user passed? | |
| 09:56:38 | yikun | we do conversion thing in group obj, see https://review.openstack.org/#/c/563375/39/nova/objects/instance_group.py@140 | |
| 10:00:28 | yikun | I'm not sure should we keep same with it. | |
| 10:00:44 | yikun | and here is some message history: | |
| 10:00:45 | yikun | http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-07-11.log.html#t2018-07-11T02:18:53 | |
| 10:00:59 | yikun | ^ gibi | |
| 10:19:27 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform metrics.update notification https://review.openstack.org/580567 | |
| 10:28:05 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559034 | |
| 10:28:55 | openstackgerrit | huanhongda proposed openstack/nova master: Microversion 2.64 - Add "deleted" time in GET server response https://review.openstack.org/574159 | |