Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-30
22:30:59 imacdonn ConnectFailure
22:31:39 imacdonn from ksa trying to do a GET request
22:32:30 fried_bunny mm
22:33:20 fried_bunny Well, I'm not happy that you're getting MissingAuthPlugin for /. But it's what you'll get for anything else you try, so that's not the end of the world.
22:34:07 imacdonn I guess that the client doesn't know that auth is not required to get the version
22:34:26 fried_bunny mordred will not be happy about that. Or maybe it's my fault.
22:35:40 fried_bunny anyway, it's sounding like to cover bases we may want to do something like .get('/resource_providers?name=bogus'), which *should* require auth, and catch both MissingAuthPlugin and ConnectFailure and convert those to PlacementNotConfigured.
22:37:23 fried_bunny imacdonn: If that's more than you want to take on, or if you want to write part of it and then hand it off, put something somewhere and flag me on it.
22:39:41 imacdonn fried_bunny: that doesn't seem too bad ... have to think through unit test implications too, though
22:40:19 fried_bunny I'd be fine just mocking .get. One case to raise MissingAuthPlugin, one to raise ConnectError. Done.
22:40:34 fried_bunny (and of course one to make it return successfully)
22:40:37 imacdonn yeah, that makes sense
22:40:39 fried_bunny (which is probably already covered elsewhere)
22:41:01 imacdonn I'll fiddle with that a bit ... and maybe we can discuss further with the others next week
22:41:37 fried_bunny Sounds great.
22:41:52 imacdonn thanks! :)
22:46:55 imacdonn just '/resource_providers' should work? That seems to require auth, and provides a meaningful result
22:53:55 fried_bunny imacdonn: If you had a lot of resource providers, it could be slowish. Adding the ?name=bogus should make it very quick (even if you have a provider named 'bogus')
22:54:32 fried_bunny I *think* that returns a result with an empty payload (as opposed to a 404).
22:54:41 imacdonn will try it
22:55:20 imacdonn re. the ConnectError ... IMO it's OK to just let that go ... it should be plenty-obvious what need to be done
22:57:39 fried_bunny imacdonn: wfm
22:58:05 fried_bunny imacdonn: Though the message about placement being required would still be useful there.
22:58:05 imacdonn raise exception.PlacementNotConfigured()
22:58:05 imacdonn except keystone_exception.MissingAuthPlugin:
22:58:05 imacdonn LOG.info(self.reportclient.get('/resource_providers?name=justcheckin'))
22:58:05 imacdonn try:
22:58:23 imacdonn confirm that that still gets a 200 ... logging is just to determine that
22:59:13 imacdonn thing is, the actual message that gets logged is "PlacementNotConfigured: This compute is not configured to talk to the placement service. Configure the [placement] section of nova.conf and restart the service."
22:59:39 imacdonn IMO, the placement service being down is not the same thing
23:01:05 fried_bunny fair and true.
23:01:22 fried_bunny imacdonn: I was thinking of the message that @safe_connect spews as being the useful one.
23:02:41 fried_bunny 'Please enable the placement service before upgrading.'
23:02:41 fried_bunny imacdonn: this one: 'Placement is optional in Newton, but required in Ocata. '
23:02:50 fried_bunny though that's probably pretty obsolete at this point.
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.

Earlier   Later