| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-26 | |||
| 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 | |
| 14:45:26 | smcginnis | melwitt: FWIW, we had to think it through again and remember exactly how all of that works, so it definitely is not at all obvious. | |
| 14:45:48 | melwitt | smcginnis: heh, thanks for the consolation :) | |
| 14:45:55 | smcginnis | ;) | |
| 14:56:51 | mriedem | TheJulia: jroll: https://review.openstack.org/548042 | |
| 15:03:01 | openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | |
| 15:09:38 | mriedem | jroll: updated that with Nova team | |
| 15:09:40 | mriedem | opps | |
| 15:09:41 | mriedem | the ML | |
| 15:10:34 | jroll | nice, thanks matt | |
| 15:32:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Clarify wording in listing instance actions for deleted instances https://review.openstack.org/548055 | |
| 15:42:21 | openstackgerrit | Piotr Bielak proposed openstack/nova-specs master: Specs for improved Nova scheduler logger. https://review.openstack.org/548056 | |
| 15:44:33 | openstackgerrit | Piotr Bielak proposed openstack/nova-specs master: Specs for improved Nova scheduler logger. https://review.openstack.org/548056 | |
| 15:45:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Update noVNC deployment docs to mention non-US keymap fix in 1.0.0 https://review.openstack.org/548057 | |
| 15:45:59 | openstackgerrit | Claudiu Belu proposed openstack/nova master: compute: Adds instance live-resize https://review.openstack.org/248581 | |
| 15:45:59 | openstackgerrit | Claudiu Belu proposed openstack/nova master: conductor: add live_resize task https://review.openstack.org/248579 | |
| 15:46:00 | openstackgerrit | Claudiu Belu proposed openstack/nova master: API: Adds instance live-resize https://review.openstack.org/184584 | |
| 16:30:12 | mriedem | dansmith: http://162.253.55.2/compute/v2.1 yeah? | |
| 16:30:22 | mriedem | so what are we saying? http://162.253.55.2/compute/v2.1/healthcheck? | |
| 16:30:32 | dansmith | mriedem: no, /compute/healthcheck I think | |
| 16:30:40 | mriedem | so http://162.253.55.2/compute/healthcheck/v2.1 | |
| 16:30:44 | dansmith | I think it _has_ to be before the versioned space | |
| 16:30:51 | dansmith | no /v2.1 I think | |
| 16:30:56 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813 | |
| 16:32:19 | efried | This should be a mordred question. I think he has strong opinions on whether the 'v2.1' should be present in the URI at all, ever (even for discovry). | |
| 17:19:10 | openstackgerrit | Merged openstack/nova-specs master: Update Provider Tree https://review.openstack.org/540111 | |
| 17:35:39 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/519636 | |
| 17:59:18 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Block swap volume attempts with encrypted volumes https://review.openstack.org/543569 | |