| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-11 | |||
| 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 | |
| 20:12:12 | bauzas | mriedem: why ? | |
| 20:12:45 | mriedem | because if you do six.text_type(ex) you don't get the message from the exception | |
| 20:13:29 | bauzas | interesting to know | |
| 20:13:35 | mriedem | https://review.openstack.org/#/c/528128/7/novaclient/tests/unit/v2/test_servers.py@1640 | |
| 20:13:40 | mriedem | if you remove it, that stuff blows up | |
| 20:13:42 | dansmith | bauzas: if we name it that, people will be pronouncing it wrong for a long time | |
| 20:14:19 | bauzas | I know :p | |
| 20:22:57 | sean-k-mooney | like meltdown and specter im sure marketing will prevent any name being chossen that would not be family frendly | |
| 20:23:26 | sean-k-mooney | or well are inside/techy jokes | |
| 20:25:42 | bauzas | dansmith: got a sec ? | |
| 20:25:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 20:25:52 | dansmith | bauzas: finishing an email, but then yes | |
| 20:25:57 | bauzas | dansmith: cool | |
| 20:26:32 | bauzas | dansmith: that's just about the libvirt issue about regenerating a wrong XML because missing allocs | |
| 20:26:35 | bauzas | so take your time | |
| 20:27:07 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 20:27:24 | efried | Could one of y'all please re+W https://review.openstack.org/#/c/521686/ (auto rebase) | |
| 20:28:25 | bauzas | efried: . | |
| 20:28:37 | efried | bauzas Merci bien. | |
| 20:32:22 | mriedem | stvnoyes: on those tempest patches, when you push the changes, you need to use git review -R | |
| 20:32:29 | mriedem | you pulled my approved changes out of the gate | |
| 20:32:52 | mriedem | stvnoyes: i thought you were going to push a separate patch on top of https://review.openstack.org/#/c/266605/ ? not add to it. | |