| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-03 | |||
| 13:57:14 | dansmith | or you mean it doesn't do the rich field thing? | |
| 13:57:35 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513 | |
| 13:58:00 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object https://review.openstack.org/461912 | |
| 13:59:20 | sdague | dansmith: it doesn't do the rich field thing | |
| 13:59:35 | dansmith | okay gotcha | |
| 13:59:52 | sdague | so with that on, the log path is through journal.send, which gives us all the extra metadata | |
| 14:00:00 | sdague | with that off, it's just going to spew to stdout | |
| 14:00:12 | dansmith | cool, just wanted to make sure it wasn't going to change more stuff | |
| 14:00:25 | sdague | systemd will send it all to the journal but without any of the fun bits | |
| 14:00:47 | sdague | dansmith: https://review.openstack.org/462163 | |
| 14:01:05 | sdague | I also tweaked the debug line, because processid is going to be in the systemd logs anyway | |
| 14:01:30 | mriedem | melwitt: heh, display_name='tempest-Ãñstáñcé-2027474072' is better than UnicodeDecodeError at least | |
| 14:01:36 | dansmith | sdague: oh i just commented it out of my patch | |
| 14:02:03 | sdague | mriedem: mriedem how did we get to that? | |
| 14:02:16 | mriedem | sdague: https://review.openstack.org/#/c/461912/ | |
| 14:02:40 | sdague | dansmith: sure, honestly, for right now I think it's fine to turn it off if we have concerns, so I'll put the general patch out there as well | |
| 14:02:40 | melwitt | mriedem: true. it's correct when I download the .txt.gz file though and open it with the less command. not sure what's up with that considering it let me copy-paste the name into the review comment from the log file I had downloaded | |
| 14:03:44 | sdague | melwitt: that really feels like something that should be happening at the logging levels below | |
| 14:04:03 | mriedem | sdague: you mean in the oslo.log handlers? | |
| 14:04:07 | jaypipes | sdague, dansmith, mriedem, bauzas: https://hangouts.google.com/call/b6rgkm2kzbh57pdgv6vey2kgxau | |
| 14:04:13 | sdague | mriedem: yeh | |
| 14:04:15 | melwitt | sdague: maybe, if we wanna convert everything to unicode before logging. which we probably do I guess | |
| 14:04:17 | mriedem | sdague: i think melwitt is just trying to figure out how to fix it | |
| 14:04:25 | sdague | mriedem: sure | |
| 14:05:29 | sdague | mriedem: you going to jump on hangout about onboarding room? | |
| 14:05:45 | mriedem | yes | |
| 14:05:57 | openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | |
| 14:06:45 | dansmith | sdague: okay | |
| 14:06:57 | openstackgerrit | John Garbutt proposed openstack/nova master: Add policy description for instance actions https://review.openstack.org/461757 | |
| 14:09:12 | dansmith | jaypipes: I came to mock, staying to mock | |
| 14:09:55 | abhishek | melwitt: Hi Melanie. I have added the changes which you had suggested. Please review https://review.openstack.org/#/c/460305/ | |
| 14:10:03 | openstackgerrit | John Garbutt proposed openstack/nova master: Add policy description for instance actions https://review.openstack.org/461757 | |
| 14:10:35 | jaypipes | dansmith: :) | |
| 14:10:35 | melwitt | abhishek: ack, will take a look | |
| 14:10:55 | abhishek | melwitt: thanks | |
| 14:10:56 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-attach-interfaces endpoint instead of stevedore https://review.openstack.org/462166 | |
| 14:19:25 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-server-groups endpoint instead of stevedore https://review.openstack.org/462169 | |
| 14:21:22 | mriedem | dansmith: https://etherpad.openstack.org/p/BOS-forum-nova-project-onboarding | |
| 14:22:56 | peter-hamilton | hi all, the bulk of the cert validation support in cursive is done: https://review.openstack.org/#/c/357202/ | |
| 14:23:23 | peter-hamilton | i'd like to get additional nova folks eyes on it before we go further (thanks again for the review jaypipes) | |
| 14:29:42 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: use plain routes list for os-virtual-interfaces https://review.openstack.org/462173 | |
| 14:34:05 | openstackgerrit | John Garbutt proposed openstack/nova master: Add policy description for os-quota-classes https://review.openstack.org/461996 | |
| 14:39:50 | efried | johnthetubaguy jaypipes mordred mriedem edmondsw FYI, the necessary ksa and requirements pieces for use-service-catalog-for-endpoints are in place, so the nova change https://review.openstack.org/#/c/458257/ is passing Jenkins and (I believe) ready for a solid review hammering. | |
| 14:39:59 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-consoles and os-console-auth-tokens endpoint instead of stevedore https://review.openstack.org/462175 | |
| 14:40:27 | jaypipes | efried: will try to review this afternoon. | |
| 14:40:34 | efried | jaypipes Thanks! | |
| 14:40:40 | edmondsw | efried roger that | |
| 14:43:54 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in extended_az and extend_ser_attrs https://review.openstack.org/461600 | |
| 14:46:48 | edmondsw | mriedem edleafe is there spec for the capabilities work that was discussed at the PTG? | |
| 14:46:53 | edmondsw | All I'm finding is https://blueprints.launchpad.net/nova/+spec/expose-host-capabilities which is languishing and doesn't talk about what was discussed at PTG | |
| 14:47:15 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for block_device_mapping instead of stevedore https://review.openstack.org/462180 | |
| 14:48:43 | jaypipes | edmondsw: are you referring to the API capabilities or are you referring to the resource provider traits stuff? | |
| 14:48:56 | edmondsw | jaypipes API | |
| 14:49:26 | edleafe | edmondsw: The api-wg spec for capabilities was abandoned: https://review.openstack.org/#/c/386555/ | |
| 14:50:47 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for user_data instead of stevedore https://review.openstack.org/462183 | |
| 14:51:01 | edmondsw | edleafe the abandon comment talks about "first fixing the immediate problems | |
| 14:51:06 | edmondsw | where is that happening? | |
| 14:51:40 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 14:52:19 | edleafe | edmondsw: as far as I know no one has picked that up | |
| 14:52:41 | edmondsw | k | |
| 14:53:01 | edleafe | edmondsw: so if you've got some spare cycles... :) | |
| 14:53:11 | edmondsw | edleafe wish I did... | |
| 14:53:24 | edleafe | heh, welcome to the club! | |
| 14:56:43 | melwitt | sdague: when I was debugging locally, I found this is where it blows up in the log formatting, it's trying to convert the message to unicode, but with the Instance object I found it only works if the encoding 'utf-8' is specified to six.text_type(), and that requires the instance arg be a string, not an Instance object type | |
| 14:56:58 | melwitt | https://github.com/openstack/oslo.log/blob/3ee14a1/oslo_log/formatters.py#L83-L93 | |
| 14:58:14 | mriedem | sdague: jaypipes: dansmith: bauzas: i assume none of us have a touchscreen laptop to project if we don't have a flip chart or whiteboard right? | |
| 14:58:34 | mriedem | that would be easier to see than a flip chart if we're expecting 40 people | |
| 14:58:54 | melwitt | so in my patch I'm doing repr(instance) followed by six.text_type(instance_repr, 'utf-8') if six.PY2 | |
| 14:59:26 | dansmith | mriedem: drawing on that kind of thing is pretty hard to do well, IMHO, but no I don't have one | |
| 14:59:55 | mriedem | dansmith: i thought they were supposed to be super awesome nowadays? | |
| 14:59:58 | mriedem | but yeah i don't have one either | |
| 15:01:22 | mriedem | efried: ah ok so this is what you needed https://github.com/openstack/requirements/commit/f7ebce161764236086957451d21d0fcae9fa86e9 | |
| 15:01:22 | melwitt | I have a msft surface and drawing pens, but I dunno how to project with it | |
| 15:01:40 | mriedem | melwitt: i assume projecting is same as always | |
| 15:01:49 | mriedem | usb adapter probably | |
| 15:02:16 | efried | mriedem Yes, that was one piece of the puzzle. | |
| 15:02:30 | melwitt | yeah, it has limited ports. it has usb though | |
| 15:04:56 | mriedem | dansmith: jaypipes: for the nova project update slides, we could talk over those tomorrow if you want. i think it would be pretty simple. i figured i could start, then jay covers any placement slides, dan covers any cells slides, and i can do the rest of the misc stuff | |
| 15:04:57 | sdague | mriedem: honestly, I also find drawing on those live quite hard. Flipboard would definitely be more useful | |
| 15:05:19 | mriedem | sdague: well someone should tell all of the marketers that for every time i watch an NFL game | |
| 15:05:24 | dansmith | mriedem: sounds good to me | |
| 15:05:25 | mriedem | because they all said it was super easy and awesome | |
| 15:11:32 | sdague | dansmith: did changing the use_journal option with your runs help? | |
| 15:12:04 | dansmith | sdague: they're not done yet | |
| 15:12:10 | sdague | dansmith: ok | |
| 15:20:15 | dansmith | mriedem: so, looking at the few other onboarding threads on the ML, I'm kinda curious... what are the other projects doing for this slot? | |
| 15:25:24 | mdbooth | dansmith: Perhaps a hazing? | |
| 15:26:43 | bauzas | dansmith: mriedem: FWIW, most of the Upstream Training attendees previously were trying to look at devstack and provide a change against a project | |
| 15:27:24 | bauzas | dansmith: mriedem: now that the Institute doesn't really provide devstack, I think they just want to have some technical discussions :) | |
| 15:29:01 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 15:31:41 | mriedem | dansmith: i've wondered the same, and i saw the cinder etherpad at one point | |
| 15:31:45 | mriedem | but can't find it now | |
| 15:32:29 | mordred | efried: nice! looking now | |
| 15:35:05 | mriedem | smcginnis: where is your guys' cinder onboarding etherpad? | |
| 15:35:12 | mriedem | kevinbenton: ^ do you guys have one? | |
| 15:35:36 | giuseppe1992 | Hello all!!! | |
| 15:38:12 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 15:38:53 | mordred | efried: oh - also - I'm making the patch to keystoneauth right now to deal with teh service type list - so hopefully we can make a follow up soon to your patch to consume that .. | |
| 15:39:31 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 15:41:45 | mordred | efried: patch looks mostly great on first read - excellent work -I've got a few nitpicky things I'll leave in just a bit | |