| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-01 | |||
| 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 | |
| 15:22:21 | mriedem | bpoulos: what's the status on getting a spec amendment for that trusted certs stuff? | |
| 15:22:49 | bpoulos | mriedem: it's written, I'm just waiting for approval to post it. | |
| 15:23:05 | mriedem | i approve | |
| 15:23:32 | bpoulos | mriedem: I wish your approval were enough :) | |