| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-03 | |||
| 21:59:15 | hansmoleman | cfriesen: of course not | |
| 21:59:38 | hansmoleman | jgwentworth: one way i thought about fixing that was adding a different migrate_server (or whatever it's called) method in conductor that's not trying to target a cell, | |
| 21:59:59 | hansmoleman | because that's how build_resources works, it's not using the @targets_cell decorator because it's called from the api for scheduling and from the compute for reschedules | |
| 22:00:10 | hansmoleman | so really we should do similar for rescheduling a resize/cold migrate | |
| 22:00:22 | cfriesen | hansmoleman: as a heads-up, I'm on vacation next week. you'll have to save up your questions. :) | |
| 22:00:31 | hansmoleman | cfriesen: damn | |
| 22:00:40 | hansmoleman | i'm leaving for china on friday | |
| 22:00:48 | cfriesen | you can try asking Dean | |
| 22:01:19 | hansmoleman | given i'm gotten through the api extension and nova/compute/* changes so far, i think i'm pretty good | |
| 22:01:30 | hansmoleman | i expect some scheduler stuff but mostly linked to what i've already seen | |
| 22:01:37 | hansmoleman | *i've | |
| 22:01:55 | cfriesen | there are some changes around the server group affinity to close some races in the scheduler | |
| 22:02:49 | hansmoleman | jgwentworth: reason i bring that up now, is if we fixed the bug that way, it's a new rpc api method which isn't backportable, | |
| 22:02:57 | hansmoleman | so we'd have to get that done before rc if we want it in rocky | |
| 22:04:49 | hansmoleman | i also thought about putting the az per host on the Selection object that we pass down for alternates, | |
| 22:04:58 | hansmoleman | but that's an rcp api version bump on the Selection object, so again, not backportable | |
| 22:06:36 | hansmoleman | ew it's actually 2 separate but very similar bugs and we'd need both | |
| 22:06:52 | hansmoleman | @targets_cell looks up the instance mapping which fails if the cell conductor isn't configured with the api db | |
| 22:07:03 | hansmoleman | looking up the host az fails if not configured for the api db | |
| 22:14:01 | hansmoleman | mnaser: i'm assuming you have [api_database]/connection configured in your cell conductors? or are you just using a single global conductor? | |
| 22:15:25 | mnaser | s/has/had/ | |
| 22:15:44 | mnaser | hansmoleman: right now just a global conductor but we'll be switching to cell conductor soon | |
| 22:33:55 | hansmoleman | jgwentworth: just added https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls for stein and to the ptg etherpad; i think it's too much change at this point to try and rush a fix for rocky | |
| 22:34:04 | hansmoleman | especially given it's been broken since pike | |
| 22:36:34 | hansmoleman | cfriesen: hmm looks like a bug upstream https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-378a96ec6159d0a2f8ec7ab71bc3843bR921 - i know we change the request spec on resize, but do we revert the request_spec.flavor on revert resize? | |
| 22:37:56 | hansmoleman | another good reason why we can't trust the request spec a lot of the time.. | |
| 22:41:10 | openstackgerrit | Merged openstack/os-vif master: Support for OVS DB TCP socket communication. https://review.openstack.org/587378 | |
| 22:45:21 | openstack | Launchpad bug 1785339 in OpenStack Compute (nova) "RequestSpec.flavor is not reverted on resize revert" [Medium,Triaged] | |
| 22:45:21 | hansmoleman | https://bugs.launchpad.net/nova/+bug/1785339 | |
| 22:45:22 | hansmoleman | busted since newton | |
| 23:14:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update RequestSpec.flavor on resize_revert https://review.openstack.org/588689 | |
| 23:18:05 | openstackgerrit | Merged openstack/nova master: Remove unused flavor_delete_info() method https://review.openstack.org/588621 | |
| 23:24:43 | hansmoleman | cfriesen: i see what you mean about affinity races https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-d29c9372baf108a281712642550918dcR88 | |
| 23:24:50 | hansmoleman | should probably report a bug for that | |
| 23:25:18 | hansmoleman | since we don't do the late anti-affinity check on compute like we do for server create and evacuate (which would be up-calls for cells v2 now too) | |
| 23:25:45 | openstack | Launchpad bug 1600251 in OpenStack Compute (nova) "live migration does not honor server group policy" [High,Confirmed] | |
| 23:25:45 | hansmoleman | oh heh https://bugs.launchpad.net/nova/+bug/1600251 | |
| 23:30:59 | hansmoleman | i think part of that is fixed with https://review.openstack.org/#/c/527799/ | |
| 23:31:08 | hansmoleman | but not sure where we re-calculate the group members prior to scheduling | |
| #openstack-nova - 2018-08-04 | |||
| 01:23:51 | openstackgerrit | Merged openstack/nova master: [placement] ensure_rc_cache only at start of process https://review.openstack.org/584086 | |
| 08:07:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor NeutronFixture https://review.openstack.org/588338 | |
| 10:37:59 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Adds a test for getting allocations API https://review.openstack.org/588886 | |
| 10:38:00 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Not use project table for user table https://review.openstack.org/588887 | |
| 17:57:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for rebuild https://review.openstack.org/588931 | |
| 18:41:57 | openstackgerrit | Merged openstack/nova master: clarify usage of upgrade_levels group https://review.openstack.org/583502 | |
| 19:00:32 | openstackgerrit | Merged openstack/nova master: Deprecate upgrade_levels options for deprecated/removed services https://review.openstack.org/588607 | |
| 20:18:24 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for live migrate with anti-affinity group https://review.openstack.org/588935 | |
| 23:00:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | |
| 23:30:28 | mriedem | cfriesen: surprised there isn't a bug upstream for this https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-34db24c0784faae38c6542db6cbcad13R130 | |
| 23:31:47 | mriedem | same for RequestSpec.pci_requests | |
| 23:32:11 | mriedem | we probably shouldn't persist those in RequestSpec, like we don't persist other per-operation stuff in the request spec now | |
| 23:32:13 | mriedem | dansmith: ^ | |
| 23:32:29 | mriedem | resize and/or rebuild could invalidate either of those fields in the request spec | |
| 23:34:01 | openstack | Launchpad bug 1763766 in OpenStack Compute (nova) "nova needs to disallow topology changes on image rebuild" [Medium,Triaged] | |
| 23:34:01 | mriedem | related https://bugs.launchpad.net/nova/+bug/1763766 | |
| #openstack-nova - 2018-08-05 | |||
| 08:42:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | |
| 16:16:30 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix server strings in reboot operation https://review.openstack.org/588981 | |
| 16:25:30 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Refactor the getid method in novaclient/base.py https://review.openstack.org/588983 | |
| 22:06:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | |
| #openstack-nova - 2018-08-06 | |||
| 01:55:15 | openstackgerrit | zhufl proposed openstack/nova master: Fix none-ascii char in doc https://review.openstack.org/588422 | |
| 02:01:27 | openstackgerrit | Pierre Blanc proposed openstack/nova master: Docs: Add guide to migrate instance with snapshot https://review.openstack.org/584442 | |
| 02:30:55 | openstackgerrit | lei zhang proposed openstack/nova master: [Docs] Update the confusing console output https://review.openstack.org/589004 | |
| 02:34:33 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add microversion info in the os-server-groups API samples https://review.openstack.org/589006 | |
| 02:52:23 | openstackgerrit | lei zhang proposed openstack/nova master: [Docs] Update the confusing console output https://review.openstack.org/589004 | |
| 03:02:09 | openstackgerrit | lei zhang proposed openstack/nova master: [Docs] Update the compute node installation steps https://review.openstack.org/589008 | |
| 04:15:31 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Don't filter out sibling sets with one core https://review.openstack.org/588570 | |
| 04:15:32 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Ensure emulator threads are always calculated https://review.openstack.org/588571 | |
| 04:15:33 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Always pass 'NUMACell.siblings' to _pack_instance_onto_cores' https://review.openstack.org/588572 | |
| 04:15:34 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: trivialfix: cleanup _pack_instance_onto_cores() https://review.openstack.org/588573 | |
| 04:15:35 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Add unit tests for EmulatorThreadsTestCase https://review.openstack.org/588574 | |
| 04:15:36 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/queens: Not use thread alloc policy for emulator thread https://review.openstack.org/588575 | |
| 07:08:29 | openstackgerrit | Chen proposed openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028 | |
| 07:15:37 | quiquell | Good morning | |
| 07:15:45 | quiquell | We have found this http://logs.openstack.org/95/583195/18/check/tripleo-ci-centos-7-containers-multinode/769edac/logs/undercloud/var/log/containers/nova/nova-scheduler.log.txt.gz | |
| 07:15:51 | quiquell | What it means ? | |
| 07:16:04 | quiquell | 2018-08-03 22:57:32.138 26 ERROR nova.scheduler.client.report [req-e207e96f-2e1c-4648-afc7-be5c51d396ef 0dd904c9905d4ab7962c306a23e8e4ff 6825b026e19a450283f76bcc85f30a43 - default default] Failed to retrieve allocation candidates from placement API for filters: RequestGroup(use_same_provider=False, resources={CUSTOM_BAREMETAL:1}, traits=[], aggregates=[]) | |
| 07:16:06 | quiquell | Got 400: {"errors": [{"status": 400, "request_id": "req-8b3d0e7f-0238-4290-99f9-4a3df8213231", "code": "placement.undefined_code", "detail": "The server could not comply with the request since it is either malformed or otherwise incorrect.\n\n Invalid resource class in resources parameter: No such resource class CUSTOM_BAREMETAL. ", "title": "Bad Request"}]}. | |
| 07:30:03 | alex_xu | quiquell: 'CUSTOM_BAREMETAL' is custom trait, you must create it by the placement api before you refer it. In the ironic case, nova-manager will create that custom trait for ironic virt driver. | |
| 07:30:29 | alex_xu | quiquell: check the nova-manager log, sounds like nova-manager doesn't startup correctly http://logs.openstack.org/95/583195/18/check/tripleo-ci-centos-7-containers-multinode/769edac/logs/undercloud/var/log/containers/nova/nova-compute.log.txt.gz#_2018-08-03_23_06_47_764 | |
| 07:30:41 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Define irrelevant-files for tempest-full-py3 job https://review.openstack.org/589039 | |
| 07:30:46 | alex_xu | quiquell: also zero node return by the ironic driver http://logs.openstack.org/95/583195/18/check/tripleo-ci-centos-7-containers-multinode/769edac/logs/undercloud/var/log/containers/nova/nova-compute.log.txt.gz#_2018-08-03_23_06_47_882 | |
| 07:31:17 | alex_xu | quiquell: sorry, s/nova-maanger/nova-compute/ | |
| 07:31:49 | alex_xu | quiquell: that means nova-compute doesn't reach to the code to create that CUSTOM_BARETAL for the ironic virt driver | |
| 07:32:17 | alex_xu | quiquell: also sorry, CUSTOM_BARETAL is custom resource class, it is not custom trait....really typo too much wrong words... | |
| 07:32:33 | quiquell | alex_xu: Do we have any log about virt driver to check why it returns 0 ? | |
| 07:34:48 | alex_xu | quiquell: i guess there isn't too much, probably needs to check ironic log to see whether ironic works correctly | |
| 07:43:29 | openstack | Launchpad bug 1745405 in OpenStack Compute (nova) "tempest-full job triggered for irrelevant changes" [Undecided,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 07:43:29 | gmann | giblet: should we mark these bug fixed/invalid now (fixed by zuul) - https://bugs.launchpad.net/nova/+bug/1745405 https://bugs.launchpad.net/nova/+bug/1745431 | |
| 07:43:30 | openstack | Launchpad bug 1745431 in OpenStack Compute (nova) "neutron-grenade job is triggered for irrelevant changes " [Undecided,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 07:59:44 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add tempest-slow job to run the tempest slow tests https://review.openstack.org/567697 | |
| 09:18:51 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when resizing https://review.openstack.org/589085 | |
| 09:40:43 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Cleanup comp_node, res_prov, services, aggregate_hosts during cell deletion https://review.openstack.org/546660 | |
| 10:06:44 | cdent | gibi: another good thing to get in: https://review.openstack.org/#/c/587772/ | |
| 10:10:51 | gibi | gmann: marked both bug invalid in nova. Thanks for the notice | |
| 10:10:55 | gibi | cdent: looking... | |
| 10:18:59 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | |
| 10:19:00 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | |
| 10:19:01 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | |
| 10:19:05 | lyarwood | dansmith: morning, nothing urgent but could you take a look at ^ specifically a question raised by mriedem on a previous patchset - https://review.openstack.org/#/c/587071/4/nova/compute/manager.py | |
| 10:19:33 | gibi | cdent: +2 | |