| iurygregory | Folks, I'm looking at https://review.opendev.org/q/project:openstack/networking-generic-switch+status:open+hashtag:ironic-week-prio since tomorrow we will need to propose a release | 00:22 |
|---|---|---|
| iurygregory | https://review.opendev.org/c/openstack/networking-generic-switch/+/984833 has +W but I think due to the chain the patch hasn't merged.. | 00:23 |
| iurygregory | https://review.opendev.org/c/openstack/networking-generic-switch/+/984832/ has -1 from JayF | 00:23 |
| iurygregory | https://review.opendev.org/c/openstack/networking-generic-switch/+/984847/ no reviews | 00:24 |
| iurygregory | https://review.opendev.org/c/openstack/networking-generic-switch/+/998062 has -1 from TheJulia | 00:24 |
| iurygregory | so I'm wondering how we want to proceed, have the final release for ngs without this patches or try to delay till Friday? (Probably need to check with release team...) | 00:25 |
| TheJulia | We release what we can, we work what we can work. I guess what I mean is we just release master branch where it is at and keep working on items as we go | 00:26 |
| iurygregory | sounds good to me | 00:28 |
| iurygregory | for ipa-builder I will see if we can get https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/970336 | 00:28 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Verify Dell firmware application via LC job after reboot https://review.opendev.org/c/openstack/ironic/+/1002260 | 00:51 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Fix get_unfinished_jobs missing non-terminal job states https://review.opendev.org/c/openstack/sushy/+/1002573 | 02:03 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Verify Dell firmware application via LC job after reboot https://review.opendev.org/c/openstack/ironic/+/1002260 | 02:09 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Verify Dell firmware application via LC job after reboot https://review.opendev.org/c/openstack/ironic/+/1002260 | 03:43 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Fix get_unfinished_jobs missing non-terminal job states https://review.opendev.org/c/openstack/sushy/+/1002580 | 03:44 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Fix get_unfinished_jobs missing non-terminal job states https://review.opendev.org/c/openstack/sushy/+/1002573 | 03:57 |
| opendevreview | Merged openstack/sushy master: Fix get_unfinished_jobs missing non-terminal job states https://review.opendev.org/c/openstack/sushy/+/1002573 | 12:27 |
| iurygregory | I have the feeling we somehow broke ironic-standalone-ipa-src because of the autodetect [ERROR] /opt/stack/ironic/devstack/lib/ironic:1889 Default interface set for deploy interface is not enabled. https://zuul.opendev.org/t/openstack/build/928f71de4de441d28798deb1ed275d12 | 12:38 |
| iurygregory | maybe it will be just a configuration in the job definition, checking now | 12:38 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Fix ironic-standalone-ipa-src configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/1002643 | 13:01 |
| iurygregory | ironic cores ^ it's green so we can fix CI | 15:18 |
| TheJulia | iurygregory: thanks! | 15:23 |
| TheJulia | so, slight concern | 15:23 |
| TheJulia | We *shouldn't* need/require a default interface | 15:24 |
| TheJulia | So in other words, I'm worried (sans a bug to detail the actual error), that we are not silently covering up another failure | 15:24 |
| TheJulia | iurygregory: ^^^ | 15:24 |
| cardoe | TheJulia: so should I pull my +W? | 15:26 |
| TheJulia | if you wouldn't mind for a moment | 15:27 |
| TheJulia | loading iury's original link | 15:27 |
| TheJulia | hmm, its going through the list in devstack, and its devstack that is hitting the breaks based upon config | 15:28 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix ironic-standalone-ipa-src configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/1002643 | 15:29 |
| TheJulia | Im find with that landing, its not an ironic bug, its a devstack bug we're trying to route around instead | 15:30 |
| TheJulia | I've rechecked it | 15:31 |
| TheJulia | Sorry for the noise | 15:31 |
| iurygregory | I was worried to just remove the config.. | 15:46 |
| iurygregory | we can give a try to see how it goes of course, I just pushed so we could unblock ipa-builder gate =) | 15:47 |
| TheJulia | I'm thinking the right answer may be more so to make devstack dumber, tbh | 15:48 |
| iurygregory | so we can't use the classic "it works on devstack" LOL | 15:50 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix ironic-standalone-ipa-src configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/1002643 | 16:07 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add Keystone-independent oauth2 auth_strategy https://review.opendev.org/c/openstack/ironic/+/1002665 | 16:35 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Restore mistakenly-removed TestRBACScopedRequests https://review.opendev.org/c/openstack/ironic/+/1002666 | 16:35 |
| TheJulia | my "sitting in the hospital" ^ tinkering. | 16:36 |
| cardoe | hjensas: TheJulia: https://review.opendev.org/c/openstack/neutron/+/1002654 I believe that should improve the testing of port binding | 16:42 |
| cardoe | TheJulia: So just wondering why you didn't choose to do it as a keystonemiddleware plugin? | 16:43 |
| TheJulia | cardoe: see https://review.opendev.org/c/openstack/ironic/+/1002665/1/ironic/common/external_oauth2.py | 16:45 |
| TheJulia | The real challenge is also reforming what the meaning of the setting is as well | 16:45 |
| TheJulia | because then we also need to track/flag if RBAC can apply... or not | 16:47 |
| TheJulia | cardoe: does that make sense? | 16:58 |
| * TheJulia assumes someone drove by and capture'd cardoe's attention | 17:36 | |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix ironic-standalone-ipa-src configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/1002643 | 17:54 |
| cardoe | TheJulia: sorry yes wolves got me... yaks setting things on fire drawing in the wolves | 17:59 |
| TheJulia | cardoe: YakNair, perhaps? | 18:00 |
| *** Unknown123 is now known as Mike-- | 18:19 | |
| cardoe | fun fact "openstack baremetal node vif attach NODEID NEUTRONPORT --port-uuid BAREMETALPORTGROUP" does not work | 18:36 |
| TheJulia | because the select is against the port table only? | 18:40 |
| cardoe | Yep | 18:46 |
| cardoe | so it's --vif-info port_group_uuid=$UUID because we have --vif-info as an escape hatch to provide any value. | 18:47 |
| cardoe | It's actually looking up the portgroup but then supplying the arg as port_uuid= | 18:47 |
| cardoe | and someone got cute and put a / in the port name | 18:47 |
| TheJulia | heh | 18:48 |
| cardoe | And we inconsistently escape the / in the lookup path. I would have thought all lookups went through a shared path | 18:48 |
| TheJulia | yeah, I think that was partially done and never extended out because someone never submitted patches upstream or soemthing | 18:48 |
| TheJulia | Insert grumbles there. | 18:48 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix ironic-standalone-ipa-src configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/1002643 | 19:23 |
| *** Unknown123 is now known as Mike-- | 19:43 | |
| opendevreview | Merged openstack/ironic-python-agent master: Fix ironic-standalone-ipa-src configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/1002643 | 21:24 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!