| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-26 | |||
| 10:56:19 | mriedem | stephenfin: ^ | |
| 10:56:26 | mriedem | that came up in the public WG room today | |
| 10:57:16 | stephenfin | mriedem: I assume that is not tracked by global requirements? | |
| 10:57:24 | dayou | melwitt, thanks for you input and help, it is very useful | |
| 10:58:49 | mriedem | stephenfin: since it's a native package, no it's not | |
| 10:59:45 | stephenfin | mriedem: Fair. +2d | |
| 10:59:57 | mriedem | thanks | |
| 11:00:25 | mriedem | stephenfin: the notes from the denver ptg said that once the fix was released in novnc, we could move forward with your patch on deprecating the keymap config option | |
| 11:00:30 | mriedem | https://review.openstack.org/#/c/483994/ | |
| 11:00:35 | mriedem | so maybe you want to refresh that | |
| 11:00:58 | stephenfin | Will do | |
| 11:02:18 | vivsoni | Hi Matt | |
| 11:04:56 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for Granular API policy https://review.openstack.org/547850 | |
| 11:05:13 | gmann_ | mriedem: alex_xu johnthetubaguy added spec for granular API policy - https://review.openstack.org/#/c/547850/ | |
| 11:06:17 | stephenfin | mriedem: If you haven't moved on too far, did OVH folks mention anything about the issue Tushar reported in that review? | |
| 11:06:27 | stephenfin | https://review.openstack.org/#/c/483994/6 | |
| 11:06:40 | stephenfin | I can fix it, but if they have it fixed already... | |
| 11:09:44 | mriedem | stephenfin: nope | |
| 11:10:36 | stephenfin | Lovely. Time to stack | |
| 11:12:48 | openstackgerrit | Merged openstack/nova master: Drop compute RPC 4.x compatibility https://review.openstack.org/543580 | |
| 11:18:48 | openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728 | |
| 11:18:48 | openstackgerrit | Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729 | |
| 11:18:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730 | |
| 11:18:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002 | |
| 11:18:50 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | |
| 11:18:50 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990 | |
| 11:23:35 | vivsoni | mriedem: Hi | |
| 11:23:58 | mriedem | vivsoni: hi? | |
| 11:24:24 | vivsoni | one of my tempest Test case is failing | |
| 11:24:28 | vivsoni | http://54.201.44.218/45/539145/2/check/3par-iscsi-driver-master-client-pip-ssa02-dsvm/e43ca1d/logs/testr_results.html.gz | |
| 11:24:41 | vivsoni | mriedem: this is something to do with force dettach | |
| 11:25:07 | vivsoni | mriedem: connector is empty and driver is not implemented that case | |
| 11:25:13 | vivsoni | can you please guide me | |
| 11:25:29 | dansmith | edleafe: last week or something you said you might be interested in doing the member_of thing for allocation_candidates.. are you interested enough to commit there or did you want to look at what is required first? | |
| 11:25:30 | mriedem | vivsoni: no idea, i'm in sessions at the ptg right now | |
| 11:25:49 | vivsoni | mriedem: ok | |
| 11:26:29 | melwitt | dayou: based on the ordering of the phases in stack.sh, it looks like you could use the 'test-config' phase in your plugin.sh to run the steps in your local.sh. see this example: https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/plugin.sh#L95 | |
| 11:26:49 | mriedem | vivsoni: i guess you're right | |
| 11:26:49 | mriedem | Feb 26 09:30:53.005199 d-p-c-p77-1659969 devstack@c-api.service[30317]: DEBUG cinder.api.openstack.wsgi [None req-f882bcca-b050-4909-afa9-95698a62c1c7 tempest-VolumesActionsTest-1767615534 tempest-VolumesActionsTest-1767615534] Action body: {"os-force_detach": {"attachment_id": "d8020aad-e1a5-4a24-8a93-764e2086100d", "connector": null}} {{(pid=30322) get_method /opt/stack/new/cinder/cinder/api/openstack/wsgi.py:986}} | |
| 11:26:54 | mriedem | vivsoni: but force detach is purely a cinder API | |
| 11:26:59 | mriedem | it has nothing to do with nova | |
| 11:27:32 | vivsoni | mriedem: ok.. it needs to be handled in cinder | |
| 11:28:14 | smcginnis | vivsoni: The driver needs to fix handling that. | |
| 11:28:47 | vivsoni | smcginnis: can you please guide me in fixing that | |
| 11:30:59 | smcginnis | vivsoni: Not really. If the driver is not passed the connector in the case of force detach, it needs to expect and handle that to detach the volume. if not connector: do_driver_stuff()? | |
| 11:32:37 | vivsoni | smcginnis: ok | |
| 11:33:56 | melwitt | mriedem: this is the queens final releases patch for us, and I was wondering, what is the diff-start entry mean? https://review.openstack.org/#/c/547519/2/deliverables/queens/nova.yaml | |
| 11:34:36 | mriedem | melwitt: that's an excellent question for the release team nerds, like scm | |
| 11:34:41 | mriedem | smcginnis: | |
| 11:34:46 | melwitt | ah, thanks. sorry | |
| 11:34:53 | mriedem | >:( | |
| 11:34:59 | mriedem | np | |
| 11:35:09 | mriedem | melwitt: it goes into the docs i believe | |
| 11:35:15 | smcginnis | melwitt: That's a reno thing. It tells reno where to compile the release note from to have one "queens" release note page rather than breaking out each mileston. | |
| 11:35:15 | smcginnis | e | |
| 11:35:39 | smcginnis | melwitt: No problem, reno can be a confusing thing. ;) | |
| 11:35:52 | mriedem | ah https://docs.openstack.org/releasenotes/nova/queens.html | |
| 11:37:54 | edleafe | dansmith: I don't have any other tasks lined up for placement, so yeah, I'll commit | |
| 12:02:31 | melwitt | smcginnis: on the diff-start thing, why is it the pike rc1 version then? how does that compile the release note for queens? | |
| 12:02:42 | melwitt | https://review.openstack.org/#/c/547519/2/deliverables/queens/nova.yaml | |
| 12:21:58 | openstackgerrit | yuanyue proposed openstack/nova master: Whether to use config drive depends on instance.configdrive https://review.openstack.org/485930 | |
| 12:28:46 | openstackgerrit | yuanyue proposed openstack/nova master: Whether to use config drive depends on instance.configdrive https://review.openstack.org/485930 | |
| 12:31:26 | openstackgerrit | Merged openstack/nova master: Update noVNC deployment docs to mention non-US keymap fix in 1.0.0 https://review.openstack.org/547985 | |
| 13:24:16 | openstackgerrit | Merged openstack/nova master: rp: GET /resource_providers?required= |
|
| 13:31:13 | dansmith | edleafe: okay this is the change I have queued up that will utilize it: https://review.openstack.org/#/c/544729/ | |
| 13:47:43 | openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728 | |
| 13:47:44 | openstackgerrit | Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729 | |
| 13:47:44 | openstackgerrit | Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730 | |
| 13:47:45 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990 | |
| 13:47:45 | openstackgerrit | Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002 | |
| 13:47:46 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | |
| 14:07:53 | jroll | melwitt: this is the pike/ironic one mentioned, if you're curious: https://github.com/openstack/nova/commit/8e77c475e1eb48703314e1b98bc9cb491027064d | |
| 14:08:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make _get_sharing_providers more efficient https://review.openstack.org/547008 | |
| 14:23:31 | melwitt | jroll: thanks | |
| 14:28:18 | hrw | melwitt: are you on ptg? | |
| 14:28:31 | melwitt | hrw: yes | |
| 14:29:07 | hrw | melwitt: would love to meet then. you are one of those few dib cores ;D | |
| 14:30:04 | mriedem | TheJulia: the ironic nova driver entry in the driver log page is out of date https://www.openstack.org/marketplace/drivers/#project=nova%20(compute)&release=all and deva is listed as the contact in the repo - who should be the contact for ironic CI now? | |
| 14:30:06 | jroll | melwitt: and the offline version, if you didn't go looking yet: https://github.com/openstack/nova/commit/8f8982d8ef381729ef12a14f37486335f31c4fd3 | |
| 14:30:47 | melwitt | hrw: dib? we'll have a nova room Wed-Fri so definitely will see you there | |
| 14:31:38 | TheJulia | mriedem: Wow, that is VERY out of date | |
| 14:31:45 | mriedem | TheJulia: http://stackalytics.com/report/driverlog?project_id=openstack%2Fnova is what i was looking at last night | |
| 14:32:17 | jroll | mriedem: I wonder if we can just put "#openstack-ironic" in there | |
| 14:32:20 | mriedem | jroll: melwitt: problems with ironic flavor migration? | |
| 14:32:24 | mriedem | also plug plug plug https://review.openstack.org/#/c/527541/ | |
| 14:32:49 | mriedem | jroll: if that works i'll add it, because mikal is listed for several other community run CI jobs in here, like qemu | |
| 14:32:53 | jroll | mriedem: nah it's good, just talking FFU and this was relevant | |
| 14:33:05 | melwitt | mriedem: discussing the fact that there are a few places where we require bringing up nova-compute during an upgrade to migrate data | |
| 14:33:20 | hrw | melwitt: I could be wrong ;) dib as diskimage-builder | |
| 14:33:23 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add check for redundant import aliases https://review.openstack.org/543995 | |
| 14:33:25 | melwitt | for the ironic flavor migration, it was realized it could be done offline in a batch command so that was added later | |
| 14:33:25 | jroll | mriedem: yeah I'd either put the irc channel or a link to the governance page for ironic (which lists the ptl) | |
| 14:34:28 | TheJulia | +1 We need to centralize information as much as possible, too many different places are going to get lost | |
| 14:34:41 | melwitt | hrw: oh, gotcha. no I'm not core on diskimage-builder | |
| 14:37:08 | hrw | my mistake. | |
| 14:39:48 | smcginnis | melwitt: The diff-start goes from the branching point, so since what became queens was from when we branched at pike RC1, that's used as the starting point for reno to scan for queens changes. | |
| 14:43:31 | melwitt | smcginnis: I see. so what about if other reno changes were added in pike RC2 or pike RC3 etc? those would get picked up for queens? | |
| 14:44:15 | smcginnis | melwitt: No, because those changes would then be in the stable/pike branch and not be present in queens. | |
| 14:44:29 | smcginnis | melwitt: Unless they were merged in master and backported to stable/pike of course. | |
| 14:44:40 | melwitt | smcginnis: okay, thank you | |