Tuesday, 2018-05-01

*** randomhack_ has joined #openstack-nova00:04
arvindn05alex_xu: had a question on your comments00:05
arvindn05https://review.openstack.org/#/c/560596/ - update scheduler to use image-traits00:06
openstackgerritMerged openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION  https://review.openstack.org/56398400:06
*** moshele has joined #openstack-nova00:07
openstackgerritMerged openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC  https://review.openstack.org/56401000:23
*** tbachman has joined #openstack-nova00:26
*** namnh has joined #openstack-nova00:27
*** Dinesh_Bhor has joined #openstack-nova00:40
*** jmlowe has quit IRC00:43
*** jmlowe has joined #openstack-nova00:44
*** yamahata has quit IRC00:49
*** takashin has joined #openstack-nova01:04
*** mlavalle has quit IRC01:05
*** moshele has quit IRC01:07
*** dklyle has quit IRC01:09
*** dklyle has joined #openstack-nova01:09
*** slaweq has joined #openstack-nova01:11
*** dklyle has quit IRC01:11
*** dklyle has joined #openstack-nova01:12
*** slaweq has quit IRC01:16
*** felipemonteiro__ has quit IRC01:18
*** gjayavelu has quit IRC01:25
*** bhujay has joined #openstack-nova01:27
*** salv-orlando has joined #openstack-nova01:31
*** salv-orlando has quit IRC01:36
*** r-daneel has joined #openstack-nova01:39
*** r-daneel_ has joined #openstack-nova01:42
*** spsurya has joined #openstack-nova01:43
*** r-daneel has quit IRC01:43
*** r-daneel_ is now known as r-daneel01:43
*** mriedem_away is now known as mriedem01:44
openstackgerritJulia Kreger proposed openstack/nova master: WIP: ironic: add instance_uuid before any other spawn activity  https://review.openstack.org/56372201:45
openstackgerritMatt Riedemann proposed openstack/nova master: Add user_id to RequestSpec  https://review.openstack.org/56534001:48
*** mriedem has quit IRC01:53
*** Zames has joined #openstack-nova01:58
*** Zames has quit IRC02:01
*** Zames has joined #openstack-nova02:08
*** fragatina has quit IRC02:08
*** fragatina has joined #openstack-nova02:09
*** fragatina has quit IRC02:09
*** Zames has quit IRC02:10
*** bkopilov has quit IRC02:10
*** dave-mccowan has joined #openstack-nova02:12
*** randomhack_ has quit IRC02:14
*** dklyle has quit IRC02:15
*** dklyle has joined #openstack-nova02:16
*** dklyle has quit IRC02:22
*** dklyle has joined #openstack-nova02:23
*** dklyle has quit IRC02:24
*** dklyle has joined #openstack-nova02:24
*** nicolasbock has quit IRC02:25
*** dklyle has quit IRC02:25
*** dklyle has joined #openstack-nova02:26
*** liverpooler has quit IRC02:27
openstackgerritArvind Nadendla proposed openstack/nova master: update scheduler to use image-traits  https://review.openstack.org/56059602:28
openstackgerritArvind Nadendla proposed openstack/nova master: update scheduler to use image-traits  https://review.openstack.org/56059602:33
*** armaan has joined #openstack-nova02:34
takashin02:38
*** wolverineav has joined #openstack-nova02:40
*** dikonoor has joined #openstack-nova02:41
*** namnh has quit IRC02:43
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (1)  https://review.openstack.org/56463902:45
*** namnh has joined #openstack-nova02:49
*** hongbin has joined #openstack-nova02:53
*** hamzy has quit IRC02:54
*** hamzy has joined #openstack-nova02:54
*** namnh has quit IRC02:54
*** slaweq has joined #openstack-nova02:56
*** itlinux has joined #openstack-nova02:58
*** slaweq has quit IRC03:01
*** namnh has joined #openstack-nova03:06
*** namnh has quit IRC03:10
openstackgerritMerged openstack/nova master: Use ConsoleAuthToken object to generate authorizations  https://review.openstack.org/32541403:14
*** dougshelley66 has quit IRC03:15
*** wolverineav has quit IRC03:16
*** wolverineav has joined #openstack-nova03:16
*** wolverineav has quit IRC03:21
*** fragatina has joined #openstack-nova03:23
*** takashin has quit IRC03:26
*** dklyle has quit IRC03:27
*** dklyle has joined #openstack-nova03:28
*** wolverineav has joined #openstack-nova03:29
*** salv-orlando has joined #openstack-nova03:32
*** itlinux has quit IRC03:32
*** fragatina has quit IRC03:34
*** bkopilov has joined #openstack-nova03:35
*** salv-orlando has quit IRC03:37
*** vladikr has quit IRC03:40
*** vladikr has joined #openstack-nova03:41
*** lbragstad has joined #openstack-nova03:49
*** yamahata has joined #openstack-nova03:50
*** dklyle has quit IRC03:50
*** dklyle has joined #openstack-nova03:50
openstackgerritJay Pipes proposed openstack/nova master: Add generation column to consumers table  https://review.openstack.org/56540003:54
openstackgerritJay Pipes proposed openstack/nova master: Add Project object  https://review.openstack.org/56540103:54
openstackgerritJay Pipes proposed openstack/nova master: Add User object to placement modeling  https://review.openstack.org/56540203:54
openstackgerritJay Pipes proposed openstack/nova master: add Consumer object  https://review.openstack.org/56540303:54
openstackgerritJay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer  https://review.openstack.org/56540403:54
openstackgerritJay Pipes proposed openstack/nova master: move consumer ensure to API layer  https://review.openstack.org/56540503:54
openstackgerritJay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id  https://review.openstack.org/56540603:54
openstackgerritJay Pipes proposed openstack/nova master: prep for consumer generation in API layer  https://review.openstack.org/56540703:54
*** yamamoto has quit IRC04:01
*** dklyle has quit IRC04:01
*** dklyle has joined #openstack-nova04:01
*** yamamoto has joined #openstack-nova04:04
*** yamamoto has quit IRC04:05
*** hongbin has quit IRC04:08
*** yamamoto has joined #openstack-nova04:15
*** yamamoto has quit IRC04:20
*** vladikr has quit IRC04:22
*** vladikr has joined #openstack-nova04:23
*** namnh has joined #openstack-nova04:26
*** Zames has joined #openstack-nova04:30
*** yamamoto has joined #openstack-nova04:31
*** Zames has quit IRC04:32
*** fragatina has joined #openstack-nova04:33
*** salv-orlando has joined #openstack-nova04:33
*** yamamoto has quit IRC04:34
*** yamamoto has joined #openstack-nova04:34
openstackgerritTetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared  https://review.openstack.org/55645004:37
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates  https://review.openstack.org/55651404:37
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs  https://review.openstack.org/56184204:37
*** salv-orlando has quit IRC04:37
*** fragatina has quit IRC04:40
*** fragatina has joined #openstack-nova04:43
*** dave-mccowan has quit IRC04:43
*** sree has joined #openstack-nova04:47
openstackgerritAditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin  https://review.openstack.org/53437104:48
*** fragatina has quit IRC04:49
*** cfriesen has quit IRC04:50
*** cfriesen_ has joined #openstack-nova04:50
*** sree has quit IRC04:50
*** vivsoni__ has joined #openstack-nova04:51
*** lbragstad has quit IRC04:51
*** sree has joined #openstack-nova04:51
*** vladikr has quit IRC04:54
*** sree has quit IRC04:55
*** vladikr has joined #openstack-nova04:57
*** wolverineav has quit IRC04:58
*** wolverineav has joined #openstack-nova04:59
*** salv-orlando has joined #openstack-nova05:01
*** salv-orlando has quit IRC05:02
*** wolverineav has quit IRC05:03
*** links has joined #openstack-nova05:03
*** fragatina has joined #openstack-nova05:07
*** wolverineav has joined #openstack-nova05:09
*** fragatina has quit IRC05:12
*** bkopilov_ has joined #openstack-nova05:19
*** takashin has joined #openstack-nova05:22
openstackgerritMerged openstack/nova master: libvirt: Drop BAD_LIBVIRT_NUMA_VERSIONS  https://review.openstack.org/56401105:22
openstackgerritmelanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399005:25
openstackgerritmelanie witt proposed openstack/nova master: Update layout docs for running console proxies  https://review.openstack.org/55748905:25
openstackgerritmelanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536705:25
*** vivsoni__ has quit IRC05:25
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2)  https://review.openstack.org/56464305:26
*** wolverineav has quit IRC05:28
*** fragatina has joined #openstack-nova05:29
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3)  https://review.openstack.org/56464505:31
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060805:36
*** jogo has quit IRC05:37
*** sdeath has joined #openstack-nova05:38
*** sree has joined #openstack-nova05:41
*** namnh has quit IRC05:44
*** psachin has joined #openstack-nova05:49
*** Eran_Kuris has joined #openstack-nova05:50
*** fragatina has quit IRC05:51
*** jogo has joined #openstack-nova05:52
*** sree has quit IRC06:00
*** namnh has joined #openstack-nova06:00
*** sree has joined #openstack-nova06:01
*** fragatina has joined #openstack-nova06:04
*** sree has quit IRC06:05
*** namnh has quit IRC06:05
*** namnh has joined #openstack-nova06:06
*** gongysh has joined #openstack-nova06:07
*** slaweq has joined #openstack-nova06:11
*** namnh has quit IRC06:11
*** homeski has quit IRC06:12
*** cfriesen_ has quit IRC06:15
*** slaweq has quit IRC06:15
*** cfriesen_ has joined #openstack-nova06:15
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948306:21
*** janki has joined #openstack-nova06:21
*** janki has quit IRC06:26
*** gongysh has quit IRC06:27
*** gongysh has joined #openstack-nova06:31
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.63 - List/Show all server migration types  https://review.openstack.org/43083906:32
*** cfriesen_ has quit IRC06:35
*** cfriesen_ has joined #openstack-nova06:35
*** namnh has joined #openstack-nova06:38
*** dikonoor has quit IRC06:38
*** namnh has quit IRC06:42
*** cfriesen__ has joined #openstack-nova06:47
*** cfriesen_ has quit IRC06:48
*** gongysh has quit IRC06:49
*** baoli has joined #openstack-nova06:52
*** armaan has quit IRC06:58
*** baoli has quit IRC06:59
*** salv-orlando has joined #openstack-nova07:04
*** rcernin has quit IRC07:06
*** salv-orlando has quit IRC07:09
*** salv-orlando has joined #openstack-nova07:10
*** gongysh has joined #openstack-nova07:12
*** gongysh has quit IRC07:13
*** salv-orlando has quit IRC07:14
*** gongysh has joined #openstack-nova07:17
*** gjayavelu has joined #openstack-nova07:17
*** baoli has joined #openstack-nova07:20
*** tianhui has quit IRC07:21
*** tianhui has joined #openstack-nova07:21
*** david-lyle has joined #openstack-nova07:22
*** david-lyle has quit IRC07:24
*** dklyle has quit IRC07:24
*** baoli has quit IRC07:25
*** zz9pzza has joined #openstack-nova07:26
*** gjayavelu has quit IRC07:30
*** cfriesen__ has quit IRC07:31
*** tomtom001 has quit IRC07:31
*** tomtom001 has joined #openstack-nova07:32
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Fix the policy argument in server-group-create  https://review.openstack.org/56469907:35
*** sdeath has quit IRC07:40
*** namnh has joined #openstack-nova07:46
*** namnh has quit IRC07:51
*** rmart04 has joined #openstack-nova07:59
*** ragiman_ has joined #openstack-nova08:00
*** mdnadeem has joined #openstack-nova08:00
*** mgoddard has joined #openstack-nova08:04
*** namnh has joined #openstack-nova08:06
*** gongysh has quit IRC08:06
*** namnh_ has joined #openstack-nova08:09
*** slaweq has joined #openstack-nova08:11
*** namnh has quit IRC08:12
*** slaweq has quit IRC08:15
*** armaan has joined #openstack-nova08:16
*** baoli has joined #openstack-nova08:16
*** namnh_ has quit IRC08:17
*** rmart04 has quit IRC08:20
*** baoli has quit IRC08:21
*** mdbooth has joined #openstack-nova08:26
*** baoli has joined #openstack-nova08:30
*** belmoreira has joined #openstack-nova08:33
*** baoli has quit IRC08:35
*** ralonsoh has joined #openstack-nova08:37
*** derekh has joined #openstack-nova08:38
*** belmoreira has quit IRC08:42
*** sambetts|afk is now known as sambetts08:50
*** yamahata has quit IRC08:53
*** gongysh has joined #openstack-nova08:55
*** moshele has joined #openstack-nova08:56
*** psachin has quit IRC08:56
*** namnh has joined #openstack-nova08:57
*** armaan has quit IRC08:59
*** armaan has joined #openstack-nova09:00
*** armaan has quit IRC09:04
*** slaweq has joined #openstack-nova09:07
*** cdent has joined #openstack-nova09:08
*** hjensas has quit IRC09:12
openstackgerritMaciej Kucia proposed openstack/nova-specs master: Fix Non-unique network names in Servers IPs API response  https://review.openstack.org/55812509:18
*** armaan has joined #openstack-nova09:19
*** Dinesh_Bhor has quit IRC09:21
*** slaweq has quit IRC09:22
*** slaweq has joined #openstack-nova09:23
*** dklyle has joined #openstack-nova09:26
*** mdbooth has quit IRC09:28
*** namnh has quit IRC09:36
*** sree has joined #openstack-nova09:37
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc  https://review.openstack.org/52952009:37
*** sree has quit IRC09:42
*** gongysh has quit IRC09:44
*** mdbooth has joined #openstack-nova09:45
*** lifeless_ is now known as lifeless09:46
*** cdent has quit IRC09:50
*** takashin has left #openstack-nova10:00
openstackgerritStephen Finucane proposed openstack/nova master: Remove get_config_vhostuser  https://review.openstack.org/56547110:12
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ  https://review.openstack.org/56547210:12
stephenfindansmith, sahid: When you're about, easy clean up I spotted after review of tx/rx queue size https://review.openstack.org/56547110:13
stephenfinClearly someone need to document what the heck is going on there because that should have been an easy spot10:13
stephenfinkashyap: Do you intend to tackle the two remaining cleanups noted here? https://review.openstack.org/#/c/564012/4/nova/virt/libvirt/driver.py10:14
*** hjensas has joined #openstack-nova10:25
*** hjensas has quit IRC10:25
*** hjensas has joined #openstack-nova10:25
*** baoli has joined #openstack-nova10:32
*** bkopilov has quit IRC10:33
*** bkopilov_ has quit IRC10:33
openstackgerritMerged openstack/python-novaclient master: Make sure microversion < 2.62 does not show host(Id) for instance actions  https://review.openstack.org/56478210:36
*** baoli has quit IRC10:36
*** dougshelley66 has joined #openstack-nova10:37
*** tbachman has quit IRC10:45
*** nicolasbock has joined #openstack-nova10:48
*** rmart04 has joined #openstack-nova10:48
*** baoli has joined #openstack-nova10:53
openstackgerritMatthew Booth proposed openstack/nova master: libvirt: Fix misleading debug msg "Instance is running"  https://review.openstack.org/56523410:58
*** nicolasbock has quit IRC11:09
*** dougshelley66 has quit IRC11:15
*** dklyle has quit IRC11:15
*** gongysh has joined #openstack-nova11:15
*** moshele has quit IRC11:21
*** slaweq has quit IRC11:22
*** nicolasbock has joined #openstack-nova11:22
*** mriedem has joined #openstack-nova11:38
*** zz9pzza has quit IRC11:39
*** dougshelley66 has joined #openstack-nova11:45
*** yamamoto has quit IRC11:45
*** slaweq has joined #openstack-nova11:50
*** moshele has joined #openstack-nova11:51
*** zz9pzza has joined #openstack-nova11:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs  https://review.openstack.org/56184211:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries  https://review.openstack.org/55804511:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree  https://review.openstack.org/55948011:52
*** pchavva has joined #openstack-nova11:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories  https://review.openstack.org/55955411:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: WIP: Support nested allocation candidates  https://review.openstack.org/56548711:52
*** jmccarthy has joined #openstack-nova11:54
*** yamamoto has joined #openstack-nova11:55
*** yamamoto has quit IRC11:55
*** dklyle has joined #openstack-nova11:56
*** armaan has quit IRC11:56
*** armaan has joined #openstack-nova11:56
*** baoli has quit IRC11:57
*** armaan has joined #openstack-nova11:57
*** dklyle has quit IRC11:58
*** david-lyle has joined #openstack-nova11:58
openstackgerritsahid proposed openstack/nova master: hardware: only take into account small pages  https://review.openstack.org/53216811:59
*** krtaylor has joined #openstack-nova12:02
*** armaan has quit IRC12:06
*** bkopilov_ has joined #openstack-nova12:06
*** dave-mccowan has joined #openstack-nova12:08
*** Eran_Kuris has quit IRC12:08
*** bkopilov has joined #openstack-nova12:10
*** lucas-hungry is now known as lucasagomes12:11
*** tbachman has joined #openstack-nova12:11
*** sree has joined #openstack-nova12:12
*** mchlumsky has quit IRC12:16
*** sree has quit IRC12:17
*** edmondsw has joined #openstack-nova12:19
*** mchlumsky has joined #openstack-nova12:20
*** yamamoto has joined #openstack-nova12:24
*** Tom-Tom has joined #openstack-nova12:26
*** Tom-Tom has quit IRC12:30
*** tomtom001 has quit IRC12:32
*** tomtom001 has joined #openstack-nova12:32
*** liverpooler has joined #openstack-nova12:37
*** baoli has joined #openstack-nova12:37
openstackgerritMatt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399012:37
openstackgerritMatt Riedemann proposed openstack/nova master: Update layout docs for running console proxies  https://review.openstack.org/55748912:37
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536712:37
mriedemmelwitt: fixed up that bottom patch so tests should pass now ^12:37
efriedō/12:40
*** baoli has quit IRC12:41
*** weshay is now known as weshay|ruck12:46
*** slaweq has quit IRC12:47
*** moshele has quit IRC12:57
*** moshele has joined #openstack-nova12:59
*** tbachman has quit IRC13:00
mriedemvdrok: 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
johnthetubaguyI think the traits support should cover the other cases now, but I could be miss-remembering13:02
johnthetubaguyhaven't heard that anyone has successfully moved between them though13:02
*** artom has joined #openstack-nova13:03
*** tinwood has quit IRC13:04
johnthetubaguyalthough that was the non-exact case, thinking about it13:04
*** tinwood has joined #openstack-nova13:04
*** hjensas has quit IRC13:05
*** lyan has joined #openstack-nova13:07
TheJuliamriedem: 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 those13:07
*** armaan has joined #openstack-nova13:07
*** lyan is now known as Guest3398513:07
jrollplease do13:08
jrollwe should probably also raise an exception for nodes that don't have a resource class at this point13:08
*** david-lyle has quit IRC13:08
*** armaan has quit IRC13:10
*** armaan_ has joined #openstack-nova13:10
jrollmriedem: should we be removing the baremetal_enabled_filters option altogether?13:10
*** slaweq has joined #openstack-nova13:11
jrollreading the deprecation reason, seems like we should13:11
TheJuliajroll: well, we set it by default upon create now, the conundrum being actual user use13:11
jrollTheJulia: we set a default resource class on node create?13:12
TheJuliajroll: we do13:12
jrollO_o13:12
jrollalso, we've been logging warnings since pike, I don't think we can hold back just because people are being lazy13:12
TheJuliawhich people though?13:13
jrolloperators?13:13
TheJuliawell, the users of the flavors are the ones impacted13:14
openstackgerritMerged openstack/nova-specs master: Return resources of entire trees in Placement  https://review.openstack.org/55946613:15
jrollTheJulia: 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 IRC13:15
TheJuliayup13:15
TheJuliaI do agree13:15
jrollalso, fwiw, there's only a default resource class if the operator sets one: https://github.com/openstack/ironic/blob/d917f32060d4f7237b9b4ecd6e0e769bc41d420c/ironic/conf/default.py#L7413:15
*** esberglu has joined #openstack-nova13:16
TheJuliahmm, I thought that was defaulted to a value, maybe I'm thinking of other config someplace13:18
*** tssurya has joined #openstack-nova13:19
mriedemthat reminds me of a bug in pike....sec13:28
mriedemhttps://bugs.launchpad.net/nova/+bug/176533413:28
openstackLaunchpad bug 1765334 in OpenStack Compute (nova) "Ironic resource class may not be put into inventory in Pike" [Undecided,New]13:28
mriedemif you set the resource class on the node after it's created, not during create, the inventory for it doesn't get represented in placement13:28
*** baoli_ has joined #openstack-nova13:29
*** baoli_ has quit IRC13:30
artomShould we update the dashboard links from https://wiki.openstack.org/wiki/Nova/GerritDashboards ? 'cuz they're showing up empty for me13:30
mriedeminadvertantly fixed in queens though so i'm not sure what, if anything, we should do about that bug in pike13:30
mriedemartom: sure, they're likely all busted due to new gerrit13:30
artommriedem, 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
jrollmriedem: any reason not to backport https://review.openstack.org/#/c/518294/ ?13:31
mriedemjroll: idk, it's tied to the vgpu blueprint, i don't know if there would be other side effects13:32
jrollheh, was hoping you'd know because I certainly don't :)13:32
*** lbragstad has joined #openstack-nova13:32
*** mdbooth has quit IRC13:32
*** openstackgerrit has quit IRC13:34
artomAh, cuz it's not Jenkins that's voting now, it's Zuul13:34
* artom tries to fix it13:34
*** Eran_Kuris has joined #openstack-nova13:35
*** mdbooth has joined #openstack-nova13:35
mriedemok so baremetal_enabled_filters is only used if use_baremetal_filters is True, and that defaults to False13:36
mriedemi'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
mriedemi 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 gate13:37
mriedemso it would be nice if someone that does both pci/numa and ironic could try that out before we removed those deprecated options13:37
artomDoes a NUMATopologyFilter even make sense for Ironic? You're getting the whole machine regardles...13:37
mriedemif your single compute installation supports both vms and bms13:38
mriedemthen you'll need to run that13:38
mriedemif you support numa + vm13:38
*** dklyle has joined #openstack-nova13:38
artommriedem, ah, in a mixed cloud case13:39
artommriedem, well, the filter does check for host_topology, and if its absent, passes the host13:39
artomWhat does an ironic host expose for that?13:39
mriedemnothing13:39
mriedemso it might be ok, and i can't remember what at the time was making me think it might not work13:39
artomAh, no, hold up, I was wrong13:39
mriedemi think i found something during code inspection13:40
artomIf you request a topology and the host doens't have it, it fails the host13:40
artomWhich... is fine, actually :)13:40
mriedemright it would skip the ironic node13:40
mriedemwhich is what we want13:40
artomAnd if you request nothing and the host has nothing, it passes13:40
artomWhich is also fine13:41
*** Tom-Tom has joined #openstack-nova13:41
mriedemowalsh: does tripleo still rely on the exact core/ram/disk filters for baremetal?13:42
mriedemartom: yeah which was shown in https://review.openstack.org/#/c/553412/13:42
artommriedem, logs for that have been purged :(13:43
artomBut the ironic job passed, so yey :)13:43
mriedemyeah i verified it ran and passed the filter in the ironic job,13:44
*** felipemonteiro__ has joined #openstack-nova13:44
mriedembut as noted, the flavors used don't have any numa topology requests13:44
artomHrmm, so it didn't fully test it13:45
artomIn fact, it didn't test the host topology bit at all :(13:45
artomAnd that's the thing we're unsure about13:45
artomSince numa_topology_filter.py L86 will evaluate to false immediately when there's no request_topology13:46
mriedemjroll: uh oh https://docs.openstack.org/ironic/latest/install/configure-compute.html still tells people to set use_baremetal_filters=True13:46
mriedemartom: right13:47
dansmithartom: if you want to get a machine with a specific topology (like more than one node) wouldn't you want the filter?13:47
jrollmriedem: oof13:47
artomdansmith, for Ironic? Wouldn't that be a custom trait?13:47
mriedemi could try and hack up a numa functional test in-tree and use the ironic driver, or a fake ironic driver13:47
artomLike SUPER_GOLD_3_NUMA_NODES or whatever?13:47
dansmithartom: ah, I guess with CUSTOM_IRONIC_SILVER type scheduling that's true13:47
*** hemna_ has joined #openstack-nova13:48
* artom still has no idea wtf the SILVER and GOLD stuff is all about13:48
dansmithartom: just service levels, if you have a couple different classes of box13:49
*** jmccarthy has quit IRC13:49
*** felipemonteiro_ has joined #openstack-nova13:49
mriedemif you combine 5 of them you get a power ranger super bot thing13:50
artomNah brah, Captain Planet13:50
mriedemlame13:50
artomDude13:50
artomHe's our hero13:50
artomGoing to bring pollution down to zero13:50
* jroll starts using 2_XEON_E5_4000_192_GB_RAM_3_NVME_1_RUST as an example resource class instead13:50
mriedemonly a canadian would think captain tree hugger is a cool super hero13:50
artomHow can you not dig that?13:51
*** jmccarthy has joined #openstack-nova13:51
mriedemspeaking of, https://adequateman.deadspin.com/why-your-childrens-television-program-sucks-paw-patrol-1758314083#_ga=2.144903474.746352317.1525182679-529658380.152105107413:51
artomYeah well, I'll be here smelling my flowers while you choke on your coal roller13:51
*** awaugama has joined #openstack-nova13:52
mriedemcoal jobs f yeah! 'merica!13:52
mriedemok, ok, so back to this13:52
mriedemi guess we can drop the exact filters13:52
dansmithyeah, so this went off the rails :)13:52
mriedemand drop the other ironic specific options in a follow up13:52
*** slaweq has joined #openstack-nova13:52
mriedemwould like owalsh or one of the other 30 tripleo heros to tell me if they still rely on those13:52
*** felipemonteiro__ has quit IRC13:53
mriedemartom: dansmith: i'm assuming you can find out?13:53
*** hjensas has joined #openstack-nova13:53
*** hjensas has quit IRC13:53
*** hjensas has joined #openstack-nova13:53
mriedemEmilienM: ^?13:53
*** slaweq has quit IRC13:53
artommriedem, I don't think we have more tools for finding out than you do :/13:53
*** slaweq has joined #openstack-nova13:53
* mriedem does the unthinkable and jumps into the #tripleo channel13:54
dansmithmriedem: I'm pretty sure they're using resource classes now13:54
*** links has quit IRC13:54
*** dklyle has quit IRC13:55
EmilienMmriedem: 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 around13:57
*** hongbin has joined #openstack-nova13:59
*** dikonoor has joined #openstack-nova14:01
*** jmccarthy has quit IRC14:01
*** dougshel_ has joined #openstack-nova14:04
*** felipemonteiro_ has quit IRC14:05
*** dougshelley66 has quit IRC14:05
*** felipemonteiro_ has joined #openstack-nova14:05
owalshEmilienM, mriedem: yea, landed in pike https://review.openstack.org/49085114:06
EmilienMowalsh: awesome14:06
mriedemok i'll push changes to docs and test modules in tripleo to remove the references14:08
mriedemget my tripleo ATC badge14:08
*** wolverineav has joined #openstack-nova14:11
*** READ10 has joined #openstack-nova14:11
*** armaan_ has quit IRC14:13
*** dikonoor has quit IRC14:14
*** armaan has joined #openstack-nova14:14
*** mriedem has quit IRC14:19
*** mriedem has joined #openstack-nova14:19
*** sree has joined #openstack-nova14:19
*** openstackgerrit has joined #openstack-nova14:20
openstackgerritJay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer  https://review.openstack.org/56540414:20
openstackgerritJay Pipes proposed openstack/nova master: move consumer ensure to API layer  https://review.openstack.org/56540514:20
openstackgerritJay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id  https://review.openstack.org/56540614:20
openstackgerritJay Pipes proposed openstack/nova master: prep for consumer generation in API layer  https://review.openstack.org/56540714:20
openstackgerritJay Pipes proposed openstack/nova master: increment consumer generation when allocs change  https://review.openstack.org/56551314:20
*** tbachman has joined #openstack-nova14:21
*** sree has quit IRC14:24
*** mlavalle has joined #openstack-nova14:24
*** felipemonteiro__ has joined #openstack-nova14:27
*** felipemonteiro_ has quit IRC14:30
*** armaan has quit IRC14:33
openstackgerritMatt Riedemann proposed openstack/nova master: Remove ExactCoreFilter ExactDiskFilter ExactRamFilter  https://review.openstack.org/50888614:33
dansmithdum dum duuuum14:34
*** slaweq has quit IRC14:34
openstackgerritJay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id  https://review.openstack.org/56540614:34
openstackgerritJay Pipes proposed openstack/nova master: prep for consumer generation in API layer  https://review.openstack.org/56540714:34
openstackgerritJay Pipes proposed openstack/nova master: increment consumer generation when allocs change  https://review.openstack.org/56551314:34
*** gongysh has quit IRC14:37
*** eharney has quit IRC14:39
openstackgerritMatt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399014:46
openstackgerritMatt Riedemann proposed openstack/nova master: Update layout docs for running console proxies  https://review.openstack.org/55748914:46
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536714:46
*** moshele has quit IRC14:47
openstackgerritMathieu Gagné proposed openstack/nova master: Add user_id to RequestSpec  https://review.openstack.org/56534014:48
*** fishbone_ has joined #openstack-nova14:49
*** bpoulos has joined #openstack-nova14:50
openstackgerritBrianna Poulos proposed openstack/nova master: Implement certificate_utils  https://review.openstack.org/47994914:50
openstackgerritBrianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths  https://review.openstack.org/56126214:50
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620414:50
openstackgerritBrianna Poulos proposed openstack/nova master: Add notification support for trusted_certs  https://review.openstack.org/56326914:50
*** eharney has joined #openstack-nova14: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
mgagneCan the CachingScheduler still be used in master? (and therefore bypassing placement API IIRC)14:54
owalshmriedem: based on the commit msg in https://review.openstack.org/565512, puppet should stop setting use_baremetal_filters and baremetal_enabled_filters?15:00
mriedemuse_baremetal_filters is deprecated15:01
mriedemowalsh: for that specific patch,15:01
mriedemthe only filers in baremetal_enabled_filters were the 3 exact filters we're removing15:01
mriedemso if baremetal_enabled_filters isn't going to have any filters in it, then use_baremetal_filters should be False15:01
*** yamamoto has quit IRC15:03
*** tbachman has quit IRC15:03
owalshok, I'll push a new patch to make it so15:03
mriedemowalsh: i'm not following, is there something wrong with https://review.openstack.org/#/c/565512/ ?15:04
*** rmart04 has quit IRC15:05
owalshmriedem: that's just a spec test, need to change the behaviour in manifests/scheduler/filter.pp15:05
mriedemah ok15:05
*** mchlumsky has quit IRC15:05
mriedemmgagne: you have to fix this again https://review.openstack.org/#/c/565340/15:06
*** Eran_Kuris has quit IRC15:06
mriedemyou dropped my edit to fix the other compat test15:06
*** cfriesen__ has joined #openstack-nova15:06
mgagnemriedem: oops, didn't see that one :O15:07
mgagneok, gimme 2m, will fix shortly15:07
openstackgerritMathieu Gagné proposed openstack/nova master: Add user_id to RequestSpec  https://review.openstack.org/56534015:09
mgagnemriedem: why do the above tests not need it?15:09
mriedemthey do need it, they are broken15:09
mriedembut that can be fixed in a separate patch15:09
mriedemthis is why i asked for that validation yesterday15:09
*** r-daneel has quit IRC15:09
*** slaweq has joined #openstack-nova15:11
mgagneoh.... wonderful15:14
*** slaweq has quit IRC15:15
*** Eran_Kuris has joined #openstack-nova15:16
*** yamamoto has joined #openstack-nova15:16
openstackgerritStephen Finucane proposed openstack/nova master: Remove get_config_vhostuser  https://review.openstack.org/56547115:19
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ  https://review.openstack.org/56547215:19
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION  https://review.openstack.org/56551715:19
*** yamamoto has quit IRC15:21
openstackgerritBrianna Poulos proposed openstack/python-novaclient master: Microversion 2.63 - Add trusted_image_certificates  https://review.openstack.org/50039615:22
mriedembpoulos: what's the status on getting a spec amendment for that trusted certs stuff?15:22
bpoulosmriedem: it's written, I'm just waiting for approval to post it.15:22
mriedemi approve15:23
*** Eran_Kuris has quit IRC15:23
bpoulosmriedem: I wish your approval were enough :)15:23
*** fragatina has quit IRC15:23
bpoulosmriedem: unfortunately, it sometimes takes a while, and it can be very frustrating for us when we want to be able to do a quick turnaround15:24
*** fragatina has joined #openstack-nova15:24
mriedemunderstandable15:24
mriedemwould i get a grant kickback if i just pushed the spec amendment myself?15:25
mriedemyou don't have to answer that15:25
*** yamamoto has joined #openstack-nova15:31
*** Eran_Kuris has joined #openstack-nova15:33
*** MasterofJOKers has quit IRC15:33
*** MasterofJOKers has joined #openstack-nova15:34
openstackgerritStephen Finucane proposed openstack/nova master: Remove get_config_vhostuser  https://review.openstack.org/56547115:35
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ  https://review.openstack.org/56547215:35
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION  https://review.openstack.org/56551715:35
*** yamamoto has quit IRC15:36
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real  https://review.openstack.org/56504315:40
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags`  https://review.openstack.org/56392615:40
*** itlinux has joined #openstack-nova15:42
*** yamamoto has joined #openstack-nova15:44
*** yamamoto has quit IRC15:44
*** vladikr has quit IRC15:45
*** hamzy has quit IRC15:46
*** r-daneel has joined #openstack-nova15:47
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags`  https://review.openstack.org/56392615:47
*** vladikr has joined #openstack-nova15:47
mriedemdansmith: 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 pcid15:48
*** ragiman_ has quit IRC15:49
*** ccamacho has quit IRC15:49
dansmithack15:50
*** tinwood has quit IRC15:51
*** tinwood has joined #openstack-nova15:51
*** r-daneel has quit IRC15:51
*** r-daneel has joined #openstack-nova15:57
*** sree has joined #openstack-nova15:57
*** gyee has joined #openstack-nova15:57
openstackgerritSurya Seetharaman proposed openstack/nova master: Make association_refresh configurable  https://review.openstack.org/56552615:58
tssuryamriedem, efried: ^15:59
efriedack16:00
*** sree has quit IRC16:01
openstackgerritJay Pipes proposed openstack/nova master: add Consumer object  https://review.openstack.org/56540316:02
openstackgerritJay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer  https://review.openstack.org/56540416:02
openstackgerritJay Pipes proposed openstack/nova master: move consumer ensure to API layer  https://review.openstack.org/56540516:02
openstackgerritJay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id  https://review.openstack.org/56540616:02
openstackgerritJay Pipes proposed openstack/nova master: prep for consumer generation in API layer  https://review.openstack.org/56540716:02
openstackgerritJay Pipes proposed openstack/nova master: increment consumer generation when allocs change  https://review.openstack.org/56551316:02
*** dougshelley66 has joined #openstack-nova16:05
efriedtssurya: Reviewed.  Looks good.16:06
tssuryaefried: thanks for the quick review,16:06
efriedsure thing.  easy one :)16:06
*** mchlumsky has joined #openstack-nova16:07
*** dougshel_ has quit IRC16:07
*** tbachman has joined #openstack-nova16:11
*** danpb1 has joined #openstack-nova16:16
melwittmriedem: thanks16:17
mriedemmelwitt: only thing left is cleaning up https://review.openstack.org/#/c/565367/16:18
melwittk, looking at it now16:19
*** ralonsoh has quit IRC16:20
melwittphew, lots o docs16:23
melwittthanks for pointing out all of those16:23
*** armaan has joined #openstack-nova16:28
openstackgerritMatt Riedemann proposed openstack/nova master: Fix shelving a paused instance  https://review.openstack.org/54166016:28
mriedemeasy fix for a super latent problem ^16:30
*** spsurya has quit IRC16:31
*** Swami has joined #openstack-nova16:36
*** bhujay has quit IRC16:36
*** lucasagomes is now known as lucas-afk16:36
*** ralonsoh has joined #openstack-nova16:36
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh'  https://review.openstack.org/53438316:37
openstackgerritStephen Finucane proposed openstack/nova master: tox: Speed things up and document them  https://review.openstack.org/53438216:37
openstackgerritStephen Finucane proposed openstack/nova master: tox: Switch basepython to 3  https://review.openstack.org/56553416:37
*** armaan has quit IRC16:38
*** armaan has joined #openstack-nova16:39
SwamiHave 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 IRC16:39
openstackgerritStephen Finucane proposed openstack/nova master: Remove stale pip-missing-reqs tox test  https://review.openstack.org/56553516:40
openstackgerritmelanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536716:42
*** sambetts is now known as sambetts|afk16:42
*** yamamoto has joined #openstack-nova16:44
*** yamamoto has quit IRC16:50
openstackgerritmelanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536716:52
mriedemmelwitt: you were too fast16:53
melwittoops16:53
*** moshele has joined #openstack-nova16:53
*** derekh has quit IRC16:56
*** hamzy has joined #openstack-nova16:56
*** moshele has quit IRC16:56
*** gjayavelu has joined #openstack-nova16:59
*** imacdonn has quit IRC16:59
*** danpb1 has quit IRC17:00
*** armaan has quit IRC17:01
*** armaan has joined #openstack-nova17:02
*** hamzy has quit IRC17:03
*** r-daneel_ has joined #openstack-nova17:05
*** mgoddard has quit IRC17:06
*** r-daneel has quit IRC17:07
*** r-daneel has joined #openstack-nova17:08
openstackgerritmelanie witt proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536717:08
*** r-daneel_ has quit IRC17:10
openstackgerritSurya Seetharaman proposed openstack/nova master: Make association_refresh configurable  https://review.openstack.org/56552617:14
*** READ10 has quit IRC17:15
*** yamahata has joined #openstack-nova17:19
*** READ10 has joined #openstack-nova17:21
openstackgerritMatt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5)  https://review.openstack.org/51464217:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6)  https://review.openstack.org/51464317:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7)  https://review.openstack.org/51464417:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9)  https://review.openstack.org/51464617:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10)  https://review.openstack.org/51464717:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12)  https://review.openstack.org/54281917:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13)  https://review.openstack.org/54667417:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667517:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804317:24
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832617:24
*** artom has joined #openstack-nova17:24
melwittmriedem: 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_23117:25
*** mdnadeem has quit IRC17:27
mriedemhmm yeah http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/local.conf.txt.gz has NOVA_CONSOLE_PROXY_COMPUTE_TLS=True17:27
mriedemdoes the config have to be in the cell?17:28
mriedemif so, it's probably writing to the wrong nova config17:28
*** itlinux has quit IRC17:28
*** mdbooth has quit IRC17:29
mriedem[vnc] is configured properly here http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/etc/nova/nova.conf.txt.gz17:29
mriedemand here http://logs.openstack.org/49/479949/71/check/nova-next/7751dc4/logs/etc/nova/nova-cpu.conf.txt.gz17:29
melwittI 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
mriedemmelwitt: i wonder if it's something with using the queens UCA17:29
mriedemlibvirt might have changed17:30
mriedemre: https://review.openstack.org/#/c/396186/7/lib/nova_plugins/functions-libvirt17:30
melwittmaybe, guh I hope not. I have no clue how that stuff works17:30
melwittbut this used to work and I can't think of anything that's changed17:30
mriedemthis 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_54317:31
melwittyeah, saw that too17:31
melwitt"no cert file" sounds like it can't find the cert file. it deploys them to /etc/pki/libvirt-vnc/17:33
melwittoh wait, but the config says /etc/pki/nova-novnc/17:34
*** mgoddard has joined #openstack-nova17:34
mriedemlooking in the libvirtd logs, i can see qemu being executed with "tls,x509verify=/etc/pki/libvirt-vnc"17:35
melwittokay, so I wonder if these are wrong in nova.conf? not sure17:36
melwitt[vnc]17:36
melwittvencrypt_ca_certs = /etc/pki/nova-novnc/ca-cert.pem17:36
mriedemdon't know, but that wouldn't really explain how it was working before17:37
mriedemprobably need to ask stephenfin or danpb17:37
melwittyeah, true. I don't even know where teh nova-novnc dir comes from17:37
mriedemit's created by devstack https://review.openstack.org/#/c/396186/7/lib/nova17:37
melwittfwiw, 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
melwittbecause I didn't want to approve the patches until there was some demonstration of the proxy running with SSL enabled17:39
*** itlinux has joined #openstack-nova17:42
*** tbachman has quit IRC17:42
melwittwonder 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@15617:43
mriedemthat's what i was thinking,17:44
mriedembecause i had to deal with that for the multiattach stuff17:44
mriedemmelwitt: this https://github.com/openstack-dev/devstack/blob/master/stackrc#L63017:45
mriedemcould add a debug patch to make sure that libvirt-qemu exists17:45
melwitthm, 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-qemu17:46
*** yamamoto has joined #openstack-nova17:47
*** ralonsoh has quit IRC17:47
*** liverpooler has quit IRC17:48
mriedemmelwitt: you should kick an intern to make sure this article from 2011 is up to date https://wiki.libvirt.org/page/VNCTLSSetup17:50
*** vladikr has quit IRC17:50
melwittheh, that's a good doc17:51
*** yamamoto has quit IRC17:52
openstackgerritAditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin  https://review.openstack.org/53437117:54
*** hamzy has joined #openstack-nova17:57
wolverineavstephenfin: mriedem - updated the review with patch addressing comments ^^ take a look when you get a chance :)17:57
*** vladikr has joined #openstack-nova18:04
melwittbased 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/52261918:06
openstackLaunchpad 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-nova18:06
*** vivsoni has quit IRC18:07
*** vivsoni has joined #openstack-nova18:07
openstackgerritAditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin  https://review.openstack.org/53437118:08
mriedemmelwitt: so probably the thing to do is check /etc/group like in stackrc and fallback to qemu:qemu if libvirt-qemu doesn't exist18:10
*** tbachman has joined #openstack-nova18:11
*** slaweq has joined #openstack-nova18:11
*** larsks has joined #openstack-nova18:14
larsksdansmith: a hack: https://github.com/larsks/os-placement-tools18:15
*** slaweq has quit IRC18:15
*** liverpooler has joined #openstack-nova18:15
mriedemtssurya: comments inline18:17
*** vladikr has quit IRC18:17
tssuryamriedem: thanks18:17
openstackgerritJulia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity  https://review.openstack.org/56372218:17
*** moshele has quit IRC18:18
mriedemlarsks: "--fox" typo18:19
larsksmriedem: hah, you fell for my clever plan.18:19
larsksAlso, thanks, fixed it.18:20
mriedemis this for busted migrations?18:20
larsksmriedem: 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
larsksMaybe they were caused by busted migrations?18:20
larsksBut this fixed the immediate problem regardless of the cause.18:21
mriedemwhat version of nova?18:21
larsksIt is now pike, but it was upgraded from ocata.  And possibly from earlier.18:21
mriedempike GA or latest stable release?18:21
mriedempost pike GA we found a number of bugs in places where we didn't cleanup allocations properly18:21
mriedemthose should all be fixed now18:21
larsksUh...whatever red hat is distributing in rhel-osp 12.18:21
mriedemheh, fair enough18:22
* mriedem wonders if this coincides with dansmith asking me about writing a cli to migrate off the caching scheduler18:23
mriedemhttps://github.com/larsks/os-placement-tools/blob/master/check_placement.py#L16 is nice18:23
larsksThanks!18:24
dansmithmriedem: nope18:24
dansmithmriedem: it does coincide with me ruminating about deleting duplicate allocations in -placement last week though18:24
mriedemwonder if anyone ever started on that osc-placement change to orchestrate setting inventory on an entire placement provider aggregate set18:25
dansmithlarsks: that looks awesome18:25
mriedemunrelated to this besides placement tooling18:25
dansmithlarsks: you should check for instance state and only agree to --fix those in ACTIVE though18:25
dansmithlarsks: before ocata (ish?) you could have legit reasons for two allocations for one instance, because that's how we did migrations18:26
dansmithbut if the instance is ACTIVE and has two, that'll always need fixing I think18:26
*** Eran_Kuris_ has joined #openstack-nova18:26
dansmithmriedem: right?18:26
mriedemand/or don't touch any instances with a non-null task_state18:26
mriedembefore pike the computes would heal allocations18:26
dansmithyeah18:27
mriedembut yeah don't fix an instance that's undergoing a task_state transition18:27
mriedemyee be warned18:27
*** Eran_Kuris has quit IRC18:29
*** vladikr has joined #openstack-nova18:31
larsksSo noted.18:31
*** READ10 has quit IRC18:32
openstackgerritJay Pipes proposed openstack/nova master: Add Project object  https://review.openstack.org/56540118:32
openstackgerritJay Pipes proposed openstack/nova master: Add User object to placement modeling  https://review.openstack.org/56540218:32
openstackgerritJay Pipes proposed openstack/nova master: add Consumer object  https://review.openstack.org/56540318:32
openstackgerritJay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer  https://review.openstack.org/56540418:32
openstackgerritJay Pipes proposed openstack/nova master: move consumer ensure to API layer  https://review.openstack.org/56540518:32
openstackgerritJay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id  https://review.openstack.org/56540618:32
openstackgerritJay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict()  https://review.openstack.org/56540718:32
openstackgerritJay Pipes proposed openstack/nova master: increment consumer generation when allocs change  https://review.openstack.org/56551318:32
melwittlooks 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_68318:36
mriedemhuh18:36
mriedemnuts18:36
melwittyeah. back to the drawing board18:36
*** vivsoni has quit IRC18:41
*** vivsoni has joined #openstack-nova18:41
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536718:42
mriedemalright https://review.openstack.org/#/q/topic:bp/convert-consoles-to-objects+status:open is ready to go18:43
mriedemand it's the last day in the runway18:43
*** fragatina has quit IRC18:44
melwittso close ...18:45
mriedemwell, efried and dansmith can make that magic happen18:46
mriedem*or18:46
melwittnote 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 through18:46
mriedemyup18:46
efriedmriedem: subtle.18:46
*** yamamoto has joined #openstack-nova18:48
*** mgoddard has quit IRC18:52
*** Eran_Kuris_ has quit IRC18:53
*** yamamoto has quit IRC18:53
*** imacdonn has joined #openstack-nova18:54
melwittdansmith: 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@107818:54
dansmithif the thing isn't enabled, then stop_service won't do anything,18:56
dansmithbut I thnk that conductor case was just for when we were moving to super conductor18:57
dansmithalthough maybe it's still required since we generate the service id for services in the cell?18:57
*** felipemonteiro__ has quit IRC18:57
*** tssurya has quit IRC18:58
*** vladikr has quit IRC18:59
*** r-daneel has quit IRC19:00
melwittI wasn't sure why we can't assume they're enabled from the earlier call to start_nova_console_proxies19:00
*** r-daneel_ has joined #openstack-nova19:00
dansmithidk, I'm kinda into something else at the moment so I can't look deeply,19:01
dansmithbut what I said is what I remember off the top of my head for that stuff19:01
melwittyeah, np, thanks for that. not looking for a deep answer, wish I had written a comment on here from back when someone explained it19:02
*** r-daneel_ is now known as r-daneel19:02
openstackgerritEric Fried proposed openstack/nova master: placement: Granular GET /allocation_candidates  https://review.openstack.org/51775719:05
melwittI 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 runs19:05
melwittso during the unstack, you can't stop the per cell processes unless you enable them first19:06
dansmiththat's what I'm saying,19:07
dansmithbecause we generate their service names from the cell we're in19:07
dansmithn-cond-cell1, etc19:07
melwittright19:07
melwittwhat 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 run19:08
*** vladikr has joined #openstack-nova19:11
*** slaweq has joined #openstack-nova19:11
*** dougshelley66 has quit IRC19:15
*** slaweq has quit IRC19:15
openstackgerritJay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API  https://review.openstack.org/56556519:21
*** vladikr has quit IRC19:22
*** mgoddard has joined #openstack-nova19:23
*** mgoddard has quit IRC19:28
*** vladikr has joined #openstack-nova19:35
*** dpawlik has joined #openstack-nova19:37
*** damien_r has joined #openstack-nova19:39
*** moshele has joined #openstack-nova19:45
*** awaugama has quit IRC19:45
*** dpawlik has quit IRC19:47
*** vladikr has quit IRC19:47
*** yamamoto has joined #openstack-nova19:49
*** moshele has quit IRC19:50
*** slaweq has joined #openstack-nova19:50
*** damien_r has quit IRC19:51
*** dougshelley66 has joined #openstack-nova19:54
*** yamamoto has quit IRC19:55
*** liverpooler has quit IRC19:56
*** vladikr has joined #openstack-nova19:59
openstackgerritOliver Walsh proposed openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set  https://review.openstack.org/56557320:00
efriedowalsh: Do we need a bug for ^ ?20:07
*** fragatina has joined #openstack-nova20:07
*** mchlumsky has quit IRC20:12
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs  https://review.openstack.org/56522520:13
openstackgerritMatt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes*  https://review.openstack.org/56557820:13
mriedemefried: i thought about that, but it's not even tagged in rc120:15
mriedemso not really a good chance of someone hitting that besides red hat QE20:15
efriedmriedem: Sure, it's only been broken for a week, but still, paperwork.20:15
efriedmriedem: If you of all people don't need a bug, I'm sure not gonna push for it :)20:16
owalshawww but  I love paperwork :-(20:16
owalshefried, mriedem: yea, I'm lazy and it's not released yet but can raise one if you think it's necessary20:18
*** tssurya has joined #openstack-nova20:19
openstackgerritMerged openstack/nova master: Add user_id to RequestSpec  https://review.openstack.org/56534020:19
*** tssurya has quit IRC20:19
openstackgerritBrianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths  https://review.openstack.org/56126220:20
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620420:20
openstackgerritBrianna Poulos proposed openstack/nova master: Add notification support for trusted_certs  https://review.openstack.org/56326920:20
melwitthm, looks like all of the third party CIs suddenly are failing20:34
*** artom has quit IRC20:34
mriedemall or just powervm?20:35
melwitthyperv CI can't resolve a hostname, some others like powervm and virtuozzo, the log files aren't found20:35
mriedemi blame zuul20:35
melwittand it seems like this has happened starting today20:35
*** itlinux has quit IRC20:39
*** vladikr has quit IRC20:41
*** itlinux has joined #openstack-nova20:41
*** randomhack_ has joined #openstack-nova20:42
randomhack_was this bug ever solved? https://bugs.launchpad.net/nova/+bug/1633033. ran into this today on a Newton in20:43
openstackLaunchpad 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 system20:44
randomhack_s/love/live20:45
randomhack_known limitation of encrypted iscsi volumes?20:46
mriedemrandomhack_: lyarwood would know but it's really late for him (UK)20:48
openstackgerritMatt Riedemann proposed openstack/nova master: Implement granular policy rules for placement  https://review.openstack.org/52442520:49
openstackgerritMatt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes*  https://review.openstack.org/56557820:49
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs  https://review.openstack.org/56522520:49
randomhack_mriedem: thanks i’ll check tomorrow earlier20:49
melwittyeah. 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.html20:49
randomhack_melwitt: thanks!20:49
melwittbecause I thought lyarwood had a depends-on patch that ran live migration on top of the series20:50
melwittrandomhack_: double check with lyarwood to make sure20:50
*** yamamoto has joined #openstack-nova20:51
*** rajinir has joined #openstack-nova20:51
*** bpoulos has quit IRC20:52
melwittthis was one of the live migration test patches https://review.openstack.org/54507420:52
mriedemmelwitt: i think that was queens20:53
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/libvirt-qemu-native-luks.html :)20:53
mriedemyeah20:53
melwittah, you're right, it was queens20:53
*** hamzy has quit IRC20:55
*** vladikr has joined #openstack-nova20:56
*** yamamoto has quit IRC20:57
melwittceph 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_84554220:59
melwittthere'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#L23921:00
*** vladikr has quit IRC21:00
melwittoh, wait, it's a TODO21:00
melwittI'll try to do the todo21:01
mriedemit's a glorious todo21:01
melwittapparently. I don't know how this used to work before if there's an inherent issue with the volume dependencies in the test21:02
melwittI don't know volumes about volumes21:02
* mriedem does a rimshot21:02
melwittthanks21:03
mriedemso 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
mriedemwe then create a 2nd server from that image-defined bdm (the volume snapshot)21:03
mriedembecause delete_on_termination=True,21:04
mriedemwhen 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
mriedembut cinder won't let you delete a volume that has snapshots,21:04
mriedemso nova-compute will get an error from cinder and ignore it21:04
melwitthmm okay21:05
mriedemin here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L242221:05
mriedemyou should see "Failed to delete volume" in the n-cpu logs for that test21:05
melwittit's weird because what's happening is it's the test trying to cleanup the volumes it had created,21:06
melwittand it can't delete the snapshot because of dependent things21:06
mriedemhttp://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_00459921:06
melwittit's not failing because of anything about n-cpu as far as I can tell21:06
mriedemi believe jbernard had a really old patch for dealing with a failure to delete snapshots because they were busy with the cinder rbd driver21:07
mriedemhttps://review.openstack.org/#/c/281550/21:08
melwittI remember this patch21:08
melwittI 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 first21:09
*** tbachman has quit IRC21:09
openstackgerritJulia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity  https://review.openstack.org/56372221:11
openstackgerritMerged openstack/nova master: Use os.rename, not mv.  https://review.openstack.org/56246321:13
*** dklyle has joined #openstack-nova21:14
SwamiI 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='80821:23
Swami6'}]), 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 IRC21:29
*** wolverineav has joined #openstack-nova21:30
*** felipemonteiro__ has joined #openstack-nova21:30
*** pchavva has quit IRC21:30
*** felipemonteiro_ has joined #openstack-nova21:33
*** itlinux has quit IRC21:34
*** wolverineav has quit IRC21:34
*** felipemonteiro__ has quit IRC21:36
*** idlemind has joined #openstack-nova21:41
*** edmondsw has quit IRC21:48
*** yamamoto has joined #openstack-nova21:53
*** yamamoto has quit IRC21:59
*** READ10 has joined #openstack-nova22:08
openstackgerritMerged openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real  https://review.openstack.org/56504322:11
melwittso we have to backport that all the way to ocata? ^22:13
*** jmlowe has quit IRC22:13
openstackgerritJay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API  https://review.openstack.org/56556522:14
dansmithmelwitt: I don't really think we do22:19
dansmithit's pretty minor22:19
dansmithIMHO22:19
melwittokay22:19
dansmiththat reminds me I need to look at the second patch there22:19
*** Guest33985 has quit IRC22:19
*** dklyle has quit IRC22:21
melwittthat'll close out the blueprint, yeah?22:21
melwitt(after it merges)22:22
*** lbragstad has quit IRC22:22
*** dklyle has joined #openstack-nova22:26
*** yassine has joined #openstack-nova22:35
*** yassine is now known as Guest7228422:36
*** rcernin has joined #openstack-nova22:36
openstackgerritEric Fried proposed openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set  https://review.openstack.org/56557322:37
efriedmriedem: Quick re+A on this one ^22:39
openstackgerritJay Pipes proposed openstack/nova master: WIP: Add a microversion for consumer generation  https://review.openstack.org/56560422:40
*** jmlowe has joined #openstack-nova22:43
*** lbragstad has joined #openstack-nova22:46
*** slaweq has quit IRC22:46
*** slaweq has joined #openstack-nova22:47
*** yamamoto has joined #openstack-nova22:55
*** artom has joined #openstack-nova22:55
*** hongbin has quit IRC22:55
*** felipemonteiro_ has quit IRC22:58
*** moshele has joined #openstack-nova22:58
*** yamamoto has quit IRC23:00
*** moshele has quit IRC23:02
*** rajinir has quit IRC23:10
*** vivsoni_ has joined #openstack-nova23:12
*** vivsoni has quit IRC23:13
*** randomhack_ has quit IRC23:20
*** r-daneel has quit IRC23:23
*** slaweq has quit IRC23:27
*** itlinux has joined #openstack-nova23:34
*** takashin has joined #openstack-nova23:35
*** itlinux has quit IRC23:39
*** esberglu has quit IRC23:41
*** esberglu has joined #openstack-nova23:41
*** esberglu has quit IRC23:41
*** mlavalle has quit IRC23:46
mriedemmelwitt: would have to ask kashyap because there is a note in the patch about future patches23:49
mriedem"(A future Nova patch will also allow ability to remove CPU flags.)"23:49
*** r-daneel has joined #openstack-nova23:50
*** dklyle has quit IRC23:51
*** yamamoto has joined #openstack-nova23:56
*** dklyle has joined #openstack-nova23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!