| opendevreview | Adam McArthur proposed openstack/ironic master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/ironic/+/906806 | 00:14 |
|---|---|---|
| opendevreview | Adam McArthur proposed openstack/ironic master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/ironic/+/906807 | 00:14 |
| opendevreview | Adam McArthur proposed openstack/ironic master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/ironic/+/906808 | 00:14 |
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/906935 | 02:15 |
| rpittau | good morning ironic! o/ | 07:44 |
| arne_wiebalck | Good morning, Ironic! | 07:45 |
| masghar | Good morning! | 10:15 |
| opendevreview | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for OSC baremetal chassis create command https://review.opendev.org/c/openstack/python-ironicclient/+/382496 | 10:26 |
| iurygregory | good morning ironic | 11:14 |
| masghar | o/ | 11:36 |
| rpittau | good night! o/ | 16:58 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Bump hacking to 6.1.0 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/907047 | 17:18 |
| opendevreview | Merged openstack/ironic-ui master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/ironic-ui/+/906783 | 17:22 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Bump hacking to 6.1.0 https://review.opendev.org/c/openstack/ironic-inspector/+/907046 | 17:39 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/906777 | 18:01 |
| opendevreview | Merged openstack/sushy master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/sushy/+/906797 | 18:14 |
| opendevreview | Merged openstack/sushy-tools master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/sushy-tools/+/906800 | 18:15 |
| opendevreview | Merged openstack/sushy-tools master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/sushy-tools/+/906801 | 18:15 |
| opendevreview | Merged openstack/sushy-tools master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/sushy-tools/+/906802 | 18:15 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Remove dependency on pytz https://review.opendev.org/c/openstack/ironic-inspector/+/906947 | 18:20 |
| frickler | JayF: seems doc job isn't fixed for inspector, either? ^^ | 18:27 |
| JayF | What the hell? I tested it! | 18:29 |
| JayF | My 0.35.1 fix should work on this anyway. | 18:29 |
| JayF | Hmm. Going to see if I can ID what breaks it. | 18:30 |
| JayF | Just out of curiousity/environmental knowledge for future | 18:30 |
| JayF | hmmmm. python 3.11 locally, python 3.10 in the job | 18:30 |
| JayF | and it broke locally. why didn't this reproduce when I was testing?! | 18:31 |
| JayF | frickler: well, I confirmed 0.35.1 fixes it, at least | 18:32 |
| JayF | and 0.35.1 makes loud noises in this case, and just doesn't fail | 18:32 |
| JayF | and I see the loud noises so I know I have a valid test | 18:32 |
| JayF | *shrug* | 18:32 |
| frickler | JayF: do you want to do a deps-on on one of those patches here on the reqs bump? just to verify in CI | 18:37 |
| JayF | That doesn't work. | 18:37 |
| JayF | https://opendev.org/openstack/ironic-inspector/src/branch/master/tox.ini#L100 | 18:38 |
| JayF | hmm. unless we set TOX_CONSTRAINTS_FILE in the gate | 18:38 |
| JayF | which we probably do, I'm guessing, because you'd know | 18:38 |
| TheJulia | I believe it is set | 18:38 |
| frickler | I think it should be set somewhere, yes | 18:39 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector master: [DNM] Eventlet science! https://review.opendev.org/c/openstack/ironic-inspector/+/907247 | 18:40 |
| TheJulia | one more meeting today | 18:45 |
| TheJulia | one more meeting | 18:45 |
| TheJulia | then I can go look at rbac with a fresh brain... (joking, there is no fresh brain after hours of meetings | 18:45 |
| TheJulia | ) | 18:45 |
| JayF | I turn into a pumpkin in about 2 hours for a doctor's appointment | 18:48 |
| JayF | had a super productive day so far, none of it pointed in the direction I thought when my day started | 18:48 |
| JayF | but hey, I have a commit in eventlet, I think that officially makes me part of the problem ;) /s | 18:48 |
| TheJulia | heh | 18:48 |
| TheJulia | \o/ | 18:48 |
| TheJulia | I need to figure out how to fix service role usage with a service project | 18:49 |
| TheJulia | downside is wsgi doesn't load config as it is loading files, it loads it later | 18:49 |
| TheJulia | so my attempt fails to work | 18:49 |
| TheJulia | because I wanted to make it optional | 18:49 |
| JayF | So with the move to explictly define wsgi entrypoints | 18:50 |
| JayF | could we make a wsgi entrypoint that somehow preloaded the conf | 18:50 |
| JayF | in a different way than running from command line? | 18:50 |
| * JayF hand waves a little | 18:50 | |
| TheJulia | dunno, I also want to make something backportable | 18:50 |
| opendevreview | Merged openstack/bifrost master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/bifrost/+/906759 | 19:05 |
| opendevreview | Merged openstack/bifrost master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/bifrost/+/906760 | 19:05 |
| opendevreview | Merged openstack/bifrost master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/bifrost/+/906761 | 19:05 |
| JayF | https://review.opendev.org/c/openstack/ironic-inspector/+/907247 science passed | 19:22 |
| TheJulia | for science! | 19:26 |
| opendevreview | Merged openstack/networking-generic-switch master: [codespell] Fixing Spelling Mistakes https://review.opendev.org/c/openstack/networking-generic-switch/+/906791 | 19:28 |
| opendevreview | Steve Baker proposed openstack/ironic master: Expose conductor online boolean for accurate alive https://review.opendev.org/c/openstack/ironic/+/873660 | 20:03 |
| opendevreview | Merged openstack/metalsmith master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/metalsmith/+/906786 | 20:24 |
| opendevreview | Merged openstack/metalsmith master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/metalsmith/+/906787 | 22:04 |
| JayF | inspector gate should be fixed | 23:15 |
| JayF | rechecking https://review.opendev.org/c/openstack/ironic-inspector/+/906947 | 23:15 |
| opendevreview | Adam McArthur proposed openstack/python-ironic-inspector-client master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/907116 | 23:22 |
| opendevreview | Adam McArthur proposed openstack/python-ironic-inspector-client master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/907117 | 23:22 |
| opendevreview | Adam McArthur proposed openstack/python-ironicclient master: [codespell] Adding Tox Target for Codespell https://review.opendev.org/c/openstack/python-ironicclient/+/906795 | 23:27 |
| opendevreview | Adam McArthur proposed openstack/python-ironicclient master: [codespell] Adding CI target for Tox Codespell https://review.opendev.org/c/openstack/python-ironicclient/+/906796 | 23:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!