TheJulia | JayF: a little file moving/link addition needed with https://review.opendev.org/c/openstack/ironic-specs/+/890164 :) | 00:00 |
---|---|---|
opendevreview | Merged openstack/bifrost master: CI: Add a non-voting SLURP upgrade job for Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/913680 | 00:54 |
opendevreview | Verification of a change to openstack/ironic master failed: Replace cinderclient usage with openstacksdk https://review.opendev.org/c/openstack/ironic/+/900265 | 01:24 |
opendevreview | Verification of a change to openstack/ironic stable/2023.2 failed: Add states.SERVICING and SERVICEWAIT to need_prepare_ramdisk https://review.opendev.org/c/openstack/ironic/+/915349 | 01:33 |
opendevreview | Steve Baker proposed openstack/ironic master: Replace cinderclient usage with openstacksdk https://review.opendev.org/c/openstack/ironic/+/900265 | 02:15 |
opendevreview | Steve Baker proposed openstack/ironic master: Remove requirement exclusions missing from global https://review.opendev.org/c/openstack/ironic/+/917507 | 02:15 |
rpittau | good morning ironic! o/ | 06:47 |
rpittau | tonyb: re pysnmp-lextudio: current status is we're waiting on some tests to be completed before giving the green light as it's a major version bump, I expect hat to move forward towards the end of May | 06:49 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Remove last traces of ubuntu focal https://review.opendev.org/c/openstack/bifrost/+/917537 | 06:59 |
*** zbitter is now known as zaneb | 08:09 | |
iurygregory | good morning Ironic | 13:30 |
TheJulia | good morning | 13:33 |
TheJulia | rpittau: I think the *what* of what changed to drive a major version bump is also really minor, albeit breaking for us. I think this is one of those cases where CI alone can give us good confidence. Besides, if it doesn't work or if it exhibits odd behavior, it is likely all eventlet | 13:36 |
opendevreview | Verification of a change to openstack/ironic stable/2023.2 failed: Add states.SERVICING and SERVICEWAIT to need_prepare_ramdisk https://review.opendev.org/c/openstack/ironic/+/915349 | 13:36 |
opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove old excludes https://review.opendev.org/c/openstack/ironic/+/917701 | 13:43 |
opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: Remove old excludes https://review.opendev.org/c/openstack/ironic-inspector/+/917702 | 13:44 |
opendevreview | Takashi Kajinami proposed openstack/ironic-lib master: Remove old excludes https://review.opendev.org/c/openstack/ironic-lib/+/917703 | 13:46 |
opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Remove old excludes https://review.opendev.org/c/openstack/ironic-python-agent/+/917704 | 13:46 |
opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Remove old excludes https://review.opendev.org/c/openstack/python-ironicclient/+/917705 | 13:47 |
opendevreview | Takashi Kajinami proposed openstack/python-ironic-inspector-client master: Remove old excludes https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/917706 | 13:48 |
opendevreview | Takashi Kajinami proposed openstack/ironic-tempest-plugin master: Remove old excludes https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/917710 | 13:57 |
TheJulia | I have very mixed feelings on this.... | 14:05 |
rpittau | TheJulia: sorry, I miss the context :) | 14:12 |
TheJulia | The sudden need to remove the exclusions, which are sort of a loss of knowledge and context and really... we had tooling at one point which would have done what Takashi did | 14:21 |
TheJulia | just... yeah, a bag of mixed feelings | 14:21 |
rpittau | oh, I was referring to the ping before, but I get what you mean in this case | 14:22 |
rpittau | I'm very surprised | 14:22 |
TheJulia | If we're going to go back to requirements pushing, It could be a positive, but if an exclusion is put in, it goes in for a very good reason. | 14:23 |
tkajinam | Technically we can keep these by updating the tooling in requirement,s but since requirements are no longer synced actively and these are all 2+ years old releases, idk if these are really useful nowadays. I also think the current excludes are not really complete one. | 14:26 |
TheJulia | I mean, if we're going to do it, we really need to just go back to a central push model since there is no way to keep everyone on the same page otherwise | 14:27 |
rpittau | unless you break CI :) | 14:28 |
TheJulia | break it! | 14:28 |
TheJulia | And then fix it, and then learn how *not* to break it | 14:29 |
tkajinam | I'm not too sure if central push model gives us benefit now. minimum version really needs to be maintained per project based on logic change. Another use cases may be maintaining known bad versions but this has been lost long time ago since we introduced global upper constraints mechanism IIUC. | 14:31 |
TheJulia | Well, central known-bad version tracking is a good thing | 14:32 |
TheJulia | Removing upper constraints also changes some of that | 14:33 |
TheJulia | I guess at the end of the day, the question is what results in the most benefit/value | 14:33 |
rpittau | leaving a bit earlier today, see you all on Thursday o/ | 14:52 |
tonyb | rpittau: Thanks for the update. | 15:11 |
opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake ironic python agent (fake_ipa) https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 16:33 |
TheJulia | I'm tagging takashi's patches as ironic-week-prio | 16:58 |
TheJulia | A starting point to get things unwedged CI wise is https://review.opendev.org/c/openstack/ironic/+/917701 | 17:19 |
opendevreview | Merged openstack/python-ironic-inspector-client master: Remove old excludes https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/917706 | 18:41 |
opendevreview | Merged openstack/ironic master: Remove old excludes https://review.opendev.org/c/openstack/ironic/+/917701 | 18:48 |
opendevreview | Merged openstack/python-ironicclient master: Remove old excludes https://review.opendev.org/c/openstack/python-ironicclient/+/917705 | 19:08 |
opendevreview | Merged openstack/ironic-python-agent master: Remove old excludes https://review.opendev.org/c/openstack/ironic-python-agent/+/917704 | 19:27 |
opendevreview | Steve Baker proposed openstack/ironic master: Replace cinderclient usage with openstacksdk https://review.opendev.org/c/openstack/ironic/+/900265 | 19:59 |
opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Remove old excludes https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/917710 | 21:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!