| TheJulia | okay, recheck party has been started | 00:02 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: Remove verbose per-node debug logging from redfish sensor collection https://review.opendev.org/c/openstack/ironic/+/976767 | 00:19 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 00:23 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 00:26 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: OVS testing patch for 'vxlan' binding model https://review.opendev.org/c/openstack/networking-generic-switch/+/972765 | 00:26 |
| TheJulia | and the spice... err... code must flow | 00:30 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 00:40 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Add Junos, Cumulus NVUE, and denote Dell OS10 as unsupported https://review.opendev.org/c/openstack/networking-generic-switch/+/972764 | 00:40 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: OVS testing patch for 'vxlan' binding model https://review.opendev.org/c/openstack/networking-generic-switch/+/972765 | 00:41 |
| cardoe | Indeed https://review.opendev.org/c/openstack/ironic/+/978020 succeeded. | 00:46 |
| TheJulia | woot | 00:55 |
| cardoe | You might not like the next one I did. | 00:56 |
| cardoe | TaskManager now has multiple versions from a typing stand point. | 00:57 |
| cardoe | Between the one linked above and TaskManager, I’ve found a bunch of bugs. | 00:58 |
| TheJulia | ugh, I haven't looked at the prior one, but that is for a fresh brain | 01:00 |
| cardoe | The other one is purely mechanical. Replacing a helper function with internal raise. | 01:14 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: docs: Reorder docs to be more sane https://review.opendev.org/c/openstack/networking-generic-switch/+/978015 | 03:46 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: ci: switch over to redfish based jobs https://review.opendev.org/c/openstack/networking-baremetal/+/978039 | 04:16 |
| TheJulia | The networking-baremetal classic job seems super sensitive and we have some fundimentally whacky networking it is built upon | 04:20 |
| cardoe | janders: since you were messing with the BIOS update stuff I think it'd be great if we could support HTTP PUT via multipart. HPE and Dell both support it. https://developer.dell.com/apis/2978/versions/7.xx/docs/Tasks/1MultipartUpdates.md | 04:25 |
| janders | cardoe++ | 04:26 |
| janders | we have no immediate push for this downstream but I do see the value | 04:26 |
| cardoe | TheJulia: https://review.opendev.org/c/openstack/nova/+/972601 I dunno if that'll solve the other one you saw cause I cannot figure out the exception they saw. Unfortunately sean misunderstood my change so he -1'd it. | 04:27 |
| cardoe | janders: It's how the iLO and iDRAC work via the web interface and its how Dell's OpenManage pushes one off firmwares since the other way it operates is using their OEM firmware repo. So the API that sushy implements today isn't actually used by the vendors own tools which is what makes me like it less. | 04:29 |
| TheJulia | cardoe: try to remind me tomorrow, I've spent a good amount of time walking that code path back and forth | 04:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Remove idrac-wsman tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978040 | 05:16 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Dial back the multi-tenancy testing https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978041 | 05:16 |
| TheJulia | cardoe: makes sense that the exceptions should be cleaned up there, but they are super skittish about catching other stuff there and providing better messages. That being said, I'm not sure how to reconcile your comment and the issue together... yet I'm having serious deja vu looking at that change | 05:22 |
| TheJulia | btw, https://review.opendev.org/c/openstack/networking-baremetal/+/978039 should clean up the excess issues on the existing networking-baremetal job failures, it is working in another window and just gets us out of the mtu or asymmetric routing issues which are happening | 05:24 |
| cardoe | So if you look at https://bugs.launchpad.net/nova/+bug/2137673 | 05:41 |
| cardoe | The error the user will get is "Build of instance f006c37e-7863-4318-859d-1315a0f9a1bf aborted: Failure prepping block device." | 05:41 |
| cardoe | If I had used the KVM/libvirt backend in nova the error would have been "It is not allowed to create an interface on external network b420e863-7d26-4c87-8974-76f6c8750640" | 05:42 |
| cardoe | Cause they have a set of "white listed" exceptions | 05:42 |
| cardoe | The problem is that the ironic driver has "prepare_networks_before_block_device_mapping" which KVM/libvirt don't have. | 05:44 |
| cardoe | And their white list exceptions for that call don't include any of the network related exceptions. | 05:45 |
| opendevreview | Merged openstack/ironic master: Add storage controller discovery to Redfish inspection https://review.opendev.org/c/openstack/ironic/+/976626 | 05:59 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: l2vni plug case with Cisco NXOS https://review.opendev.org/c/openstack/networking-generic-switch/+/968377 | 06:08 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish BIOS settings update to not require IPA https://review.opendev.org/c/openstack/ironic/+/977569 | 06:08 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish BIOS settings update to not require IPA https://review.opendev.org/c/openstack/ironic/+/977569 | 06:10 |
| opendevreview | Merged openstack/ironic master: Enable parallel processes in sphinx-build to speed up tox "docs" target https://review.opendev.org/c/openstack/ironic/+/976807 | 06:29 |
| opendevreview | Merged openstack/networking-baremetal master: ci: switch over to redfish based jobs https://review.opendev.org/c/openstack/networking-baremetal/+/978039 | 07:24 |
| rpittau | good morning ironic! o/ | 07:27 |
| opendevreview | Jacob Anders proposed openstack/ironic master: fix redfish BIOS settings update to not require IPA https://review.opendev.org/c/openstack/ironic/+/977569 | 12:30 |
| opendevreview | cid proposed openstack/ironic master: Fix pickle error when oslo.service uses spawn mode https://review.opendev.org/c/openstack/ironic/+/977266 | 12:44 |
| opendevreview | cid proposed openstack/ironic master: Fix pickle error when oslo.service uses spawn mode https://review.opendev.org/c/openstack/ironic/+/977266 | 12:48 |
| cardoe | TheJulia: https://review.opendev.org/c/openstack/ironic/+/978020 is what ya wanted me to ping you about? | 14:48 |
| TheJulia | no, the nova change | 14:53 |
| TheJulia | unless that change was the one you were talking about late last night | 14:54 |
| *** dking is now known as Guest3853 | 14:56 | |
| *** Guest3853 is now known as dking | 14:56 | |
| TheJulia | i foudn the infamous tempurl 401 error | 15:17 |
| cardoe | TheJulia: https://review.opendev.org/c/openstack/nova/+/972601 was the nova change. | 16:03 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix tempurl timeout bug https://review.opendev.org/c/openstack/ironic/+/978098 | 16:06 |
| TheJulia | yeah | 16:07 |
| TheJulia | so ^^^ is the most annoying CI bug | 16:09 |
| TheJulia | I'd really appreciate it for some more reviews on https://review.opendev.org/q/hashtag:ironic-week-prio+and+project:openstack/networking-generic-switch+and+status:open so we can sort of start to wrap up the networking plugins for the cycle | 16:12 |
| TheJulia | I also had to rebase and make some minor fixes in rebase, so some changes lost their +2's | 16:12 |
| opendevreview | Merged openstack/networking-generic-switch master: l2vni plug case with Cisco NXOS https://review.opendev.org/c/openstack/networking-generic-switch/+/968377 | 16:13 |
| TheJulia | woot woot | 16:13 |
| opendevreview | Jason Paroly proposed openstack/ironic-tempest-plugin master: Remove flavor dependency from InspectorBasicTest https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978106 | 16:28 |
| TheJulia | So, ^^^ raises a question, do the existing inspector tests work with the new implementaiton, or should we rip/repalce them? | 16:30 |
| dtantsur | TheJulia: I think they haven't been updated :( | 16:30 |
| TheJulia | :( | 16:32 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: follow-up on trunk reconciler https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 17:03 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Update ovn documentation based upon VXLAN discoveries https://review.opendev.org/c/openstack/ironic/+/977546 | 17:25 |
| cardoe | Well I'll give a cookie for https://review.opendev.org/c/openstack/ironic/+/978020 and https://review.opendev.org/c/openstack/ironic/+/976494 | 17:27 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix tempurl timeout bug https://review.opendev.org/c/openstack/ironic/+/978098 | 17:28 |
| cardoe | I've reviewed all 4 of your patches. | 17:33 |
| TheJulia | cid: getting there regarding https://review.opendev.org/c/openstack/ironic/+/977266 | 17:40 |
| TheJulia | Left a comment regarding it | 17:40 |
| TheJulia | cardoe: cookie per review? ;) | 17:42 |
| cid | Alright, tks. I will take a look at a moment. | 17:43 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix tox doc target errors related to oslo_service backend exceptions https://review.opendev.org/c/openstack/ironic/+/977989 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix tox doc target complaining about missing oslo.reports https://review.opendev.org/c/openstack/ironic/+/977990 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix docstrings for trait-based-networking related code https://review.opendev.org/c/openstack/ironic/+/977991 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Mark our own sphinx doc extensions as parallel read safe https://review.opendev.org/c/openstack/ironic/+/977992 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Various docstring fixes https://review.opendev.org/c/openstack/ironic/+/977993 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add osprofiler.opts as an import https://review.opendev.org/c/openstack/ironic/+/977994 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Suppress autodoc warnings https://review.opendev.org/c/openstack/ironic/+/977995 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix documentation related build warnings throughout Ironic's docs https://review.opendev.org/c/openstack/ironic/+/977996 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Treat warnings as errors in sphinx-build for docs tox target https://review.opendev.org/c/openstack/ironic/+/977997 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Documentation for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/975686 | 17:51 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix tox doc target errors related to oslo_service backend exceptions https://review.opendev.org/c/openstack/ironic/+/977989 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix tox doc target complaining about missing oslo.reports https://review.opendev.org/c/openstack/ironic/+/977990 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix docstrings for trait-based-networking related code https://review.opendev.org/c/openstack/ironic/+/977991 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Mark our own sphinx doc extensions as parallel read safe https://review.opendev.org/c/openstack/ironic/+/977992 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Various docstring fixes https://review.opendev.org/c/openstack/ironic/+/977993 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add osprofiler.opts as an import https://review.opendev.org/c/openstack/ironic/+/977994 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Suppress autodoc warnings https://review.opendev.org/c/openstack/ironic/+/977995 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Fix documentation related build warnings throughout Ironic's docs https://review.opendev.org/c/openstack/ironic/+/977996 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Treat warnings as errors in sphinx-build for docs tox target https://review.opendev.org/c/openstack/ironic/+/977997 | 17:53 |
| opendevreview | Clif Houck proposed openstack/ironic master: Documentation for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/975686 | 17:53 |
| clif | Sorry for the spam >_< | 17:53 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix up the behavior of serial number reading on iDRAC https://review.opendev.org/c/openstack/ironic/+/978112 | 17:53 |
| * TheJulia blinks | 17:53 | |
| TheJulia | clif: Pizza sized cookies, pls | 17:53 |
| cardoe | I think we should land that iDRAC change this cycle. | 17:54 |
| clif | I just love pushing code and then realizng the very bottom commit still isn't fixed | 17:54 |
| TheJulia | cardoe: which idrac change? | 17:54 |
| cardoe | In this cycle we changed the inspection generically in the redfish driver to duplicate data for working around Dell. | 17:54 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/978112 | 17:54 |
| cardoe | So rather than changing the data surface for a release this just moves the fix into the idrac driver. | 17:54 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: split off helper func in inspection_rules for testing https://review.opendev.org/c/openstack/ironic/+/977497 | 18:00 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix inspection rules loaded from a file to not fail in logging https://review.opendev.org/c/openstack/ironic/+/978116 | 18:00 |
| cardoe | The "split off func in inspection_rules for testing" had 2 core's give a +2 but it failed now that I re-enabled the missing pep8 checks. | 18:01 |
| cardoe | The last change fixes a crasher when using inspection rules and running conductor with debug. | 18:01 |
| cardoe | And with those two changes, inspection rules works as documented. | 18:01 |
| opendevreview | Merged openstack/networking-baremetal master: Add conductor group sharding support https://review.opendev.org/c/openstack/networking-baremetal/+/948432 | 18:02 |
| opendevreview | Merged openstack/networking-baremetal master: Trunk port reconciliation for L2VNI attachments https://review.opendev.org/c/openstack/networking-baremetal/+/974619 | 18:02 |
| TheJulia | JFYI, https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978041 will save about 1500 seconds on some of the network testing | 18:05 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: add helper tool for testing inspection rules https://review.opendev.org/c/openstack/ironic/+/973381 | 18:05 |
| cardoe | and that's the script we've integrated to test inspection tools. | 18:06 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Remove idrac-wsman tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978040 | 18:31 |
| rm_work[m] | TheJulia: do you think https://review.opendev.org/c/openstack/ironic/+/971581 should be a fairly straightforward backport? | 18:51 |
| rm_work[m] | It looks like it's not exactly complex | 18:51 |
| opendevreview | Adam Harwell proposed openstack/ironic stable/2025.2: Fix async periodic on redfish for servicewait https://review.opendev.org/c/openstack/ironic/+/978125 | 18:54 |
| rm_work[m] | I did the basic gerrit cherry-pick to 2025.2 to see if it would be clean and it does appear to be | 18:54 |
| TheJulia | yeah, that should be pretty good | 18:54 |
| rm_work[m] | will try to go to 2025.1 as well just because we do still run that | 18:55 |
| rm_work[m] | thanks! | 18:55 |
| TheJulia | I +2'ed it | 18:55 |
| TheJulia | and tagged it | 18:55 |
| opendevreview | Adam Harwell proposed openstack/ironic stable/2025.1: Fix async periodic on redfish for servicewait https://review.opendev.org/c/openstack/ironic/+/978126 | 18:55 |
| TheJulia | heh, okay | 18:55 |
| rm_work[m] | We're trying to stay as close to master as possible, I'm seeing how close we can get to carrying zero patches downstream :P | 18:56 |
| TheJulia | oooh, exciting! | 18:56 |
| rm_work[m] | We have a couple more but they are vendor specific which is... annoying | 18:57 |
| rm_work[m] | I'm contemplating what exactly is TOO vendor specific | 18:57 |
| rm_work[m] | some of these things COULD be useful to others maybe, IDK | 18:57 |
| TheJulia | Sometimes yeah, sometimes we might also look at an item and go "hey, if we do this, it oculd be more applicable", but it really takes that dialog in code review | 18:58 |
| TheJulia | The key aspect really just being, set context upfront | 18:58 |
| rm_work[m] | yep, looks like I have one more to do but it doesn't cherry-pick cleanly so I'll look into it today: https://review.opendev.org/c/openstack/ironic/+/971459 | 18:59 |
| TheJulia | ++ | 18:59 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: vxlan: Arista EOS and vendor neutral SONiC https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 | 19:02 |
| opendevreview | Merged openstack/ironic master: Support pluggable API middleware via entry points https://review.opendev.org/c/openstack/ironic/+/976494 | 19:16 |
| opendevreview | Merged openstack/networking-baremetal master: vlxan: explicitly prevent heiarchical agent triggered binding https://review.opendev.org/c/openstack/networking-baremetal/+/977402 | 19:29 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 19:46 |
| cardoe | If someone else wants to review and workflow https://review.opendev.org/c/openstack/ironic/+/977989 and the one above it, that'll cause the entire docs cleanup series by clif to land. | 19:54 |
| TheJulia | This is the way. | 19:56 |
| cardoe | TheJulia: I fixed your pep8 feedback on https://review.opendev.org/c/openstack/ironic/+/977497 | 20:17 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: Fix async periodic on redfish for servicewait https://review.opendev.org/c/openstack/ironic/+/978125 | 20:21 |
| cardoe | ^ there I fixed the pep8 issue that prevented it. TheJulia you had +2'd that but now it'll pass. | 20:21 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: Fix async periodic on redfish for servicewait https://review.opendev.org/c/openstack/ironic/+/978126 | 20:22 |
| cardoe | Same tehre | 20:22 |
| cardoe | Get that merge pipeline nice and full | 20:23 |
| TheJulia | Muahahahahahaha | 20:24 |
| cardoe | alright... https://review.opendev.org/q/hashtag:ironic-week-prio+status:open is cleaned up as well. | 20:31 |
| cardoe | Most everything in there is mergeable or top of mind maybe? | 20:31 |
| TheJulia | rm_work[m]: looks like 978126 is failing ci :\ | 20:37 |
| TheJulia | cardoe: yeah, looks like not everything is churning due to mappings, but that is also fine | 20:37 |
| cardoe | TheJulia: I fixed 978126 | 20:45 |
| TheJulia | docs target failure | 20:45 |
| TheJulia | https://zuul.opendev.org/t/openstack/build/f6288b6da3054a98922db3ac164259b1 | 20:45 |
| opendevreview | Merged openstack/ironic master: Fix tempurl timeout bug https://review.opendev.org/c/openstack/ironic/+/978098 | 20:52 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Remove flavor dependency from InspectorBasicTest https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/978106 | 20:52 |
| opendevreview | Merged openstack/ironic master: Fix tox doc target errors related to oslo_service backend exceptions https://review.opendev.org/c/openstack/ironic/+/977989 | 20:52 |
| opendevreview | Merged openstack/ironic master: Fix tox doc target complaining about missing oslo.reports https://review.opendev.org/c/openstack/ironic/+/977990 | 20:52 |
| opendevreview | Merged openstack/networking-generic-switch master: vxlan: follow-up cisco nxos/substrate review feedback. https://review.opendev.org/c/openstack/networking-generic-switch/+/977999 | 20:56 |
| TheJulia | vxlan items are quickly disappearing into a merged state :) | 20:56 |
| TheJulia | rpittau: I think if we can get the last bits merged in the next 24 hours, we can likely go ahead and look at ngs/n-b releases next week | 20:57 |
| TheJulia | https://review.opendev.org/q/topic:%22ironic-vxlan%22+status:open | 20:57 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977317 | 21:02 |
| opendevreview | Michael Sherman proposed openstack/networking-generic-switch master: Allow reusing ssh connections https://review.opendev.org/c/openstack/networking-generic-switch/+/978141 | 21:07 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977317 | 21:12 |
| cardoe | I think that'll fix the 2025.1 stuff | 21:13 |
| opendevreview | Michael Sherman proposed openstack/networking-generic-switch master: Allow reusing ssh connections https://review.opendev.org/c/openstack/networking-generic-switch/+/978141 | 21:14 |
| cardoe | https://review.opendev.org/c/openstack/python-ironicclient/+/977406 that's an easy backport | 21:14 |
| rm_work[m] | are there known issues with 2024.2 in CI? | 21:30 |
| rm_work[m] | this doesn't look like it's related to the CR: | 21:30 |
| rm_work[m] | https://zuul.opendev.org/t/openstack/build/6a764d5732384f29a2dd53839366d302 | 21:30 |
| TheJulia | looks like https://review.opendev.org/c/openstack/networking-generic-switch/+/972763 is the major change holding back a flood of ngs stuffs | 21:32 |
| opendevreview | Merged openstack/ironic master: Fix docstrings for trait-based-networking related code https://review.opendev.org/c/openstack/ironic/+/977991 | 21:39 |
| opendevreview | Merged openstack/ironic master: Mark our own sphinx doc extensions as parallel read safe https://review.opendev.org/c/openstack/ironic/+/977992 | 21:39 |
| TheJulia | rm_work[m]: so that is an issue with setuptools, I think the only way to "fix" that is to pin setuptools on the branch, we've been impacted by that all over the place as of recent | 21:41 |
| TheJulia | <82.0.0 | 21:41 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: follow-up on trunk reconciler https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 21:42 |
| opendevreview | Michael Sherman proposed openstack/networking-generic-switch master: Allow reusing ssh connections https://review.opendev.org/c/openstack/networking-generic-switch/+/978147 | 21:45 |
| opendevreview | Michael Sherman proposed openstack/networking-generic-switch master: Allow reusing ssh connections https://review.opendev.org/c/openstack/networking-generic-switch/+/978141 | 21:46 |
| shermanm | blah, somehow I thought it wouldn't spam on a --WIP change | 21:48 |
| opendevreview | Verification of a change to openstack/ironic master failed: Various docstring fixes https://review.opendev.org/c/openstack/ironic/+/977993 | 22:00 |
| rm_work[m] | anyone happen to know if it's possible for git-review to use HTTPS instead of SSH? I kinda feel like the answer is no here, since I think the only configured auth is SSH-pubkey | 22:02 |
| TheJulia | rm_work[m]: I think it is possible... I think. | 22:10 |
| TheJulia | its target url, and there is a password in the gui for such | 22:10 |
| opendevreview | Michael Sherman proposed openstack/networking-generic-switch master: Allow reusing ssh connections https://review.opendev.org/c/openstack/networking-generic-switch/+/978141 | 22:30 |
| cardoe | TheJulia, rm_work[m]: as soon as https://review.opendev.org/c/openstack/ironic/+/977317 merges then I'll backport it to 2024.2 | 22:56 |
| cardoe | rm_work[m]: it is possible to use HTTPS. I used to when SSH at a random port was blocekd. | 22:56 |
| opendevreview | Verification of a change to openstack/ironic master failed: Various docstring fixes https://review.opendev.org/c/openstack/ironic/+/977993 | 23:17 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!