| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-06 | |||
| 17:08:16 | artom | It happens a lot | |
| 17:08:18 | mriedem | since it's just hitting APIs, | |
| 17:08:40 | artom | OOOIC | |
| 17:08:44 | mriedem | so while unskipping it could be ok in the gate b/c of how most of the devstack jobs are setup (ovs w/o hybrid plug or lb), | |
| 17:08:51 | artom | Upstream gate would be fine, but we'd break someone else who runs it with hybrid plug | |
| 17:08:54 | mriedem | that doesn't mean someone, like your downstream ci, could break on it | |
| 17:09:02 | mriedem | correct | |
| 17:09:11 | mriedem | i'm basically convincing myself on your behalf | |
| 17:09:12 | artom | Our downstream CI is permanently on fire anyways | |
| 17:09:17 | mriedem | b/c of tripleo | |
| 17:09:18 | mriedem | sure | |
| 17:09:19 | mriedem | :) | |
| 17:09:20 | artom | (Did I say that out loud? It's not true, obviously) | |
| 17:09:59 | sean-k-mooney | sure... | |
| 17:10:44 | artom | mriedem, what I selfishly *don't* want to do is have my fix depend on your tempest patch | |
| 17:11:13 | artom | Because your tempest patch might be stuck in recheck hell for a while, and I'm in a *massive* hurry ;) | |
| 17:11:40 | artom | But... I think you made the case for me that it's better the other way around anyways | |
| 17:11:48 | efried | kashyap, mriedem: https://blueprints.launchpad.net/nova/+spec/pick-guest-arch-based-on-host-arch-in-libvirt-driver for train? | |
| 17:12:12 | sean-k-mooney | efried: proably not | |
| 17:12:20 | mriedem | efried: no | |
| 17:12:25 | efried | ight, thx | |
| 17:12:31 | mriedem | imo | |
| 17:12:43 | efried | just wanted to tag series goal if so, so it shows up on dashboard things. | |
| 17:12:50 | efried | NOT tagging. | |
| 17:13:06 | sean-k-mooney | do we have a NEXT tag or somthing we can use | |
| 17:13:08 | mriedem | artom: i have a hard time knowing what your team considers a priority ever really... | |
| 17:13:10 | sean-k-mooney | e.g. after train | |
| 17:13:27 | mriedem | until the boss (mdbooth) shows up asking for reviews | |
| 17:13:49 | sean-k-mooney | mriedem: our relase folk considers it a blocker for the osp 15 beta | |
| 17:13:56 | artom | mriedem, speaking for myself I try not to push that angle too much, because what's important for upstream isn't necessarily what's important for us | |
| 17:14:07 | artom | Or vice versa | |
| 17:14:23 | mriedem | sean-k-mooney: did those people never realize this has always been broken? | |
| 17:14:29 | sean-k-mooney | but its a beta so its not going to break production deployments | |
| 17:14:36 | artom | I just feel weird saying "well this thing is super important for *us*, so dump everything else and work on my thing" | |
| 17:14:41 | mriedem | unless you guys just changed to hybrid plug | |
| 17:14:55 | artom | But if you'd rather I just be up front about it, I can do that too | |
| 17:15:01 | mriedem | btw, whatever happened with russell ovn'ing the world at rh? | |
| 17:15:03 | sean-k-mooney | mriedem: its because its showing up in the downstream ci and that makes them unhappy | |
| 17:15:11 | mriedem | sean-k-mooney: oh easy, just stop doing testing :) | |
| 17:15:17 | mriedem | then you're in the same state as before | |
| 17:15:21 | mriedem | heads in the sand and whatnot | |
| 17:16:13 | melwitt | I think we switched our default network backend to ovn recently. sean-k-mooney correct me | |
| 17:16:18 | mriedem | artom: if you're cool with dropping the nova-live-migratoin rename + resize revert stuff (i still thing the rename is good at some point, just not while hurrying for a fix) i'm ok with adding a depends-on from my unskip to your fix | |
| 17:16:25 | sean-k-mooney | hehe well for 15 we were planning to test with ovn but it has other issues | |
| 17:16:39 | mriedem | ooo the plot thickens | |
| 17:16:48 | artom | mriedem, 🤝 | |
| 17:17:01 | artom | (That's a handshake shaped like a yellow heart) | |
| 17:17:10 | mriedem | ok - yeah pidgin doesn't render those | |
| 17:17:29 | melwitt | I can see it. looks cool | |
| 17:17:35 | artom | sean-k-mooney, so what's geneve? another name for OVN? | |
| 17:17:40 | sean-k-mooney | melwitt: yes but ovn has only works when we disabel waiting for the network-evnets on live migation so i think they wanted to test ml2/ovn with iptabls as a fallback plan | |
| 17:17:45 | artom | 'cuz that's the default networking thingee in our ospd custom job | |
| 17:17:59 | sean-k-mooney | geneve is a l3 tunneling protocol like vxlan | |
| 17:18:20 | sean-k-mooney | and its the protocol ovn uses for its networking by defualt | |
| 17:19:02 | sean-k-mooney | artom: you are refering to the neutron type driver which for ovn would be geneve | |
| 17:19:16 | artom | For the geneve option set the NETWORK_OVN paremeter to yes." | |
| 17:19:16 | artom | sean-k-mooney, "Choose the network variant (default for OSP15 is 'geneve') | |
| 17:19:26 | artom | (Form the Jenkins job build page) | |
| 17:19:32 | artom | So yeah, looks like OSP15 default is OVN | |
| 17:20:13 | sean-k-mooney | mriedem: but to your point yes this has always been broken in one way or another | |
| 17:21:09 | sean-k-mooney | but we i think we only started waithing in stien so its only showing up now as they are testing Stien/osp15 | |
| 17:21:09 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 17:22:08 | artom | sean-k-mooney, I guess you can abandon https://review.opendev.org/#/c/660782/ now, since we're settling on https://review.opendev.org/#/c/663405/1 | |
| 17:23:20 | sean-k-mooney | am sure i never intended it to merge anyway so if you dont need it for testing then sure ill abandon | |
| 17:24:05 | artom | We could replace it with another DNM on top of mriedem's tempest patch to run tempest-slow with hybrid plug | |
| 17:24:12 | artom | For the extra confidence | |
| 17:24:34 | artom | But we've seen it pass already, not sure how necessary a new patch would be | |
| 17:24:51 | sean-k-mooney | well i can add a depens on to matts change too but ok ill just abandon it | |
| 17:25:13 | artom | Well no you can't, because your change was in the multinode job | |
| 17:25:30 | artom | Whereas mriedem's change unksips a test in tempest-slow | |
| 17:25:53 | sean-k-mooney | yes but may change didnt diable tempest-slow | |
| 17:26:00 | sean-k-mooney | but anyway its abandoned | |
| 17:26:20 | sean-k-mooney | you know how to reporduce this if you need to so i dont need to keep a patch open | |
| 17:26:43 | artom | I need lunch | |
| 18:02:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1829479 and bug 1817833 https://review.opendev.org/663737 | |
| 18:02:27 | mriedem | cfriesen: sean-k-mooney: ^ | |
| 18:02:27 | openstack | bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,Triaged] https://launchpad.net/bugs/1829479 | |
| 18:02:28 | openstack | bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Undecided,In progress] https://launchpad.net/bugs/1817833 - Assigned to xulei (605423512-j) | |
| 18:02:32 | mriedem | bugtastic | |
| 18:05:57 | sean-k-mooney | mriedem_lunch: the comment help but ya that looks correct | |
| 18:32:47 | efried | I wonder, if we made a bug whose title referenced another bug, and that bug's title referenced the first, could we make patchbot spin forever? | |
| 18:41:13 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664 | |
| 18:41:14 | openstackgerrit | Eric Fried proposed openstack/nova master: Use OpenStack SDK for placement https://review.opendev.org/656023 | |
| 18:41:35 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899 | |
| 18:41:44 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027 | |
| 18:41:52 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028 | |
| 18:42:01 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690 | |
| 18:50:52 | openstackgerrit | Merged openstack/nova master: Add testing guide for down cells https://review.opendev.org/650167 | |
| 20:04:45 | mriedem | artom: i guess you missed these comments https://review.opendev.org/#/c/644881/19//COMMIT_MSG@11 | |
| 20:27:48 | mriedem | dansmith et al, I'm +2 on the nova/cyborg spec now https://review.opendev.org/#/c/603955/ | |
| 20:49:09 | mriedem | melwitt: see what you think about my proposed wording in this "specify az on unshelve" spec and if you agree i'll update it https://review.opendev.org/#/c/624689/10/specs/train/approved/support-specifying-az-when-restore-shelved-server.rst@62 | |
| 20:58:00 | efried | nova meeting in 3 minutes in #openstack-meeting | |
| 20:59:11 | melwitt | mriedem: ack | |
| 21:02:16 | efried | nova meeting now | |
| 21:07:23 | mriedem | no meeting?! | |
| 21:07:28 | mriedem | was reviewing something... | |
| 21:28:25 | melwitt | mriedem_afk: +1 to proposed wording | |
| 21:43:02 | openstackgerrit | Dustin Cowles proposed openstack/nova master: WIP: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691 | |
| 21:53:14 | tonyb | kashyap: I don't but it's on my list of things to get workign this year | |
| 21:53:19 | tonyb | kashyap: 'sup? | |