Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-09
16:44:14 sean-k-mooney i was thinking more doing set.flags in the base test case
16:44:29 mriedem nova.conf doesn't have anything to do with test execution timeout...
16:44:47 sean-k-mooney i was wondering if we could modify libvirt timeouts in nova.conf
16:45:02 mriedem nope
16:45:11 openstackgerrit Merged openstack/nova master: Implement update_provider_tree https://review.opendev.org/667417
16:45:20 mriedem configuring libvirt is kind of a libvirt.conf thing right?
16:45:50 sean-k-mooney ya i guess i think these timeout are comeing form teh python binding or the c lib rather then libvirt
16:47:03 sean-k-mooney ill give it another 15mins and if its still timing out ten ill run with OS_TEST_TIMEOUT=30 currently its timing out after 160s
16:50:40 sean-k-mooney actully i might make that OS_TEST_TIMEOUT=8 im not seeing any test over about 4.5 seconds
16:51:09 sean-k-mooney oh there is 1 at 13s
16:55:52 dansmith artom: I assume you're working on feedback on the resize revert patches yeah?
17:09:49 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912
17:12:50 openstackgerrit Merged openstack/nova master: xenapi: implement update_provider_tree https://review.opendev.org/667435
17:12:58 openstackgerrit Merged openstack/nova master: Deprecate non-update_provider_tree compat code https://review.opendev.org/667442
17:13:06 Sundar gibi, efried: Re. https://review.opendev.org/#/c/657464/, there was some discussion over naming, which got resolved in the review of the spec https://review.opendev.org/#/c/603955/12/specs/train/approved/nova-cyborg-interaction.rst@307 . Can we resume the review of 657464?
17:13:07 openstackgerrit Merged openstack/nova master: Update COMPUTE_STATUS_DISABLED from set_host_enabled compute call https://review.opendev.org/668743
17:14:02 efried Sundar: As far as I'm concerned, absolutely. Not much I can do though, since I own the patch :)
17:31:49 openstackgerrit Merged openstack/nova master: Add placement request pre-filter compute_status_filter https://review.opendev.org/668752
17:31:56 openstackgerrit Merged openstack/nova master: Refactor HostAPI.service_update https://review.opendev.org/668756
17:32:05 openstackgerrit Merged openstack/nova master: Sync COMPUTE_STATUS_DISABLED from API https://review.opendev.org/654596
17:38:33 artom dansmith, yeah
17:41:43 dansmith cool
17:54:07 openstackgerrit Merged openstack/nova master: Follow up for pre-filter-disabled-computes series https://review.opendev.org/668986
18:51:04 mriedem +2 on the host/hypervisor_hostname on server create api change again https://review.opendev.org/#/c/645520/
19:13:49 openstackgerrit Merged openstack/nova master: Cleanup quota user docs https://review.opendev.org/662573
19:16:19 openstackgerrit Matt Riedemann proposed openstack/nova master: nova-lvm: Disable [validation]/run_validation in tempest.conf https://review.opendev.org/662176
19:30:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts https://review.opendev.org/666767
19:34:37 cdent mriedem: I think you already know this, but my coverage run locked up and stayed locked (until I forced it dead just now) in nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_check_can_live_migrate_dest_file_backed_discard_bad_libvirt
19:37:43 mriedem i also killed mine
19:37:49 mriedem but posted a change
19:37:56 mriedem which will probably timeout :)
19:39:51 mriedem ah yeah
19:39:51 mriedem @mock.patch.object(fakelibvirt.Connection, 'getVersion')
19:39:51 mriedem @mock.patch.object(fakelibvirt.Connection, 'getLibVersion')
19:40:01 mriedem but that doesn't mock out conn = self.get_connection()
19:50:35 sean-k-mooney ya i have seen the same
19:51:57 sean-k-mooney i ws just checking it before i pip out to the shops to get dinner but i was thinking maybe we could run the test in two batchs
19:52:33 sean-k-mooney run the funtional test only first then unit tests only then have coverage combine the combine the two into one set of results
19:54:41 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: remove unused Service.get_by_compute_host mocks https://review.opendev.org/669934
20:01:06 mriedem sean-k-mooney: huh https://github.com/nedbat/coveragepy/blob/6b72dcbd98c69a7e8c5333d2926c7f871161db7e/coverage/collector.py#L115
20:01:09 mriedem the docs are really not clear about that
20:04:20 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912
20:06:28 mriedem efried: can you re-approve https://review.opendev.org/#/c/666042/ since i just had to rebase and fix the pep8 thing due to the newly enforced hacking rules
20:06:58 efried ...
20:08:22 efried done
20:10:28 mriedem thanks
21:05:37 mriedem dansmith: you might have an opinion on this https://review.opendev.org/#/c/669545/
21:07:42 dansmith hrm
21:07:57 dansmith I hate to add a traceback, but probably makes sense in the diaper case
21:08:22 dansmith is there a bug filed that shows what the actual issue was?
21:08:34 dansmith because we should definitely not take that path and log a traceback fro anything we can handle
21:09:09 mriedem no bug as far as i know
21:09:30 mriedem looks like something with nova - or some python-*client downstream - trying to get a response.status_code
21:09:34 mriedem and response is None
21:09:49 mriedem i thought of you b/c of the change i linked in there
21:12:41 dansmith yeah, and that case makes sense to not dump stack I think
21:12:54 dansmith but if we're not handling it, dumping is the right thing to do I think
21:13:16 dansmith point is, I think we should make sure we've fixed the problem that this would be dumping stack for
21:13:23 dansmith in addition to dumping stack for the future
21:14:11 mriedem if you're +2 i'll follow your lead
21:14:18 mriedem into the bowels of hell
21:15:28 dansmith commented
21:26:52 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: Replace a wiki link with API ref guide link https://review.opendev.org/669027
21:28:20 openstackgerrit Eric Fried proposed openstack/nova master: Perf: Use dicts for ProviderTree roots https://review.opendev.org/637225
21:28:55 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add a guide to add a new microversion support https://review.opendev.org/667002
21:36:17 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: Add links to novaclient contributor guide https://review.opendev.org/669028
22:39:05 efried dustinc: Bottom SDK patch is in merge conflict, not sure when that happened. I'm going to rebase it and the placement patch and leave the ironic series to you, mkay?
22:39:48 dustinc Yeah I have it taken care of in local branch, will push it this week
22:40:13 dustinc Thanks for heads up
22:40:42 openstackgerrit Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664
22:40:42 openstackgerrit Eric Fried proposed openstack/nova master: Use OpenStack SDK for placement https://review.opendev.org/656023
22:44:31 efried dustinc: ^
22:44:31 efried be careful not to regress to a previous commit, then. You should pull the bottom patch and rebase your series onto it.
22:45:57 dustinc efried: hmm ok, yeah I had already rebased and resolved that conflict locally so will have to figure that out before I push then
22:46:11 efried right. lmk if you need help.
22:49:17 openstackgerrit Merged openstack/nova master: Fix GET /servers/detail host_status performance regression https://review.opendev.org/666042
22:49:38 openstackgerrit Merged openstack/nova master: libvirt: remove unused Service.get_by_compute_host mocks https://review.opendev.org/669934
23:23:34 openstackgerrit Merged openstack/nova master: doc: Replace a wiki link with API ref guide link https://review.opendev.org/669027
23:56:05 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Deprecate cells v1 and extension commands and APIs https://review.opendev.org/669597
#openstack-nova - 2019-07-10
00:04:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add a live migration regression test https://review.opendev.org/641200
00:59:39 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Fix GET /servers/detail host_status performance regression https://review.opendev.org/669958
01:01:31 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Fix GET /servers/detail host_status performance regression https://review.opendev.org/669958
01:20:07 openstackgerrit Merged openstack/python-novaclient master: Add irrelevant files in dsvm job again https://review.opendev.org/668293
01:27:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.opendev.org/614036
01:27:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.opendev.org/631123
01:27:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.opendev.org/627892
01:27:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.opendev.org/631581
01:27:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.opendev.org/633853
01:27:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_dest compute method https://review.opendev.org/633293
01:27:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtDestTask https://review.opendev.org/627890
01:27:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.opendev.org/634832
01:27:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova.compute.utils.delete_image https://review.opendev.org/637605
01:27:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtSourceTask https://review.opendev.org/627891
01:27:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor ComputeManager.remove_volume_connection https://review.opendev.org/642183
01:27:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Add power_on kwarg to ComputeDriver.spawn() method https://review.opendev.org/642590
01:27:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Add finish_snapshot_based_resize_at_dest compute method https://review.opendev.org/635080
01:27:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add FinishResizeAtDestTask https://review.opendev.org/635646
01:27:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.opendev.org/614035
01:27:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.opendev.org/635668
01:27:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Plumb allow_cross_cell_resize into compute API resize() https://review.opendev.org/635684

Earlier   Later