Thursday, 2023-09-21

*** ravlew is now known as Guest80804:18
opendevreviewMerged openstack/networking-generic-switch master: Honor ngs_save_configuration setting when using batch commands  https://review.opendev.org/c/openstack/networking-generic-switch/+/88640504:41
opendevreviewMerged openstack/networking-generic-switch master: Do not make actual device changes in bind_port()  https://review.opendev.org/c/openstack/networking-generic-switch/+/84759204:41
opendevreviewMerged openstack/networking-generic-switch master: Fake: support adding a random sleep and injecting failures  https://review.opendev.org/c/openstack/networking-generic-switch/+/87479304:51
dtantsurJayF, I do have a soft objection against sushy-tools 1.0 since we don't have any stability of backward compatibility promise for it06:57
dtantsurenough people try it in production already...06:57
iurygregorygood morning Ironic11:05
opendevreviewOpenStack Release Bot proposed openstack/networking-baremetal stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/networking-baremetal/+/89602711:13
opendevreviewOpenStack Release Bot proposed openstack/networking-baremetal stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/networking-baremetal/+/89602811:13
opendevreviewOpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/networking-baremetal/+/89602911:13
opendevreviewOpenStack Release Bot proposed openstack/ironic-ui stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-ui/+/89603011:16
opendevreviewOpenStack Release Bot proposed openstack/ironic-ui stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-ui/+/89603111:16
opendevreviewOpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-ui/+/89603211:16
opendevreviewOpenStack Release Bot proposed openstack/bifrost stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89603311:16
opendevreviewOpenStack Release Bot proposed openstack/bifrost stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89603411:16
opendevreviewOpenStack Release Bot proposed openstack/bifrost master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89603611:16
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent/+/89603711:18
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent/+/89603811:18
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent/+/89603911:18
opendevreviewOpenStack Release Bot proposed openstack/networking-generic-switch stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604011:19
opendevreviewOpenStack Release Bot proposed openstack/networking-generic-switch stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604111:19
opendevreviewOpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604211:19
opendevreviewOpenStack Release Bot proposed openstack/ironic-inspector stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-inspector/+/89604311:20
opendevreviewOpenStack Release Bot proposed openstack/ironic-inspector stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-inspector/+/89604411:20
opendevreviewOpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-inspector/+/89604511:20
opendevreviewOpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/89604611:21
opendevreviewOpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/89604711:21
opendevreviewOpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/89604811:21
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89604911:23
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89605011:23
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89605111:23
iurygregorySPAM TIME11:27
dtantsuroh lol11:35
dtantsurpity that https://review.opendev.org/c/openstack/bifrost/+/891693/ did not go in bifrost11:37
iurygregorywe can probably backport ? since is something for testenv 11:40
dtantsuryeah, I guess so11:40
dtantsurokay, I feel like we've approved all of them11:42
dtantsur(bifrost needs more work, as usual, but that's after the DIB images are published)11:42
iurygregoryngs will need backports to 2023.2 11:44
iurygregoryto fix CI failures11:44
opendevreviewMerged openstack/ironic-ui stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-ui/+/89603011:45
opendevreviewMerged openstack/ironic-ui stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-ui/+/89603111:46
opendevreviewMerged openstack/ironic-python-agent stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent/+/89603711:46
opendevreviewMerged openstack/ironic-python-agent stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent/+/89603811:46
opendevreviewMerged openstack/networking-baremetal stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/networking-baremetal/+/89602711:47
opendevreviewMerged openstack/networking-baremetal stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/networking-baremetal/+/89602811:47
opendevreviewMerged openstack/ironic-ui master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-ui/+/89603211:48
opendevreviewMerged openstack/ironic-python-agent-builder master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89605111:48
opendevreviewMerged openstack/ironic-python-agent master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent/+/89603911:51
opendevreviewVerification of a change to openstack/ironic-prometheus-exporter master failed: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/89604811:52
opendevreviewVerification of a change to openstack/networking-generic-switch stable/2023.2 failed: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604011:52
opendevreviewVerification of a change to openstack/networking-generic-switch stable/2023.2 failed: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604111:52
opendevreviewMerged openstack/networking-baremetal master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/networking-baremetal/+/89602911:52
iurygregoryI was expecting that for ngs =X11:53
iurygregorybut I wasn't expecting that for IPE...11:53
iurygregoryok, seems like is just POST_FAILURE in some jobs11:54
opendevreviewVerification of a change to openstack/ironic-inspector master failed: Update master for stable/2023.2  https://review.opendev.org/c/openstack/ironic-inspector/+/89604511:54
opendevreviewMerged openstack/bifrost master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89603611:56
opendevreviewMerged openstack/networking-generic-switch master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604211:59
dtantsuriurygregory, could you approve https://review.opendev.org/c/openstack/ironic/+/895778 please? It's not ideal to fix the release note after branching12:06
dtantsurbonus for approving https://review.opendev.org/c/openstack/ironic/+/863999 :)12:07
iurygregorylooking now12:07
iurygregorydtantsur, I'm ok with 863999, just wondering about JayF comment on the releasenote (but since this would be a doc change I'm going to approve)12:19
dtantsurIf you mean the 2nd comment (about the docs), I think I've actually addressed it, but Gerrit keeps showing it12:24
iurygregoryyeah, it was the 2nd12:25
iurygregorythank you gerrit!12:25
opendevreviewMerged openstack/ironic-inspector stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-inspector/+/89604312:26
opendevreviewVerification of a change to openstack/ironic-python-agent-builder stable/2023.2 failed: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89604913:05
opendevreviewVerification of a change to openstack/ironic-python-agent-builder stable/2023.2 failed: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89605013:05
opendevreviewMerged openstack/ironic-prometheus-exporter stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/89604613:09
opendevreviewMerged openstack/ironic-prometheus-exporter stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/89604713:09
opendevreviewVerification of a change to openstack/ironic-python-agent master failed: Add mlnx deploy_step entry to enable deploy time firmware  https://review.opendev.org/c/openstack/ironic-python-agent/+/89337113:24
opendevreviewMerged openstack/ironic master: Redfish: wait for secure boot state change if it's not immediate  https://review.opendev.org/c/openstack/ironic/+/86399913:29
opendevreviewMerged openstack/ironic master: Trivial: attach versions to release series  https://review.opendev.org/c/openstack/ironic/+/89577813:29
JayFBranch was cut for ngs before I could update the patch. I'll backport those patches and get a release cut before everything is officially done13:35
TheJuliathanks JayF13:46
opendevreviewJake Hutchinson proposed openstack/bifrost master: Bifrost NTP configuration  https://review.opendev.org/c/openstack/bifrost/+/89569114:34
opendevreviewMerged openstack/bifrost stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89603314:44
opendevreviewMerged openstack/bifrost stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89603414:44
opendevreviewDmitriy Stremkovskiy proposed openstack/sushy-tools master: Improving nova search filter  https://review.opendev.org/c/openstack/sushy-tools/+/89613014:56
JayFdtantsur: I think your feedback came in too late and the releases PR is already landed. We do explicitly document in those repos not to use them in production, and we do have 1.x.y releases of virtualBMC and virtualPDU14:57
dtantsurokay, no worries14:57
JayFAt this point if someone is using those in production against our explicit request... I sort of feel like they've made their own bed at that point14:58
JayFAnd yet we all know it's true. 🫥14:58
TheJuliaWe've also not made any really breaking changes to sushy in a while14:58
* TheJulia quickly knocks on wood14:58
JayFI talked to mgoddard in a dm about tenks. The usage pattern for that is to check out directly from a repo, they do not use the releases nor do they expect anyone else to.15:05
JayFBut yet we do have an old release cut and in pypi.15:05
JayFIs there anyone else invested in this project who cares about how releases are handled? With this information, I think we could consider this a deliverable we never release similar to molteniron15:05
TheJuliaAdded Third Party CI as a topic to the etherpad, I think I've basically captured the back and forth from yesterday, albeit in likely less words15:11
TheJuliayeah, likely15:11
TheJuliaspeaking of molteniron, if third party CI goes away completely... we can close up molteniron15:12
TheJuliaJayF: I also updated the etherpad link for the PTG bot/page15:13
JayFThank you. I do think there might be value in having a separately scheduled thing with hpe aside from the larger community chat, but I can go either way15:16
TheJuliaThere are definitely some bigger questions we as a community need to discuss15:16
-opendevstatus- NOTICE: The lists.openinfra.dev and lists.starlingx.io sites will be offline briefly for migration to a new server15:31
dtantsurshould I be mildly worried that https://tarballs.opendev.org/openstack/ironic-python-agent/tinyipa/files/ does not have 2023.2?16:13
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/2023.2: Switch testing to stable/2023.2  https://review.opendev.org/c/openstack/bifrost/+/89614316:13
dtantsurthis ^^ will probably fail because of it16:13
dtantsurhmm, something weird happens. not sure if it's transient https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_2de/69f66cf866e2ffba949b3d4bc20d2a177bd96bc3/post/ironic-python-agent-build-image-tinyipa/2def070/job-output.txt16:15
dtantsurokay, now I'm more than mildly worried: https://zuul.opendev.org/t/openstack/builds?job_name=ironic-python-agent-build-image-tinyipa&project=openstack/ironic-python-agent16:15
dtantsurit seems possible that repo.tinycorelinux.net is too slow, or the download anyhow gets stuck16:18
TheJuliahas anything landed on those repos yet?16:18
TheJuliaewww16:18
TheJuliaouch16:18
dtantsurwhat I don't understand is how the check-image job manages to pass. It does the same thing!16:19
TheJuliaI can look in  a few16:21
* TheJulia added yet another item to the etherpad.16:21
dtantsurthx, would be great (a bit late here)16:23
opendevreviewMerged openstack/networking-generic-switch stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604016:33
opendevreviewMerged openstack/networking-generic-switch stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/networking-generic-switch/+/89604116:33
opendevreviewJay Faulkner proposed openstack/networking-generic-switch stable/2023.2: CI fix: Use the un-deprecated v3 etcd API  https://review.opendev.org/c/openstack/networking-generic-switch/+/89611016:39
opendevreviewJay Faulkner proposed openstack/networking-generic-switch stable/2023.2: Honor ngs_save_configuration setting when using batch commands  https://review.opendev.org/c/openstack/networking-generic-switch/+/89611116:39
opendevreviewJay Faulkner proposed openstack/networking-generic-switch stable/2023.2: Do not make actual device changes in bind_port()  https://review.opendev.org/c/openstack/networking-generic-switch/+/89611216:40
opendevreviewJay Faulkner proposed openstack/networking-generic-switch stable/2023.2: Fake: support adding a random sleep and injecting failures  https://review.opendev.org/c/openstack/networking-generic-switch/+/89611316:42
JayFthose (except the CI patch itself) are all depends-on the CI patch, so should be able to pass CI first run16:43
opendevreviewMerged openstack/ironic-python-agent-builder stable/2023.2: Update .gitreview for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89604916:59
opendevreviewMerged openstack/ironic-python-agent-builder stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89605016:59
opendevreviewJulia Kreger proposed openstack/ironic-python-agent-builder master: CI: Change image-build to use Ubuntu Jammy  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89614617:31
JayFlanded ^ as a single core as it's an (attempt) to fix post jobs for ipa in ci17:32
JayFLooking at openstack/ironic open patches, seeing if it's good for me to cut the release17:37
JayFI think we're good? 17:37
JayFI will submit the PR for Ironic 2023.2-assocated release unless someone says otherwise; sometime after lunch my time (probably in around 2 hours)17:38
TheJulianone from me17:42
TheJuliaobjections, that is17:43
iurygregoryno objections from me, just do it!17:58
JayFIronic 22.0.0 in the pipes https://review.opendev.org/c/openstack/releases/+/89615618:19
JayFFYI: https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035191.html -- tl;dr: I do not intend to be the Ironic PTL in D cycle.18:58
TheJuliaAn excellent email :)18:59
JayFAfter a chat with QA team, I'm going to cut our 2023.2 release for ironic-tempest-plugin as well (responsibility is shared but the ideal state is we cut it when release is cut for ironic, apparently)19:08
JayFsomeone had already done this but it's still good info19:10
JayF(in this case, when they tag {series}-last as something goes into EM, they cut a release for current version of that tempest plugin; ours hadn't changed since then)19:22
TheJuliadtantsur: lets talk httpboot tomorrow20:02
JayFheads up: I will not be around upstream very much tomorrow; if anyone needs me urgently please use some comm method that hits my phone21:02
opendevreviewMerged openstack/ironic-python-agent-builder master: CI: Change image-build to use Ubuntu Jammy  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/89614621:03
opendevreviewMaksim Malchuk proposed openstack/bifrost master: Drop hardcode from the ironic-inspector and optimise defaults  https://review.opendev.org/c/openstack/bifrost/+/89616821:58
mmalchukgood night Ironic o/22:06
mmalchukJayF congrats with next turn PTL22:06
JayFthank you22:06
JayFcores, please land https://review.opendev.org/q/status:open+-is:wip+repo:openstack/networking-generic-switch+branch:stable/2023.222:11
JayFthanks!22:11
opendevreviewMerged openstack/networking-generic-switch stable/2023.2: CI fix: Use the un-deprecated v3 etcd API  https://review.opendev.org/c/openstack/networking-generic-switch/+/89611022:22

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!