| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-01 | |||
| 13:28:54 | 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:30:16 | 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:23 | mriedem | inadvertantly fixed in queens though so i'm not sure what, if anything, we should do about that bug in pike | |
| 13:30:42 | mriedem | artom: sure, they're likely all busted due to new gerrit | |
| 13:31:11 | artom | mriedem, I was secretly hoping someone else would do it, or at least show me how ;) | |
| 13:31:20 | artom | (Not the wiki part, the gerrit URL part) | |
| 13:31:22 | jroll | mriedem: any reason not to backport https://review.openstack.org/#/c/518294/ ? | |
| 13:32:03 | mriedem | jroll: idk, it's tied to the vgpu blueprint, i don't know if there would be other side effects | |
| 13:32:27 | jroll | heh, was hoping you'd know because I certainly don't :) | |
| 13:34:15 | artom | Ah, cuz it's not Jenkins that's voting now, it's Zuul | |
| 13:36:09 | mriedem | ok so baremetal_enabled_filters is only used if use_baremetal_filters is True, and that defaults to False | |
| 13:36:35 | 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:37:03 | 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:25 | 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:57 | artom | Does a NUMATopologyFilter even make sense for Ironic? You're getting the whole machine regardles... | |
| 13:38:34 | mriedem | if your single compute installation supports both vms and bms | |
| 13:38:39 | mriedem | then you'll need to run that | |
| 13:38:45 | mriedem | if you support numa + vm | |
| 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 | |