| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-11 | |||
| 17:31:10 | edleafe | dansmith: aren't we treating them as opaque blobs, as originally designed? I know there was the doubking up hack, but that should no longer be necessary | |
| 17:31:21 | cdent | edleafe: they've changed shape | |
| 17:31:27 | cdent | and they get manipulated | |
| 17:31:34 | cdent | so not really opaque (last I checked) | |
| 17:32:00 | bauzas | dansmith: not sure you saw the long convo I had with mdbooth but before I leave for 2.5 hours (will be back later), just food for thoughts | |
| 17:32:07 | cdent | I can't remember why they get manipulated | |
| 17:32:20 | cdent | My ability to index this stuff for memory is broken | |
| 17:33:02 | dansmith | edleafe: not in the scheduler | |
| 17:33:12 | dansmith | edleafe: I'm not having any problems with the code that treats it as opaque | |
| 17:33:14 | bauzas | dansmith: since we create mediated devices on the fly (or allocate them) based on allocations, and generate the guest config with that, thanks to allocations, it means that when you power off and then power on again, you get a guest that no longer has VGPUs because we don't pass (yet) the allocations to power_on() | |
| 17:33:37 | dansmith | bauzas: okay | |
| 17:33:56 | bauzas | dansmith: everything works seamlessly when booting, so https://review.openstack.org/#/c/528832/ is ready for reviews, but I'll need to address that issue in a follow-up | |
| 17:34:31 | homeski | Question for anyone ... | |
| 17:34:45 | homeski | I have a instance booted from volume, where the volume gets attached using the IDE bus | |
| 17:34:50 | homeski | is there a way, using OSP CLI, to have the boot device remounted using virtio ? | |
| 17:34:57 | homeski | ight now I have to manually `virsh edit` the XML | |
| 17:35:04 | homeski | Also, if I attach any additional volumes, they always get attached using IDE, even if I have `disk_bus=virtio` set on the volume metadata | |
| 17:35:10 | homeski | I have to keep messing with the XML manually | |
| 17:35:44 | dansmith | edleafe: and we can't get rid of the doubling until rocky, AFAIK, because pike computes still depends on it | |
| 17:36:22 | dansmith | homeski: see topic please | |
| 17:36:40 | homeski | gotcha, thx | |
| 17:36:54 | edleafe | dansmith: sure, what I was asking was if that was a one-off thing, and that we aren't doing any more of it | |
| 17:37:19 | dansmith | edleafe: that's the biggest wart, yeah | |
| 17:39:09 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.generation_for https://review.openstack.org/532922 | |
| 17:46:08 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 17:57:40 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: WIP: Do not normalize allocation_ratios https://review.openstack.org/532924 | |
| 17:58:13 | maciejjozefczyk | mriedem: https://review.openstack.org/532924 | |
| 17:58:58 | maciejjozefczyk | mriedem: IMO instead doing additional if statements in RT we should do it clean and set default values in config files as it should be done eariler (starting from newton) - if I understand the problem correctly | |
| 17:59:18 | maciejjozefczyk | its draft, without test changes | |
| 17:59:25 | maciejjozefczyk | tests* | |
| 17:59:51 | mriedem | maciejjozefczyk: i don't know about the implications of changing the defaults in the config file | |
| 17:59:56 | mriedem | that's all bauzas territory | |
| 18:00:11 | mriedem | i don't know the history behind why we had 0.0 in config and hard-coded values in code | |
| 18:01:55 | maciejjozefczyk | mriedem: It was (at least I can read it from comments) about backwards compatibility to Liberty, i think | |
| 18:02:12 | maciejjozefczyk | bauzas: hello :) could you please check https://review.openstack.org/532924 if the idea is ok for you? thanks! | |
| 18:05:43 | dansmith | mriedem: for the overcommit ratios? | |
| 18:05:55 | dansmith | allocation_ratio yeah | |
| 18:06:10 | dansmith | that is our sentinel to the scheduler to obey what the compute reports in the db instead of using config | |
| 18:06:14 | dansmith | when we moved that stuff | |
| 18:06:22 | dansmith | if 0.0, then 16.0 for cpu, etc | |
| 18:06:47 | dansmith | heh that patch just undoes it without historical context I think | |
| 18:06:52 | dansmith | bauzas: ^ | |
| 18:07:00 | dansmith | he has the full story in his head I bet | |
| 18:08:03 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Pass limit to /allocation_requests https://review.openstack.org/531517 | |
| 18:10:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244 | |
| 18:10:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle swapping to a multiattach volume https://review.openstack.org/532928 | |
| 18:10:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: [api] Allow multi-attach in compute api https://review.openstack.org/271047 | |
| 18:10:49 | mriedem | ildikov: gibi: johnthetubaguy: ^ swap volume change is in the middle | |
| 18:11:24 | ildikov | mriedem: gorgeous, thank you! | |
| 19:24:28 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | |
| 19:24:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Raise conflict exception on RP create 409 https://review.openstack.org/532563 | |
| 19:24:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | |
| 19:24:30 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | |
| 19:24:30 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | |
| 19:24:31 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 19:24:31 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | |
| 19:24:32 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 19:24:48 | openstackgerrit | Ed Leafe proposed openstack/nova master: Fix invalid UUIDs in test_compute.py https://review.openstack.org/531479 | |
| 19:28:23 | mriedem | edleafe: comments in the resize + alternates patch https://review.openstack.org/#/c/526436/ | |
| 19:28:33 | mriedem | as far as i can tell, there is nothing in there claiminig resources on the alternate hosts | |
| 19:30:10 | mriedem | dansmith: we need to get this fixed before rc1 https://review.openstack.org/#/q/topic:bug/1741307+project:openstack/nova | |
| 19:30:41 | dansmith | okay | |
| 19:38:31 | sean-k-mooney | mriedem: if we do a release of os-vif on monday are you ok with bumping the minium version for queens | |
| 19:40:23 | mriedem | i won't be around on monday | |
| 19:40:32 | mriedem | i was going to do the os-vif release today, and os-traits for that matter | |
| 19:40:35 | mriedem | now that you mention it | |
| 19:40:59 | mriedem | dansmith: melwitt: thanks for getting those; i'll add something to the ptg etherpad about migrating caching scheduler people to filter scheduler | |
| 19:41:30 | sean-k-mooney | mriedem: it would be nice to include https://review.openstack.org/#/q/status:open+project:openstack/os-vif+branch:master+topic:bug/1672812 and https://review.openstack.org/#/q/status:open+project:openstack/os-vif+branch:master+topic:bug/1666917 in queens | |
| 19:42:41 | mriedem | jesus | |
| 19:42:48 | mriedem | and what requires the minimum version bumps for those? | |
| 19:43:09 | mriedem | with jay on his deathbed i'm not sure how that's going to get done | |
| 19:43:28 | sean-k-mooney | mriedem: sorry i ment to say upperconstriats not minium version | |
| 19:43:37 | mriedem | u-c bumps are automatic | |
| 19:43:46 | mriedem | g-r bumps are when some consumer needs to rely on the changes | |
| 19:44:06 | mriedem | and presumably we're doing this work so something can use it | |
| 19:44:56 | sean-k-mooney | its used by os-vif itself it allow use to not use process exec to create port and insted use the openvswitch python libs | |
| 19:45:17 | mriedem | sean-k-mooney: ok, but that's not a super high priority is it? | |
| 19:45:22 | mriedem | at this point in queens i think that's a bit risky | |
| 19:45:29 | mriedem | and would be better deferred to an early release in rocky | |
| 19:46:33 | sean-k-mooney | its just a nice to have yes so it we branched today and did a release it would not break anything | |
| 19:47:07 | mriedem | has claudiub seen this? https://review.openstack.org/#/c/487405/ | |
| 19:47:16 | mriedem | i think the hyperv driver is using os-vif so how are they not blowing up? | |
| 19:47:49 | sean-k-mooney | the code that is pertected by the if would not fail if the if was removed so it does not explode | |
| 19:48:52 | sean-k-mooney | that said checking if an interface exists via a file exits check under sys on windows is not exactly well fucntional | |
| 19:49:20 | mriedem | the other 2 things in https://review.openstack.org/#/q/status:open+project:openstack/os-vif+branch:master+topic:bug/1672812 are not really something we need to get released | |
| 19:49:24 | mriedem | the testing related patches i mean | |
| 19:49:40 | mriedem | so i'm going to push an os-vif release today, and if something else gets in afteward before next thursday, bauzas or someone can propose a release | |
| 19:50:00 | sean-k-mooney | mriedem: ok that sound fine to me | |
| 19:50:18 | sean-k-mooney | as i said they are just nice to have not required | |
| 19:50:37 | mriedem | sean-k-mooney: kit me up when rocky rolls around and we can push for the native ovs ones | |
| 19:50:39 | mriedem | *hit me up | |
| 19:51:46 | sean-k-mooney | sure. we are keeping support for process exec and not enableing it by default anyway. we had push out makeing that change to S instead. | |
| 19:52:13 | sean-k-mooney | has a name been chosen for S yet by the way | |
| 20:07:05 | mriedem | squish | |
| 20:07:47 | mriedem | shizer? | |
| 20:10:22 | bauzas | mriedem: sylvain :p | |
| 20:10:31 | mriedem | there is nothing german about that | |
| 20:10:35 | mriedem | shizer it is | |
| 20:11:28 | bauzas | schnauze :p | |
| 20:12:09 | bauzas | mriedem: question for https://review.openstack.org/#/c/528128/7/novaclient/exceptions.py | |