Friday, 2024-07-12

TheJuliaAnyway, either some help or future hardware type would be helpful. Downside of snmp is even though it is old and everything else, it is still quite popular.00:04
opendevreviewJacob Anders proposed openstack/sushy-tools master: [WIP] Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950001:38
opendevreviewJacob Anders proposed openstack/sushy-tools master: Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950004:23
opendevreviewOpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/ironic/+/92380904:43
rpittaugood morning ironic! o/07:23
rpittauTheJulia: thanks for the email on snmp07:24
rpittauJayF: I believe I shared it all here https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/921998, plus some patches that are merged on DIB, I see you added the one for the profile change which I didn't have the time to do (downstream priorities overrule)07:26
iurygregorygood morning ironic09:20
opendevreviewJacob Anders proposed openstack/sushy-tools master: Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950010:49
opendevreviewJacob Anders proposed openstack/sushy-tools master: Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950011:02
opendevreviewJacob Anders proposed openstack/sushy-tools master: Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950011:20
opendevreviewJacob Anders proposed openstack/sushy-tools master: Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950011:24
sylvrHello ! it seems I have an issue with my ironic service being unable to store/process the introspection data from my IPAramdisk12:22
TheJuliasylvr: what are the symptoms your encountering?12:44
sylvrall my nodes are stuck in enroll state and the driver_info field are empty12:46
sylvrwhen checking the logs on the IPA, everything looks fine (to me at least)12:46
sylvrand the logs on the seed/bifrost don't report error12:47
TheJuliaSo your expecting inspection to populate all of your driver_info fields?12:50
sylvrno, I configured them with kayobe (username/password) but the ipmi_address should be populated by node with the result of the introspection right ?12:52
TheJuliaNo guarentee it will and far less likely on newer hardware because that requires the inband ipmi bridge to be enabled *and* the host to be configured with a static (non-dhcp/dynamic) address which can be determiend over the IPMI bridge12:52
sylvrI'm using old hardware12:53
TheJuliaFurthermore you cannot inspect a node we have zero information about12:53
sylvrthe node are registered in bifrost, they're booting through PXE the IPAramdisk and the heartbeat is OK12:54
TheJuliaso a network port is known12:54
TheJuliais the ipmi inbahd interface enabled on the hardware?12:55
TheJuliainband12:55
sylvrI'm guessing yes since I can run "ipmitool lan print" locally on the node (after ssh on the IPA) and get the BMC addresses12:56
TheJuliaokay12:56
TheJuliaso did you create an introspection rule to publish the ipmi_address? or do you have hardware discovery enabled?12:56
TheJuliaif you have nodes pre-created, discovery won't work12:56
TheJuliasince you already have nodes12:57
TheJuliaso the only thing I can think is an introspection rule12:57
sylvr(auto)discovery, the rules are set uisng Kayobe12:57
TheJuliaso yeah, you've short circuited the process then by having them enrolled in advance12:57
sylvrwhat do you mean ?12:57
TheJuliaSo if a node is already known by ironic, then discovery path where ipmi_address is set, won't be executed nor set12:58
TheJuliasince it is no longer discovery12:58
sylvrthe node are known by ironic when they're booting the IPAramdisk, not before in my case12:59
TheJuliadiscovery only works when the node is unknown to ironic13:00
TheJuliai.e. the ramdisk boots and Ironic knows nothing about the node13:00
sylvryes13:00
TheJuliawhat do you get with ipmitool lan print?13:00
iurygregoryCisco why you do this to me... 13:01
TheJuliaiurygregory: "Because we can!"13:01
iurygregoryTheJulia, https://paste.opendev.org/show/be1nxWaZ5Hhn3BZYYTx9/13:01
sylvrTheJulia : https://pastebin.com/br9AVjm713:02
iurygregorythis is just interesting...13:02
TheJuliahow so?13:02
TheJuliasylvr: okay then, what about inspector's logs? What do you see in them?13:03
sylvrgrep -i error /var/log/kolla/ironic/ironic.log : https://pastebin.com/2U3H4irD13:07
sylvrerror in /var/log/kolla/ironic-inspector/inspector.log :  https://pastebin.com/6mD7NrPi13:07
sylvrinfo&debug in /var/log/kolla/ironic-inspector/inspector.log : https://pastebin.com/0ekcTuK313:07
iurygregoryTheJulia, just the whole description of the params XD is weird since I never saw in any hardware before13:09
rpittauTheJulia, JayF, iurygregory, can you please have a look at cid's patch  https://review.opendev.org/c/openstack/ironic/+/923570? kind of a sensitive change, thanks!13:19
iurygregoryrpittau, will add to my list13:20
rpittauthanks!13:21
iurygregoryrpittau, TheJulia when someone asks "is it imperative to have the ejectmedia available? can we do it manually if needed?"13:22
iurygregoryhow should I react to it?13:22
iurygregory"M4 are EoL for Cisco, and due to their policies i can not open a case for EoL hw"13:23
rpittaufacepalm intensifies13:23
iurygregorycan I cry?13:23
opendevreviewMerged openstack/ironic-inspector master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/ironic-inspector/+/92381113:23
opendevreviewcid proposed openstack/ironic master: Follow-up to change; #922951 breaks RAID  https://review.opendev.org/c/openstack/ironic/+/92357013:25
sylvrTheJulia : I need to disconnect from IRC, joinable via mail13:36
opendevreviewMerged openstack/sushy-tools master: Add support for BIOS update emulation  https://review.opendev.org/c/openstack/sushy-tools/+/90950013:37
opendevreviewVerification of a change to openstack/ironic master failed: Localdev: disable caching and related periodic  https://review.opendev.org/c/openstack/ironic/+/92345313:42
TheJuliaHave a wonderful weekend folks, I'm taking the day off13:53
rpittauTheJulia: enjoy!13:55
iurygregoryTheJulia, enjoy14:01
iurygregoryrpittau, the EjectMedia works, but they give a strange response =X   {"error":{    "code":"InternalError",    "message":"/usr/local/share/lua/5.1/dkjson.lua:397: bad argument #1 to 'strfind' (string expected, got nil)"  }, can we handle this in sushy? =( I know is not the best scenario but seems like Cisco won't provide firmware updates for the hardware they are using...14:03
TheJulia... that is pretty bad14:09
rpittauiurygregory: that really look like a bug in the firmware, and I don't think should be on us to do anything honestly14:09
TheJuliaI wonder what the string is and what evil can be performed14:10
* TheJulia wonders if that is a CVE...14:10
TheJuliait does leak context into the internals of th eBMC14:10
TheJuliaPowered by Lua14:10
TheJulias/th\ eBMC/the\ BMC/14:10
rpittauthe problem is that all the is gone out of support14:10
iurygregoryyeah14:10
TheJuliastill a potential issue14:10
rpittauyep, that's why it should not be used in production :)14:11
iurygregoryand we have customer case.. I'm talking with the TSE =X14:11
TheJuliayeah, best to have customer apply vendor pressure14:11
TheJuliaAlthough, some countryies/governmants might find that to be cve-ey in nature and expect that issue to be fixed by hardware vendor if the hardware was shipped at all in the last 5 years14:12
iurygregorysadly M4 are EoL for Cisco, and due to their policies i can not open a case for EoL hw,14:12
iurygregoryis it imperative to have the ejectmedia available?14:12
iurygregorycan we do it manually if needed?14:12
iurygregorythis was the answer from the customer when I told it was a bad firmware..14:12
TheJuliai think it is imperitive, we shouldn't leave a bootable thing which has injected it's own bootloader record14:12
TheJuliathe value is automation and avoiding the sharp edges to the technology14:13
TheJuliathis is a nearly unavoidable sharp edge to that server line14:13
rpittauif they want to press the eject media button on hundreds of server who are we to stop them?14:13
iurygregoryrpittau, will ironic like they doing this? XD14:14
iurygregorythem*14:14
TheJulianot responsible if they bootup tot he wrong workload before they hit eject14:14
rpittauthis ^14:14
iurygregorythe install will probably go crazy if they eject at the wrong time14:15
TheJuliahttps://www.youtube.com/watch?v=1umEXpGHc0E14:15
iurygregoryand if ironic ejects and this message causes error in the process the install will also fail14:15
iurygregoryso I'm like "I have no idea on what to do now"14:16
TheJuliaNothing you can do14:19
TheJuliaI forgot how sad that song was14:20
TheJuliaI should have remembered given what happened to a friend around when it was released14:20
rpittau:/14:21
TheJuliaOn a plus side, YouTube has started 90’s alt rock hits14:26
iurygregoryrpittau, do you want me to close the bug mentioning is the hardware with this problem is eol? or we should consider a fix for the insert at least?14:33
rpittauthe insert part may be useful and that's actually something that we can handle, so I guess we can leave it open14:34
iurygregoryrpittau, awesome. I will keep that open and improve some tests when I'm back from PTO14:35
iurygregorynow I'm not in a hurry to wrap up the patch14:35
rpittauyep, thanks :)14:35
nick_hi all. i'm troubleshooting a problem running ironic in kubernetes (with openstack-helm). in kubernetes, a volume cannot contain a hard link to anything in a different volume. so i'm trying to set the config to use a single volume. i came across this tmpdir setting: https://opendev.org/openstack/ironic/src/branch/master/ironic/common/utils.py#L254-L256 -  is the only way to set tmpdir via the `TMPDIR` environment variable?14:41
TheJulianick_: As a service project, we're unaware of what openstack-helm has put in place configuration wise. We generally take such configuration via a configuration file, but it looks like there is some fallback logic logic which calls this helper https://docs.python.org/3/library/tempfile.html#tempfile.gettempdir14:48
TheJuliafigures....14:49
* TheJulia sighs14:49
JayFrpittau: cid had a minor typo fix, your +2 was lost here (mine was added) https://review.opendev.org/c/openstack/ironic/+/92357014:53
JayFrpittau: so you can land it if ya want14:53
rpittauoh! didn't see the sneaky print there :)14:53
rpittauapproved!14:54
nick_so i can set tmpdir in ironic.conf? i tried putting it in the default section but ironic didn't honor it. it only worked when i used a TMPDIR environment variable i set. but i'd much prefer to put this in ironic.conf if possible14:54
JayFprint('cool')14:54
JayFnick_: let me grep the same config file for "temp" and "tmp", I have an environment to generate one14:55
JayFbut even if you have to set an ENV, you should be able to do that e.g. in a systemd service file14:55
JayFnick_: [DEFAULT]/tmpdir -- it's documented in the sample config14:56
TheJuliahttps://github.com/openstack/ironic/blob/master/ironic/conf/default.py#L41714:58
TheJuliatempdir, not tmpdir14:58
nick_i see, it's `tempdir`, not `tmpdir`14:58
rpittaubye everyone have a great weekend! o/15:03
nick_thanks all!15:05
opendevreviewMerged openstack/ironic master: Localdev: disable caching and related periodic  https://review.opendev.org/c/openstack/ironic/+/92345315:45
opendevreviewVerification of a change to openstack/ironic master failed: Follow-up to change; #922951 breaks RAID  https://review.opendev.org/c/openstack/ironic/+/92357016:05
MatthewHello all, excited to joining this chat room as a OpenStack lover!16:15
*** Matthew is now known as Guest128916:15
JayFo/16:50
cidHello, you're welcome Matthew17:16
opendevreviewVerification of a change to openstack/ironic master failed: Follow-up to change; #922951 breaks RAID  https://review.opendev.org/c/openstack/ironic/+/92357017:58
opendevreviewcid proposed openstack/python-ironicclient master: Add CLI support for runbooks  https://review.opendev.org/c/openstack/python-ironicclient/+/92410120:10
opendevreviewcid proposed openstack/python-ironicclient master: Add CLI support for runbooks  https://review.opendev.org/c/openstack/python-ironicclient/+/92410120:24
iurygregorybye everyone, see you on July 22  o/20:32
opendevreviewcid proposed openstack/ironic master: Self-Service via Runbooks  https://review.opendev.org/c/openstack/ironic/+/92214220:55
cidHappy weekend o//21:02
JayFo/21:03
opendevreviewMerged openstack/ironic master: Follow-up to change; #922951 breaks RAID  https://review.opendev.org/c/openstack/ironic/+/92357021:11

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