| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-12-14 | |||
| 10:30:02 | mdbooth | + # the subclass constructors have a different signature. | |
| 10:30:02 | mdbooth | + # NOTE(mdbooth): we spec a subclass of imagebackend.Image here because | |
| 10:30:39 | mdbooth | lyarwood: I assume I hit some particular issue to cause me to do that, because my first attempt would almost certainly have been what you did | |
| 10:31:54 | mdbooth | lyarwood: It makes sense, though, because they do have different signatures. | |
| 10:33:11 | lyarwood | mdbooth: hmm yeah but we only use the returned mock as a class remember | |
| 10:33:38 | lyarwood | mdbooth: the side_effect then provides the same signature as the subclass constructor | |
| 10:34:21 | mdbooth | lyarwood: I can't remember why I did this. I just assume there was a reason for it, because it doesn't seem like the first thing I'd have thought of. | |
| 10:34:56 | lyarwood | yeah indeed | |
| 10:35:50 | mdbooth | lyarwood: Anyway, assuming all *current* tests pass I'm happy to put my +1 on your patch as it. Future problems can be addressed if they arise. | |
| 10:36:04 | lyarwood | mdbooth: ack thanks | |
| 10:44:20 | lyarwood | https://review.openstack.org/#/q/topic:bug/1785827+(status:open+OR+status:merged) - if any stable cores are online this series is almost finished, just the Pike changes to +W | |
| 10:58:58 | egonzalez | hi, did something get merged in the last few hours (maybe yesterday evening UTC) which can cause the following error? | |
| 10:59:28 | egonzalez | http://paste.openstack.org/show/737292/ | |
| 10:59:49 | egonzalez | Error gathering result from cell 00000000-0000-0000-0000-000000000000: ProgrammingError: (pymysql.err.ProgrammingError) (1146, u"Table 'nova_cell0.services' doesn't exist") | |
| 11:02:50 | egonzalez | added other log at http://paste.openstack.org/show/737293/ while executing nova-api command | |
| 11:14:43 | mdbooth | lyarwood: I don't understand the revert_resize problem, btw, and without the instance.host check the workaround executes on _hard_reboot | |
| 11:15:39 | mdbooth | lyarwood: So the revert_resize thing is that we execute cleanup on the dest, right? | |
| 11:15:57 | lyarwood | mdbooth: argh yeah we'd lose console.log with hard_reboot, the directory is recreated however. | |
| 11:16:21 | lyarwood | mdbooth: yes, revert resize cleanup is on the dest | |
| 11:16:28 | mdbooth | console.log, kernel, and ramdisk, but yeah | |
| 11:16:41 | lyarwood | mdbooth: it should recreate kernel and ramdisk no? | |
| 11:16:54 | mdbooth | lyarwood: Unless they've been deleted in glance. | |
| 11:17:00 | lyarwood | true | |
| 11:17:53 | mdbooth | lyarwood: So, is the revert_resize thing a different bug? | |
| 11:18:12 | mdbooth | Because your workaround wouldn't trigger on revert_resize. | |
| 11:18:27 | lyarwood | mdbooth: with the host check removed it should | |
| 11:18:41 | lyarwood | mdbooth: we call cleanup with destroy_disks=False and no migrate_data | |
| 11:18:43 | mdbooth | lyarwood: Right, but it didn't before. | |
| 11:19:07 | mdbooth | So is that a separate latent bug? | |
| 11:19:08 | lyarwood | mdbooth: yeah, ah crap I didn't add the bug to the commit did I | |
| 11:19:25 | lyarwood | mdbooth: yes, it's listed in the conf change | |
| 11:19:43 | lyarwood | mdbooth: I'll respin the commit message now | |
| 11:21:52 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216 | |
| 11:22:15 | kashyap | mdbooth: lyarwood: ^ I surely went overboard with the long commit message (that I spent 2 hours writing) | |
| 11:22:41 | kashyap | But I made sure they're in readable chunks. | |
| 11:24:04 | openstackgerrit | zhangyujun proposed openstack/nova master: Should not raise while restore power on failed https://review.openstack.org/624854 | |
| 11:24:06 | kashyap | Oh, it is WIP! | |
| 11:26:11 | lyarwood | mdbooth: kernel, ramdisk etc are set against the flavor right? | |
| 11:27:28 | lyarwood | ah, image props, my bad | |
| 11:27:41 | mdbooth | lyarwood: Right. I think they can also be specified on the command line. | |
| 11:28:13 | mdbooth | I don't think anybody actually uses them, but we probably shouldn't assume that, especially not for a backport. | |
| 11:28:26 | mdbooth | s/command line/api/ | |
| 11:32:48 | lyarwood | mdbooth: yeah, adding a note about it now. | |
| 11:33:04 | mdbooth | lyarwood: How about... changing revert_resize? | |
| 11:34:04 | lyarwood | mdbooth: to pass migrate_data? | |
| 11:34:06 | mdbooth | lyarwood: Put the migration check back in, then turn the cleanup in revert_resize into a migration cleanup by moving it after revert_migration_context() | |
| 11:34:50 | lyarwood | mdbooth: iirc the migration data in resize doesn't have is_shared_block_storage set | |
| 11:34:55 | mdbooth | Then it doesn't execute during _hard_reboot() so we don't need to think ourselves in knots about the potential implications of that. | |
| 11:34:56 | lyarwood | mdbooth: that's only used in the live migration flows | |
| 11:35:42 | mdbooth | lyarwood: If you move the destroy() call after revert_migration_context(), your workaround would execute for revert_resize() | |
| 11:35:46 | mdbooth | Which is what you want. | |
| 11:37:19 | lyarwood | mdbooth: you'd also need to move terminate_connection then as well | |
| 11:37:41 | lyarwood | that opens up another can of worms tbh | |
| 11:38:53 | mdbooth | I drew a diagram of Nova's cleanup workflows: http://www.cartoonaday.com/images/cartoons/2017/05/rube-goldberg-machine-cartoon-A.jpg | |
| 11:39:11 | lyarwood | lol | |
| 11:39:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | |
| 11:40:09 | lyarwood | -2 - I can't see any gaffer tape in that diagram. | |
| 11:40:27 | mdbooth | Hehe. That's v2 | |
| 11:41:36 | lyarwood | but seriously for this workaround I'm not going to move things around in the compute layer, the whole point is that this is an opt-in self contained backportable hack for operators. | |
| 11:46:10 | lyarwood | kashyap: ack btw, looks pretty straight forward, happy to help write the missing tests and land this while you're out if it would help. | |
| 11:48:15 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473 | |
| 11:48:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | |
| 11:54:21 | lyarwood | kashyap: and by straight forward I was just talking about the openstack/nova change, not the actions required to actually use this (ie changes in TripleO). Nice work with the commit and documentation in general for this. | |
| 12:09:58 | kashyap | lyarwood: Unless there's a big rush to land this, I'd just like to get back and write the tests, as a practise. | |
| 12:10:08 | kashyap | And, thanks! | |
| 13:04:58 | gibi | tssurya: the DownCellFixtur patch looks good to me | |
| 13:05:05 | tssurya | gibi: thanks :) | |
| 14:33:23 | mdbooth | Do we have a shelve/unshelve test on rbd currently? | |
| 14:37:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527 | |
| 14:37:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | |
| 14:37:02 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528 | |
| 14:37:02 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239 | |
| 14:37:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529 | |
| 14:37:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240 | |
| 14:37:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 14:37:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459 | |
| 14:37:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject networks with QoS policy https://review.openstack.org/570079 | |
| 14:37:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port https://review.openstack.org/570078 | |
| 14:37:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor PortResourceRequestBasedSchedulingTestBase https://review.openstack.org/624080 | |
| 14:37:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421 | |
| 14:37:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that allocated PF matches the used PF https://review.openstack.org/623543 | |
| 14:51:09 | mriedem | mdbooth: we have a ceph job which runs tempest tests, | |
| 14:51:13 | mriedem | and tempest has shelve/unshelve tests, | |
| 14:51:16 | mriedem | so yes we should | |
| 14:52:01 | mriedem | the ceph job has been broken since the summit though | |
| 14:52:29 | mriedem | oh maybe that was fixed and just encrypted volumes are broken | |
| 14:53:12 | mriedem | https://github.com/openstack/devstack-plugin-ceph/commit/7728027cdd575f7d43c97b4b7a9345c69898c26f | |
| 14:53:12 | mriedem | ah, so, the encrypted volume tests were just recently enabled in the ceph job | |
| 15:00:03 | mdbooth | mriedem: Just about to join a meeting, but I have a suspicion it might be broken if we're using ceph clone for glance images | |
| 15:00:28 | mdbooth | Because when you go to delete the image on unshelve it fails because the backing image is still in use. | |
| 15:02:48 | jangutter | mriedem: thanks very much for your review on https://review.openstack.org/#/c/607610/ ... at the risk of really messing up your Friday, would you have a few minutes to chat about things no-one wants to chat about? | |
| 15:11:58 | mriedem | mdbooth: like this? https://review.openstack.org/#/c/473897/ | |
| 15:12:44 | openstack | Launchpad bug 1697935 in devstack-plugin-ceph "re-enable compute shelve tests" [Undecided,In progress] - Assigned to Eric Harney (eharney) | |
| 15:12:44 | mriedem | https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1697935 | |
| 15:12:50 | mriedem | mdbooth: probably want to talk to eharney | |
| 15:12:58 | mriedem | jangutter: sure | |
| 15:13:56 | jangutter | mriedem: thanks! I am trying to make the "upgrade" sequence less vague. | |
| 15:14:23 | jangutter | mriedem: os-vif's two bits, the plug-in and the object model, has Nova between them. | |
| 15:15:08 | jangutter | mriedem: so the only sane way I see to adopt the interfaced added in the object model, would be to do three updates, or two with compat code in the middle that's hard to test. | |
| 15:15:32 | jangutter | mriedem: os-vif object model -> nova -> os-vif plugin | |