Earlier  
Posted Nick Remark
#openstack-nova - 2018-03-30
23:03:07 imacdonn yeah
23:13:42 imacdonn eep .... 27 unit tests fail with the code-change... this is going to be fun :)
23:14:55 fried_bunny imacdonn: In how many test classes? You should be able to mock the reportclient.get in their respective setUp()s.
23:16:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527
23:16:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Update the cells FAQs and scheduler maintenance docs. https://review.openstack.org/556459
23:16:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Add --enable and --disable options to nova-manage update_cell https://review.openstack.org/555416
23:19:31 imacdonn at least 6 different classes :/
23:26:25 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test live_migration_wait_for_vif_plug=True https://review.openstack.org/558006
23:53:27 fried_bunny leakypipes: still around?
#openstack-nova - 2018-03-31
00:27:49 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Return anchor providers in a_r and p_s https://review.openstack.org/558014
00:27:59 fried_bunny leakypipes: tetsuro: ^
00:28:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423
00:28:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604
00:28:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170
00:28:15 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370
00:28:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333
00:28:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947
00:28:17 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: compute: use port binding extended API during live migration https://review.openstack.org/551371
00:28:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334
00:28:18 openstackgerrit Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537
00:28:18 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Port binding based on events during live migration https://review.openstack.org/434870
00:45:42 tetsuro @fried_bunny: Thank you! Let me look into that on Monday.
00:45:47 tetsuro It's already Saturday here. I should go to see the cherry blossoms today :)
00:53:20 openstackgerrit Merged openstack/nova stable/pike: compute: Cleans up allocations after failed resize https://review.openstack.org/548584
01:07:03 Spaz-Home Cherry blossumes starting to bloom in Korea
01:07:14 Spaz-Home I wish I could head into the mountains
05:44:31 Kumar hello
05:45:42 Kumar I am finding difficulty on running guru meditation report while selinux mode set as enforcing
05:46:02 Kumar here is the trace:
05:46:03 Kumar nova-api: File "/usr/lib64/python2.7/site-packages/psutil/_pslinux.py", line 1092, in wrapper
05:46:03 Kumar nova-api: return self._proc.ppid()
05:46:04 Kumar nova-api: Unable to run Guru Meditation Report!
05:46:04 Kumar nova-api: AccessDenied: psutil.AccessDenied (pid=1)
05:46:04 Kumar nova-api: raise AccessDenied(self.pid, self._name)
05:46:11 Kumar any solution?
06:49:08 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add microversion to support extra_specs in flavor API. https://review.openstack.org/554159
07:01:12 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
07:27:02 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145
09:42:24 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [placement] Add test for provider summaries https://review.openstack.org/558044
09:42:25 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add unrequested resources to provider_summaries https://review.openstack.org/558045
14:30:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix cancel_all_events event name parsing https://review.openstack.org/558059
15:25:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334
15:25:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333
15:25:14 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Port binding based on events during live migration https://review.openstack.org/434870
15:25:14 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: compute: use port binding extended API during live migration https://review.openstack.org/551371
15:25:15 openstackgerrit Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537
16:20:49 openstack Launchpad bug 1760322 in OpenStack Compute (nova) "Traits not synced if first retrieval fails" [Undecided,New]
16:20:49 fried_bunny cdent: here's one for ya: https://bugs.launchpad.net/nova/+bug/1760322
16:29:33 openstackgerrit Eric Fried proposed openstack/nova master: Test case: traits don't sync if first access fails https://review.openstack.org/558066
16:29:44 fried_bunny cdent: There's the test case ^
16:44:57 openstackgerrit Merged openstack/nova master: Fix allocation_candidates not to ignore shared RPs https://review.openstack.org/533396
16:48:22 openstackgerrit Eric Fried proposed openstack/nova master: Don't declare traits synced if they ain't https://review.openstack.org/558068
16:48:34 fried_bunny cdent: And a (wrong) fix ^
16:48:47 fried_bunny leakypipes: I'm sure you know how to do this right (see commit message)
18:27:48 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
18:29:54 openstackgerrit Eric Fried proposed openstack/nova master: Add unrequested resources to provider_summaries https://review.openstack.org/558045
18:29:54 openstackgerrit Eric Fried proposed openstack/nova master: [placement] Add test for provider summaries https://review.openstack.org/558044
18:29:55 openstackgerrit Eric Fried proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437
18:29:55 openstackgerrit Eric Fried proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
18:29:56 openstackgerrit Eric Fried proposed openstack/nova master: Return anchor providers in a_r and p_s https://review.openstack.org/558014
18:33:45 imacdonn fried_bunny: you around ?
18:33:53 fried_bunny imacdonn: Yeaux
18:34:21 fried_bunny you admining my placement hacking?
18:34:29 fried_bunny heh, admiring
18:34:37 fried_bunny Not sure what admining is. But it sounds profitable.
18:34:58 imacdonn admining is a thankless task ... if it has sys- as a prefix, at least
18:35:21 fried_bunny oh, as in "doing admin things". I read it as "ad mining"
18:35:43 fried_bunny I agree admin-ing doesn't sound as profitable
18:36:09 imacdonn and it may actually be a real word! https://english.stackexchange.com/questions/115422/administrating-vs-admining
18:36:47 imacdonn er anyway, I was poking at the unit tests for this placement-is-configured check thing
18:37:05 imacdonn all the tests that create a compute service as a fixture are failing
18:37:23 imacdonn I don't know much about fixtures
18:37:23 fried_bunny (article declaims NOT a real word - slang only)
18:37:46 fried_bunny Have you posted anything on gerrit yet that I can have a gander at?
18:38:05 imacdonn negative .. wanted to get it to pass tox first
18:38:40 imacdonn simple test case where it fails.... http://paste.openstack.org/show/718086/
18:38:57 imacdonn I don't know how to mock things in a fixture, if that even makes sense
18:41:53 fried_bunny yeah, okay; it must be getting mocked out *somewhere* or a shit ton of the other tests would be failing previously. I wonder if we're just mocking out the report client everywhere.
18:42:06 fried_bunny How many failures are you seeing when you run the world?
18:42:26 imacdonn there are like 26 other tests that fail .. I just picked that one because it's really simple
18:43:25 fried_bunny How many different files/classes are the failures in?
18:43:48 imacdonn at least 6
18:44:15 imacdonn you just sparked a thought, though ... now looking for mentions of reportclient under tests/
18:44:23 imacdonn like ... nova/tests/unit/compute/test_compute_mgr.py: @mock.patch.object(self.compute, 'reportclient')
18:44:47 imacdonn not sure if any use fixtures, though
18:45:38 fried_bunny They wouldn't need to, if they all mock the reportclient. A fixture is basically a way to make a complicated mock/mock.patch.
18:45:54 fried_bunny Do all of the failures have self.start_service('compute') in the stack trace?
18:46:32 imacdonn certainly the majority of them do
18:47:48 imacdonn the nova.tests.unit.api.openstack.compute.test_services.ServicesTestV253 are a bit different
18:48:15 fried_bunny Well, just to try things out, find nova.test.TestCase.start_service
18:48:29 imacdonn actually scratch that .. it just looks different in the trace due to multi-line
18:48:38 fried_bunny and under the name == 'compute' conditional, add:
18:50:09 fried_bunny self.useFixture(nova_fixtures.PlacementFixture())
18:50:45 imacdonn I thought about something like that ... but I also don't know where the fixture gets its config from
18:50:58 fried_bunny It uses the conf patcher :)
18:51:12 fried_bunny ...which is instantiated further down the line.
18:51:25 imacdonn ok, but where does that get its config from :)
18:52:13 fried_bunny It's *possible* it gets layered.
18:52:19 fried_bunny Which is kinda cool.
18:52:39 fried_bunny I think you make conf changes in tests via self.flags

Earlier   Later