Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-26
21:31:28 kashyap Sorry, haven't checked the thread yet. (It's late, and I'm slowly getting back to thinking in English, after 3-ish hours of straight Dutch.)
21:31:53 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM: Add proc_units_factor conf option https://review.openstack.org/554688
21:31:56 dansmith melwitt: ah I see now, I guess kashyap was confused?
21:32:19 melwitt okay, when you get a chance to check the thread, I'd try it out and see if it works. danpb used that technique to limit ListOpt choices for VNC auth_schemes for the console TLS stuff
21:33:07 cfriesen kashyap: I would suggest that it would make sense to do your check at LibvirtDriver.init_host(), then in a followup patch move the sanity checks from LibvirtDriver._get_guest_cpu_model_config() to init_host().
21:33:28 dansmith cfriesen: looks to me like it will be checked early during config parsing now
21:33:30 openstackgerrit Eric Fried proposed openstack/nova master: Slugification utilities for placement names https://review.openstack.org/556628
21:33:55 dansmith which is what we originally suggested
21:34:17 cfriesen sure, if we can restrict it via the config parsing for backport that'd be even better.
21:34:59 kashyap melwitt: Ah, this one: https://github.com/openstack/nova/blob/cd15c3d/nova/conf/vnc.py#L226,232
21:35:11 melwitt yes
21:35:12 kashyap melwitt: Interesting pointer; I'll give it a whirl and see what comes
21:35:43 melwitt cool, hope it works, else we have a bug in the vnc opts too :P
21:37:17 kashyap LOL
21:44:26 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Migrate tempest-dsvm-cells job to an in-tree job definition https://review.openstack.org/556656
21:49:28 mriedem so uh,
21:49:46 mriedem sahid would likely explode if i mentioned this, but https://review.openstack.org/#/c/458820/ could arguably require a microversion right?
21:49:59 mriedem i.e. i create an sriov port in neutron with trusted=true,
21:50:22 mriedem and try to create a server with that port, but nova api isn't new enough to move that in the pci request for scheduling,
21:50:31 mriedem so i get a server without a trusted VF
21:51:36 mriedem thought about this because of the bandwidth-aware scheduling stuff that relies on the qos policy on the ports in neutron, which impacts scheduling behavior in nova
21:52:28 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Filter resource providers by forbidden traits in db https://review.openstack.org/556472
21:52:29 openstackgerrit Chris Dent proposed openstack/nova master: Support forbidden traits in allocation_candidates in db https://review.openstack.org/556660
21:53:40 cdent edleafe, efried, jaypipes: that ^ batch of stuff is the initial db support for forbidden traits. most of the rest of the code is done too, but needs to be extracted from a lump of random stuff. I'm not super confident I've got enough tests yet.
21:54:18 efried ack
22:05:22 openstackgerrit Matt Riedemann proposed openstack/osc-placement stable/queens: Do not depend on jenkins user in devstack gate https://review.openstack.org/556666
22:05:36 openstackgerrit Matt Riedemann proposed openstack/osc-placement stable/queens: Migrate legacy-osc-placement-dsvm-functional job in-tree https://review.openstack.org/556635
22:19:44 openstackgerrit Eric Fried proposed openstack/nova master: Handle agg generation conflict in report client https://review.openstack.org/556669
22:20:02 efried dansmith, cdent, jaypipes, edleafe ^
22:20:39 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Create/lookup API services in cell0 on WSGI app startup https://review.openstack.org/556670
22:27:14 mriedem dansmith: i can't convince myself that there would be any upgrade impact with this ^
22:27:28 mriedem even if you had mistakenly started up API services pointing at cell1
22:29:05 dansmith well, you'd have some nova-api type services that will still list out of a service listing
22:29:06 dansmith from cell1
22:29:16 mriedem service list filters out api services
22:29:19 dansmith we don't need this wsgi_app stuff for metadata?
22:29:21 mriedem https://github.com/openstack/nova/blob/88f6c3b7b892f2a05066ac6f0e3353f4ce80f6b0/nova/api/openstack/compute/services.py#L48
22:29:50 dansmith hmm
22:30:07 mriedem is nova-metadata global or per-cell?
22:31:16 melwitt I think it's global
22:31:24 dansmith yeah, it hits api db
22:31:49 dansmith I really didn't think services would filter out api, that's kindof confusing to me
22:32:07 mriedem the os-services REST API is mostly biased to compute services
22:32:15 mriedem i.e. things you can enable/disable
22:32:21 mriedem and service group stuff
22:32:23 dansmith it shows scheduler
22:32:29 mriedem i know, but...
22:32:30 dansmith which is probably why I'm thinking that,
22:32:31 dansmith although,.
22:32:36 dansmith now that I think of it,
22:32:41 mriedem that all predates host mappings too
22:32:48 dansmith we didn't even have api service records until we needed it for SERVICE_VERSION
22:32:48 mriedem and this api relies on host mappings to do any actions on services
22:32:51 dansmith so that makes sense now
22:32:52 mriedem so it's restricted to compute
22:34:00 mriedem this is as close as we get to nova-metadata being global for cells v2 in our docs https://docs.openstack.org/nova/latest/user/cellsv2-layout.html#neutron-metadata-api-proxy
22:34:04 dansmith so I guess no upgrade impact, through a weird set of reasons
22:34:28 mriedem alternatively to this, i could base it on if CONF.database.connection is None
22:34:31 mriedem but..
22:35:05 mriedem but that just seems weird
22:35:54 mriedem anyway, there might be more fallout before https://review.openstack.org/#/c/555346/ is done
22:35:54 dansmith I guess I'm not sure why we should make this change
22:35:58 dansmith I mean, it's okay I guess,
22:36:21 mriedem ease of configuration/deployment i guess?
22:36:22 mriedem https://bugs.launchpad.net/nova/+bug/1757472
22:36:22 dansmith but it seems like more of a departure to have [database] be unconfigured
22:36:23 openstack Launchpad bug 1757472 in OpenStack Compute (nova) "Required to define database/connection when running services for nova_api cell" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)
22:36:24 dansmith I guess
22:36:44 mriedem would have to ask belmiro, but i also just triaged another duplicate of this same bug
22:37:03 mriedem if i've got access to the api db and cell0 is in there, then i don't really need the config
22:37:33 dansmith is that really the only place you'll blow up by not having that there?
22:37:46 mriedem we'll see
22:37:52 mriedem the service wouldn't start in devstack w/o it
22:38:00 dansmith yeah
22:38:01 mriedem i just rev'ed the devstack patch
22:38:31 dansmith I don't really see the justification, especially since config is where db urls and credentials are _supposed_ to be
22:38:38 dansmith but if it's really just that one place, then I guess meh
22:40:07 mriedem it certainly makes the db sync for cell0 a pain in the ass https://review.openstack.org/#/c/555346/3/lib/nova@742
22:40:17 mriedem if you're not using simple_cell_setup anyway
22:41:06 dansmith well, --all-cells will make that easier
22:41:19 dansmith but that's also kindof a weirdness
22:41:21 dansmith anyway, whatever
22:41:25 dansmith see what else breaks I guess
22:52:18 openstackgerrit melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381
22:52:19 openstackgerrit melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
22:52:19 openstackgerrit melanie witt proposed openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414
23:11:07 cfriesen why do we still call plug_vifs() when we call _create_domain_and_network() with vifs_already_plugged=True ?
#openstack-nova - 2018-03-27
00:55:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Create/lookup API services in cell0 on WSGI app startup https://review.openstack.org/556670
01:28:46 openstackgerrit melanie witt proposed openstack/nova master: rbd: use MAX_AVAIL stat for reporting bytes available https://review.openstack.org/556692
01:33:50 alex_xu_ efried: I replied again https://review.openstack.org/#/c/554305/2, a long version, hope I explain clear that
01:35:36 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
02:09:20 phuongnh join #openstack-dib
02:28:40 openstackgerrit jichenjc proposed openstack/nova master: Avoid live migrate to same host https://review.openstack.org/542689
02:45:52 openstackgerrit Artom Lifshitz proposed openstack/nova-specs master: NUMA-aware live migration https://review.openstack.org/552722
03:23:49 openstackgerrit Artom Lifshitz proposed openstack/nova-specs master: NUMA-aware live migration https://review.openstack.org/552722
04:10:33 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host to API and Conductor https://review.openstack.org/556513
04:11:44 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Record the host info in EventReporter https://review.openstack.org/556746
04:29:33 openstackgerrit Jake Yip proposed openstack/nova master: Add --until_deleted_at to db archive_deleted_rows https://review.openstack.org/556751
05:10:05 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
05:10:06 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: support shared and nested allocation candidates https://review.openstack.org/556514

Earlier   Later