| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-01 | |||
| 16:49:57 | cdent | yeah, personally I’d prefer that the logs never have ansi codes in them | |
| 16:50:14 | superdan | I also use the solarized theme for all of this kind of ting, which is super low impact, especially compared to what you've got here | |
| 16:50:14 | cdent | but I think systemd’s journalctl doesn’t want me to feel that way | |
| 16:50:15 | fried_rice | cdent You can make that happen by leaving -a off of journalctl | |
| 16:50:30 | fried_rice | superdan What is this "solarized" theme? | |
| 16:50:44 | superdan | fried_rice: http://ethanschoonover.com/solarized | |
| 16:50:48 | cdent | fried_rice: right, but my point is that if I download a log from CI, it should _not_ be doctored | |
| 16:51:05 | cdent | if the CI is including ansi, then I end up with a mess | |
| 16:51:33 | fried_rice | cdent Hum, right. See, to me that's what I *do* want, which is why I started this whole thing. | |
| 16:51:50 | cdent | (see b) abve) | |
| 16:52:29 | fried_rice | I only know how to make code-included logs show up colorized. But there must be a way to just strip out the color codes. | |
| 16:52:30 | cdent | ansi escape sequences in log strings strikes me as some kind of weird punishment | |
| 16:52:42 | cdent | probably | |
| 16:52:44 | fried_rice | Oh, jeez, I feel they make it WAY more readable. | |
| 16:53:23 | superdan | they do sometimes, but other times they get in the way depending on how you're looking at it | |
| 16:53:34 | superdan | if I need to open it in an editor or something to apply a macro or something, | |
| 16:53:37 | superdan | then it's a total mess | |
| 16:53:58 | superdan | if I can have the option, then cool. if I have no option, I'd lean towards clean logs | |
| 17:03:52 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix disk size during live migration with disk over-commit https://review.openstack.org/524681 | |
| 17:06:30 | figleaf | mriedem: just got back from having my teeth scraped. Reading back, yeah, 409 seems like the best choice | |
| 17:09:01 | cdent | when you say it like that, it gives me the fingernails chalkboard sensation, or the styrofoam on cardboard things. thank you very little | |
| 17:09:30 | mriedem | i had that done on tuesday | |
| 17:09:33 | mriedem | so refreshing | |
| 17:25:15 | mriedem | oh shelve http://lists.openstack.org/pipermail/openstack-dev/2017-December/125124.html | |
| 17:25:19 | mriedem | you'll be the death of me | |
| 17:25:23 | mriedem | alaski: ^ maybe you have ideas | |
| 17:25:26 | mriedem | from 2013 | |
| 17:27:06 | cdent | shelve-tastic | |
| 17:31:41 | mriedem | fried_rolls: failure here looks real | |
| 17:31:42 | mriedem | https://review.openstack.org/#/c/524618/ | |
| 17:31:45 | mriedem | http://logs.openstack.org/18/524618/1/check/openstack-tox-functional/81f3dd8/testr_results.html.gz | |
| 17:31:51 | mriedem | cur_rp_gen = self._resource_providers[rp_uuid]['generation'] TypeError: 'NoneType' object has no attribute '__getitem__' | |
| 17:32:44 | mriedem | superdan: want to push this through stable? else i will https://review.openstack.org/#/c/509961 | |
| 17:33:29 | superdan | not really, but I will | |
| 17:33:49 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix incorrect known vcpuset when CPUPinningUnknown raised https://review.openstack.org/523644 | |
| 17:39:10 | mriedem | johnthetubaguy: bauzas: lyarwood: if you're still around, we should get these in https://review.openstack.org/#/q/I1a46ef1503be2febcd20f4594f44344d05525446 | |
| 17:39:24 | mriedem | fixes a regression introduced in a cve fix from a week or two ago | |
| 17:45:34 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix TypeError of _get_project_id when project_id is None https://review.openstack.org/524687 | |
| 17:46:20 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Vzstorage: synchronize volume connect https://review.openstack.org/524688 | |
| 17:50:50 | johnthetubaguy | mriedem: seems important, +2 | |
| 17:51:20 | mriedem | thanks | |
| 17:54:43 | alaski | mriedem: so the idea with unshelving was that it should look like nothing had happened. So you boot instance from image A, image_ref points at A. Resize/migration should work. You shelve/offload/unshelve and have an instance running again, with image_ref A. It's supposed to be like nothing happened so resize/migration should still work. What's unclear to me, since I've been out of the loop for a bit, | |
| 17:54:45 | alaski | is why resize/migration fails specifically after an unshelve. | |
| 17:57:04 | alaski | I think the argument against updating image_ref was that something like a rebuild should use the original image, not the snapshot. But outside of that I don't recall any reasons not to update the image_ref | |
| 17:58:16 | mriedem | ok so create with image A, shelve to create image B, unshelve with image B, rebuild with image A again | |
| 17:58:58 | alaski | yeah. image B is an artifact of shelving but from a user point of view it never really exists. | |
| 17:59:25 | mriedem | hmm, except it does, until we delete it | |
| 17:59:43 | alaski | shelve/unshelve ideally has the vm exactly as it was before shelving, no updates to the vm at all. | |
| 17:59:46 | mriedem | anywho, thanks, didn't think about the rebuild thing, but this all seems very slight of hand and confusing | |
| 18:00:01 | mriedem | well, | |
| 18:00:11 | mriedem | we still don't do that properly with volumes and ports completely i don't think | |
| 18:00:17 | alaski | yes, that whole feature is very much slight of hand and confusing | |
| 18:00:29 | mriedem | i will think on those over a 1000 calorie burrito | |
| 18:01:59 | alaski | "ideally" that's how shelve works, in practice it clearly does not. I won't claim that the ideal is achievable, so it may be better to be pragmatic and just document it well. | |
| 18:02:43 | alaski | maybe you'll come out of your food coma with an epiphany | |
| 18:08:39 | openstackgerrit | Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733 | |
| 18:22:38 | rabel | hi there. i need your thoughts on my last comment to https://review.openstack.org/#/c/402632/12 :) . it is about the datastore selection in vmware driver. | |
| 18:27:17 | cdent | rabel: I suspect rgerganov is going to be best placed to answer that and he won’t be back around until Monday, but I can try to make sure he sees it | |
| 18:29:31 | rabel | cdent: that would be great. thank you | |
| 18:29:43 | cdent | ✔ | |
| 19:15:28 | openstackgerrit | Merged openstack/nova stable/pike: Log consumer uuid when retrying claims in the scheduler https://review.openstack.org/509961 | |
| 19:16:55 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 19:16:55 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 19:23:21 | sdaniel_ | Hi, I need some help about nova services :( | |
| 19:29:23 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 19:29:59 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] re-use existing conf with auth token middleware https://review.openstack.org/523403 | |
| 19:30:39 | cdent | mriedem: I came up with a somewhat different approach for the test of that ^ because the middleware is very kwargy instead of a attrtibutey | |
| 19:34:09 | sdaniel_ | I had to reconfigurate the network type from provider to self-service | |
| 19:34:30 | sdaniel_ | and after that know of my nova services won't come up | |
| 19:34:44 | sdaniel_ | (after i restarted them manually) | |
| 19:35:28 | sdaniel_ | I re-created the whole nova DBs and re-installed the nova packages, but nothing helped | |
| 19:36:42 | sdaniel_ | I cant restart the services | |
| 19:36:50 | sdaniel_ | I got this error in syslog: TypeError: __init__() got an unexpected keyword argument 'retry_on_request' | |
| 19:37:17 | sdaniel_ | Only just nova cant restart, neutron can | |
| 19:50:22 | openstackgerrit | Merged openstack/nova stable/pike: Refined fix for validating image on rebuild https://review.openstack.org/523212 | |
| 19:56:02 | openstackgerrit | Merged openstack/python-novaclient master: Move zuulv3 jobs to project repo https://review.openstack.org/522099 | |
| 19:56:33 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 19:58:25 | openstackgerrit | Merged openstack/nova stable/ocata: Refined fix for validating image on rebuild https://review.openstack.org/523427 | |
| 19:58:30 | openstackgerrit | Merged openstack/nova stable/newton: Refined fix for validating image on rebuild https://review.openstack.org/523434 | |
| 20:25:58 | fried_rice | mriedem Yup, was just starting to look at that before "lunch". Should have it fixed here shortly. | |
| 20:34:06 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | |
| 20:34:06 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | |
| 20:34:07 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | |
| 20:35:07 | superdan | fried_rice: I hope "lunch" was something super well-deserving of quotes | |
| 20:35:44 | fried_rice | superdan Tapping out people bigger and stronger than me. The actual eating is happening now, at my desk. | |
| 20:36:29 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | |
| 20:36:55 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | |
| 20:39:22 | mriedem | i hope you're not eating the aforementioned tapped out people | |
| 20:40:28 | coffee_cat | lol :) | |
| 20:41:22 | cdent | “tapped out people and fried_rice”, it’s what’s for dinner | |
| 20:41:41 | fried_rice | You could say I ate them for lunch. | |
| 20:54:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Store shelved snapshot image ref with instance after unshelve https://review.openstack.org/524726 | |
| 20:58:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Check the return code when forcing TCG mode with libguestfs https://review.openstack.org/524727 | |
| 21:05:06 | mriedem | coffee_cat: already found a problem which i didn't think about before :) | |
| 21:05:09 | mriedem | but easy fix | |
| 21:05:44 | coffee_cat | mriedem: I would've been disappointed otherwise :) | |
| 21:05:56 | mriedem | i don't think we supported multi-cell when this was written | |
| 21:06:03 | mriedem | but now, we do | |
| 21:06:04 | mriedem | bigly | |
| 21:06:33 | mriedem | days since multi-cell multi-attach accidents: 0 | |