| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-03 | |||
| 09:04:44 | openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log endpoint instead of stevedore https://review.openstack.org/461647 | |
| 09:19:35 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049 | |
| 09:27:29 | openstackgerrit | Alex Xu proposed openstack/nova master: Add policy description for os-quota-classes https://review.openstack.org/461996 | |
| 09:27:30 | openstackgerrit | Alex Xu proposed openstack/nova master: Add policy description for extensions https://review.openstack.org/461995 | |
| 09:37:02 | johnthetubaguy | gmann: alex_xu: just looking at this ips API, seems like we should have deprecated that? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/ips.py | |
| 09:41:55 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 09:42:14 | asettle | Good morning \o/ it's been a whole week since I've been here | |
| 09:42:31 | asettle | I've got a quick nova doc patch ifs omeone could take a wee lookie | |
| 09:42:34 | asettle | https://review.openstack.org/#/c/462009/2 | |
| 09:47:19 | johnthetubaguy | asettle: seems like a good add to me | |
| 09:47:44 | asettle | Thanks johnthetubaguy :) I just did my little edit, and +2 | |
| 09:48:41 | openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add doc for resource provider usages https://review.openstack.org/450105 | |
| 09:52:30 | alex_xu | johnthetubaguy: I remember we talk about in the before | |
| 09:53:34 | johnthetubaguy | alex_xu: ah, I forgot about that one completely | |
| 09:54:54 | alex_xu | johnthetubaguy: I can't remeber the detail also, try to search the log | |
| 09:59:13 | openstackgerrit | Gábor Antal proposed openstack/nova master: [WIP] Transform aggregate.update_metadata notification https://review.openstack.org/460625 | |
| 10:01:32 | alex_xu | johnthetubaguy: there is a roadmap for the interface API in the bottom of https://etherpad.openstack.org/p/nova-os-interfaces, it didn't have record about ips :( but I guess that isn't something we want | |
| 10:01:49 | johnthetubaguy | alex_xu: ah, oops | |
| 10:02:29 | alex_xu | johnthetubaguy: here is the talk, it is very long http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-04-11.log.html#t2017-04-11T14:02:10 if you are interesting | |
| 10:10:58 | openstackgerrit | Abhishek Sharma M proposed openstack/nova master: Catching OverQuota Exception https://review.openstack.org/460305 | |
| 10:47:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove dead code https://review.openstack.org/460662 | |
| 10:50:41 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Require OVS 2.8 and kernel 4.8 for offload https://review.openstack.org/460093 | |
| 10:55:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ensure sample policy help text correctly wrapped https://review.openstack.org/460097 | |
| 11:25:31 | asettle | johnthetubaguy: could I Get your email with something? How much do you know about Compute schedulers? | |
| 11:26:51 | sdague | melwitt: yeh, there was a reason this broke down on me before, and I really don't remember what it is | |
| 11:28:32 | sfinucan | sdague: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today? | |
| 11:40:12 | sdague | sfinucan: question on https://review.openstack.org/#/c/431037/10 (though +2 for now anyway) | |
| 12:01:41 | mordred | efried: the keystoneauth change has landed - so I think next step is getting you a ksa patch cut, yeah? | |
| 12:19:09 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/461446 | |
| 12:23:07 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/462124 | |
| 12:46:43 | alex_xu | nova api meeting is in 14 mins at #openstack-meeting-4 | |
| 12:53:10 | johnthetubaguy | asettle: I know bits, but there might be better folks to ask, whats up? | |
| 12:54:52 | asettle | johnthetubaguy: I'm staring at this bug: https://bugs.launchpad.net/openstack-manuals/+bug/1684261 | |
| 12:54:52 | openstack | Launchpad bug 1684261 in openstack-manuals "AggregateImagePropertiesIsolation example doesn't actually indicate how it works" [Low,Confirmed] - Assigned to Alexandra Settle (alexandra-settle) | |
| 12:54:54 | efried | mordred - lbragstad did that yesterday; 2.20.0 is out. | |
| 12:54:55 | asettle | And I admit, I'm also very confused. | |
| 12:55:44 | efried | It's working its way through requirements now. The bot proposed u-c bump, which merged; and I proposed g-r bump, which merged. Waiting for the bot to propose nova requirements.txt bump, but I think the nova change set will now verify, cause it uses u-c, right? | |
| 12:55:55 | lbragstad | efried mordred https://pypi.python.org/pypi/keystoneauth1/2.20.0 | |
| 12:58:37 | johnthetubaguy | asettle: good question, sounds like we need a better worked example. We have more docs in here: https://docs.openstack.org/developer/nova/filter_scheduler.html | |
| 13:00:29 | efried | mriedem Do you want me to include links to merged and/or proposed change sets in the use-service-catalog-for-endpoints edit? https://review.openstack.org/#/c/461481/ | |
| 13:00:43 | alex_xu | nova api meeting is running at #openstack-meeting-4 | |
| 13:01:26 | mriedem | efried: nope | |
| 13:01:53 | asettle | johnthetubaguy: yeah I was reading that this morning. I couldn't, admittedly, find too many good examples to transfer across. The only thing I could find for AggregateImagePropertiesIsolation is the literal explanation about halfway down the page. | |
| 13:04:44 | mriedem | dansmith: can you hit this simple grenade change for using wsgi on master https://review.openstack.org/#/c/460543/ ? | |
| 13:05:01 | mriedem | i.e. the devstack change to use uwsgi on master depends on that https://review.openstack.org/#/c/457715/ | |
| 13:06:33 | mriedem | EmilienM: we've said in the past that when we make changes to grenade that we should give the deployment teams a heads up, do you think this change warrants that https://review.openstack.org/#/c/460543/ ? | |
| 13:06:46 | mriedem | that's needed for using uwsgi for nova in devstack on master https://review.openstack.org/#/c/457715/ | |
| 13:08:40 | openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257 | |
| 13:13:31 | bauzas | sched.org, you <3 | |
| 13:24:49 | dansmith | mriedem: yep got it | |
| 13:26:15 | sfinucan | sdague: One more review here for you, if you would be so kind https://review.openstack.org/460097 | |
| 13:37:25 | openstackgerrit | John Garbutt proposed openstack/nova master: Add policy description for fping https://review.openstack.org/461763 | |
| 13:43:12 | dansmith | sdague: so your native call logging hack suggestion... | |
| 13:43:38 | dansmith | sdague: I dunno if you saw, but we're actually missing logs, and the two requests don't line up. we're not just seeing a big delay | |
| 13:43:43 | dansmith | do you still think it's worth trying that? | |
| 13:46:11 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Use six.text_type() when logging Instance object https://review.openstack.org/461912 | |
| 13:53:44 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513 | |
| 13:54:36 | sdague | dansmith: sure | |
| 13:54:47 | dansmith | okay | |
| 13:54:50 | sdague | dansmith: you want me to push it out there? | |
| 13:55:10 | sdague | honestly, the only thing it loses us is the search by request-id, which would be nice | |
| 13:55:12 | dansmith | sdague: I'll just slam it into my patch | |
| 13:55:16 | sdague | but not hugely a thing | |
| 13:55:25 | dansmith | I'm going to put something else to log in the delete path | |
| 13:55:32 | sdague | any idea if there was an exception in this path? | |
| 13:55:33 | dansmith | wait, what? | |
| 13:55:59 | sdague | wait what what? | |
| 13:57:12 | dansmith | commenting the use_journal thing means we don't log request_id? | |
| 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 | 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: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: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 | |