Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-01
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-" as part of the mask. This means if the admin specifies "hw:cpu_realtime_mask=1" they actually get all the cpus, which seems odd.
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
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

Earlier   Later