| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-01 | |||
| 21:16:55 | melwitt | the way the bug is worded, I'm not certain whether they're saying evacuate is broken with ceph out-of-the-box | |
| 21:17:09 | mnaser | i am almost certain this is a manifestation of another bug | |
| 21:17:13 | melwitt | so I was wondering if you've ever done evacuate successfully | |
| 21:17:14 | mnaser | of a poorly upgraded ceph cluster | |
| 21:17:15 | mnaser | one second | |
| 21:17:19 | melwitt | k | |
| 21:17:20 | mnaser | yes, evacuate works all the time, but one second | |
| 21:18:00 | openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 21:18:08 | mnaser | melwitt: http://lists.ceph.com/pipermail/ceph-users-ceph.com/2017-September/020722.html | |
| 21:18:10 | mnaser | alongside doc there | |
| 21:18:39 | melwitt | thanks | |
| 21:18:42 | mnaser | saaame exact behaviour afaik as original reported issue | |
| 21:19:36 | melwitt | oh, this is familiar ... I remember an internal bug we had about this | |
| 21:19:55 | mnaser | a lot of people were affected by this because the step to tell you to do this was forgotten in the original ceph upgrade docs | |
| 21:20:23 | melwitt | a-ha | |
| 21:21:24 | melwitt | going to try to find that internal bug, the conclusion was to add something to a config? to allow the blacklist command | |
| 21:21:48 | melwitt | and I had first seen the idea on an upstream cinder bug | |
| 21:23:52 | mnaser | melwitt: http://docs.ceph.com/docs/master/releases/luminous/#upgrade-from-jewel-or-kraken | |
| 21:23:58 | mnaser | specifically #6 | |
| 21:27:15 | melwitt | ty | |
| 21:29:42 | openstack | Launchpad bug 1760065 in kolla-ansible "ceph luminous insufficient nova caps " [Undecided,New] | |
| 21:29:42 | melwitt | https://bugs.launchpad.net/kolla-ansible/+bug/1760065 | |
| 21:31:04 | openstack | Launchpad bug 1773449 in tripleo "VM rbd backed block devices inconsistent after unexpected host outage" [High,Fix released] - Assigned to Giulio Fidente (gfidente) | |
| 21:31:04 | melwitt | I think I found the bug I was thinking of, it wasn't for cinder though https://bugs.launchpad.net/nova/+bug/1773449 | |
| 21:31:49 | melwitt | particularly comment #31 | |
| 21:33:40 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | |
| 21:33:41 | openstackgerrit | Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033 | |
| 21:33:42 | openstackgerrit | Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane https://review.openstack.org/584598 | |
| 21:33:43 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599 | |
| 21:33:44 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648 | |
| 21:33:45 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034 | |
| 21:33:46 | openstackgerrit | Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | |
| 21:33:47 | openstackgerrit | Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 21:35:15 | efried | melwitt: FYI, this series is "finished" (ready for review) ^ | |
| 21:35:38 | melwitt | efried: coolness, thanks for the heads up | |
| 21:35:47 | mriedem | time to slot up stein runways etherpad it looks like | |
| 21:36:19 | melwitt | yeah, I need to make some etherpads. need a retro one too for the PTG. and probably another one I'm forgetting | |
| 21:38:24 | mriedem | stein review priorities? | |
| 21:44:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Rollback failed evacuated guest https://review.openstack.org/588087 | |
| 21:44:16 | mriedem | mdbooth and the red hat gang would probably love to punch holes in ^ | |
| 21:49:30 | sean-k-mooney | mriedem: you -w it so im guessing your not to keen on its current state either | |
| 21:49:46 | mriedem | no, rollback is tricky | |
| 21:49:53 | mriedem | plus see the todo about bdm attachments | |
| 21:50:38 | sean-k-mooney | mriedem: how many states can a vm in a failed evecuate be in by the way. in theor the compute service on teh host you were evacuating from is down so can we even check the current status of the vm | |
| 21:52:44 | mriedem | unless the source came back up and changed the state on the instance in the db which is why the evac on the dest failed in the first place | |
| 21:53:49 | sean-k-mooney | right if this rollback is happening the vm is not running on the dest, may or may not be running on the source, and the source node may or may not be up | |
| 21:55:29 | sean-k-mooney | just reading the commit and bug for context but ya the rollback is non trivial | |
| 22:00:25 | openstack | Launchpad bug 1784782 in OpenStack Compute (nova) "API: flavors - Cannot list all public and private flavors by default" [Undecided,New] | |
| 22:00:25 | mriedem | https://bugs.launchpad.net/nova/+bug/1784782 is news to me that you can pass GET /flavors?is_public=None as admin to list all flavors | |
| 22:02:11 | sean-k-mooney | mriedem: am ya that seams unintentional that is_public=None would have that effect | |
| 22:02:23 | mriedem | it was very intentional when it was added | |
| 22:02:52 | sean-k-mooney | mriedem: im confused is that ment to work or not? | |
| 22:03:08 | mriedem | https://review.openstack.org/#/c/32713/ | |
| 22:03:32 | mriedem | https://review.openstack.org/#/c/27403/ | |
| 22:03:58 | mriedem | our api-ref doesn't mention anything about this, just says is_public must be a boolean | |
| 22:04:04 | sean-k-mooney | huh ok i did not know that | |
| 22:04:09 | mriedem | me neither | |
| 22:04:31 | sean-k-mooney | im guessing there are no tests for this and we changed something recently | |
| 22:05:16 | mriedem | test_list_detail_flavors_with_is_public_filter_as_string_none | |
| 22:05:21 | mriedem | test_list_flavors_with_is_public_filter_as_string_none | |
| 22:05:30 | mriedem | i don't see any functional tests, but there are unit tests | |
| 22:06:22 | sean-k-mooney | also accourding to git blame nothing has changed in that fuction for 5 years | |
| 22:06:38 | sean-k-mooney | https://github.com/openstack/nova/blame/master/nova/api/openstack/compute/flavors.py#L89 | |
| 22:07:54 | sean-k-mooney | i have a devstack deployment i did like 2 hours ago ill see if i can figure out how to test this with curl | |
| 22:08:28 | mriedem | create a private flavor, | |
| 22:08:32 | mriedem | get an admin token | |
| 22:08:41 | mriedem | curl /flavors?is_public='None' | |
| 22:12:19 | sean-k-mooney | that seams to work | |
| 22:12:33 | sean-k-mooney | ill post that to pastbin | |
| 22:12:38 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Remove redundant _update()s https://review.openstack.org/588091 | |
| 22:15:07 | sean-k-mooney | http://paste.openstack.org/show/727100/ | |
| 22:15:32 | sean-k-mooney | i guss i should create some private flavor to see if they are missing one sec | |
| 22:16:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix GET /flavors?is_public description https://review.openstack.org/588092 | |
| 22:19:56 | sean-k-mooney | mriedem: ya it still works http://paste.openstack.org/show/727101/ | |
| 22:20:09 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: only _init_compute_node on startup https://review.openstack.org/588094 | |
| 22:20:20 | sean-k-mooney | ill add that as a comment on the bug | |
| 22:20:39 | mriedem | sean-k-mooney: thanks | |
| 22:21:46 | sean-k-mooney | also if you can figure out how to get onto my network in the next hour there a free admin token for my devstack cloud lol | |
| 22:26:56 | mriedem | i'm already there, | |
| 22:27:07 | mriedem | checking out your kenny loggins fan directory | |
| 22:28:14 | sean-k-mooney | whey would i need a fan dir when youtube has danger zone on a 4 hour loop https://www.youtube.com/watch?v=K7dXpZCEKls&index=5&list=PLG2eb1MxWbfEQRhkJpzq0YeKOTyfpY3Db&t=0s | |
| 22:28:50 | mriedem | ha | |
| 22:29:00 | mriedem | don't forget about caddyshack | |
| 22:30:46 | mriedem | https://www.youtube.com/watch?v=U8SSdyflGN4 has that guy from the doobie brothers in it | |
| 22:30:53 | mriedem | twice the smoothness | |
| 22:30:58 | sean-k-mooney | mriedem: ah yes i have not seen that movie in like 2 decades | |
| 22:31:13 | mriedem | ^ is what i used to get pumped before all summit talks | |
| 22:32:26 | sean-k-mooney | :) | |
| 22:53:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: fix resize user guide link https://review.openstack.org/588097 | |
| #openstack-nova - 2018-08-02 | |||
| 01:39:30 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Use common functions in granular fixture https://review.openstack.org/588113 | |
| 02:01:03 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Use common functions in NonSharedStorageFixture https://review.openstack.org/588114 | |
| 02:30:54 | openstackgerrit | Merged openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636 | |
| 03:13:00 | openstackgerrit | Vishakha Agarwal proposed openstack/nova master: 'Updated_at' is NULL when show aggregate info https://review.openstack.org/580271 | |
| 06:26:11 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Fix a typo in comment in resource_provider.py https://review.openstack.org/588145 | |
| 06:58:19 | openstackgerrit | huanhongda proposed openstack/nova master: Destroy evacuated instance while unset nova-compute forced_down https://review.openstack.org/587807 | |
| 07:14:37 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Increase max_unit in placement test fixture https://review.openstack.org/588158 | |
| 07:14:38 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Refactor AllocationFixture in placement test https://review.openstack.org/588159 | |
| 07:58:39 | jaosorior | gibi: do unversioned notifications still work to date? or are they already being deprecated? | |
| 08:20:04 | jaosorior | or anybody that knows about notifications ^^ | |
| 08:36:55 | gibi | jaosorior: hi! unversioned still works, not even deprecated yet | |