| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-01 | |||
| 19:46:28 | melwitt | I got confused when I started thinking about passthrough of microversions when using osc | |
| 19:46:41 | melwitt | and it made me think of novaclient and the --os-compute-api-version thing | |
| 19:47:00 | mriedem | yeah, it would be the same if there was a python-placementclient thing | |
| 19:47:11 | mriedem | but we've intentionally always avoided that | |
| 19:47:15 | melwitt | ok yeah, I see now 'openstack resource class list -h' | |
| 19:47:29 | mriedem | b/c we have osc for cli and openstacksdk for sdk | |
| 19:48:05 | melwitt | "Return a list of all resource classes. This command requires at least --os- | |
| 19:48:06 | melwitt | placement-api-version 1.2." | |
| 19:48:28 | melwitt | thanks | |
| 20:18:16 | openstackgerrit | Merged openstack/nova master: Nix unused raise_if_custom_resource_class_pre_v1_1 https://review.openstack.org/575847 | |
| 20:42:32 | openstack | bug 1784705 in OpenStack Compute (nova) queens "ResourceTracker.stats can leak across multiple ironic nodes" [High,In progress] https://launchpad.net/bugs/1784705 - Assigned to Matt Riedemann (mriedem) | |
| 20:42:32 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Add recreate test for RT.stats bug 1784705 https://review.openstack.org/588076 | |
| 20:42:33 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Make ResourceTracker.stats node-specific https://review.openstack.org/588077 | |
| 21:03:42 | mriedem | ooo goody 5 new bugs in triage | |
| 21:04:48 | melwitt | nooo | |
| 21:04:57 | mriedem | well one is against mitaka so.... | |
| 21:05:49 | melwitt | yeah, just opened that one | |
| 21:06:21 | melwitt | I've noticed a fair bit of bugs being added to nova that used to be on other projects lately | |
| 21:06:45 | melwitt | or something. really old bugs coming back as new | |
| 21:15:59 | melwitt | mnaser: have you ever used evacuate with ceph-backed instances? looking at this bug and wondering if evacuate doesn't work with ceph period or if it's only if you've enabled the ceph "exclusive-lock" feature on a disk https://bugs.launchpad.net/nova/+bug/1781878 | |
| 21:16:00 | openstack | Launchpad bug 1781878 in OpenStack Compute (nova) "VM fails to boot after evacuation when it uses ceph disk" [Undecided,New] | |
| 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, | |