| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-02-24 | |||
| 11:00:17 | bauzas | sean-k-mooney: I fully cleaned all my local eggs but I still had the issue | |
| 11:00:51 | bauzas | sean-k-mooney: and I finally was able to fix it by asking tox to use sdist (ie. removing use_develop in tox.ini) | |
| 11:01:30 | bauzas | weirod | |
| 11:01:33 | bauzas | weirdo | |
| 11:03:54 | bauzas | stephenfin: you're a pbr expert (heh), any idea why https://tox.readthedocs.io/en/latest/example/general.html#avoiding-expensive-sdist doesn't work for me and why by not asking use_develop=True works ? | |
| 11:04:00 | bauzas | for the nova repo | |
| 11:12:50 | bauzas | stephenfin: context is http://paste.openstack.org/show/789922/ and looks like I can't use use_develop, but I rather need to create a packaged install | |
| 11:32:32 | stephenfin | bauzas: Not sure, but I don't think that's the issue. I think the issue is with the latest major version of oslo.config | |
| 11:32:49 | stephenfin | We've been seeing issues with other projects like oslo.policy because of it | |
| 11:34:22 | stephenfin | bauzas: What do you mean it doesn't work though? | |
| 11:34:43 | bauzas | stephenfin: like I said, I get http://paste.openstack.org/show/789922/ by not modifying tox.ini | |
| 11:35:15 | bauzas | stephenfin: but if i ask to not using the development mode (ie. https://github.com/openstack/nova/blob/master/tox.ini#L11) then it works | |
| 11:35:30 | stephenfin | and if you remove that, things work? Can you give me the output until 'py37 run-test:' ? | |
| 11:36:32 | bauzas | stephenfin: yes, but WDYM by py36 run-test N | |
| 11:36:36 | bauzas | whoops | |
| 11:36:41 | bauzas | py37 run-test | |
| 11:36:57 | stephenfin | line 7 in the paste you provided | |
| 11:37:11 | stephenfin | I want to see what the difference in the commands is if you remove usedevelop | |
| 11:37:32 | bauzas | ah ah ok | |
| 11:38:32 | bauzas | stephenfin: http://paste.openstack.org/show/789923/ | |
| 11:39:14 | bauzas | stephenfin: also see the git diff tox.ini http://paste.openstack.org/show/789924/ | |
| 11:40:25 | stephenfin | you're also ignoring upper constraints too. Why's that necessary? | |
| 11:47:15 | sean-k-mooney | that is strange | |
| 11:47:23 | sean-k-mooney | bauzas: did your env break again | |
| 11:54:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Make image fixtures simpler to reference https://review.opendev.org/690403 | |
| 13:11:14 | openstackgerrit | Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494 | |
| 13:26:52 | bauzas | stephenfin: because of https://review.opendev.org/#/c/709389/5 that wasn't merged yet | |
| 13:30:15 | openstackgerrit | Alexandre arents proposed openstack/nova master: Avoid allocation leak when deleting instance stuck in BUILD https://review.opendev.org/702368 | |
| 13:54:55 | bauzas | ERROR: Could not find a version that satisfies the requirement gitdb2===4.0.1 (from -c https://releases.openstack.org/constraints/upper/master (line 777)) (from versions: 2.0.0, 2.0.1, 2.0.2, 2.0.3, 2.0.4, 2.0.5, 2.0.6, 3.0.0, 3.0.1, 3.0.2, 3.0.3, 3.0.3.post1, 4.0.2) | |
| 13:55:36 | bauzas | but https://review.opendev.org/#/c/709389/ is merged and I can see gitdb==4.0.2 in https://opendev.org/openstack/requirements/raw/branch/master/upper-constraints.txt | |
| 13:56:17 | aarents | bauzas: got the same issue since this morning | |
| 13:56:31 | bauzas | aarents: was due to some bug, see the change | |
| 13:57:31 | aarents | bauzas: ok | |
| 14:02:23 | bauzas | mmm, removed my local pip cache, the URL was cached | |
| 14:07:10 | stephenfin | merged now, thankfully | |
| 14:11:13 | bauzas | okay, still puzzled by stestr not knowing the run command | |
| 14:11:30 | bauzas | stephenfin: looks like 'run' is a pbr entrypoint | |
| 14:11:40 | bauzas | https://github.com/mtreinish/stestr/blob/master/setup.cfg#L40 | |
| 14:11:48 | bauzas | and stestr relies on cliff | |
| 14:12:10 | bauzas | so I suspect something messed up with entrypoints when I use develop | |
| 14:12:19 | stephenfin | bauzas: '--remote_debug-port' is a nova config option though | |
| 14:12:50 | bauzas | stephenfin: sorry, where ? | |
| 14:12:56 | stephenfin | http://paste.openstack.org/show/789922/ | |
| 14:12:59 | stephenfin | from that | |
| 14:13:15 | stephenfin | what seems to be happening is arguments from some command are being registered against others | |
| 14:13:30 | stephenfin | it's like argparse is maintaining some kind of global state | |
| 14:14:31 | stephenfin | bauzas: As proof, try this: virtualenv .venv; source .venv; pip install stestr; stestr -h | |
| 14:14:35 | bauzas | oh, you're right | |
| 14:14:42 | stephenfin | you'll get a totally different output | |
| 14:14:47 | bauzas | I can't see any of the usual stestr options | |
| 14:14:51 | stephenfin | exactly | |
| 14:15:01 | stephenfin | I just haven't figured out why | |
| 14:15:41 | bauzas | I'm just creating a venv as a tox target | |
| 14:15:43 | stephenfin | We worked around it in glance with some patch that I'll need to go root for, but the ultimate issue is that there is some kind of global state being maintained by cliff or oslo.config that there shouldn't be | |
| 14:15:49 | bauzas | to see whether stestr is messed up too | |
| 14:16:39 | stephenfin | bauzas: I assume you're not seeing this issue on master? | |
| 14:16:54 | bauzas | I'm seeing it on any branches | |
| 14:16:55 | bauzas | but, | |
| 14:17:18 | bauzas | if I reclone nova to another directory, it then works | |
| 14:17:31 | bauzas | so the issue is only on the main git repo | |
| 14:17:34 | stephenfin | Backup anything you want and 'git clean -xfd' ? | |
| 14:17:56 | stephenfin | Maybe you have a hangover pyc file that the interpreter is loading | |
| 14:18:04 | stephenfin | I've seen that cause other issues in the past | |
| 14:18:24 | bauzas | maybe | |
| 14:18:41 | stephenfin | ('git clean -xfd' will wipe all untracked files including those ignored by '.gitgnore' files, btw) | |
| 14:18:56 | stephenfin | (I know you probably know that already) | |
| 14:21:31 | bauzas | indeed, but haven't thought about it | |
| 14:23:15 | bauzas | it does fix the problem | |
| 14:23:36 | bauzas | have no f*** idea what the issue was, but now stestr works with develop mode | |
| 14:23:57 | bauzas | (which is understandable, since I asked to delete all pyc's) | |
| 14:36:09 | efried | I'm going to take at least the morning, possibly the day. Email if anything needs my attention. o/ | |
| 14:36:20 | gibi | efried_pto: o/ | |
| 15:06:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/689278 | |
| 15:06:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Unplug VIFs as part of cleanup of networks https://review.opendev.org/663382 | |
| 15:06:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix incorrect vm and task state after build failure race https://review.opendev.org/689388 | |
| 15:08:04 | stephenfin | gibi: Think you could take a look at ^ this week? | |
| 15:10:04 | elod | lyarwood: hi, do we need to wait for any patch in Rocky (for final release before Extended Maintenance transition?) | |
| 15:11:02 | lyarwood | elod: there's nothing from my side that can't wait until after EM | |
| 15:11:24 | lyarwood | elod: RH isn't shipping Rocky anymore FWIW | |
| 15:12:22 | elod | lyarwood: i see, good. Then I guess we could create a release patch | |
| 15:12:31 | gibi | stephenfin: queued | |
| 15:12:50 | elod | lyarwood: i can try to do that (i've never done before) | |
| 15:13:31 | gibi | stephenfin: I will start with those tomorrow (currently deep in a pike re-schedule bug downstream) | |
| 15:13:40 | elod | lyarwood: is that enough for the release or are there anything else that need to be done? | |
| 15:14:20 | lyarwood | elod: that's it AFAIK, efried_pto will need to bless it as PTL but that's about it | |
| 15:16:30 | elod | lyarwood: sounds good \o/ then i'll create the patch and add you then efried_pto as reviewer | |
| 15:18:00 | lyarwood | elod: awesome thanks | |
| 15:34:06 | elod | lyarwood: here it is: https://review.opendev.org/#/c/709554/ i think 18.2.4 will be fine, but please double check | |
| 15:53:22 | bauzas | efried_pto: stephenfin: thanks for reviewing https://review.opendev.org/#/c/589085/ | |
| 15:53:50 | bauzas | FWIW, I haven't yet provided a ML thread about this because I also want to fix the resize revert | |
| 15:53:59 | bauzas | so, I'll provide two other changes | |
| 15:54:14 | bauzas | once it's done, I'll also provide a ML thread | |
| 17:27:03 | openstackgerrit | Merged openstack/nova master: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/679502 | |
| 19:48:40 | openstackgerrit | melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600 | |
| 20:00:32 | openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Fix os-simple-tenant-usage result order https://review.opendev.org/709603 | |
| #openstack-nova - 2020-02-25 | |||
| 00:49:49 | sean-k-mooney | dansmith: i think https://review.opendev.org/#/c/709641/2 will create a tempest full multi node job with a cyborg flavor. it will take a while too run but it should be close if it does not fully work so ill take a look at it agian tomorow | |
| 02:11:11 | openstackgerrit | wangjiajing proposed openstack/nova master: Perfect parameter of method. https://review.opendev.org/709652 | |
| 03:18:09 | openstackgerrit | Merged openstack/nova master: Deprecate base rules in favor of new rules https://review.opendev.org/701624 | |
| 05:28:38 | openstackgerrit | Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772 | |
| 05:29:03 | openstackgerrit | MIKI Nobuhiro proposed openstack/nova master: Enable instance to be booted with neutron SR-IOV ports https://review.opendev.org/709658 | |