| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-16 | |||
| 12:17:37 | gibi | let me mock the RPC call to see if that helps | |
| 12:19:28 | gibi | sean-k-mooney: yep the same mock helps | |
| 12:19:40 | gibi | pushing a fix... | |
| 12:19:47 | sean-k-mooney | cool | |
| 12:20:12 | sean-k-mooney | so it was just trying to hit rabbit and timeing out or somehting wehn enableding/disabling rabbit | |
| 12:20:49 | gibi | sean-k-mooney: yes. since https://review.opendev.org/#/q/If32bca070185937ef83f689b7163d965a89ec10a the service disable/enable api call call down to the compute so the compute can update traits in placement | |
| 12:21:23 | gibi | in these tests there is no compute so we are waiting for a timeout that is then catched and droped by the caller | |
| 12:21:34 | gibi | s/catched/caught/ | |
| 12:22:15 | sean-k-mooney | im surpised no one completed about long running ci jobs | |
| 12:22:28 | sean-k-mooney | although i guess tempest will always be longer | |
| 12:27:16 | gibi | I saw the slow functional tests locally but somehow I did not found the same for the unit test, I guess functional takes longer so I always waited for that | |
| 12:28:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Avoid timeout from service update api unit tests https://review.opendev.org/676923 | |
| 12:29:06 | sean-k-mooney | i just noticed because i went to grab my phone form my bedroom and then got a glass of water and noticed it was still running and it looked like it was hung. | |
| 12:29:14 | sean-k-mooney | then it updated with a long test | |
| 12:30:30 | sean-k-mooney | im seeing some other errors too. i just checkout out the latest master and ran them and im getting "TypeError: missing a required argument: 'project_id'" | |
| 12:31:28 | sean-k-mooney | i wonder if i could have some stale pyc files? | |
| 12:32:24 | efried | stephenfin: Thanks for catching up the privsep series. Hopefully I'll get a chance to +A those suckers today. | |
| 12:36:36 | gibi | sean-k-mooney: the rest of the py27 run was clean for me top of master | |
| 12:37:29 | sean-k-mooney | gibi: im seeing this on py36 http://paste.openstack.org/show/757964/ | |
| 12:37:42 | sean-k-mooney | im cloneing a fresh repo to see if it makes any difference | |
| 12:37:59 | sean-k-mooney | there are 3 other failure all in the sam test class | |
| 12:40:31 | sean-k-mooney | the project id is not set in the context object | |
| 12:40:36 | sean-k-mooney | oh it is | |
| 12:40:47 | sean-k-mooney | its uuids.my_tenant | |
| 12:44:28 | gibi | sean-k-mooney: py36 was clean for me too | |
| 12:45:06 | sean-k-mooney | ill know in a second if the clean repo is clean | |
| 12:45:37 | sean-k-mooney | i delete the pyc files and it still happened so i dont know what it could be in the other one | |
| 12:46:10 | sean-k-mooney | oh ya i have to wait for those tests without the rpc stub... | |
| 12:47:54 | sean-k-mooney | nope same 4 failtures on a clean clone | |
| 12:48:01 | gibi | if it still fails for you then it might be the virtualenv. I did not recreate that | |
| 12:49:15 | sean-k-mooney | http://paste.openstack.org/show/757965/ | |
| 12:50:20 | openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Test sdk Connection(strict_proxies) https://review.opendev.org/676926 | |
| 12:50:28 | efried | mordred: There's the sniffer ^ | |
| 12:50:52 | mordred | efried: fingers crossed | |
| 12:56:54 | sean-k-mooney | gibi it looks like an issue with " mocked_client = mock.create_autospec(client.Client)" | |
| 12:57:23 | sean-k-mooney | i progess passted that but when the client is used we get teh exception | |
| 12:57:24 | efried | what? Autospec causing problems? Never! | |
| 12:57:48 | gibi | sean-k-mooney: I still cannot reproduce it with a recreated virtualenv. Im using Python 3.6.9 | |
| 12:58:22 | sean-k-mooney | Python 3.6.8 | |
| 12:58:30 | sean-k-mooney | that is on rhel8 | |
| 12:58:37 | sean-k-mooney | ill try it on a ubunut system | |
| 13:00:58 | gibi | sean-k-mooney: I'm on 16.04 | |
| 13:01:29 | gibi | sean-k-mooney: with ppa:deadsnakes/ppa as the python source | |
| 13:01:46 | sean-k-mooney | it passed for my on python 3.6.8 on popos which is based ubuntu 18.04 | |
| 13:01:53 | sean-k-mooney | so this might be a rhel8 issue | |
| 13:02:33 | sean-k-mooney | i keep finding rhel8 issues that make me wonder why im trying to using... | |
| 13:02:59 | sean-k-mooney | i am so tempted to just install ubunut 18.04 but not quite yet... | |
| 13:06:05 | sean-k-mooney | gibi: thanks for checking :) | |
| 13:21:57 | KeithMnemonic | Can someone please see if they can WF+1 this change now https://review.opendev.org/#/c/674916/ ? | |
| 13:25:58 | efried | KeithMnemonic: You'll need a stable. mriedem is on vacation. dansmith or melwitt should be able to help once they arrive on the scene. (They're US Pacific, so it might be a bit.) | |
| 13:27:58 | KeithMnemonic | efried: thanks! | |
| 13:33:50 | dansmith | efried: 'scuse me? | |
| 13:34:39 | efried | dansmith: KeithMnemonic is looking for a second stable to push https://review.opendev.org/#/c/674916/ | |
| 13:34:50 | dansmith | efried: I know, | |
| 13:34:52 | efried | and yes, I know you're an early bird | |
| 13:34:58 | dansmith | mkay. | |
| 13:35:02 | efried | I was giving you an excuse to finish your coffee | |
| 13:35:09 | dansmith | hrmf. | |
| 13:35:13 | efried | you're welcome. | |
| 13:43:29 | KeithMnemonic | dansmith: Thanks! | |
| 13:44:17 | dansmith | yup | |
| 13:47:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Update docstring of 'revert_resize' function https://review.opendev.org/665976 | |
| 13:47:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Recalculate 'RequestSpec.numa_topology' on resize https://review.opendev.org/662522 | |
| 13:47:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Add '_setup_compute_services' helper https://review.opendev.org/663102 | |
| 13:47:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Cleanup of '_test_resize' helper test https://review.opendev.org/664245 | |
| 13:51:12 | aspiers | stephenfin: I've extracted this as requested https://review.opendev.org/#/c/638680/38//COMMIT_MSG@43 - will submit soon | |
| 13:51:44 | aspiers | stephenfin: you said you have another use for it https://review.opendev.org/#/c/638680/38/nova/tests/functional/libvirt/integrated_helpers.py@24 so thought I'd mention | |
| 13:52:21 | stephenfin | aspiers: Sweet | |
| 14:09:56 | yonglihe | stephenfin: do you have bandwidth to review https://review.opendev.org/#/c/621476/ , and you are NUMA guy. 😃 | |
| 14:12:39 | yonglihe | Hope this not too surprise 😄 | |
| 14:42:43 | efried | (failed, just starting to dig now) | |
| 14:42:43 | efried | mordred, dustinc: results are in https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b | |
| 14:43:15 | dustinc | Sigh.. | |
| 14:43:19 | mordred | efried: poop | |
| 14:43:35 | mordred | efried: I blame ... uhm ... blast, jaypipes isn't here anymore | |
| 14:44:07 | sean-k-mooney | mordred: blame friday and the gates hate of our weekends | |
| 14:44:07 | aspiers | :) | |
| 14:44:25 | sean-k-mooney | or at least efried weekend | |
| 14:46:25 | openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | |
| 14:46:26 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964 | |
| 14:47:25 | aspiers | stephenfin: ^^^ | |
| 14:49:11 | efried | We got the right sdk, I think https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/job-output.txt#924 | |
| 14:49:14 | sean-k-mooney | has the battle for 2.76 concluded by the way or is the power-update patch still under review | |
| 14:52:30 | efried | It looks like we're now raising VirtDriverNotReady properly when we get the failure to connect to ironic ++ | |
| 14:52:38 | efried | https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/controller/logs/screen-n-cpu.txt.gz?severity=1#935 | |
| 14:54:48 | efried | ...but then we stop getting that error, which implies that we got the connection good, which is what we expect --- but we still get ComputeHostNotFound | |
| 14:55:00 | efried | dustinc: It's possible that this ^ is an actual bug on the nova side | |
| 14:57:56 | mordred | efried: yay! that does seem like the sequence of actions we want | |
| 14:58:40 | efried | Reckon a little extra logging would help out here. | |
| 15:08:35 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make _rever_allocation nested allocation aware https://review.opendev.org/676138 | |
| 15:08:35 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate server with ports having resource request https://review.opendev.org/655113 | |
| 15:08:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140 | |
| 15:15:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497 | |
| 15:15:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972 | |
| 15:22:33 | openstackgerrit | Chris Dent proposed openstack/nova stable/stein: Add an issue releasenote for placement eventlet stall https://review.opendev.org/676973 | |
| 15:23:00 | cdent | efried, belmoreira : first draft ^ | |
| 15:33:56 | melwitt | cdent: fyi I had this similar reno going, is it for the same thing or no? https://review.opendev.org/662095 | |
| 15:34:35 | cdent | same issue, different context, so maybe worth having both? | |
| 15:34:50 | cdent | Not sure whether combining they will be useful or not | |
| 15:34:53 | melwitt | ok. I'm on a call right now but skimmed over | |