| *** harlowja has joined #openstack-nova | 00:03 | |
| *** yangyapeng has quit IRC | 00:06 | |
| *** annegentle has joined #openstack-nova | 00:07 | |
| *** yangyapeng has joined #openstack-nova | 00:07 | |
| *** harlowja has quit IRC | 00:09 | |
| *** yangyapeng has quit IRC | 00:11 | |
| *** moshele has joined #openstack-nova | 00:13 | |
| *** edleafe has quit IRC | 00:14 | |
| *** harlowja has joined #openstack-nova | 00:14 | |
| *** edleafe has joined #openstack-nova | 00:16 | |
| *** yangyapeng has joined #openstack-nova | 00:18 | |
| *** edmondsw has joined #openstack-nova | 00:19 | |
| *** thorst has joined #openstack-nova | 00:20 | |
| *** yangyapeng has quit IRC | 00:22 | |
| *** thorst has quit IRC | 00:23 | |
| *** Swami has quit IRC | 00:24 | |
| *** jaypipes has quit IRC | 00:24 | |
| *** edmondsw has quit IRC | 00:24 | |
| *** yassine has joined #openstack-nova | 00:32 | |
| *** thorst has joined #openstack-nova | 00:32 | |
| mriedem | gmann: these don't require microversion changes, correct? | 00:33 |
|---|---|---|
| openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling error availiable change to available https://review.openstack.org/501043 | 00:34 |
| *** esberglu has joined #openstack-nova | 00:34 | |
| gmann | mriedem, yes. only thing i want to confirm from alex_xu about restricting the int convertible string as limit like '1' it used to be valid and converted by utils previously and now it will be 400 | 00:34 |
| gmann | i think we discussed it in original spec but i do not remember the consensus . | 00:35 |
| mriedem | gmann: if a microversion bump is required then i think we need a spec, | 00:36 |
| mriedem | otherwise i'm ok with specless | 00:36 |
| gmann | mriedem, yea, if so it need spec. we will discuss it in today meeting for all cases and ll update you | 00:37 |
| mriedem | thanks | 00:37 |
| openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling mistake availiable change to available https://review.openstack.org/501045 | 00:38 |
| *** esberglu has quit IRC | 00:38 | |
| *** rajathagasthya has quit IRC | 00:40 | |
| openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling mistake prefered change to preferred https://review.openstack.org/501046 | 00:41 |
| *** baoli has joined #openstack-nova | 00:41 | |
| *** hshiina has joined #openstack-nova | 00:42 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: WIP spelling error availiable change to available https://review.openstack.org/501043 | 00:43 |
| *** gouthamr has quit IRC | 00:45 | |
| *** baoli_ has joined #openstack-nova | 00:45 | |
| *** baoli has quit IRC | 00:46 | |
| openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling mistake intergration change to integration https://review.openstack.org/501047 | 00:47 |
| alex_xu | mriedem: gmann it needn't microversion, I think just just add query params validation for the exist API and keep it same behavour for the API | 00:47 |
| gmann | alex_xu, but we are doing non negative integer for limit - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/keypairs.py#L109 | 00:49 |
| *** dave-mccowan has quit IRC | 00:49 | |
| *** gouthamr has joined #openstack-nova | 00:49 | |
| gmann | https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/validation/parameter_types.py#L438 | 00:49 |
| gmann | alex_xu, this is only case change the behavior for 'int' 200 -> 400 | 00:50 |
| alex_xu | gmann: that isn't backward-compatible? | 00:50 |
| alex_xu | gmann: I remember there are some validation in the python code for the limits, that should check the non-negative value also | 00:50 |
| gmann | alex_xu, humm | 00:51 |
| gmann | alex_xu, i found the flow like this | 00:51 |
| gmann | 1. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/openstack/common.py#L189 | 00:51 |
| gmann | 2. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/openstack/common.py#L202 | 00:51 |
| *** jichen has joined #openstack-nova | 00:51 | |
| gmann | 3. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/utils.py#L883 | 00:52 |
| gmann | 3rd one convert the '1' to 1 | 00:52 |
| *** zhurong has joined #openstack-nova | 00:53 | |
| *** TuanLA has joined #openstack-nova | 00:54 | |
| alex_xu | gmann: there is parameter 'min_value', that is limited to non-negative? | 00:54 |
| * alex_xu rush to next meeting... | 00:55 | |
| *** mriedem has quit IRC | 00:55 | |
| *** harlowja has quit IRC | 00:57 | |
| *** dixiaoli has joined #openstack-nova | 00:57 | |
| *** harlowja has joined #openstack-nova | 01:01 | |
| openstackgerrit | Merged openstack/nova master: tests: Remove useless test https://review.openstack.org/483031 | 01:02 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certificates to REST API https://review.openstack.org/486204 | 01:02 |
| *** phuongnh has joined #openstack-nova | 01:05 | |
| *** Apoorva_ has joined #openstack-nova | 01:05 | |
| *** nicolasbock has quit IRC | 01:06 | |
| openstackgerrit | Merged openstack/nova master: Cleanup allocations on invalid dest node during live migration https://review.openstack.org/498861 | 01:06 |
| *** thorst has quit IRC | 01:06 | |
| *** rajathagasthya has joined #openstack-nova | 01:07 | |
| alex_xu | gmann: I may not clear the question | 01:08 |
| alex_xu | the value of '1' will get 400? | 01:08 |
| openstackgerrit | Merged openstack/nova master: iso8601.is8601.Utc No Longer Exists https://review.openstack.org/498287 | 01:09 |
| *** Apoorva has quit IRC | 01:09 | |
| *** Apoorva_ has quit IRC | 01:10 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 01:14 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 01:15 |
| *** yamamoto has joined #openstack-nova | 01:15 | |
| *** yangyapeng has joined #openstack-nova | 01:15 | |
| cdent | dansmith: some of the jiggery pokery in that ^ is starting to get a bit wack | 01:15 |
| openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param https://review.openstack.org/500396 | 01:15 |
| *** nicolasbock has joined #openstack-nova | 01:21 | |
| *** hongbin has joined #openstack-nova | 01:22 | |
| *** cdent has quit IRC | 01:23 | |
| *** thorst has joined #openstack-nova | 01:24 | |
| *** scottda_ has quit IRC | 01:24 | |
| *** thorst has quit IRC | 01:25 | |
| openstackgerrit | jichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | 01:38 |
| *** thorst has joined #openstack-nova | 01:39 | |
| *** thorst has quit IRC | 01:39 | |
| *** markvoelker has joined #openstack-nova | 01:43 | |
| *** baoli_ has quit IRC | 01:44 | |
| *** baoli has joined #openstack-nova | 01:45 | |
| *** vikrant has joined #openstack-nova | 01:48 | |
| *** itlinux has joined #openstack-nova | 01:54 | |
| *** edmondsw has joined #openstack-nova | 01:58 | |
| *** edmondsw has quit IRC | 02:02 | |
| openstackgerrit | jichenjc proposed openstack/nova master: conf: Remove quota_networks options https://review.openstack.org/501068 | 02:05 |
| *** thorst has joined #openstack-nova | 02:07 | |
| *** thorst has quit IRC | 02:07 | |
| *** thorst has joined #openstack-nova | 02:07 | |
| *** vikrant has quit IRC | 02:08 | |
| *** thorst has quit IRC | 02:12 | |
| *** yamahata has quit IRC | 02:14 | |
| *** xinliang has quit IRC | 02:17 | |
| *** markvoelker has quit IRC | 02:18 | |
| openstackgerrit | Chenghui Yu proposed openstack/nova master: Remove usage of kwarg retry_on_request in API https://review.openstack.org/501073 | 02:21 |
| *** esberglu has joined #openstack-nova | 02:22 | |
| *** coreywright has quit IRC | 02:26 | |
| *** esberglu has quit IRC | 02:26 | |
| *** gbarros has joined #openstack-nova | 02:27 | |
| *** baoli has quit IRC | 02:28 | |
| *** baoli has joined #openstack-nova | 02:29 | |
| *** xinliang has joined #openstack-nova | 02:30 | |
| *** thorst has joined #openstack-nova | 02:34 | |
| *** thorst has quit IRC | 02:34 | |
| *** baoli has quit IRC | 02:40 | |
| *** coreywright has joined #openstack-nova | 02:44 | |
| *** liusheng has quit IRC | 02:51 | |
| *** jichen has quit IRC | 02:52 | |
| *** liusheng has joined #openstack-nova | 02:52 | |
| *** zhurong has quit IRC | 02:58 | |
| *** baoli has joined #openstack-nova | 02:59 | |
| *** penick has joined #openstack-nova | 03:06 | |
| *** abhi89 has joined #openstack-nova | 03:10 | |
| *** gvrangan has joined #openstack-nova | 03:10 | |
| *** penick has quit IRC | 03:10 | |
| *** yangyape_ has joined #openstack-nova | 03:11 | |
| *** baoli has quit IRC | 03:11 | |
| *** moshele has quit IRC | 03:11 | |
| *** yangyap__ has joined #openstack-nova | 03:14 | |
| *** yangyape_ has quit IRC | 03:14 | |
| *** yangyapeng has quit IRC | 03:14 | |
| *** jichen has joined #openstack-nova | 03:15 | |
| *** markvoelker has joined #openstack-nova | 03:15 | |
| *** yangyap__ has quit IRC | 03:19 | |
| *** yangyapeng has joined #openstack-nova | 03:19 | |
| *** nicolasbock has quit IRC | 03:20 | |
| *** yamamoto_ has joined #openstack-nova | 03:25 | |
| *** links has joined #openstack-nova | 03:28 | |
| *** yamamoto has quit IRC | 03:28 | |
| *** yangyape_ has joined #openstack-nova | 03:30 | |
| *** yangyapeng has quit IRC | 03:30 | |
| *** hshiina has quit IRC | 03:34 | |
| *** annegentle has quit IRC | 03:37 | |
| *** takashin has joined #openstack-nova | 03:44 | |
| *** gouthamr has quit IRC | 03:46 | |
| *** edmondsw has joined #openstack-nova | 03:46 | |
| *** udesale has joined #openstack-nova | 03:46 | |
| *** markvoelker has quit IRC | 03:48 | |
| *** rajathagasthya has quit IRC | 03:48 | |
| *** edmondsw has quit IRC | 03:51 | |
| *** adisky__ has quit IRC | 03:55 | |
| *** gvrangan has quit IRC | 03:56 | |
| *** kiennt has joined #openstack-nova | 03:58 | |
| *** hongbin has quit IRC | 03:58 | |
| *** hshiina has joined #openstack-nova | 03:59 | |
| *** gyee has quit IRC | 04:00 | |
| *** gbarros has quit IRC | 04:02 | |
| *** zhurong has joined #openstack-nova | 04:08 | |
| *** tbh_ has joined #openstack-nova | 04:09 | |
| *** esberglu has joined #openstack-nova | 04:10 | |
| *** esberglu has quit IRC | 04:14 | |
| *** gcb has quit IRC | 04:19 | |
| *** trungnv has quit IRC | 04:19 | |
| *** gcb has joined #openstack-nova | 04:21 | |
| *** thorst has joined #openstack-nova | 04:26 | |
| *** psachin has joined #openstack-nova | 04:31 | |
| *** thorst has quit IRC | 04:31 | |
| *** trungnv has joined #openstack-nova | 04:37 | |
| *** trungnv has quit IRC | 04:37 | |
| *** trinaths has joined #openstack-nova | 04:42 | |
| openstackgerrit | Pooja Jadhav proposed openstack/nova master: Fix ValueError if invalid max_rows passed to db purge https://review.openstack.org/500771 | 04:43 |
| *** armax has quit IRC | 04:44 | |
| *** markvoelker has joined #openstack-nova | 04:45 | |
| *** gibi has quit IRC | 04:48 | |
| *** gvrangan has joined #openstack-nova | 04:48 | |
| *** zxy has quit IRC | 04:49 | |
| *** zxy has joined #openstack-nova | 04:49 | |
| *** moshele has joined #openstack-nova | 05:00 | |
| itlinux | there are 4 compute nodes.. and those images are cirros.. | 05:03 |
| *** diga has joined #openstack-nova | 05:04 | |
| *** gvrangan has quit IRC | 05:04 | |
| *** claudiub|2 has joined #openstack-nova | 05:06 | |
| *** yamamoto_ has quit IRC | 05:07 | |
| *** yamamoto has joined #openstack-nova | 05:09 | |
| itlinux | hello team.. | 05:11 |
| itlinux | looks like the scheduler gets stuck when I try to spin up few machines at one time.. | 05:11 |
| itlinux | like 4 or 5 | 05:11 |
| itlinux | I have 4 compute nodes and the images are cirros | 05:11 |
| itlinux | I checked the hypervisors are up | 05:11 |
| *** zxy has quit IRC | 05:14 | |
| *** zxy has joined #openstack-nova | 05:14 | |
| *** markvoelker has quit IRC | 05:18 | |
| *** shan has joined #openstack-nova | 05:19 | |
| *** kairo_ has joined #openstack-nova | 05:20 | |
| *** kairo has quit IRC | 05:22 | |
| *** zxy has quit IRC | 05:22 | |
| *** zxy has joined #openstack-nova | 05:22 | |
| *** moshele has quit IRC | 05:23 | |
| *** Tom_ has joined #openstack-nova | 05:24 | |
| *** pcaruana has joined #openstack-nova | 05:27 | |
| *** gvrangan has joined #openstack-nova | 05:33 | |
| *** yamahata has joined #openstack-nova | 05:39 | |
| *** ratailor has joined #openstack-nova | 05:41 | |
| *** liverpooler has joined #openstack-nova | 05:46 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Update doc to indicate nova-network deprecated https://review.openstack.org/500654 | 05:47 |
| *** yamahata has quit IRC | 05:48 | |
| openstackgerrit | jichenjc proposed openstack/nova master: conf: Remove quota_networks options https://review.openstack.org/501068 | 05:49 |
| gmann | alex_xu, i mean, if we pass any int as string in limit like '1' not 1, it will be 400 in current case of keypair. because of schema | 05:49 |
| itlinux | hello nova team.. anyone has a tweak since when I spin up some vms the compute that gets like two vms takes forever.. | 05:49 |
| itlinux | like 100sec for a cirros image.. | 05:49 |
| itlinux | sounds a little too much! | 05:50 |
| *** wlfightup has quit IRC | 05:53 | |
| *** yamahata has joined #openstack-nova | 05:53 | |
| *** udesale__ has joined #openstack-nova | 05:53 | |
| gmann | alex_xu, but when i ran it without schema it is also same(400). cannot this convert '1' -? 1 - https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/utils.py#L883 | 05:55 |
| *** udesale has quit IRC | 05:56 | |
| *** wlfightup has joined #openstack-nova | 05:56 | |
| *** yangyapeng has joined #openstack-nova | 05:57 | |
| *** yangyape_ has quit IRC | 05:57 | |
| *** esberglu has joined #openstack-nova | 05:58 | |
| *** esberglu has quit IRC | 06:03 | |
| *** yamahata has quit IRC | 06:03 | |
| openstackgerrit | jichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | 06:04 |
| *** dixiaoli has quit IRC | 06:05 | |
| *** dixiaoli has joined #openstack-nova | 06:05 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance doc for nova services https://review.openstack.org/499536 | 06:09 |
| openstackgerrit | jichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError https://review.openstack.org/469779 | 06:09 |
| alex_xu | gmann: that schema should work with '1', not sure I missed something | 06:11 |
| gmann | alex_xu, oh, 1 sec | 06:13 |
| *** yamahata has joined #openstack-nova | 06:17 | |
| *** moshele has joined #openstack-nova | 06:18 | |
| openstackgerrit | Merged openstack/nova master: Add uuid to migration object and migrate-on-load https://review.openstack.org/496934 | 06:19 |
| *** thorst has joined #openstack-nova | 06:27 | |
| *** udesale__ has quit IRC | 06:28 | |
| *** links has quit IRC | 06:29 | |
| *** shan has quit IRC | 06:29 | |
| *** ratailor has quit IRC | 06:29 | |
| *** psachin has quit IRC | 06:29 | |
| *** yingjun has joined #openstack-nova | 06:30 | |
| *** thorst has quit IRC | 06:32 | |
| *** yamamoto has quit IRC | 06:34 | |
| *** yangyapeng has quit IRC | 06:35 | |
| *** yamamoto has joined #openstack-nova | 06:35 | |
| *** yangyapeng has joined #openstack-nova | 06:35 | |
| *** sridharg has joined #openstack-nova | 06:36 | |
| *** adisky__ has joined #openstack-nova | 06:38 | |
| openstackgerrit | Merged openstack/nova master: Add uuid online migration for migrations https://review.openstack.org/498589 | 06:40 |
| *** ssmith has quit IRC | 06:43 | |
| *** sahid has joined #openstack-nova | 06:51 | |
| *** udesale has joined #openstack-nova | 06:55 | |
| *** links has joined #openstack-nova | 06:55 | |
| *** ratailor has joined #openstack-nova | 06:55 | |
| *** rcernin has joined #openstack-nova | 06:57 | |
| openstackgerrit | Gao Zexu proposed openstack/os-traits master: Update the documentation urls https://review.openstack.org/501122 | 06:57 |
| *** udesale__ has joined #openstack-nova | 06:57 | |
| *** udesale__ has quit IRC | 06:58 | |
| *** udesale__ has joined #openstack-nova | 06:59 | |
| *** psachin has joined #openstack-nova | 06:59 | |
| *** gibi has joined #openstack-nova | 07:00 | |
| *** udesale has quit IRC | 07:00 | |
| *** andreas_s has joined #openstack-nova | 07:01 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for used_limits function https://review.openstack.org/499091 | 07:02 |
| *** yingjun has quit IRC | 07:06 | |
| *** udesale__ has quit IRC | 07:09 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP:Remove quota_networks and enable_network_quota https://review.openstack.org/501068 | 07:09 |
| *** udesale__ has joined #openstack-nova | 07:10 | |
| jichen | gmann: hi, in https://review.openstack.org/#/c/501057/2/nova/tests/unit/api/openstack/compute/test_config_drive.py, you mentioned :i like to have some uuid as project id here instead of removing. | 07:11 |
| jichen | I think in v2.18, we have: | 07:11 |
| jichen | 2.18 Establishes a set of routes that makes project_id an optional construct in v2.1. | 07:12 |
| jichen | why we still need add project id in the test ? | 07:12 |
| *** udesale__ has quit IRC | 07:15 | |
| *** udesale has joined #openstack-nova | 07:15 | |
| *** markvoelker has joined #openstack-nova | 07:15 | |
| gmann | jichen, yea, but we do have tests to test both | 07:21 |
| *** jpena|off is now known as jpena | 07:22 | |
| gmann | jichen, i do not have string opinion on this but i we can discuss whether it is ok to remove the project_id from tests | 07:22 |
| *** edmondsw has joined #openstack-nova | 07:22 | |
| gmann | jichen, its unit test only right? does it matter anywhere. i mean url is really dummy things in unit tests right | 07:23 |
| *** kairo_ is now known as kairo | 07:23 | |
| *** takashin has quit IRC | 07:23 | |
| *** kairo has quit IRC | 07:23 | |
| *** kairo has joined #openstack-nova | 07:23 | |
| *** edmondsw has quit IRC | 07:26 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202 | 07:28 |
| *** tesseract has joined #openstack-nova | 07:30 | |
| *** ralonsoh has joined #openstack-nova | 07:32 | |
| jichen | gmann: yeah, we should have test for both..I will see whether it's possible to at least include some project id into test | 07:34 |
| jichen | gmann: and yes, it's in unit test , but PS#1 shows a lot of error if I use /v2.1/fake/xxx, so I tried to move to /v2.1/xxx and it works locally | 07:35 |
| jichen | so I think at least in some cases it matters :) | 07:35 |
| *** mdrabe has quit IRC | 07:36 | |
| gmann | jichen, can we use some dummy UUID in this change and later we decide whether to remove project-id in unit tests or not as whole | 07:37 |
| openstackgerrit | sahid proposed openstack/os-vif master: ovs-hybrid: should permanently keep MAC entries https://review.openstack.org/501132 | 07:37 |
| jichen | ok, that's doable, I think you are saying is /v2.1/0000-0000-00000-00000/xxx (a uuid format) instead /v2.1/fake/xxx ,right? | 07:38 |
| gmann | something like '11111111-1111- ' | 07:38 |
| gmann | jichen, yea | 07:38 |
| jichen | gmann: ok, let me try it, thanks for comments | 07:39 |
| gmann | jichen, thanks | 07:39 |
| *** hieulq has quit IRC | 07:40 | |
| *** TuanLA has quit IRC | 07:40 | |
| *** TuanLA has joined #openstack-nova | 07:41 | |
| *** hieulq has joined #openstack-nova | 07:41 | |
| *** claudiub|2 has quit IRC | 07:43 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: slowly live-migration to ensure network is ready https://review.openstack.org/497457 | 07:43 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add method to configure migration speed https://review.openstack.org/497456 | 07:43 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/497455 | 07:44 |
| *** esberglu has joined #openstack-nova | 07:46 | |
| *** markvoelker has quit IRC | 07:49 | |
| openstackgerrit | Merged openstack/nova master: Refactor LiveMigrationTask._find_destination https://review.openstack.org/498874 | 07:50 |
| *** esberglu has quit IRC | 07:50 | |
| openstackgerrit | Merged openstack/nova master: Add missing unit tests for FilterScheduler._get_all_host_states https://review.openstack.org/496803 | 07:54 |
| *** hoonetorg has quit IRC | 07:58 | |
| *** udesale has quit IRC | 07:58 | |
| *** udesale has joined #openstack-nova | 07:59 | |
| *** hferenc has joined #openstack-nova | 08:00 | |
| *** edmondsw has joined #openstack-nova | 08:03 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Add functional for live migrate delete https://review.openstack.org/499583 | 08:08 |
| *** alexpilotti has quit IRC | 08:14 | |
| *** ociuhandu_ has quit IRC | 08:14 | |
| *** hoonetorg has joined #openstack-nova | 08:14 | |
| *** phuongnh has quit IRC | 08:15 | |
| *** phuongnh has joined #openstack-nova | 08:16 | |
| *** yangyapeng has quit IRC | 08:19 | |
| *** yangyapeng has joined #openstack-nova | 08:20 | |
| *** gcb has quit IRC | 08:23 | |
| *** phuongnh has quit IRC | 08:23 | |
| *** phuongnh has joined #openstack-nova | 08:24 | |
| *** liverpooler has quit IRC | 08:25 | |
| *** jaosorior has quit IRC | 08:27 | |
| *** lucas-afk is now known as lucasagomes | 08:28 | |
| *** thorst has joined #openstack-nova | 08:28 | |
| *** thorst has quit IRC | 08:33 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm https://review.openstack.org/465160 | 08:37 |
| *** yamahata has quit IRC | 08:38 | |
| *** suresh12 has quit IRC | 08:41 | |
| *** markvoelker has joined #openstack-nova | 08:46 | |
| *** alexchadin has joined #openstack-nova | 08:47 | |
| *** derekh has joined #openstack-nova | 08:51 | |
| *** alexpilotti has joined #openstack-nova | 08:54 | |
| *** moshele has quit IRC | 08:54 | |
| *** tbh_ has quit IRC | 08:58 | |
| *** moshele has joined #openstack-nova | 09:03 | |
| *** hferenc has quit IRC | 09:05 | |
| *** ociuhandu has joined #openstack-nova | 09:08 | |
| *** sambetts|afk is now known as sambetts | 09:10 | |
| *** lpetrut has joined #openstack-nova | 09:13 | |
| *** rabel has joined #openstack-nova | 09:14 | |
| *** yangyape_ has joined #openstack-nova | 09:15 | |
| *** hshiina has quit IRC | 09:15 | |
| *** ociuhandu has quit IRC | 09:18 | |
| *** openstackgerrit has quit IRC | 09:18 | |
| *** yangyapeng has quit IRC | 09:18 | |
| *** yangyape_ has quit IRC | 09:19 | |
| *** yangyapeng has joined #openstack-nova | 09:19 | |
| *** markvoelker has quit IRC | 09:19 | |
| *** ociuhandu has joined #openstack-nova | 09:20 | |
| *** adisky__ has quit IRC | 09:32 | |
| *** yamamoto has quit IRC | 09:40 | |
| *** jichen has quit IRC | 09:41 | |
| stephenfin | bauzas: Is my point about not using 'prelude' in reno notes correct here? https://review.openstack.org/#/c/459166/ | 09:41 |
| *** gvrangan has quit IRC | 09:42 | |
| *** alexchadin has quit IRC | 09:42 | |
| bauzas | stephenfin: looking | 09:42 |
| bauzas | stephenfin: that's correct | 09:43 |
| bauzas | stephenfin: I did a bit of writing in the doc about that | 09:43 |
| bauzas | sec, finding it | 09:43 |
| bauzas | (that new docs rearrangement is making me finding my kids :) ) | 09:43 |
| bauzas | stephenfin: https://docs.openstack.org/nova/latest/contributor/releasenotes.html | 09:44 |
| stephenfin | bauzas: Perfect. I'll leave a note for mikal saying as much. Ta :) | 09:47 |
| bauzas | stephenfin: just did | 09:47 |
| bauzas | honestly, that privsep effort is important, so I'll see how I can help with reviews | 09:48 |
| stephenfin | Yup, I started on that yesterday. It's getting pretty big and has been there for a while now | 09:49 |
| *** dixiaoli_ has joined #openstack-nova | 09:51 | |
| *** dixiaoli has quit IRC | 09:52 | |
| *** ociuhandu_ has joined #openstack-nova | 09:52 | |
| bauzas | good opportunity then for understanding how privsep works :) | 09:53 |
| bauzas | I barely understand it, but those changes will help | 09:53 |
| *** sdague has joined #openstack-nova | 09:53 | |
| *** ociuhandu has quit IRC | 09:54 | |
| *** yangyapeng has quit IRC | 09:56 | |
| *** yangyapeng has joined #openstack-nova | 09:56 | |
| *** claudiub|2 has joined #openstack-nova | 09:57 | |
| *** dixiaoli_ has quit IRC | 09:57 | |
| *** nicolasbock has joined #openstack-nova | 10:02 | |
| *** dixiaoli has joined #openstack-nova | 10:02 | |
| *** diga has quit IRC | 10:06 | |
| *** nicolasbock has quit IRC | 10:07 | |
| *** openstackgerrit has joined #openstack-nova | 10:08 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082 | 10:08 |
| *** yamamoto has joined #openstack-nova | 10:11 | |
| *** abhi89 has quit IRC | 10:11 | |
| *** edmondsw has quit IRC | 10:13 | |
| *** diga has joined #openstack-nova | 10:13 | |
| *** jaosorior has joined #openstack-nova | 10:14 | |
| *** yamamoto has quit IRC | 10:16 | |
| *** markvoelker has joined #openstack-nova | 10:17 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082 | 10:17 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add ``HostPortProfileInfo`` class https://review.openstack.org/441590 | 10:17 |
| *** ociuhandu_ has quit IRC | 10:18 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: [WIP] Add tests for 'limit' filter as string for GET keypair API https://review.openstack.org/501202 | 10:23 |
| *** yangyapeng has quit IRC | 10:25 | |
| *** yangyapeng has joined #openstack-nova | 10:25 | |
| *** thorst has joined #openstack-nova | 10:29 | |
| *** yangyapeng has quit IRC | 10:29 | |
| openstackgerrit | sahid proposed openstack/nova master: objects: add new field MASK for emulator threads policy https://review.openstack.org/501205 | 10:31 |
| openstackgerrit | sahid proposed openstack/nova master: objects: add cpu_emulator_threads_mask field https://review.openstack.org/501206 | 10:31 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: add method to get emulator threads mask https://review.openstack.org/501207 | 10:31 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: handle/check emulator threads mask constraints https://review.openstack.org/501208 | 10:31 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: add method to compute emulator threads placement https://review.openstack.org/501209 | 10:31 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure emulator threads of a guest based on mask https://review.openstack.org/501210 | 10:31 |
| *** yamamoto has joined #openstack-nova | 10:32 | |
| *** thorst has quit IRC | 10:34 | |
| *** nicolasbock has joined #openstack-nova | 10:36 | |
| *** dixiaoli has quit IRC | 10:36 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: [DNM] Fir testing only https://review.openstack.org/501211 | 10:37 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: [DNM] For testing only https://review.openstack.org/501211 | 10:38 |
| *** gszasz has joined #openstack-nova | 10:40 | |
| *** kiennt has quit IRC | 10:41 | |
| *** suresh12 has joined #openstack-nova | 10:42 | |
| *** tbachman has quit IRC | 10:42 | |
| *** Tom_ has quit IRC | 10:43 | |
| *** tbachman has joined #openstack-nova | 10:43 | |
| *** Tom has joined #openstack-nova | 10:43 | |
| *** suresh12 has quit IRC | 10:47 | |
| *** tbachman has quit IRC | 10:47 | |
| *** Tom has quit IRC | 10:48 | |
| *** gaurangt has quit IRC | 10:48 | |
| *** yamamoto has quit IRC | 10:48 | |
| *** markvoelker has quit IRC | 10:49 | |
| *** udesale has quit IRC | 10:50 | |
| openstackgerrit | wanghongtaozz proposed openstack/nova master: spelling mistake https://review.openstack.org/501215 | 10:50 |
| *** gaurangt has joined #openstack-nova | 10:52 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 10:52 |
| *** liverpooler has joined #openstack-nova | 10:52 | |
| *** zhurong has quit IRC | 10:54 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 10:55 |
| openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 10:55 |
| openstackgerrit | sahid proposed openstack/nova master: network: add command to configure trusted mode for VFs https://review.openstack.org/458513 | 10:55 |
| *** TuanLA has quit IRC | 10:55 | |
| *** alexchadin has joined #openstack-nova | 10:58 | |
| *** alexchadin has quit IRC | 10:58 | |
| *** alexchadin has joined #openstack-nova | 10:58 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 10:58 |
| *** nicolasbock has quit IRC | 11:07 | |
| *** nicolasbock has joined #openstack-nova | 11:09 | |
| *** phuongnh has quit IRC | 11:12 | |
| *** hrw has quit IRC | 11:17 | |
| *** psachin has quit IRC | 11:19 | |
| *** ratailor has quit IRC | 11:20 | |
| *** links has quit IRC | 11:20 | |
| *** lucasagomes is now known as lucas-hungry | 11:21 | |
| *** rabel has quit IRC | 11:23 | |
| *** rabel has joined #openstack-nova | 11:24 | |
| *** abhi89 has joined #openstack-nova | 11:27 | |
| *** trinaths has left #openstack-nova | 11:28 | |
| openstackgerrit | sahid proposed openstack/nova master: compute: reset instance events https://review.openstack.org/420026 | 11:29 |
| *** yangyapeng has joined #openstack-nova | 11:29 | |
| *** hrw has joined #openstack-nova | 11:29 | |
| openstackgerrit | sahid proposed openstack/nova master: compute: reset instance events https://review.openstack.org/420026 | 11:30 |
| *** ratailor has joined #openstack-nova | 11:31 | |
| *** links has joined #openstack-nova | 11:32 | |
| *** psachin has joined #openstack-nova | 11:32 | |
| *** Tom has joined #openstack-nova | 11:37 | |
| *** dave-mccowan has joined #openstack-nova | 11:39 | |
| *** nicolasbock has quit IRC | 11:40 | |
| *** Tom has quit IRC | 11:42 | |
| *** yamamoto has joined #openstack-nova | 11:45 | |
| *** Tom_ has joined #openstack-nova | 11:45 | |
| *** markvoelker has joined #openstack-nova | 11:46 | |
| *** smatzek has joined #openstack-nova | 11:47 | |
| *** smatzek has quit IRC | 11:49 | |
| *** smatzek has joined #openstack-nova | 11:49 | |
| *** Tom_ has quit IRC | 11:49 | |
| *** mamandle has joined #openstack-nova | 11:49 | |
| *** lpetrut has quit IRC | 11:51 | |
| *** tbachman has joined #openstack-nova | 11:52 | |
| *** nicolasbock has joined #openstack-nova | 11:52 | |
| *** kylek3h has quit IRC | 11:52 | |
| *** kylek3h has joined #openstack-nova | 11:53 | |
| *** alexchadin has quit IRC | 11:57 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | 11:57 |
| *** tbachman has quit IRC | 11:57 | |
| *** kylek3h has quit IRC | 11:57 | |
| *** jpena is now known as jpena|lunch | 12:00 | |
| *** gvrangan has joined #openstack-nova | 12:01 | |
| *** tbachman has joined #openstack-nova | 12:04 | |
| *** hrw has quit IRC | 12:04 | |
| *** yangyapeng has quit IRC | 12:05 | |
| *** yangyapeng has joined #openstack-nova | 12:05 | |
| *** gcb has joined #openstack-nova | 12:05 | |
| *** udesale has joined #openstack-nova | 12:07 | |
| mamandle | mriedem: after commit 1b2f3c7bb25667527e7e1d2afd84d5d5533e7751, the evacuated instance cleanup is failing in delete_allocation_for_evacuated_instance complaining that the compute node key is not present in resource_tracker compute_nodes . I looked a bit into it and it | 12:07 |
| *** weshay_PTO is now known as weshay | 12:08 | |
| *** kylek3h has joined #openstack-nova | 12:09 | |
| mamandle | it looks like we need to ensure that resources have initialized (via _update_available_resource) before this is called? | 12:09 |
| *** yangyapeng has quit IRC | 12:09 | |
| *** edmondsw has joined #openstack-nova | 12:10 | |
| *** kylek3h has quit IRC | 12:10 | |
| *** ociuhandu has joined #openstack-nova | 12:10 | |
| *** edmondsw has quit IRC | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:11 | |
| *** hrw has joined #openstack-nova | 12:11 | |
| *** thorst has joined #openstack-nova | 12:11 | |
| *** thorst has quit IRC | 12:12 | |
| *** thorst has joined #openstack-nova | 12:13 | |
| *** yamamoto has quit IRC | 12:13 | |
| *** diga has quit IRC | 12:14 | |
| *** hrw has quit IRC | 12:14 | |
| *** mamandle has quit IRC | 12:18 | |
| *** markvoelker has quit IRC | 12:20 | |
| *** ratailor has quit IRC | 12:20 | |
| *** hrw has joined #openstack-nova | 12:25 | |
| *** markvoelker has joined #openstack-nova | 12:28 | |
| *** lpetrut has joined #openstack-nova | 12:28 | |
| *** yamamoto has joined #openstack-nova | 12:28 | |
| *** jaypipes has joined #openstack-nova | 12:30 | |
| *** lucas-hungry is now known as lucasagomes | 12:34 | |
| *** lyan has joined #openstack-nova | 12:37 | |
| openstackgerrit | Merged openstack/nova master: Hyper-V: Perform proper cleanup after cold migration https://review.openstack.org/486955 | 12:39 |
| *** alexchadin has joined #openstack-nova | 12:40 | |
| *** kylek3h has joined #openstack-nova | 12:40 | |
| openstackgerrit | Merged openstack/nova master: Add release note for requiring shred 8.22 or above. https://review.openstack.org/501022 | 12:42 |
| openstackgerrit | Merged openstack/nova master: Update doc to indicate nova-network deprecated https://review.openstack.org/500654 | 12:43 |
| openstackgerrit | Merged openstack/nova master: Replace http with https for doc links in nova https://review.openstack.org/500693 | 12:43 |
| *** trinath has joined #openstack-nova | 12:45 | |
| stephenfin | sdague: Now that Queens is open, could you take another look at this patch? https://review.openstack.org/#/c/466799/ | 12:46 |
| trinath | Hi, While compiling qemu 2.6 from source, "./configure --target-list=aarch64-softmmu --enable-kvm --enable-vhost-net --enable-vnc --enable-curses " , I get the error " ERROR: "cc" cannot build an executable (is your linker broken?)". | 12:46 |
| trinath | please help me resolve this issue. | 12:46 |
| *** scottda_ has joined #openstack-nova | 12:49 | |
| *** gmann_ has joined #openstack-nova | 12:55 | |
| sdague | stephenfin: +2 | 12:55 |
| stephenfin | thank you :) | 12:55 |
| stephenfin | trinath: Totally wrong channel, buddy. You want #qemu or Google | 12:56 |
| *** udesale has quit IRC | 12:56 | |
| *** pchavva has joined #openstack-nova | 13:00 | |
| alex_xu | nova api meeting is running at #openstack-meeting-4 | 13:01 |
| *** mikal has quit IRC | 13:05 | |
| *** xyang1 has joined #openstack-nova | 13:05 | |
| *** slaweq has quit IRC | 13:05 | |
| *** vladikr has joined #openstack-nova | 13:07 | |
| *** adisky__ has joined #openstack-nova | 13:07 | |
| *** david-lyle has quit IRC | 13:09 | |
| *** david-lyle has joined #openstack-nova | 13:09 | |
| *** takedakn has joined #openstack-nova | 13:10 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno: note that custom resources are not supported https://review.openstack.org/501252 | 13:10 |
| *** lucasxu has joined #openstack-nova | 13:11 | |
| *** baoli has joined #openstack-nova | 13:11 | |
| *** mikal has joined #openstack-nova | 13:12 | |
| *** jpena|lunch is now known as jpena | 13:12 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: reno: note that custom resources are not supported https://review.openstack.org/500521 | 13:13 |
| *** mriedem has joined #openstack-nova | 13:13 | |
| *** gouthamr has joined #openstack-nova | 13:14 | |
| *** trinath has quit IRC | 13:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Hyper-V: Perform proper cleanup after cold migration https://review.openstack.org/501254 | 13:15 |
| *** smatzek has quit IRC | 13:16 | |
| *** liverpooler has quit IRC | 13:16 | |
| *** yangyapeng has joined #openstack-nova | 13:16 | |
| *** liverpooler has joined #openstack-nova | 13:16 | |
| *** gmann__ has joined #openstack-nova | 13:17 | |
| *** gmann_ has quit IRC | 13:19 | |
| *** yangyapeng has quit IRC | 13:20 | |
| *** tidwellr has joined #openstack-nova | 13:23 | |
| *** esberglu has joined #openstack-nova | 13:24 | |
| *** sean-k-mooney has quit IRC | 13:25 | |
| openstackgerrit | Merged openstack/nova master: Enhance doc for nova services https://review.openstack.org/499536 | 13:25 |
| *** evrardjp has joined #openstack-nova | 13:26 | |
| *** gvrangan has quit IRC | 13:26 | |
| evrardjp | hello guys, sorry for my glitch in your meeting. | 13:26 |
| alex_xu | jaypipes: mriedem bauzas, a team in the intel is working on upstream CIT https://github.com/opencit/opencit-openstack-extensions/blob/v3.2/nova/scheduler/asset_tag_filter.py, I think it is a upgrade version of trustedfilter. I know we have some problem with the trustedfilter, performance, depend on external service. I'm thinking is it ok using traits API instead of the filter implementation, is a | 13:27 |
| alex_xu | right direction? | 13:27 |
| bauzas | alex_xu: I saw your proposal for the PTG | 13:27 |
| bauzas | alex_xu: MHO is that I think we don't need to see a TrustedFilter | 13:27 |
| evrardjp | I was in the wrong chan. | 13:27 |
| bauzas | I mean, in the tree | 13:27 |
| alex_xu | bauzas: yea, the team reaches to me recently, try to figure out the things | 13:27 |
| alex_xu | bauzas: why | 13:28 |
| bauzas | alex_xu: but in case operators or deployers want to use a custom scheduler filter for that, I'm totally fine | 13:28 |
| openstackgerrit | Merged openstack/nova master: Fix ValueError if invalid max_rows passed to db purge https://review.openstack.org/500771 | 13:28 |
| bauzas | alex_xu: for the exact reason you mentioned | 13:28 |
| bauzas | reasons* even | 13:28 |
| openstackgerrit | Merged openstack/nova master: Mark LXC as missing for swap volume support https://review.openstack.org/482216 | 13:29 |
| alex_xu | bauzas: ok, is it ok for external agent report trusted info as traits to the placement service directly? it sounds like out of nova totatly also | 13:29 |
| gmann__ | mriedem: query schema BP is good without microversion. i updated info in BP - https://blueprints.launchpad.net/nova/+spec/json-schema-validation-for-query-param | 13:29 |
| *** catintheroof has joined #openstack-nova | 13:29 | |
| bauzas | alex_xu: also, the TrustedFilter is deprecated and will be removed by Queens | 13:29 |
| openstackgerrit | Merged openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | 13:30 |
| bauzas | alex_xu: we can surely discuss that in the PTG | 13:30 |
| *** thegreenhundred has joined #openstack-nova | 13:30 | |
| alex_xu | bauzas: yea, i know | 13:30 |
| alex_xu | bauzas: yea, that will be great, try to figure out a way we liked | 13:30 |
| bauzas | alex_xu: if things need to be proposed to Nova for helping operators to have that feature, that's fine by me | 13:30 |
| *** annegentle has joined #openstack-nova | 13:31 | |
| alex_xu | bauzas: ok, probably I need to talk with that team what is the key thing needs to integrate with nova | 13:31 |
| bauzas | alex_xu: I just want to make sure we don't create an in-tree module that would need a 3rd party system that is not tested in the gate :) | 13:31 |
| jaypipes | alex_xu: who is using this openCIT technology? | 13:31 |
| bauzas | and again, removing TrustedFilter from upstream doesn't mean operators or deployers could not use it | 13:32 |
| alex_xu | bauzas: ok :) | 13:32 |
| alex_xu | jaypipes: I don't know :) | 13:32 |
| bauzas | well, I know that :) | 13:32 |
| bauzas | Intel customers, I'd say | 13:32 |
| alex_xu | bauzas: better answer than me :) | 13:32 |
| bauzas | honestly, the OAT feature is nice | 13:33 |
| bauzas | that's how it's implemented in Nova that is bad to me | 13:33 |
| bauzas | s/bad/wrong | 13:33 |
| jaypipes | bauzas: OAT? | 13:34 |
| alex_xu | bauzas: yea, so I'm thinking we have placement API, we can have something report to the placement API direclty out of Nova | 13:34 |
| bauzas | jaypipes: Open Attestation Service or something like that | 13:34 |
| *** tbachman has quit IRC | 13:35 | |
| bauzas | jaypipes: that's basically some agent running on computes that verify whether the compute kernel is not modified | 13:35 |
| bauzas | s/modified/tainted I'd say | 13:35 |
| jaypipes | alex_xu: sure, you can do that. but then aren't we opening up the "trust" attack vector to anyone who can write a trait for a resource provider? ;) | 13:35 |
| *** gmann__ has quit IRC | 13:35 | |
| bauzas | jaypipes: that necessarly supposes that the Placement service can't be tainted | 13:36 |
| jaypipes | bauzas: zactly ;) | 13:36 |
| bauzas | that's an operator problem honestly :) | 13:36 |
| *** burt has joined #openstack-nova | 13:36 | |
| bauzas | if they want to go that direction, fine by me | 13:36 |
| alex_xu | jaypipes: so you mean the only right way is access the OAT server in each scheduling... | 13:37 |
| bauzas | jaypipes: OpenAttestation | 13:37 |
| bauzas | oop | 13:37 |
| bauzas | jaypipes: https://wiki.openstack.org/wiki/OpenAttestation | 13:37 |
| *** gbarros has joined #openstack-nova | 13:38 | |
| bauzas | oh interesting | 13:39 |
| bauzas | OAT has been superseded by OpenCIT | 13:39 |
| mriedem | gmann: ok approved | 13:39 |
| mriedem | thanks for following up | 13:39 |
| mriedem | bauzas: jaypipes: i'm +2 on dan's migration allocations spec if one of you wants to take a gander https://review.openstack.org/#/c/498510/ | 13:40 |
| bauzas | AFAICS, instead of querying a manifest, the compute agent sends a report to the attestation server | 13:40 |
| bauzas | mriedem: for sure | 13:40 |
| jaypipes | alex_xu: no, I'm saying that the "trust" part of all of this stuff would be dependent on how much you trust the caller to the placement API that is setting traits. Obviously, the Placement API doesn't verify callers or attest to their content. | 13:40 |
| bauzas | my review bag is pretty empty those days | 13:40 |
| *** ssmith has joined #openstack-nova | 13:40 | |
| bauzas | jaypipes: well, the placement API accepts credentials, so the real problem is whether they accept Keystone tokens as safe enough or not | 13:41 |
| *** jaosorior has quit IRC | 13:41 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout https://review.openstack.org/498821 | 13:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! doc: Add contents page https://review.openstack.org/498820 | 13:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages https://review.openstack.org/498819 | 13:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add configuration index page https://review.openstack.org/498818 | 13:41 |
| jaypipes | bauzas: right... | 13:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add user index page https://review.openstack.org/498817 | 13:41 |
| bauzas | jaypipes: but again, it's not my problem :) | 13:41 |
| stephenfin | ralonsoh: You about. Question about binding profiles | 13:41 |
| jaypipes | damn you stephenfin :) was just about to +W the bottom of that. | 13:41 |
| mriedem | bauzas: if your review bag is empty, this bug fix and the changes below it are needed for pike https://review.openstack.org/#/c/499878/ | 13:42 |
| mriedem | once that's done and backports are merged i'll cut a release | 13:42 |
| *** hrw has quit IRC | 13:43 | |
| alex_xu | jaypipes: yea, i see that, so we don't have that problem for the existed trusted filter | 13:43 |
| stephenfin | jaypipes: You still can - it was a rebase to move the dodgy contents patch to the end | 13:43 |
| bauzas | mriedem: I'm already on https://review.openstack.org/#/q/topic:bp/request-spec-use-by-compute | 13:43 |
| stephenfin | :) | 13:43 |
| bauzas | jaypipes: mriedem: dansmith: I'm gonna ask a question for https://review.openstack.org/#/c/498510/5/specs/queens/approved/migration-allocations.rst | 13:43 |
| mriedem | bauzas: cool - gerrit somehow auto-changed my topic branch which was annoying there | 13:43 |
| *** alexchadin has quit IRC | 13:43 | |
| *** wlfightup has quit IRC | 13:43 | |
| *** wlfightup has joined #openstack-nova | 13:43 | |
| *** zxy has quit IRC | 13:43 | |
| *** yangyapeng has joined #openstack-nova | 13:43 | |
| *** zxy has joined #openstack-nova | 13:44 | |
| bauzas | jaypipes: mriedem: dansmith: if we consider that allocations need to be set/corrected by the conductor for migrations (and we already do that), could we consider having a single boot request to have the simple allocation claim to be also done in the conductor ? | 13:44 |
| bauzas | I know it has been a long conversation previously | 13:44 |
| *** hrw has joined #openstack-nova | 13:44 | |
| bauzas | and I don't want to open wounds | 13:45 |
| bauzas | but I just want to consider how that could help us having a single driver using Placement | 13:45 |
| *** cleong has joined #openstack-nova | 13:45 | |
| alex_xu | jaypipes: is it another layer problem, since you can attach traits to the resource provider, basically you already get the admin user in the system... | 13:45 |
| *** alexchadin has joined #openstack-nova | 13:45 | |
| jaypipes | alex_xu: I'm not saying it's a problem, per se. I'm just saying that the OpenCTI team should be aware of the fact that communication with the placement API itself is not attested. | 13:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Make eventlet hub use a monotonic clock https://review.openstack.org/434327 | 13:46 |
| alex_xu | jaypipes: yea, i see, so let me tell the opencit team... | 13:47 |
| bauzas | jaypipes: alex_xu: tbh, the current implementation of the TrustedFilter is already considering that the communication to the OpenCIT system is not having a man in the middle | 13:47 |
| bauzas | or that what is passed by the OpenCIT server is correct | 13:47 |
| *** yangyapeng has quit IRC | 13:48 | |
| bauzas | that's just adding a proxy, but the design is the same | 13:48 |
| jaypipes | bauzas: k. but this is opening up another communication channel. that's all I was saying. | 13:48 |
| bauzas | jaypipes: sure, I'm just trying to explain that given the current implementation, those folks don't see that as a problem already :p | 13:48 |
| jaypipes | bauzas: cool. | 13:48 |
| *** hrw has quit IRC | 13:49 | |
| alex_xu | bauzas: yea | 13:49 |
| *** crushil has quit IRC | 13:50 | |
| mriedem | bauzas: the only reason conductor is setting/correcting allocations during a move operation is because of bugs | 13:50 |
| *** liverpooler has quit IRC | 13:50 | |
| bauzas | I know :( | 13:50 |
| mriedem | bauzas: well, or failures in certain cases, like live migration pre-check failing after we've already claimed on the dest host in the scheduler | 13:50 |
| mriedem | but the force cases are bugs, | 13:50 |
| *** hrw has joined #openstack-nova | 13:51 | |
| bauzas | but since the scheduler doesn't know whether it's a move or a boot, we need to do something like that | 13:51 |
| mriedem | and i plan on having the force scenarios still call the scheduler but with a skip_filters flag | 13:51 |
| mriedem | the scheduler, since pike, determines if it's a move by checking for existing allocations on another node | 13:51 |
| mriedem | and doubles those up | 13:51 |
| bauzas | honestly, dansmith proposed to have the scheduler code to be run by the conductor service, and I just think all of this tends to that | 13:51 |
| mriedem | the issue with force is that we bypass the scheduler altogether | 13:51 |
| bauzas | yup | 13:52 |
| mriedem | so are you asking if we should stop doing the claim in the scheduler and move that to conductor now in queens? | 13:52 |
| *** liverpooler has joined #openstack-nova | 13:52 | |
| bauzas | mriedem: yep, I'm wondering if we should reconsider the opportunity to have claims done by the conductor for the reasons you mentioned | 13:53 |
| bauzas | #1 if you force, then you need to reconcile claims | 13:53 |
| mriedem | i think we can still do force and have the scheduler handle the claim | 13:53 |
| mriedem | i'm going to write a bp for that | 13:53 |
| mriedem | since it's going to be an rpc change and require some consideration | 13:53 |
| bauzas | #2 for move operations, there are a list of corner cases depending on the success of the move that require the conductor to reconcile allocations as well | 13:53 |
| mriedem | sure, but for #2 we can also fail on the compute and need to cleanup allocations, | 13:54 |
| bauzas | good point | 13:54 |
| mriedem | which we're seeing with prep_resize failing, evacuate moveclaim failing, and unshelve claim failing | 13:54 |
| mriedem | i do'nt really see this any different from cleaning up ports and volume attachments | 13:54 |
| mriedem | which happen both in conductor sometimes and in the compute | 13:54 |
| bauzas | about the force thingy, what if placement returns "Sorry"' to the scheduler if it claims ? | 13:54 |
| *** hongbin has joined #openstack-nova | 13:55 | |
| mriedem | then we fail | 13:55 |
| bauzas | I'm not sure operators would accept that | 13:55 |
| mriedem | they are going to have to | 13:55 |
| bauzas | LOL | 13:55 |
| mriedem | if you force to a compute today, the claim on the compute could still fail | 13:55 |
| mriedem | except live migration doesn't do a claim | 13:56 |
| mriedem | but evacuate does | 13:56 |
| bauzas | that's more complicated : | 13:56 |
| mriedem | maybe those claims never fail because we pass an empty limits dict | 13:56 |
| mriedem | so the claim just considers unlimited cpu/ram/disk | 13:56 |
| bauzas | since we don't call the scheduler, the scheduler isn't passing limits to the conductor which eventually gives them to the compute | 13:56 |
| bauzas | so we don't really verify the resource usage | 13:56 |
| mriedem | yeah i just said the same thing | 13:57 |
| mriedem | honestly i didn't realize that's how things worked until last week when writing some functional tests for this | 13:57 |
| bauzas | you typed too fast, damn you | 13:57 |
| jaypipes | mriedem, bauzas, alex_xu: https://twitter.com/jaypipes/status/905429571012616192 | 13:57 |
| *** moshele has quit IRC | 13:57 | |
| mriedem | jaypipes: i can't tell if they look freaked out like that because of the storm or if that's just normal | 13:57 |
| bauzas | jaypipes: orly? :( you're going to be impacted ? :( | 13:57 |
| jaypipes | bauzas: yeah. we're probably going to evacuate tonight or tomorrow morning. | 13:58 |
| bauzas | jaypipes: le woof is hugging them | 13:58 |
| jaypipes | mriedem: just normal. | 13:58 |
| jaypipes | heh | 13:58 |
| mriedem | bauzas: point is, we need the resource allocation representation in placement to be accurate, | 13:58 |
| mriedem | and since the RT isn't adjusting claims anymore, | 13:58 |
| mriedem | after the fact i mean, | 13:58 |
| mriedem | we can't just not create the allocations, | 13:58 |
| *** crushil has joined #openstack-nova | 13:58 | |
| mriedem | with the force stuff before with claims, the RT would still report the usage on the compute node to the scheduler and if it was full then that compute node is out of the running for placement decisions | 13:59 |
| mriedem | if we don't report the allocations to placement, then things are incorrect for the scheduler | 13:59 |
| mriedem | regardless of force | 13:59 |
| mriedem | plus, as i said in the ML, i think force is a bad idea anyway | 14:00 |
| mriedem | i'm semi ok with continuing to ignore the filters (even though the live migration force scenario still checks ram and compute filters, but in conductor code) | 14:00 |
| mriedem | but i'm not ok with bypassing placement | 14:00 |
| mriedem | bauzas: so maybe what you want to see in https://review.openstack.org/#/c/499399/ is a release note that says, evacuate with force=True may fail due to placement | 14:01 |
| mriedem | and at some point an update to the api-ref for the force flag on evacuate and live migrate | 14:02 |
| mriedem | i.e. the force flag bypasses the scheduler filters but can still fail due to placement | 14:02 |
| *** Tom_ has joined #openstack-nova | 14:03 | |
| *** Tom_ has quit IRC | 14:03 | |
| mriedem | we merged the same for live migrate for the pike GA but didn't have a release note for that https://review.openstack.org/#/c/496727/ | 14:03 |
| mriedem | i could add a release note for the 16.0.1 release | 14:04 |
| bauzas | mriedem: so, I agree with you about having "something" that would reconcile allocations if you force | 14:04 |
| mriedem | since the RT isn't going to do that once all computes are upgraded to pike, | 14:05 |
| bauzas | the fact is, some operators for example want to live-migrate a whole host to another one with the price of degradation | 14:05 |
| dansmith | we can audit without heal | 14:05 |
| mriedem | it's better to try and allocate in the controller and fail fast than do it in the compute after we've already live migrated or evacuated something | 14:05 |
| *** mamandle has joined #openstack-nova | 14:05 | |
| bauzas | because that's an emergency situation | 14:05 |
| bauzas | and they don't want their customers to know that the host is bad | 14:05 |
| mriedem | if they are live migrating, the customer doesn't know | 14:06 |
| mriedem | if it moved or not | 14:06 |
| dansmith | well, the customer will know of course | 14:06 |
| bauzas | sure, but they want to migrate sooner than later and somehow want to migrate the host to some other they know | 14:06 |
| dansmith | minimum disruption, but that's all | 14:06 |
| bauzas | agreed | 14:07 |
| mriedem | btw, unrelated, but can we get this merged https://review.openstack.org/#/c/500968/ to unblock the requirements upper-constraints change from merging? | 14:07 |
| mriedem | ^ is needed to get nova unit tests to pass with the latest os-xenapi lib | 14:07 |
| dansmith | I'm not sure I understand what the concern is, I just started reading | 14:07 |
| dansmith | all we have to do is make sure force does a caim first, and if that fails, fail, right? | 14:07 |
| mriedem | dansmith: which is exactly what this does https://review.openstack.org/#/c/496727/ | 14:08 |
| mriedem | and https://review.openstack.org/#/c/499399/ | 14:08 |
| dansmith | right | 14:08 |
| mriedem | we're really talking about two things and diverged from the first, which is bauzas wants to consider moving the claim code out of the scheduler and into conductor | 14:09 |
| mriedem | and then we went down the force flag rathole | 14:09 |
| alex_xu | jamespage: bauzas there also a proposal in the intel about integrate an agent which managing cpu l3 cache into the nova. I'm also thinking report the l3 cache inventory by the agent to the placement directly is option, but there missed a way for the agent to know new instance boot up in the node except polling the libvirt | 14:09 |
| *** jaosorior_ has joined #openstack-nova | 14:09 | |
| mriedem | alex_xu: listen for notifications | 14:09 |
| alex_xu | jaypipes: ^ | 14:09 |
| alex_xu | jamespage: sorry for pointed to wrong person | 14:09 |
| alex_xu | mriedem: but the notification isn't for single host, we must filter the info | 14:10 |
| mriedem | alex_xu: if you want a hypervisor agnostic solution, listen for notifications rather than polling libvirt directly | 14:10 |
| jaypipes | alex_xu: you mean RDT/CAT, right? | 14:10 |
| alex_xu | jaypipes: yea | 14:10 |
| *** armax has joined #openstack-nova | 14:10 | |
| mriedem | alex_xu: the instance.create.end payload would have the host/node information in it so yeah you'd have to filter but that doesn't seem hard | 14:11 |
| bauzas | I don't get why it requires libvirt to be polled | 14:11 |
| bauzas | and I second mriedem about subscribing to the notifications bus | 14:12 |
| kashyap | bauzas: What requires libvirt to be polled? /me read it out of context; will read the scrollback | 14:12 |
| *** Tom_ has joined #openstack-nova | 14:12 | |
| alex_xu | mriedem: yea, that won't be so hard, that probably each node have a agent to listen the notification | 14:12 |
| mriedem | bauzas: xenserver ci isn't going to care about unit test changes https://review.openstack.org/#/c/500968/ | 14:12 |
| bauzas | holy ship, you're right | 14:12 |
| alex_xu | bauzas: kashyap an external agent try to know there is new instance boot up | 14:13 |
| mriedem | alex_xu: plus then you can use the fancy new versioned notifications and provide feedback to gibi | 14:13 |
| alex_xu | mriedem: yeah | 14:13 |
| gibi | yeah :) | 14:14 |
| kashyap | alex_xu: Ah, it's the L3 CAT interface. I see some in-progress design work of exposing it in libvirt upstream | 14:14 |
| alex_xu | kashyap: yea, looks like libvirt interface is slow | 14:14 |
| kashyap | alex_xu: Yeah, there's a new thread that started a day ago | 14:14 |
| kashyap | Did you see that? About the new RFC / design | 14:14 |
| *** ekuris has quit IRC | 14:15 | |
| alex_xu | kashyap: not yet, another team member follow that, probably I will tell him | 14:15 |
| *** yamahata has joined #openstack-nova | 14:15 | |
| *** priteau has joined #openstack-nova | 14:16 | |
| mdbooth | melwitt: You about? Spotted something potentially funky looking at your change https://review.openstack.org/#/c/498983/ | 14:16 |
| mdbooth | Not directly related to your patch | 14:16 |
| *** yamamoto has quit IRC | 14:17 | |
| alex_xu | kashyap: thanks for the info | 14:17 |
| mdbooth | I couldn't see where we wrap swap_volume in any kind of lock | 14:17 |
| *** takedakn has quit IRC | 14:17 | |
| kashyap | alex_xu: Here it is: "Yet another RFC for [Intel] CAT" - https://www.redhat.com/archives/libvir-list/2017-September/msg00072.html | 14:17 |
| mdbooth | e.g. by setting task_state | 14:17 |
| mdbooth | I expect I just missed it, though. | 14:17 |
| *** ociuhandu has quit IRC | 14:17 | |
| *** yangyapeng has joined #openstack-nova | 14:18 | |
| mdbooth | Do you happen to know if there is any locking around that method? | 14:18 |
| *** tuanla_fujitsu has joined #openstack-nova | 14:18 | |
| *** tuanla__ has joined #openstack-nova | 14:18 | |
| mdbooth | If there isn't, the save/restore xml after a long-running rebase mechanism would be potentially pretty bad. | 14:19 |
| *** smatzek has joined #openstack-nova | 14:19 | |
| kashyap | mdbooth: I read your comment this morning, and did find your task exclusion observation interesting | 14:19 |
| gibi | mriedem: could you confirm that we want to delete the allocation of the instance during _destroy_evacuated_instances if the instance failed an evacuation? | 14:19 |
| gibi | mriedem: context is in https://review.openstack.org/#/c/499237/ | 14:19 |
| kashyap | Hmm, although the patch is a strict improvement as-is, now I wonder about your comment - where is the bug lurking... | 14:19 |
| dansmith | mriedem: I'm going to be in an airport when the cells meeting is going on | 14:19 |
| dansmith | mriedem: I can try to run it from there, but no promises | 14:19 |
| alex_xu | kashyap: thanks! | 14:20 |
| mriedem | dansmith: let's just skip | 14:20 |
| *** priteau has quit IRC | 14:20 | |
| *** priteau_ has joined #openstack-nova | 14:20 | |
| dansmith | mriedem: cool | 14:20 |
| gibi | mriedem: I'm asking because then a subsequent rebuild needs to allocate resources again | 14:20 |
| *** annegentle has quit IRC | 14:21 | |
| *** priteau_ has quit IRC | 14:21 | |
| *** yangyapeng has quit IRC | 14:22 | |
| *** priteau has joined #openstack-nova | 14:22 | |
| mriedem | gibi: hmm, and a rebuild to the same host won't do a claim, and the RT won't auto-heal the allocations on the node | 14:22 |
| mriedem | and rebuild (not evacuate) bypasses the scheduler | 14:23 |
| mriedem | that's a good point | 14:23 |
| *** eharney has joined #openstack-nova | 14:23 | |
| gibi | mriedem: yes, that is my fear | 14:23 |
| gibi | mriedem: in the other hand we could put the instance to error state after the failed evac and keep the allocation | 14:24 |
| gibi | mriedem: this way the rebuild doesn't need to allocate | 14:24 |
| mriedem | i'm in the middle of something and will have to digest the replies in that review later | 14:24 |
| mriedem | gibi: we might also just have to leave this until the ptg | 14:24 |
| gibi | mriedem: sure | 14:24 |
| mriedem | could you add it to the ptg etherpad in case it has to wait until next week? | 14:24 |
| *** hogepodge has joined #openstack-nova | 14:24 | |
| gibi | mriedem: adding... | 14:25 |
| hogepodge | mriedem: initial schedule is up, let me know if times work for you, leave notes if things need to be shuffled. It's all preliminary right now and subject to change. https://etherpad.openstack.org/p/InteropDenver2017PTG | 14:26 |
| *** Dave has quit IRC | 14:27 | |
| mriedem | hogepodge: ok thanks | 14:27 |
| *** awaugama has joined #openstack-nova | 14:30 | |
| *** links has quit IRC | 14:31 | |
| gibi | mriedem: I added it to the etherpad | 14:32 |
| *** hrw has quit IRC | 14:34 | |
| *** gbarros has quit IRC | 14:35 | |
| *** hrw has joined #openstack-nova | 14:35 | |
| mriedem | gibi: thanks | 14:36 |
| hogepodge | dtantsur|afk: ^^ | 14:40 |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle exception on adding secgroup https://review.openstack.org/465173 | 14:41 |
| openstackgerrit | Merged openstack/nova master: rbd: Remove unnecessary 'encode' calls https://review.openstack.org/412356 | 14:41 |
| stephenfin | Hurrah ^ | 14:42 |
| *** Dave has joined #openstack-nova | 14:43 | |
| *** hrw has quit IRC | 14:43 | |
| openstackgerrit | Merged openstack/nova master: Remove qpid description in doc https://review.openstack.org/499087 | 14:44 |
| *** annegentle has joined #openstack-nova | 14:44 | |
| *** gbarros has joined #openstack-nova | 14:45 | |
| *** tbachman has joined #openstack-nova | 14:46 | |
| *** hrw has joined #openstack-nova | 14:46 | |
| *** mdrabe has joined #openstack-nova | 14:48 | |
| *** alexchadin has quit IRC | 14:51 | |
| *** slaweq has joined #openstack-nova | 14:53 | |
| dansmith | mriedem: dtantsur|afk: see this ironic migration thing? https://review.openstack.org/#/c/501025/2 | 14:54 |
| *** kairo has quit IRC | 14:54 | |
| *** kairo has joined #openstack-nova | 14:54 | |
| mriedem | now i do | 14:56 |
| esberglu | I'm getting the following unauthorized command error from nova-rootwrap intermittently | 14:58 |
| esberglu | http://paste.openstack.org/show/620547/ | 14:58 |
| mriedem | sdague: looks like https://review.openstack.org/#/c/457636/ is happy - the devstack patch to install the osc-placement plugin | 14:58 |
| mriedem | http://logs.openstack.org/36/457636/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ad2b234/logs/pip2-freeze.txt.gz | 14:59 |
| esberglu | In /etc/nova/rootwrap.d/compute.filters I have the following line | 14:59 |
| esberglu | tee: CommandFilter, tee, root | 14:59 |
| esberglu | Anyone have any idea why sometimes that filter is failing to match? | 14:59 |
| *** marst has joined #openstack-nova | 15:01 | |
| *** efried_zzz is now known as efried | 15:01 | |
| *** kairo has quit IRC | 15:02 | |
| *** kairo has joined #openstack-nova | 15:02 | |
| *** andreas_s_ has joined #openstack-nova | 15:02 | |
| sdague | mriedem: yep | 15:03 |
| *** rabel has quit IRC | 15:03 | |
| *** tbachman has quit IRC | 15:03 | |
| *** felipemonteiro has joined #openstack-nova | 15:03 | |
| *** felipemonteiro_ has joined #openstack-nova | 15:05 | |
| *** andreas_s has quit IRC | 15:05 | |
| stephenfin | sdague: Is the RXTX factor flavor key still applicable with a neutron backend? | 15:07 |
| * stephenfin thinks it was you discussing it a while back | 15:07 | |
| sdague | stephenfin: I thought it was nova net (and maybe xen) only. But I don't know. | 15:08 |
| mriedem | hmm, did we ever say in the pike release notes that conductor now needs it's nova.conf to have the [placement] section filled in? | 15:08 |
| *** felipemonteiro has quit IRC | 15:08 | |
| mriedem | stephenfin: no, i plan on deprecating that out of the api | 15:08 |
| mriedem | just need to write the spec | 15:08 |
| mriedem | and yes, it's really only nova net + xen | 15:09 |
| stephenfin | mriedem: Excellent. I'm going to put that in the docs and mark bug 1688054 as invalid | 15:09 |
| openstack | bug 1688054 in openstack-manuals "Flavors in Administrator Guide - confusing description for rxtx factor" [Medium,Confirmed] https://launchpad.net/bugs/1688054 | 15:09 |
| mriedem | stephenfin: that doesn't make the bug invalid | 15:09 |
| mriedem | if the docs are wrong for how it's used today | 15:09 |
| sdague | esberglu: where in the code is that called? | 15:09 |
| *** kairo has quit IRC | 15:09 | |
| stephenfin | Yeah, that's a good point. OK, I'll clarify the intent a little better so | 15:10 |
| stephenfin | ...and join the flavor/flavor2 files | 15:10 |
| stephenfin | #RefactoringFTW | 15:10 |
| *** david-lyle has quit IRC | 15:10 | |
| *** itlinux has quit IRC | 15:11 | |
| efried | esberglu IT or OOT? SDE or traditional? Is it a snapshot operation that's being tested? | 15:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 15:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | 15:13 |
| *** david-lyle has joined #openstack-nova | 15:15 | |
| efried | Okay, I think I answered my own questions esberglu | 15:16 |
| *** yamamoto has joined #openstack-nova | 15:17 | |
| efried | sdague: This is being done as part of the snapshot operation in our OOT driver: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/mgmt.py#L97-L100 | 15:17 |
| *** tbachman has joined #openstack-nova | 15:18 | |
| *** chyka has joined #openstack-nova | 15:19 | |
| esberglu | Sorry got pulled away for a few | 15:19 |
| esberglu | efried: Yeah this is OOT snaphot | 15:19 |
| efried | esberglu And presumably this is causing the test to fail. | 15:20 |
| esberglu | efried: Yep | 15:20 |
| efried | But the _tee_as_root doesn't raise, right? We just fail to find the disk in the next part of the code? | 15:20 |
| *** chyka has quit IRC | 15:21 | |
| *** chyka has joined #openstack-nova | 15:21 | |
| esberglu | efried: The nova-rootwrap command raises when trying to execute that tee command | 15:24 |
| efried | esberglu Oh. Boo. | 15:24 |
| *** gmann has quit IRC | 15:24 | |
| esberglu | efried: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/mgmt.py#L58 | 15:24 |
| *** gmann has joined #openstack-nova | 15:24 | |
| esberglu | It calls that execute which goes into the rootwrap filters and should match the tee line | 15:24 |
| esberglu | But sometimes it isn't matching that tee filter | 15:25 |
| efried | Which rootwrap filter? | 15:25 |
| *** yamamoto has quit IRC | 15:25 | |
| esberglu | tee: CommandFilter, tee, root | 15:25 |
| esberglu | in /etc/nova/rootwrap.d/compute.filters | 15:26 |
| *** suresh12 has joined #openstack-nova | 15:26 | |
| esberglu | That line should allow tee to be run as root with any parameters | 15:26 |
| efried | esberglu Sorry, I mean are we using RootwrapDaemonHelper or RootwrapProcessHelper? | 15:27 |
| *** priteau_ has joined #openstack-nova | 15:29 | |
| *** priteau has quit IRC | 15:29 | |
| *** gszasz has quit IRC | 15:29 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for force live migration allocations https://review.openstack.org/501314 | 15:30 |
| mriedem | bauzas: here you go ^ | 15:30 |
| bauzas | mriedem: roger. just in a meeting now | 15:30 |
| *** gszasz has joined #openstack-nova | 15:31 | |
| *** priteau has joined #openstack-nova | 15:31 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for force live migration allocations https://review.openstack.org/501314 | 15:31 |
| efried | esberglu Can you try setting `use_rootwrap_daemon` to True in the compute conf and see if the problem magically disappears? | 15:31 |
| efried | esberglu That's in the [DEFAULT] section | 15:31 |
| esberglu | efried: Yeah that isn't set currently. I can give it a try | 15:32 |
| efried | esberglu Though if non-daemon is busted, that seems like it oughtta be a bug to nova. | 15:32 |
| *** tbachman has quit IRC | 15:33 | |
| openstackgerrit | Merged openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146 | 15:34 |
| sdague | efried: yeh, though the real fix would be to get rid of it entirely for privsep | 15:34 |
| *** priteau_ has quit IRC | 15:34 | |
| sdague | be aware, that mikal's patch series is doing that | 15:34 |
| efried | sdague "it" which? | 15:34 |
| efried | The Process version? | 15:34 |
| sdague | https://review.openstack.org/#/c/489438 | 15:35 |
| sdague | that's 5 patches in | 15:35 |
| *** andreas_s_ has quit IRC | 15:35 | |
| sdague | and it will probably take some time to land, but rootwrap for tee is removed at that point | 15:35 |
| sdague | efried: the tee call at all | 15:36 |
| *** priteau has quit IRC | 15:36 | |
| efried | sdague Mm. And the resolution is to decorate my function with this privsep gizmo and then do a regular ol `with open(): write()` ? | 15:37 |
| sdague | yep | 15:37 |
| sdague | but his first patch has to land before that will work | 15:37 |
| *** felipemonteiro__ has joined #openstack-nova | 15:38 | |
| efried | esberglu ^^ FYI. Need to add that to the watchlist so we can react when it lands. | 15:38 |
| esberglu | efried: Will do | 15:39 |
| *** tbachman has joined #openstack-nova | 15:40 | |
| efried | esberglu It would seem we could transition as soon as https://review.openstack.org/#/c/459166/ is in. | 15:40 |
| efried | But in the meantime, <shrug> let's see if using the rootwrap daemon helps. | 15:40 |
| *** priteau has joined #openstack-nova | 15:40 | |
| *** felipemonteiro_ has quit IRC | 15:41 | |
| *** priteau has quit IRC | 15:41 | |
| *** priteau has joined #openstack-nova | 15:42 | |
| *** felipemonteiro_ has joined #openstack-nova | 15:43 | |
| *** burt has quit IRC | 15:43 | |
| *** tbachman_ has joined #openstack-nova | 15:43 | |
| stephenfin | jaypipes: https://review.openstack.org/#/c/498817/ and friends have their gate +1 now, if you were idle enough :) | 15:44 |
| *** tbachman has quit IRC | 15:44 | |
| *** tbachman_ is now known as tbachman | 15:44 | |
| *** burt has joined #openstack-nova | 15:46 | |
| *** tidwellr has quit IRC | 15:46 | |
| *** priteau has quit IRC | 15:46 | |
| *** felipemonteiro__ has quit IRC | 15:46 | |
| *** priteau has joined #openstack-nova | 15:46 | |
| *** priteau_ has joined #openstack-nova | 15:47 | |
| *** priteau_ has quit IRC | 15:48 | |
| *** priteau__ has joined #openstack-nova | 15:50 | |
| *** moshele has joined #openstack-nova | 15:50 | |
| *** priteau has quit IRC | 15:51 | |
| *** tylerderosagrund has joined #openstack-nova | 15:52 | |
| *** gyee has joined #openstack-nova | 15:55 | |
| dansmith | mriedem: yeah I had marked that as -W ahead of me writing the release note, but then I wiped it when I pushed a little fix | 15:56 |
| mriedem | i don't know how the tests are even passing | 15:57 |
| *** moshele has quit IRC | 15:57 | |
| dansmith | mriedem: because you missed that there is a patch before this? | 15:57 |
| mriedem | yes | 15:57 |
| *** claudiub has joined #openstack-nova | 15:58 | |
| *** tuanla_fujitsu has quit IRC | 15:59 | |
| *** tuanla__ has quit IRC | 15:59 | |
| esberglu | efried: ^ 459166 just got W+1 | 16:00 |
| *** lucasxu has quit IRC | 16:00 | |
| *** lpetrut_ has joined #openstack-nova | 16:00 | |
| esberglu | efried: Oh wait nvm looking at the wrong thing | 16:00 |
| efried | esberglu Ya know, we could patch that guy in and make our own patch that uses it... | 16:01 |
| *** claudiub|2 has quit IRC | 16:01 | |
| *** lpetrut has quit IRC | 16:01 | |
| *** lpetrut__ has joined #openstack-nova | 16:03 | |
| *** suresh12 has quit IRC | 16:04 | |
| *** david-lyle has quit IRC | 16:04 | |
| *** dklyle has joined #openstack-nova | 16:04 | |
| openstackgerrit | Merged openstack/nova master: Remove unneeded attributes from context https://review.openstack.org/498615 | 16:05 |
| *** lpetrut_ has quit IRC | 16:05 | |
| openstackgerrit | Merged openstack/nova master: Pass config object to oslo_reports https://review.openstack.org/485575 | 16:06 |
| *** cdent has joined #openstack-nova | 16:06 | |
| openstackgerrit | Merged openstack/nova master: Enhance support matrix document https://review.openstack.org/482020 | 16:07 |
| *** lucasagomes is now known as lucas-afk | 16:07 | |
| openstackgerrit | Merged openstack/nova master: trivial: Remove some single use function from utils https://review.openstack.org/491513 | 16:08 |
| *** gbarros has quit IRC | 16:08 | |
| *** gbarros has joined #openstack-nova | 16:10 | |
| *** yamahata has quit IRC | 16:11 | |
| *** itlinux has joined #openstack-nova | 16:14 | |
| *** dklyle has quit IRC | 16:15 | |
| *** dtantsur|afk is now known as dtantsur | 16:16 | |
| *** dave-mccowan has quit IRC | 16:17 | |
| *** david-lyle has joined #openstack-nova | 16:18 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342 | 16:18 |
| dtantsur | dansmith: thanks, I'll check it tomorrow - day full of errands today :( | 16:20 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025 | 16:20 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Split out the core of the ironic flavor migration https://review.openstack.org/501024 | 16:20 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/501343 | 16:20 |
| dansmith | dtantsur: ack, np | 16:20 |
| *** Tom_ has quit IRC | 16:21 | |
| *** krtaylor has quit IRC | 16:21 | |
| dtantsur | cdent: any specific schedule plans for API SIG PTG? | 16:21 |
| cdent | dansmith: could you give me some early feedback on whether you think this is the right approach: https://review.openstack.org/#/c/501051/ (it’s related to post /allocations) | 16:21 |
| dtantsur | cdent: I'm figuring out conflicts with https://etherpad.openstack.org/p/InteropDenver2017PTG | 16:21 |
| *** shan has joined #openstack-nova | 16:21 | |
| dims | jaypipes : when is the hurricane expected? | 16:22 |
| dansmith | cdent: yeah I know I need to do that, sorry | 16:22 |
| dansmith | dims: weekend I think | 16:22 |
| dansmith | dims: meanwhile portland is on fire and florida refuses to share the water: the | 16:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/486660 | 16:23 |
| cdent | dtantsur: edleafe and I talked about that yesterday and we’re going to try to cook a schedule, but as there aren’t specific topics (nobody signed up for guided reviews) that are really on the agenda, it is mostly going to be workshopping guideline improvements and whatever comes up | 16:23 |
| dansmith | http://www.oregonlive.com/wildfires/index.ssf/2017/09/eagle_creek_fire_rages_in_colu.html | 16:23 |
| cdent | no worries dansmith | 16:23 |
| dims | dansmith : y i heard :) | 16:23 |
| dtantsur | cdent: surprisingly to me, the guided review idea has not got too much traction with other ironic team members.. | 16:23 |
| dims | dansmith : how far is it away from you? | 16:23 |
| dansmith | dims: not far enough, but not currently a risk for us | 16:24 |
| dansmith | dims: but it's crazy smoky outside and ash is falling on everything | 16:24 |
| *** tidwellr has joined #openstack-nova | 16:25 | |
| dansmith | walking outside is like getting a face full of camp fire | 16:25 |
| dims | dansmith : dear lord | 16:25 |
| dansmith | meanwhile, florida is all like "oh water, what are we going to do with all this water? did we mention we have a lot of water on the way?" | 16:25 |
| *** gvrangan has joined #openstack-nova | 16:25 | |
| cdent | that’s just rude | 16:25 |
| dansmith | ikr | 16:25 |
| mnaser | ahem im trying to troubleshoot (what i think is a nova bug).. any idea why instances can be seen in the list, but viewing them returns a 404? | 16:26 |
| mnaser | ex: nova list shows them, nova show <uuid> says no server matching :X | 16:26 |
| dansmith | mnaser: instance mapping missing? | 16:26 |
| dims | dansmith : jaypipes : hopefully you will both make it to ptg | 16:26 |
| dansmith | dims: thanks :) | 16:26 |
| *** gszasz has quit IRC | 16:27 | |
| dansmith | mnaser: sounds like cell mappings are in place, so it can iterate and list, but no instance mapping for when it needs to go from instance uuid -> cell | 16:27 |
| dansmith | mnaser: look in api database, instance_mappings table, see if there is one for the instance you're trying | 16:27 |
| *** rajathagasthya has joined #openstack-nova | 16:27 | |
| mnaser | i wonder why the mapping didnt go in then (customer says they recently failed builds) | 16:27 |
| dansmith | mnaser: idk, should get an instance mapping once you're scheduled (or map you to cell0 if you fail) so we'd need to know what happened exactly to figure it out | 16:28 |
| mnaser | /var/log/nova/nova-conductor.log:2017-09-06 10:42:20.512 829 WARNING nova.scheduler.utils [req-dd7ba87f-8539-4642-b238-3d1e4e4a686c 8873fec3d88b45c09af91ac942917650 eb60ffb2745a40e9b76223e5481e7c5e - default default] [instance: 90ecb4cc-8ea9-4419-9a3d-f67c9436df82] Setting instance to ERROR state.: TooManyInstances: Quota exceeded for ram: Requested 32768, but already used 716800 of 716800 ram | 16:28 |
| mnaser | i wonder if it fails there meaning it doesn't end up getting scheduled? | 16:28 |
| dansmith | mnaser: yeah so that should map you to cell0 | 16:28 |
| mnaser | time to check if it did | 16:29 |
| dansmith | mnaser: well, it seems like it's not mapped, but after confirming that, see if it's in the cell0 database | 16:29 |
| *** Apoorva has joined #openstack-nova | 16:29 | |
| mnaser | MariaDB [nova_cell0]> select * from instances where id='90ecb4cc-8ea9-4419-9a3d-f67c9436df82'; => empty set | 16:29 |
| dansmith | mnaser: uuid | 16:30 |
| mnaser | oh duh | 16:30 |
| mnaser | empty | 16:30 |
| *** felipemonteiro__ has joined #openstack-nova | 16:30 | |
| dansmith | check the other? | 16:30 |
| mnaser | select * from instance_mappings where instance_uuid='90ecb4cc-8ea9-4419-9a3d-f67c9436df82'; => cell_id is null | 16:30 |
| *** itlinux_ has joined #openstack-nova | 16:31 | |
| mnaser | i can see it in my default clel db | 16:31 |
| mnaser | (one that was created upgrading to ocata) | 16:31 |
| mnaser | and the only one other than cell0 | 16:32 |
| dansmith | okay that's weird | 16:32 |
| dansmith | mnaser: can you file a bug with those details? | 16:32 |
| *** itlinux has quit IRC | 16:32 | |
| *** licanwei has quit IRC | 16:32 | |
| mnaser | dansmith will do and ill do a bit more research too | 16:32 |
| mnaser | maybe it happens when you hit quotas | 16:32 |
| dansmith | mnaser: also, you should be able to do map_instances on your non-cell0 cell and fix that mapping I think | 16:32 |
| *** felipemonteiro__ has quit IRC | 16:33 | |
| dansmith | although you might have to nuke the instance mapping row first, which is also a bug for maintainabiity | 16:33 |
| dansmith | mnaser: thanks | 16:33 |
| *** felipemonteiro__ has joined #openstack-nova | 16:33 | |
| *** annegentle has quit IRC | 16:33 | |
| *** felipemonteiro_ has quit IRC | 16:34 | |
| openstackgerrit | Merged openstack/nova master: Glance download: only fsync files https://review.openstack.org/451094 | 16:36 |
| openstackgerrit | Merged openstack/nova master: spelling mistake https://review.openstack.org/501215 | 16:36 |
| mnaser | dansmith found the bug im pretty sure | 16:36 |
| mnaser | https://github.com/openstack/nova/blob/a6c429bd7916211aa04d96caa54443ad92cb99ff/nova/conductor/manager.py#L992-L1006 | 16:37 |
| mnaser | no bury in there | 16:37 |
| mnaser | unlike the other failure cases | 16:37 |
| mnaser | i guess i can work around it now by setting quota/recheck_quota=false for now.. but i think that's the failure case | 16:37 |
| *** dave-mccowan has joined #openstack-nova | 16:38 | |
| *** slaweq has quit IRC | 16:39 | |
| *** gszasz has joined #openstack-nova | 16:39 | |
| *** tidwellr has quit IRC | 16:43 | |
| *** jpena is now known as jpena|off | 16:43 | |
| *** penick has joined #openstack-nova | 16:44 | |
| *** lucasxu has joined #openstack-nova | 16:47 | |
| dansmith | mnaser: that's not the bug | 16:47 |
| dansmith | mnaser: at that point we've created it in the one cell, so we can no longer bury it in cell0 | 16:47 |
| dansmith | mnaser: so the bug is that we need to finish updating the mapping in that case, but apparently we aren't | 16:47 |
| dansmith | mnaser: this is pike or ocata? | 16:47 |
| mnaser | pike | 16:48 |
| *** mamandle is now known as manasm | 16:48 | |
| *** gszasz has quit IRC | 16:48 | |
| mnaser | this use case would launch like 30 instances at once | 16:48 |
| dansmith | okay | 16:48 |
| mnaser | s/user use case/ | 16:48 |
| mnaser | and i guess it goes over quota in that big burst | 16:48 |
| dansmith | mnaser: L1042 is the problem | 16:49 |
| dansmith | meaning, we never run L1042, but we should | 16:49 |
| *** shan7 has joined #openstack-nova | 16:49 | |
| dansmith | in that quota casw | 16:49 |
| mnaser | so technically we should be saving before checking the quota again | 16:49 |
| manasm | mriedem: did you get a chance to look at my comment regarding commit 1b2f3c7bb25667527e7e1d2afd84d5d5533e7751 earlier today? | 16:49 |
| openstackgerrit | Merged openstack/nova master: Put base policy rules at first https://review.openstack.org/500736 | 16:50 |
| * mnaser has to run | 16:50 | |
| mnaser | if folks are busy i dont mind picking up the work cause its a bit critical for us | 16:50 |
| mnaser | just need a little pointing to what has to be done | 16:50 |
| *** felipemonteiro_ has joined #openstack-nova | 16:50 | |
| *** gbarros has quit IRC | 16:51 | |
| dansmith | mnaser: sure, I'll help guide when you're back to work on it. thanks for excellent triage | 16:51 |
| manasm | the evacuated instance cleanup is failing in delete_allocation_for_evacuated_instance complaining that the compute node key is not present in resource_tracker compute_nodes. It looks like we need to ensure that resources have initialized (via _update_available_resource) before this is called? | 16:52 |
| *** shan has quit IRC | 16:53 | |
| mriedem | mnaser: did you report a bug yet? | 16:53 |
| *** annegentle has joined #openstack-nova | 16:53 | |
| mriedem | manasm: i didn't that no, but i think gibi has a related bug open | 16:53 |
| *** shan has joined #openstack-nova | 16:54 | |
| mriedem | about a PUT /allocations with an empty body/ | 16:54 |
| mriedem | ? | 16:54 |
| *** felipemonteiro__ has quit IRC | 16:54 | |
| *** shan7 has quit IRC | 16:54 | |
| *** sambetts is now known as sambetts|afk | 16:54 | |
| *** psachin has quit IRC | 16:55 | |
| *** dave-mccowan has quit IRC | 16:56 | |
| manasm | mriedem: gibi: can you please help me with defect number so that I can confirm if its the same? | 16:56 |
| *** dave-mccowan has joined #openstack-nova | 16:57 | |
| *** lpetrut_ has joined #openstack-nova | 16:59 | |
| mriedem | manasm: https://bugs.launchpad.net/nova/+bug/1714924 ? | 16:59 |
| openstack | Launchpad bug 1714924 in OpenStack Compute (nova) "placement api fails when nova tries to delete resource allocation after failed evacuation" [Medium,Confirmed] | 16:59 |
| * mriedem eats lunch | 16:59 | |
| *** gbarros has joined #openstack-nova | 17:00 | |
| *** annegentle has quit IRC | 17:00 | |
| *** lpetrut__ has quit IRC | 17:01 | |
| *** derekh has quit IRC | 17:01 | |
| *** sahid has quit IRC | 17:01 | |
| *** rcernin has quit IRC | 17:02 | |
| *** moshele has joined #openstack-nova | 17:03 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | 17:03 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166 | 17:04 |
| *** annegentle has joined #openstack-nova | 17:04 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | 17:04 |
| *** yarokifor has quit IRC | 17:04 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | 17:04 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | 17:04 |
| sdague | oh, a wild mikal | 17:05 |
| *** krtaylor has joined #openstack-nova | 17:05 | |
| mikal | sdague: I _think_ I've addressed your concerns on https://review.openstack.org/#/c/459166/33, but your comments are welcome as always | 17:05 |
| *** jaosorior_ has quit IRC | 17:05 | |
| *** suresh12 has joined #openstack-nova | 17:06 | |
| *** yangyapeng has joined #openstack-nova | 17:06 | |
| *** harlowja has quit IRC | 17:07 | |
| mikal | sdague: I know I've broken things later in the series with your requested changes, but I am going through and fixing those up now (it will take a while because of test runs) | 17:07 |
| *** harlowja has joined #openstack-nova | 17:07 | |
| dansmith | sdague: what percentage of the remaining gate resources would you say mikal's set above consumes? :) | 17:08 |
| *** felipemonteiro_ has quit IRC | 17:08 | |
| sdague | dansmith: meh, I think we've been pegged all day already anyway | 17:09 |
| sdague | mikal: the patch above that has an easy fix | 17:09 |
| *** lpetrut_ has quit IRC | 17:09 | |
| *** felipemonteiro_ has joined #openstack-nova | 17:09 | |
| sdague | which is just delete the uml section | 17:09 |
| dansmith | heh | 17:09 |
| *** yangyapeng has quit IRC | 17:10 | |
| sdague | stephenfin: you deleted uml support at some point, right? | 17:10 |
| *** felipemonteiro__ has joined #openstack-nova | 17:11 | |
| sdague | dansmith: well it looks like zuul went super grumpy anyway and just merge conflicted the whole stack on itself | 17:12 |
| sdague | so no tests being run | 17:12 |
| dansmith | karma is a bitch | 17:12 |
| *** shan has quit IRC | 17:13 | |
| kmalloc | sdague: I want to talk with you re some limit stuff being stored in keystone before the PTG session if you'll have a moment or two | 17:13 |
| *** yamamoto has joined #openstack-nova | 17:13 | |
| kmalloc | I want to make sure I'm not way off in the weeds. we can meet somewhere around the room(s), but just want to make sure i'm not thinking about this side-angled. | 17:14 |
| *** felipemonteiro_ has quit IRC | 17:14 | |
| sdague | kmalloc: sure, though I think the bigger question is finding someone else to drive the limits/quota work, because I definitely don't have the bw on it any more. | 17:14 |
| sdague | but, yet, I'm happy to chat up current thinking | 17:14 |
| *** moshele has quit IRC | 17:14 | |
| kmalloc | right. i expect someone will have to help drive it | 17:15 |
| *** kristian__ has joined #openstack-nova | 17:15 | |
| kmalloc | but i know you and I spent a chunk of time going over the details and requirements :) | 17:15 |
| kmalloc | i also don't have the bw to drive it | 17:15 |
| kmalloc | =/ | 17:15 |
| sdague | yeh, I would be more inclined to spend time figuring out if there is someone that wants to own it, and then supporting them. I don't think it makes a ton of sense to work on details until that is figured out | 17:16 |
| *** JoseMello has joined #openstack-nova | 17:16 | |
| manasm | mriedem: thanks for bug link, will check and revert. | 17:17 |
| *** yamamoto has quit IRC | 17:17 | |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| openstackgerrit | Erik Berg proposed openstack/nova master: Fix binary name. https://review.openstack.org/501359 | 17:20 |
| *** Tom_ has joined #openstack-nova | 17:21 | |
| *** manasm has quit IRC | 17:21 | |
| *** moshele has joined #openstack-nova | 17:21 | |
| mikal | dansmith: hey, perhaps these could have been reviewed five months ago when they were written then? | 17:22 |
| dansmith | mikal: no I'm pretty sure nobody can look at them until they're more than three months old. isn't that a rule or something? | 17:23 |
| *** cfriesen has quit IRC | 17:23 | |
| dansmith | mikal: I'm poking fun at the sad state of the gate, not your set of course :) | 17:23 |
| *** lpetrut has joined #openstack-nova | 17:25 | |
| *** tidwellr has joined #openstack-nova | 17:25 | |
| *** Tom_ has quit IRC | 17:25 | |
| *** suresh12 has quit IRC | 17:26 | |
| mikal | dansmith: possibly... my plan is to keep growing the series and hold the gate to ransom until you give into my demands. | 17:26 |
| *** suresh12 has joined #openstack-nova | 17:26 | |
| dansmith | mikal: seems like a solid strategy to me | 17:26 |
| mikal | dansmith: I know, but I am hoping to emotionally pressure you into rubber stamping my thing | 17:26 |
| dansmith | hah | 17:26 |
| mikal | Also, UML WTF? | 17:26 |
| mikal | Also, said thing caused cburgess a heart attack, so clearly its ready for prime time | 17:27 |
| *** jamesdenton has joined #openstack-nova | 17:27 | |
| dansmith | I run a for-profit public cloud on UML | 17:28 |
| dansmith | still waiting for customers though | 17:28 |
| *** baoli has quit IRC | 17:28 | |
| sdague | mikal: I had thought that stephenfin had actually removed that, but maybe not | 17:28 |
| mikal | sdague: or this is a dangling bit? | 17:28 |
| sdague | I went through git log -S on it, it looks like it was put in place in the before times, for CI testing | 17:28 |
| mikal | sdague: I found some dead code in libvirt.vif as well on this little journey | 17:29 |
| sdague | I think that uml references are fair to just delete | 17:29 |
| sdague | I guess we should probably get final sign off from fearless leader on that, maybe I'll push up a blanket patch for it | 17:29 |
| dansmith | I'll bet you a dollar that dear leader thinks UML is a diagram technique | 17:30 |
| sdague | heh | 17:30 |
| sdague | yeh, I did have to explain it to folks | 17:30 |
| mikal | We should deprecate it | 17:30 |
| mikal | Because that's always fun | 17:30 |
| *** yamahata has joined #openstack-nova | 17:31 | |
| *** baoli has joined #openstack-nova | 17:31 | |
| *** moshele has quit IRC | 17:32 | |
| *** kristia__ has joined #openstack-nova | 17:32 | |
| *** adisky__ has quit IRC | 17:32 | |
| jaypipes | dims: currently the hurricane is projected to hit Miami on Sunday midday | 17:33 |
| *** shan has joined #openstack-nova | 17:33 | |
| jaypipes | dims: but that means Sarasota will be no-fly and my flight leaves Sunday morning. | 17:34 |
| *** kristian__ has quit IRC | 17:36 | |
| dims | jaypipes : too close ... :( any evac orders? | 17:37 |
| *** slaweq has joined #openstack-nova | 17:39 | |
| *** cfriesen has joined #openstack-nova | 17:40 | |
| jaypipes | dims: not in sarasota, only the keys | 17:40 |
| dansmith | the keys.. aka "the small sand bars formerly known as the keys" | 17:42 |
| jaypipes | dansmith: right. | 17:42 |
| *** slaweq has quit IRC | 17:42 | |
| *** slaweq has joined #openstack-nova | 17:43 | |
| jaypipes | dansmith: you know... "embers in a fire pit formerly known as Portland" | 17:43 |
| dansmith | jaypipes: yep | 17:44 |
| dansmith | we're past 20,000 acres now | 17:44 |
| dansmith | they've found the 15 year old that started it with a firework | 17:45 |
| dansmith | unfortunately they have not yet released his name | 17:45 |
| cburgess | mikal What caused me a heart attack? | 17:47 |
| mriedem | those fires made it all hazy here in SE MN last weekend | 17:48 |
| mriedem | really ruined my fun | 17:48 |
| mikal | cburgess: privsep being the wave of the future | 17:48 |
| cburgess | mikal I'm over it. I think you see/agree with most of my concerns and I'll bring them up in the patch train over the next few days as I review it. | 17:48 |
| mikal | cburgess: yeah I know, but I think the restart thing will be hard. We can discuss next week as performance art in front of everyone else. | 17:49 |
| cburgess | Ugh... ok | 17:49 |
| mikal | cburgess: also, I ate your carrots. You're welcome. | 17:49 |
| cburgess | Because thats probably my biggest concern. | 17:49 |
| cburgess | mikal lol ok. Anne might be grumpy at you. | 17:49 |
| mikal | cburgess: I didn't eat the _all_, just some | 17:50 |
| cburgess | cool | 17:50 |
| mikal | cburgess: let's just deny it was me and it will be fine | 17:50 |
| cburgess | mikal lol | 17:50 |
| *** gbarros has quit IRC | 17:50 | |
| jaypipes | dansmith: have they begun building the hangman's platform for said child? or has that burned down as well? | 17:51 |
| dansmith | jaypipes: I've been welding it with fire-proof steel in my garage waiting for the announcement | 17:51 |
| jaypipes | dansmith: in other news, +W'd the migration allocation spec | 17:51 |
| jaypipes | dansmith: well played. | 17:51 |
| cburgess | jaypipes So thats real? I saw something on facebook but you know you can't trust facebook. | 17:51 |
| dansmith | jaypipes: if this isn't an advertisement for not having children, I don't know what is | 17:51 |
| jaypipes | dansmith: :) ++ | 17:52 |
| dansmith | "Just look at the widespread devastation that can result from a single child. JUST SAY NO" | 17:52 |
| mriedem | yeah, because drunk 40 year old yokuls shooting off guns never caused any harm | 17:53 |
| mriedem | YEEEEEEEEEEHHHHHAW! | 17:53 |
| cburgess | mriedem We probably shouldn't have them either. | 17:53 |
| dansmith | heh | 17:53 |
| mikal | Heh | 17:53 |
| *** nicolasbock has quit IRC | 17:53 | |
| jaypipes | mriedem: who you calling drunk? | 17:53 |
| mriedem | yokuls | 17:53 |
| mriedem | duh | 17:53 |
| mikal | Google had multiple fibre outages in Oregon because of yokels using the fibre for target practise | 17:54 |
| dansmith | mriedem: problem is, that yokul would go to jail for this, but the kid will get a slap on the wrist and community service | 17:54 |
| jaypipes | mriedem: I'll let you know I done drunk my dranks responsibally | 17:54 |
| * mriedem set a fire in a field once when he was 14 on accident... | 17:54 | |
| cburgess | mriedem In dansmith's world I think we have to take you out back and shoot you now. | 17:55 |
| mriedem | told the authorities it was some other bad kids that they just missed when we got there | 17:55 |
| jaypipes | "accident". yeah, right, mriedem. | 17:55 |
| dansmith | mriedem: well, this kid wasn't even _that_ smart | 17:55 |
| mriedem | "geez i'm real sorry mister" | 17:55 |
| dansmith | cburgess: retroactive "was a kid once" punishment? harsh | 17:55 |
| cburgess | dansmith Thats how I role. | 17:55 |
| * dansmith was never a kid | 17:56 | |
| dansmith | just ask my parents | 17:56 |
| cburgess | lol | 17:56 |
| mriedem | the old benjamin button disease | 17:56 |
| openstackgerrit | Merged openstack/nova-specs master: Add migration-allocations spec https://review.openstack.org/498510 | 17:56 |
| dansmith | ran my middle school bedroom like an office with an inbox and credenza, made my mom call me on the internal phone system if she needed something | 17:56 |
| jaypipes | nice | 17:57 |
| jaypipes | I should do that with Julie and the pugs. | 17:57 |
| jaypipes | "excuse me, Fiona. you need to make an appointment." | 17:57 |
| dansmith | and then put nothing but an answering machine on the other end? | 17:57 |
| jaypipes | dansmith: indeed. | 17:57 |
| cburgess | Now I have this image of a small dansmith sitting a little desk -2ing requests from his mother. | 17:57 |
| dansmith | cburgess: not far off | 17:57 |
| jaypipes | lmao | 17:57 |
| dims | cburgess : LOL | 17:59 |
| mikal | "Press 1 to ask about my grades, press 2 to complain about my girlfriend, press 3 to..." | 18:00 |
| *** Tom_ has joined #openstack-nova | 18:00 | |
| *** ralonsoh has quit IRC | 18:01 | |
| cburgess | mikal lol | 18:01 |
| *** itlinux_ has quit IRC | 18:01 | |
| *** harlowja has quit IRC | 18:02 | |
| mikal | "Press 4 to ask what that loud fan noise coming from the garage is" | 18:03 |
| openstackgerrit | Merged openstack/nova master: Make xen unit tests work with os-xenapi>=0.3.0 https://review.openstack.org/500968 | 18:03 |
| mikal | Answer, by the way, its a loud fan | 18:03 |
| dansmith | heh | 18:04 |
| dansmith | I did have a server catch fire in my room one time | 18:04 |
| mikal | Haven't we all... | 18:04 |
| dansmith | still have the (remnants of the) motherboard | 18:04 |
| mikal | Anyways, its time for me to brave LA traffic and go to the airport. | 18:04 |
| mikal | I shall say sweet sweet lies about you all at the opendev thing | 18:05 |
| mikal | "Oh yes, they're quite competent. Have you asked them how they feel about remote compute nodes?" | 18:05 |
| *** Tom_ has quit IRC | 18:05 | |
| mikal | "They love that stuff, want more of it. Please send many patches." | 18:05 |
| cburgess | lol | 18:05 |
| mikal | "Ask for Dan, he'll help" | 18:06 |
| melwitt | you're going to opendev? so am I | 18:06 |
| * dansmith scowls at mikal and cburgess | 18:07 | |
| cburgess | dansmith Wait what did I do? | 18:08 |
| dansmith | cburgess: you encouraged him by laughing at his jokes | 18:08 |
| cburgess | dansmith Oh that... | 18:08 |
| *** egonzalez has joined #openstack-nova | 18:10 | |
| openstackgerrit | Merged openstack/nova master: doc: Remove deprecated call to sphinx.util.compat https://review.openstack.org/498824 | 18:14 |
| *** ssmith has quit IRC | 18:17 | |
| *** Swami has joined #openstack-nova | 18:21 | |
| *** thorst has quit IRC | 18:22 | |
| *** thorst has joined #openstack-nova | 18:23 | |
| *** ijw has joined #openstack-nova | 18:24 | |
| *** ijw has quit IRC | 18:26 | |
| *** ijw has joined #openstack-nova | 18:26 | |
| mnaser | dansmith / mriedem: https://bugs.launchpad.net/nova/+bug/1715462 | 18:26 |
| openstack | Launchpad bug 1715462 in OpenStack Compute (nova) "Instances failing quota recheck end up with no assigned cell" [Undecided,New] | 18:26 |
| *** thorst has quit IRC | 18:27 | |
| sdague | cburgess: what's your biggest concern on privsep? | 18:28 |
| sdague | I apparently didn't see that in the patch (or glossed past it) | 18:28 |
| mnaser | i guess a test to make sure that a cell is assigned if the quota recheck fails would be #1 and then the fix to show that its working would be the correct path? | 18:28 |
| dansmith | mnaser: that'd be ideal yeah | 18:28 |
| mnaser | ok, ill try to work on a test first | 18:28 |
| cburgess | sdague I haven't reviewed the patch this was a review mikal and I did verbally in my dinning room this morning. | 18:28 |
| *** krtaylor has quit IRC | 18:28 | |
| *** thorst has joined #openstack-nova | 18:29 | |
| sdague | cburgess: were mimosas involved? | 18:29 |
| cburgess | sdague I have a few concerns but the big one is around how the privsep daemons get started. I would prefer that if a nova component detects that there is no daemon it just started/restarted it. Having to restart the entire compute/conductor/whatever process is faily heavy weight just to get the privsep daemon running again. | 18:29 |
| cburgess | sdague Oh I wish.. that would have made it better | 18:30 |
| sdague | cburgess: yeah, the watchdogging seems reasonable | 18:30 |
| mnaser | dansmith mind if i bother you a bit as i figure out the best way to go about this with small questions? https://github.com/openstack/nova/blob/master/nova/tests/unit/conductor/test_conductor.py#L1722-L1760 -- would it be beneficial to just add an assert there instead of writing a whole new test? | 18:30 |
| *** jaosorior has joined #openstack-nova | 18:30 | |
| openstackgerrit | Merged openstack/nova master: trivial: Remove "vif" script https://review.openstack.org/491443 | 18:30 |
| *** hrw has quit IRC | 18:30 | |
| *** tesseract has quit IRC | 18:31 | |
| dansmith | mnaser: since that test is specifically aimed at checking that late quota check I think that's probably fine | 18:31 |
| *** krtaylor has joined #openstack-nova | 18:31 | |
| mnaser | alright cool, i'll give it a shot, lets hope it fails :> | 18:31 |
| openstackgerrit | Merged openstack/nova master: trivial: Remove files from 'tools' https://review.openstack.org/491444 | 18:31 |
| cburgess | sdague Cool so we agree and mikal did in theory when I went to the gym this morning but now he claims there might be issues and promises performance art in Denver. | 18:31 |
| *** thorst has quit IRC | 18:33 | |
| mnaser | woo, i have a failing test case | 18:36 |
| mnaser | can i get some advice on how would be the ideal way of where to move the cell_mapping saving part? should all of it be moved earlier or should i refactor it into a private fucntion such as _update_instance_mapping(instance, cell) and then call that in the exception handling part of the quota recheck? | 18:39 |
| openstackgerrit | Erik Berg proposed openstack/nova master: Fix binary name. https://review.openstack.org/501359 | 18:39 |
| mnaser | that way we don't risk introducing other weird problems that might come up from shuffling the order of things | 18:39 |
| dansmith | mnaser: this is a sticky place we have to be super careful | 18:42 |
| dansmith | give me a few to read | 18:42 |
| mnaser | dansmith: no problem! that's why i thought by refactoring it into a function and calling it in exception handling, we touch the *least* amount of codepath possible (but i'm sure folks know the codebase far more than me :-) | 18:43 |
| *** thorst has joined #openstack-nova | 18:46 | |
| dansmith | mnaser: so I think the right thing to do is really to map the instance right after we create it above | 18:46 |
| dansmith | mnaser: so that if we end up with any instance created (which will show up in a list) it'll have a corresponding map show that show will work | 18:46 |
| dansmith | s/show that/so that/ | 18:46 |
| mnaser | dansmith so right after the with(..) block spanning 988-990? | 18:47 |
| dansmith | mnaser: year | 18:48 |
| *** bauzas has quit IRC | 18:48 | |
| dansmith | heh, that was either yeah or yar | 18:48 |
| mnaser | dansmith in my research of the code, i found the _populate_instance_mapping function which seemed pretty robust at setting the instance mapping | 18:48 |
| *** bauzas has joined #openstack-nova | 18:48 | |
| mnaser | would it beneficial to use that instead? | 18:48 |
| *** ijw has quit IRC | 18:49 | |
| mnaser | we have the host in there so we can pass it (but i'll take what you think is best overall) | 18:49 |
| dansmith | mnaser: yeah if that works should be okay | 18:50 |
| *** tbachman has quit IRC | 18:50 | |
| mnaser | dansmith cool, i'll get on this and see if it affects any other tests as well | 18:51 |
| dansmith | cool | 18:51 |
| *** jaosorior has quit IRC | 18:53 | |
| mnaser | yay, that specific test is passing now, i'll just rerun all the conductor tests because that code shuffle might have affected other tests | 18:55 |
| *** esberglu has quit IRC | 18:55 | |
| *** gvrangan has quit IRC | 18:55 | |
| *** esberglu has joined #openstack-nova | 18:55 | |
| dansmith | yeah, entirely possible because that extra thing isn't mocked out now | 18:55 |
| *** shan has quit IRC | 18:56 | |
| *** ijw has joined #openstack-nova | 18:57 | |
| *** tidwellr has quit IRC | 18:59 | |
| mnaser | only one test failing after that.. a bit less painless than i expected :> | 18:59 |
| dansmith | run functional tests too? | 18:59 |
| *** esberglu has quit IRC | 19:00 | |
| *** ijw has quit IRC | 19:02 | |
| *** ijw has joined #openstack-nova | 19:03 | |
| *** catinthe_ has joined #openstack-nova | 19:04 | |
| *** avolkov has quit IRC | 19:05 | |
| mnaser | nope, didnt do that yet, just the uni test of conductor | 19:05 |
| dansmith | might scare up another failure or two in there depending | 19:05 |
| *** catintheroof has quit IRC | 19:06 | |
| *** catintheroof has joined #openstack-nova | 19:06 | |
| dansmith | mnaser: I have to head to the airport in a few but will be back online from there | 19:06 |
| *** esberglu has joined #openstack-nova | 19:06 | |
| mnaser | dansmith np, thank you for your help so far | 19:06 |
| dansmith | np | 19:06 |
| openstackgerrit | Merged openstack/nova master: doc: Add user index page https://review.openstack.org/498817 | 19:07 |
| *** tidwellr has joined #openstack-nova | 19:08 | |
| openstackgerrit | Merged openstack/nova master: doc: Add configuration index page https://review.openstack.org/498818 | 19:08 |
| *** ijw has quit IRC | 19:08 | |
| *** catinthe_ has quit IRC | 19:10 | |
| *** moshele has joined #openstack-nova | 19:10 | |
| sdague | hmmmm.... https://bugs.launchpad.net/nova/+bug/1715463 doesn't seem good | 19:15 |
| openstack | Launchpad bug 1715463 in OpenStack Compute (nova) "binary name gets confused under upgrades of osapi_compute and metadata" [High,Incomplete] - Assigned to Ebbex (eb4x) | 19:15 |
| *** rajathagasthya has quit IRC | 19:15 | |
| *** tbachman has joined #openstack-nova | 19:16 | |
| *** moshele has quit IRC | 19:16 | |
| cdent | sdague: the fix didn’t get linked to the bug yet (because the bug came after the code): https://review.openstack.org/#/c/501359/ | 19:21 |
| cdent | it’s effectively a logic problem in the code | 19:22 |
| cdent | name change in the wrong place | 19:22 |
| sdague | yeh, so this the deployment change after the upgrade, so they come up wsgi for the first time? | 19:22 |
| *** abhi89 has quit IRC | 19:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails https://review.openstack.org/499878 | 19:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass migration from API to conductor for evacuate https://review.openstack.org/500176 | 19:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation https://review.openstack.org/499877 | 19:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure https://review.openstack.org/499874 | 19:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Create allocations against forced dest host during evacuate https://review.openstack.org/499399 | 19:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Refactor out claim_resources_on_destination into a utility https://review.openstack.org/499718 | 19:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Modernize set_vm_state_and_notify https://review.openstack.org/499799 | 19:23 |
| cdent | sdague: dunno | 19:26 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 19:28 |
| *** itlinux has joined #openstack-nova | 19:29 | |
| ildikov | mriedem: hi | 19:29 |
| mriedem | o/ | 19:30 |
| ildikov | mriedem: just wanted to ask how to move forward with the Cinder-Nova open reviews? | 19:30 |
| mriedem | looks like that dependent cinderclient change is released and in g-r | 19:31 |
| *** crushil has quit IRC | 19:31 | |
| mriedem | so i can start looking at https://review.openstack.org/#/c/493323/ later | 19:31 |
| ildikov | mriedem: yep, the cinderclient is all set and the test runs looked clean so far | 19:32 |
| ildikov | mriedem: I also uploaded the specs and left the multi-attach in WIP for now, but happy to get feedback on both | 19:33 |
| *** ijw has joined #openstack-nova | 19:33 | |
| *** itlinux has quit IRC | 19:33 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken link https://review.openstack.org/501391 | 19:34 |
| *** pcaruana has quit IRC | 19:34 | |
| *** rmcall has joined #openstack-nova | 19:39 | |
| *** ijw has quit IRC | 19:39 | |
| *** ijw has joined #openstack-nova | 19:40 | |
| *** lbragstad has quit IRC | 19:43 | |
| *** pchavva has quit IRC | 19:44 | |
| *** ijw has quit IRC | 19:45 | |
| *** lbragstad has joined #openstack-nova | 19:45 | |
| mnaser | conductor tests all passing with that change, functional are all passing so far so hopefully i can push this up soon :> | 19:46 |
| *** kuzko has quit IRC | 19:48 | |
| *** kuzko has joined #openstack-nova | 19:50 | |
| *** awaugama has quit IRC | 19:51 | |
| *** annegentle has quit IRC | 19:51 | |
| *** suresh12 has quit IRC | 19:53 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix test_rpc_consumer_isolation for oslo.messaging 5.31.0 https://review.openstack.org/501400 | 19:55 |
| *** suresh12 has joined #openstack-nova | 19:55 | |
| *** ijw has joined #openstack-nova | 19:58 | |
| *** Apoorva has quit IRC | 20:00 | |
| *** tylerderosagrund has quit IRC | 20:00 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025 | 20:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/501343 | 20:00 |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Fix broken URLs https://review.openstack.org/501402 | 20:02 |
| *** eharney has quit IRC | 20:04 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova stable/pike: Fix broken link https://review.openstack.org/501403 | 20:05 |
| *** kristian__ has joined #openstack-nova | 20:05 | |
| *** kristian__ has quit IRC | 20:07 | |
| *** sridharg has quit IRC | 20:08 | |
| *** kristia__ has quit IRC | 20:08 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:09 | |
| *** crushil has joined #openstack-nova | 20:09 | |
| openstackgerrit | Merged openstack/nova master: HyperV: Perform proper cleanup after failed instance spawns https://review.openstack.org/499690 | 20:10 |
| *** dave-mccowan has quit IRC | 20:11 | |
| *** liverpooler has quit IRC | 20:11 | |
| *** felipemonteiro__ has quit IRC | 20:12 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 20:12 |
| mnaser | dansmith ^ | 20:12 |
| dansmith | wooo | 20:12 |
| dansmith | melwitt: ^ | 20:13 |
| melwitt | sweet | 20:13 |
| *** Apoorva has joined #openstack-nova | 20:14 | |
| melwitt | thanks for running with that mnaser | 20:15 |
| mnaser | melwitt np! :) | 20:15 |
| *** vvargaszte has joined #openstack-nova | 20:16 | |
| *** vvargaszte has quit IRC | 20:16 | |
| *** suresh12 has quit IRC | 20:20 | |
| *** rajathagasthya has joined #openstack-nova | 20:21 | |
| *** tinwood has quit IRC | 20:22 | |
| *** tinwood has joined #openstack-nova | 20:23 | |
| sdague | looking at old patches, is this still a thing - https://review.openstack.org/#/c/375400/ ? | 20:24 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 20:24 |
| *** annegentle has joined #openstack-nova | 20:25 | |
| *** suresh12 has joined #openstack-nova | 20:26 | |
| *** vladikr has quit IRC | 20:26 | |
| mriedem | sdague: yeah that doesn't seem worth it right now, plus yeah we don't want to touch older release notes if we can help it | 20:27 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 20:27 |
| *** smatzek has quit IRC | 20:29 | |
| *** awaugama has joined #openstack-nova | 20:31 | |
| *** jmlowe_ has quit IRC | 20:33 | |
| *** egonzalez has quit IRC | 20:33 | |
| mriedem | mnaser: melwitt: dansmith: one concern in that change related to mapping the instance before we create the bdms/tags in the cell | 20:33 |
| mriedem | this is why this gets all really gorpy | 20:34 |
| *** jmlowe has joined #openstack-nova | 20:34 | |
| dansmith | mriedem: yeah I was thinking about that, but if we just have the buildrequest we have less info visible right | 20:34 |
| * dansmith actually reads the comment | 20:34 | |
| melwitt | yeah, I was similarly concerned but not sure if it's a problem yet | 20:34 |
| *** Apoorva_ has joined #openstack-nova | 20:36 | |
| dansmith | won't we keep using the buildrequest if present? | 20:36 |
| dansmith | like, that's the lock we use to say "okay now you can look at the instance" | 20:36 |
| dansmith | like, when we remove the buildrequest, I mean | 20:36 |
| *** vvargaszte has joined #openstack-nova | 20:37 | |
| mriedem | no we look for the instance mapping first | 20:38 |
| mriedem | https://github.com/openstack/nova/blob/b79492f70257754f960eaf38ad6a3f56f647cb3d/nova/compute/api.py#L2240 | 20:38 |
| dansmith | ah, and use the cell_mapping I guess | 20:39 |
| *** dave-mccowan has joined #openstack-nova | 20:39 | |
| *** vladikr has joined #openstack-nova | 20:39 | |
| mriedem | so in the before times, you couldn't apply tags to a server until it was active, | 20:40 |
| mriedem | but in pike you can create a server with tags now | 20:40 |
| *** Apoorva has quit IRC | 20:40 | |
| dansmith | mriedem: do we need to wait until later to do that? couldn't we do those two steps after the create before this save? and then check the quota? | 20:40 |
| mriedem | idk | 20:41 |
| dansmith | alternately we could just do this if we fail the quota check, it just seemed better to me to do it in fewer places | 20:41 |
| mriedem | this is all wonky why it's all done in two steps | 20:41 |
| dansmith | yeah | 20:41 |
| mnaser | added a docstring while we figure out the rest :> | 20:41 |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 20:41 |
| mriedem | although, | 20:42 |
| mriedem | it was split into two parts due to melwitt's quota check change | 20:42 |
| *** lucasxu has quit IRC | 20:42 | |
| melwitt | I didn't change the 2-partness for the quota check | 20:42 |
| mriedem | https://review.openstack.org/#/c/416521/ | 20:42 |
| mriedem | melwitt: i mean the 2 loops over the tuple | 20:43 |
| mriedem | that was a single for loop before | 20:43 |
| melwitt | oh, I see (looking at the patch now) | 20:43 |
| dansmith | hm why is that? | 20:43 |
| mriedem | have to create them all to count | 20:44 |
| mriedem | and then continue creating stuff | 20:44 |
| melwitt | at the time I was thinking, just check all of the quota first before creating a bunch of other resources if it's just gonna possibly fail in the middle | 20:44 |
| dansmith | why not do that at the end I mean | 20:44 |
| dansmith | yeah, but, then we're kinda checking the quota without everything created, which is a little odd | 20:44 |
| melwitt | we could. I was probably just thinking it would be less wasteful, why create BDMs and all of that if it's gonna fail the recheck anyway | 20:44 |
| dansmith | I mean, I guess it's just checking instances | 20:45 |
| melwitt | yeah | 20:45 |
| mriedem | if we did just quota check at the end, then we also have to cleanup the bdms and tags | 20:45 |
| mriedem | in addition to the stuff that _cleanup_build_artifacts is already removing | 20:45 |
| dansmith | do we? | 20:45 |
| dansmith | we delete the instance.. are those things cleaned up by compute normally or something? | 20:46 |
| mriedem | it wouldn't be mapped to a host | 20:46 |
| mriedem | so no compute involved | 20:46 |
| dansmith | no, | 20:46 |
| dansmith | this would be like a local delete | 20:46 |
| dansmith | because we obviously need not call to compute in this case, so I'm asking in a normal local delete where compute is down, | 20:46 |
| dansmith | do we have to clean up bdms and tags separately? | 20:47 |
| mriedem | no, don't think so, those get removed via the db api | 20:47 |
| mriedem | when you delete the instance | 20:47 |
| *** slaweq has quit IRC | 20:47 | |
| dansmith | instance delete will delete BDMs | 20:47 |
| mriedem | https://github.com/openstack/nova/blob/b79492f70257754f960eaf38ad6a3f56f647cb3d/nova/db/sqlalchemy/api.py#L1886 | 20:48 |
| dansmith | and tags | 20:48 |
| mriedem | https://github.com/openstack/nova/blob/b79492f70257754f960eaf38ad6a3f56f647cb3d/nova/db/sqlalchemy/api.py#L1894 | 20:48 |
| dansmith | yeah | 20:48 |
| dansmith | so when you delete the instance after the quota check those will get cleaned up if we have created them right? | 20:48 |
| mriedem | well we don't delete the instance here, | 20:49 |
| mriedem | we just set it to ERROR state | 20:49 |
| mriedem | and then the user does the (local) delete | 20:49 |
| dansmith | right | 20:50 |
| mriedem | which should cleanup the bdms/tags | 20:50 |
| openstackgerrit | Merged openstack/nova master: Fix broken link https://review.openstack.org/501391 | 20:50 |
| mriedem | well, the bdms and tags wouldn't even exist in the cell db | 20:50 |
| mriedem | if the quota check fails | 20:50 |
| mriedem | oh but if we moved it to the end.. | 20:50 |
| mriedem | yeah i think that would work | 20:51 |
| *** prometheanfire has left #openstack-nova | 20:51 | |
| dansmith | we can also do what mnaser wanted to do in the first place, which is just do this quick map fixup if we fail this quota check, | 20:52 |
| melwitt | cells meeting in 8 minutes? | 20:52 |
| dansmith | but then we will have an instance without those things in error state, and we'll be doing it from yet another place | 20:52 |
| *** lpetrut has quit IRC | 20:52 | |
| dansmith | I kinda want to keep it less spaghetti-like and do it in one place, personally | 20:52 |
| *** annegentle has quit IRC | 20:52 | |
| dansmith | melwitt: I'm in an airport so we said this morning we'd cancel | 20:52 |
| melwitt | oh, cool | 20:53 |
| dansmith | if you have things to discuss we can, because I'm clearly connected, but, I didn't see the point | 20:53 |
| melwitt | no, just forgot to check the log | 20:53 |
| dansmith | ack | 20:53 |
| mriedem | another alternative is just don't delete the build request in _cleanup_build_artifacts, or is that crazy? | 20:54 |
| dansmith | no, I think we can't do that | 20:54 |
| mriedem | the problem is that we haven't mapped the instance to a cell, and _cleanup_build_artifacts removes the build request, so the API literally can't find it anywhere | 20:54 |
| mriedem | yeah? | 20:54 |
| dansmith | or you mean in the failure case only? | 20:54 |
| mriedem | yes | 20:54 |
| *** chyka has quit IRC | 20:54 | |
| *** chyka_ has joined #openstack-nova | 20:54 | |
| mriedem | but then you'd have a build request in the api and an instance in the cell, unmapped | 20:55 |
| dansmith | mriedem: we wouldn't know where to find the instance record to mark it as deleted when they deleted the buildreq, so we'd leave that undeleted but unfindable instance forever | 20:55 |
| mriedem | and that instance in the cell db wouldn't get removed, | 20:55 |
| mriedem | unless _cleanup_build_artifacts removes it | 20:55 |
| dansmith | right | 20:55 |
| *** vvargaszte has quit IRC | 20:55 | |
| mriedem | so, change _cleanup_build_artifacts to delete the instance from the cell and leave the build request for API requests? | 20:55 |
| mriedem | unless we just overhaul this code to remove the 2 loops and do the quota check at the end | 20:56 |
| melwitt | doing that would remove the instance fault message which explains the quota check failed | 20:56 |
| mriedem | melwitt: can't the build request have a fault on it? | 20:56 |
| dansmith | and we have the instance in the cell db, that's better than a buildreq, so we should use it | 20:56 |
| melwitt | I didn't know it can | 20:56 |
| dansmith | mriedem: I don't think we ever update that instance do we? | 20:56 |
| dansmith | because it's fake right? | 20:56 |
| mriedem | build request doesn't have a fault field, so yeah, can't do that | 20:57 |
| melwitt | it has an instance field which would have a fault field I guess | 20:57 |
| dansmith | no, I think the instance is fake and constructed on the fly | 20:57 |
| melwitt | oh, okay | 20:57 |
| mriedem | the BuildRequest.instance is persisted | 20:58 |
| mriedem | it's just serialized json | 20:58 |
| mriedem | but, that starts to get weird... | 20:59 |
| dansmith | yeah, I guess so | 20:59 |
| mriedem | managing state on an instance nested in a build requst | 20:59 |
| dansmith | I'm not sure why that's better though | 20:59 |
| dansmith | yeah | 20:59 |
| dansmith | this makes a bunch of other regular code just be used for anything else you do to this dead instance | 20:59 |
| mriedem | it's only better if we want to keep the 2 loops b/c we're afraid of changing them | 20:59 |
| *** chyka_ has quit IRC | 21:00 | |
| dansmith | I ain't skeered | 21:00 |
| mriedem | dude, | 21:00 |
| mriedem | i shit my pants everytime we touch nova these days | 21:00 |
| dansmith | hah | 21:00 |
| mnaser | (which is totally why i thought putting a tiny little call/fixup in the exception handling = less scary | 21:01 |
| mnaser | :p | 21:01 |
| melwitt | well, doesn't moving the quota check to the end and having only one loop address the issue fine since we know the BDMs and tags will get cleaned up at delete time? | 21:01 |
| mriedem | melwitt: i think so | 21:01 |
| melwitt | so the fake instance thing isn't the only way to get rid of the two loop thing | 21:01 |
| dansmith | mnaser: yeah, but we already have lots of "clean this up in this snowflake way" calls | 21:01 |
| *** krtaylor has quit IRC | 21:01 | |
| *** kuzko has quit IRC | 21:01 | |
| mriedem | a test that creates an instance with a bdm and tag and asserts those are removed from the db would make me feel safer | 21:01 |
| dansmith | melwitt: the fake instance is the way to keep the two loops he said | 21:01 |
| dansmith | melwitt: fearing change | 21:02 |
| *** eharney has joined #openstack-nova | 21:02 | |
| mriedem | i fear the landmines | 21:02 |
| mriedem | i have literally 2 toes left | 21:02 |
| *** dave-mcc_ has joined #openstack-nova | 21:02 | |
| *** mriedem has quit IRC | 21:02 | |
| melwitt | oh. well, my thinking is the two loops are new and it's probably better to put it back to one now that we know we can | 21:02 |
| dansmith | the easier way to do that is just do what mnaser wanted to do in the first place | 21:02 |
| dansmith | melwitt: I agree | 21:02 |
| dansmith | melwitt: and also, the fake instance in a build request is a special case and we should have that oooonly in the case where we haven't created the instance yet or can't, | 21:03 |
| *** mriedem has joined #openstack-nova | 21:03 | |
| mnaser | for me it's the fact that it makes it more comfortable to backport this sort of thing (which i hope we can do to stable/pike) | 21:03 |
| dansmith | so putting more things in that basket when we already have an instance created in a real cell seems wrong to me | 21:03 |
| dansmith | mnaser: right, so, | 21:03 |
| dansmith | I was going to suggest: | 21:03 |
| dansmith | how about we let mnaser do the simpler but uglier thing of only mapping if we hit that quota check, | 21:03 |
| melwitt | dansmith: yeah, agreed | 21:03 |
| mriedem | and then collapse the loops in queens only | 21:03 |
| mriedem | i like that | 21:03 |
| dansmith | backport that mofo' and then do this cleanup separately with a functional test to get rid of the two loop | 21:03 |
| dansmith | yeah | 21:04 |
| mnaser | i think that would be best: so that stable can remain stable and you have plenty of time to review a much bigger change like that :> | 21:04 |
| mnaser | and you're not worried about stable breaking | 21:04 |
| *** kuzko has joined #openstack-nova | 21:04 | |
| mriedem | mnaser: btw, how is it that vexxhost is already on pike code? | 21:04 |
| dansmith | I think that's reasonable | 21:04 |
| *** dave-mccowan has quit IRC | 21:05 | |
| dansmith | mriedem: you mean "thank you for already being on pike" right? | 21:05 |
| mnaser | mriedem puppet+rdo packages, trusting the openstack CI and puppet-openstack-integration we use | 21:05 |
| mnaser | aha | 21:05 |
| mriedem | dansmith: yes | 21:05 |
| dansmith | thought so | 21:05 |
| dansmith | :) | 21:05 |
| mriedem | i'm just impressed and happy i mean | 21:05 |
| mnaser | honestly it was an easy upgrade | 21:05 |
| *** chyka has joined #openstack-nova | 21:05 | |
| melwitt | yeah, it's awesome ppl are running pike | 21:05 |
| mnaser | newton->ocata was a bit of uh a mess | 21:05 |
| mnaser | 400,000 records being mapped in the new default cell? | 21:05 |
| mriedem | well, placement and cells v2 | 21:05 |
| mnaser | takes a little while :p | 21:06 |
| *** chyka has quit IRC | 21:06 | |
| mnaser | i backported the placement puppet manifests so we had placement running in newton so it was *one less thing* to worry about | 21:06 |
| mriedem | smart | 21:06 |
| dansmith | https://twitter.com/get_offmylawn/status/905537805903941633 | 21:07 |
| mnaser | :> | 21:07 |
| melwitt | f yeah | 21:07 |
| mnaser | also well | 21:08 |
| mnaser | running vanilla openstack means we don't much around a lot :) | 21:08 |
| *** chyka has joined #openstack-nova | 21:09 | |
| *** cleong has quit IRC | 21:09 | |
| jaypipes | mnaser: ++ :) | 21:09 |
| mnaser | so do we come to quorum to leave things as is except call set cell mapping in the exception handling code? | 21:09 |
| melwitt | running vanilla openstack is the way to go. I have been part of running custom openstack and regretted doing it that way | 21:09 |
| dansmith | mnaser: yeah | 21:09 |
| mnaser | or i'll add a note as well | 21:10 |
| mriedem | TODO i'd say | 21:11 |
| mnaser | damn | 21:11 |
| mriedem | you'll also have to keep track of which cells each instance goes into | 21:11 |
| mnaser | um | 21:11 |
| mnaser | i dont know what cell | 21:11 |
| mnaser | yeah | 21:11 |
| mnaser | i'll have to figure out a clean way to do that | 21:11 |
| mriedem | just save that into a dict in the first loop | 21:11 |
| mnaser | alright | 21:11 |
| *** slaweq has joined #openstack-nova | 21:12 | |
| *** itlinux has joined #openstack-nova | 21:12 | |
| mriedem | everything already goes into that instances list variable, and is sometimes None for things that failed, but for things that did get mapped, i think you could map the instance uuid to the cell in a dict and then use those 2 variables to do the mapping in the failure block | 21:12 |
| mriedem | s/mapped/created/ | 21:12 |
| *** chyka_ has joined #openstack-nova | 21:13 | |
| mriedem | hmm, i think i just realized another bug here | 21:13 |
| dansmith | um | 21:14 |
| *** chyka has quit IRC | 21:14 | |
| dansmith | hah yesh | 21:14 |
| mriedem | we're always mapping to the last cell | 21:14 |
| dansmith | yeah | 21:14 |
| mriedem | :) | 21:14 |
| *** Apoorva_ has quit IRC | 21:14 | |
| dansmith | oof | 21:14 |
| * mriedem reports a new bug | 21:14 | |
| *** Apoorva has joined #openstack-nova | 21:14 | |
| *** gouthamr has quit IRC | 21:15 | |
| dansmith | mnaser: so you might want to stack your change on top of the fix for that bug | 21:15 |
| dansmith | which I can cook up | 21:15 |
| mnaser | okay cool, ill work on the stuff to set mappings | 21:15 |
| *** krtaylor has joined #openstack-nova | 21:16 | |
| dansmith | mriedem: I'm guessing that this would have uncovered that if I had ever finished it: https://review.openstack.org/#/c/452006/ | 21:17 |
| dansmith | ah, no, | 21:18 |
| dansmith | that's the migrate one | 21:18 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1715493 | 21:18 |
| openstack | Launchpad bug 1715493 in OpenStack Compute (nova) "Instances always get mapped into the last processed cell in conductor" [High,Triaged] | 21:18 |
| *** chyka_ has quit IRC | 21:19 | |
| dansmith | I guess this bug came from splitting that loop amirite? | 21:19 |
| mriedem | yup | 21:19 |
| *** tidwellr has quit IRC | 21:20 | |
| mriedem | so when we combine we can remove both workarounds | 21:20 |
| dansmith | yeah | 21:20 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 21:20 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 21:20 |
| melwitt | ugh :( | 21:20 |
| mriedem | plus, if mnaser's patch creates a instance -> cell mapping dict to keep track, the 2nd loop could use that too | 21:20 |
| dansmith | yeah | 21:20 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | 21:20 |
| dansmith | gonna work on tests, but pushed this up in case I have to run ^ | 21:21 |
| * mnaser wil rebase on that | 21:21 | |
| *** yangyapeng has joined #openstack-nova | 21:22 | |
| *** chyka has joined #openstack-nova | 21:23 | |
| dansmith | hmm, we kinda have a test for this, I'm not sure why it's not failing | 21:25 |
| *** gbarros has joined #openstack-nova | 21:25 | |
| *** yangyapeng has quit IRC | 21:27 | |
| mnaser | dansmith is it okay that when i try to do git review with your patch below mine, it mentions that it will submit two commits? | 21:29 |
| * mnaser is not an expert at stacking gerrit changes | 21:29 | |
| *** crushil has quit IRC | 21:29 | |
| dansmith | mnaser: yep | 21:29 |
| mnaser | okay cool | 21:29 |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 21:29 |
| mnaser | oh it didnt send both, nice. that's leveraging your cell_instance_cache | 21:30 |
| mnaser | oh i should update the commit msg | 21:30 |
| melwitt | mnaser: I usually double check to make sure the commit hash of the dependent change is the same as what shows on the review being rebased upon | 21:30 |
| *** crushil has joined #openstack-nova | 21:30 | |
| melwitt | that's how you can tell whether it will push more than just your change | 21:30 |
| mnaser | melwitt oh, if it matches it wont submit it? i'm just terrified of the embarassement that happens sometimes when an irc channel gets spammed :p | 21:30 |
| mriedem | mnaser: git review -R also avoids the rebase of the base changes | 21:31 |
| melwitt | mnaser: yeah. if the hash hasn't changed it won't submit it. so you can know before you do it | 21:31 |
| mriedem | melwitt: btw i think i figured out the git review / rebase author change thing, | 21:31 |
| mriedem | it happens when rebasing on a series that involves a merge conflict, | 21:31 |
| mriedem | so you rebase, merge conflict, fix, git add / commit, git rebase continue | 21:31 |
| mriedem | and that has changed the author | 21:31 |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 21:32 |
| melwitt | weird. I feel like I've rebased other ppl's changes with merge conflict and it didn't change the author before | 21:32 |
| mnaser | there, that should have most of it covered | 21:32 |
| melwitt | but, that's great data to go on. I'll pay attention next time | 21:32 |
| dansmith | okay, the test I thought we had was build not schedule_and_build | 21:33 |
| *** Apoorva_ has joined #openstack-nova | 21:33 | |
| dansmith | I have to board soon, but I can work on tests for this on the plane unless someone else wants to? | 21:33 |
| mnaser | i'd help but it sits outside my relm of knowledge :) have a safe flight though dansmith | 21:34 |
| dansmith | mnaser: np, thanks | 21:35 |
| mnaser | time to go figure out why horizon pike is logging users out now :> thanks for the help everyone, i'll keep an eye out on reviews for the patch | 21:35 |
| mriedem | mnaser: one issue in your patch | 21:36 |
| mnaser | sure whats up | 21:36 |
| mnaser | oh /me looks | 21:36 |
| mriedem | comment inline | 21:36 |
| melwitt | dansmith: I can write the tests for your change if you want to not do it on the plane | 21:36 |
| *** Apoorva has quit IRC | 21:37 | |
| mnaser | mriedem took me a while to figure out what you meant till i realized that super hidden for loop in the top lol | 21:38 |
| mriedem | :) everything else looks good | 21:39 |
| mriedem | sdague: low hanging fruit review https://review.openstack.org/#/c/499001/ | 21:42 |
| mriedem | skip more cells v1 random failure tests | 21:42 |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 21:42 |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 21:44 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 21:44 |
| *** harlowja has joined #openstack-nova | 21:45 | |
| openstackgerrit | Merged openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166 | 21:45 |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 21:48 |
| *** priteau__ has quit IRC | 21:50 | |
| openstackgerrit | Chris Dent proposed openstack/nova-specs master: Add a spec for POST /allocations in placement https://review.openstack.org/499259 | 21:52 |
| *** edmondsw has quit IRC | 21:52 | |
| *** cdent has quit IRC | 21:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add release note for force live migration allocations https://review.openstack.org/501477 | 21:55 |
| *** catintheroof has quit IRC | 21:56 | |
| *** dave-mcc_ has quit IRC | 21:57 | |
| *** lyan has quit IRC | 21:58 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 21:58 |
| *** priteau has joined #openstack-nova | 21:58 | |
| *** chyka has quit IRC | 22:02 | |
| *** priteau has quit IRC | 22:03 | |
| *** ijw has quit IRC | 22:03 | |
| *** awaugama has quit IRC | 22:04 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | 22:05 |
| *** ijw has joined #openstack-nova | 22:05 | |
| *** xyang1 has quit IRC | 22:09 | |
| *** erlon has quit IRC | 22:11 | |
| *** ijw has quit IRC | 22:11 | |
| *** suresh12 has quit IRC | 22:13 | |
| mriedem | melwitt: i'm going to step away to make dinner but if you get that test written ping me and i'll take a look tonight | 22:15 |
| melwitt | mriedem: okay. I wasn't sure if dansmith got my message before boarding. so I wasn't sure if he's already working on it on the plane | 22:16 |
| *** kylek3h has quit IRC | 22:18 | |
| *** baoli has quit IRC | 22:18 | |
| *** edmondsw has joined #openstack-nova | 22:23 | |
| *** suresh12 has joined #openstack-nova | 22:24 | |
| *** jaypipes has quit IRC | 22:24 | |
| *** edmondsw has quit IRC | 22:28 | |
| *** suresh12 has quit IRC | 22:29 | |
| *** ijw has joined #openstack-nova | 22:35 | |
| *** JoseMello has quit IRC | 22:35 | |
| *** krtaylor has quit IRC | 22:37 | |
| *** slaweq has quit IRC | 22:37 | |
| *** Apoorva_ has quit IRC | 22:40 | |
| *** Apoorva has joined #openstack-nova | 22:41 | |
| *** suresh12 has joined #openstack-nova | 22:42 | |
| mnaser | do live migration jobs often fail | 22:46 |
| mnaser | or did i break something :> | 22:46 |
| *** vladikr has quit IRC | 22:47 | |
| mnaser | Details: (LiveAutoBlockMigrationV225Test:test_live_block_migration) Server 44884988-d3f2-4e32-a367-4962acda6346 failed to reach ACTIVE status and task state "None" within the required time (196 s). Current status: BUILD. Current task state: scheduling. | 22:47 |
| *** penick has quit IRC | 22:50 | |
| *** gouthamr has joined #openstack-nova | 22:52 | |
| *** thegreenhundred has quit IRC | 22:55 | |
| *** gbarros has quit IRC | 22:55 | |
| *** scottda_ has quit IRC | 23:00 | |
| *** vladikr has joined #openstack-nova | 23:00 | |
| *** felipemonteiro_ has quit IRC | 23:01 | |
| *** Apoorva_ has joined #openstack-nova | 23:01 | |
| *** cdent has joined #openstack-nova | 23:04 | |
| *** Apoorva has quit IRC | 23:05 | |
| *** ijw has quit IRC | 23:07 | |
| *** cdent has quit IRC | 23:08 | |
| *** felipemonteiro has joined #openstack-nova | 23:09 | |
| *** marst has quit IRC | 23:11 | |
| *** itlinux has quit IRC | 23:16 | |
| openstackgerrit | Merged openstack/nova master: Amend uuid4 hacking rule https://review.openstack.org/488315 | 23:21 |
| openstackgerrit | Merged openstack/nova stable/pike: Fix broken link https://review.openstack.org/501403 | 23:21 |
| *** kylek3h has joined #openstack-nova | 23:23 | |
| *** suresh12 has quit IRC | 23:26 | |
| *** kylek3h has quit IRC | 23:29 | |
| *** Dave has quit IRC | 23:33 | |
| *** suresh12_ has joined #openstack-nova | 23:34 | |
| *** yangyapeng has joined #openstack-nova | 23:38 | |
| *** slaweq has joined #openstack-nova | 23:38 | |
| *** baoli has joined #openstack-nova | 23:38 | |
| *** baoli has quit IRC | 23:39 | |
| mriedem | mnaser: ah ./nova/conductor/manager.py:1017:20: F821 undefined name 'instance_mapping_cache' | 23:40 |
| *** baoli has joined #openstack-nova | 23:40 | |
| mnaser | oopsie | 23:40 |
| mriedem | that's a bug in dan's change | 23:40 |
| mnaser | oh thats not my path | 23:40 |
| mnaser | yeah | 23:40 |
| mriedem | i'll fix quick | 23:41 |
| *** hongbin has quit IRC | 23:42 | |
| *** hongbin has joined #openstack-nova | 23:43 | |
| *** yangyapeng has quit IRC | 23:43 | |
| *** slaweq has quit IRC | 23:43 | |
| *** hongbin has quit IRC | 23:44 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452 | 23:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408 | 23:44 |
| *** suresh12_ has quit IRC | 23:47 | |
| *** suresh12 has joined #openstack-nova | 23:48 | |
| *** Dave has joined #openstack-nova | 23:49 | |
| *** baoli has quit IRC | 23:51 | |
| openstackgerrit | Merged openstack/nova master: Add release note for force live migration allocations https://review.openstack.org/501314 | 23:58 |
| *** priteau has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!