| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-27 | |||
| 14:11:21 | johnthetubaguy | jroll: that was my assumption | |
| 14:12:06 | figleaf | jroll: there will probably be an "if _is_ironic():" check added to that code to look for the resource class | |
| 14:12:12 | jroll | johnthetubaguy: I also don't see that n:1 service:resource_class in the code | |
| 14:12:25 | johnthetubaguy | jroll: the resource class request's presence just always dominates any other request, that seems fine | |
| 14:13:57 | cdent | Is it normal for us to be as confused about stuff as we've been? | |
| 14:14:04 | johnthetubaguy | figleaf: I guess we can make it nested, once we have nested? | |
| 14:14:43 | johnthetubaguy | cdent: thats probably yes, I prefer the "can we do better at this" question, I hope thats a yes too | |
| 14:15:00 | cdent | just checking | |
| 14:15:04 | johnthetubaguy | jroll: I am looking here: https://review.openstack.org/#/c/404472/23/nova/compute/resource_tracker.py@525 | |
| 14:15:11 | johnthetubaguy | cdent: its good to check these things | |
| 14:15:18 | cdent | I'm trying to decide whether it is reasonable to be chill or panicky | |
| 14:15:57 | jroll | johnthetubaguy: yeah, so that's for a single ironic node, and that code will run once per ironic node | |
| 14:16:29 | openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Update pci device fields from deleted object https://review.openstack.org/426243 | |
| 14:16:46 | figleaf | cdent: it's more the overlapping terminology that confuses me | |
| 14:16:56 | johnthetubaguy | jroll: doh, I see the loop now | |
| 14:17:05 | figleaf | "compute" can mean multiple things | |
| 14:18:00 | johnthetubaguy | figleaf: sorry, I see what you were trying to tell me now | |
| 14:18:08 | jroll | figleaf: I've learned to always say compute *service* (nova-compute process) or compute *node* (the hypervisor/hardware), it seems to help | |
| 14:18:33 | cdent | jroll: if you said compute service to me, I'd assume you meant all of nova | |
| 14:19:00 | cdent | language is hard | |
| 14:19:00 | jroll | cdent: ahem, "nova-compute service" | |
| 14:19:02 | jroll | heh | |
| 14:21:39 | openstackgerrit | Sam Betts proposed openstack/nova: Change order of _cleanup_deploy and _unprovision in Ironic virt https://review.openstack.org/422678 | |
| 14:21:40 | jroll | through an IV | |
| 14:23:48 | cdent | jroll++ | |
| 14:26:30 | artom | Intra-vineous :D | |
| 14:26:38 | figleaf | johnthetubaguy: in that case, let's make some wine! | |
| 14:28:24 | openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Update pci device fields from deleted object https://review.openstack.org/426243 | |
| 14:36:49 | johnthetubaguy | figleaf: best idea I heard all day | |
| 14:39:19 | BlackDex | hello again. I'm having trouble with nova-consoleauth | |
| 14:39:31 | BlackDex | i have an ha setup, with hacluster and a vip | |
| 14:39:37 | BlackDex | 3 systems | |
| 14:40:01 | BlackDex | if i use horizon to view the console i have a 1 in 3 times luck to get the console to view | |
| 14:40:42 | BlackDex | i have configured memcached, but nothing is getting into memcached, i can connect to memcached from the nova-controle nodes | |
| 14:47:20 | superdan | that scheduler change failed on the old side due to volumes and neutron | |
| 14:47:21 | cdent | johnthetubaguy, dansmith : looks like the latest multinode grenade failed on timeouts this time | |
| 14:47:25 | cdent | jinx | |
| 14:47:27 | superdan | heh | |
| 14:48:16 | johnthetubaguy | is that a real something is broken timeout, or a you hit an unlucky node timeout? | |
| 14:49:18 | johnthetubaguy | cdent: did we get the correct min_version back now? | |
| 14:49:28 | cdent | no logs for the new side | |
| 14:49:53 | superdan | johnthetubaguy: we didn't get far enough to look | |
| 14:50:16 | johnthetubaguy | I was worried you would say that | |
| 14:50:36 | johnthetubaguy | heh | |
| 14:50:43 | mriedem | superdan: congrats https://review.openstack.org/319379 | |
| 14:51:17 | mriedem | now we just cleanup the release notes for cells v2 and maybe upgrade docs and we're good for ocata i think | |
| 14:51:35 | jroll | what's the best way to talk to the placement service in devstack? just grab a keystone token and use curl? | |
| 14:51:50 | mriedem | jroll: yeah, althouh mlavalle has an osc plugin somewhere | |
| 14:52:17 | jroll | mriedem: cool, thanks | |
| 14:52:19 | cdent | jroll: a) probably yes b) i'm not sure if mlavalle's thing is "done" c) I use gabbi yaml files, run throuh gabbi-run | |
| 14:52:37 | johnthetubaguy | figleaf: cdent: I am struggling to understand this line: https://review.openstack.org/#/c/404472/23/nova/compute/resource_tracker.py@548 | |
| 14:52:41 | openstackgerrit | Artom Lifshitz proposed openstack/nova: Pass APIVersionRequest to extensions https://review.openstack.org/425876 | |
| 14:52:41 | openstackgerrit | Artom Lifshitz proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759 | |
| 14:53:05 | johnthetubaguy | figleaf: cdent: I feel I am missing something big and obvious | |
| 14:53:10 | jroll | cdent: thanks | |
| 14:53:11 | cdent | jroll: in case you're curious: https://gist.github.com/cdent/a9590764fbc7402d450fa36df14f35e0 | |
| 14:53:42 | jroll | cdent: I need to poll for resources in our devstack plugin, curl seems like the right thing for now | |
| 14:53:52 | cdent | jroll: for that, yeah | |
| 14:54:18 | cdent | (although, just because we're almost on the topic, gabbi can poll) | |
| 14:54:48 | cdent | johnthetubaguy: the cleanup or that the exception happens? | |
| 14:55:14 | johnthetubaguy | cdent: I am wondering if we need cleanup in other cases, like when an ironic node is there, but there is no allocation | |
| 14:55:26 | johnthetubaguy | cdent: maybe the API just does that for us already? | |
| 14:55:56 | johnthetubaguy | ah, I think it does | |
| 14:56:04 | cdent | I don't think I'm sufficiently contextualized to really follow what you're saying? | |
| 14:56:43 | openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Fix tag attribute disappearing https://review.openstack.org/426030 | |
| 14:57:08 | johnthetubaguy | cdent: sorry, I was worried about us not always deleting the old style ironic inventories, but I didn't realise the API would do that for us when we put the inventories | |
| 14:57:39 | mriedem | superdan: cdent: jaypipes: bauzas: johnthetubaguy: melwitt: fyi i'm starting to make a list of things to do before rc1 https://etherpad.openstack.org/p/nova-ocata-rc1-todos | |
| 14:57:44 | johnthetubaguy | its just when we have allocations we need to do better | |
| 14:57:47 | johnthetubaguy | mriedem: perfect | |
| 14:57:51 | mriedem | jroll: https://github.com/miguellavalle/python-placementclient | |
| 14:58:39 | jroll | mriedem: cool, I'll wait til that's more "official" to depend on it in devstack, I think | |
| 14:59:43 | cdent | mriedem: do we want to be concerned with the traffic breadth from the resource tracker that I wrote about in: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110953.html ? There are two bug fixes in progress related to that. | |
| 15:01:05 | mriedem | cdent: maybe? throw it into the etherpad, i haven't read that thread in detail yet | |
| 15:01:08 | mriedem | i know we're super chatty | |
| 15:01:19 | openstackgerrit | Jordan Pittier proposed openstack/nova: Fix unspecified bahavior on GET /servers/detail?tenant_id=X as admin https://review.openstack.org/426259 | |
| 15:01:19 | cdent | k, will do | |
| 15:01:50 | mriedem | the RT is chatty i mean when using the report client | |
| 15:02:03 | mriedem | i think i pointed something out about that in the resource classes series that's still up | |
| 15:02:17 | mriedem | but now i can't remember what it was, i think something about updating inventory twice when we didn't need to | |
| 15:03:06 | cdent | mriedem: yeah | |
| 15:03:22 | mriedem | but thanks for doing an audit, it's easy to not care about performance when you're just trying to get something functional | |
| 15:03:25 | cdent | I think in order to maintain our self-healing we still need to report more than we want | |
| 15:03:34 | cdent | s/want/could/ | |
| 15:03:49 | cdent | but at the moment it is very redundant, and some if it just because of old bugs | |
| 15:04:11 | cdent | mriedem: was worried about what would happen when this suddenly landed on a big deployment | |
| 15:04:17 | superdan | cdent: so one thing to consider, while we see how this goes, | |
| 15:04:33 | superdan | cdent: is are we likely to make more trouble by doing what we're doing than we have now.. | |
| 15:04:35 | superdan | because currently, | |
| 15:04:54 | superdan | we do this, but over rpc and into the same database as everything else, | |
| 15:05:17 | superdan | with placement, it's over http, probably a little more compact, and to a potentially separate database | |
| 15:05:29 | superdan | and probably more efficient horizontally-scaled under apache | |
| 15:05:46 | superdan | so we could assume it's at least a wash for ocata and see how it goes with some early testers | |
| 15:05:51 | cdent | certainly the horizontal scale capabilities are more gooderer | |
| 15:05:57 | mriedem | and we're still doing the rpc to single db thing | |
| 15:06:07 | superdan | you didn't identify anything that is worse than today right? | |
| 15:06:10 | mriedem | will be nice in the future when we can drop that load from rpc traffic | |
| 15:06:15 | cdent | so yeah, we are continuing the existing, and adding more on top | |
| 15:06:16 | superdan | yeah | |
| 15:06:48 | superdan | yeah, I guess I forgot we're doing both, but, theoretically to different places | |
| 15:06:49 | superdan | but yeah | |