Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-01
13:39:04 artom mriedem, ah, in a mixed cloud case
13:39:18 artom mriedem, well, the filter does check for host_topology, and if its absent, passes the host
13:39:27 artom What does an ironic host expose for that?
13:39:35 mriedem nothing
13:39:49 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:51 artom Ah, no, hold up, I was wrong
13:40:08 mriedem i think i found something during code inspection
13:40:09 artom If you request a topology and the host doens't have it, it fails the host
13:40:17 artom Which... is fine, actually :)
13:40:23 mriedem right it would skip the ironic node
13:40:27 mriedem which is what we want
13:40:54 artom And if you request nothing and the host has nothing, it passes
13:41:02 artom Which is also fine
13:42:05 mriedem owalsh: does tripleo still rely on the exact core/ram/disk filters for baremetal?
13:42:41 mriedem artom: yeah which was shown in https://review.openstack.org/#/c/553412/
13:43:26 artom mriedem, logs for that have been purged :(
13:43:57 artom But the ironic job passed, so yey :)
13:44:47 mriedem yeah i verified it ran and passed the filter in the ironic job,
13:44:55 mriedem but as noted, the flavors used don't have any numa topology requests
13:45:17 artom Hrmm, so it didn't fully test it
13:45:33 artom In fact, it didn't test the host topology bit at all :(
13:45:37 artom And that's the thing we're unsure about
13:46:14 artom Since numa_topology_filter.py L86 will evaluate to false immediately when there's no request_topology
13:46:53 mriedem jroll: uh oh https://docs.openstack.org/ironic/latest/install/configure-compute.html still tells people to set use_baremetal_filters=True
13:47:08 mriedem artom: right
13:47:12 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:33 jroll mriedem: oof
13:47:34 artom dansmith, for Ironic? Wouldn't that be a custom trait?
13:47:40 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:50 artom Like SUPER_GOLD_3_NUMA_NODES or whatever?
13:47:53 dansmith artom: ah, I guess with CUSTOM_IRONIC_SILVER type scheduling that's true
13:49:28 dansmith artom: just service levels, if you have a couple different classes of box
13:50:08 mriedem if you combine 5 of them you get a power ranger super bot thing
13:50:31 artom Nah brah, Captain Planet
13:50:37 mriedem lame
13:50:41 artom Dude
13:50:45 artom He's our hero
13:50:55 artom Going to bring pollution down to zero
13:50:58 mriedem only a canadian would think captain tree hugger is a cool super hero
13:51:02 artom How can you not dig that?
13:51:39 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:48 artom Yeah well, I'll be here smelling my flowers while you choke on your coal roller
13:52:16 mriedem coal jobs f yeah! 'merica!
13:52:27 mriedem ok, ok, so back to this
13:52:34 mriedem i guess we can drop the exact filters
13:52:36 dansmith yeah, so this went off the rails :)
13:52:42 mriedem and drop the other ironic specific options in a follow up
13:52:58 mriedem would like owalsh or one of the other 30 tripleo heros to tell me if they still rely on those
13:53:07 mriedem artom: dansmith: i'm assuming you can find out?
13:53:09 mriedem EmilienM: ^?
13:53:45 artom mriedem, I don't think we have more tools for finding out than you do :/
13:54:57 dansmith mriedem: I'm pretty sure they're using resource classes now
13:57:18 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
14:06:17 owalsh EmilienM, mriedem: yea, landed in pike https://review.openstack.org/490851
14:06:47 EmilienM owalsh: awesome
14:08:03 mriedem ok i'll push changes to docs and test modules in tripleo to remove the references
14:08:12 mriedem get my tripleo ATC badge
14:20:18 openstackgerrit Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404
14:20:20 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
14:20:21 openstackgerrit Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406
14:20:22 openstackgerrit Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407
14:20:23 openstackgerrit Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513
14:33:44 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove ExactCoreFilter ExactDiskFilter ExactRamFilter https://review.openstack.org/508886
14:34:02 dansmith dum dum duuuum
14:34:44 openstackgerrit Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406
14:34:46 openstackgerrit Jay Pipes proposed openstack/nova master: prep for consumer generation in API layer https://review.openstack.org/565407
14:34:47 openstackgerrit Jay Pipes proposed openstack/nova master: increment consumer generation when allocs change https://review.openstack.org/565513
14:46:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
14:46:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Update layout docs for running console proxies https://review.openstack.org/557489
14:46:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367
14:48:03 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340
14:50:18 openstackgerrit Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
14:50:19 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
14:50:20 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
14:50:21 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
14:52:13 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:54:51 mgagne Can the CachingScheduler still be used in master? (and therefore bypassing placement API IIRC)
15:00:23 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:01:16 mriedem use_baremetal_filters is deprecated
15:01:23 mriedem owalsh: for that specific patch,
15:01:38 mriedem the only filers in baremetal_enabled_filters were the 3 exact filters we're removing
15:01:50 mriedem so if baremetal_enabled_filters isn't going to have any filters in it, then use_baremetal_filters should be False
15:03:14 owalsh ok, I'll push a new patch to make it so
15:04:46 mriedem owalsh: i'm not following, is there something wrong with https://review.openstack.org/#/c/565512/ ?
15:05:19 owalsh mriedem: that's just a spec test, need to change the behaviour in manifests/scheduler/filter.pp
15:05:27 mriedem ah ok
15:06:43 mriedem mgagne: you have to fix this again https://review.openstack.org/#/c/565340/
15:06:49 mriedem you dropped my edit to fix the other compat test
15:07:01 mgagne mriedem: oops, didn't see that one :O
15:07:16 mgagne ok, gimme 2m, will fix shortly
15:09:03 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340
15:09:07 mgagne mriedem: why do the above tests not need it?
15:09:19 mriedem they do need it, they are broken
15:09:27 mriedem but that can be fixed in a separate patch
15:09:36 mriedem this is why i asked for that validation yesterday
15:14:28 mgagne oh.... wonderful
15:19:42 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471
15:19:43 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472
15:19:44 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517
15:22:01 openstackgerrit Brianna Poulos proposed openstack/python-novaclient master: Microversion 2.63 - Add trusted_image_certificates https://review.openstack.org/500396

Earlier   Later