ameya49 | Julia : 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 triggered | 02:41 |
---|---|---|
opendevreview | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/843371 | 03:15 |
opendevreview | Verification of a change to openstack/ironic master failed: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/842277 | 04:07 |
*** mnasiadka_ is now known as mnasiadka | 05:08 | |
*** tkajinam_ is now known as tkajinam | 05:11 | |
opendevreview | Merged openstack/ironic master: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/842277 | 07:39 |
opendevreview | Maksim Malchuk proposed openstack/tenks master: Use Yoga upper constraints to avoid Python version conflict https://review.opendev.org/c/openstack/tenks/+/843424 | 10:20 |
opendevreview | Mark Goddard proposed openstack/tenks master: Use Yoga upper constraints to avoid Python version conflict https://review.opendev.org/c/openstack/tenks/+/843427 | 10:35 |
iurygregory | good morning Ironic | 11:42 |
jssfr | good morning iurygregory | 11:58 |
*** mnaser_ is now known as mnaser | 12:41 | |
TheJulia | good morning | 12:53 |
iurygregory | good morning TheJulia =) | 13:02 |
* TheJulia hopes dib ipa builds are fixed | 13:04 | |
* TheJulia waits patiently | 13:04 | |
iurygregory | did we merge any patches? | 13:09 |
TheJulia | Hmm | 13:17 |
* TheJulia blinks at irccloud for loosing a message | 13:17 | |
TheJulia | Anyway, yes, it looks like the default change | 13:17 |
opendevreview | Merged openstack/tenks master: Use Yoga upper constraints to avoid Python version conflict https://review.opendev.org/c/openstack/tenks/+/843427 | 13:35 |
TheJulia | metalsmith is still doing centos8 | 13:44 |
TheJulia | iurygregory, arne_wiebalck_, This is kind of impressive https://paste.openstack.org/show/814627/ | 13:50 |
iurygregory | centos8 is just the user image in metalsmith no? | 13:52 |
TheJulia | I'm not sure yet | 13:52 |
TheJulia | possibly, I changed job logs | 13:52 |
iurygregory | WOW re the paste | 13:52 |
mgoddard | Hi, is anyone aware of a way to automatically create port groups for nodes? | 13:59 |
TheJulia | to articulate them, I think the conundrum was always static configs on switches versus dynamic lacp | 14:10 |
TheJulia | I *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 problematic | 14:11 |
mgoddard | TheJulia: I'm looking at static LACP config, using James Denton's recent patch to NGS | 14:13 |
mgoddard | was hoping for some way to create the port groups using e.g. ironic inspector | 14:13 |
mgoddard | annoyingly it doesn't give the port group name in the LLDP data | 14:14 |
TheJulia | That is just for human consumption right?\ | 14:15 |
mgoddard | I need to set local_link_connection.port_id=bondX on the ports | 14:19 |
TheJulia | Hmm, that is the remote side port... | 14:21 |
TheJulia | I guess the only way to do that would be to consolidate the view of both sides | 14:21 |
mgoddard | one thing that might work is for the IPA ramdisk to detect LACP, then setup a bond, before running LLDP | 14:22 |
TheJulia | to iterate though all of the interfaces attempting to setup bonds | 14:23 |
mgoddard | probably I don't get the switch_port_link_aggregation_id field because LACP wasn't negotiated | 14:23 |
TheJulia | does the lacp exchange include the remote port name? | 14:23 |
TheJulia | I thought i did, at least on cisco, but that might not work on others | 14:23 |
TheJulia | oh, well. | 14:23 |
mgoddard | Yes, we get the remote port name, just not the port group to which it belongs | 14:23 |
mgoddard | Really, LLDP should give us enough info to know which links should attempt LACP together | 14:24 |
TheJulia | true | 14:25 |
mgoddard | But it seems not | 14:25 |
mgoddard | switch_port_description | bond member of bond16 | 14:25 |
mgoddard | close, but not quite :) | 14:25 |
TheJulia | heh | 14:25 |
JasonF | TheJulia: that's some cool data | 15:26 |
*** JasonF is now known as JayF | 15:26 | |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Guard shared device/cluster filesystems https://review.opendev.org/c/openstack/ironic-python-agent/+/839084 | 16:25 |
TheJulia | well, ipa building seems to be happier | 16:25 |
TheJulia | iurygregory: yeah, I think metalsmith just deployes 8 | 16:29 |
TheJulia | deploys | 16:29 |
TheJulia | https://review.opendev.org/c/openstack/ironic-python-agent/+/843162 <-- ironic-cores. Moves the minimum to a more sane non-python2 supporting version | 16:34 |
iurygregory | TheJulia, yeah from what I recall this was what dmitry told me | 16:45 |
opendevreview | Julia Kreger proposed openstack/ironic master: increase disk_erasure_coconcurrency https://review.opendev.org/c/openstack/ironic/+/838592 | 16:50 |
TheJulia | I'm afraid that has no unit tests, but then again, it is all IPA side behavior/operation | 16:51 |
TheJulia | stripped wip/added reno, etc | 16:51 |
TheJulia | iurygregory: we should just do a super simple skiplevel job | 16:57 |
TheJulia | ngs is broken on dss key lenghts | 16:57 |
iurygregory | ouch | 16:58 |
iurygregory | D: | 16:58 |
TheJulia | "Unknown exception: q must be exactly 160, 224, or 256 bits long" | 16:58 |
TheJulia | https://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.txt | 16:58 |
iurygregory | "funny" | 16:59 |
iurygregory | I like the fact the exception has "q" O.o | 17:00 |
iurygregory | because it doesn't make any sense LOL | 17:00 |
TheJulia | https://storyboard.openstack.org/#!/story/2010053 | 17:11 |
TheJulia | It is interpretting the rsa key as a dss key | 17:11 |
TheJulia | but seems to detonate after upgrade | 17:12 |
opendevreview | Verification of a change to openstack/ironic master failed: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/842597 | 17:28 |
iurygregory | oh jesus, rsa as dsa | 17:59 |
iurygregory | TheJulia, I'm wondering if we can try to pin versions for paramiko | 18:00 |
iurygregory | https://github.com/paramiko/paramiko/issues/1839#issuecomment-1132447370 | 18:00 |
arne_wiebalck_ | iurygregory: sorry if I missed it, but is the Ironic team dinner for the summit already settled? | 18:04 |
iurygregory | arne_wiebalck_, seems like it can be Tuesday/Wednesday based on the votes in the etherpad | 18:05 |
iurygregory | I was going to wait till this friday | 18:05 |
arne_wiebalck_ | iurygregory: ++ | 18:05 |
* arne_wiebalck_ just wants to make sure he does not miss it | 18:05 | |
iurygregory | :D | 18:06 |
arne_wiebalck_ | cheese fondue for everyone! | 18:06 |
arne_wiebalck_ | :-D | 18:06 |
iurygregory | XD | 18:06 |
TheJulia | iurygregory: I guess is tarted in paramiko 2.7.x | 18:35 |
TheJulia | so... maybe?! | 18:35 |
iurygregory | maybe.. | 18:54 |
TheJulia | I'm not entirely sure if anyone would freak if we pin paramiko back... | 18:59 |
* TheJulia goes to codesearch | 18:59 | |
TheJulia | nova requires 2.7.1 | 18:59 |
TheJulia | so any explicit pin we put in, will conflict with g-r | 19:00 |
TheJulia | cinder requires 2.7.2 | 19:01 |
TheJulia | same with manila | 19:01 |
iurygregory | ouch =( | 19:44 |
iurygregory | maybe worth a thread in openstack-discuss to see ? | 19:45 |
opendevreview | Merged openstack/ironic master: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/842597 | 19:54 |
opendevreview | Julia Kreger proposed openstack/ironic stable/yoga: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843511 | 20:22 |
opendevreview | Julia Kreger proposed openstack/ironic stable/xena: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843512 | 20:22 |
opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843513 | 20:23 |
opendevreview | Julia Kreger proposed openstack/ironic stable/victoria: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843514 | 20:23 |
opendevreview | Julia Kreger proposed openstack/ironic stable/ussuri: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843515 | 20:23 |
opendevreview | Julia Kreger proposed openstack/ironic stable/train: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843516 | 20:23 |
TheJulia | iurygregory: I guess we need to just close all of the pre-train branches | 20:24 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/20.2: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843517 | 20:24 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/20.0: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843518 | 20:24 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/19.0: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843519 | 20:24 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.1: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843520 | 20:25 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.0: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843521 | 20:25 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/16.2: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843522 | 20:25 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/16.1: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843523 | 20:25 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/15.2: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843524 | 20:25 |
opendevreview | Julia Kreger proposed openstack/ironic bugfix/15.1: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic/+/843525 | 20:26 |
TheJulia | Yeah, can we nuke older bugfix branches tooooo?! | 20:26 |
TheJulia | I guess on a plus side, this gives us a solid understanding of CI's state | 20:26 |
stevebaker[m] | good morning | 21:12 |
TheJulia | o/ | 21:47 |
iurygregory | TheJulia, for older branches pre-train ++ for bugfix some of them we can =) | 22:21 |
iurygregory | sorry I was at the gym =) | 22:21 |
iurygregory | maybe a topic for our weekly meeting on monday? | 22:24 |
TheJulia | iurygregory: I'm out on monday. It is a holiday here | 22:44 |
iurygregory | ohhh | 23:25 |
iurygregory | well, I can discuss the topic with the folks present if you want | 23:26 |
iurygregory | or we can just send and email to see if anyone is against the idea | 23:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!