Thursday, 2022-05-26

ameya49Julia : Im was running existing ironic_tempest_plugin test_introspection_basic test case, where after introspection if finished, it calls boot_instance method where create_server is been triggered02:41
opendevreviewOpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/ironic-inspector/+/84337103:15
opendevreviewVerification of a change to openstack/ironic master failed: devstack: use CentOS 9 for DIB IPA builds  https://review.opendev.org/c/openstack/ironic/+/84227704:07
*** mnasiadka_ is now known as mnasiadka05:08
*** tkajinam_ is now known as tkajinam05:11
opendevreviewMerged openstack/ironic master: devstack: use CentOS 9 for DIB IPA builds  https://review.opendev.org/c/openstack/ironic/+/84227707:39
opendevreviewMaksim Malchuk proposed openstack/tenks master: Use Yoga upper constraints to avoid Python version conflict  https://review.opendev.org/c/openstack/tenks/+/84342410:20
opendevreviewMark Goddard proposed openstack/tenks master: Use Yoga upper constraints to avoid Python version conflict  https://review.opendev.org/c/openstack/tenks/+/84342710:35
iurygregorygood morning Ironic11:42
jssfrgood morning iurygregory11:58
*** mnaser_ is now known as mnaser12:41
TheJuliagood morning12:53
iurygregorygood morning TheJulia =)13:02
* TheJulia hopes dib ipa builds are fixed13:04
* TheJulia waits patiently13:04
iurygregorydid we merge any patches?13:09
TheJuliaHmm13:17
* TheJulia blinks at irccloud for loosing a message13:17
TheJuliaAnyway, yes, it looks like the default change13:17
opendevreviewMerged openstack/tenks master: Use Yoga upper constraints to avoid Python version conflict  https://review.opendev.org/c/openstack/tenks/+/84342713:35
TheJuliametalsmith is still doing centos813:44
TheJuliaiurygregory, arne_wiebalck_, This is kind of impressive https://paste.openstack.org/show/814627/13:50
iurygregorycentos8 is just the user image in metalsmith no?13:52
TheJuliaI'm not sure yet13:52
TheJuliapossibly, I changed job logs13:52
iurygregoryWOW re the paste13:52
mgoddardHi, is anyone aware of a way to automatically create port groups for nodes?13:59
TheJuliato articulate them, I think the conundrum was always static configs on switches versus dynamic lacp14:10
TheJuliaI *think* with dynamic lacp, it should be totally doable, although hjensas recently became aware of some switches out there that forward LACP broadcasts.... so it may slightly problematic14:11
mgoddardTheJulia: I'm looking at static LACP config, using James Denton's recent patch to NGS14:13
mgoddardwas hoping for some way to create the port groups using e.g. ironic inspector 14:13
mgoddardannoyingly it doesn't give the port group name in the LLDP data14:14
TheJuliaThat is just for human consumption right?\14:15
mgoddardI need to set local_link_connection.port_id=bondX on the ports14:19
TheJuliaHmm, that is the remote side port...14:21
TheJuliaI guess the only way to do that would be to consolidate the view of both sides14:21
mgoddardone thing that might work is for the IPA ramdisk to detect LACP, then setup a bond, before running LLDP14:22
TheJuliato iterate though all of the interfaces attempting to setup bonds14:23
mgoddardprobably I don't get the switch_port_link_aggregation_id field because LACP wasn't negotiated14:23
TheJuliadoes the lacp exchange include the remote port name?14:23
TheJuliaI thought i did, at least on cisco, but that might not work on others14:23
TheJuliaoh, well.14:23
mgoddardYes, we get the remote port name, just not the port group to which it belongs14:23
mgoddardReally, LLDP should give us enough info to know which links should attempt LACP together14:24
TheJuliatrue14:25
mgoddardBut it seems not14:25
mgoddardswitch_port_description              | bond member of bond1614:25
mgoddardclose, but not quite :)14:25
TheJuliaheh14:25
JasonFTheJulia: that's some cool data15:26
*** JasonF is now known as JayF15:26
opendevreviewJulia Kreger proposed openstack/ironic-python-agent master: Guard shared device/cluster filesystems  https://review.opendev.org/c/openstack/ironic-python-agent/+/83908416:25
TheJuliawell, ipa building seems to be happier16:25
TheJuliaiurygregory: yeah, I think metalsmith just deployes 816:29
TheJuliadeploys16:29
TheJuliahttps://review.opendev.org/c/openstack/ironic-python-agent/+/843162 <-- ironic-cores. Moves the minimum to a more sane non-python2 supporting version16:34
iurygregoryTheJulia, yeah from what I recall this was what dmitry told me16:45
opendevreviewJulia Kreger proposed openstack/ironic master: increase disk_erasure_coconcurrency  https://review.opendev.org/c/openstack/ironic/+/83859216:50
TheJuliaI'm afraid that has no unit tests, but then again, it is all IPA side behavior/operation16:51
TheJuliastripped wip/added reno, etc16:51
TheJuliaiurygregory: we should just do a super simple skiplevel job16:57
TheJuliangs is broken on dss key lenghts16:57
iurygregoryouch16:58
iurygregoryD:16:58
TheJulia"Unknown exception: q must be exactly 160, 224, or 256 bits long"16:58
TheJuliahttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_bb0/840530/12/check/ironic-grenade-multinode-multitenant/bb0df26/controller/logs/screen-q-svc.txt16:58
iurygregory"funny"16:59
iurygregoryI like the fact the exception has "q" O.o17:00
iurygregorybecause it doesn't make any sense LOL17:00
TheJuliahttps://storyboard.openstack.org/#!/story/201005317:11
TheJuliaIt is interpretting the rsa key as a dss key17:11
TheJuliabut seems to detonate after upgrade17:12
opendevreviewVerification of a change to openstack/ironic master failed: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84259717:28
iurygregoryoh jesus, rsa as dsa17:59
iurygregoryTheJulia, I'm wondering if we can try to pin versions for paramiko18:00
iurygregoryhttps://github.com/paramiko/paramiko/issues/1839#issuecomment-113244737018:00
arne_wiebalck_iurygregory: sorry if I missed it, but is the Ironic team dinner for the summit already settled?18:04
iurygregoryarne_wiebalck_, seems like it can be Tuesday/Wednesday based on the votes in the etherpad18:05
iurygregoryI was going to wait till this friday18:05
arne_wiebalck_iurygregory: ++18:05
* arne_wiebalck_ just wants to make sure he does not miss it18:05
iurygregory:D18:06
arne_wiebalck_cheese fondue for everyone!18:06
arne_wiebalck_:-D18:06
iurygregoryXD18:06
TheJuliaiurygregory: I guess is tarted in paramiko 2.7.x18:35
TheJuliaso... maybe?!18:35
iurygregorymaybe..18:54
TheJuliaI'm not entirely sure if anyone would freak if we pin paramiko back...18:59
* TheJulia goes to codesearch18:59
TheJulianova requires 2.7.118:59
TheJuliaso any explicit pin we put in, will conflict with g-r19:00
TheJuliacinder requires 2.7.219:01
TheJuliasame with manila19:01
iurygregoryouch =(19:44
iurygregorymaybe worth a thread in openstack-discuss to see ?19:45
opendevreviewMerged openstack/ironic master: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84259719:54
opendevreviewJulia Kreger proposed openstack/ironic stable/yoga: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351120:22
opendevreviewJulia Kreger proposed openstack/ironic stable/xena: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351220:22
opendevreviewJulia Kreger proposed openstack/ironic stable/wallaby: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351320:23
opendevreviewJulia Kreger proposed openstack/ironic stable/victoria: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351420:23
opendevreviewJulia Kreger proposed openstack/ironic stable/ussuri: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351520:23
opendevreviewJulia Kreger proposed openstack/ironic stable/train: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351620:23
TheJuliaiurygregory: I guess we need to just close all of the pre-train branches20:24
opendevreviewJulia Kreger proposed openstack/ironic bugfix/20.2: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351720:24
opendevreviewJulia Kreger proposed openstack/ironic bugfix/20.0: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351820:24
opendevreviewJulia Kreger proposed openstack/ironic bugfix/19.0: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84351920:24
opendevreviewJulia Kreger proposed openstack/ironic bugfix/18.1: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84352020:25
opendevreviewJulia Kreger proposed openstack/ironic bugfix/18.0: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84352120:25
opendevreviewJulia Kreger proposed openstack/ironic bugfix/16.2: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84352220:25
opendevreviewJulia Kreger proposed openstack/ironic bugfix/16.1: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84352320:25
opendevreviewJulia Kreger proposed openstack/ironic bugfix/15.2: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84352420:25
opendevreviewJulia Kreger proposed openstack/ironic bugfix/15.1: CI: Removing ironic job queue  https://review.opendev.org/c/openstack/ironic/+/84352520:26
TheJuliaYeah, can we nuke older bugfix branches tooooo?!20:26
TheJuliaI guess on a plus side, this gives us a solid understanding of CI's state20:26
stevebaker[m]good morning21:12
TheJuliao/21:47
iurygregoryTheJulia, for older branches pre-train ++ for bugfix some of them we can =)22:21
iurygregorysorry I was at the gym =)22:21
iurygregorymaybe a topic for our weekly meeting on monday?22:24
TheJuliaiurygregory: I'm out on monday. It is a holiday here22:44
iurygregoryohhh23:25
iurygregorywell, I can discuss the topic with the folks present if you want23:26
iurygregoryor we can just send and email to see if anyone is against the idea23:26

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