Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-01
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: devref and reno for nova-{api,metadata}-wsgi scripts https://review.openstack.org/461289
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: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: Move to proper target_cell calling convention https://review.openstack.org/458634
18:29:06 openstackgerrit Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635
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
18:36:44 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
18:38:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fixed-ips endpoint instead of stevedore https://review.openstack.org/461479
18:53:21 jaypipes I think dansmith is going to add some vomit to his snot when he sees the SQL in my next patch.
18:56:33 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480
18:59:37 cdent vomitty snot! \o/
19:01:37 edleafe that reminds me... lunchtime!
19:04:54 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-availability-zone endpoint instead of stevedore https://review.openstack.org/461475
19:05:14 openstackgerrit Eric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa https://review.openstack.org/461481
19:05:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fixed-ips endpoint instead of stevedore https://review.openstack.org/461479
19:05:26 efried mriedem mordred jaypipes ^^
19:05:39 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480
19:19:19 dtroyer I've got a question on the interaction between aggregates and cells… I have a Mitaka DevStack configuration that has n-cell enabled; the hosts from "nova host-list" look like cell!region@vxd-mitaka, which aggregate-add-host does not like. Is this not a valid thing? Or am I missing something here?
19:19:43 jaypipes efried: gonna be while...
19:20:03 mriedem dtroyer: cells v1 doesn't support aggregates
19:20:05 efried jaypipes Don't think there's a hurry. mriedem asked me to update based on [glance]api_servers decision.
19:20:25 dtroyer mriedem: cool, that's what I needed to know. is there a way from the API to know if I have cells v1 or v2?
19:21:09 mriedem dtroyer: well, one thing i guess is GET /os-cells would return something for cells v1 if enabled https://developer.openstack.org/api-ref/compute/#cells-os-cells-capacities
19:21:17 mriedem that's admin-only by default
19:21:19 mriedem the api i mean
19:21:36 mriedem but cells v1 runs the separate n-cell service, that's how devstack checks it
19:21:46 mriedem dtroyer: cells v2 is baked in, not a separate service or API
19:21:55 dtroyer mriedem: thanks. I'm trying to do graceful failures in OSC in the aggregate commands so admin-only is OK
19:21:58 mriedem dtroyer: so starting in ocata, you're running cells v2 by design
19:22:00 dtroyer ah, gotcha
19:22:10 mriedem even if you're running cells v1, you have cells v2 :)
19:22:14 mriedem consider your mind blown
19:22:45 mriedem dtroyer: the other thing would probably be the /os-services API
19:23:08 mriedem https://github.com/openstack/nova/blob/master/nova/cmd/cells.py#L41
19:23:08 dtroyer mriedem: one more question… I have a note that nova-net required cells v1, that doesn't seem right to me…
19:23:23 mriedem if you're running cells, there should be a nova-cells service binary in the GET /os-services response
19:23:35 mriedem i think anyway
19:23:43 mriedem dtroyer: other way around
19:23:47 mriedem well,
19:24:01 mriedem cells v1 support in-tree requires nova-net,
19:24:17 dtroyer ok, that explains a couple of things. thanks
19:24:29 mriedem dtroyer: in ocata we made nova-network only start under a cells v1 config
19:24:32 mriedem that's probably the confusion
19:24:48 dtroyer I bet so
19:25:07 mriedem we basically just continued to disable nova-net in ocata,
19:25:15 mriedem and switched all jobs to use neutron except for the cells v1 job
19:31:13 dansmith jaypipes: I don't hate your sql, I admire it.
19:31:18 dansmith jaypipes: I just note that I shan't be debugging it if something goes wrong
19:31:34 cdent tshirt
19:39:21 openstackgerrit Eric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa https://review.openstack.org/461481
19:53:09 openstackgerrit Ed Leafe proposed openstack/nova master: Remove the can_host column https://review.openstack.org/461494
19:56:21 melwitt jaypipes: fyi I pushed an update to the quotas series attempting to address your comments https://review.openstack.org/#/c/446239
19:56:29 jaypipes melwitt: yup, reviewing it now :)
19:56:37 jaypipes while waiting for functional tests to complete..
19:57:01 melwitt jaypipes: cool, thanks :)
20:02:24 ildikov jaypipes: around for a meeting on #openstack-meeting-3? :)
20:03:03 jaypipes ildikov: I'm sorry, Ildiko, I'm right in the middle of debugging a nasty little feature and really need to focus on it.
20:03:06 jaypipes sorry :(
20:03:16 jaypipes ildikov: will read back from channel later.
20:03:29 ildikov jaypipes: ok, let's catch up later then
20:04:52 jaypipes thx ildikov
21:10:04 openstackgerrit Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257
21:16:54 mriedem oh boy, so my daughter got home and there is a note from the teacher in her backpack about inappropriate behavior,
21:17:09 mriedem e.g. a worksheet of sight words to fille out, every entry has the word "butt" in it
21:17:34 mriedem she at least spelled it correctly...
21:23:48 cdent \o/
21:25:39 dansmith mriedem: sight words?
21:26:08 mriedem dansmith: yeah, like simple single-syllable words that they should recognize immediately
21:26:12 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513
21:26:14 mriedem they, here, were, are, them
21:26:22 mriedem butt
21:26:25 dansmith mriedem: and butt is inappropriate?

Earlier   Later