| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-04 | |||
| 19:38:27 | jaypipes | bauzas: cool. will review promptly when you push. | |
| 19:38:30 | bauzas | edleafe: could you pease upload your last point for https://review.openstack.org/#/c/460177/ for leaving me providing UTs ? | |
| 19:39:11 | dansmith | sdague: can you think of any reason why we might be missing logs from one of the api workers or something like that? | |
| 19:39:21 | melwitt | dansmith: I dunno, on the surface it looks like we have to be missing some logs | |
| 19:39:39 | dansmith | melwitt: yeah, seems like impossible that we're not missing logs | |
| 19:40:11 | mriedem | dansmith: haven't been following - did you pull the journal down and check there? | |
| 19:40:22 | dansmith | mriedem: was just about to | |
| 19:41:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | |
| 19:41:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 19:42:08 | dansmith | I only see it in nova-compute logs | |
| 19:43:11 | dansmith | in the journal I mean | |
| 19:44:44 | dansmith | I see it in scheduler too | |
| 19:44:52 | dansmith | but no api or anywhere else | |
| 19:44:56 | melwitt | dansmith: since you're depending on devstack change, did you rebase that to get the use_journal change. it looks like you did already | |
| 19:45:04 | dansmith | and I see it in neutron-server | |
| 19:45:10 | dansmith | melwitt: yep | |
| 19:45:26 | sdague | dansmith: what's going on? | |
| 19:45:36 | dansmith | sdague: we think we're missing nova-api logs | |
| 19:45:48 | dansmith | well, and nova-compute logs actually | |
| 19:45:55 | dansmith | we see instances that were booted and used by tempest, | |
| 19:46:08 | dansmith | but zero evidence of that uuid in nova api logs, very little in compute (not even the boot part) | |
| 19:46:12 | dansmith | in scheduler, and neutron | |
| 19:46:15 | dansmith | so it clearly exists | |
| 19:46:25 | dansmith | but it's like shockingly absent from the other services | |
| 19:47:09 | dansmith | sdague: that's the case in the text logs and in the raw journal I pulled down | |
| 19:47:17 | dansmith | (unified journal ftmfw by the way) | |
| 19:47:22 | sdague | right, they'll be the same | |
| 19:47:31 | sdague | ok, give me the example in question | |
| 19:47:34 | dansmith | well, I know they should be, but.. checking :) | |
| 19:47:42 | dansmith | http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/ | |
| 19:47:50 | dansmith | instance db767e42-2dbe-48d7-868d-b766d9879eb6 | |
| 19:48:44 | dansmith | and if you look for the NoVNCConsoleTestJSON tenant/user, | |
| 19:48:45 | dansmith | you get one hit in scheduler and nothing else | |
| 19:49:08 | sdague | so, it is entirely possible that the default rate limitters for journald are getting tripped | |
| 19:49:14 | sdague | I wonder if we can find out about that | |
| 19:49:25 | dansmith | oh interesting | |
| 19:49:32 | sdague | dansmith: you have the journals all down already? | |
| 19:49:40 | dansmith | sdague: just one journal, right? | |
| 19:49:41 | sdague | search for -k systemd | |
| 19:49:46 | dansmith | devstack.journal.gz? | |
| 19:50:02 | sdague | well, there are usual multiples of them because they roll over | |
| 19:50:17 | sdague | but, yeh, only looks like one in there | |
| 19:50:32 | dansmith | -k is dmesg, which you can't do on a file | |
| 19:50:45 | dansmith | unless you mean something else | |
| 19:50:46 | dansmith | -u? | |
| 19:51:23 | dansmith | journalctl | grep systemd yields not much of relevance | |
| 19:51:39 | sdague | -t | |
| 19:51:40 | sdague | sorry | |
| 19:51:53 | dansmith | same as the grep | |
| 19:52:05 | edleafe | bauzas: should be ready to upload soon. I've already created a unit test. | |
| 19:52:11 | sdague | ok, so default seems to be 1000 messages per 30s | |
| 19:52:17 | sdague | I am sure we are exceeding that | |
| 19:52:23 | dansmith | surely at startup | |
| 19:54:05 | sdague | let me put in a fix for that | |
| 19:54:13 | dansmith | cool thanks | |
| 19:54:26 | dansmith | link me and I'll depend on it and recheck | |
| 19:56:18 | melwitt | dansmith: so the websocketproxy goes direct to the instance? it doesn't need to know the compute host MQ right | |
| 19:56:44 | dansmith | it goes direct to the host | |
| 19:56:59 | dansmith | consoleauth validate_token does talk to the compute mq, but it does the looku | |
| 19:57:17 | melwitt | okay. we'll just have to see the logs then, nothing is jumping out at me | |
| 19:57:20 | dansmith | I think the token in memcache must have the ip/port that the proxy needs to connect to | |
| 19:57:21 | dansmith | yeah | |
| 19:57:28 | dansmith | hopefully something dumb | |
| 19:57:32 | dansmith | simple and dumb | |
| 19:57:34 | dansmith | like me. | |
| 19:57:44 | sdague | dansmith: I think this is it - https://review.openstack.org/462725 | |
| 19:57:46 | melwitt | lol :P | |
| 19:58:42 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | |
| 19:58:44 | dansmith | sdague: ack thanks | |
| 19:58:56 | dansmith | sdague: if that's really happening we probably want to merge that like.. yesterday :P | |
| 19:59:03 | mriedem | gibi: i rebased and commented in https://review.openstack.org/#/c/419730/ for adding keypairs to InstancePayload - summary is i don't think we should lazy load instance.keypairs for every instance action, for the same reason I didn't want to do that for instance.tags | |
| 20:00:26 | sdague | dansmith: yep agreed, I want to make sure it looks sane | |
| 20:00:37 | sdague | hey, it's all about learning new things :) | |
| 20:02:13 | dansmith | sdague: I'm not complaining :) | |
| 20:03:11 | sdague | also, what's up with the linuxbridge job? | |
| 20:03:17 | sdague | that seem things to be hating me today | |
| 20:04:24 | dansmith | sdague: that was an impressively incomprehensible exclamation | |
| 20:04:45 | openstackgerrit | Ed Leafe proposed openstack/nova master: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | |
| 20:04:55 | edleafe | bauzas: Here ya go ^^ | |
| 20:05:33 | dansmith | what is this nonsense? http://logs.openstack.org/79/462679/1/check/gate-nova-tox-functional-ubuntu-xenial/c760042/testr_results.html.gz | |
| 20:05:37 | bauzas | edleafe: okay, I'll update the next top changes | |
| 20:07:06 | melwitt | dansmith: socket_options, duh | |
| 20:07:26 | melwitt | :D | |
| 20:10:49 | cdent | dansmith: might be mismatched versions of urllib3 and wsgi-intercept | |
| 20:11:03 | cdent | I saw that a while back but thought it was better. | |
| 20:11:03 | dansmith | cdent: it's broken is what it is | |
| 20:11:20 | dansmith | looks rather fundamental, so wondering if it's worth a recheck | |
| 20:11:25 | cdent | yessir, but I think it got better. are your deps maybe stale? | |
| 20:11:35 | dansmith | that patch is new from today | |
| 20:11:39 | cdent | hmmm | |
| 20:12:00 | dansmith | I guess edleafe already rechecked it | |
| 20:12:15 | cdent | i'll mess with it a bit | |
| 20:12:19 | cdent | (locally) | |
| 20:12:24 | mriedem | can't provide new key_name on rebuild? | |
| 20:17:28 | melwitt | mriedem: I don't think you can specify anything other than the image. there have been requests to allow new userdata and such, but the answer has been "just boot a new instance" if you're looking at re-imaging it anyway | |
| 20:18:04 | mriedem | do we rebuild the config drive during rebulid? | |
| 20:18:32 | melwitt | I don't think so | |
| 20:18:50 | melwitt | but not sure, not a config drive expert | |
| 20:19:24 | mriedem | i'm looking, just this keypair notification patch has me thinking about weird stuf | |
| 20:19:29 | mriedem | https://review.openstack.org/#/c/419730/ | |