| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-17 | |||
| 16:17:03 | openstackgerrit | Merged openstack/nova stable/ocata: Functional test for regression bug #1713783 https://review.openstack.org/505160 | |
| 16:17:11 | openstackgerrit | Merged openstack/nova master: Deduplicate notification samples Rocky - 1 https://review.openstack.org/560296 | |
| 16:32:04 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Pass the correct image to build_request_spec in conductor.rebuild_instance https://review.openstack.org/516404 | |
| 16:32:05 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Set error state after failed evacuation https://review.openstack.org/518733 | |
| 16:42:48 | openstackgerrit | Jay Pipes proposed openstack/nova master: support multiple member_of qparams https://review.openstack.org/561315 | |
| 16:46:35 | openstackgerrit | Chris Dent proposed openstack/nova master: Parse forbidden in extra_specs https://review.openstack.org/561677 | |
| 16:52:34 | mriedem | melwitt: if we can get this novaclient change in https://review.openstack.org/#/c/561520/ we can do a novaclient release to align with rocky-1 | |
| 16:52:55 | mriedem | which we should probably do either way; i think i waited too long in queens to release novaclient and it caused some havoc with OSC and horizon late in queens | |
| 16:53:21 | melwitt | mriedem: okay, yeah I did want to release novaclient for r-1. I'll take a look | |
| 16:58:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate notification samples Rocky - 4 https://review.openstack.org/560731 | |
| 16:58:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate notification samples Rocky - 5 https://review.openstack.org/560734 | |
| 16:58:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate notification samples Rocky - 6 https://review.openstack.org/560737 | |
| 16:58:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_id to instance action notifications https://review.openstack.org/553288 | |
| 17:09:32 | melwitt | mriedem: why do you think disabling the UCA could cause the ceph job problem again? problems occur when installs by pip get mixed with installs by apt-get. devstack will only pip install over a apt-get installed package if upper-constraints aren't being met | |
| 17:12:37 | melwitt | with the Pike UCA, the version of python-paste installed by apt-get did not meet upper-constraints so devstack pip installed a newer version over it. so the matching python-pastedeploy package installed by apt-get had to be manually pip force installed in order to have paste and pastedeploy both installed by pip so pastedeploy could refer to modules in paste | |
| 17:13:10 | mriedem | melwitt: so that's not a problem if you're just using stock ubuntu 16.04? | |
| 17:13:51 | melwitt | it depends on what versions of python-paste and python-pastedeploy are in the repo for 16.04 and if only *one* of them does not meet upper-constraints | |
| 17:16:08 | melwitt | problems occur when only one of them does not meet upper-constraints because then devstack will upgrade only one with pip and then they can't refer to each other anymore. really, the hack probably should have been to pip force install both of the packages to guarantee they will always be able to find each other | |
| 17:16:58 | jaypipes | mriedem, melwitt, dansmith: any of you seen "nova.exception.InvalidDiskInfo: Disk info file is invalid: qemu-img failed to execute on /fake/path : Unexpected error while running command." pop up in unit test failures recently? | |
| 17:17:33 | melwitt | jaypipes: I have not | |
| 17:17:50 | mriedem | unit tests? | |
| 17:17:55 | mriedem | i probably know the one that is causing it | |
| 17:18:23 | jaypipes | mriedem: ya. if I run tox -epy27, I get a single failure, always in nova.tests.unit.virt.test_images.QemuTestCase.test_qemu_info_with_errors | |
| 17:18:29 | mriedem | https://review.openstack.org/#/c/553067/ | |
| 17:19:08 | openstackgerrit | Chris Dent proposed openstack/nova master: Parse forbidden in extra_specs https://review.openstack.org/561677 | |
| 17:20:30 | openstackgerrit | Jay Pipes proposed openstack/nova master: support multiple member_of qparams https://review.openstack.org/561315 | |
| 17:20:48 | jaypipes | mriedem, melwitt, dansmith: only seems to happen for me locally. so not really concerned about it... | |
| 17:21:11 | mriedem | probably a bad global mock on os.exists | |
| 17:22:09 | melwitt | do those tests require that you have qemu-img installed maybe? | |
| 17:29:18 | jaypipes | melwitt: not sure... I can check | |
| 17:35:02 | dansmith | sean-k-mooney: if you're still around and can re-apply your +1 to this I think that'd be good: https://review.openstack.org/#/c/484997/20 | |
| 17:37:53 | sean-k-mooney | dansmith: done. it looks like v20 is just some pep8 fixs/whitespace clean up for v19 so ya it looks fine to me | |
| 17:38:25 | dansmith | sean-k-mooney: yes, just want your +1 on there :) | |
| 17:38:31 | dansmith | s/just/I just/ | |
| 17:44:52 | mriedem | melwitt: unit tests should never require that you have native packages installed | |
| 17:45:12 | melwitt | I know, but I've run into issues with that in the past | |
| 17:52:54 | melwitt | Kevin_Zheng: your blueprint "Add request_id to instance action notifications [END DATE: 2018-05-01]" has been added to a review runway. please let us know if the next two weeks are not a good time for fast iteration on review https://etherpad.openstack.org/p/nova-runways-rocky | |
| 17:53:38 | dansmith | I assume we won't hear back on that until later this afternoon yeah? | |
| 17:54:07 | melwitt | maybe not. I didn't know what time zone he is | |
| 17:55:27 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Update forbidden spec to reflect chosen nova-side format https://review.openstack.org/562012 | |
| 17:55:30 | dansmith | melwitt: china | |
| 17:55:41 | melwitt | a-ha, cool | |
| 17:55:59 | melwitt | the series is half approved already and currently actively reviewed/responded so I expect it's going to be okay | |
| 18:07:39 | imacdonn | mriedem dansmith efried: need to decide what to do with https://review.openstack.org/#/c/558089/ - I'm leaning back towards just removing the check in init_host(). If it needs something more than that, I probably need to let it go to someone else | |
| 18:07:58 | openstackgerrit | Merged openstack/python-novaclient master: Fix comments in novaclient/tests/unit/fakes.py https://review.openstack.org/561772 | |
| 18:08:48 | efried | imacdonn: I thought we decided to basically leave the old check as is but switch to checking auth_type instead of region | |
| 18:09:07 | imacdonn | efried: I tried that, but it gets all complicated with unit testing again | |
| 18:09:33 | efried | You can't just replace os_region_name with auth_type throughout? | |
| 18:10:38 | imacdonn | efried: no, because all the tests that create a compute service try to actually use placement if auth_type is set, and we're back to messing around with fixtures (and it needs a valid auth_url, etc.) | |
| 18:10:59 | efried | eek, really? Hum, okay. | |
| 18:14:13 | mriedem | so zvm was moved out of runways? i don't see a log note for it | |
| 18:14:41 | dansmith | mriedem: there was ML traffic about it | |
| 18:15:02 | mriedem | sure but | |
| 18:15:06 | mriedem | sherbit | |
| 18:15:12 | dansmith | not in the lo because it's back in the queue I guess | |
| 18:15:13 | mriedem | sorbet? | |
| 18:15:37 | dansmith | sherbert | |
| 18:16:47 | mriedem | ok, haircut time it is | |
| 18:19:12 | melwitt | yeah, I didn't think to put log notes because it's back in the queue. but I'll add log notes to it now, I think that's a good idea | |
| 18:19:58 | dansmith | I did it | |
| 18:20:00 | dansmith | with a link to the ML | |
| 18:20:12 | melwitt | you rock, thanks | |
| 18:38:39 | openstackgerrit | Merged openstack/nova-specs master: Update forbidden spec to reflect chosen nova-side format https://review.openstack.org/562012 | |
| 18:40:54 | openstackgerrit | Merged openstack/nova master: Deduplicate notification samples Rocky - 2 https://review.openstack.org/560299 | |
| 18:41:05 | openstackgerrit | Merged openstack/nova master: Deduplicate notification samples Rocky - 3 https://review.openstack.org/560727 | |
| 18:41:21 | openstackgerrit | Merged openstack/nova master: doc: BFV instances and IsolatedHostsFilter https://review.openstack.org/543494 | |
| 18:42:18 | sean-k-mooney | anyone know where the people that work on diskimage-builder hang out on irc? #openstack-infra? | |
| 18:42:44 | dansmith | probs | |
| 18:42:59 | imacdonn | sean-k-mooney: there's supposed to be a #openstack-dib , according to https://wiki.openstack.org/wiki/IRC | |
| 18:43:17 | sean-k-mooney | imacdonn: ah thanks :) | |
| 18:43:54 | sean-k-mooney | i need to bookmark that wiki | |
| 18:47:23 | dansmith | lyarwood: on this https://review.openstack.org/#/c/561704 -- are we sure that the bit you're getting from qemu-img is the allocated part and not the would-be-allocated part? | |
| 18:50:53 | lyarwood | dansmith: it's the allocated part, virtual size is the potential allocation but thats the same for preallocated files like the example in the commit | |
| 18:51:44 | dansmith | lyarwood: yeah, I'm poking around with qemu-img right now | |
| 18:53:13 | dansmith | the fallocate command you have doesn't work for me, | |
| 18:53:27 | dansmith | but the neckbeard way of making a sparse file does show: | |
| 18:53:37 | dansmith | disk size: 1.0M | |
| 18:53:37 | dansmith | virtual size: 1.0G (1074790400 bytes) | |
| 18:53:37 | dansmith | dan@eagle:~$ qemu-img info -f raw testsparse.img | grep size | |
| 18:55:00 | lyarwood | dansmith: ah on osx? fallocate -n is odd and tbh I had no idea about it until this came up | |
| 18:55:07 | dansmith | lyarwood: nope, linux | |
| 18:55:21 | dansmith | fallocate: fallocate failed: keep size mode is unsupported | |
| 18:55:41 | lyarwood | dansmith: try -l 1M | |
| 18:56:04 | dansmith | it doesn't matter, I created a sparse file with dd | |
| 18:56:13 | lyarwood | dansmith: but anyway, it's weird as it leaves the EOF at the start of the file but allocates blocks past that. | |
| 18:56:22 | dansmith | because I'm old school cool | |
| 18:56:51 | dansmith | oh I see, a little different than sparse | |
| 18:56:58 | dansmith | that seems fairly evil | |
| 18:57:30 | dansmith | but, doesn't matter for what I was looking for | |
| 18:57:37 | lyarwood | kk | |
| 19:02:23 | dansmith | jaypipes: wanna send that ^ ? | |
| 19:02:28 | dansmith | pretty easy | |
| 19:03:57 | openstackgerrit | iain MacDonnell proposed openstack/nova master: Remove placement config check https://review.openstack.org/558089 | |
| 19:13:35 | jaypipes | dansmith: sorry, send what? | |
| 19:13:48 | dansmith | jaypipes: https://review.openstack.org/#/c/561704 | |
| 19:14:05 | dansmith | quick fix to look at the actual allocated size of a disk instead of just what the dumb ol' operating system thinks | |
| 19:14:11 | dansmith | (i.e. accounting for sparse files) | |
| 19:14:13 | jaypipes | ack. on itr. | |
| 19:23:55 | jaypipes | dansmith: /me wondering what "crudini" is... | |
| 19:24:10 | dansmith | eh? | |