| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-28 | |||
| 19:44:09 | edleafe | Really helpful when comparing expected vs. actual in tests | |
| 19:44:23 | efried | edleafe Nice, thanks! | |
| 19:44:42 | openstackgerrit | Merged openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088 | |
| 19:49:31 | jaypipes | efried: actually... if you are looking for something to do... | |
| 19:49:55 | efried | jaypipes Tell me. I think I could just about handle a deep rebase right now. | |
| 19:50:01 | jaypipes | efried: if you wouldn't mind fixing those little issues you found on https://review.openstack.org/#/c/523192/, that would be super useful since I'm currently rebasing again the n-r-p series. | |
| 19:50:14 | efried | ight | |
| 19:50:20 | jaypipes | efried: since that's an important bug. | |
| 19:50:26 | efried | lemme propose this ksa release and I'll hit that... | |
| 19:50:27 | jaypipes | efried: and will make mriedem happy. | |
| 19:50:30 | jaypipes | ++ | |
| 19:50:33 | jaypipes | thanks much | |
| 19:54:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove deprecated TrustedFilter https://review.openstack.org/506864 | |
| 19:58:09 | openstackgerrit | Eric Fried proposed openstack/nova master: Use oslo_db Session in resource_provider.py https://review.openstack.org/523192 | |
| 19:58:27 | efried | jaypipes ^ -- though I doubt mriedem was witholding his +2 for those issues. | |
| 19:59:03 | mriedem | i'm withholding my +2 for other reasons | |
| 19:59:08 | efried | at least this way we get more test runs. | |
| 19:59:34 | efried | mriedem Care to comment? | |
| 20:00:02 | mriedem | i will leave you in suspense | |
| 20:02:43 | openstackgerrit | Merged openstack/nova master: Remove 'nova-manage quota refresh' command https://review.openstack.org/521829 | |
| 20:05:04 | efried | jaypipes https://bugs.launchpad.net/nova/+bug/1731668 got reassigned to me, and I can't seem to punt it back to you. Why is that? | |
| 20:05:04 | openstack | Launchpad bug 1731668 in OpenStack Compute (nova) "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [High,In progress] - Assigned to Eric Fried (efried) | |
| 20:05:36 | mriedem | efried: are you on the nova bug team? | |
| 20:05:52 | efried | mriedem How would I know? | |
| 20:05:55 | mriedem | https://launchpad.net/~nova-bugs | |
| 20:06:00 | efried | Sorry, I didn't mean that to sound rude | |
| 20:06:08 | efried | I literally meant, how would I know if that's the case? | |
| 20:06:30 | mriedem | should show your membership in there somewhere | |
| 20:06:34 | mriedem | or under your launchpad profile | |
| 20:06:46 | mriedem | https://launchpad.net/~efried/+participation | |
| 20:06:47 | mriedem | so no | |
| 20:06:52 | mriedem | so join the nova bug team | |
| 20:06:59 | efried | ight | |
| 20:07:42 | mriedem | there are no rules | |
| 20:08:31 | efried | There's a busted link at https://wiki.openstack.org/wiki/Nova/BugTriage | |
| 20:08:42 | mriedem | wikis are free to update | |
| 20:09:05 | mriedem | but you mean http://138.197.2.61/ yes? | |
| 20:09:19 | efried | yeah | |
| 20:09:26 | efried | I wouldn't know what to update it to. | |
| 20:09:40 | mriedem | let me see if i can find it | |
| 20:10:20 | mriedem | http://45.55.105.55:3000/dashboard/db/openstack-bugs | |
| 20:10:31 | mriedem | 45.55.105.55:8082/bugs-dashboard.html | |
| 20:10:37 | mriedem | http://45.55.105.55:8082/bugs-dashboard.html | |
| 20:11:01 | mriedem | might be dead at this point since markus_z was maintaining that | |
| 20:13:39 | efried | Fix Committed and Fix Released -- this is no longer accurate: https://wiki.openstack.org/wiki/Bugs ? | |
| 20:13:55 | efried | Cause we mark Fix Released as soon as it's in master; and we don't use Fix Committed at all - right? | |
| 20:14:17 | mriedem | once something is merged on master it's marked fix released | |
| 20:14:25 | mriedem | i think things get marked fix committed on stable branches | |
| 20:14:41 | mriedem | there is no auto-release of those for stable releases i don't think, although dhelmann used to run a script for that | |
| 20:14:43 | mriedem | idk anymore | |
| 20:14:59 | mriedem | nor is there a proposed/* branch anymore | |
| 20:15:23 | efried | Since it's OpenStack-wide, I'll refrain from editing that page; long as I have some understanding of what the real deal is. | |
| 20:16:00 | efried | Mebbe I'll mention it to dhellmann | |
| 20:16:14 | mriedem | bauzas and stephenfin gave a talk at the summit about bug triage | |
| 20:16:21 | mriedem | you can blast them with your questions | |
| 20:23:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix some incorrect option references for scheduler filters https://review.openstack.org/521645 | |
| 20:23:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate the IronicHostManager https://review.openstack.org/521648 | |
| 20:26:20 | openstackgerrit | Merged openstack/nova master: libvirt: remove extraneous retry assignment in cleanup method https://review.openstack.org/409199 | |
| 20:47:39 | mriedem | efried: jaypipes: maybe you guys can confirm the issue i raised in this osc-placement allocations patch https://review.openstack.org/#/c/457534/10 | |
| 20:47:58 | mriedem | but looking at how PUT /allocations/{consumer_id} works before 1.12, we can get some weird behavior | |
| 20:48:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | |
| 20:48:16 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 20:48:17 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 20:48:17 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 20:48:18 | openstackgerrit | Jay Pipes proposed openstack/nova master: ProviderTree.uuid_set() https://review.openstack.org/520243 | |
| 20:48:18 | openstackgerrit | Jay Pipes proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643 | |
| 20:48:19 | openstackgerrit | Jay Pipes proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree https://review.openstack.org/520663 | |
| 20:48:20 | openstackgerrit | Jay Pipes proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates https://review.openstack.org/521097 | |
| 20:48:20 | openstackgerrit | Jay Pipes proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | |
| 20:48:21 | openstackgerrit | Jay Pipes proposed openstack/nova master: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | |
| 20:48:21 | openstackgerrit | Jay Pipes proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | |
| 20:48:22 | openstackgerrit | Jay Pipes proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 20:48:44 | jaypipes | mriedem: gonna take a break for a bit, but will look at that later. | |
| 20:49:34 | mriedem | smoke'em if you got'em | |
| 20:49:34 | melwitt | mriedem: about the console TLS series, you might have you skim it to see what I mean. it's an effort to break the work into chunks. an already merged patch added a base class for a SecurityProxy. the first unmerged patch in the series lays some groundwork by adding a base class and a None auth scheme. the second patch adds a vencrypt auth scheme, and the final patch adds a RFBSecurityProxy which actually does the encryption | |
| 20:49:51 | efried | mriedem ack; but sounds like it oughtta be solved in the CLI - not a bug in the placement API? | |
| 20:50:15 | mriedem | efried: that's what i said, i think it's a limitation in the api until 1.12 and we have to handle it in the cli | |
| 20:50:42 | melwitt | mriedem: so I'm not seeing a better way to do the reno except at the end https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/websocket-proxy-to-host-security | |
| 20:50:42 | mriedem | melwitt: well, we could just hold off on approving the first unmerged patch with the new option until people are happy with the entire series | |
| 20:50:43 | efried | mriedem rgr. Though "limitation" might be a bit strong. | |
| 20:50:55 | melwitt | mriedem: okay, that works | |
| 21:00:30 | openstackgerrit | Merged openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 21:25:43 | melwitt | weird, yet another time when it says merge conflict and I rebase locally but there was no conflict ... | |
| 21:27:29 | melwitt | mriedem: should I just try the rebase button on https://review.openstack.org/#/c/498983 ? it says merge conflict but there's no conflict to resolve when I pull it down and rebase locally | |
| 21:28:35 | mriedem | sure | |
| 21:29:07 | melwitt | k, trying that now | |
| 21:29:34 | openstackgerrit | melanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume https://review.openstack.org/498983 | |
| 21:44:50 | penick | Hey folks, we're gearing up to replace our quota by flavor (and az) patches with something we could actually upstream. At the moment (based on hazy recollection of conversations at the summit) the direction i'm heading in is to do quota by custom resource class, wherein a resource class could be a combination of AZ and Flavor. Does that sound sane? | |
| 21:45:05 | mriedem | mlavalle: is this the neutron port binding extension that i'd want for the live migration thingy? http://paste.openstack.org/show/627643/ | |
| 21:45:21 | penick | Wanted to see if anyone had some input before we started working on the spec. | |
| 21:45:58 | melwitt | penick: I think what we discussed was leveraging the quota class facility we already have and adding the ability to tag flavors with a quota class | |
| 21:47:37 | penick | melwitt: ah, I also have to do quota by flavor and AZ as well. Can I make that work using quota classes too? | |
| 21:47:45 | mriedem | mlavalle: i'm guessing it's not, since it looks like that extension is just an indication of the binding:profile stuff on the port, i.e. https://developer.openstack.org/api-ref/network/v2/index.html#port-binding-extended-attributes | |
| 21:48:02 | mlavalle | mriedem: it's not | |
| 21:48:26 | mlavalle | that is the existing extension | |
| 21:49:00 | melwitt | penick: this gives a high level view of what quota classes are https://docs.openstack.org/nova/pike/user/quotas.html so it's already possible to set quota on a per class basis i.e. "openstack quota set --class myclass --instances 5" | |
| 21:49:52 | melwitt | it's just that currently there is no way to specify a quota class with any one request. rackspace used to do it via their own custom paste middleware, IIRC | |
| 21:51:52 | penick | oh ok, I didn't realize that it could be custom resource classes too, not just the typical resources like instances, ram, etc | |
| 21:52:08 | mriedem | not the same thing | |
| 21:52:16 | mriedem | resource classes are a placement concept | |