| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-11 | |||
| 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 | gibi | gmann: I think you see this https://bugs.launchpad.net/nova/+bug/1746863 | |
| 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: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 | |
| 10:44:03 | openstackgerrit | Van Hung Pham proposed openstack/nova master: convert py35 jobs to py3 https://review.openstack.org/581663 | |
| 10:46:15 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update queued-for-delete from the ComputeAPI during deletion. https://review.openstack.org/566813 | |
| 10:48:01 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update queued-for-delete from the ComputeAPI during deletion/restoration https://review.openstack.org/566813 | |
| 10:55:53 | gibi | yikun: it is on my list | |
| 10:57:31 | cnf | hi, anyone know what would cause the creation of a new instance to give me "Block Device Mapping is Invalid" ? | |
| 10:57:42 | cnf | on -cinder, someone said it looks like a nova issue | |
| 11:11:52 | yikun | gibi: ok, thanks :) | |
| 11:28:41 | openstackgerrit | huanhongda proposed openstack/nova master: Microversion 2.64 - Add "deleted" time in GET server response https://review.openstack.org/574159 | |
| 12:26:12 | gibi | yikun: did I expressed my opininon in every open issue in https://review.openstack.org/#/c/563401/28/nova/notifications/objects/server_group.py@41 or did I missed something? | |
| 12:27:08 | gibi | yikun: ahh the conversion thing | |
| 12:27:12 | gibi | yikun: i missed that | |
| 12:27:22 | yikun | yep, : ) | |
| 12:27:33 | yikun | conversion | |
| 12:27:46 | gibi | yikun: would it be hard to make the int conversion in the notification body as well? | |
| 12:28:30 | yikun | No, just need put some duplicate code into notification obj. | |
| 12:29:10 | yikun | like https://review.openstack.org/#/c/563375/39/nova/objects/instance_group.py@140 | |
| 12:30:36 | gibi | yikun: I see. We can only store a DictOfString even if your values will be in different types in the future | |
| 12:31:19 | gibi | yikun: can we call the rules property from the notification payload generation? let me look a bit deeper in the code... | |
| 12:32:11 | gibi | yikun: I guess we call the rules property but then we store the result in a DictOfString in the payload as well | |
| 12:32:23 | gibi | yikun: which converts it back to string | |
| 12:32:57 | yikun | yes, so if we don't do convert, we will get back a "3" | |
| 12:32:58 | gibi | yikun: as we don't have a Dict type with variable value type | |
| 12:33:11 | gibi | yikun: we cannot define a proper field in the payload class | |
| 12:33:44 | gibi | yikun: and the payload class is something we think about as a contract for the consumers | |
| 12:33:49 | yikun | yes, at least I can't see a DictField type in ovo. :) | |