*** randomhack_ has joined #openstack-nova | 00:04 | |
arvindn05 | alex_xu: had a question on your comments | 00:05 |
---|---|---|
arvindn05 | https://review.openstack.org/#/c/560596/ - update scheduler to use image-traits | 00:06 |
openstackgerrit | Merged openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION https://review.openstack.org/563984 | 00:06 |
*** moshele has joined #openstack-nova | 00:07 | |
openstackgerrit | Merged openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC https://review.openstack.org/564010 | 00:23 |
*** tbachman has joined #openstack-nova | 00:26 | |
*** namnh has joined #openstack-nova | 00:27 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:40 | |
*** jmlowe has quit IRC | 00:43 | |
*** jmlowe has joined #openstack-nova | 00:44 | |
*** yamahata has quit IRC | 00:49 | |
*** takashin has joined #openstack-nova | 01:04 | |
*** mlavalle has quit IRC | 01:05 | |
*** moshele has quit IRC | 01:07 | |
*** dklyle has quit IRC | 01:09 | |
*** dklyle has joined #openstack-nova | 01:09 | |
*** slaweq has joined #openstack-nova | 01:11 | |
*** dklyle has quit IRC | 01:11 | |
*** dklyle has joined #openstack-nova | 01:12 | |
*** slaweq has quit IRC | 01:16 | |
*** felipemonteiro__ has quit IRC | 01:18 | |
*** gjayavelu has quit IRC | 01:25 | |
*** bhujay has joined #openstack-nova | 01:27 | |
*** salv-orlando has joined #openstack-nova | 01:31 | |
*** salv-orlando has quit IRC | 01:36 | |
*** r-daneel has joined #openstack-nova | 01:39 | |
*** r-daneel_ has joined #openstack-nova | 01:42 | |
*** spsurya has joined #openstack-nova | 01:43 | |
*** r-daneel has quit IRC | 01:43 | |
*** r-daneel_ is now known as r-daneel | 01:43 | |
*** mriedem_away is now known as mriedem | 01:44 | |
openstackgerrit | Julia Kreger proposed openstack/nova master: WIP: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 01:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340 | 01:48 |
*** mriedem has quit IRC | 01:53 | |
*** Zames has joined #openstack-nova | 01:58 | |
*** Zames has quit IRC | 02:01 | |
*** Zames has joined #openstack-nova | 02:08 | |
*** fragatina has quit IRC | 02:08 | |
*** fragatina has joined #openstack-nova | 02:09 | |
*** fragatina has quit IRC | 02:09 | |
*** Zames has quit IRC | 02:10 | |
*** bkopilov has quit IRC | 02:10 | |
*** dave-mccowan has joined #openstack-nova | 02:12 | |
*** randomhack_ has quit IRC | 02:14 | |
*** dklyle has quit IRC | 02:15 | |
*** dklyle has joined #openstack-nova | 02:16 | |
*** dklyle has quit IRC | 02:22 | |
*** dklyle has joined #openstack-nova | 02:23 | |
*** dklyle has quit IRC | 02:24 | |
*** dklyle has joined #openstack-nova | 02:24 | |
*** nicolasbock has quit IRC | 02:25 | |
*** dklyle has quit IRC | 02:25 | |
*** dklyle has joined #openstack-nova | 02:26 | |
*** liverpooler has quit IRC | 02:27 | |
openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | 02:28 |
openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | 02:33 |
*** armaan has joined #openstack-nova | 02:34 | |
takashin | 02:38 | |
*** wolverineav has joined #openstack-nova | 02:40 | |
*** dikonoor has joined #openstack-nova | 02:41 | |
*** namnh has quit IRC | 02:43 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (1) https://review.openstack.org/564639 | 02:45 |
*** namnh has joined #openstack-nova | 02:49 | |
*** hongbin has joined #openstack-nova | 02:53 | |
*** hamzy has quit IRC | 02:54 | |
*** hamzy has joined #openstack-nova | 02:54 | |
*** namnh has quit IRC | 02:54 | |
*** slaweq has joined #openstack-nova | 02:56 | |
*** itlinux has joined #openstack-nova | 02:58 | |
*** slaweq has quit IRC | 03:01 | |
*** namnh has joined #openstack-nova | 03:06 | |
*** namnh has quit IRC | 03:10 | |
openstackgerrit | Merged openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414 | 03:14 |
*** dougshelley66 has quit IRC | 03:15 | |
*** wolverineav has quit IRC | 03:16 | |
*** wolverineav has joined #openstack-nova | 03:16 | |
*** wolverineav has quit IRC | 03:21 | |
*** fragatina has joined #openstack-nova | 03:23 | |
*** takashin has quit IRC | 03:26 | |
*** dklyle has quit IRC | 03:27 | |
*** dklyle has joined #openstack-nova | 03:28 | |
*** wolverineav has joined #openstack-nova | 03:29 | |
*** salv-orlando has joined #openstack-nova | 03:32 | |
*** itlinux has quit IRC | 03:32 | |
*** fragatina has quit IRC | 03:34 | |
*** bkopilov has joined #openstack-nova | 03:35 | |
*** salv-orlando has quit IRC | 03:37 | |
*** vladikr has quit IRC | 03:40 | |
*** vladikr has joined #openstack-nova | 03:41 | |
*** lbragstad has joined #openstack-nova | 03:49 | |
*** yamahata has joined #openstack-nova | 03:50 | |
*** dklyle has quit IRC | 03:50 | |
*** dklyle has joined #openstack-nova | 03:50 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add generation column to consumers table https://review.openstack.org/565400 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add Project object https://review.openstack.org/565401 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add User object to placement modeling https://review.openstack.org/565402 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add Consumer object https://review.openstack.org/565403 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406 | 03:54 |
openstackgerrit | Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407 | 03:54 |
*** yamamoto has quit IRC | 04:01 | |
*** dklyle has quit IRC | 04:01 | |
*** dklyle has joined #openstack-nova | 04:01 | |
*** yamamoto has joined #openstack-nova | 04:04 | |
*** yamamoto has quit IRC | 04:05 | |
*** hongbin has quit IRC | 04:08 | |
*** yamamoto has joined #openstack-nova | 04:15 | |
*** yamamoto has quit IRC | 04:20 | |
*** vladikr has quit IRC | 04:22 | |
*** vladikr has joined #openstack-nova | 04:23 | |
*** namnh has joined #openstack-nova | 04:26 | |
*** Zames has joined #openstack-nova | 04:30 | |
*** yamamoto has joined #openstack-nova | 04:31 | |
*** Zames has quit IRC | 04:32 | |
*** fragatina has joined #openstack-nova | 04:33 | |
*** salv-orlando has joined #openstack-nova | 04:33 | |
*** yamamoto has quit IRC | 04:34 | |
*** yamamoto has joined #openstack-nova | 04:34 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450 | 04:37 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | 04:37 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | 04:37 |
*** salv-orlando has quit IRC | 04:37 | |
*** fragatina has quit IRC | 04:40 | |
*** fragatina has joined #openstack-nova | 04:43 | |
*** dave-mccowan has quit IRC | 04:43 | |
*** sree has joined #openstack-nova | 04:47 | |
openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 04:48 |
*** fragatina has quit IRC | 04:49 | |
*** cfriesen has quit IRC | 04:50 | |
*** cfriesen_ has joined #openstack-nova | 04:50 | |
*** sree has quit IRC | 04:50 | |
*** vivsoni__ has joined #openstack-nova | 04:51 | |
*** lbragstad has quit IRC | 04:51 | |
*** sree has joined #openstack-nova | 04:51 | |
*** vladikr has quit IRC | 04:54 | |
*** sree has quit IRC | 04:55 | |
*** vladikr has joined #openstack-nova | 04:57 | |
*** wolverineav has quit IRC | 04:58 | |
*** wolverineav has joined #openstack-nova | 04:59 | |
*** salv-orlando has joined #openstack-nova | 05:01 | |
*** salv-orlando has quit IRC | 05:02 | |
*** wolverineav has quit IRC | 05:03 | |
*** links has joined #openstack-nova | 05:03 | |
*** fragatina has joined #openstack-nova | 05:07 | |
*** wolverineav has joined #openstack-nova | 05:09 | |
*** fragatina has quit IRC | 05:12 | |
*** bkopilov_ has joined #openstack-nova | 05:19 | |
*** takashin has joined #openstack-nova | 05:22 | |
openstackgerrit | Merged openstack/nova master: libvirt: Drop BAD_LIBVIRT_NUMA_VERSIONS https://review.openstack.org/564011 | 05:22 |
openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 05:25 |
openstackgerrit | melanie witt proposed openstack/nova master: Update layout docs for running console proxies https://review.openstack.org/557489 | 05:25 |
openstackgerrit | melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 05:25 |
*** vivsoni__ has quit IRC | 05:25 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2) https://review.openstack.org/564643 | 05:26 |
*** wolverineav has quit IRC | 05:28 | |
*** fragatina has joined #openstack-nova | 05:29 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645 | 05:31 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 05:36 |
*** jogo has quit IRC | 05:37 | |
*** sdeath has joined #openstack-nova | 05:38 | |
*** sree has joined #openstack-nova | 05:41 | |
*** namnh has quit IRC | 05:44 | |
*** psachin has joined #openstack-nova | 05:49 | |
*** Eran_Kuris has joined #openstack-nova | 05:50 | |
*** fragatina has quit IRC | 05:51 | |
*** jogo has joined #openstack-nova | 05:52 | |
*** sree has quit IRC | 06:00 | |
*** namnh has joined #openstack-nova | 06:00 | |
*** sree has joined #openstack-nova | 06:01 | |
*** fragatina has joined #openstack-nova | 06:04 | |
*** sree has quit IRC | 06:05 | |
*** namnh has quit IRC | 06:05 | |
*** namnh has joined #openstack-nova | 06:06 | |
*** gongysh has joined #openstack-nova | 06:07 | |
*** slaweq has joined #openstack-nova | 06:11 | |
*** namnh has quit IRC | 06:11 | |
*** homeski has quit IRC | 06:12 | |
*** cfriesen_ has quit IRC | 06:15 | |
*** slaweq has quit IRC | 06:15 | |
*** cfriesen_ has joined #openstack-nova | 06:15 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 06:21 |
*** janki has joined #openstack-nova | 06:21 | |
*** janki has quit IRC | 06:26 | |
*** gongysh has quit IRC | 06:27 | |
*** gongysh has joined #openstack-nova | 06:31 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.63 - List/Show all server migration types https://review.openstack.org/430839 | 06:32 |
*** cfriesen_ has quit IRC | 06:35 | |
*** cfriesen_ has joined #openstack-nova | 06:35 | |
*** namnh has joined #openstack-nova | 06:38 | |
*** dikonoor has quit IRC | 06:38 | |
*** namnh has quit IRC | 06:42 | |
*** cfriesen__ has joined #openstack-nova | 06:47 | |
*** cfriesen_ has quit IRC | 06:48 | |
*** gongysh has quit IRC | 06:49 | |
*** baoli has joined #openstack-nova | 06:52 | |
*** armaan has quit IRC | 06:58 | |
*** baoli has quit IRC | 06:59 | |
*** salv-orlando has joined #openstack-nova | 07:04 | |
*** rcernin has quit IRC | 07:06 | |
*** salv-orlando has quit IRC | 07:09 | |
*** salv-orlando has joined #openstack-nova | 07:10 | |
*** gongysh has joined #openstack-nova | 07:12 | |
*** gongysh has quit IRC | 07:13 | |
*** salv-orlando has quit IRC | 07:14 | |
*** gongysh has joined #openstack-nova | 07:17 | |
*** gjayavelu has joined #openstack-nova | 07:17 | |
*** baoli has joined #openstack-nova | 07:20 | |
*** tianhui has quit IRC | 07:21 | |
*** tianhui has joined #openstack-nova | 07:21 | |
*** david-lyle has joined #openstack-nova | 07:22 | |
*** david-lyle has quit IRC | 07:24 | |
*** dklyle has quit IRC | 07:24 | |
*** baoli has quit IRC | 07:25 | |
*** zz9pzza has joined #openstack-nova | 07:26 | |
*** gjayavelu has quit IRC | 07:30 | |
*** cfriesen__ has quit IRC | 07:31 | |
*** tomtom001 has quit IRC | 07:31 | |
*** tomtom001 has joined #openstack-nova | 07:32 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix the policy argument in server-group-create https://review.openstack.org/564699 | 07:35 |
*** sdeath has quit IRC | 07:40 | |
*** namnh has joined #openstack-nova | 07:46 | |
*** namnh has quit IRC | 07:51 | |
*** rmart04 has joined #openstack-nova | 07:59 | |
*** ragiman_ has joined #openstack-nova | 08:00 | |
*** mdnadeem has joined #openstack-nova | 08:00 | |
*** mgoddard has joined #openstack-nova | 08:04 | |
*** namnh has joined #openstack-nova | 08:06 | |
*** gongysh has quit IRC | 08:06 | |
*** namnh_ has joined #openstack-nova | 08:09 | |
*** slaweq has joined #openstack-nova | 08:11 | |
*** namnh has quit IRC | 08:12 | |
*** slaweq has quit IRC | 08:15 | |
*** armaan has joined #openstack-nova | 08:16 | |
*** baoli has joined #openstack-nova | 08:16 | |
*** namnh_ has quit IRC | 08:17 | |
*** rmart04 has quit IRC | 08:20 | |
*** baoli has quit IRC | 08:21 | |
*** mdbooth has joined #openstack-nova | 08:26 | |
*** baoli has joined #openstack-nova | 08:30 | |
*** belmoreira has joined #openstack-nova | 08:33 | |
*** baoli has quit IRC | 08:35 | |
*** ralonsoh has joined #openstack-nova | 08:37 | |
*** derekh has joined #openstack-nova | 08:38 | |
*** belmoreira has quit IRC | 08:42 | |
*** sambetts|afk is now known as sambetts | 08:50 | |
*** yamahata has quit IRC | 08:53 | |
*** gongysh has joined #openstack-nova | 08:55 | |
*** moshele has joined #openstack-nova | 08:56 | |
*** psachin has quit IRC | 08:56 | |
*** namnh has joined #openstack-nova | 08:57 | |
*** armaan has quit IRC | 08:59 | |
*** armaan has joined #openstack-nova | 09:00 | |
*** armaan has quit IRC | 09:04 | |
*** slaweq has joined #openstack-nova | 09:07 | |
*** cdent has joined #openstack-nova | 09:08 | |
*** hjensas has quit IRC | 09:12 | |
openstackgerrit | Maciej Kucia proposed openstack/nova-specs master: Fix Non-unique network names in Servers IPs API response https://review.openstack.org/558125 | 09:18 |
*** armaan has joined #openstack-nova | 09:19 | |
*** Dinesh_Bhor has quit IRC | 09:21 | |
*** slaweq has quit IRC | 09:22 | |
*** slaweq has joined #openstack-nova | 09:23 | |
*** dklyle has joined #openstack-nova | 09:26 | |
*** mdbooth has quit IRC | 09:28 | |
*** namnh has quit IRC | 09:36 | |
*** sree has joined #openstack-nova | 09:37 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520 | 09:37 |
*** sree has quit IRC | 09:42 | |
*** gongysh has quit IRC | 09:44 | |
*** mdbooth has joined #openstack-nova | 09:45 | |
*** lifeless_ is now known as lifeless | 09:46 | |
*** cdent has quit IRC | 09:50 | |
*** takashin has left #openstack-nova | 10:00 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471 | 10:12 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472 | 10:12 |
stephenfin | dansmith, sahid: When you're about, easy clean up I spotted after review of tx/rx queue size https://review.openstack.org/565471 | 10:13 |
stephenfin | Clearly someone need to document what the heck is going on there because that should have been an easy spot | 10:13 |
stephenfin | kashyap: Do you intend to tackle the two remaining cleanups noted here? https://review.openstack.org/#/c/564012/4/nova/virt/libvirt/driver.py | 10:14 |
*** hjensas has joined #openstack-nova | 10:25 | |
*** hjensas has quit IRC | 10:25 | |
*** hjensas has joined #openstack-nova | 10:25 | |
*** baoli has joined #openstack-nova | 10:32 | |
*** bkopilov has quit IRC | 10:33 | |
*** bkopilov_ has quit IRC | 10:33 | |
openstackgerrit | Merged openstack/python-novaclient master: Make sure microversion < 2.62 does not show host(Id) for instance actions https://review.openstack.org/564782 | 10:36 |
*** baoli has quit IRC | 10:36 | |
*** dougshelley66 has joined #openstack-nova | 10:37 | |
*** tbachman has quit IRC | 10:45 | |
*** nicolasbock has joined #openstack-nova | 10:48 | |
*** rmart04 has joined #openstack-nova | 10:48 | |
*** baoli has joined #openstack-nova | 10:53 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Fix misleading debug msg "Instance is running" https://review.openstack.org/565234 | 10:58 |
*** nicolasbock has quit IRC | 11:09 | |
*** dougshelley66 has quit IRC | 11:15 | |
*** dklyle has quit IRC | 11:15 | |
*** gongysh has joined #openstack-nova | 11:15 | |
*** moshele has quit IRC | 11:21 | |
*** slaweq has quit IRC | 11:22 | |
*** nicolasbock has joined #openstack-nova | 11:22 | |
*** mriedem has joined #openstack-nova | 11:38 | |
*** zz9pzza has quit IRC | 11:39 | |
*** dougshelley66 has joined #openstack-nova | 11:45 | |
*** yamamoto has quit IRC | 11:45 | |
*** slaweq has joined #openstack-nova | 11:50 | |
*** moshele has joined #openstack-nova | 11:51 | |
*** zz9pzza has joined #openstack-nova | 11:52 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | 11:52 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 11:52 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 11:52 |
*** pchavva has joined #openstack-nova | 11:52 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554 | 11:52 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: WIP: Support nested allocation candidates https://review.openstack.org/565487 | 11:52 |
*** jmccarthy has joined #openstack-nova | 11:54 | |
*** yamamoto has joined #openstack-nova | 11:55 | |
*** yamamoto has quit IRC | 11:55 | |
*** dklyle has joined #openstack-nova | 11:56 | |
*** armaan has quit IRC | 11:56 | |
*** armaan has joined #openstack-nova | 11:56 | |
*** baoli has quit IRC | 11:57 | |
*** armaan has joined #openstack-nova | 11:57 | |
*** dklyle has quit IRC | 11:58 | |
*** david-lyle has joined #openstack-nova | 11:58 | |
openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | 11:59 |
*** krtaylor has joined #openstack-nova | 12:02 | |
*** armaan has quit IRC | 12:06 | |
*** bkopilov_ has joined #openstack-nova | 12:06 | |
*** dave-mccowan has joined #openstack-nova | 12:08 | |
*** Eran_Kuris has quit IRC | 12:08 | |
*** bkopilov has joined #openstack-nova | 12:10 | |
*** lucas-hungry is now known as lucasagomes | 12:11 | |
*** tbachman has joined #openstack-nova | 12:11 | |
*** sree has joined #openstack-nova | 12:12 | |
*** mchlumsky has quit IRC | 12:16 | |
*** sree has quit IRC | 12:17 | |
*** edmondsw has joined #openstack-nova | 12:19 | |
*** mchlumsky has joined #openstack-nova | 12:20 | |
*** yamamoto has joined #openstack-nova | 12:24 | |
*** Tom-Tom has joined #openstack-nova | 12:26 | |
*** Tom-Tom has quit IRC | 12:30 | |
*** tomtom001 has quit IRC | 12:32 | |
*** tomtom001 has joined #openstack-nova | 12:32 | |
*** liverpooler has joined #openstack-nova | 12:37 | |
*** baoli has joined #openstack-nova | 12:37 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 12:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update layout docs for running console proxies https://review.openstack.org/557489 | 12:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 12:37 |
mriedem | melwitt: fixed up that bottom patch so tests should pass now ^ | 12:37 |
efried | ō/ | 12:40 |
*** baoli has quit IRC | 12:41 | |
*** weshay is now known as weshay|ruck | 12:46 | |
*** slaweq has quit IRC | 12:47 | |
*** moshele has quit IRC | 12:57 | |
*** moshele has joined #openstack-nova | 12:59 | |
*** tbachman has quit IRC | 13:00 | |
mriedem | vdrok: TheJulia: johnthetubaguy: are we ready for this now? https://review.openstack.org/#/c/508886/ or are people still needing this, i.e. do we have gaps in custom resource classes for ironic scheduling? | 13:01 |
johnthetubaguy | I think the traits support should cover the other cases now, but I could be miss-remembering | 13:02 |
johnthetubaguy | haven't heard that anyone has successfully moved between them though | 13:02 |
*** artom has joined #openstack-nova | 13:03 | |
*** tinwood has quit IRC | 13:04 | |
johnthetubaguy | although that was the non-exact case, thinking about it | 13:04 |
*** tinwood has joined #openstack-nova | 13:04 | |
*** hjensas has quit IRC | 13:05 | |
*** lyan has joined #openstack-nova | 13:07 | |
TheJulia | mriedem: my experience as of recent is that people are just now waking up to resource classes, but I think it should be safe to nuke those | 13:07 |
*** armaan has joined #openstack-nova | 13:07 | |
*** lyan is now known as Guest33985 | 13:07 | |
jroll | please do | 13:08 |
jroll | we should probably also raise an exception for nodes that don't have a resource class at this point | 13:08 |
*** david-lyle has quit IRC | 13:08 | |
*** armaan has quit IRC | 13:10 | |
*** armaan_ has joined #openstack-nova | 13:10 | |
jroll | mriedem: should we be removing the baremetal_enabled_filters option altogether? | 13:10 |
*** slaweq has joined #openstack-nova | 13:11 | |
jroll | reading the deprecation reason, seems like we should | 13:11 |
TheJulia | jroll: well, we set it by default upon create now, the conundrum being actual user use | 13:11 |
jroll | TheJulia: we set a default resource class on node create? | 13:12 |
TheJulia | jroll: we do | 13:12 |
jroll | O_o | 13:12 |
jroll | also, we've been logging warnings since pike, I don't think we can hold back just because people are being lazy | 13:12 |
TheJulia | which people though? | 13:13 |
jroll | operators? | 13:13 |
TheJulia | well, the users of the flavors are the ones impacted | 13:14 |
openstackgerrit | Merged openstack/nova-specs master: Return resources of entire trees in Placement https://review.openstack.org/559466 | 13:15 |
jroll | TheJulia: sure, in the end, users are always the impacted people. but, like most cases, it's a result of operators not tending to their cloud properly. it's like if an operator didn't deploy the placement api before it was required, users are impacted because the operator didn't do their job, ya know? | 13:15 |
*** slaweq has quit IRC | 13:15 | |
TheJulia | yup | 13:15 |
TheJulia | I do agree | 13:15 |
jroll | also, fwiw, there's only a default resource class if the operator sets one: https://github.com/openstack/ironic/blob/d917f32060d4f7237b9b4ecd6e0e769bc41d420c/ironic/conf/default.py#L74 | 13:15 |
*** esberglu has joined #openstack-nova | 13:16 | |
TheJulia | hmm, I thought that was defaulted to a value, maybe I'm thinking of other config someplace | 13:18 |
*** tssurya has joined #openstack-nova | 13:19 | |
mriedem | that reminds me of a bug in pike....sec | 13:28 |
mriedem | https://bugs.launchpad.net/nova/+bug/1765334 | 13:28 |
openstack | Launchpad bug 1765334 in OpenStack Compute (nova) "Ironic resource class may not be put into inventory in Pike" [Undecided,New] | 13:28 |
mriedem | if you set the resource class on the node after it's created, not during create, the inventory for it doesn't get represented in placement | 13:28 |
*** baoli_ has joined #openstack-nova | 13:29 | |
*** baoli_ has quit IRC | 13:30 | |
artom | Should we update the dashboard links from https://wiki.openstack.org/wiki/Nova/GerritDashboards ? 'cuz they're showing up empty for me | 13:30 |
mriedem | inadvertantly fixed in queens though so i'm not sure what, if anything, we should do about that bug in pike | 13:30 |
mriedem | artom: sure, they're likely all busted due to new gerrit | 13:30 |
artom | mriedem, I was secretly hoping someone else would do it, or at least show me how ;) | 13:31 |
artom | (Not the wiki part, the gerrit URL part) | 13:31 |
jroll | mriedem: any reason not to backport https://review.openstack.org/#/c/518294/ ? | 13:31 |
mriedem | jroll: idk, it's tied to the vgpu blueprint, i don't know if there would be other side effects | 13:32 |
jroll | heh, was hoping you'd know because I certainly don't :) | 13:32 |
*** lbragstad has joined #openstack-nova | 13:32 | |
*** mdbooth has quit IRC | 13:32 | |
*** openstackgerrit has quit IRC | 13:34 | |
artom | Ah, cuz it's not Jenkins that's voting now, it's Zuul | 13:34 |
* artom tries to fix it | 13:34 | |
*** Eran_Kuris has joined #openstack-nova | 13:35 | |
*** mdbooth has joined #openstack-nova | 13:35 | |
mriedem | ok so baremetal_enabled_filters is only used if use_baremetal_filters is True, and that defaults to False | 13:36 |
mriedem | i've got a note that we can't remove that stuff until we know all filters work with ironic, and i had a suspicion at one point that NUMATopologyFilter wouldn't work with ironic nodes, but can't really remember why now, | 13:36 |
mriedem | i posted a devstack change to enable the NUMATopologyFilter and it passed the ironic CI, but none of the requests would have stressed that filter since we don't do PCI/NUMA testing in the normal gate | 13:37 |
mriedem | so it would be nice if someone that does both pci/numa and ironic could try that out before we removed those deprecated options | 13:37 |
artom | Does a NUMATopologyFilter even make sense for Ironic? You're getting the whole machine regardles... | 13:37 |
mriedem | if your single compute installation supports both vms and bms | 13:38 |
mriedem | then you'll need to run that | 13:38 |
mriedem | if you support numa + vm | 13:38 |
*** dklyle has joined #openstack-nova | 13:38 | |
artom | mriedem, ah, in a mixed cloud case | 13:39 |
artom | mriedem, well, the filter does check for host_topology, and if its absent, passes the host | 13:39 |
artom | What does an ironic host expose for that? | 13:39 |
mriedem | nothing | 13:39 |
mriedem | so it might be ok, and i can't remember what at the time was making me think it might not work | 13:39 |
artom | Ah, no, hold up, I was wrong | 13:39 |
mriedem | i think i found something during code inspection | 13:40 |
artom | If you request a topology and the host doens't have it, it fails the host | 13:40 |
artom | Which... is fine, actually :) | 13:40 |
mriedem | right it would skip the ironic node | 13:40 |
mriedem | which is what we want | 13:40 |
artom | And if you request nothing and the host has nothing, it passes | 13:40 |
artom | Which is also fine | 13:41 |
*** Tom-Tom has joined #openstack-nova | 13:41 | |
mriedem | owalsh: does tripleo still rely on the exact core/ram/disk filters for baremetal? | 13:42 |
mriedem | artom: yeah which was shown in https://review.openstack.org/#/c/553412/ | 13:42 |
artom | mriedem, logs for that have been purged :( | 13:43 |
artom | But the ironic job passed, so yey :) | 13:43 |
mriedem | yeah i verified it ran and passed the filter in the ironic job, | 13:44 |
*** felipemonteiro__ has joined #openstack-nova | 13:44 | |
mriedem | but as noted, the flavors used don't have any numa topology requests | 13:44 |
artom | Hrmm, so it didn't fully test it | 13:45 |
artom | In fact, it didn't test the host topology bit at all :( | 13:45 |
artom | And that's the thing we're unsure about | 13:45 |
artom | Since numa_topology_filter.py L86 will evaluate to false immediately when there's no request_topology | 13:46 |
mriedem | jroll: uh oh https://docs.openstack.org/ironic/latest/install/configure-compute.html still tells people to set use_baremetal_filters=True | 13:46 |
mriedem | artom: right | 13:47 |
dansmith | artom: if you want to get a machine with a specific topology (like more than one node) wouldn't you want the filter? | 13:47 |
jroll | mriedem: oof | 13:47 |
artom | dansmith, for Ironic? Wouldn't that be a custom trait? | 13:47 |
mriedem | i could try and hack up a numa functional test in-tree and use the ironic driver, or a fake ironic driver | 13:47 |
artom | Like SUPER_GOLD_3_NUMA_NODES or whatever? | 13:47 |
dansmith | artom: ah, I guess with CUSTOM_IRONIC_SILVER type scheduling that's true | 13:47 |
*** hemna_ has joined #openstack-nova | 13:48 | |
* artom still has no idea wtf the SILVER and GOLD stuff is all about | 13:48 | |
dansmith | artom: just service levels, if you have a couple different classes of box | 13:49 |
*** jmccarthy has quit IRC | 13:49 | |
*** felipemonteiro_ has joined #openstack-nova | 13:49 | |
mriedem | if you combine 5 of them you get a power ranger super bot thing | 13:50 |
artom | Nah brah, Captain Planet | 13:50 |
mriedem | lame | 13:50 |
artom | Dude | 13:50 |
artom | He's our hero | 13:50 |
artom | Going to bring pollution down to zero | 13:50 |
* jroll starts using 2_XEON_E5_4000_192_GB_RAM_3_NVME_1_RUST as an example resource class instead | 13:50 | |
mriedem | only a canadian would think captain tree hugger is a cool super hero | 13:50 |
artom | How can you not dig that? | 13:51 |
*** jmccarthy has joined #openstack-nova | 13:51 | |
mriedem | speaking of, https://adequateman.deadspin.com/why-your-childrens-television-program-sucks-paw-patrol-1758314083#_ga=2.144903474.746352317.1525182679-529658380.1521051074 | 13:51 |
artom | Yeah well, I'll be here smelling my flowers while you choke on your coal roller | 13:51 |
*** awaugama has joined #openstack-nova | 13:52 | |
mriedem | coal jobs f yeah! 'merica! | 13:52 |
mriedem | ok, ok, so back to this | 13:52 |
mriedem | i guess we can drop the exact filters | 13:52 |
dansmith | yeah, so this went off the rails :) | 13:52 |
mriedem | and drop the other ironic specific options in a follow up | 13:52 |
*** slaweq has joined #openstack-nova | 13:52 | |
mriedem | would like owalsh or one of the other 30 tripleo heros to tell me if they still rely on those | 13:52 |
*** felipemonteiro__ has quit IRC | 13:53 | |
mriedem | artom: dansmith: i'm assuming you can find out? | 13:53 |
*** hjensas has joined #openstack-nova | 13:53 | |
*** hjensas has quit IRC | 13:53 | |
*** hjensas has joined #openstack-nova | 13:53 | |
mriedem | EmilienM: ^? | 13:53 |
*** slaweq has quit IRC | 13:53 | |
artom | mriedem, I don't think we have more tools for finding out than you do :/ | 13:53 |
*** slaweq has joined #openstack-nova | 13:53 | |
* mriedem does the unthinkable and jumps into the #tripleo channel | 13:54 | |
dansmith | mriedem: I'm pretty sure they're using resource classes now | 13:54 |
*** links has quit IRC | 13:54 | |
*** dklyle has quit IRC | 13:55 | |
EmilienM | mriedem: hey, saw your question. Thanks for asking. I also think we rely on resource classes now but I would double check with dtantsur|afk or bfournier, let me poke around | 13:57 |
*** hongbin has joined #openstack-nova | 13:59 | |
*** dikonoor has joined #openstack-nova | 14:01 | |
*** jmccarthy has quit IRC | 14:01 | |
*** dougshel_ has joined #openstack-nova | 14:04 | |
*** felipemonteiro_ has quit IRC | 14:05 | |
*** dougshelley66 has quit IRC | 14:05 | |
*** felipemonteiro_ has joined #openstack-nova | 14:05 | |
owalsh | EmilienM, mriedem: yea, landed in pike https://review.openstack.org/490851 | 14:06 |
EmilienM | owalsh: awesome | 14:06 |
mriedem | ok i'll push changes to docs and test modules in tripleo to remove the references | 14:08 |
mriedem | get my tripleo ATC badge | 14:08 |
*** wolverineav has joined #openstack-nova | 14:11 | |
*** READ10 has joined #openstack-nova | 14:11 | |
*** armaan_ has quit IRC | 14:13 | |
*** dikonoor has quit IRC | 14:14 | |
*** armaan has joined #openstack-nova | 14:14 | |
*** mriedem has quit IRC | 14:19 | |
*** mriedem has joined #openstack-nova | 14:19 | |
*** sree has joined #openstack-nova | 14:19 | |
*** openstackgerrit has joined #openstack-nova | 14:20 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407 | 14:20 |
openstackgerrit | Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513 | 14:20 |
*** tbachman has joined #openstack-nova | 14:21 | |
*** sree has quit IRC | 14:24 | |
*** mlavalle has joined #openstack-nova | 14:24 | |
*** felipemonteiro__ has joined #openstack-nova | 14:27 | |
*** felipemonteiro_ has quit IRC | 14:30 | |
*** armaan has quit IRC | 14:33 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove ExactCoreFilter ExactDiskFilter ExactRamFilter https://review.openstack.org/508886 | 14:33 |
dansmith | dum dum duuuum | 14:34 |
*** slaweq has quit IRC | 14:34 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406 | 14:34 |
openstackgerrit | Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407 | 14:34 |
openstackgerrit | Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513 | 14:34 |
*** gongysh has quit IRC | 14:37 | |
*** eharney has quit IRC | 14:39 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 14:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update layout docs for running console proxies https://review.openstack.org/557489 | 14:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 14:46 |
*** moshele has quit IRC | 14:47 | |
openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340 | 14:48 |
*** fishbone_ has joined #openstack-nova | 14:49 | |
*** bpoulos has joined #openstack-nova | 14:50 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 14:50 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 14:50 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 14:50 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 14:50 |
*** eharney has joined #openstack-nova | 14:51 | |
fishbone_ | hello all, does anyone know what could cause instance log error: pywintypes.com_error: (-2147352567, 'Exception occurred.', (0, 'Session', 'Access is denied. ', None, 0, -2147024891), None); in windows instances? | 14:52 |
mgagne | Can the CachingScheduler still be used in master? (and therefore bypassing placement API IIRC) | 14:54 |
owalsh | mriedem: based on the commit msg in https://review.openstack.org/565512, puppet should stop setting use_baremetal_filters and baremetal_enabled_filters? | 15:00 |
mriedem | use_baremetal_filters is deprecated | 15:01 |
mriedem | owalsh: for that specific patch, | 15:01 |
mriedem | the only filers in baremetal_enabled_filters were the 3 exact filters we're removing | 15:01 |
mriedem | so if baremetal_enabled_filters isn't going to have any filters in it, then use_baremetal_filters should be False | 15:01 |
*** yamamoto has quit IRC | 15:03 | |
*** tbachman has quit IRC | 15:03 | |
owalsh | ok, I'll push a new patch to make it so | 15:03 |
mriedem | owalsh: i'm not following, is there something wrong with https://review.openstack.org/#/c/565512/ ? | 15:04 |
*** rmart04 has quit IRC | 15:05 | |
owalsh | mriedem: that's just a spec test, need to change the behaviour in manifests/scheduler/filter.pp | 15:05 |
mriedem | ah ok | 15:05 |
*** mchlumsky has quit IRC | 15:05 | |
mriedem | mgagne: you have to fix this again https://review.openstack.org/#/c/565340/ | 15:06 |
*** Eran_Kuris has quit IRC | 15:06 | |
mriedem | you dropped my edit to fix the other compat test | 15:06 |
*** cfriesen__ has joined #openstack-nova | 15:06 | |
mgagne | mriedem: oops, didn't see that one :O | 15:07 |
mgagne | ok, gimme 2m, will fix shortly | 15:07 |
openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340 | 15:09 |
mgagne | mriedem: why do the above tests not need it? | 15:09 |
mriedem | they do need it, they are broken | 15:09 |
mriedem | but that can be fixed in a separate patch | 15:09 |
mriedem | this is why i asked for that validation yesterday | 15:09 |
*** r-daneel has quit IRC | 15:09 | |
*** slaweq has joined #openstack-nova | 15:11 | |
mgagne | oh.... wonderful | 15:14 |
*** slaweq has quit IRC | 15:15 | |
*** Eran_Kuris has joined #openstack-nova | 15:16 | |
*** yamamoto has joined #openstack-nova | 15:16 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471 | 15:19 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472 | 15:19 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517 | 15:19 |
*** yamamoto has quit IRC | 15:21 | |
openstackgerrit | Brianna Poulos proposed openstack/python-novaclient master: Microversion 2.63 - Add trusted_image_certificates https://review.openstack.org/500396 | 15:22 |
mriedem | bpoulos: what's the status on getting a spec amendment for that trusted certs stuff? | 15:22 |
bpoulos | mriedem: it's written, I'm just waiting for approval to post it. | 15:22 |
mriedem | i approve | 15:23 |
*** Eran_Kuris has quit IRC | 15:23 | |
bpoulos | mriedem: I wish your approval were enough :) | 15:23 |
*** fragatina has quit IRC | 15:23 | |
bpoulos | mriedem: unfortunately, it sometimes takes a while, and it can be very frustrating for us when we want to be able to do a quick turnaround | 15:24 |
*** fragatina has joined #openstack-nova | 15:24 | |
mriedem | understandable | 15:24 |
mriedem | would i get a grant kickback if i just pushed the spec amendment myself? | 15:25 |
mriedem | you don't have to answer that | 15:25 |
*** yamamoto has joined #openstack-nova | 15:31 | |
*** Eran_Kuris has joined #openstack-nova | 15:33 | |
*** MasterofJOKers has quit IRC | 15:33 | |
*** MasterofJOKers has joined #openstack-nova | 15:34 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471 | 15:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472 | 15:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517 | 15:35 |
*** yamamoto has quit IRC | 15:36 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043 | 15:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926 | 15:40 |
*** itlinux has joined #openstack-nova | 15:42 | |
*** yamamoto has joined #openstack-nova | 15:44 | |
*** yamamoto has quit IRC | 15:44 | |
*** vladikr has quit IRC | 15:45 | |
*** hamzy has quit IRC | 15:46 | |
*** r-daneel has joined #openstack-nova | 15:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926 | 15:47 |
*** vladikr has joined #openstack-nova | 15:47 | |
mriedem | dansmith: kashyap's https://review.openstack.org/#/q/topic:bug/1750829+status:open changes should be ready, to lift the cpu features flags restrictions so you can specify more than just pcid | 15:48 |
*** ragiman_ has quit IRC | 15:49 | |
*** ccamacho has quit IRC | 15:49 | |
dansmith | ack | 15:50 |
*** tinwood has quit IRC | 15:51 | |
*** tinwood has joined #openstack-nova | 15:51 | |
*** r-daneel has quit IRC | 15:51 | |
*** r-daneel has joined #openstack-nova | 15:57 | |
*** sree has joined #openstack-nova | 15:57 | |
*** gyee has joined #openstack-nova | 15:57 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make association_refresh configurable https://review.openstack.org/565526 | 15:58 |
tssurya | mriedem, efried: ^ | 15:59 |
efried | ack | 16:00 |
*** sree has quit IRC | 16:01 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: add Consumer object https://review.openstack.org/565403 | 16:02 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404 | 16:02 |
openstackgerrit | Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405 | 16:02 |
openstackgerrit | Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406 | 16:02 |
openstackgerrit | Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407 | 16:02 |
openstackgerrit | Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513 | 16:02 |
*** dougshelley66 has joined #openstack-nova | 16:05 | |
efried | tssurya: Reviewed. Looks good. | 16:06 |
tssurya | efried: thanks for the quick review, | 16:06 |
efried | sure thing. easy one :) | 16:06 |
*** mchlumsky has joined #openstack-nova | 16:07 | |
*** dougshel_ has quit IRC | 16:07 | |
*** tbachman has joined #openstack-nova | 16:11 | |
*** danpb1 has joined #openstack-nova | 16:16 | |
melwitt | mriedem: thanks | 16:17 |
mriedem | melwitt: only thing left is cleaning up https://review.openstack.org/#/c/565367/ | 16:18 |
melwitt | k, looking at it now | 16:19 |
*** ralonsoh has quit IRC | 16:20 | |
melwitt | phew, lots o docs | 16:23 |
melwitt | thanks for pointing out all of those | 16:23 |
*** armaan has joined #openstack-nova | 16:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix shelving a paused instance https://review.openstack.org/541660 | 16:28 |
mriedem | easy fix for a super latent problem ^ | 16:30 |
*** spsurya has quit IRC | 16:31 | |
*** Swami has joined #openstack-nova | 16:36 | |
*** bhujay has quit IRC | 16:36 | |
*** lucasagomes is now known as lucas-afk | 16:36 | |
*** ralonsoh has joined #openstack-nova | 16:36 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383 | 16:37 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Speed things up and document them https://review.openstack.org/534382 | 16:37 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Switch basepython to 3 https://review.openstack.org/565534 | 16:37 |
*** armaan has quit IRC | 16:38 | |
*** armaan has joined #openstack-nova | 16:39 | |
Swami | Have a question on PCI-Passthrough. What is the right approach, should we create a neutron port with vnic_type=direct-physical and assign it to the VM or directly create a flavor with Alias and create a VM without specifying the network. | 16:39 |
*** artom has quit IRC | 16:39 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove stale pip-missing-reqs tox test https://review.openstack.org/565535 | 16:40 |
openstackgerrit | melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 16:42 |
*** sambetts is now known as sambetts|afk | 16:42 | |
*** yamamoto has joined #openstack-nova | 16:44 | |
*** yamamoto has quit IRC | 16:50 | |
openstackgerrit | melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 16:52 |
mriedem | melwitt: you were too fast | 16:53 |
melwitt | oops | 16:53 |
*** moshele has joined #openstack-nova | 16:53 | |
*** derekh has quit IRC | 16:56 | |
*** hamzy has joined #openstack-nova | 16:56 | |
*** moshele has quit IRC | 16:56 | |
*** gjayavelu has joined #openstack-nova | 16:59 | |
*** imacdonn has quit IRC | 16:59 | |
*** danpb1 has quit IRC | 17:00 | |
*** armaan has quit IRC | 17:01 | |
*** armaan has joined #openstack-nova | 17:02 | |
*** hamzy has quit IRC | 17:03 | |
*** r-daneel_ has joined #openstack-nova | 17:05 | |
*** mgoddard has quit IRC | 17:06 | |
*** r-daneel has quit IRC | 17:07 | |
*** r-daneel has joined #openstack-nova | 17:08 | |
openstackgerrit | melanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 17:08 |
*** r-daneel_ has quit IRC | 17:10 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make association_refresh configurable https://review.openstack.org/565526 | 17:14 |
*** READ10 has quit IRC | 17:15 | |
*** yamahata has joined #openstack-nova | 17:19 | |
*** READ10 has joined #openstack-nova | 17:21 | |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 17:24 |
*** artom has joined #openstack-nova | 17:24 | |
melwitt | mriedem: I noticed yesterday that it appears nova-next isn't running console proxy with SSL anymore. says "no cert file" but I see the cert stuff being set up in the devstack log. don't yet know what's wrong http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/devstacklog.txt.gz#_2018-05-01_15_14_45_231 | 17:25 |
*** mdnadeem has quit IRC | 17:27 | |
mriedem | hmm yeah http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/local.conf.txt.gz has NOVA_CONSOLE_PROXY_COMPUTE_TLS=True | 17:27 |
mriedem | does the config have to be in the cell? | 17:28 |
mriedem | if so, it's probably writing to the wrong nova config | 17:28 |
*** itlinux has quit IRC | 17:28 | |
*** mdbooth has quit IRC | 17:29 | |
mriedem | [vnc] is configured properly here http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/etc/nova/nova.conf.txt.gz | 17:29 |
mriedem | and here http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/etc/nova/nova-cpu.conf.txt.gz | 17:29 |
melwitt | I would think the config doesn't have the be in the cell until my changes (and the above log I linked is on a change unrelated to my per cell proxy stuff) | 17:29 |
mriedem | melwitt: i wonder if it's something with using the queens UCA | 17:29 |
mriedem | libvirt might have changed | 17:30 |
mriedem | re: https://review.openstack.org/#/c/396186/7/lib/nova_plugins/functions-libvirt | 17:30 |
melwitt | maybe, guh I hope not. I have no clue how that stuff works | 17:30 |
melwitt | but this used to work and I can't think of anything that's changed | 17:30 |
mriedem | this is where that check happens http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/devstacklog.txt.gz#_2018-05-01_15_05_38_543 | 17:31 |
melwitt | yeah, saw that too | 17:31 |
melwitt | "no cert file" sounds like it can't find the cert file. it deploys them to /etc/pki/libvirt-vnc/ | 17:33 |
melwitt | oh wait, but the config says /etc/pki/nova-novnc/ | 17:34 |
*** mgoddard has joined #openstack-nova | 17:34 | |
mriedem | looking in the libvirtd logs, i can see qemu being executed with "tls,x509verify=/etc/pki/libvirt-vnc" | 17:35 |
melwitt | okay, so I wonder if these are wrong in nova.conf? not sure | 17:36 |
melwitt | [vnc] | 17:36 |
melwitt | vencrypt_ca_certs = /etc/pki/nova-novnc/ca-cert.pem | 17:36 |
mriedem | don't know, but that wouldn't really explain how it was working before | 17:37 |
mriedem | probably need to ask stephenfin or danpb | 17:37 |
melwitt | yeah, true. I don't even know where teh nova-novnc dir comes from | 17:37 |
mriedem | it's created by devstack https://review.openstack.org/#/c/396186/7/lib/nova | 17:37 |
melwitt | fwiw, I was the one who got the devstack patch to actually run with SSL because there were minor things wrong in the devstack changes originally because things had changed over time (the series has been around forever) | 17:38 |
melwitt | because I didn't want to approve the patches until there was some demonstration of the proxy running with SSL enabled | 17:39 |
*** itlinux has joined #openstack-nova | 17:42 | |
*** tbachman has quit IRC | 17:42 | |
melwitt | wonder if the chown user:group needs to be updated again, that's what I had to change last time https://review.openstack.org/#/c/396186/7/lib/nova_plugins/functions-libvirt@156 | 17:43 |
mriedem | that's what i was thinking, | 17:44 |
mriedem | because i had to deal with that for the multiattach stuff | 17:44 |
mriedem | melwitt: this https://github.com/openstack-dev/devstack/blob/master/stackrc#L630 | 17:45 |
mriedem | could add a debug patch to make sure that libvirt-qemu exists | 17:45 |
melwitt | hm, yeah. this would be the qemu user and group. originally it was just qemu:qemu and I had to update it to libvirt-qemu:libvirt-qemu | 17:46 |
*** yamamoto has joined #openstack-nova | 17:47 | |
*** ralonsoh has quit IRC | 17:47 | |
*** liverpooler has quit IRC | 17:48 | |
mriedem | melwitt: you should kick an intern to make sure this article from 2011 is up to date https://wiki.libvirt.org/page/VNCTLSSetup | 17:50 |
*** vladikr has quit IRC | 17:50 | |
melwitt | heh, that's a good doc | 17:51 |
*** yamamoto has quit IRC | 17:52 | |
openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 17:54 |
*** hamzy has joined #openstack-nova | 17:57 | |
wolverineav | stephenfin: mriedem - updated the review with patch addressing comments ^^ take a look when you get a chance :) | 17:57 |
*** vladikr has joined #openstack-nova | 18:04 | |
melwitt | based on this old bug, it's recommended to use qemu:qemu but it depends on what the distro has done https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/522619 | 18:06 |
openstack | Launchpad bug 522619 in libvirt (Ubuntu) "libvirt launced kvm / qemu system processes run as root by default" [Wishlist,Fix released] | 18:06 |
*** moshele has joined #openstack-nova | 18:06 | |
*** vivsoni has quit IRC | 18:07 | |
*** vivsoni has joined #openstack-nova | 18:07 | |
openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 18:08 |
mriedem | melwitt: so probably the thing to do is check /etc/group like in stackrc and fallback to qemu:qemu if libvirt-qemu doesn't exist | 18:10 |
*** tbachman has joined #openstack-nova | 18:11 | |
*** slaweq has joined #openstack-nova | 18:11 | |
*** larsks has joined #openstack-nova | 18:14 | |
larsks | dansmith: a hack: https://github.com/larsks/os-placement-tools | 18:15 |
*** slaweq has quit IRC | 18:15 | |
*** liverpooler has joined #openstack-nova | 18:15 | |
mriedem | tssurya: comments inline | 18:17 |
*** vladikr has quit IRC | 18:17 | |
tssurya | mriedem: thanks | 18:17 |
openstackgerrit | Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 18:17 |
*** moshele has quit IRC | 18:18 | |
mriedem | larsks: "--fox" typo | 18:19 |
larsks | mriedem: hah, you fell for my clever plan. | 18:19 |
larsks | Also, thanks, fixed it. | 18:20 |
mriedem | is this for busted migrations? | 18:20 |
larsks | mriedem: well, maybe. We hit a situation on Friday at boston university where we were unable to schedule new instances on an empty compute node. It turns out there were a number of stale allocations there (and elsewhere in the cluster). | 18:20 |
larsks | Maybe they were caused by busted migrations? | 18:20 |
larsks | But this fixed the immediate problem regardless of the cause. | 18:21 |
mriedem | what version of nova? | 18:21 |
larsks | It is now pike, but it was upgraded from ocata. And possibly from earlier. | 18:21 |
mriedem | pike GA or latest stable release? | 18:21 |
mriedem | post pike GA we found a number of bugs in places where we didn't cleanup allocations properly | 18:21 |
mriedem | those should all be fixed now | 18:21 |
larsks | Uh...whatever red hat is distributing in rhel-osp 12. | 18:21 |
mriedem | heh, fair enough | 18:22 |
* mriedem wonders if this coincides with dansmith asking me about writing a cli to migrate off the caching scheduler | 18:23 | |
mriedem | https://github.com/larsks/os-placement-tools/blob/master/check_placement.py#L16 is nice | 18:23 |
larsks | Thanks! | 18:24 |
dansmith | mriedem: nope | 18:24 |
dansmith | mriedem: it does coincide with me ruminating about deleting duplicate allocations in -placement last week though | 18:24 |
mriedem | wonder if anyone ever started on that osc-placement change to orchestrate setting inventory on an entire placement provider aggregate set | 18:25 |
dansmith | larsks: that looks awesome | 18:25 |
mriedem | unrelated to this besides placement tooling | 18:25 |
dansmith | larsks: you should check for instance state and only agree to --fix those in ACTIVE though | 18:25 |
dansmith | larsks: before ocata (ish?) you could have legit reasons for two allocations for one instance, because that's how we did migrations | 18:26 |
dansmith | but if the instance is ACTIVE and has two, that'll always need fixing I think | 18:26 |
*** Eran_Kuris_ has joined #openstack-nova | 18:26 | |
dansmith | mriedem: right? | 18:26 |
mriedem | and/or don't touch any instances with a non-null task_state | 18:26 |
mriedem | before pike the computes would heal allocations | 18:26 |
dansmith | yeah | 18:27 |
mriedem | but yeah don't fix an instance that's undergoing a task_state transition | 18:27 |
mriedem | yee be warned | 18:27 |
*** Eran_Kuris has quit IRC | 18:29 | |
*** vladikr has joined #openstack-nova | 18:31 | |
larsks | So noted. | 18:31 |
*** READ10 has quit IRC | 18:32 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add Project object https://review.openstack.org/565401 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add User object to placement modeling https://review.openstack.org/565402 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add Consumer object https://review.openstack.org/565403 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 18:32 |
openstackgerrit | Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513 | 18:32 |
melwitt | looks like libvirt-qemu is the right user and group, actually logged in the devstack log http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/devstacklog.txt.gz#_2018-05-01_15_04_42_683 | 18:36 |
mriedem | huh | 18:36 |
mriedem | nuts | 18:36 |
melwitt | yeah. back to the drawing board | 18:36 |
*** vivsoni has quit IRC | 18:41 | |
*** vivsoni has joined #openstack-nova | 18:41 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 18:42 |
mriedem | alright https://review.openstack.org/#/q/topic:bp/convert-consoles-to-objects+status:open is ready to go | 18:43 |
mriedem | and it's the last day in the runway | 18:43 |
*** fragatina has quit IRC | 18:44 | |
melwitt | so close ... | 18:45 |
mriedem | well, efried and dansmith can make that magic happen | 18:46 |
mriedem | *or | 18:46 |
melwitt | note that once the "convert websocketproxy" change is approved, we need to ask frickler to lift the -2 on the first devstack sandwich change so it can go through | 18:46 |
mriedem | yup | 18:46 |
efried | mriedem: subtle. | 18:46 |
*** yamamoto has joined #openstack-nova | 18:48 | |
*** mgoddard has quit IRC | 18:52 | |
*** Eran_Kuris_ has quit IRC | 18:53 | |
*** yamamoto has quit IRC | 18:53 | |
*** imacdonn has joined #openstack-nova | 18:54 | |
melwitt | dansmith: do you remember why we have to "enable before stop" for the conductor fleet? I had the same thing for the console proxies https://review.openstack.org/#/c/484973/12/lib/nova@1078 | 18:54 |
dansmith | if the thing isn't enabled, then stop_service won't do anything, | 18:56 |
dansmith | but I thnk that conductor case was just for when we were moving to super conductor | 18:57 |
dansmith | although maybe it's still required since we generate the service id for services in the cell? | 18:57 |
*** felipemonteiro__ has quit IRC | 18:57 | |
*** tssurya has quit IRC | 18:58 | |
*** vladikr has quit IRC | 18:59 | |
*** r-daneel has quit IRC | 19:00 | |
melwitt | I wasn't sure why we can't assume they're enabled from the earlier call to start_nova_console_proxies | 19:00 |
*** r-daneel_ has joined #openstack-nova | 19:00 | |
dansmith | idk, I'm kinda into something else at the moment so I can't look deeply, | 19:01 |
dansmith | but what I said is what I remember off the top of my head for that stuff | 19:01 |
melwitt | yeah, np, thanks for that. not looking for a deep answer, wish I had written a comment on here from back when someone explained it | 19:02 |
*** r-daneel_ is now known as r-daneel | 19:02 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 19:05 |
melwitt | I think it might be because start_* and stop_* aren't going to happen in the same run (stack.sh vs unstack.sh) and there's not going to be any saving of state about the cell services between those runs | 19:05 |
melwitt | so during the unstack, you can't stop the per cell processes unless you enable them first | 19:06 |
dansmith | that's what I'm saying, | 19:07 |
dansmith | because we generate their service names from the cell we're in | 19:07 |
dansmith | n-cond-cell1, etc | 19:07 |
melwitt | right | 19:07 |
melwitt | what I was missing is that there's no saving of state after a stack.sh so there's no way the teardown would "save" the fact that the services were created and enabled by the stack.sh run | 19:08 |
*** vladikr has joined #openstack-nova | 19:11 | |
*** slaweq has joined #openstack-nova | 19:11 | |
*** dougshelley66 has quit IRC | 19:15 | |
*** slaweq has quit IRC | 19:15 | |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API https://review.openstack.org/565565 | 19:21 |
*** vladikr has quit IRC | 19:22 | |
*** mgoddard has joined #openstack-nova | 19:23 | |
*** mgoddard has quit IRC | 19:28 | |
*** vladikr has joined #openstack-nova | 19:35 | |
*** dpawlik has joined #openstack-nova | 19:37 | |
*** damien_r has joined #openstack-nova | 19:39 | |
*** moshele has joined #openstack-nova | 19:45 | |
*** awaugama has quit IRC | 19:45 | |
*** dpawlik has quit IRC | 19:47 | |
*** vladikr has quit IRC | 19:47 | |
*** yamamoto has joined #openstack-nova | 19:49 | |
*** moshele has quit IRC | 19:50 | |
*** slaweq has joined #openstack-nova | 19:50 | |
*** damien_r has quit IRC | 19:51 | |
*** dougshelley66 has joined #openstack-nova | 19:54 | |
*** yamamoto has quit IRC | 19:55 | |
*** liverpooler has quit IRC | 19:56 | |
*** vladikr has joined #openstack-nova | 19:59 | |
openstackgerrit | Oliver Walsh proposed openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set https://review.openstack.org/565573 | 20:00 |
efried | owalsh: Do we need a bug for ^ ? | 20:07 |
*** fragatina has joined #openstack-nova | 20:07 | |
*** mchlumsky has quit IRC | 20:12 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs https://review.openstack.org/565225 | 20:13 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 20:13 |
mriedem | efried: i thought about that, but it's not even tagged in rc1 | 20:15 |
mriedem | so not really a good chance of someone hitting that besides red hat QE | 20:15 |
efried | mriedem: Sure, it's only been broken for a week, but still, paperwork. | 20:15 |
efried | mriedem: If you of all people don't need a bug, I'm sure not gonna push for it :) | 20:16 |
owalsh | awww but I love paperwork :-( | 20:16 |
owalsh | efried, mriedem: yea, I'm lazy and it's not released yet but can raise one if you think it's necessary | 20:18 |
*** tssurya has joined #openstack-nova | 20:19 | |
openstackgerrit | Merged openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340 | 20:19 |
*** tssurya has quit IRC | 20:19 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 20:20 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 20:20 |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 20:20 |
melwitt | hm, looks like all of the third party CIs suddenly are failing | 20:34 |
*** artom has quit IRC | 20:34 | |
mriedem | all or just powervm? | 20:35 |
melwitt | hyperv CI can't resolve a hostname, some others like powervm and virtuozzo, the log files aren't found | 20:35 |
mriedem | i blame zuul | 20:35 |
melwitt | and it seems like this has happened starting today | 20:35 |
*** itlinux has quit IRC | 20:39 | |
*** vladikr has quit IRC | 20:41 | |
*** itlinux has joined #openstack-nova | 20:41 | |
*** randomhack_ has joined #openstack-nova | 20:42 | |
randomhack_ | was this bug ever solved? https://bugs.launchpad.net/nova/+bug/1633033. ran into this today on a Newton in | 20:43 |
openstack | Launchpad bug 1633033 in OpenStack Compute (nova) "live migration with encrypted volume fails" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood) | 20:43 |
randomhack_ | love migration of an instance with encrypted volume resulted in volume being mounted without decryption and corrupted the file system | 20:44 |
randomhack_ | s/love/live | 20:45 |
randomhack_ | known limitation of encrypted iscsi volumes? | 20:46 |
mriedem | randomhack_: lyarwood would know but it's really late for him (UK) | 20:48 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425 | 20:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 20:49 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs https://review.openstack.org/565225 | 20:49 |
randomhack_ | mriedem: thanks i’ll check tomorrow earlier | 20:49 |
melwitt | yeah. IIRC, it will work as of pike when using qemu native luks decryption https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/libvirt-qemu-native-luks.html | 20:49 |
randomhack_ | melwitt: thanks! | 20:49 |
melwitt | because I thought lyarwood had a depends-on patch that ran live migration on top of the series | 20:50 |
melwitt | randomhack_: double check with lyarwood to make sure | 20:50 |
*** yamamoto has joined #openstack-nova | 20:51 | |
*** rajinir has joined #openstack-nova | 20:51 | |
*** bpoulos has quit IRC | 20:52 | |
melwitt | this was one of the live migration test patches https://review.openstack.org/545074 | 20:52 |
mriedem | melwitt: i think that was queens | 20:53 |
mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/libvirt-qemu-native-luks.html :) | 20:53 |
mriedem | yeah | 20:53 |
melwitt | ah, you're right, it was queens | 20:53 |
*** hamzy has quit IRC | 20:55 | |
*** vladikr has joined #openstack-nova | 20:56 | |
*** yamamoto has quit IRC | 20:57 | |
melwitt | ceph job failing 100% with SnapshotIsBusy during a volume delete now. le sigh http://logs.openstack.org/49/479949/71/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/94fbc69/logs/screen-c-vol.txt.gz#_May_01_15_45_34_845542 | 20:59 |
melwitt | there's even a note from you in the tempest test about this (I think?) and yet it's still happening https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py#L239 | 21:00 |
*** vladikr has quit IRC | 21:00 | |
melwitt | oh, wait, it's a TODO | 21:00 |
melwitt | I'll try to do the todo | 21:01 |
mriedem | it's a glorious todo | 21:01 |
melwitt | apparently. I don't know how this used to work before if there's an inherent issue with the volume dependencies in the test | 21:02 |
melwitt | I don't know volumes about volumes | 21:02 |
* mriedem does a rimshot | 21:02 | |
melwitt | thanks | 21:03 |
mriedem | so in this test, we boot from volume and then create a volume-backed server snapshot, so we have an image with bdmv2 metadata pointing at the volume snapshot, | 21:03 |
mriedem | we then create a 2nd server from that image-defined bdm (the volume snapshot) | 21:03 |
mriedem | because delete_on_termination=True, | 21:04 |
mriedem | when we go to delete the 2nd server, the compute service will attempt to delete the 2nd volume which is the root disk for the 2nd server, created from the volume snapshot, | 21:04 |
mriedem | but cinder won't let you delete a volume that has snapshots, | 21:04 |
mriedem | so nova-compute will get an error from cinder and ignore it | 21:04 |
melwitt | hmm okay | 21:05 |
mriedem | in here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2422 | 21:05 |
mriedem | you should see "Failed to delete volume" in the n-cpu logs for that test | 21:05 |
melwitt | it's weird because what's happening is it's the test trying to cleanup the volumes it had created, | 21:06 |
melwitt | and it can't delete the snapshot because of dependent things | 21:06 |
mriedem | http://logs.openstack.org/49/479949/71/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/94fbc69/logs/screen-n-cpu.txt.gz#_May_01_15_45_20_004599 | 21:06 |
melwitt | it's not failing because of anything about n-cpu as far as I can tell | 21:06 |
mriedem | i believe jbernard had a really old patch for dealing with a failure to delete snapshots because they were busy with the cinder rbd driver | 21:07 |
mriedem | https://review.openstack.org/#/c/281550/ | 21:08 |
melwitt | I remember this patch | 21:08 |
melwitt | I wasn't sure if this is an rbd-specific SnapshotIsBusy or if it's just because of how the test itself doesn't delete the volume first | 21:09 |
*** tbachman has quit IRC | 21:09 | |
openstackgerrit | Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 21:11 |
openstackgerrit | Merged openstack/nova master: Use os.rename, not mv. https://review.openstack.org/562463 | 21:13 |
*** dklyle has joined #openstack-nova | 21:14 | |
Swami | I am trying to debug a PCI-Passthrough issue, what I am seeing is there are two pci-requests that are being sent to the compute to claim even though there is only one VM that is created. This happens when the first attempt to create the VM errors out and then when we retry creating an instance. [InstancePCIRequest(alias_name='intel10fb',count=1,is_new=False,request_id=None,spec=[{dev_type='type-PF',product_id='10fb',vendor_id='808 | 21:23 |
Swami | 6'}]), InstancePCIRequest(alias_name=None,count=1,is_new=False,request_id=13befe5f-478f-4f4c-aa72-78cce84d942d,spec=[{dev_type='type-PF',physical_network='physnet2'}])]. The odd thing that I see between the two requests is one has a 'request_id' as None, but the other has a 'request_id' uuid. | 21:23 |
*** wolverineav has quit IRC | 21:29 | |
*** wolverineav has joined #openstack-nova | 21:30 | |
*** felipemonteiro__ has joined #openstack-nova | 21:30 | |
*** pchavva has quit IRC | 21:30 | |
*** felipemonteiro_ has joined #openstack-nova | 21:33 | |
*** itlinux has quit IRC | 21:34 | |
*** wolverineav has quit IRC | 21:34 | |
*** felipemonteiro__ has quit IRC | 21:36 | |
*** idlemind has joined #openstack-nova | 21:41 | |
*** edmondsw has quit IRC | 21:48 | |
*** yamamoto has joined #openstack-nova | 21:53 | |
*** yamamoto has quit IRC | 21:59 | |
*** READ10 has joined #openstack-nova | 22:08 | |
openstackgerrit | Merged openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043 | 22:11 |
melwitt | so we have to backport that all the way to ocata? ^ | 22:13 |
*** jmlowe has quit IRC | 22:13 | |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API https://review.openstack.org/565565 | 22:14 |
dansmith | melwitt: I don't really think we do | 22:19 |
dansmith | it's pretty minor | 22:19 |
dansmith | IMHO | 22:19 |
melwitt | okay | 22:19 |
dansmith | that reminds me I need to look at the second patch there | 22:19 |
*** Guest33985 has quit IRC | 22:19 | |
*** dklyle has quit IRC | 22:21 | |
melwitt | that'll close out the blueprint, yeah? | 22:21 |
melwitt | (after it merges) | 22:22 |
*** lbragstad has quit IRC | 22:22 | |
*** dklyle has joined #openstack-nova | 22:26 | |
*** yassine has joined #openstack-nova | 22:35 | |
*** yassine is now known as Guest72284 | 22:36 | |
*** rcernin has joined #openstack-nova | 22:36 | |
openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set https://review.openstack.org/565573 | 22:37 |
efried | mriedem: Quick re+A on this one ^ | 22:39 |
openstackgerrit | Jay Pipes proposed openstack/nova master: WIP: Add a microversion for consumer generation https://review.openstack.org/565604 | 22:40 |
*** jmlowe has joined #openstack-nova | 22:43 | |
*** lbragstad has joined #openstack-nova | 22:46 | |
*** slaweq has quit IRC | 22:46 | |
*** slaweq has joined #openstack-nova | 22:47 | |
*** yamamoto has joined #openstack-nova | 22:55 | |
*** artom has joined #openstack-nova | 22:55 | |
*** hongbin has quit IRC | 22:55 | |
*** felipemonteiro_ has quit IRC | 22:58 | |
*** moshele has joined #openstack-nova | 22:58 | |
*** yamamoto has quit IRC | 23:00 | |
*** moshele has quit IRC | 23:02 | |
*** rajinir has quit IRC | 23:10 | |
*** vivsoni_ has joined #openstack-nova | 23:12 | |
*** vivsoni has quit IRC | 23:13 | |
*** randomhack_ has quit IRC | 23:20 | |
*** r-daneel has quit IRC | 23:23 | |
*** slaweq has quit IRC | 23:27 | |
*** itlinux has joined #openstack-nova | 23:34 | |
*** takashin has joined #openstack-nova | 23:35 | |
*** itlinux has quit IRC | 23:39 | |
*** esberglu has quit IRC | 23:41 | |
*** esberglu has joined #openstack-nova | 23:41 | |
*** esberglu has quit IRC | 23:41 | |
*** mlavalle has quit IRC | 23:46 | |
mriedem | melwitt: would have to ask kashyap because there is a note in the patch about future patches | 23:49 |
mriedem | "(A future Nova patch will also allow ability to remove CPU flags.)" | 23:49 |
*** r-daneel has joined #openstack-nova | 23:50 | |
*** dklyle has quit IRC | 23:51 | |
*** yamamoto has joined #openstack-nova | 23:56 | |
*** dklyle has joined #openstack-nova | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!