| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-05 | |||
| 13:55:07 | mdbooth | An alternate idea would be the ability to mark a -1 as 'I looked at that, and have no intention of addressing it'. | |
| 13:55:30 | mdbooth | Then people might filter on unaddressed -1s instead. | |
| 13:56:05 | mhenkel | stephenfin: sorry, wrong channel | |
| 13:57:53 | mdbooth | artom: Meta reviews would be good, though. | |
| 13:58:00 | artom | mdbooth, also, I suspect just showing who the -1 is from would help that | |
| 13:58:05 | mdbooth | Although, who would do them? | |
| 14:00:07 | stephenfin | mdbooth, artom: If you're using the dashboards from gerrit-dash-creator, they shouldn't matter. The only -1s used to filter stuff there are core ones https://github.com/openstack/gerrit-dash-creator/tree/master/dashboards | |
| 14:00:14 | stephenfin | Just as an aside | |
| 14:00:27 | mdbooth | stephenfin: Ah, that's awesome | |
| 14:00:59 | mdbooth | Well, somewhat elitist, but better that than exclude based on drive-bys | |
| 14:01:02 | mdbooth | ;) | |
| 14:01:25 | stephenfin | Elitist, but the best we've got ;) | |
| 14:03:58 | openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | |
| 14:06:30 | hrw | speaking of reviews... anyone with free time to take a look at two simple aarch64 related patches? https://review.openstack.org/#/c/530965/ takes care of cpu_mode and https://review.openstack.org/#/c/489951/ makes aarch64 bootable out-of-the-box | |
| 14:16:53 | mriedem | kashyap: bad news, the shareable disk flag doesn't work when using raw images either http://paste.openstack.org/show/639185/ | |
| 14:21:50 | efried | sean-k-mooney Howdy. That's the top patch in *one* of the nrp series, yes. | |
| 14:25:34 | ildikov | mriedem: kashyap: :( | |
| 14:25:36 | openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381 | |
| 14:25:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add debug output for selected page size https://review.openstack.org/530662 | |
| 14:26:12 | ildikov | mriedem: which versions gave you that error? | |
| 14:28:58 | ildikov | mriedem: I also saw this 'qcow2 would suffer metadata corruption in such case' in the latest comment of this bug: https://bugzilla.redhat.com/show_bug.cgi?id=1415250 | |
| 14:28:59 | openstack | bugzilla.redhat.com bug 1415250 in openstack-nova "QEMU image file locking (RHOS)" [High,Post] - Assigned to lyarwood | |
| 14:32:05 | stephenfin | mdbooth: What's the name of the pattern where you create an instance of object() and use that as a unique identifier? | |
| 14:33:03 | mdbooth | stephenfin: I don't know. I stole it from zzzeek | |
| 14:33:41 | mdbooth | It's a sentinel, though, so that might be what it's called? | |
| 14:34:18 | stephenfin | Yup, sentinel is the word I was looking for | |
| 14:38:59 | mriedem | ildikov: this is still qemu 2.10 and libvirt 3.6 | |
| 14:39:36 | mriedem | kashyap was saying the libvirt devs were saying that qcow2 wouldn't work before libvirt 3.10 for shareable disks but raw should work, so this test run is setting CONF.libvirt.images_type=raw and CONF.use_cow_images=False | |
| 14:39:38 | mriedem | but still doesn't work | |
| 14:40:43 | mnaser | did i hit a really weird race condition possibly...? "Conflict updating instance e8cc3ace-2441-413e-bee7-d3a667e3298f. Expected: {'task_state': [u'migrating']}. Actual: {'task_state': None}" | |
| 14:41:13 | mriedem | possibly, might depend on what it's trying to change the task_state to | |
| 14:41:34 | mnaser | reset-state --active and doing a live-migration again triggers it.. let me check | |
| 14:41:59 | Roamer` | okay, so I'll try to also reach johnthetubaguy through a couple of other channels, but maybe I should also try to start a conversation here... What do people think about adding a pretty much trivial StorPool volume attachment driver as in https://review.openstack.org/140733/ ? I know that milestone 1 has passed and we're actually getting pretty close to milestone 2, but the Cinder and os-brick dri | |
| 14:42:05 | Roamer` | vers made it in last week, so right now we're in a funny situation where the main use case for the StorPool driver still needs manual patching. There's actually a major downstream OpenStack distributor with a client that wants to use StorPool, and the distributor would strongly prefer if all three StorPool drivers made it into OpenStack... so is there a chance for that? :) | |
| 14:42:14 | mdbooth | mriedem: I don't think those config params would affect volumes anyway fwiw. | |
| 14:43:06 | mriedem | was wondering that, | |
| 14:43:15 | mriedem | i guess those are only used for ephemeral imagebackend devices | |
| 14:43:21 | mdbooth | Yep | |
| 14:43:23 | mriedem | ok | |
| 14:43:23 | mriedem | gah | |
| 14:43:38 | mriedem | alright, trying again w/o pike uca so we use qemu 2.5 | |
| 14:45:09 | mriedem | yikun: i think we should probably just add uuid to the response for the server migrations API in this change as well https://review.openstack.org/#/c/330406/ - is that something you think you'll have time for? if not, i can add that part of the change | |
| 14:50:58 | mlavalle | mriedem, hongbin: https://bugs.launchpad.net/neutron/+bug/1718605 is approved | |
| 14:50:59 | openstack | Launchpad bug 1718605 in neutron "[RFE] Support sub-string matching when filtering port with IP address" [Wishlist,Triaged] - Assigned to hongbin (hongbin034) | |
| 14:51:30 | hongbin | mlavalle: great. thanks | |
| 14:55:31 | ildikov | mriedem: got it, sadness, I guess it means we're falling back to play with the versions in Devstack then | |
| 14:56:28 | mriedem | ildikov: yeah | |
| 14:56:38 | mriedem | mlavalle: nice | |
| 14:57:59 | fried_rice | figleaf What's the status of the alternate hosts series (for purposes of the sched mtg agenda)? | |
| 14:58:06 | fried_rice | figleaf Are there open patches? | |
| 14:58:17 | fried_rice | figleaf Or just that bug? | |
| 15:01:02 | mriedem | fried_rice: i can tell you | |
| 15:01:11 | fried_rice | Tell me, tell me. | |
| 15:01:19 | mriedem | resize is the last piece https://review.openstack.org/#/c/526436/ | |
| 15:01:29 | mriedem | but while figleaf was testing that, he found a race | |
| 15:01:35 | mriedem | so he's working on fixing that race | |
| 15:01:42 | mriedem | https://bugs.launchpad.net/nova/+bug/1741125 | |
| 15:01:43 | openstack | Launchpad bug 1741125 in OpenStack Compute (nova) "Instance resize always fails when rescheduling" [High,Confirmed] - Assigned to Ed Leafe (ed-leafe) | |
| 15:03:21 | fried_rice | mriedem Thanks. Is the forked bug https://bugs.launchpad.net/nova/+bug/1741307 also related to alt hosts? | |
| 15:03:22 | openstack | Launchpad bug 1741307 in OpenStack Compute (nova) "Resize always fails when using the CachingScheduler" [High,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 15:03:47 | mriedem | fried_rice: not to alternate hosts, just something i noticed when he pointed out the reschedule issue with resize | |
| 15:03:57 | fried_rice | cool, thanks mriedem | |
| 15:04:00 | mriedem | resize just doesn't work with the caching scheduler, period | |
| 15:04:03 | melwitt | mriedem: do you think this libvirt patch has had enough bake time to propose backports now? https://review.openstack.org/#/c/400384/ | |
| 15:04:06 | mriedem | because of the migration allocation swap stuff | |
| 15:04:31 | mriedem | melwitt: heh, can you give it about another 18 months for us to know? | |
| 15:04:32 | mriedem | :) | |
| 15:04:41 | melwitt | D: | |
| 15:05:02 | mriedem | melwitt: honestly we probably won't know until people actually upgrade to start using that, and mnaser will probably be the first one once queens is released | |
| 15:05:17 | mriedem | melwitt: i'm fine with doing backports though | |
| 15:05:20 | mriedem | just not to newton | |
| 15:06:12 | mriedem | ildikov: the devstack patch is now just enablement https://review.openstack.org/#/c/266633/ | |
| 15:06:18 | mnaser | i'll gladly take release notes for us :D | |
| 15:06:19 | mriedem | the patch on top of that turns on multiattach and disables the pike uca | |
| 15:07:18 | melwitt | mriedem: okay, cool. I shall propose them then. thanks | |
| 15:13:45 | mnaser | if there's any stable cores that can have a look at this backport: https://review.openstack.org/#/c/529385 | |
| 15:17:40 | openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381 | |
| 15:17:41 | mriedem | dansmith: ^ | |
| 15:17:49 | mriedem | the stable thing, not the notificatoins thing | |
| 15:17:49 | dansmith | mriedem: dun already got it | |
| 15:17:55 | mriedem | sympatico | |
| 15:17:56 | dansmith | bottom one needs a recheck? | |
| 15:18:10 | lyarwood | it already has | |
| 15:18:13 | dansmith | ah nm, | |
| 15:18:14 | dansmith | yeah | |
| 15:18:17 | lyarwood | should be in the gate by now | |
| 15:18:26 | dansmith | not quite | |
| 15:18:27 | mriedem | still check queue, but going | |
| 15:18:35 | lyarwood | urgh | |
| 15:18:52 | mriedem | dansmith: btw, i had to fix a test in https://review.openstack.org/#/c/530982/ | |
| 15:19:34 | mriedem | lyarwood: since you're around https://review.openstack.org/#/q/I1aa3ca6cc70cef65d24dec1e7db9491c9b73f7ab | |
| 15:19:40 | mriedem | that should be the final thing for newton | |
| 15:20:04 | mriedem | melwitt: this is a simple bp driver feature parity change https://review.openstack.org/#/c/448034/ | |
| 15:20:08 | mriedem | needs final +2 | |
| 15:20:12 | lyarwood | mriedem: ack | |
| 15:20:36 | melwitt | mriedem: ack, will review | |
| 15:20:43 | mriedem | thanks | |
| 15:21:08 | mriedem | i see the allocation candidates limit change is also approved https://review.openstack.org/#/c/513526/ - nice, might get at least 2 bps merged today | |
| 15:21:28 | openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate service status notification samples https://review.openstack.org/531381 | |
| 15:22:35 | mriedem | andreykurilin: can you take a look at https://review.openstack.org/#/c/528128/ ? it will close out that bp and it's holding up other microversion changes in novaclient | |