| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-01 | |||
| 15:34:02 | openstackgerrit | Matt Riedemann proposed openstack/nova master: db api: add get_service_by_uuid https://review.openstack.org/461428 | |
| 15:34:27 | lbragstad | johnthetubaguy I'm going to propose a series of follow-on patches with more detail | |
| 15:35:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add Service.get_by_uuid method https://review.openstack.org/461429 | |
| 15:35:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: db api: add service_get_by_uuid https://review.openstack.org/461428 | |
| 15:36:02 | cfriesen_ | on another note...who's the person to talk to around virt.hardware.vcpus_realtime_topology() currently? | |
| 15:37:23 | cfriesen_ | sdague: no python processes running after unstack.sh/clean.sh | |
| 15:38:21 | sdague | cfriesen_: hmm... I don't know, things are starting fine here. How old are your clones of all the projects? Maybe something is required from keystone that I didn't realized | |
| 15:41:50 | cfriesen_ | sdague: it's been a while...and the box itself was upgraded in place from 14.04 to 16.04. maybe I should blow it away and start from scratch. | |
| 15:41:51 | jaypipes | cdent: thanks very much! | |
| 15:42:12 | cdent | the font is... concerning | |
| 15:42:15 | cdent | but you've made the most of it | |
| 15:42:49 | jaypipes | cdent: as I mentioned, it's the Mirantis Boston template... | |
| 15:42:54 | jaypipes | nothing I can do about that. | |
| 15:42:56 | cdent | yeah, I know | |
| 15:43:08 | cdent | that's why I think it is great that you've "made the most of it' | |
| 15:43:21 | sdague | cfriesen_: the 403 typically means there is a credentials mismatch with the keystone server | |
| 15:43:34 | sdague | cfriesen_: if you want to keep debugging it, that's the thread to follow | |
| 15:43:40 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-floating-ips endpoint instead of stevedore https://review.openstack.org/461431 | |
| 15:44:21 | sdague | it must be part of the cleanup, because it works in clean environments | |
| 15:45:23 | cfriesen_ | jaypipes: you +2'd this https://review.openstack.org/#/c/356383/ last year....just wondering why we have the implied "0- |
|
| 15:46:48 | jaypipes | cfriesen_: how DARE you question my judgment! | |
| 15:46:58 | jaypipes | cfriesen_: heh, j/k :) will take a look. | |
| 15:47:38 | cfriesen_ | jaypipes: thanks...I also notice that this option doesn't seem to be documented in the manuals anywhere | |
| 15:47:48 | cfriesen_ | so I might have to fix that | |
| 15:47:59 | jaypipes | cfriesen_: that would be awesome if you did. ++ | |
| 15:48:36 | cfriesen_ | jaypipes: I've got another manual fix under review for what happens when flavor/image specify different things for cpu policy, cpu thread policy, mempage size | |
| 15:48:59 | jaypipes | cfriesen_: link me and I'll add it to my queue. | |
| 15:49:08 | cfriesen_ | jaypipes: cool, will do | |
| 15:49:18 | jaypipes | danke | |
| 15:54:03 | cdent | jaypipes: the new hotness on placement+uwsgi can be teased out of the results of https://review.openstack.org/#/c/456717/ but it's not immediately apparent without doing some digging so I'm not sure if it is worth it. May be easier/more simple for the sake of the presentation to say "it's just a simple wsgi app, run it with anything!" | |
| 16:02:43 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-floating-ip-pools endpoint instead of stevedore https://review.openstack.org/461436 | |
| 16:11:21 | dansmith | mriedem: hmm, looks like I never uploaded the patch set that fixed the setting mapped=1 thing | |
| 16:11:34 | dansmith | mriedem: and it's not in my local tree since I pulled from gerrit over this at some point | |
| 16:11:47 | dansmith | that method looked quite different in that version | |
| 16:13:30 | openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | |
| 16:13:38 | mtreinish | mriedem: ^^^ part 2 to close out the spec | |
| 16:13:45 | mriedem | mtreinish: excellent | |
| 16:15:16 | openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | |
| 16:15:36 | jaypipes | mriedem: looks like another unit test would be affected by the compute_nodes.uuid unique index: http://logs.openstack.org/87/454887/4/check/gate-nova-python27-ubuntu-xenial/e5b8b91/testr_results.html.gz | |
| 16:15:45 | jaypipes | mriedem: I blame dansmith ;) | |
| 16:15:50 | mriedem | gah | |
| 16:15:51 | mriedem | ok | |
| 16:24:00 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-floating-ips-bulk endpoint instead of stevedore https://review.openstack.org/461439 | |
| 16:28:55 | jaypipes | cfriesen_: reviewed. good stuff. | |
| 17:23:32 | cfriesen_ | but anyone that actually cares should probably specify which they want, so I'm not too worried about it. | |
| 17:23:41 | jaypipes | cdent, edleafe: maybe it would help if I just push the remainder of the patch series so you can see the end result. | |
| 17:23:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Make discover_hosts only query for unmapped ComputeNode records https://review.openstack.org/427903 | |
| 17:24:19 | dansmith | mriedem: I was thinking, we could make the scheduler call the host discovery on SIGHUP, which may be more straightforward (i.e. you have to SIGHUP the scheduler to notice new hosts | |
| 17:24:25 | edleafe | jaypipes: I sure hope so :) | |
| 17:24:35 | jaypipes | edleafe: thanks for the encouragement. | |
| 17:24:42 | cdent | jaypipes: could be, but it is also very useful for the rest of us who are not in your brain to actually talk about it and figure out the larger context of what's going on. For example my latest comment is basically about what "typical" is. | |
| 17:24:57 | cdent | think of this as an educational opportunity™ | |
| 17:25:08 | cdent | a union of souls over one page | |
| 17:25:55 | jaypipes | cdent, edleafe: are you essentially asking me to add a "agg_ids" argument to _get_providers_sharing_capacity() that would winnow the resulting provider ID list to only the aggregates that have been previously selected in ResourceProviderList.get_all_by_filters()? | |
| 17:25:59 | openstackgerrit | Chris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly https://review.openstack.org/461456 | |
| 17:26:11 | cfriesen_ | jaypipes: what would you say to something like this? | |
| 17:26:59 | edleafe | jaypipes: no, not if you are then going to use that in a future join expression. | |
| 17:27:20 | edleafe | jaypipes: there wasn't any indication in the code that this would be happening | |
| 17:27:24 | jaypipes | edleafe: precisely my plan, yes. | |
| 17:28:05 | jaypipes | edleafe: what do you mean there wasn't any indication in the code? you mean the code comments didn't say how I'd be using the result of _get_providers_sharing_capacity()? | |
| 17:28:07 | cdent | jaypipes: I'm not asking for a specific action, I'm simply trying to understand the intent and more importantly the context within which the stuff is going to operate. without that I can't review very effectively | |
| 17:28:25 | edleafe | jaypipes: your since-corrected comments also made it sound like it was going to be "in the same aggregate" | |
| 17:28:27 | jaypipes | cdent: k, then lemme push more patches that show that. | |
| 17:28:41 | jaypipes | edleafe: k, lemme push some patches. | |
| 17:29:44 | cdent | my new strategy for everything is to be 5yo and ask "why". Soon everyone will hate me. | |
| 17:30:02 | edleafe | cdent: "soon"?? | |
| 17:30:13 | cfriesen_ | cdent: your punishment is to watch all episodes of "Paw Patrol" | |
| 17:30:14 | cdent | edleafe: just because you already hate me | |
| 17:30:39 | cdent | cfriesen_: my mom doesn't let me watch older kids tv | |
| 17:33:25 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-agents endpoint instead of stevedore https://review.openstack.org/461459 | |
| 17:39:16 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: get providers sharing capacity https://review.openstack.org/460798 | |
| 17:40:04 | jaypipes | cdent: ^^ removed the aggregate stuff from the test case.. | |
| 17:40:15 | jaypipes | working on follow up patches now. gimme about half hour. | |
| 17:41:17 | cdent | thanks jaypipes, it wasn't a huge sin that stuff being in there, it was just that it was misleading with regard to the thing being tested (without more insight into what's going on). It really did make me think that I was missing something in your code. | |
| 17:41:25 | cdent | will watch out for the new stuff | |
| 17:42:57 | dansmith | mriedem: what do you want me to put as the message for the NovaException in that coding error case? I don't want to reveal too much to the user | |
| 17:43:12 | dansmith | just "Internal Error" ? | |
| 17:43:32 | mriedem | umm | |
| 17:43:52 | mriedem | default is msg_fmt = _("An unknown exception occurred.") | |
| 17:43:59 | mriedem | maybe that's good enough | |
| 17:44:04 | dansmith | ack | |
| 17:45:28 | jaypipes | cdent: yep, agree with ya. | |
| 17:46:30 | openstackgerrit | Dan Smith proposed openstack/nova master: re-Allow adding computes with no ComputeNodes to aggregates https://review.openstack.org/460616 | |
| 17:47:35 | openstackgerrit | Chris Dent proposed openstack/nova master: Add pbr-installed wsgi application for metadata api https://review.openstack.org/459413 | |
| 17:47:35 | openstackgerrit | Chris Dent proposed openstack/nova master: devref and reno for nova-{api,metadata}-wsgi scripts https://review.openstack.org/461289 | |
| 17:47:40 | cdent | I think I got those nits mriedem ^ | |
| 17:47:49 | cdent | but could very well have add more | |
| 17:48:53 | openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | |
| 18:05:00 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-assisted-volume-snapshots endpoint https://review.openstack.org/461470 | |
| 18:05:34 | mriedem | cdent: yup, one tiny issue | |
| 18:05:43 | cdent | yay! | |
| 18:06:12 | cdent | nice catch | |
| 18:08:56 | openstackgerrit | Chris Dent proposed openstack/nova master: devref and reno for nova-{api,metadata}-wsgi scripts https://review.openstack.org/461289 | |
| 18:16:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473 | |
| 18:24:17 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-availability-zone endpoint instead of stevedore https://review.openstack.org/461475 | |
| 18:29:05 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | |
| 18:29:06 | openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | |
| 18:29:06 | openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | |
| 18:29:07 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | |
| 18:29:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473 | |