| *** tetsuro has joined #openstack-nova | 00:03 | |
| *** wolverineav has quit IRC | 00:09 | |
| *** wolverineav has joined #openstack-nova | 00:09 | |
| *** slaweq has joined #openstack-nova | 00:11 | |
| *** awaugama has quit IRC | 00:12 | |
| *** igordc has joined #openstack-nova | 00:13 | |
| *** wolverineav has quit IRC | 00:14 | |
| *** tbachman has quit IRC | 00:14 | |
| *** slaweq has quit IRC | 00:16 | |
| *** igordc has quit IRC | 00:18 | |
| *** erlon has joined #openstack-nova | 00:33 | |
| *** wolverineav has joined #openstack-nova | 00:34 | |
| *** wolverineav has quit IRC | 00:36 | |
| *** wolverineav has joined #openstack-nova | 00:38 | |
| *** wolverineav has quit IRC | 00:38 | |
| *** wolverineav has joined #openstack-nova | 00:38 | |
| *** rambo_li has joined #openstack-nova | 00:39 | |
| *** cdent has quit IRC | 00:39 | |
| *** brinzhang has joined #openstack-nova | 00:48 | |
| *** Swami has quit IRC | 00:49 | |
| *** gyee has quit IRC | 00:58 | |
| *** jcosmao has quit IRC | 01:04 | |
| openstackgerrit | Merged openstack/nova master: Only construct SchedulerReportClient on first access from API https://review.openstack.org/623246 | 01:06 |
|---|---|---|
| openstackgerrit | Merged openstack/nova master: DRY up SchedulerReportClient init https://review.openstack.org/623247 | 01:06 |
| openstackgerrit | Merged openstack/nova stable/rocky: Note the aggregate allocation ratio restriction in scheduler docs https://review.openstack.org/623546 | 01:06 |
| *** tbachman has joined #openstack-nova | 01:09 | |
| *** slaweq has joined #openstack-nova | 01:10 | |
| *** slaweq has quit IRC | 01:15 | |
| *** takashin has joined #openstack-nova | 01:17 | |
| *** psachin has joined #openstack-nova | 01:30 | |
| *** takashin has quit IRC | 01:35 | |
| *** takashin has joined #openstack-nova | 01:37 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:39 | |
| *** rodolof has quit IRC | 01:40 | |
| *** Dinesh_Bhor has quit IRC | 01:43 | |
| *** macza has quit IRC | 01:44 | |
| *** tiendc has joined #openstack-nova | 01:46 | |
| *** takashin has left #openstack-nova | 02:00 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:01 | |
| *** mrsoul has quit IRC | 02:07 | |
| *** cfriesen has quit IRC | 02:07 | |
| *** bhagyashris has joined #openstack-nova | 02:11 | |
| *** munimeha1 has quit IRC | 02:25 | |
| *** mhen has quit IRC | 02:26 | |
| *** sapd1_ has joined #openstack-nova | 02:27 | |
| *** sapd1 has quit IRC | 02:29 | |
| *** mhen has joined #openstack-nova | 02:30 | |
| *** dave-mccowan has joined #openstack-nova | 02:38 | |
| *** itlinux has joined #openstack-nova | 02:39 | |
| *** naichuans has joined #openstack-nova | 02:50 | |
| openstackgerrit | Dave McCowan proposed openstack/nova stable/rocky: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623507 | 02:54 |
| openstackgerrit | Dave McCowan proposed openstack/nova stable/queens: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623509 | 02:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Don't persist RequestSpec.requested_destination https://review.openstack.org/611945 | 02:57 |
| *** wolverineav has quit IRC | 03:03 | |
| *** wolverineav has joined #openstack-nova | 03:04 | |
| *** wolverineav has quit IRC | 03:08 | |
| *** slaweq has joined #openstack-nova | 03:11 | |
| *** hongbin has joined #openstack-nova | 03:14 | |
| *** hongbin has quit IRC | 03:15 | |
| *** slaweq has quit IRC | 03:15 | |
| *** hongbin has joined #openstack-nova | 03:16 | |
| *** Hazelesque has quit IRC | 03:43 | |
| *** lbragstad has joined #openstack-nova | 03:50 | |
| *** Hazelesque has joined #openstack-nova | 03:51 | |
| *** lbragstad has quit IRC | 03:51 | |
| *** Dinesh_Bhor has quit IRC | 03:52 | |
| *** READ10 has joined #openstack-nova | 03:52 | |
| *** brinzh has joined #openstack-nova | 04:02 | |
| *** dave-mccowan has quit IRC | 04:03 | |
| *** liuyulong has quit IRC | 04:04 | |
| *** brinzhang has quit IRC | 04:06 | |
| *** udesale has joined #openstack-nova | 04:10 | |
| *** psachin has quit IRC | 04:10 | |
| *** slaweq has joined #openstack-nova | 04:11 | |
| *** lbragstad has joined #openstack-nova | 04:13 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:14 | |
| *** rambo_li has quit IRC | 04:14 | |
| *** slaweq has quit IRC | 04:16 | |
| *** psachin has joined #openstack-nova | 04:29 | |
| *** diga has joined #openstack-nova | 04:30 | |
| *** READ10 has quit IRC | 04:30 | |
| *** hongbin has quit IRC | 04:34 | |
| *** _alastor_ has quit IRC | 04:54 | |
| *** rambo_li has joined #openstack-nova | 05:00 | |
| *** bhagyashris has quit IRC | 05:01 | |
| openstackgerrit | Merged openstack/nova stable/queens: Refix disk size during live migration with disk over-commit https://review.openstack.org/602478 | 05:01 |
| *** Dinesh_Bhor has quit IRC | 05:01 | |
| *** _alastor_ has joined #openstack-nova | 05:02 | |
| *** wolverineav has joined #openstack-nova | 05:08 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** slaweq has quit IRC | 05:16 | |
| openstackgerrit | Merged openstack/nova stable/queens: Fix stacktraces with redis caching backend https://review.openstack.org/606896 | 05:16 |
| openstackgerrit | Merged openstack/nova stable/queens: Add regression test for bug 1797580 https://review.openstack.org/611944 | 05:17 |
| openstack | bug 1797580 in OpenStack Compute (nova) queens "NoValidHost during live migration after cold migrating to a specified host" [High,In progress] https://launchpad.net/bugs/1797580 - Assigned to Matt Riedemann (mriedem) | 05:17 |
| openstackgerrit | Merged openstack/nova stable/queens: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/608179 | 05:17 |
| openstackgerrit | Merged openstack/nova stable/queens: Ignore VirtDriverNotReady in _sync_power_states periodic task https://review.openstack.org/607730 | 05:17 |
| *** rambo_li has quit IRC | 05:22 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:23 | |
| *** _alastor_ has quit IRC | 05:34 | |
| *** dklyle has joined #openstack-nova | 05:38 | |
| *** ratailor has joined #openstack-nova | 05:51 | |
| *** wolverineav has quit IRC | 05:51 | |
| *** izza_ has joined #openstack-nova | 06:06 | |
| izza_ | hi...i created instance with VGPU flavor in Openstack TripleO rocky version with Windows Server 2012 R2 guest OS,but as checked, graphics dirver is not visible in the guest OS | 06:07 |
| izza_ | we're using the tesla P100 | 06:07 |
| izza_ | can someone here help us about my issue | 06:07 |
| openstackgerrit | Sujeet Banerjee proposed openstack/nova master: Hot fix for RMQ failures causing nova out of sync https://review.openstack.org/625150 | 06:10 |
| *** rambo_li has joined #openstack-nova | 06:12 | |
| *** lpetrut has joined #openstack-nova | 06:13 | |
| *** izza_ has quit IRC | 06:18 | |
| *** sridharg has joined #openstack-nova | 06:25 | |
| *** bhagyashris_ has joined #openstack-nova | 06:34 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143 | 06:59 |
| *** rcernin has quit IRC | 07:03 | |
| *** slaweq has joined #openstack-nova | 07:11 | |
| *** pcaruana has joined #openstack-nova | 07:12 | |
| *** Luzi has joined #openstack-nova | 07:13 | |
| *** slaweq has quit IRC | 07:16 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143 | 07:20 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142 | 07:20 |
| *** erlon has quit IRC | 07:23 | |
| *** dpawlik has joined #openstack-nova | 07:38 | |
| *** slaweq has joined #openstack-nova | 07:41 | |
| *** slaweq has quit IRC | 07:47 | |
| *** Dinesh_Bhor has quit IRC | 07:48 | |
| *** psachin has quit IRC | 07:48 | |
| *** sum12 has joined #openstack-nova | 07:50 | |
| *** slaweq has joined #openstack-nova | 07:51 | |
| sum12 | Hello, I wanted to understand node discovery for cell_v2, so should a hostmapping be already created when nova-manage discover_hosts is run ? | 07:51 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142 | 07:55 |
| *** lpetrut has quit IRC | 07:58 | |
| *** jangutter has joined #openstack-nova | 08:04 | |
| *** sahid has joined #openstack-nova | 08:07 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:08 | |
| *** rambo_li has quit IRC | 08:12 | |
| *** markvoelker has joined #openstack-nova | 08:16 | |
| *** Shadowphax has joined #openstack-nova | 08:20 | |
| *** imacdonn has quit IRC | 08:24 | |
| *** imacdonn has joined #openstack-nova | 08:24 | |
| *** tssurya has joined #openstack-nova | 08:32 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:33 | |
| *** rambo_li has joined #openstack-nova | 08:34 | |
| *** tetsuro has quit IRC | 08:37 | |
| *** rambo_li_ has joined #openstack-nova | 08:37 | |
| *** rambo_li_ has quit IRC | 08:38 | |
| *** priteau has joined #openstack-nova | 08:39 | |
| *** cdent has joined #openstack-nova | 08:50 | |
| *** brinzh has quit IRC | 09:06 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add DownCellFixture https://review.openstack.org/614810 | 09:10 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657 | 09:10 |
| *** erlon has joined #openstack-nova | 09:13 | |
| *** k_mouza has joined #openstack-nova | 09:13 | |
| tssurya | gibi: good morning! Whenever you have the time could you have a look at https://review.openstack.org/#/c/614810 (after the CI passes)? I have added more comments, thanks! | 09:16 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623507 | 09:20 |
| *** ccamacho has joined #openstack-nova | 09:20 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623509 | 09:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623510 | 09:21 |
| *** k_mouza has quit IRC | 09:21 | |
| *** k_mouza has joined #openstack-nova | 09:22 | |
| *** erlon has quit IRC | 09:22 | |
| openstackgerrit | zhangyujun proposed openstack/nova master: Should not raise while restore power on failed https://review.openstack.org/624854 | 09:30 |
| *** yan0s has joined #openstack-nova | 09:40 | |
| *** ttsiouts has joined #openstack-nova | 09:45 | |
| *** Dinesh_Bhor has quit IRC | 09:46 | |
| *** rodolof has joined #openstack-nova | 09:48 | |
| *** rodolof has quit IRC | 09:56 | |
| *** rodolof has joined #openstack-nova | 09:56 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | 10:11 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: fixtures: Return a mocked class instead of method within fake_imagebackend https://review.openstack.org/619804 | 10:18 |
| lyarwood | mdbooth: ^ rebased this away from either of our work as neither requires it anymore, can you hit it and let me know if it's still useful? | 10:18 |
| lyarwood | that's just the fixtures change FWIW | 10:19 |
| lyarwood | the other workaround hasn't really changed but reviews still welcome | 10:19 |
| mdbooth | lyarwood: ack | 10:19 |
| *** erlon has joined #openstack-nova | 10:21 | |
| openstackgerrit | Vlad Gusev proposed openstack/nova stable/pike: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/604039 | 10:24 |
| openstackgerrit | Vlad Gusev proposed openstack/nova stable/pike: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/607544 | 10:24 |
| lyarwood | ^ LGTM btw. | 10:29 |
| mdbooth | lyarwood: I was just looking at my very similar fake_imagebackend patch | 10:29 |
| mdbooth | I commented: | 10:29 |
| mdbooth | + # NOTE(mdbooth): we spec a subclass of imagebackend.Image here because | 10:30 |
| mdbooth | + # the subclass constructors have a different signature. | 10:30 |
| mdbooth | + fake_image_class = mock.create_autospec(imagebackend.Flat) | 10:30 |
| mdbooth | + fake_image_class.side_effect = image_init | 10:30 |
| 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:30 |
| mdbooth | lyarwood: It makes sense, though, because they do have different signatures. | 10:31 |
| lyarwood | mdbooth: hmm yeah but we only use the returned mock as a class remember | 10:33 |
| lyarwood | mdbooth: the side_effect then provides the same signature as the subclass constructor | 10:33 |
| 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 |
| lyarwood | yeah indeed | 10:34 |
| *** Dinesh_Bhor has joined #openstack-nova | 10:35 | |
| 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:35 |
| lyarwood | mdbooth: ack thanks | 10:36 |
| *** markvoelker has quit IRC | 10:36 | |
| *** Dinesh_Bhor has quit IRC | 10:36 | |
| *** markvoelker has joined #openstack-nova | 10:37 | |
| *** markvoelker has quit IRC | 10:41 | |
| 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:44 |
| egonzalez | hi, did something get merged in the last few hours (maybe yesterday evening UTC) which can cause the following error? | 10:58 |
| egonzalez | http://paste.openstack.org/show/737292/ | 10:59 |
| egonzalez | Error gathering result from cell 00000000-0000-0000-0000-000000000000: ProgrammingError: (pymysql.err.ProgrammingError) (1146, u"Table 'nova_cell0.services' doesn't exist") | 10:59 |
| *** bhagyashris_ has quit IRC | 10:59 | |
| egonzalez | added other log at http://paste.openstack.org/show/737293/ while executing nova-api command | 11:02 |
| *** rambo_li has quit IRC | 11:04 | |
| *** cdent has quit IRC | 11:09 | |
| *** diga has quit IRC | 11:09 | |
| mdbooth | lyarwood: I don't understand the revert_resize problem, btw, and without the instance.host check the workaround executes on _hard_reboot | 11:14 |
| mdbooth | lyarwood: So the revert_resize thing is that we execute cleanup on the dest, right? | 11:15 |
| lyarwood | mdbooth: argh yeah we'd lose console.log with hard_reboot, the directory is recreated however. | 11:15 |
| lyarwood | mdbooth: yes, revert resize cleanup is on the dest | 11:16 |
| mdbooth | console.log, kernel, and ramdisk, but yeah | 11:16 |
| lyarwood | mdbooth: it should recreate kernel and ramdisk no? | 11:16 |
| *** markvoelker has joined #openstack-nova | 11:16 | |
| mdbooth | lyarwood: Unless they've been deleted in glance. | 11:16 |
| lyarwood | true | 11:17 |
| *** derekh has joined #openstack-nova | 11:17 | |
| mdbooth | lyarwood: So, is the revert_resize thing a different bug? | 11:17 |
| mdbooth | Because your workaround wouldn't trigger on revert_resize. | 11:18 |
| lyarwood | mdbooth: with the host check removed it should | 11:18 |
| lyarwood | mdbooth: we call cleanup with destroy_disks=False and no migrate_data | 11:18 |
| mdbooth | lyarwood: Right, but it didn't before. | 11:18 |
| mdbooth | So is that a separate latent bug? | 11:19 |
| lyarwood | mdbooth: yeah, ah crap I didn't add the bug to the commit did I | 11:19 |
| lyarwood | mdbooth: yes, it's listed in the conf change | 11:19 |
| lyarwood | mdbooth: I'll respin the commit message now | 11:19 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216 | 11:21 |
| kashyap | mdbooth: lyarwood: ^ I surely went overboard with the long commit message (that I spent 2 hours writing) | 11:22 |
| kashyap | But I made sure they're in readable chunks. | 11:22 |
| openstackgerrit | zhangyujun proposed openstack/nova master: Should not raise while restore power on failed https://review.openstack.org/624854 | 11:24 |
| kashyap | Oh, it is WIP! | 11:24 |
| lyarwood | mdbooth: kernel, ramdisk etc are set against the flavor right? | 11:26 |
| lyarwood | ah, image props, my bad | 11:27 |
| mdbooth | lyarwood: Right. I think they can also be specified on the command line. | 11:27 |
| mdbooth | I don't think anybody actually uses them, but we probably shouldn't assume that, especially not for a backport. | 11:28 |
| mdbooth | s/command line/api/ | 11:28 |
| lyarwood | mdbooth: yeah, adding a note about it now. | 11:32 |
| mdbooth | lyarwood: How about... changing revert_resize? | 11:33 |
| lyarwood | mdbooth: to pass migrate_data? | 11:34 |
| 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 |
| lyarwood | mdbooth: iirc the migration data in resize doesn't have is_shared_block_storage set | 11:34 |
| 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 |
| lyarwood | mdbooth: that's only used in the live migration flows | 11:34 |
| mdbooth | lyarwood: If you move the destroy() call after revert_migration_context(), your workaround would execute for revert_resize() | 11:35 |
| mdbooth | Which is what you want. | 11:35 |
| *** gary_perkins has quit IRC | 11:37 | |
| lyarwood | mdbooth: you'd also need to move terminate_connection then as well | 11:37 |
| lyarwood | that opens up another can of worms tbh | 11:37 |
| 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:38 |
| lyarwood | lol | 11:39 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | 11:39 |
| lyarwood | -2 - I can't see any gaffer tape in that diagram. | 11:40 |
| mdbooth | Hehe. That's v2 | 11:40 |
| 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:41 |
| *** tbachman has quit IRC | 11:42 | |
| *** Luzi has quit IRC | 11:44 | |
| 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:46 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473 | 11:48 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | 11:48 |
| *** gary_perkins has joined #openstack-nova | 11:51 | |
| 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. | 11:54 |
| *** ttsiouts has quit IRC | 12:05 | |
| *** ttsiouts has joined #openstack-nova | 12:05 | |
| 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:09 |
| kashyap | And, thanks! | 12:10 |
| *** ttsiouts has quit IRC | 12:10 | |
| *** udesale has quit IRC | 12:11 | |
| *** ttsiouts has joined #openstack-nova | 12:16 | |
| *** pcaruana has quit IRC | 12:21 | |
| *** pcaruana has joined #openstack-nova | 12:22 | |
| *** pcaruana is now known as pcaruana|intw| | 12:25 | |
| *** k_mouza_ has joined #openstack-nova | 12:25 | |
| *** k_mouza has quit IRC | 12:28 | |
| *** k_mouza_ has quit IRC | 12:30 | |
| *** k_mouza has joined #openstack-nova | 12:30 | |
| *** ratailor has quit IRC | 12:34 | |
| *** _alastor_ has joined #openstack-nova | 12:35 | |
| *** rodolof has quit IRC | 12:40 | |
| *** rodolof has joined #openstack-nova | 12:41 | |
| *** tbachman has joined #openstack-nova | 12:53 | |
| *** tbachman has quit IRC | 13:00 | |
| gibi | tssurya: the DownCellFixtur patch looks good to me | 13:04 |
| tssurya | gibi: thanks :) | 13:05 |
| *** k_mouza has quit IRC | 13:05 | |
| *** tbachman has joined #openstack-nova | 13:06 | |
| *** kaisers_ has joined #openstack-nova | 13:08 | |
| *** dave-mccowan has joined #openstack-nova | 13:09 | |
| *** _alastor_ has quit IRC | 13:14 | |
| *** Shadowphax has quit IRC | 13:15 | |
| *** dave-mccowan has quit IRC | 13:15 | |
| *** mlavalle has joined #openstack-nova | 13:26 | |
| *** derekh has quit IRC | 13:47 | |
| *** tiendc has quit IRC | 13:53 | |
| *** mriedem has joined #openstack-nova | 13:56 | |
| *** pcaruana|intw| has quit IRC | 14:05 | |
| *** k_mouza has joined #openstack-nova | 14:12 | |
| *** derekh has joined #openstack-nova | 14:24 | |
| *** udesale has joined #openstack-nova | 14:27 | |
| *** pcaruana has joined #openstack-nova | 14:31 | |
| *** dave-mccowan has joined #openstack-nova | 14:32 | |
| mdbooth | Do we have a shelve/unshelve test on rbd currently? | 14:33 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port https://review.openstack.org/570078 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject networks with QoS policy https://review.openstack.org/570079 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor PortResourceRequestBasedSchedulingTestBase https://review.openstack.org/624080 | 14:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that allocated PF matches the used PF https://review.openstack.org/623543 | 14:37 |
| *** tbachman has quit IRC | 14:38 | |
| *** dave-mccowan has quit IRC | 14:43 | |
| *** jmlowe has quit IRC | 14:48 | |
| *** panda|off is now known as panda | 14:50 | |
| *** psachin has joined #openstack-nova | 14:51 | |
| mriedem | mdbooth: we have a ceph job which runs tempest tests, | 14:51 |
| mriedem | and tempest has shelve/unshelve tests, | 14:51 |
| mriedem | so yes we should | 14:51 |
| mriedem | the ceph job has been broken since the summit though | 14:52 |
| mriedem | oh maybe that was fixed and just encrypted volumes are broken | 14:52 |
| mriedem | ah, so, the encrypted volume tests were just recently enabled in the ceph job | 14:53 |
| mriedem | https://github.com/openstack/devstack-plugin-ceph/commit/7728027cdd575f7d43c97b4b7a9345c69898c26f | 14:53 |
| *** adrianc has quit IRC | 14:57 | |
| *** awaugama has joined #openstack-nova | 14:58 | |
| *** cfriesen has joined #openstack-nova | 14:58 | |
| *** kaisers_ is now known as kaisers_away | 14:58 | |
| *** adrianc has joined #openstack-nova | 14:59 | |
| 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 |
| mdbooth | Because when you go to delete the image on unshelve it fails because the backing image is still in use. | 15:00 |
| *** mchlumsky has joined #openstack-nova | 15:00 | |
| 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:02 |
| *** jmlowe has joined #openstack-nova | 15:04 | |
| *** ivve has joined #openstack-nova | 15:05 | |
| *** tbachman has joined #openstack-nova | 15:05 | |
| *** markvoelker has quit IRC | 15:06 | |
| mriedem | mdbooth: like this? https://review.openstack.org/#/c/473897/ | 15:11 |
| mriedem | https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1697935 | 15:12 |
| openstack | Launchpad bug 1697935 in devstack-plugin-ceph "re-enable compute shelve tests" [Undecided,In progress] - Assigned to Eric Harney (eharney) | 15:12 |
| mriedem | mdbooth: probably want to talk to eharney | 15:12 |
| mriedem | jangutter: sure | 15:12 |
| jangutter | mriedem: thanks! I am trying to make the "upgrade" sequence less vague. | 15:13 |
| jangutter | mriedem: os-vif's two bits, the plug-in and the object model, has Nova between them. | 15:14 |
| 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 |
| jangutter | mriedem: os-vif object model -> nova -> os-vif plugin | 15:15 |
| mdbooth | mriedem: Thanks | 15:15 |
| *** rambo_li has joined #openstack-nova | 15:18 | |
| openstackgerrit | Merged openstack/nova master: Add DownCellFixture https://review.openstack.org/614810 | 15:19 |
| *** psachin has quit IRC | 15:21 | |
| mriedem | jangutter: i'm not sure that clears up anything for me | 15:23 |
| mriedem | my understanding is, | 15:23 |
| mriedem | we're going to ignore backward compat for os-vif object versions, | 15:23 |
| mriedem | make some changes to add the base reference/profile stuff in os-vif, | 15:23 |
| mriedem | release with a major version, | 15:23 |
| mriedem | make nova depend on that major version and start using new code from the library, | 15:24 |
| mriedem | and then....? | 15:24 |
| mriedem | something in train? | 15:24 |
| mriedem | meanwhile kuryr-kubernetes just eats it? | 15:24 |
| *** dpawlik has quit IRC | 15:24 | |
| jangutter | mriedem: for now, the new interfaces will not break backwards compatibility just yet, they're additive... | 15:25 |
| jangutter | mriedem: you're 100% right that the problem comes when _existing_ plugins transition to use them. | 15:26 |
| jangutter | mriedem: the spec does not cover the actual process of doing the backwards-incompatible changes. | 15:27 |
| mriedem | and kuryr-k8s has a problem b/c they have plugins https://github.com/openstack/kuryr-kubernetes/blob/master/setup.cfg#L26 | 15:27 |
| mriedem | as does the agilio one | 15:27 |
| mriedem | ooo does kuryr-k8s actually do vif translation too? https://github.com/openstack/kuryr-kubernetes/blob/master/setup.cfg#L36 | 15:28 |
| mriedem | https://github.com/openstack/kuryr-kubernetes/blob/master/kuryr_kubernetes/os_vif_util.py#L303 | 15:28 |
| *** psachin has joined #openstack-nova | 15:29 | |
| mriedem | well, maybe that's closer to our os_vif_util | 15:29 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/os_vif_util.py | 15:29 |
| jangutter | mriedem: The plan, during Stein is to use the new interfaces only for new plugins. (If I understand correctly). | 15:30 |
| mriedem | will there be a new plugin as of https://review.openstack.org/#/c/567148/ ? | 15:32 |
| jangutter | mriedem: in fact, enforcing the new interfaces on existing plugins (with the hairy backlevel code) is something I tried and Jay said "let's don't do that". | 15:32 |
| jangutter | mriedem: yes and no - that plugin will be the os-vif yank out of the vrouter vif type. | 15:32 |
| jangutter | mriedem: and no, the dominoes have not fallen, and the tag has not gotten applied to https://github.com/Juniper/contrail-nova-vif-driver/ yet. | 15:33 |
| mriedem | by vrouter you mean this? https://github.com/Juniper/contrail-nova-vif-driver/blob/208f0d51b6e0c1c44771e96c062588a40491b09d/setup.py#L30 | 15:34 |
| jangutter | mriedem: yep - the very same. | 15:34 |
| mriedem | so that's going to move into it's own plugin repo or something? | 15:34 |
| mriedem | i don't know what "yank out" means here | 15:34 |
| jangutter | mriedem: nope, it's going to be in-tree there. | 15:34 |
| *** ivve has quit IRC | 15:34 | |
| mriedem | and why was jay opposed to backward compat | 15:34 |
| mriedem | ? | 15:34 |
| jangutter | mriedem: https://blueprints.launchpad.net/nova/+spec/vrouter-os-vif-conversion | 15:35 |
| mriedem | b/c it's trouble for the sake of a versioned interface that isn't going over the wire? | 15:35 |
| jangutter | mriedem: https://review.openstack.org/#/c/572081/8/os_vif/objects/vif.py@272 | 15:36 |
| mriedem | oh i see so vrouter is still legacy | 15:36 |
| *** k_mouza has quit IRC | 15:36 | |
| mriedem | https://review.openstack.org/#/c/571325/6/nova/virt/libvirt/vif.py | 15:36 |
| mriedem | unless contrail? confused... | 15:36 |
| jangutter | mriedem: YOU ARE NOT THE ONLY CONFUSED PERSON! | 15:36 |
| jangutter | mriedem: from what I could piece together, vrouter used to be there, then they gained vhost-user support and added ANOTHER vif. | 15:37 |
| jangutter | mriedem: that VIF got added as an os-vif plugin. | 15:37 |
| jangutter | mriedem: and 'vrouter' was left behind. | 15:37 |
| jangutter | mriedem: alone and unloved. | 15:37 |
| mriedem | so the vrouter stuff in nova legacy vif code is unused b/c if you have the contrail plugin loaded, we'll load that up first and *not* need to fallback to the legacy vif code, | 15:38 |
| mriedem | if you do'nt have the contrail os-vif plugin installed on the system, we will use the legacy vrouter fallback | 15:38 |
| jangutter | mriedem: the contrail plugin (as is) only does vhost-user (DPDK) plugging, not tap plugging that 'vrouter' needs. | 15:39 |
| jangutter | mriedem: current contrail / tungsten / kernel solutions _still_ use the Nova path :-( | 15:40 |
| mriedem | so, in stien, os-vif will add the new composable object model things, do a major release, | 15:44 |
| mriedem | nova depends on that and starts to use it for new plugin support, | 15:44 |
| mriedem | there will be the old and new os-vif interfaces for a full release (S->T) | 15:45 |
| mriedem | in that time, | 15:45 |
| mriedem | existing plugins will need to move over to the new model, | 15:45 |
| mriedem | and then the old stuff will be dropped sometime in T with a major release of os-vif, | 15:45 |
| mriedem | and kuryr-k8s is ok with that plan? | 15:45 |
| *** rambo_li has quit IRC | 15:45 | |
| mriedem | and hopefully no one is storing a serialized version of these things with the old model in a db somewhere... | 15:46 |
| mriedem | i.e. the object model will always be recalculated at the time it's needed | 15:46 |
| jangutter | mriedem: I don't think os-vif is needs to do a major release until we've ironed out the upgrade path -> sean-k-mooney and jaypipes will likely assign some fun stuff to me. | 15:46 |
| jangutter | mriedem: pinning object version numbers would not break backwards compatibility? | 15:48 |
| *** dpawlik has joined #openstack-nova | 15:48 | |
| aspiers | mriedem: Have made some progress on your questions. More detail soon but quick summary for now is that q35 machine type doesn't support IDE which means we'll have to jump through an extra hoop or two in certain cases (configdrive defaults to iso9660 which currently means ide bus on x86_64), but given the existing virtio-scsi support I'm optimistic this won't be a big hurdle (to mix metaphors). It will | 15:49 |
| mriedem | pinning how? where? | 15:49 |
| aspiers | almost certainly require extra documentation though, and possibly extra requirements on any images which need to work with SEV+configdrive. | 15:49 |
| aspiers | I think similar caveats will apply for volumes attached via IDE too | 15:50 |
| jangutter | mriedem: i.e. not incrementing the version numbers of the objects. | 15:50 |
| aspiers | but these caveats already apply when q35 machine type is requested via image metadata, so it's not an entirely new limitation | 15:50 |
| mriedem | aspiers: yeah i saw that yesterday, thanks. if we're already documenting things for sev limitations, i guess saying "config drive will require vfat" which, since that's per-host today, means you'd have sev instances restricted to certain hosts - which people would probably do anyway given the reserved memory restrictions, | 15:51 |
| mriedem | i had thought there was an image property to tell nova which config drive format to use, but i only see img_config_drive to tell nova to use config drive or not | 15:51 |
| mriedem | maybe it's just not documented... | 15:51 |
| aspiers | mriedem: yeah exactly, there's also the hardware requirement for SEV support they already have to consider :-) | 15:52 |
| mriedem | i guess there isn't an image property for the config drive format, | 15:52 |
| aspiers | I didn't see one | 15:52 |
| mriedem | we could certainly add that, seems weird that it's per-host config and not per-instance | 15:52 |
| aspiers | it looks hardcoded to me | 15:52 |
| *** dpawlik has quit IRC | 15:52 | |
| mriedem | it's a config option | 15:53 |
| aspiers | right | 15:53 |
| mriedem | i'd like to avoid conditional logic on the type by adding things like "if SEV use vfat" etc | 15:53 |
| aspiers | yes I agree | 15:53 |
| aspiers | don't want to introduce any more snowflake code than strictly necessary | 15:54 |
| aspiers | mriedem: but it made me wonder, is there any good reason why x86_64 is the only architecture still using ide for configdrive? | 15:54 |
| mriedem | jangutter: i thought the spec said that for the new composable interface stuff there wouldn't be version changes on the object model | 15:54 |
| aspiers | I looked at the git commit history and it seemed to be just a legacy thing | 15:55 |
| mriedem | aspiers: idk, it could just be really old code | 15:55 |
| mriedem | i'm not really familiar with that though, mdbooth might be more help | 15:55 |
| aspiers | originally it was x86_64 only, and later other archs were added which couldn't use ide | 15:55 |
| aspiers | so yeah, I think it's just really old code | 15:55 |
| aspiers | but even if so, I'm guessing changing it would break some guests which are maybe expecting the config drive to be IDE | 15:56 |
| mriedem | maybe, | 15:56 |
| aspiers | one of our qemu/libvirt gurus recommends virtio-scsi | 15:56 |
| mriedem | if the guest knows what it wants for the config drive format, we should really have an image property or flavor extra spec for that | 15:56 |
| aspiers | that sounds like a good idea ot me | 15:57 |
| aspiers | BTW I also found https://blueprints.launchpad.net/nova/+spec/virtio-scsi-by-default | 15:57 |
| aspiers | which is per-host | 15:57 |
| aspiers | so maybe that could help | 15:57 |
| aspiers | well, I guess that wouldn't affect configdrive | 15:57 |
| jangutter | mriedem: as far as I understand (and that's a follow-on question later), changing the object hash means you _should_ bump the version. we'll be changing the hash, but deciding not to bump the version. | 15:57 |
| mriedem | "2) VMs will be able to have more than 26 volumes attached, up to 255." | 15:57 |
| mriedem | that overlaps with a bp melwitt is working on | 15:58 |
| aspiers | oh, interesting | 15:58 |
| mriedem | https://blueprints.launchpad.net/nova/+spec/conf-max-attach-volumes | 15:58 |
| melwitt | did someone say... volumes | 15:58 |
| aspiers | :D | 15:58 |
| aspiers | I also found caveats here https://access.redhat.com/solutions/1758693 but not sure if they still apply | 15:58 |
| aspiers | I don't have access to the rest of that article ;-P | 15:58 |
| mriedem | jangutter: yes you should bump the versoin whenever you change the object model | 15:59 |
| mriedem | like new fields, new methods, method signature changes, etc | 15:59 |
| mriedem | however, if nothing is using the os-vif objects over the wire, it's mostly just a best practice | 16:00 |
| mriedem | so like i said on the spec, it's not the end of the world if the rules are bent during a transition here for hte library | 16:00 |
| aspiers | mriedem: any other configdrive experts I should poll on this? just mdbooth? | 16:00 |
| mriedem | i.e. ultimately we get the same thing from os-vif as we do from os-brick, and os-brick is just random dicts | 16:00 |
| mriedem | we take a thing from cinder, pass it to brick, get a dict back, pass that to cinder, | 16:01 |
| mriedem | and the vendor backends all understand what is in their special dict | 16:01 |
| jangutter | mriedem: yes - and Jay keeps trying to get me to say that out loud in our open-plan office. | 16:01 |
| mriedem | same thing with os-vif | 16:01 |
| mriedem | if this channel is our open plan office, i just said it out loud | 16:01 |
| mriedem | it makes upgrades extremely brittle, | 16:01 |
| mriedem | and really can/should only support N-1 at most lockstep upgrades between nova/cinder and nova/neutron, | 16:02 |
| aspiers | If the only clean way to solve this is adding a per-image property for configdrive bus, I could submit a bp/spec for that. I guess we'd also have to decide whether that's a strict dependency on the critical path to the SEV MVP, or something which can proceed in parallel. | 16:02 |
| mriedem | but no one has cared enough yet to take that on | 16:02 |
| mriedem | aspiers: mikal has historically been the goto config drive person | 16:02 |
| aspiers | gotcha, thanks | 16:02 |
| mriedem | but you would likely have to email him | 16:02 |
| melwitt | aspiers: fwiw there's not additional caveats on the article beyond what shows in that "preview" | 16:02 |
| jangutter | mriedem: yes - and the trouble is that os-vif, even though it's pretty new, has already gotten some crufty bits Sean and Jay wants to clean up. | 16:03 |
| aspiers | melwitt: thanks! | 16:03 |
| mriedem | aspiers: that doesn't need it's own bp, it would just be a work item in your existing sev spec | 16:03 |
| mriedem | i.e. you have two options as a deployer: | 16:03 |
| mriedem | 1. config the host(s) to use vfat for everything, and use host aggregates to pin sev instances to those hosts (you might do this anyway as noted) | 16:04 |
| mriedem | 2. if you mix instances on hosts (sev and non-sev), then you can add an image property to control the config drive format for the sev instances | 16:04 |
| aspiers | yup, that makes sense | 16:05 |
| mriedem | jay is always looking to clean up crufty bits, with fire usually | 16:05 |
| jangutter | mriedem: so the core question is - should I cut down that section (saying - the adoption plan for these interfaces will be ironed out. Promise!) | 16:05 |
| jangutter | mriedem: or scale up -> this is the twelve-point plan for world domination, we have everything in hand. | 16:06 |
| mriedem | jangutter: so going back to what i asked earlier http://paste.openstack.org/show/737315/ - is that correctly capturing the plan? | 16:06 |
| aspiers | mriedem: beginning to sound like we need yet another patchset then... I feel increasingly guilty each time I submit a new one ;-/ | 16:06 |
| mriedem | jangutter: it doesn't need to be either extreme, | 16:06 |
| mriedem | it can be "here is the proposed high-level plan, details will be ironed out during implementation" | 16:06 |
| mriedem | but if http://paste.openstack.org/show/737315/ is the high level plan, then throw that in the spec | 16:06 |
| jangutter | mriedem: I think, the last line is probably going to be - we'll work with kuryr to help them during the transition. | 16:06 |
| mriedem | ok | 16:07 |
| mriedem | then shoot the moon i guess | 16:07 |
| *** dklyle has quit IRC | 16:07 | |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add requested_networks to RequestSpec https://review.openstack.org/570201 | 16:07 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add instance hard delete https://review.openstack.org/570202 | 16:07 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: [WIP] Enable rebuild for instances in cell0 https://review.openstack.org/570203 | 16:07 |
| *** dklyle has joined #openstack-nova | 16:07 | |
| aspiers | mriedem: regarding IDE attached volumes I'm thinking that this could just be a documented limitation, similarly to the boot disk limitation against virtio-blk | 16:07 |
| *** efried has joined #openstack-nova | 16:08 | |
| aspiers | I don't think there are any other limitations re. volumes but still confirming that | 16:08 |
| jangutter | mriedem: many, many, many thanks. And one more indulgence. | 16:08 |
| jangutter | mriedem: Looking at ovo I didn't find good docs describing the effects of inheritance and composition on object versions.... | 16:09 |
| *** jobewan has joined #openstack-nova | 16:09 | |
| mriedem | jangutter: they probably don't exist | 16:09 |
| jangutter | mriedem: composition's always been "use a map" or something. | 16:09 |
| mriedem | jangutter: i will have to defer to dansmith on that | 16:09 |
| jangutter | mriedem: before I forget all of this and have to relearn it -> where would the best place be to commit it, onto the ovo docs? | 16:10 |
| mriedem | probably yeah | 16:10 |
| mriedem | https://docs.openstack.org/oslo.versionedobjects/latest/user/index.html | 16:10 |
| jangutter | mriedem: thanks, I'll jot up my notes, fall into a Christmas coma, wake up and try to figure out what this madman wrote. Thanks very very very much for braving the madness. | 16:11 |
| *** dklyle has quit IRC | 16:13 | |
| mriedem | jangutter: so, are you going to update the spec at all today? or should i just approve and you'll follow up or...? | 16:15 |
| *** tbachman has quit IRC | 16:15 | |
| mriedem | b/c i'm here next week but gone the week of xmas, and then back the first week of jan but out the week of jan 10 | 16:15 |
| mriedem | jan 10 is spec freeze | 16:15 |
| mriedem | jangutter: if you're just going to follow up on the spec with clarifications and cleanups, but no major technical changes in direction, i'll just approve now and you can follow up | 16:16 |
| jangutter | mriedem: I'm respinning today with the changes, even in a coma I have some limited brain function. | 16:16 |
| mriedem | ok | 16:16 |
| jangutter | mriedem: I'm not going to disappear, but I won't be on 24/7. | 16:17 |
| *** pcaruana has quit IRC | 16:20 | |
| *** spatel has joined #openstack-nova | 16:22 | |
| spatel | sean-k-mooney: morning.. | 16:23 |
| mriedem | my enthusiasm for this spec will disappear however :) | 16:23 |
| mriedem | i want to be this lady when i grow up https://deadspin.com/phoenix-woman-dunks-on-suns-owner-robert-sarver-at-city-1831084576 | 16:23 |
| spatel | sean-k-mooney: I am seeing these errors in my nova.log on compute node: every minute duration.. | 16:23 |
| spatel | ostack-compute-sriov-01 nova-compute:2018-12-14 11:22:22.161 40288 WARNING nova.pci.utils [req-0d87b5e4-6ece-4beb-880c-51c7c5835a66 - - - - -] No net device was found for VF 0000:03:09.0: PciDeviceNotFoundById: PCI device 0000:03:09.0 not found | 16:23 |
| *** ivve has joined #openstack-nova | 16:24 | |
| *** dklyle has joined #openstack-nova | 16:24 | |
| spatel | Everything working great just trying to understand WARNING mesg.. | 16:24 |
| *** jmlowe has quit IRC | 16:27 | |
| aspiers | mriedem: BTW thanks a lot again for your continued patience and help :) I can probably submit an updated patchset in the next few hours if you think that makes sense? | 16:32 |
| *** igordc has joined #openstack-nova | 16:33 | |
| *** kaisers_away is now known as kaisers_ | 16:37 | |
| *** gyee has joined #openstack-nova | 16:37 | |
| *** rodolof has quit IRC | 16:38 | |
| mriedem | aspiers: sure, but you're still waiting on volume-backed support test results or...? | 16:38 |
| lyarwood | mriedem / melwitt ; https://review.openstack.org/#/c/618478/ - quick question about this workaround, is it going to need a releasenote and if so which section should I use? I'm assuming `issues` but the docs suggest that's only used towards GA. | 16:39 |
| mriedem | other | 16:39 |
| *** rodolof has joined #openstack-nova | 16:39 | |
| lyarwood | kk, I'll sort that out now. | 16:39 |
| aspiers | mriedem: not test results - just confirmation that there are no other bus-related limitations with q35 | 16:39 |
| mriedem | lyarwood: note that my responding to you does not imply me wanting that change :) | 16:40 |
| aspiers | mriedem: we've already proven the ide issue (although that hardly required proof) | 16:40 |
| *** dklyle has quit IRC | 16:41 | |
| lyarwood | mriedem: yeah I know, I know, it's garbage... | 16:41 |
| mriedem | lyarwood: it doesn't mean *you're* garbage though! | 16:41 |
| lyarwood | mriedem: awww thanks | 16:41 |
| * mriedem hugs lee | 16:41 | |
| melwitt | lol | 16:42 |
| *** Shilpa has quit IRC | 16:42 | |
| *** wolverineav has joined #openstack-nova | 16:44 | |
| dansmith | mriedem: splattered some more comments on your spec if you want to try to read those before I fade into holiday | 16:45 |
| *** rodolof has quit IRC | 16:45 | |
| dansmith | mriedem: I guess at this point, the plan just needs to try like real hard to get that merged the first week in january before you disappear and the spec deadline comes eh? | 16:45 |
| *** rodolof has joined #openstack-nova | 16:45 | |
| *** belmoreira has quit IRC | 16:47 | |
| *** igordc has quit IRC | 16:48 | |
| *** igordc has joined #openstack-nova | 16:48 | |
| mriedem | dansmith: yeah | 16:51 |
| dansmith | well, if I don't come back next year, I wish you luck | 16:51 |
| mriedem | thank you | 16:51 |
| *** kaisers_ is now known as kaisers_away | 16:52 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543 | 16:53 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310 | 16:53 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311 | 16:53 |
| mriedem | dansmith: i think you will very much enjoy this bug fix while you're here https://review.openstack.org/#/c/619061/ | 16:56 |
| mriedem | fixes the pg job, | 16:56 |
| mriedem | should improve our cell mapping query by project_id code | 16:56 |
| mriedem | signed off by zzzeek himself | 16:56 |
| melwitt | mriedem, dansmith: should we... adjust the spec deadline because of the holidays? it kinda sucks that there's so much unactionable time between now and the deadline | 16:56 |
| dansmith | mriedem: I think you very much don't know what I enjoy, apparently | 16:56 |
| *** mdbooth has quit IRC | 16:56 | |
| mriedem | dansmith: oh but there is stuff about camping and dirty cartoons in it | 16:56 |
| dansmith | melwitt: the holidays come the same time every year, so I don't think this is much of a surprise | 16:56 |
| dansmith | mriedem: ORLY | 16:56 |
| melwitt | I don't think it's a surprise, but anyway, was just a thought | 16:57 |
| mriedem | melwitt: i say leave it - people have plenty of actionable time for months leading up to the freeze | 16:57 |
| dansmith | melwitt: and we've had a lot of time open for spec review, I'm not sure that extending the deadline is going to change that | 16:57 |
| melwitt | ok, fair enough | 16:58 |
| mriedem | melwitt: what you could do is sort out what from the bucket is stuff we should actually focus on before the freeze, and then get that out to the ML | 16:58 |
| mriedem | because there is likely lots of cruft, but a few that just need another +2 or are things we should focus on in stein yet | 16:58 |
| dansmith | melwitt: sorry I should have thrown a smiley on the end of that.. meant to sound snarky, not dickly :) | 16:58 |
| dansmith | mriedem: ++ | 16:59 |
| dansmith | help focus the last week of reviews in jan | 16:59 |
| mriedem | at this point i'm mostly looking to get the amd sev and jan's thing in | 16:59 |
| mriedem | i mean for things i'm actively reviewing | 16:59 |
| melwitt | ok, that's good feedback. I'll do that | 16:59 |
| mriedem | there are a couple of other specs that have had lots of review and just need a push over the line | 16:59 |
| *** kaisers_away is now known as kaisers_ | 17:00 | |
| * melwitt nods | 17:00 | |
| *** kaisers_ is now known as kaisers_away | 17:00 | |
| kashyap | Heay folks, so I was reading the scrollback here. I have posted a "simple thing" -- https://review.openstack.org/#/c/625216/ (libvirt: Support native TLS for migration and disks over NBD) | 17:01 |
| *** kaisers_away is now known as kaisers_ | 17:01 | |
| kashyap | See the commit message (I spent full 2 hours fiddling with it) for the "essay" :D | 17:01 |
| *** ttsiouts has quit IRC | 17:02 | |
| mriedem | doesn't sound simple | 17:02 |
| melwitt | that looks like that should be at least a blueprint, if not a spec | 17:02 |
| kashyap | :-) | 17:02 |
| mriedem | live migration + tls + anything = doom | 17:02 |
| kashyap | Truer words were never said | 17:02 |
| *** k_mouza has joined #openstack-nova | 17:03 | |
| mriedem | "oh you're on/not on shared storage and/or volume-backed? sorry!" | 17:03 |
| kashyap | mriedem: Ha! _That_ is what is fixed | 17:03 |
| kashyap | This whole week I (almost) singuarly focused on it | 17:03 |
| kashyap | Even wrote this detailed doc, with a start, middle, and an end: https://kashyapc.fedorapeople.org/Native-TLS/Setup-for-NBD-and-migration-streams-over-TLS.rst.txt | 17:03 |
| kashyap | You can do a "TLS journey" if you follow that doc with concentration and focus. | 17:04 |
| mriedem | can i do *anything* else? | 17:04 |
| kashyap | What more ... I even have evidence files (yet to upload) that shows precisely what to find :D | 17:04 |
| kashyap | mriedem: You are free to do anything you fancy. You know that | 17:04 |
| mriedem | i'm just joking of course | 17:04 |
| kashyap | :D | 17:04 |
| * mriedem hugs kashyap | 17:04 | |
| mriedem | i'm real fucking huggy today | 17:05 |
| kashyap | I see mriedem is giving free hugs all around | 17:05 |
| kashyap | mriedem: See -- you didn't take up my offer to get you that drink (damned if I'll remember the name) you wanted | 17:05 |
| kashyap | But _of course_ ... when you're with SO in Europe | 17:05 |
| mriedem | root beer | 17:05 |
| *** tssurya has quit IRC | 17:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Migrate upgrade checks to oslo.upgradecheck https://review.openstack.org/603499 | 17:05 |
| kashyap | Not even the last thing you want to do is hang out with me :D | 17:05 |
| mriedem | or "rooted beer" as an EU might call it | 17:06 |
| kashyap | Ah, yes. | 17:06 |
| kashyap | mriedem: So the nice thing is that: now Nova guests can have encryption (*without* the penalty of "libvirtd tunnelling") for all migration streams | 17:06 |
| kashyap | Namely: Instance guest RAM, device state + disks over NDB | 17:07 |
| * kashyap stops with the marketing | 17:07 | |
| *** udesale has quit IRC | 17:07 | |
| *** wolverineav has quit IRC | 17:07 | |
| kashyap | And goes to upload his "evidence files" | 17:07 |
| kashyap | Oh ... the fly in the ointment is I'll be on PTO next week (if I don't take them, they'll disappear). So only be back in 1st week of Jan to come back to write tests | 17:08 |
| kashyap | So ... saying: "No, you wretch, I will not even click on the URL without tests" is perfectly reasonable | 17:09 |
| mriedem | i seem to remember something about nbd + the tunneling thing years ago from danpb | 17:09 |
| mriedem | but it's very hazy now | 17:10 |
| mriedem | or maybe it was markmc | 17:10 |
| mriedem | idk | 17:10 |
| * mriedem goes to lunch | 17:10 | |
| *** mriedem is now known as mriedem_lunch | 17:10 | |
| *** erlon_ has joined #openstack-nova | 17:10 | |
| kashyap | mriedem_lunch: When you get back from lunch, as of today: | 17:10 |
| kashyap | - Nova has 'live_migration_tunnelled' -- but if you use that, you can't do "block migration" | 17:11 |
| *** dklyle has joined #openstack-nova | 17:11 | |
| kashyap | That is one of the major cases we fix. Thus, having encryption for all streams | 17:11 |
| kashyap | (By "streams", migration stream + NBD stream) | 17:12 |
| kashyap | And going forward, we will deprecate the 'live_migration_tunnelled', because it has _awful_ performance impact and latency | 17:13 |
| kashyap | As there'll be no compelling reason to use it. | 17:13 |
| kashyap | melwitt: Hi, forgot to address your comment | 17:14 |
| *** erlon has quit IRC | 17:14 | |
| kashyap | melwitt: I _thought_ I posted a spec, but only settled with "bug", as it's technically a "bug": https://bugs.launchpad.net/nova/+bug/1798796 | 17:14 |
| openstack | Launchpad bug 1798796 in OpenStack Compute (nova) "libvirt: Use VIR_MIGRATE_TLS to get QEMU's native TLS support for migration and NBD" [Medium,In progress] - Assigned to Kashyap Chamarthy (kashyapc) | 17:14 |
| kashyap | But, I know what you mean. | 17:15 |
| melwitt | calling that a bug is a stretch IMO, if you need a commit message that long and at least a certain version of QEMU etc | 17:15 |
| melwitt | but maybe that's just me | 17:16 |
| kashyap | No, you're fully right. Given the length of the commit message | 17:16 |
| kashyap | I might as well just put it in a spec | 17:16 |
| kashyap | What do you suggest: a spec-less Blueprint, or a spec? | 17:17 |
| * kashyap goes to register a BP to begin with | 17:17 | |
| melwitt | I usually say, start with a specless blueprint (you need a blueprint either way) and then usually we discuss in the nova meeting and decide whether a spec is needed | 17:17 |
| dansmith | I agree with melwitt, and I think it's a spec given the commit message is already nearly a spec | 17:17 |
| dansmith | well, I agree with melwitt as of 30s ago :P | 17:17 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | 17:17 |
| kashyap | dansmith: Okido. As you know, I'm not averse to (efficient and useful!) words | 17:18 |
| *** efried has quit IRC | 17:18 | |
| kashyap | Sorry for being a lazy pig and not writing it until the last moment | 17:18 |
| dansmith | in said meeting, I would just say "if you need a spec in the commit message it should probably be a spec" :) | 17:18 |
| kashyap | I accept any penalty you may incur on me. | 17:18 |
| kashyap | dansmith: No...that's too quippy | 17:18 |
| kashyap | Okay, before the my brain cache gets flushed, might as well just do it. Here we go. | 17:19 |
| melwitt | heh. well, true if it were specless it would have to be like, literally the commit message was enough to explain it and no one had further questions and no additional details were needed | 17:19 |
| melwitt | which is unlikely | 17:20 |
| kashyap | melwitt: Did you read the commit message? :-) | 17:20 |
| dansmith | the code is really simple, | 17:20 |
| dansmith | so it seems overkill for a spec, but the commit message is long, and there's apparently another tome of reference material from kashyap | 17:20 |
| dansmith | so I dunno | 17:20 |
| kashyap | Yeah, except the part about _which_ of the 4 flags go together, and _which_ I shold not mix | 17:20 |
| melwitt | kashyap: yes :) it is a mini spec | 17:21 |
| kashyap | dansmith: From enabling point of view, it's straight-forward | 17:21 |
| kashyap | melwitt: LOL | 17:21 |
| kashyap | Yeah, I spent a whole 2 hours to write. I'm a slow-as-molasses with writing down thoghts | 17:21 |
| dansmith | so i could be maybe persuaded that it should just be a BP, if the commit message was trimmed down to non-kashyap standards and the patch included docs changes to explain what the commit message is | 17:22 |
| melwitt | another advantage of the spec is it goes into our docs as nice reference place | 17:22 |
| dansmith | yes, the commit message is documenting usage, which isn't right, IMHO | 17:22 |
| kashyap | dansmith: The TODO will be gone from the commit. Isn' it quite trimmed? | 17:22 |
| kashyap | Yeah, I agree with both of you | 17:22 |
| dansmith | kashyap: no | 17:23 |
| kashyap | This is so frelling fiddly because it took me the whole dammed WEEK to set it up! | 17:23 |
| dansmith | kashyap: if you need rst formatting and section headers in your commit message, you're doing it wrong | 17:23 |
| melwitt | haha | 17:23 |
| kashyap | dansmith: I dare you: | 17:23 |
| dansmith | melwitt: amirite? :) | 17:23 |
| kashyap | No | 17:23 |
| melwitt | dansmith: yup | 17:24 |
| kashyap | dansmith: So long as the commit message is _useful_, readable, conveys meaningful information, the format doesn't matter | 17:24 |
| kashyap | You've seen kernel commit messages, they go longer :D | 17:24 |
| kashyap | News at 11: "This is not the kernel" | 17:24 |
| dansmith | kashyap: they use them for a totally different purpose | 17:24 |
| kashyap | dansmith: All the while I was writing that commit, I thought: "A cover letter would be nice" ... which is the "spec" :-) | 17:25 |
| kashyap | dansmith: And yes: as you asked earlier: I of course want to write docs with the patch. | 17:26 |
| melwitt | kashyap: yeah, I think you have great info there, just better in a design doc rather than commit message, IMHO | 17:27 |
| kashyap | Yeah, agreed. I just wrote that to not lose the context in the head. | 17:27 |
| kashyap | Will move it around. | 17:27 |
| melwitt | and commit message refers to that design doc (blueprint) so we can get to it | 17:27 |
| melwitt | yeah, I understand | 17:27 |
| kashyap | At the risk of being banned from the channel, if I might say ... I'm a bit old-fashioned in the sense that: A commit should explain without requiring network access :-) | 17:28 |
| melwitt | heh | 17:28 |
| kashyap | Sorry, it's a habit I learnt from the "mailing list" folks. | 17:29 |
| melwitt | these days it feels like there's nothing you can do without network access | 17:29 |
| kashyap | That's why I try to not "break people's flow of reading" by saying: here you must click on this link, and then, get distrated by the Reddit tab next to it | 17:29 |
| kashyap | And then forget that you were reading the commit message :D | 17:29 |
| *** markvoelker has joined #openstack-nova | 17:31 | |
| melwitt | I'll put this bp (after you create it) on the "first week of jan" list since it sounds like the code change is simple, just a matter of docs and capturing details | 17:32 |
| kashyap | Thanks, one messy thing (even the libvirt storage folks said) is the item I noted in the 1023434th line on the commit: | 17:33 |
| kashyap | "Figure out _which_ combination of migration flags are valid. E.g. VIR_MIGRATE_NON_SHARED_INC + VIR_MIGRATE_TLS: To migrate the NBD stream over TLS. And what about VIR_MIGRATE_NON_SHARED_DISK?" | 17:33 |
| kashyap | I need to carefully re-review what flags Nova uses, and what combinations we allow. | 17:33 |
| kashyap | It's actually not "messy"; just a logic thing I need to work out. | 17:34 |
| *** k_mouza_ has joined #openstack-nova | 17:34 | |
| melwitt | ok | 17:35 |
| *** markvoelker has quit IRC | 17:35 | |
| *** bnemec is now known as beekneemech | 17:37 | |
| *** k_mouza has quit IRC | 17:38 | |
| kashyap | melwitt: When you get a minute: one comment on benchmarks | 17:38 |
| *** psachin has quit IRC | 17:39 | |
| kashyap | Benchmarks of non-native TLS vs. "native TLS", i.e. DanPB did some of it when he was doing the QEMU work | 17:39 |
| kashyap | But I don't have the URL ready; I'll check w/ him and add that details in the "Q&A" with the reviewers | 17:39 |
| *** k_mouza_ has quit IRC | 17:39 | |
| kashyap | Hope it's not a "hard requirement" | 17:39 |
| *** macza has joined #openstack-nova | 17:45 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310 | 17:45 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311 | 17:45 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543 | 17:45 |
| melwitt | kashyap: I don't think so, since it's opt-in via conf option | 17:45 |
| *** macza has quit IRC | 17:45 | |
| *** derekh has quit IRC | 17:47 | |
| openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement os-vif generic datapath offloads https://review.openstack.org/607610 | 17:50 |
| *** dklyle has quit IRC | 17:51 | |
| *** kaisers_ is now known as kaisers_away | 17:51 | |
| *** wolverineav has joined #openstack-nova | 17:52 | |
| *** wolverineav has quit IRC | 17:52 | |
| *** wolverineav has joined #openstack-nova | 17:52 | |
| kashyap | melwitt: Okido, noted. | 17:52 |
| melwitt | would be certainly nice to include in the docs and/or spec when you find it | 17:53 |
| jangutter | dansmith: so... I'm looking for a place to jot down the stuff I learned about ovo and versioning before I forget it and have to rediscover it. | 17:55 |
| jangutter | dansmith: mostly the things about the great "composition vs inheritance" debate of 2018. | 17:55 |
| dansmith | jangutter: yeah, I'm not sure what to tell you, but it seems like somewhere in ovo is the right place | 17:56 |
| jangutter | dansmith: even if it ends up being an abandoned doc review, want to do something in January or so... | 17:57 |
| dansmith | jangutter: january sounds god :) | 17:57 |
| dansmith | *good | 17:57 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478 | 17:58 |
| jangutter | dansmith: I was half-afraid that "just do it as a blog post" would be the go-to thing, but I'll try to make it generic enough that it's not a pain to update. | 17:59 |
| *** yan0s has quit IRC | 17:59 | |
| *** dklyle has joined #openstack-nova | 18:00 | |
| dansmith | jangutter: yeah docs somewhere would be better I think, even if it has to start in nova, but just a text file in ovo would be better than nothing | 18:00 |
| kashyap | melwitt: When you have a moment, as promised: https://blueprints.launchpad.net/nova/+spec/support-qemu-native-tls-for-live-migration | 18:00 |
| melwitt | thanks | 18:00 |
| melwitt | aw, I was expecting a paste of your commit message in there | 18:01 |
| kashyap | melwitt: Does it make sense? My eyes are weary from reading my own sentences | 18:01 |
| kashyap | melwitt: I thought it's not "allowed"? | 18:01 |
| melwitt | why not? | 18:01 |
| kashyap | Can post the rest of it there as well. As the Blueprint says "this is shown in feature texts" | 18:02 |
| kashyap | Implying: "be mindful what you post" | 18:02 |
| kashyap | Okay, let me go post the rest | 18:02 |
| melwitt | oh, I didn't know that. it's not shown in any of our docs | 18:02 |
| kashyap | Ah, okay, then. | 18:03 |
| melwitt | we use launchpad just for the milestone tracking and such, for organizing things | 18:03 |
| *** jangutter has quit IRC | 18:05 | |
| *** kaisers_away is now known as kaisers_ | 18:06 | |
| *** kaisers_ is now known as kaisers_away | 18:07 | |
| kashyap | Yep, noted. | 18:11 |
| kashyap | melwitt: When you can, please refresh and see if it's better | 18:11 |
| * kashyap hopes mriedem_lunch won't get tarred and feathered by mriedem_lunch for writing a "mini spec" in a Blueprint | 18:12 | |
| kashyap | No ... he is a good mood today; I even got the Christmas present of a hug from him | 18:12 |
| kashyap | Err, that "/me" sentence got borked. Let me rewrite | 18:12 |
| *** igordc has quit IRC | 18:13 | |
| *** itlinux has quit IRC | 18:13 | |
| * kashyap hopes _he_ won't get tarred and feathered by mriedem_lunch for writing a "mini spec" in a Blueprint :D | 18:13 | |
| *** itlinux has joined #openstack-nova | 18:13 | |
| melwitt | kashyap: looks fine to me. I didn't think anyone would have a problem with it, so if someone does, sorry for leading you astray. tell them it's my fault :) | 18:16 |
| kashyap | Hehe | 18:16 |
| kashyap | melwitt: No problem. If people ask me questions & convince me to write a spec, and I can't defend myself, I will of course write one without making excuses :-) | 18:16 |
| kashyap | Okay, I'll make dinner, then | 18:17 |
| melwitt | kashyap: yeah, I think the thought is, either way we want the details documented somewhere. so you can either add docs changes to your patch that explains all the version needed, flags to use, benchmark link, reference material link, etc. or, you can put that stuff in a spec. for now, I thought since you wrote that commit message, why not paste it in the blueprint for a starting point | 18:18 |
| melwitt | you'll need a release note too, in your patch | 18:19 |
| kashyap | Yeah, absolutely | 18:20 |
| kashyap | Was just writing that in the TODO :-) | 18:21 |
| *** sridharg has quit IRC | 18:21 | |
| *** sahid has quit IRC | 18:21 | |
| *** wolverineav has quit IRC | 18:23 | |
| *** dklyle has quit IRC | 18:25 | |
| *** Swami has joined #openstack-nova | 18:26 | |
| *** wolverineav has joined #openstack-nova | 18:26 | |
| *** rodolof has quit IRC | 18:27 | |
| *** wolverineav has quit IRC | 18:28 | |
| *** wolverineav has joined #openstack-nova | 18:28 | |
| *** mlavalle has quit IRC | 18:35 | |
| *** kaisers_away is now known as kaisers_ | 18:37 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216 | 18:38 |
| *** kaisers_ has quit IRC | 18:42 | |
| *** N3l1x has joined #openstack-nova | 18:42 | |
| *** N3l1x has quit IRC | 18:42 | |
| *** N3l1x has joined #openstack-nova | 18:43 | |
| *** dklyle has joined #openstack-nova | 18:46 | |
| *** mriedem_lunch is now known as mriedem | 18:49 | |
| *** tbachman has joined #openstack-nova | 18:51 | |
| *** mriedem has quit IRC | 18:52 | |
| *** spatel has quit IRC | 18:53 | |
| *** dklyle has quit IRC | 18:54 | |
| *** mriedem has joined #openstack-nova | 18:56 | |
| *** igordc has joined #openstack-nova | 18:59 | |
| openstackgerrit | Merged openstack/nova stable/queens: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623509 | 19:07 |
| *** dklyle has joined #openstack-nova | 19:11 | |
| *** dklyle has quit IRC | 19:21 | |
| *** wolverineav has quit IRC | 19:25 | |
| *** wolverineav has joined #openstack-nova | 19:25 | |
| *** ttsiouts has joined #openstack-nova | 19:28 | |
| *** wolverineav has quit IRC | 19:30 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fail to live migration if instance has a NUMA topology https://review.openstack.org/611088 | 19:31 |
| *** jmlowe has joined #openstack-nova | 19:32 | |
| *** wolverineav has joined #openstack-nova | 19:38 | |
| *** burt has joined #openstack-nova | 19:40 | |
| *** dklyle has joined #openstack-nova | 19:42 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: add live migration timeout action https://review.openstack.org/619143 | 19:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove live_migration_progress_timeout config https://review.openstack.org/619142 | 19:50 |
| *** dklyle has quit IRC | 19:55 | |
| *** macza has joined #openstack-nova | 20:01 | |
| *** ttsiouts has quit IRC | 20:04 | |
| *** ttsiouts has joined #openstack-nova | 20:04 | |
| *** betherly has joined #openstack-nova | 20:09 | |
| *** betherly has quit IRC | 20:13 | |
| melwitt | started the "first week of jan" etherpad here https://etherpad.openstack.org/p/nova-stein-blueprint-spec-freeze | 20:15 |
| melwitt | will send it to the ML when I have more filled out | 20:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix up force live migration completion docs https://review.openstack.org/625339 | 20:16 |
| mriedem | dansmith: so i want to test triggering a post-copy in the gate, but the problem is these tiny cirros instances live migrate in less than a second, and our completion timeout is in seconds (integers) so i need it to take at least a second. wondering if you're aware of any libvirt guest APIs where I can inject something into the guest to basically stall the live migration and force the timeout and switch to post-copy | 20:27 |
| mriedem | i know sahid had that bandwidth thing... | 20:27 |
| mriedem | hey we even have a config option for this... live_migration_bandwidth | 20:30 |
| *** igordc has quit IRC | 20:31 | |
| *** ttsiouts has quit IRC | 20:33 | |
| dansmith | mriedem: the only thing you can do to stall it from the inside is to dirty memory like a banshee | 20:33 |
| dansmith | iirc, danpb had a little C program that would behave badly | 20:33 |
| dansmith | but it'd be pretty dependent on how fast the machine was that you're testing on | 20:34 |
| dansmith | but no, not aware of anything libvirty to help with that | 20:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test for force complete live migrate with post-copy https://review.openstack.org/625343 | 20:35 |
| openstackgerrit | Merged openstack/nova master: Remove lock on SchedulerReportClient._create_client https://review.openstack.org/624770 | 20:35 |
| openstackgerrit | Merged openstack/nova stable/pike: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623510 | 20:35 |
| mriedem | yeah ok maybe that won't work then | 20:36 |
| mriedem | but i tried! | 20:36 |
| *** priteau has quit IRC | 20:37 | |
| *** wolverineav has quit IRC | 20:43 | |
| *** wolverineav has joined #openstack-nova | 20:44 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec to implement os-vif generic datapath offloads https://review.openstack.org/607610 | 20:44 |
| *** dtrainor has quit IRC | 20:45 | |
| *** wolverineav has quit IRC | 20:49 | |
| *** betherly has joined #openstack-nova | 20:50 | |
| *** tbachman has quit IRC | 20:50 | |
| *** betherly has quit IRC | 20:55 | |
| *** wolverineav has joined #openstack-nova | 20:58 | |
| *** pbandark has joined #openstack-nova | 20:59 | |
| *** mchlumsky has quit IRC | 21:01 | |
| pbandark | hi.. i am facing issue while creating instance on openstack. the instance is getting created. but, when I check the console logs of the vm, i can see kernel panic. https://pastebin.com/dZ2uTuFa any suggestion would be really helpful. | 21:02 |
| *** ttsiouts has joined #openstack-nova | 21:03 | |
| openstackgerrit | Merged openstack/nova-specs master: Spec to implement os-vif generic datapath offloads https://review.openstack.org/607610 | 21:07 |
| *** aloga has quit IRC | 21:11 | |
| *** aloga has joined #openstack-nova | 21:11 | |
| *** jobewan has quit IRC | 21:16 | |
| *** dklyle has joined #openstack-nova | 21:27 | |
| *** dklyle has quit IRC | 21:32 | |
| *** dklyle has joined #openstack-nova | 21:33 | |
| *** mchlumsky has joined #openstack-nova | 21:35 | |
| *** N3l1x has quit IRC | 21:38 | |
| openstackgerrit | Ben Nemec proposed openstack/nova master: Migrate upgrade checks to oslo.upgradecheck https://review.openstack.org/603499 | 21:46 |
| mriedem | beekneemech: heh forgot about the functional test | 21:51 |
| mriedem | dansmith: replied to your replies and new questions in the cross-cell resize spec https://review.openstack.org/#/c/616037/ - good things in there again, thanks. good news is since i wrote that, i think quite a bit of this is at least more clear, so some of the open questions are closing, which is nice. | 21:52 |
| beekneemech | mriedem: Yeah, looks like it just got moved so it wouldn't have shown up in the conflict list. | 21:52 |
| mriedem | dansmith: so you can exit for the year knowing it's sort of coming together | 21:52 |
| mriedem | probably hairiest issue is how much existing code do we want to hack vs just writing something entirely new from scratch | 21:53 |
| *** mchlumsky has quit IRC | 21:53 | |
| mriedem | i'm not sure i'll have the energy to poc an alternative next week, but maybe | 21:53 |
| mriedem | i have already split a bunch of shit out | 21:53 |
| *** rodolof has joined #openstack-nova | 21:54 | |
| *** dklyle has quit IRC | 21:54 | |
| *** igordc has joined #openstack-nova | 21:56 | |
| *** dklyle has joined #openstack-nova | 21:56 | |
| *** ttsiouts has quit IRC | 22:00 | |
| mriedem | calling it quits o/ | 22:01 |
| *** mriedem has quit IRC | 22:01 | |
| *** dklyle has quit IRC | 22:02 | |
| *** awaugama has quit IRC | 22:03 | |
| *** ttsiouts has joined #openstack-nova | 22:07 | |
| *** igordc has quit IRC | 22:08 | |
| *** igordc has joined #openstack-nova | 22:10 | |
| *** dklyle has joined #openstack-nova | 22:13 | |
| *** betherly has joined #openstack-nova | 22:14 | |
| *** k_mouza has joined #openstack-nova | 22:15 | |
| *** dklyle has quit IRC | 22:18 | |
| *** rodolof has quit IRC | 22:18 | |
| *** betherly has quit IRC | 22:18 | |
| *** ttsiouts has quit IRC | 22:19 | |
| *** k_mouza has quit IRC | 22:23 | |
| *** betherly has joined #openstack-nova | 22:24 | |
| *** betherly has quit IRC | 22:29 | |
| *** ivve has quit IRC | 22:29 | |
| *** slaweq has quit IRC | 22:33 | |
| *** dklyle has joined #openstack-nova | 22:37 | |
| *** dklyle has quit IRC | 22:41 | |
| *** dklyle has joined #openstack-nova | 22:42 | |
| *** wolverineav has quit IRC | 22:51 | |
| openstackgerrit | Krzysztof Opasiak proposed openstack/nova master: Fix server IPs with non-unique network names https://review.openstack.org/625371 | 22:53 |
| *** macza has quit IRC | 22:56 | |
| *** dklyle has quit IRC | 22:57 | |
| *** wolverineav has joined #openstack-nova | 22:59 | |
| *** wolverineav has quit IRC | 23:03 | |
| *** wolverineav has joined #openstack-nova | 23:06 | |
| *** k_mouza has joined #openstack-nova | 23:07 | |
| *** lbragstad has quit IRC | 23:08 | |
| *** k_mouza has quit IRC | 23:08 | |
| *** lbragstad has joined #openstack-nova | 23:17 | |
| *** lbragstad has quit IRC | 23:22 | |
| openstackgerrit | Merged openstack/nova master: Remove GROUP BY clause from CellMapping.get_by_project_id https://review.openstack.org/619061 | 23:30 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
