| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent stable/wallaby: WIP: Multipath Hardware path handling https://review.opendev.org/c/openstack/ironic-python-agent/+/837784 | 00:15 |
|---|---|---|
| opendevreview | Merged x/sushy-oem-idrac stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/x/sushy-oem-idrac/+/832510 | 01:30 |
| opendevreview | Merged x/sushy-oem-idrac stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/x/sushy-oem-idrac/+/832511 | 01:46 |
| opendevreview | Merged x/sushy-oem-idrac master: Add Python3 Zed unit tests https://review.opendev.org/c/x/sushy-oem-idrac/+/832512 | 02:08 |
| arne_wiebalck | Good morning, Ironic! | 08:53 |
| janders | good morning arne_wiebalck o/ | 09:08 |
| arne_wiebalck | hey janders o/ | 09:08 |
| dtantsur | morning folks | 09:55 |
| dtantsur | effectively a Friday today :) | 09:55 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-lib master: Update jobs names https://review.opendev.org/c/openstack/ironic-lib/+/837705 | 10:46 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Use Werkzeug modern version https://review.opendev.org/c/openstack/ironic-python-agent/+/837661 | 10:47 |
| iurygregory | good morning Ironic | 10:56 |
| dtantsur | hey iurygregory | 11:53 |
| dtantsur | how is the CI feeling today? any updates? | 12:07 |
| * dtantsur really wants https://review.opendev.org/c/openstack/ironic/+/837379 merged | 12:07 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a note about duplicate parameters with %default% https://review.opendev.org/c/openstack/ironic/+/837860 | 12:10 |
| dtantsur | iurygregory: ^^^ | 12:10 |
| iurygregory | dtantsur, ack! | 12:11 |
| iurygregory | Tks! | 12:11 |
| iurygregory | I'm wondering if CI is back to normal.. | 12:12 |
| * iurygregory checks the thread in the ML | 12:13 | |
| iurygregory | ok, seems like we can give a try to merge things I would say (based on the emails on the thread [all] Devstack jobs are failing due to a git security fix | 12:39 |
| * iurygregory hopes CI will be happy <3 | 12:40 | |
| dtantsur | arne_wiebalck: hey! do you plan on a tweet with the recording of the SIG meeting? | 13:34 |
| dtantsur | I'd be really curious to check it out and share further | 13:35 |
| arne_wiebalck | I will ... is it up already? | 13:36 |
| * arne_wiebalck checking ... | 13:36 | |
| arne_wiebalck | Oh, it is! stevebaker[m] FTW ! | 13:36 |
| arne_wiebalck | dtantsur: done, thanks for the ping! | 13:41 |
| arne_wiebalck | and thanks again stevebaker[m] for the ultrafast editing! | 13:42 |
| opendevreview | Verification of a change to openstack/ironic master failed: Allow reusing defaults in per-node kernel_append_params https://review.opendev.org/c/openstack/ironic/+/837379 | 13:59 |
| iurygregory | dtantsur, seems like metalsmith is sad ^ | 14:09 |
| iurygregory | seems like the same problem from devstack? Versioning for this project requires either an sdist tarball, or access to an upstream git repository. It's also possible that there is a mismatch between the package name in setup.cfg and the argument given to pbr.version.VersionInfo. Project name metalsmith was given, but was not able to be found. | 14:10 |
| iurygregory | shall we disable the job for now to get this merged today? | 14:11 |
| dtantsur | iurygregory: we don't have any ideas for fixing it? | 14:12 |
| dtantsur | I think we can lie to pbr about versions, but I wonder if anyone is fixing it? | 14:13 |
| iurygregory | well, for devstack they fixed it seems | 14:13 |
| iurygregory | but I'm wondering why it didn't work for metalsmith... | 14:13 |
| dtantsur | iurygregory: it's not a devstack location... | 14:14 |
| * dtantsur understands the problem but asks the infra first if they want a global fix | 14:16 | |
| iurygregory | ok, devstack was running but only failed to install metalsmith after | 14:18 |
| dtantsur | https://opendev.org/openstack/metalsmith/src/branch/master/playbooks/integration/initial-setup.yaml#L21 | 14:18 |
| dtantsur | this is not affected by the fix | 14:18 |
| ayoung | rpittau, it seemed to work on a clean just now. Did something get changed/committed? | 14:19 |
| iurygregory | yeah | 14:19 |
| iurygregory | ayoung, I don't think rpittau is around today (is a bank holiday if I recall) | 14:20 |
| ayoung | iurygregory, thanks...the issue was that I had built my own ipxe.efi, but was using someone elses | 14:20 |
| ayoung | 2 different issues, of course | 14:20 |
| iurygregory | ayoung, oh =)yeah | 14:20 |
| ayoung | the issue with DNS was not a DNS issue at all. | 14:20 |
| ayoung | the ipxe.efi I was using was built specifically for redirecting to a central http server for images. I wanted a generic, fetch from the same server that hosted the tftp binary | 14:21 |
| ayoung | once I realized it was a hard coded value, it made sense | 14:21 |
| ayoung | WRT to the werkzeug, I will know in a few minutes if it is still borked | 14:22 |
| ayoung | but instpector is running on a clean right now, which is a good sign | 14:22 |
| ayoung | OTOH, I get this error | 14:22 |
| ayoung | [ 145.327836] ironic-python-agent[2598]: 2022-04-14 14:22:45.185 2598 WARNING ironic_python_agent.ironic_api_client [-] Failed looking up node with addresses '0c:42:a1:5a:97:44,0c:42:a1:5a:97:45,02:e2:84:91:2d:89' at http://192.168.116.64:6385. Error 404: Resource could not be found.. Check if inspection has completed | 14:22 |
| ayoung | It looks like it checks ALL of the interfaces instead of just one (the one for the port created when I enroll the node) | 14:23 |
| ayoung | Do I need to kick of an explicit inspect? | 14:23 |
| opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Add a workaround to fix the CI https://review.opendev.org/c/openstack/metalsmith/+/837886 | 14:24 |
| dtantsur | iurygregory: maybe ^^ | 14:25 |
| iurygregory | dtantsur, nice one | 14:25 |
| dtantsur | ayoung: the agent does not know which NIC you enrolled | 14:25 |
| dtantsur | so it has to list all of them | 14:25 |
| iurygregory | this fix reminds me when vscode asks "Do you trust the authors of the files in this folder?" | 14:26 |
| * dtantsur is wondering if we need an image building service for ironic | 15:02 | |
| dtantsur | iurygregory: https://review.opendev.org/c/openstack/metalsmith/+/837886 passed the CI, wanna approve? | 15:53 |
| iurygregory | dtantsur, right now! :D | 15:54 |
| iurygregory | +W since we don't have much people around today | 15:54 |
| iurygregory | after it merges I will put a recheck in your patch | 15:54 |
| dtantsur | thank you! | 15:57 |
| iurygregory | yw | 15:59 |
| dtantsur | Starting the long weekend, see you on Tuesday folks | 16:13 |
| dtantsur | o/ | 16:13 |
| iurygregory | bye dtantsur o/ enjoy the long weekend! | 16:14 |
| opendevreview | Merged openstack/metalsmith master: Add a workaround to fix the CI https://review.opendev.org/c/openstack/metalsmith/+/837886 | 17:48 |
| opendevreview | Verification of a change to openstack/ironic master failed: Allow reusing defaults in per-node kernel_append_params https://review.opendev.org/c/openstack/ironic/+/837379 | 18:11 |
| bkra | Hello hello, is it possible to set a customized dict on a Node object for automation purposes? | 18:24 |
| bkra | Im aware of the Traits feature but this only supports string values stuff in a list | 18:24 |
| bkra | NVM, I suppose I can use the --extra for that case. | 18:31 |
| opendevreview | Merged openstack/ironic master: Allow reusing defaults in per-node kernel_append_params https://review.opendev.org/c/openstack/ironic/+/837379 | 21:21 |
| JayF | bkra: you just want to essentially store metadata about your nodes in the DB alongside? | 21:26 |
| JayF | I think node.extra is the place for that | 21:26 |
| JayF | So you're on the right track :) | 21:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!