| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-02 | |||
| 10:09:36 | openstackgerrit | gaobin proposed openstack/nova master: Improve the properties of the api https://review.openstack.org/614730 | |
| 10:20:44 | openstackgerrit | Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407 | |
| 10:25:23 | openstack | Launchpad bug 1789423 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset is_public field" [Medium,Triaged] - Assigned to Mohammed Naser (mnaser) | |
| 10:25:23 | mnaser | hey -- trying to fix https://bugs.launchpad.net/nova/+bug/1789423 locally, got it working but i need to get my pep8 to pass.. | |
| 10:25:27 | mnaser | http://paste.openstack.org/show/733952/ | |
| 10:25:36 | mnaser | any ideas how to ideally clean that up to fit 80 chars? | |
| 11:23:12 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Default embedded instance.flavor.is_public attribute https://review.openstack.org/615134 | |
| 11:24:59 | mnaser | ^ if someone can help with that briefly | |
| 11:42:43 | jroll | mnaser: I'd do Flavor(\n and indent the rest by whatever looks most readable | |
| 11:43:39 | jroll | mnaser: probably this if it fits, I didn't count: http://paste.openstack.org/show/734030/ | |
| 11:44:18 | jroll | or maybe http://paste.openstack.org/show/734031/ | |
| 11:44:50 | jroll | of course you could always assign those to a variable above where you define fake_flavor, too | |
| 11:45:59 | openstackgerrit | huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833 | |
| 11:50:09 | openstackgerrit | Jose Castro Leon proposed openstack/nova master: Fixes bug on attaching volume scsi on an instance boot from iso https://review.openstack.org/615141 | |
| 12:11:31 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | |
| 12:13:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | |
| 12:14:12 | openstackgerrit | huanhongda proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/611833 | |
| 12:23:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | |
| 12:32:21 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786 | |
| 12:34:52 | openstackgerrit | Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787 | |
| 12:39:41 | johnthetubaguy | jaypipes: melwitt: suddenly noticed we can do per-project instance count checks from the CellMapping table. I worked this out before and keep forgetting about it. | |
| 12:46:08 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180 | |
| 12:49:55 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Default embedded instance.flavor.is_public attribute https://review.openstack.org/615134 | |
| 12:50:07 | mnaser | thanks for the input jroll ! % | |
| 12:50:15 | jroll | :) | |
| 12:56:06 | jaypipes | johnthetubaguy: yeah, that's what melwitt added the queue_for_delete column I think in the API DB's instance_mappings table, right? | |
| 13:06:21 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180 | |
| 13:34:01 | sri_ | Hello folks, quick quastion Does Ceph monitor IPs are hardcoded in the database <block_device_mapping> ? did anybody tried replacing ceph monitors(IP) recently ? | |
| 13:39:22 | johnthetubaguy | leakypipes: yeah, mind like a sieve | |
| 13:40:04 | mriedem | sri_: you might be looking for https://review.openstack.org/#/c/579004/ | |
| 13:40:23 | openstack | Launchpad bug 1452641 in nova (Ubuntu) "Static Ceph mon IP addresses in connection_info can prevent VM startup" [Medium,In progress] - Assigned to Corey Bryant (corey.bryant) | |
| 13:40:23 | mriedem | https://bugs.launchpad.net/nova/+bug/1452641 | |
| 13:41:36 | openstackgerrit | Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380 | |
| 13:54:03 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328 | |
| 13:55:53 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | |
| 13:59:36 | openstackgerrit | Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787 | |
| 13:59:36 | openstackgerrit | Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786 | |
| 14:00:10 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add local_root to block_device_info https://review.openstack.org/529029 | |
| 14:00:10 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | |
| 14:00:11 | openstackgerrit | Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380 | |
| 14:00:11 | openstackgerrit | Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363 | |
| 14:07:52 | mnaser | is there any spec that proposes merging image backend + image cache (the thing that i kept talking about but never did?) | |
| 14:08:28 | fried_rice | tssurya, mriedem, leakypipes: Was the CERN workaround for refresh interval based on setting CONF.compute.resource_provider_association_refresh to a high value, or was there some component of out-of-tree hackage as well? | |
| 14:09:18 | leakypipes | fried_rice: UIAM, they disabled it entirely. | |
| 14:09:31 | fried_rice | leakypipes: By hacking code? | |
| 14:09:42 | dansmith | yup | |
| 14:10:05 | fried_rice | okay. I would like to know exactly what that patch looked like at some point soon if possible. | |
| 14:10:23 | mnaser | https://github.com/openstack/nova/blob/a0563e754ce5228df73252ea9e18821781c06d80/nova/scheduler/client/report.py#L843-L853 | |
| 14:10:23 | mnaser | im gonna assume | |
| 14:10:24 | mnaser | return True | |
| 14:10:28 | mnaser | :p | |
| 14:12:37 | fried_rice | It would be `return False`. That's probably it, but it's going to make a difference if it was something subtly different. | |
| 14:13:03 | openstack | Launchpad bug 1800472 in OpenStack Compute (nova) "nova.tests.functional.test_server_group.ServerGroupTestV264.test_boot_servers_with_affinity_no_valid_host intermittently failing with "OpenStackApiNotFoundException: Item not found"" [Medium,Triaged] | |
| 14:13:03 | mriedem | dansmith: so i think i have a lead on https://bugs.launchpad.net/nova/+bug/1800472 | |
| 14:13:28 | mriedem | it appears that we're getting the 404 because we're looking up the nova-api service version when fetching the instance and thinking it's version is 0 | |
| 14:13:44 | mriedem | so it goes down the old pre-cellsv2 path and tries to get the instance out of the local-to-api db | |
| 14:13:50 | mriedem | which with the cells fixture should still default to cell1 | |
| 14:13:59 | mriedem | but the instance is buried in cell0 which is why the lookup would fail | |
| 14:14:06 | SteelyDan | orly | |
| 14:14:18 | SteelyDan | surely we can remove the old paths now yeah? | |
| 14:16:04 | mriedem | i think so, | |
| 14:16:26 | mriedem | it's also weird since the api fixture does create a service record, and that should be in the default cell1 db | |
| 14:16:34 | mriedem | so i'm not sure why we're failing to find it, or with the correct version | |
| 14:22:44 | SteelyDan | ack | |
| 14:23:40 | sean-k-mooney | fried_rice: did my reivew of https://review.openstack.org/#/c/614886/ make sense | |
| 14:24:06 | fried_rice | sean-k-mooney: Hadn't gotten to it yet. Thank you for looking at it. | |
| 14:26:21 | sean-k-mooney | no worries. tl;dr ya i like the direction some nits inline but nothing major | |
| 14:27:01 | sean-k-mooney | fried_rice: we proably can do more then you do in the patch but it a solid first step if we decide to go that way | |
| 14:27:31 | fried_rice | sean-k-mooney: The random jitter thing is a fine idea, but would be a totally independent piece of work. (I will respond accordingly.) | |
| 14:28:20 | sean-k-mooney | fried_rice: ya it should not be in this patch and hopefully its something we can get for free form oslo with the right settings | |
| 14:28:47 | sean-k-mooney | if not it proably should be added to oslo first | |
| 14:29:27 | fried_rice | sean-k-mooney: As for changing the default, I doubt that's going to fly. For a change of this magnitude, we'll probably want to let it sit in experimental-optional state (where e.g. CERN and mnaser can try it out) for a while, and then consider changing the default in a separate patch, maybe in a future release. | |
| 14:30:24 | sean-k-mooney | fried_rice: ya i was debating that too. definetly a different patch | |
| 14:30:42 | sean-k-mooney | making it a train+ thing i could also see | |
| 14:31:12 | sean-k-mooney | did we actully agree on Train as the T release name by the way | |
| 14:32:52 | fried_rice | sean-k-mooney: The poll is still open afaik | |
| 14:32:56 | fried_rice | til next week | |
| 14:42:46 | sri_ | mriedem, yes :), I was also looking for proceees to update mon ip's in older versions of openstack, we have one of the stack running in Liberty | |
| 14:45:45 | sri_ | mriedem, thank you for your help :) | |
| 14:47:02 | mriedem | yw | |
| 14:50:00 | mriedem | SteelyDan: great, so removing that service version check is easy peasy. what's not is that a buttload of tests for the api rely on that code since they stub out an instance in the db w/o a related mapping, and we require the mapping now to lookup the instance. gonna be fun hunting those tests down. | |
| 14:50:12 | SteelyDan | hah nice | |
| 14:50:53 | jangutter | sean-k-mooney: the wiki has a very special icon next to the proposal. | |
| 14:51:04 | mriedem | ugh, rest api controller tests stubbing out the db api | |
| 14:51:05 | mriedem | fml | |
| 14:53:15 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Unified limits ideas https://review.openstack.org/615180 | |
| 14:54:40 | jroll | SteelyDan: I'm having trouble coming up with a solution for my comment here, other than managing it in the database or something like etcd. mind taking a look to see if you can come up with anything? https://review.openstack.org/#/c/609709/3/specs/stein/approved/ironic-conductor-groups.rst@163 | |
| 14:55:53 | SteelyDan | jroll: why do you need to differentiate those two? | |
| 14:56:16 | SteelyDan | does default group imply it will take over everything? or will it look for a default group that ironic has nothing in since it's expected to be partitioned? | |
| 14:56:52 | jroll | SteelyDan: the latter | |
| 14:57:11 | SteelyDan | so what's the problem then? | |
| 14:57:42 | SteelyDan | if ironic is partitioned and you start up a compute with no config, it will look for the default group, find it empty and report no nodes (if I'm reading you correctly) | |
| 14:58:23 | jroll | SteelyDan: oh, I'm thinking of some nodes are partitioned and some are in the default group | |
| 14:58:40 | jroll | so we need to differentiate between "manage all nodes" and "manage only the default group" | |
| 14:58:52 | SteelyDan | why aren't those the same? meaning, | |
| 14:59:06 | SteelyDan | why isn't the default group inclusive of all nodes in ironic by default, and empty if you want strict partitioning? | |
| 14:59:40 | SteelyDan | and if you just want one group or something, then any node not configured would manage the default group (i.e. the rest) | |
| 14:59:49 | SteelyDan | I mean, I'm saying words without knowing how this works on the ironic side of course | |
| 15:00:22 | SteelyDan | even still, two config options could solve this right? manage_a_group=True|False, group_i_manage=[] | |
| 15:00:51 | jroll | right now you could have some "ungrouped" (default group) and some grouped | |