| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-29 | |||
| 08:38:25 | gibi | alex_xu: answered in https://review.openstack.org/#/c/639608 | |
| 08:38:57 | gibi | alex_xu: instead of revert and re-add the functional reproduce can I just push a patch that changes what exception we throw in the functiional test? | |
| 09:10:50 | openstackgerrit | Magnus Bergman proposed openstack/nova master: Adding tests to demonstrate bug #1821824 https://review.openstack.org/647957 | |
| 09:10:51 | openstack | bug 1821824 in OpenStack Compute (nova) "Forbidden traits in flavor properties don't work" [Undecided,In progress] https://launchpad.net/bugs/1821824 - Assigned to Magnus Bergman (magnusbe) | |
| 09:13:05 | trident | gibi: There. Got stuck in some other work yesterday, but here it is. First commit with new functional and unit tests demonstrating the forbidden traits issue. Next commit will add the fix as well. | |
| 09:14:31 | gibi | trident: ack. I will look at it shortly | |
| 09:23:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle placement error during re-schedule https://review.openstack.org/639608 | |
| 09:23:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix exception type in test_boot_reschedule_fill_provider_mapping_raises https://review.openstack.org/648651 | |
| 09:25:06 | gibi | alex_xu: ^^ | |
| 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) | |