| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-29 | |||
| 09:29:04 | openstackgerrit | Magnus Bergman proposed openstack/nova master: Fix bug preventing forbidden traits from working https://review.openstack.org/648653 | |
| 09:31:19 | trident | gibi: I hope I got it right with the commits in series. | |
| 09:32:17 | gibi | trident: based on a quick look in gerrit you put the patches in a good order. I will review both | |
| 09:32:30 | trident | Great, thanks! | |
| 09:48:07 | gibi | trident: left comments in https://review.openstack.org/#/c/647957 | |
| 09:50:13 | gibi | trident: some formal changes needed but the test your created seems good to me | |
| 10:03:17 | trident | gibi: Thanks! Will look into it shortly! | |
| 10:09:59 | openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Support adding the reason behind a server lock https://review.openstack.org/638629 | |
| 10:23:29 | openstackgerrit | sean mooney proposed openstack/nova master: make lower-constratints env use lower-constratins https://review.openstack.org/645392 | |
| 10:38:33 | openstackgerrit | Surya Seetharaman proposed openstack/python-novaclient master: [WIP] Support adding the reason behind a server lock https://review.openstack.org/648659 | |
| 11:00:08 | tssurya | stephenfin: any idea on how to fix http://logs.openstack.org/29/638629/7/check/openstack-tox-docs/f14d940/job-output.txt.gz#_2019-03-29_10_28_35_792167 , seems like the docs CI is failing..not sure if there is a bug already | |
| 11:07:35 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Support adding the reason behind a server lock https://review.openstack.org/648662 | |
| 11:08:08 | openstackgerrit | Merged openstack/nova master: Error out migration when confirm_resize fails https://review.openstack.org/647546 | |
| 11:12:41 | openstackgerrit | sean mooney proposed openstack/nova master: make lower-constratints env use lower-constratins https://review.openstack.org/645392 | |
| 11:17:19 | openstackgerrit | sean mooney proposed openstack/nova master: make lower-constratints env use lower-constratins https://review.openstack.org/645392 | |
| 11:19:20 | sean-k-mooney | by the way i have confimed that cdent's version works too https://review.openstack.org/#/c/622972/14 | |
| 11:20:06 | sean-k-mooney | so im fine with abandoning my version in favor of his if people prefer but we should merge one of the two | |
| 11:25:40 | stephenfin | tssurya: \o/ Looks like Sphinx 2.0 has broken some stuff | |
| 11:25:45 | stephenfin | I think that got released today | |
| 11:26:15 | stephenfin | tssurya: What repo is that against? I didn't think we were still using oslosphinx anywhere | |
| 11:29:15 | sean-k-mooney | launchpad is broken... | |
| 11:30:04 | sean-k-mooney | i should just never file bugs buzilla was broken when i tried yesterday too. | |
| 11:32:03 | openstackgerrit | sean mooney proposed openstack/nova master: make lower-constraints env use lower-constraints https://review.openstack.org/645392 | |
| 11:35:34 | openstackgerrit | sean mooney proposed openstack/nova master: make lower-constraints env use lower-constraints https://review.openstack.org/645392 | |
| 11:39:36 | openstackgerrit | Helena proposed openstack/nova-specs master: Spec for a new nova virt driver to manage an RSD, composable infrastructure deployment https://review.openstack.org/648665 | |
| 11:50:59 | stephenfin | tssurya: https://review.openstack.org/648666 | |
| 12:11:03 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: docs: Remove cruft from 'conf.py' https://review.openstack.org/648670 | |
| 12:11:04 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: docs: Fix multiple issues with the 'redirect' extension https://review.openstack.org/648671 | |
| 12:11:05 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: docs: Switch to openstackdocstheme https://review.openstack.org/648672 | |
| 12:11:27 | stephenfin | gibi, bauzas: Can you look at those to unblock tssurya/anyone else proposing specs ^ | |
| 12:11:49 | stephenfin | (https://review.openstack.org/648666 might be helpful for additional context) | |
| 12:19:45 | gibi | stephenfin: on a meeting I will look at it latae | |
| 12:19:47 | gibi | later | |
| 12:49:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Only call _fill_provider_mapping if claim succeeds https://review.openstack.org/648676 | |
| 13:06:04 | elbragstad | gmann nice - thanks for those | |
| 13:09:15 | fried_rice | gibi: Earlier mlavalle asked about https://bugs.launchpad.net/neutron/+bug/1821058 / https://review.openstack.org/#/c/645173/ | |
| 13:09:16 | openstack | Launchpad bug 1821058 in neutron "[RFE] Port binding event extended information for Nova" [Wishlist,Confirmed] | |
| 13:09:16 | fried_rice | I can see you're already reviewing the latter. | |
| 13:09:16 | fried_rice | Any initial thoughts on this work? | |
| 13:11:46 | gibi | fried_rice: my position is that the problems listed in the spec are valid, however it is not clear to me how the proposed solution will impact nova (and os-vif) especially how much dependency will be added between nova and the neutron backends | |
| 13:12:11 | gibi | fried_rice: but I haven't read sean-k-mooney's reply in the spec yet | |
| 13:12:37 | fried_rice | gibi: Okay. I suppose it will be something for the xproj session at the ptg. But obviously the more we can have resolved before then, the better. | |
| 13:12:53 | gibi | fried_rice: totally agree | |
| 13:13:01 | gibi | fried_rice: this will be a cross project effort | |
| 13:13:36 | sean-k-mooney | fried_rice: so i have been putting of writing the nova side of the spec abit untill i get a vague yes or no form the neutron side | |
| 13:14:01 | fried_rice | sean-k-mooney: At some point we'll want it tied to a bp | |
| 13:14:02 | sean-k-mooney | there are likely other ways to adress the issues i that were raised | |
| 13:14:15 | fried_rice | but I guess there's no point in filing one of those if we're still not sure we want to do it at all. | |
| 13:14:25 | sean-k-mooney | fried_rice: i can file that now if you like. | |
| 13:14:38 | fried_rice | sean-k-mooney: Or it doesn't exist :P | |
| 13:15:00 | sean-k-mooney | well i can file a blueprint for the geneal problem of fix neutron event handleing | |
| 13:15:20 | sean-k-mooney | the solution is still open | |
| 13:15:38 | fried_rice | sean-k-mooney: it will require *something* in nova? | |
| 13:18:07 | tssurya | stephenfin: thanks for the fix ! | |
| 13:20:09 | sean-k-mooney | fried_rice: yes | |
| 13:21:05 | fried_rice | sean-k-mooney: Then yes please, let's get a bp on the docket | |
| 13:21:32 | sean-k-mooney | fried_rice: put simplely if we want to ensure that networking is configured correclty for all instance lifecycle operations and not just spawn/power on then we need to wait for the netwrok events in several placees in the code | |
| 13:21:52 | fried_rice | several places in the nova code | |
| 13:21:55 | sean-k-mooney | fried_rice: we cant today simpley because we dont know what neutron backend is being used and if it will send them | |
| 13:22:04 | sean-k-mooney | fried_rice: its all in the virt dirvres | |
| 13:22:34 | fried_rice | interesting problem. | |
| 13:23:10 | sean-k-mooney | interesting proablem i have been trinig to fix for 4 years... but yes :) | |
| 13:24:15 | sean-k-mooney | fried_rice: we have some comments like https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2846-L2855 in the code where we know we would like to wait but cant | |
| 13:24:51 | sean-k-mooney | but https://bugs.launchpad.net/nova/+bug/1813789 is also jsut another sideffect of the same large problem | |
| 13:24:53 | openstack | Launchpad bug 1813789 in OpenStack Compute (nova) "Evacuate test intermittently fails with network-vif-plugged timeout exception" [Medium,In progress] - Assigned to Artom Lifshitz (notartom) | |
| 13:35:38 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/648686 | |
| 13:36:15 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/648687 | |
| 13:37:04 | alex_xu | gibi: yea, I'm ok with another fix | |
| 13:38:12 | gibi | alex_xu: cool, I pushed the test fix below the actual bugfix | |
| 13:38:42 | alex_xu | gibi: thanks, checking now | |
| 13:38:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete allocations even if _confirm_resize raises https://review.openstack.org/647566 | |
| 13:38:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add more details to confirmResize troubleshooting https://review.openstack.org/647741 | |
| 13:39:25 | gibi | stephenfin: did you missed a depends-on in https://review.openstack.org/#/c/648670/ ? | |
| 13:39:36 | gibi | stephenfin: the doc job fails | |
| 13:41:13 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Temporarily mutate migration object in finish_revert_resize https://review.openstack.org/648688 | |
| 13:44:00 | stephenfin | gibi: Nope, just saw that. It's _another_ dependency that's borked (yasfb) | |
| 13:44:06 | stephenfin | working on a fix for that now | |
| 13:45:38 | gibi | stephenfin: ack | |
| 13:48:25 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Temporarily mutate migration object in finish_revert_resize https://review.openstack.org/648691 | |
| 13:54:54 | bauzas | finucannot: the change is broken, right? https://review.openstack.org/#/c/648670/1 | |
| 13:55:02 | bauzas | oh, nevermind, haven't seen above | |
| 13:55:05 | finucannot | bauzas: See above :) | |
| 13:56:37 | bauzas | finucannot: yeah I'm tired | |
| 13:57:16 | bauzas | and I'm torn in between using PapaOurs or bauwser for my Friday nick | |
| 13:57:24 | bauzas | people should vote | |
| 13:57:53 | artom | Because of the implication | |
| 14:06:09 | bauzas | temka: DaddyBear has many more implications than PapaOurs | |
| 14:06:22 | temka | bauzas, quite | |
| 14:07:29 | bauzas | hum, DaddyBear isn't registered | |
| 14:08:13 | temka | They tend to be on a different sort of register | |
| 14:08:16 | temka | (I'm so sorry) | |
| 14:09:59 | DaddyBear | there, registered | |
| 14:10:12 | DaddyBear | but I'm still the real PapaOurs | |
| 14:18:37 | Zara | had to check what channel I was in for a sec, there | |
| 14:18:48 | dansmith | heh | |
| 14:18:56 | dansmith | "different sort of register" .. that was a good one | |
| 14:19:10 | dansmith | I normally don't like to encourage temka but that one was worth it | |
| 14:19:14 | cdent | temka++ | |
| 14:19:29 | temka | I said I was sorry! | |
| 14:34:38 | edleafe | temka: I'm just glad I wasn't the only one who thought that! | |