Earlier  
Posted Nick Remark
#openstack-nova - 2018-12-04
17:42:33 mriedem this is the libvirt code fwiw https://github.com/openstack/nova/blob/5f648dda49a6d5fe5ecfd7dddcb5f7dc3d6b51a6/nova/virt/libvirt/driver.py#L8609
17:43:13 cfriesen maybe add a new "topology" section?
17:43:14 mriedem so there are 3 options: (1) include it directly in the server response details, (2) add a new subresource endpoint to servers (for show on a single server only), (3) make it part of diagnostics (which is also per server)
17:43:26 mriedem at this point you might want to take those 3 options to the ML
17:43:37 cfriesen okay, can do.
17:43:52 sean-k-mooney well we are already passing in the instance to that function and its in the dirver so that seam very doable
17:43:59 mriedem b/c alex_xu and gmann will probably also have input
17:44:17 mriedem sean-k-mooney: b/c the information they get comes from the driver,
17:44:25 mriedem for what this wants, we don't really need the driver,
17:44:32 mriedem we're just dumping info out of the db
17:45:19 sean-k-mooney mriedem: that is true altough i wonder if any of the driver treat the extra specs different
17:45:43 mriedem idk, i don't deal in numa
17:45:53 mriedem nova/virt/hardware.py is where i stop
17:46:07 sean-k-mooney cfriesen: efried: would praobly be another person to take a look
17:46:19 efried wha, wha?
17:46:28 efried how far back am I reading?
17:46:34 mriedem just post to the ML and let efried catch up there
17:46:43 sean-k-mooney efried: https://review.openstack.org/#/c/612256/7/specs/stein/approved/show-server-numa-topology.rst
17:46:58 sean-k-mooney but ya we can disscuss on the ML too
17:47:48 sean-k-mooney efried: i was just wondering if we could make feature like this work well across powervm, libvirt, hyperv ectra
17:48:12 efried sean-k-mooney: PowerVM doesn't expose or allow you to muck with its numa topology.
17:49:22 sean-k-mooney but would i have info taht might make debuing for an admin simpler when dong device passhtou/cpu pinning/numa that could be useful to expose
17:50:01 sean-k-mooney i was just wondering how driver specific it would be or not
17:50:36 sean-k-mooney anyway im going to go grab food. ill be back later
17:54:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix guestfs.set_backend_settings call https://review.openstack.org/524748
18:31:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499
18:31:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995
18:33:37 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.openstack.org/612497
19:34:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995
19:34:29 mriedem jaypipes: i was just updating ^ and leaving comments probably while you were
19:34:35 mriedem see if your comments are addressed
19:34:50 mriedem oh wait
19:34:56 mriedem nvm, was thinking of https://review.openstack.org/609995
19:37:20 mriedem jaypipes: dansmith: bauzas: efried: i'm +1 on this series https://review.openstack.org/#/q/topic:bp/initial-allocation-ratios+(status:open+OR+status:merged) because i wanted to wait to +2 so that all of the changes can be approved together, since 2 and 3 in the series are kind of inter-dependent a bit,
19:37:45 mriedem so i'm looking for another +2 on the last 2 changes - the ones that deal with the online data migration
19:38:23 efried I'll look if you get desperate. But the discussion in placement has me convinced I am a tard when it comes to anything involving "data migration".
19:38:43 mriedem :/
19:43:00 jaypipes mriedem: ack. will look in a bit.
19:43:40 cfriesen maybe a weird question, but how does nova-manage handle authentication? is it based off the config file and whoever is allowed to run the executable?
19:44:30 cfriesen I'm trying to sort out dependencies
19:49:47 melwitt cfriesen: yes. I think it doesn't yet [or may never] do anything like call out to REST APIs (doesn't need keystone auth). the auth info for the database access is part of the [database]connection and [api_database]connection settings in the nova config file. and generally admin RequestsContext are used so nova-manage is able to access any database record
19:51:33 cfriesen melwitt: thanks, that confirms what I thought.
20:07:30 prometheanfire mriedem: ping :D
20:07:39 prometheanfire you remember what we've done in the past for libvirt fails on rhel based stuff and uppdating stable branches? https://review.openstack.org/622548 the intention is that they would backport to ocata afaik
20:12:02 mriedem prometheanfire: well, upper-constraints is less of an issue, it's lower-constraints that really matters for stable
20:12:29 mriedem i have no idea what the support matrix is for stable/ocata wrt centors
20:12:31 mriedem *centos
20:12:40 prometheanfire mriedem: true, problem is libvirt starting to fail due to a minor release...
20:13:14 prometheanfire on gentoo I've always left it uncapped so the latest stable is always used
20:13:24 prometheanfire haven't had complaints, yet
20:14:11 mriedem so centos 7.6 (released yesterday) has libvirt 4.5 and therefore libvirt-python 4.4 from u-c fails to build with it, because it needs to be >= 4.5 right?
20:14:31 prometheanfire something like that, ya
20:15:31 prometheanfire the author would know more I imagine
20:15:44 mriedem i left a comment, only wondering why 4.10 is used over 4.5
20:15:46 mriedem to match libvirt
20:16:27 prometheanfire mainly because it's the latest release and using a newer one is fine (and hopefully future proofs stuff a little)
20:20:01 mriedem ok, shrug; it's weird having something in u-c that is so dependent on native package versions
20:20:25 mriedem it would also be weird to me to update libvirt-python in stable/ocata u-c at this point, especially given what limited CI coverage we run on ocata now
20:21:22 mriedem i know nova doesn't run centos jobs on ocata (or anywhere ever)
20:21:25 prometheanfire ya, iirc libvirt-python is the only thing like that
20:21:40 mriedem what version of centos does infra support for ocata?
20:22:08 mriedem anyway, this is probably something for the -stable or -qa channel or something
20:22:11 mriedem not really a nova thing
20:27:17 prometheanfire k
20:56:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Note the aggregate allocation ratio restriction in scheduler docs https://review.openstack.org/620713
20:56:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add docs for (initial) allocation ratio configuration https://review.openstack.org/622588
20:56:37 mriedem yay docs
20:59:26 sean-k-mooney efried: you attended the GPU on openstack for sciense session in berlin
21:00:06 efried yes
21:00:24 efried sean-k-mooney: was that a question or a statement (because you saw me talk in the video)?
21:00:40 sean-k-mooney well statement i just finished watching it
21:00:54 sean-k-mooney did ye speak to them much after the video
21:01:25 sean-k-mooney looking at the options they were setting the missed a couple of things.
21:02:24 sean-k-mooney do you know if the benchmarks they were running are opensouce
21:02:40 sean-k-mooney i know its been a few monts so if you dont rememeber no worries
21:03:36 efried sean-k-mooney: I can give you the guy's email if it wasn't on the charts. I'm sure he would love to hear from you.
21:04:55 sean-k-mooney its on teh title slide
21:05:12 sean-k-mooney i might reach out to team
21:05:17 sean-k-mooney them
21:07:55 sean-k-mooney sicne they set isolcpus and did not use a dedicated emulator thread that will cause signifciant over head on 1 of the vm cores
21:32:26 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Use ksa adapter for cinder client https://review.openstack.org/508345
21:34:47 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
21:35:30 openstackgerrit Dave McCowan proposed openstack/nova master: Prevent logging of encryption key value https://review.openstack.org/622605
21:36:40 openstackgerrit Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844
22:44:38 mriedem gd it's fun having stuff queued up for at least 9-10 hours before tests run on it, each day, taking about a week to merge code, weeeee
22:45:36 sean-k-mooney mriedem is taht only after the zuul change or has it been that way for a while
23:53:16 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Provider config file schema and loading https://review.openstack.org/622622
23:57:13 efried jaypipes: code for the provider config YAML blueprint ^
23:57:30 efried other than test, it's actually complete I think.
#openstack-nova - 2018-12-05
01:46:15 openstackgerrit zhongshengping proposed openstack/nova master: Update mailinglist from dev to discuss https://review.openstack.org/621827
01:51:16 qwebirc716083 join #openstack-nova
02:38:43 openstackgerrit Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960
02:42:37 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
02:51:51 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Handle tags in _bury_in_cell0 https://review.openstack.org/621856
03:21:26 openstackgerrit Merged openstack/nova master: Remove utils.execute() calls from xenapi. https://review.openstack.org/619700
04:41:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Body verification for the lock action https://review.openstack.org/622835
05:47:44 izza_ hi guys
05:48:01 izza_ can anyone help us here on windows provisioning on openstack
06:30:30 openstackgerrit Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474
06:34:28 openstackgerrit Yongli He proposed openstack/nova master: Add numa topology and pci devices into server detail API https://review.openstack.org/621476

Earlier   Later