| *** zbitter is now known as zaneb | 03:35 | |
| iurygregory | good morning | 11:36 |
|---|---|---|
| opendevreview | cid proposed openstack/bifrost master: Fix codespell reported errors https://review.opendev.org/c/openstack/bifrost/+/920410 | 11:50 |
| opendevreview | cid proposed openstack/ironic master: Flexible IPMI credential persistence method configuration https://review.opendev.org/c/openstack/ironic/+/917229 | 12:06 |
| opendevreview | cid proposed openstack/ironic master: Provision ARM (aarch64) fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic/+/915441 | 12:19 |
| TheJulia | good morning | 13:34 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Assert URL consistency for agent_url https://review.opendev.org/c/openstack/ironic/+/921294 | 14:38 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Update the redfish interoperability profile https://review.opendev.org/c/openstack/ironic/+/920574 | 15:22 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Render the redfish interop profile in the docs https://review.opendev.org/c/openstack/ironic/+/921298 | 15:22 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Update the redfish interoperability profile https://review.opendev.org/c/openstack/ironic/+/920574 | 15:54 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Tox environment to verify the redfish profile https://review.opendev.org/c/openstack/ironic/+/921302 | 15:54 |
| JayF | Hello from Geneva o/ | 16:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Update the redfish interoperability profile https://review.opendev.org/c/openstack/ironic/+/920574 | 16:21 |
| dtantsur | I've never thought how much work this ^^ would be... | 16:21 |
| dtantsur | JayF: hey-hey, how are you enjoying Europe? | 16:21 |
| JayF | anything that both 1) seems easy and 2) hasn't yet been done usually is a hard problem waiting to pounce | 16:21 |
| JayF | dtantsur: I'm exhausted, TBH, but glad to be on the last leg | 16:21 |
| JayF | dtantsur: you going to be there tomorrow? | 16:21 |
| dtantsur | nop | 16:22 |
| JayF | dtantsur: I hope so, I brought your socks (they are the last not-yet-given-away ironic socks) | 16:22 |
| JayF | dang | 16:22 |
| JayF | you might lose out to rpittau then, I don't think he has gotten any either | 16:22 |
| dtantsur | heh :D | 16:22 |
| JayF | and I'm not going to re-attempt to get them posted to .eu, apparently it's impossible to get mail in | 16:22 |
| dtantsur | yeah, not worth it | 16:24 |
| dtantsur | (you can send from Switzerland lol) | 16:24 |
| dtantsur | ((or even from France)) | 16:25 |
| JayF | I considered it, but I don't love the visual of being able to barely speak the language trying to ship something -- it'd look suspicious and I really try to not look sus about anything when I'm outta my home country. | 16:25 |
| JayF | I know I'm a guest :D | 16:26 |
| JayF | I am in Switzerland now; took the train down this afternoon | 16:26 |
| dtantsur | I mean, you can walk to France (you probably will, I remember we crossed the border to get lunch) | 16:26 |
| JayF | Ah, makes sense, but still the same bit applies | 16:27 |
| dtantsur | but yeah, don't bother with the socks :D | 16:27 |
| JayF | I tried :D | 16:27 |
| dtantsur | JayF: if rpittau won't take them, maybe they'll fit masghar :) | 16:27 |
| * JayF will bribe people to cheer for his speech /s | 16:27 | |
| TheJulia | brrraaaaaiiinssss | 16:37 |
| JayF | I actually need to do a dry run of my talk for thursday, if someone wants to offer me twenty minutes. Otherwise I'll distract my non-openstacky friend :D | 16:38 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Assert URL consistency for agent_url https://review.opendev.org/c/openstack/ironic/+/921294 | 17:22 |
| opendevreview | Merged openstack/ironic master: GRUB conf template compatibility with arm server https://review.opendev.org/c/openstack/ironic/+/920836 | 17:47 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Change [agent]require_tls to True by default https://review.opendev.org/c/openstack/ironic/+/921317 | 18:03 |
| cid | TheJulia: #noqa here is for pep8? https://review.opendev.org/c/openstack/ironic/+/921294 | 18:07 |
| TheJulia | cid: looks like we broke the bandit ci job (https://zuul.opendev.org/t/openstack/builds?job_name=ironic-tox-bandit&project=openstack/ironic) with https://review.opendev.org/c/openstack/ironic/+/920276 | 18:08 |
| TheJulia | cid: re, noqa, yes | 18:08 |
| cid | well well, no logs | 18:10 |
| TheJulia | The job apparently just didn't run, the neutron folks copied the change and noticed their bandit job stopped running | 18:11 |
| opendevreview | Ihar Hrachyshka proposed openstack/ironic master: Revert "Fix negative lookahead assertion syntax error" https://review.opendev.org/c/openstack/ironic/+/921319 | 18:12 |
| TheJulia | hence ^ | 18:14 |
| cid | Hmm. Probably because of the two steps regex | 18:14 |
| JayF | +2 on the revert, it's easy to forget that the job just disappears in some failure modes | 18:17 |
| TheJulia | I guess at some point, we should ask ourselves, do we *really* need to exclude the yaml | 18:22 |
| JayF | it's a bandit job. it probably uses less resources than ... all the other jobs | 18:23 |
| JayF | lol | 18:23 |
| TheJulia | lol | 18:25 |
| TheJulia | yeah | 18:25 |
| clarkb | the reason is the meaning of the regex changes subtley. It went from saying files in tools/ other than bandit.yml are irrelevant to all files other than tools/bandit.yml are irrelevant | 18:28 |
| clarkb | and yes I often feel like we try to over optimize when jobs run and then are continually surprised when jobs don't run | 18:29 |
| cid | My exact thought. Matches all, then tries exclusion. | 18:31 |
| TheJulia | Yeah, the cost/benefit of optimizing needs to be visited. much like someone messaged me downstream wanting to use hardware which they were struggling with to which I finally asked them to consider the business value of going down the path which is already hurting them | 18:32 |
| TheJulia | free is not free if it takes you a couple weeks to wedge free in | 18:32 |
| TheJulia | cid: did you make that change elsewhere? | 18:35 |
| cid | Let me see... | 18:35 |
| cid | ironic-python-agent, | 18:36 |
| cid | and ironic-inspector | 18:37 |
| cid | https://review.opendev.org/c/openstack/ironic-inspector/+/920281 | 18:37 |
| cid | https://review.opendev.org/c/openstack/ironic-inspector/+/920281 | 18:37 |
| cid | Correct link: https://review.opendev.org/c/openstack/ironic-python-agent/+/920280 | 18:37 |
| TheJulia | so we could revert, or just remove the regex? | 18:42 |
| cid | Yea | 18:48 |
| cid | I will also research if there was a better way to do that | 18:49 |
| JayF | I think you should just follow TheJulia's advice and see if removing the exclude still leaves the job passing | 18:49 |
| JayF | and if so, just accept that simpler is probably better than saving a couple minutes in a VM | 18:50 |
| cid | So, do not exclude any of the contents of tools/ | 18:54 |
| cid | Making a push now... | 18:54 |
| TheJulia | Yeah, if it fails if someone tweaked a script, it likely is something which needs to be fixed due to a check being changed in bandit | 18:55 |
| TheJulia | meaning, likely legitimate anyway | 18:55 |
| cid | ++ | 18:57 |
| opendevreview | cid proposed openstack/ironic master: Follow up to https://review.opendev.org/c/openstack/ironic/+/920276 https://review.opendev.org/c/openstack/ironic/+/921329 | 19:30 |
| cid | o/ | 19:59 |
| opendevreview | Curt Moore proposed openstack/ironic master: Correct bond_mode enum value for 802.3ad https://review.opendev.org/c/openstack/ironic/+/921340 | 21:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Change [agent]require_tls to True by default https://review.opendev.org/c/openstack/ironic/+/921317 | 21:30 |
| TheJulia | dtantsur: question of the day, if one http boots, but uses HTTPBoot to get a URL via DHCP, do we feel that is sane enough that we can embed the agent token.... | 23:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!