Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-17
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 dan@eagle:~$ qemu-img info -f raw testsparse.img | grep size
18:53:37 dansmith virtual size: 1.0G (1074790400 bytes)
18:53:37 dansmith disk size: 1.0M
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?
19:24:11 openstackgerrit Merged openstack/nova master: Remove mox in test_serversV21.py (1) https://review.openstack.org/560821
19:25:00 jaypipes dansmith: +W'd it.
19:25:32 lyarwood jaypipes: ^_^ https://pypi.org/project/crudini/ aka openstack-config
19:26:02 lyarwood well, downstream at least, no idea if that wrapper was upstream at any point
19:26:50 dansmith jaypipes: pre-allocation of an image (even qcow) is for performance
19:27:59 jaypipes lyarwood: ah, cool. thx :)
19:28:45 jaypipes dansmith: oh? wasn't aware of that. is the performance due to contiguous blocks being grouped together?

Earlier   Later