| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-02 | |||
| 08:41:53 | Spazmotic | Oi... | |
| 08:41:58 | Spazmotic | not you.. I take back my morning to you! | |
| 08:42:12 | Spazmotic | :) | |
| 08:42:48 | gibi | happy Friday everyone :) | |
| 08:42:55 | Spazmotic | Morning gibi | |
| 08:43:54 | Spazmotic | If anyone gets some time to review this today I would appreciate it. https://review.openstack.org/#/c/538415/ Would like to try to pack this one away before I leave for my trip | |
| 08:45:29 | Spazmotic | johnthetubaguy if you're working today and have time, that is the one that we discussed before, with live migration assert_can_migrate | |
| 08:56:00 | ameeda | https://review.openstack.org/#/c/526900/28/nova/db/sqlalchemy/migrate_repo/versions/378_instance_system_metadata_sqlite_upgrade.sql | |
| 08:56:00 | ameeda | cloud you please help me to figure out this issue https://review.openstack.org/#/c/526900/ ? the error of functional test appears after I added | |
| 09:19:26 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161 | |
| 09:43:29 | gibi | edleafe: thanks for simplification in https://review.openstack.org/#/c/539323 , LGTM | |
| 09:48:06 | openstackgerrit | James E. Blair proposed openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326 | |
| 09:55:21 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/540073 | |
| 09:55:52 | openstackgerrit | Merged openstack/nova stable/ocata: Stop _undefine_domain erroring if domain not found https://review.openstack.org/508640 | |
| 09:56:00 | openstackgerrit | Merged openstack/nova master: doc: placement upgrade notes for queens https://review.openstack.org/539658 | |
| 09:59:03 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501 | |
| 10:18:11 | lyarwood | mdbooth / kashyap ; would you mind taking a look at the following LM tempest changes regarding encrypted volumes - https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/libvirt-qemu-native-luks | |
| 10:18:53 | kashyap | lyarwood: Hiya. Racing to do the slides for FOSDEM tomm :-( | |
| 10:18:57 | lyarwood | just want to ensure I'm not confusing live migration, live block migration, live migration with volumes attached etc | |
| 10:19:05 | lyarwood | k np | |
| 10:19:06 | kashyap | Sure, will look | |
| 10:19:09 | ildikov | gibi: hi :) | |
| 10:19:13 | mdbooth | lyarwood: Sure | |
| 10:19:22 | kashyap | But Monday I'll have full concentration back here | |
| 10:19:36 | ildikov | gibi: if you have a minute this multi-attach related change needs one more +2: https://review.openstack.org/#/c/532689/ :) | |
| 10:19:54 | Spazmotic | Opened page for FOSDEM and thought it said Bruges | |
| 10:20:19 | lyarwood | urgh so the cleanup duplicates the above test, sorry missed that | |
| 10:20:52 | lyarwood | ah no it doesn't, think I might need more coffee | |
| 10:20:56 | lyarwood | brb | |
| 10:22:31 | kashyap | Spazmotic: It is Brussels, not Bruges :-) | |
| 10:22:47 | Spazmotic | Hehe yeah I saw that after.. but made me audible go, "In Bruges!?" | |
| 10:49:08 | andreaf | mriedem_parent, ildikov: wow, https://review.openstack.org/#/c/266605/ merged! 266605! I didn't see a number in the 200k in a while :] | |
| 10:49:29 | andreaf | it surely is a pain to merge a patch in Tempest :P | |
| 10:50:12 | gibi | ildikov: hello :) I will try to look at it before end of today | |
| 10:59:43 | ildikov | andreaf: lol, I would say cross-project work has its challenges :) | |
| 11:00:01 | ildikov | andreaf: I mean overall | |
| 11:00:22 | ildikov | gibi: sounds good, thanks :) | |
| 11:01:03 | ildikov | gibi: the patch is small, but an important one for testing the new multi-attach stuff constantly | |
| 11:01:08 | andreaf | ildikov yes especially if QA is one of the projects :D | |
| 11:01:35 | ildikov | andreaf: that was the easy part this time :D | |
| 11:01:45 | Spazmotic | Ugh just found a more simple way to do my own commit | |
| 11:01:48 | Spazmotic | damn you eyes | |
| 11:04:16 | openstackgerrit | Chris Dent proposed openstack/nova master: Remove microversion fallback code from report client https://review.openstack.org/528794 | |
| 11:10:27 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 11:18:25 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049 | |
| 12:17:34 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 12:45:36 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes https://review.openstack.org/536350 | |
| 13:09:25 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501 | |
| 13:19:17 | Spazmotic | giblet always makes me think of a tiny pig. | |
| 13:19:23 | Spazmotic | like real tiny, like ant sized | |
| 13:20:33 | efried | @/ | |
| 13:21:58 | giblet | Spazmotic: this giblet is full of birthday cake today so it is not so small of a pig now | |
| 13:22:16 | Spazmotic | Birthday pig.. sounds delicious. | |
| 13:22:39 | Spazmotic | Happy birthday man, if it is your own cake :) | |
| 13:22:40 | giblet | the cake was delicious too. :) | |
| 13:22:57 | giblet | Spazmotic: Yes, it is. Thank you | |
| 13:23:24 | cdent | happy birthday giblet | |
| 13:24:31 | giblet | cdent: thanks | |
| 13:25:13 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179 | |
| 13:26:03 | efried | Wow, is it Friday already?? | |
| 13:30:44 | Spazmotic | That spec made my eyes fall out of my head so I assume it's super good | |
| 13:32:30 | fried_rice | Which one? | |
| 13:32:50 | Spazmotic | Yours.. impressive I feel dumber for even being too close to it.. I'll save that for when I want to make one later hehe | |
| 13:34:17 | fried_rice | Spazmotic: I wouldn't model your bp off of that one. *Some* folks thought it was too long. | |
| 13:38:12 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049 | |
| 13:57:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test https://review.openstack.org/540386 | |
| 13:59:13 | giblet | edleafe: I figured out how can we ensure that no new UUID validation warning is merged to nova: https://review.openstack.org/#/c/540386 | |
| 14:08:35 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179 | |
| 14:11:42 | mriedem | lyarwood: mdbooth: can you comment on melwitt's question here https://review.openstack.org/#/c/539739/2/nova/virt/libvirt/driver.py@7430 | |
| 14:11:43 | openstackgerrit | Eric Fried proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712 | |
| 14:11:51 | mriedem | i don't see anything ever setting connection_info['volume_id'] | |
| 14:12:44 | figleaf | giblet: nice | |
| 14:13:08 | mdbooth | mriedem: When I was working on an HLR for a telco many, many moons ago, we'd have called that sort of request tromboning :) | |
| 14:13:27 | figleaf | giblet: working on fixing the merge conflict | |
| 14:13:59 | mriedem | mdbooth: ok i don't know what that means | |
| 14:14:10 | ildikov | mriedem: when we did the connection_info translation for the new attach API we didn't explicitly add the volume_id field to 'data' there so it doesn't seem to come from Cinder either | |
| 14:14:18 | lyarwood | mriedem: yup looking now | |
| 14:15:33 | mdbooth | mriedem: It was a request sent externally only to be bounced back to an internal source :) | |
| 14:16:16 | mdbooth | mriedem: Ah, that's interesting. I made some comments about that in review. | |
| 14:16:23 | mriedem | mdbooth: ok? i asked you guys because you were reviewing/working on the luks patch that i'm trying to fix. | |
| 14:16:54 | mdbooth | mriedem: Sorry, IRC failure. My mind was just wandering... | |
| 14:17:12 | mdbooth | mriedem: So I don't think we should be fetching stuff out of connection_info at all | |
| 14:17:27 | mriedem | this line specifically can be using bdm.volume_id | |
| 14:17:33 | mdbooth | I let it pass in review because I believed we always set volume_id in practise | |
| 14:17:38 | mdbooth | in connection_info | |
| 14:17:40 | mriedem | i was just asking if there was some specific logic as to why it was using connection_info | |
| 14:17:52 | lyarwood | mriedem: vol['id'] no? | |
| 14:18:10 | cdent | fried_rice: you'll probably appreciate the fun of https://bugs.launchpad.net/nova/+bug/1747001 | |
| 14:18:11 | openstack | Launchpad bug 1747001 in OpenStack Compute (nova) "Use of parse.urlencode with dict in nova/tests/unit/scheduler/client/test_report.py can result in unpredictable query strings and thus unreliable tests" [Low,Triaged] | |
| 14:18:13 | mdbooth | No, it was only out of expediency to avoid a larger refactor | |
| 14:18:14 | lyarwood | mriedem: we don't set the id on these LibvirtLiveMigrateBDMInfo objects | |
| 14:18:22 | mriedem | lyarwood: vol['id'] would be the primary key of the bdm | |
| 14:18:41 | mriedem | block_device_info) | |
| 14:18:41 | mriedem | block_device_mapping = driver.block_device_info_get_mapping( | |
| 14:18:46 | mdbooth | Although not *much* larger... | |
| 14:18:48 | lyarwood | right sorry, volume_id | |
| 14:19:01 | mriedem | so i'll fix in my change, good thing mel noticed it, | |
| 14:19:11 | mriedem | was just wondering if there was any good reason behind this being here, and i guess there isn't | |
| 14:19:14 | lyarwood | that and the detach lookup | |
| 14:19:36 | fried_rice | cdent: Appreciating the fun... | |