Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-11
00:12:31 tetsuro good morning
00:27:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_virt_drivers.py https://review.openstack.org/559878
00:28:33 gmann mriedem: yea, it is not in valid sort key as well as in ignore list also so this gives 400 - https://github.com/openstack/nova/blob/94b150bd79e5dfb4bdbb1aa9437c103ad5b80543/nova/api/openstack/compute/schemas/servers.py#L379
00:30:07 gmann tetsuro: morning
00:33:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (2/3) https://review.openstack.org/550648
00:33:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc (3/3) https://review.openstack.org/550659
00:34:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
00:34:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/test_utils.py https://review.openstack.org/557883
00:49:49 openstack Launchpad bug 1762876 in OpenStack Compute (nova) "KeyError during move operation functional tests" [Medium,Confirmed]
00:49:49 mriedem if anyone wants to chase down a functional test race bug over breakfast https://bugs.launchpad.net/nova/+bug/1762876
00:55:26 naichuans mriedem: Hi, Matt. Currently we have no test environments about live-migration in citrix CI, but we do have test jobs about it in Citrix XenRT, it is a internal test environment. And all the test related with the two patches are passed.https://review.openstack.org/#/c/489451/ and https://review.openstack.org/#/c/554154/
00:56:56 naichuans efried: Thank you for the feedback. Will follow up blueprint nested-resource-providers-allocation-candidates
01:31:00 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
01:31:01 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
01:31:02 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
01:31:41 Kevin_Zheng mriedem strange, I thought that one went well..
02:08:29 eandersson How does database pools work with workers? Does the conductor have one db pool per proccess?
02:08:48 eandersson and is the nova_api different?
02:33:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/api/openstack/compute/test_hosts.py https://review.openstack.org/559266
02:57:24 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
02:57:25 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
02:57:26 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
03:21:17 openstackgerrit Merged openstack/nova master: PowerVM: Add proc_units_factor conf option https://review.openstack.org/554688
03:30:51 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Enhance nova-specs webpage and clean up repo https://review.openstack.org/551802
04:21:40 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
04:21:41 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
04:21:42 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
04:46:24 openstackgerrit Nakanishi Tomotaka proposed openstack/nova master: Test Compute API in multiple cells https://review.openstack.org/560260
05:25:24 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
05:40:44 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
05:48:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in test_neutron_security_groups.py https://review.openstack.org/560267
05:49:29 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Return resources of entire trees in Placement https://review.openstack.org/559466
06:03:40 openstackgerrit Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078
06:03:41 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605
06:03:42 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
06:03:43 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
06:03:44 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
06:04:40 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
06:07:37 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Return resources of entire trees in Placement https://review.openstack.org/559466
06:18:01 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
06:57:55 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041
07:12:33 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041
07:13:21 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add microversion to support extra_specs in flavor API. https://review.openstack.org/554159
07:18:53 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
07:19:24 openstackgerrit Bence Romsics proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
07:19:25 openstackgerrit Bence Romsics proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
07:19:26 openstackgerrit Bence Romsics proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
07:27:49 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 1 https://review.openstack.org/560296
07:38:26 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 2 https://review.openstack.org/560299
07:44:06 takashin mriedem: Are you around?
07:44:39 takashin mriedem: Could you look https://bugs.launchpad.net/nova/+bug/1762941 ?
07:44:40 openstack Launchpad bug 1762941 in OpenStack Compute (nova) "testtools.matchers._impl.MismatchError: 'completed' != u'running' in test_bug_1718512.TestRequestSpecRetryReschedule.test_resize_with_reschedule_then_live_migrate" [Undecided,New]
08:06:02 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 2 https://review.openstack.org/560299
08:28:24 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
08:28:43 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 1 https://review.openstack.org/560296
08:28:59 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041
08:34:53 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 2 https://review.openstack.org/560299
08:39:06 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183
08:39:07 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
08:39:08 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
08:39:09 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
08:39:10 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
08:39:11 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
08:39:12 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
08:39:13 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
08:39:14 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
08:39:15 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
08:39:16 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
08:52:50 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
09:06:21 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
09:06:22 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
09:06:23 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
09:06:24 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
09:06:25 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
09:19:28 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 1 https://review.openstack.org/560296
09:38:33 kashyap mdbooth: Hi, would you happen to have a minute to debug a unit test failure that's driving me nuts?
09:39:02 kashyap mdbooth: That's the one: http://paste.openstack.org/show/718887/
09:39:11 mdbooth kashyap: Not this morning I'm afraid
09:39:18 kashyap mdbooth: Ah, no problem.
09:43:14 mdbooth kashyap: Without any kind of in depth look, and assuming you're just seeing teeth-gnashing weirdness, I suggest either:
09:43:22 mdbooth * You're not mocking what you think you're mocking
09:43:58 mdbooth * self._host.has_min_version returns something disgustingly pythonic with obtuse notions of true and false
09:45:12 kashyap mdbooth: I think it's the former; the "mock" thing.
09:45:47 kashyap I ran through PDB, and both: libvirt_driver.MIN_VIRTUOZZO_VERSION (from the test) and the MIN_VIRTUOZZO_VERSION from driver.py print the same version
09:47:41 kashyap Anyway; thanks for the look.
09:48:56 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add microversion to support extra_specs in flavor API. https://review.openstack.org/554159
09:49:35 mdbooth kashyap: You're mocking driver._conn btw
09:49:44 mdbooth But the test uses driver._host
09:49:47 mdbooth s/test/code/
09:51:42 mdbooth kashyap: Unfortunately your problem is vastly more interesting than what I'm actually supposed to be doing.
09:51:57 kashyap :-)
09:52:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 2 https://review.openstack.org/560299
09:53:57 mdbooth kashyap: Don't assume that the previous unit test was correct. It may have worked only by coincidence. This isn't uncommon.
09:54:13 kashyap This doesn't fly either:
09:54:14 kashyap + as mock_getver:
09:54:14 kashyap + with mock.patch.object(driver.init_host, 'getVersion') \
09:54:31 kashyap mdbooth: Sigh, yeah, I recall johnthetubaguy saying our MIN_VERSIONS testing has been a "joke"
09:54:32 mdbooth driver.init_host is a function
09:54:42 mdbooth That doesn't really make sense
09:55:36 mdbooth If you look at Host.has_min_version

Earlier   Later