Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-19
15:35:12 sean-k-mooney so nova-live-migration will by python 2
15:35:16 mriedem it's not
15:35:16 sean-k-mooney ya i know
15:35:19 efried nope
15:35:31 stephenfin sean-k-mooney: it can't be. if it was, it wouldn't work for us :)
15:35:33 mriedem hi https://review.opendev.org/#/c/694787/
15:36:07 sean-k-mooney im looking at https://github.com/openstack/nova/blob/master/.zuul.yaml and i dont see python 3 enabled
15:36:08 efried because https://opendev.org/openstack/nova/src/branch/master/playbooks/legacy/nova-live-migration/run.yaml#L50 rite?
15:36:16 mriedem it's in the legacy playbook
15:36:17 mriedem yes that
15:36:37 mriedem cue sean-k-mooney saying we should really move those jobs to zuulv3
15:36:41 stephenfin mriedem: on that, you need to keep the sphinx on py2 line in doc/requirements.txt
15:36:50 stephenfin i tried removing it and got the same thing you're seeing
15:36:53 stephenfin the tool is dumb
15:37:17 sean-k-mooney ah ok
15:37:22 gmann ? anything changed ?
15:37:44 dansmith gmann: no, we're just taking bets on whether or not you're really going to merge the devstack change today
15:37:52 openstackgerrit Matt Riedemann proposed openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9 https://review.opendev.org/694787
15:38:17 dansmith gmann: efried and I will have to meet on the playground after school if not, and he'll kick my ass
15:38:28 gmann ok :)
15:43:20 openstackgerrit Merged openstack/nova master: functional: Make '_wait_for_state_change' behave consistently https://review.opendev.org/689180
15:47:02 gmann mriedem: sean-k-mooney for legacy jobs i am explicitly setting py3 via d-g var for other projects. waiting to move them to zuulv3 +py3 might take time.
15:49:31 gmann stephenfin: on requirement changes. I am removing the py2 only req also for other projects and if any project break anyone (which is rare case for all non-core/small projects) then we revert otherwise keep dropping those.
15:50:01 stephenfin ack
15:50:17 mriedem stephenfin: i'm going to drop the misleading comment on https://review.opendev.org/#/c/694789/2/playbooks/legacy/nova-grenade-multinode/run.yaml@35
15:50:45 stephenfin fine by me
15:52:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job https://review.opendev.org/694789
15:52:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job https://review.opendev.org/694789
16:00:44 openstackgerrit Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/694756
16:02:10 openstackgerrit Archit Modi proposed openstack/nova stable/pike: Revert "openstack server create" to "nova boot" in nova docs https://review.opendev.org/693236
16:05:39 openstackgerrit Archit Modi proposed openstack/nova stable/pike: Revert "openstack server create" to "nova boot" in nova docs https://review.opendev.org/693236
16:15:14 artom sean-k-mooney, FWIW, I'm not going to hassly you about the softer wording stuff in your NUMA rebuild patches. There's still enough stuff there to warrant a respin on both (albeit the top one is just typos)
16:15:17 artom *hassle
16:18:24 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Make '_IntegratedTestBase' subclass 'InstanceHelperMixin' https://review.opendev.org/689182
16:18:24 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Unify '_build_minimal_create_server_request' implementations https://review.opendev.org/695024
16:20:52 sean-k-mooney artom: i am going ot resping them for the typos yes
16:21:18 artom And the unit tests :)
16:21:24 artom Mostly the unit tests
16:24:01 sean-k-mooney i actully intionally did not add them but yes im going to add some for the neew fucntion
16:25:00 sean-k-mooney mainly because functional tests and exsiting test provide equal or better coverage in general but as noted ill add the test for the function you highlighted
16:29:31 openstackgerrit Thierry Carrez proposed openstack/nova master: Start README.rst with a better title https://review.opendev.org/695025
16:32:35 mgoddard mriedem: hi. The more I pick at this thing (rebalance) the more it comes apart :)
16:32:43 openstackgerrit Merged openstack/nova master: (Temporarily) readd bare support for py27 https://review.opendev.org/695007
16:33:11 gmann everyone, sent the ML for making devstack py3 default and other suggestion we agreed today- http://lists.openstack.org/pipermail/openstack-discuss/2019-November/010938.html
16:33:35 mgoddard mriedem: I have a few related but separable changes, and a functional tests. Does nova tend to prefer separate commits where possible, or shall I bung it all in one?
16:34:29 mriedem mgoddard: i can't speak for all of nova but i prefer separate commits if possible - easier to review, easier to revert if necessary
16:34:49 mgoddard mriedem: sure.
16:35:01 mriedem mgoddard: and i'm not surprised rebalance has issues since it's never had functional testing
16:35:16 kashyap mgoddard: On separate commits, this is a good reminder: https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes
16:35:24 mriedem the only way i found the race bug in the first place was digging into ironic multinode grenade logs looking for some other issue
16:35:29 kashyap (Bearing in mind things like bisectability, etc)
16:35:41 mgoddard mriedem: separate commits should be quite neat as each will fix a different part of the functional test
16:36:20 bauzas gibi: mriedem: do you think it would be worth inspecting the args when calling PlacementFixture ?
16:36:44 bauzas (related to the microversion/version issue)
16:37:15 mgoddard thanks kashyap. I generally prefer them, although it's not universal
16:39:32 efried bauzas: can you point me to the change in question please?
16:39:44 efried (It's somewhere on my list, been meaning to weigh in, but don't want to dig it up rn)
16:40:39 efried if this is a functional test, the PlacementFixture should be doing the right thing with its kwargs. And there should usually not be a reason to (re-)mock a method on the fixture.
16:40:44 bauzas efried: https://review.opendev.org/#/c/670112/8/nova/cmd/manage.py@2867
16:41:10 bauzas context is that I used a wrong argument : 'microversion' instead of 'version'
16:41:39 bauzas so I could rather use utils.getargspec()
16:41:49 efried I understand, gimme sec.
16:41:53 bauzas I mean, in the PlacementFixture
16:42:45 bauzas efried: ie. something around there https://github.com/openstack/nova/blob/master/nova/tests/functional/fixtures.py#L103
16:43:43 efried oh, crap, yeah, I remember trying to unwind this _fake_* gorp a while back and back-burnering it because it turned out to be hard.
16:44:37 efried yeah, this is (still) awful to have in the way of the actual placement fixture
16:44:52 efried it will mask all kinds of problems, like the one you're running into.
16:45:58 efried egads, we should also be making self._client a sdk proxy now as well.
16:46:07 bauzas you know what ? I'll provide a separate change
16:46:11 bauzas efried: ^
16:46:15 efried not that it's hugely different
16:46:28 efried bauzas: Totally agree a separate change is in order.
16:46:34 bauzas k
16:47:08 efried If you can't completely kill the _fake_* methods, they should at least be avoiding massaging the args as much as possible.
16:47:21 gmann stephenfin: can you rebase this on top of slaweq patch to dropneutron-grenade from devstack (commented on review) - https://review.opendev.org/#/c/649097/10
16:48:30 stephenfin ack
16:51:20 openstackgerrit Merged openstack/nova stable/pike: Revert "openstack server create" to "nova boot" in nova docs https://review.opendev.org/693236
16:52:23 stephenfin gmann: done
16:55:10 gmann stephenfin: thanks, only thing we need to care is neutron-grenade job which will not work on devstack-py3-default. slaweq has patches up to migrate its usage to grenade-py3 but not all are merged. so i am making neutron-greande explicitly say py2
16:56:03 stephenfin gmann: Makes sense. Do we need https://review.opendev.org/#/c/649097/ to depend on that so?
16:56:42 gmann stephenfin: yeah, i am pushing the neutron patch (neutron-grenade is in neutron repo) and ping you
16:58:11 gibi bauzas: tested your placement audit patch. The basic functionality seems to work with even bandwidth involved. But I think --delete is broken somehow. (See my comment in the review)
16:58:23 bauzas gibi: k, will look
16:58:37 bauzas gibi: and thanks :)
16:59:02 gibi np
16:59:34 bauzas gibi: I can't see your comment on delete ?
16:59:45 bauzas anyway, let's discuss this tomorrow
16:59:49 bauzas thanks gibi a lot
17:00:20 bauzas ah, nevermind
17:00:23 bauzas Ctrl-R
17:04:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'os-consoles' API https://review.opendev.org/687907
17:04:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova-console' service, 'console' RPC API https://review.opendev.org/687908
17:04:57 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909
17:59:41 mriedem dansmith: you want to move this bottom cross-cell patch along? https://review.opendev.org/#/c/642591/ - you should already be familiar with this one
18:10:57 dansmith mriedem: yeah, sorry I need to re-find my way on those
18:13:50 mriedem bauzas: easy bug fix https://review.opendev.org/#/c/694756/
18:15:17 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Ussuri" https://review.opendev.org/695056
18:58:22 openstackgerrit Boris Bobrov proposed openstack/nova master: Also enable iommu for virtio controllers and video in libvirt https://review.opendev.org/684825
19:19:49 openstackgerrit Merged openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9 https://review.opendev.org/694787
19:32:00 efried mriedem: Do you remember seeing/fixing this recently? https://zuul.opendev.org/t/openstack/build/ac433b3305ba454f845990f98a3bf481/log/logs/grenade.sh.txt.gz#38749

Earlier   Later