| *** diurnalist has quit IRC | 00:16 | |
| TheJulia | Things go | 00:23 |
|---|---|---|
| TheJulia | Hey folks, I would really appreciate reviews on https://review.opendev.org/#/c/740360/2 I just spent 3 hours looking at a bug that was actually the same exact bug all along | 00:24 |
| patchbot | patch 740360 - ironic - Remove locks before RPC bus is started - 2 patch sets | 00:24 |
| *** Goneri has quit IRC | 00:39 | |
| *** SpamapS has quit IRC | 00:44 | |
| *** diurnalist has joined #openstack-ironic | 00:46 | |
| *** portdirect has quit IRC | 00:51 | |
| *** gmann has quit IRC | 00:51 | |
| *** gmann has joined #openstack-ironic | 00:51 | |
| *** portdirect has joined #openstack-ironic | 00:51 | |
| *** seongsoocho has quit IRC | 00:52 | |
| *** johnhaan has quit IRC | 00:52 | |
| *** pas-ha has quit IRC | 00:53 | |
| *** seongsoocho has joined #openstack-ironic | 00:54 | |
| *** pas-ha has joined #openstack-ironic | 00:55 | |
| *** SpamapS has joined #openstack-ironic | 00:58 | |
| *** ociuhandu has joined #openstack-ironic | 01:01 | |
| *** ociuhandu has quit IRC | 01:05 | |
| *** johnhaan has joined #openstack-ironic | 01:08 | |
| *** ricolin has joined #openstack-ironic | 01:21 | |
| *** mkrai has joined #openstack-ironic | 01:44 | |
| *** yaawang has quit IRC | 01:45 | |
| *** yaawang has joined #openstack-ironic | 01:46 | |
| *** cdearborn has quit IRC | 01:53 | |
| *** diurnalist has quit IRC | 02:24 | |
| *** rloo has quit IRC | 02:29 | |
| *** rcernin has quit IRC | 02:59 | |
| *** rcernin has joined #openstack-ironic | 03:03 | |
| *** SpamapS has quit IRC | 03:06 | |
| *** SpamapS has joined #openstack-ironic | 03:09 | |
| *** SpamapS has quit IRC | 03:15 | |
| *** uzumaki has quit IRC | 03:42 | |
| *** uzumaki has joined #openstack-ironic | 04:06 | |
| *** gyee has quit IRC | 04:10 | |
| *** diurnalist has joined #openstack-ironic | 04:31 | |
| *** diurnalist has quit IRC | 04:35 | |
| *** rh-jelabarre has quit IRC | 04:48 | |
| *** Lucas_Gray has quit IRC | 04:49 | |
| *** abdysn has joined #openstack-ironic | 05:01 | |
| *** ociuhandu has joined #openstack-ironic | 05:03 | |
| *** uzumaki has quit IRC | 05:03 | |
| *** SpamapS has joined #openstack-ironic | 05:06 | |
| *** ociuhandu has quit IRC | 05:07 | |
| *** waleedm has joined #openstack-ironic | 06:20 | |
| waleedm | Hi guys, I see now in master we couldn't get all data from BareMetal introspection data | 06:21 |
| waleedm | all info that was under “extra.” is not exist anymore and there was information about all the network devices and their capabilities which is also not exist. | 06:21 |
| waleedm | how to get them now ?! | 06:23 |
| *** uzumaki has joined #openstack-ironic | 06:28 | |
| *** mkrai has quit IRC | 06:43 | |
| *** diurnalist has joined #openstack-ironic | 06:43 | |
| *** diurnalist has quit IRC | 06:48 | |
| arne_wiebalck | Good morning, ironic! | 06:49 |
| janders | good morning arne_wiebalck | 06:51 |
| arne_wiebalck | Hey janders o/ | 06:51 |
| rpittau | good morning ironic! o/ | 06:51 |
| arne_wiebalck | Hey rpittau o/ | 06:51 |
| rpittau | hey arne_wiebalck :) | 06:51 |
| janders | good morning rpittau o/ | 06:52 |
| rpittau | hey janders :) | 06:52 |
| *** yolanda has quit IRC | 06:52 | |
| *** yolanda has joined #openstack-ironic | 06:52 | |
| arne_wiebalck | waleedm: My first guess would be that the extra-hardware collector is not passed to the ipa-inspection-collectors parameter on the command line of the agent RAM disk ... did you confirm it is there? | 06:53 |
| arne_wiebalck | waleedm: command line of the RAM disk kernel, that is | 06:54 |
| waleedm | kernel --timeout 60000 http://192.168.24.1:8088/agent.kernel ipa-inspection-callback-url=http://192.168.24.1:5050/v1/continue ipa-inspection-collectors=default,extra-hardware,numa-topology,logs systemd.journald.forward_to_console=yes BOOTIF=${mac} ipa-debug=1 ipa-inspection-dhcp-all-interfaces=1 ipa-collect-lldp=1 initrd=agent.ramdisk || goto retry_boot | 06:57 |
| waleedm | initrd --timeout 60000 http://192.168.24.1:8088/agent.ramdisk || goto retry_boot | 06:57 |
| rpittau | kind of urgent request, please someone approve https://review.opendev.org/743248 it fixes CI in ironic-python-agent ussuri branch and we have a release pending 2 patches blocked because of CI broken :) | 07:02 |
| patchbot | patch 743248 - ironic-python-agent (stable/ussuri) - Update jobs - 1 patch set | 07:02 |
| iurygregory | good morning arne_wiebalck janders rpittau and Ironic! | 07:08 |
| janders | good morning iurygregory o/ | 07:09 |
| waleedm | arne_wiebalck, so seems it's there, right? | 07:10 |
| rpittau | hey iurygregory :) | 07:10 |
| rpittau | waleedm: it looks like it, it would be interesting to see the ironic-python-agent logs | 07:14 |
| *** dougsz has joined #openstack-ironic | 07:21 | |
| rpittau | anyone with a minute to spare please review https://review.opendev.org/743043 the change on ipa side has merged | 07:23 |
| patchbot | patch 743043 - ironic-python-agent-builder - Update name of ipa job - 1 patch set | 07:23 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/ussuri: Prevent un-needed iscsi cleanup https://review.opendev.org/743032 | 07:23 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/ussuri: Ignore devices with size 0 when collecting inventory https://review.opendev.org/741171 | 07:24 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Fix attribute error in `set_boot_image()` https://review.opendev.org/708495 | 07:28 |
| *** rcernin has quit IRC | 07:32 | |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Fix attribute error in `set_boot_image()` https://review.opendev.org/708495 | 07:32 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Fix attribute error in `set_boot_image()` https://review.opendev.org/708495 | 07:34 |
| *** diurnalist has joined #openstack-ironic | 07:41 | |
| *** diurnalist has quit IRC | 07:46 | |
| *** ociuhandu has joined #openstack-ironic | 07:52 | |
| *** ociuhandu has quit IRC | 07:52 | |
| *** ociuhandu has joined #openstack-ironic | 07:53 | |
| arne_wiebalck | waleedm: the next thing I would check is whether python-hardware is installed in the agent RAM disk, since this is the tool used to collect all the extra data I think | 08:00 |
| arne_wiebalck | waleedm: and, as rpittau suggested, the IPA logs may help understand the issue | 08:00 |
| *** lucasagomes has joined #openstack-ironic | 08:06 | |
| waleedm | arne_wiebalck, https://filebin.net/aip0rx347gvg9bk6/journal__1_?t=zi1qttby | 08:08 |
| arne_wiebalck | waleedm: this looks like the IPA collects all the data, right? | 08:10 |
| waleedm | yes but this's the output | 08:11 |
| waleedm | https://filebin.net/qxrunxqutfa0mxc9/cloudx-node8-ilo.json?t=7yj95gpu | 08:11 |
| waleedm | no extra data | 08:12 |
| waleedm | no PCI address for the NICs | 08:12 |
| *** dougsz has quit IRC | 08:38 | |
| rpittau | waleedm: did you check the inspector logs for any errors/warnings ? | 08:40 |
| *** derekh has joined #openstack-ironic | 08:41 | |
| *** noama has joined #openstack-ironic | 08:44 | |
| rpittau | waleedm: also, the ipa logs you posted seem truncated, is that all ? | 08:48 |
| *** rcernin has joined #openstack-ironic | 08:48 | |
| *** priteau has joined #openstack-ironic | 08:49 | |
| noama | hi | 08:51 |
| noama | it from my setup, yes this is what collected in undercloud | 08:52 |
| *** dougsz has joined #openstack-ironic | 08:52 | |
| *** ociuhandu_ has joined #openstack-ironic | 08:53 | |
| *** rcernin has quit IRC | 08:54 | |
| waleedm | rpittau, Noam is the setup owner :) | 08:55 |
| *** diurnalist has joined #openstack-ironic | 08:56 | |
| noama | i got this is the inspector logs | 08:56 |
| noama | 2020-07-28 11:54:14.674 8 DEBUG ironic_inspector.process [-] [node: 755b3a2a-e3a4-464d-9ed8-f8decf0cf4bf state processing MAC 98:03:9b:9f:ce:86 BMC 10.209.225.77] Running post-processing hook extra_hardware _run_post_hooks /usr/lib/python3.6/site-packages/ironic_inspector/process.py:268 | 08:56 |
| *** ociuhandu has quit IRC | 08:56 | |
| noama | ironic_inspector.plugins.extra_hardware [-] [node: 755b3a2a-e3a4-464d-9ed8-f8decf0cf4bf state processing MAC 98:03:9b:9f:ce:86 BMC 10.209.225.77] Extra hardware data was not in a recognised format (eDeploy), and will not be forwarded to introspection rules | 08:56 |
| noama | 755b3a2a-e3a4-464d-9ed8-f8decf0cf4bf state processing MAC 98:03:9b:9f:ce:86 BMC 10.209.225.77] Deleting "data" key from introspection data as it is assumed unusable by introspection rules. Raw data is stored in swift before_update /usr/lib/python3.6/site-packages/ironic_inspector/plugins/extra_hardware.py:65 | 08:56 |
| noama | ironic_inspector.process [-] [node: 755b3a2a-e3a4-464d-9ed8-f8decf0cf4bf state processing MAC 98:03:9b:9f:ce:86 BMC 10.209.225.77] Running post-processing hook lldp_basic _run_post_hooks /usr/lib/python3.6/site-packages/ironic_inspector/process.py:268 | 08:56 |
| rpittau | noama: hi! Please don't post long multiple lines here, use an external tool, like http://paste.openstack.org/ | 08:57 |
| noama | ok, sorry http://paste.openstack.org/show/796361/ | 08:57 |
| *** diurnalist has quit IRC | 09:00 | |
| noama | it seems this is the commit https://github.com/openstack/ironic-inspector/commit/c044ebd19826e825132434ff4f1010c213fb136b | 09:03 |
| noama | https://review.opendev.org/#/c/728096/ | 09:04 |
| patchbot | patch 728096 - ironic-inspector - extra_hardware: stop storing raw data in swift (MERGED) - 1 patch set | 09:04 |
| *** ricolin has quit IRC | 09:07 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Remove locks before RPC bus is started https://review.opendev.org/740360 | 09:12 |
| *** dtantsur|afk is now known as dtantsur | 09:12 | |
| dtantsur | morning ironic | 09:12 |
| *** Lucas_Gray has joined #openstack-ironic | 09:14 | |
| rpittau | noama: thanks for the pointer | 09:14 |
| noama | morning | 09:14 |
| noama | dtantsur can you explain this change https://review.opendev.org/#/c/728096/ and how can i get the extra hardware info | 09:15 |
| patchbot | patch 728096 - ironic-inspector - extra_hardware: stop storing raw data in swift (MERGED) - 1 patch set | 09:15 |
| rpittau | dtantsur: good morning! Wdyt about the issue mentioned by noama ? It looks like the extra-hardware data is being ignored | 09:15 |
| dtantsur | noama: fetch introspection data with API or CLI. if you have the extra-hardware plugin enabled, the processed data will be in 'extra' | 09:16 |
| dtantsur | "Extra hardware data was not in a recognised format (eDeploy), and will not be forwarded to introspection rules" means that you're receiving something invalid though | 09:17 |
| noama | dtantsur: the extra info is not exist any more, i use "openstack baremetal introspection data save" | 09:18 |
| dtantsur | well, because you've received invalid data | 09:19 |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Enhance certificate verification for ilo harware type https://review.opendev.org/743490 | 09:19 |
| rpittau | I was looking at the ipa code but I don't see any change that could've influenced that | 09:19 |
| iurygregory | good morning dtantsur =) | 09:20 |
| noama | dtantsur: so something on the server did not run correctly? | 09:21 |
| dtantsur | noama: something on the ramdisk. you can see the received data in debug logs. | 09:22 |
| janders | good morning dtantsur o/ | 09:23 |
| *** ociuhandu_ has quit IRC | 09:25 | |
| janders | while working on https://review.opendev.org/#/c/442383/ I'm having some trouble with the MultiOpt format of the clean_step_add_params option | 09:28 |
| patchbot | patch 442383 - ironic - [WIP]Generic way to configure clean step priorites... (ABANDONED) - 1 patch set | 09:28 |
| janders | would you be able to point me to a code snippet that parses MultiOpt configs? | 09:28 |
| janders | it seems unusual (or I'm totally missing it in the ironic codebase) | 09:29 |
| noama | dtantsur: http://paste.openstack.org/show/796365/ it does not contain the "extra" tree | 09:29 |
| *** ociuhandu has joined #openstack-ironic | 09:31 | |
| *** ociuhandu has quit IRC | 09:31 | |
| *** ociuhandu has joined #openstack-ironic | 09:31 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 09:32 |
| rpittau | janders: I'm not sure we use MultiOpt in ironic at all, have you checked the oslo_config reference? https://docs.openstack.org/oslo.config/latest/reference/api/oslo_config.html#oslo_config.cfg.MultiOpt | 09:33 |
| dtantsur | noama: from what I can gather, you have [] somewhere in your data, which is not correct (although one could argue we should ignore it) | 09:35 |
| dtantsur | it's probably an issue in python-hardware | 09:36 |
| janders | rpittau for reference here's where I picked it up from https://review.opendev.org/#/c/362615/10/specs/approved/generic-clean-priority-configuration.rst@59 | 09:37 |
| patchbot | patch 362615 - ironic-specs - Generic way to configure clean step priorites in A... (ABANDONED) - 10 patch sets | 09:37 |
| iurygregory | janders, https://docs.openstack.org/oslo.config/latest/configuration/format.html may help | 09:38 |
| rpittau | dtantsur, noama, latest version of python-hardware is 0.25.0 and it was released on March 30 | 09:38 |
| * dtantsur is making inspector more robust | 09:38 | |
| janders | rpittau iurygregory thank you, will start with browsing these two docs | 09:39 |
| dtantsur | janders: ironic-inspector has a MultiStrOpt | 09:49 |
| dtantsur | in ironic_inspector.conf.pci_devices | 09:49 |
| janders | dtantsur thank you, looking now :) | 09:50 |
| janders | yeah inspector way of handling this makes perfect sense | 09:52 |
| janders | the abandoned code tried treating a list like a dict and that didn't work very well | 09:52 |
| janders | (or at least it seems that was the problem) | 09:53 |
| janders | this example helps a lot, thanks dtantsur | 09:54 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 09:55 |
| *** k_mouza has joined #openstack-ironic | 09:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Relax extra_hardware data validation by default https://review.opendev.org/743498 | 09:58 |
| dtantsur | nao<TAB> :( | 09:58 |
| dtantsur | anyway, this relaxes validation a bit ^^^ | 09:58 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 10:01 |
| noama | dtantsur: thanks man, ill patch it to my server and test it | 10:01 |
| dtantsur | ah, I used wrong tab completion :D | 10:02 |
| dtantsur | sorry, had an early start today | 10:02 |
| dtantsur | (and spent 2+ hours in a line) | 10:02 |
| noama | dtantsur: pretty good to early start | 10:02 |
| dtantsur | not when you need to face some bureaucracy :) | 10:03 |
| noama | dtantsur: nope, but thanks | 10:03 |
| noama | dtantsur: ill ping you later the results | 10:03 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Add support for retrieving unprocessed data https://review.opendev.org/743504 | 10:19 |
| dtantsur | not sure how we missed ^^^ but we did apparently | 10:19 |
| *** rcernin has joined #openstack-ironic | 10:19 | |
| noama | dtantsur: the cli will out json with the unprocessed data? | 10:27 |
| dtantsur | noama: ignore the unprocessed data for now, it just occurred to me that we don't expose it | 10:28 |
| dtantsur | I think the first patch should be enough for you | 10:28 |
| noama | ok | 10:28 |
| openstackgerrit | John Garbutt proposed openstack/networking-generic-switch master: WIP: Attempt to batch up cmds https://review.opendev.org/743283 | 10:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Remove built-in API documentation in favour of API-REF https://review.opendev.org/743506 | 10:40 |
| *** uzumaki has quit IRC | 10:40 | |
| *** rcernin has quit IRC | 10:43 | |
| openstackgerrit | John Garbutt proposed openstack/networking-generic-switch master: WIP: Attempt to batch up cmds https://review.opendev.org/743283 | 10:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: CI: run doc8 on api-ref https://review.opendev.org/743507 | 10:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Relax extra_hardware data validation by default https://review.opendev.org/743498 | 10:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add non-voting jobs with the openstack ansible collection from source https://review.opendev.org/743508 | 11:03 |
| *** uzumaki has joined #openstack-ironic | 11:09 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Enhance certificate verification for ilo harware type https://review.opendev.org/743490 | 11:11 |
| *** lucasagomes has quit IRC | 11:11 | |
| *** lucasagomes has joined #openstack-ironic | 11:12 | |
| *** rloo has joined #openstack-ironic | 11:13 | |
| *** k_mouza has quit IRC | 11:13 | |
| *** k_mouza has joined #openstack-ironic | 11:19 | |
| *** janders has quit IRC | 11:19 | |
| *** k_mouza has quit IRC | 11:19 | |
| *** k_mouza has joined #openstack-ironic | 11:20 | |
| *** janders has joined #openstack-ironic | 11:23 | |
| noama | dtantsur: extra data is now back again | 11:25 |
| dtantsur | nice | 11:25 |
| openstackgerrit | John Garbutt proposed openstack/networking-generic-switch master: WIP: Attempt to batch up cmds https://review.opendev.org/743283 | 11:29 |
| *** k_mouza has quit IRC | 11:34 | |
| *** diurnalist has joined #openstack-ironic | 11:38 | |
| *** diurnalist has quit IRC | 11:42 | |
| *** k_mouza has joined #openstack-ironic | 11:45 | |
| *** ricolin has joined #openstack-ironic | 11:47 | |
| *** k_mouza has quit IRC | 11:53 | |
| *** priteau has quit IRC | 11:55 | |
| *** priteau has joined #openstack-ironic | 11:56 | |
| *** tkajinam has quit IRC | 12:02 | |
| *** noama has quit IRC | 12:05 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:09 | |
| *** uzumaki has quit IRC | 12:12 | |
| rpittau | dtantsur: if you have a minute this fixes ipa CI in ussuri https://review.opendev.org/743248 | 12:13 |
| patchbot | patch 743248 - ironic-python-agent (stable/ussuri) - Update jobs - 1 patch set | 12:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client master: Support retrieving unprocessed data https://review.opendev.org/743518 | 12:13 |
| dtantsur | rpittau: I guess we're not worried too much about temporary losing the coverage on partition images? | 12:13 |
| rpittau | aren't we running wholedisk+partition in tempest now ? | 12:15 |
| dtantsur | rpittau: not until https://review.opendev.org/#/c/741445/ lands | 12:15 |
| patchbot | patch 741445 - ironic-tempest-plugin - Update basic_ops - 13 patch sets | 12:15 |
| dtantsur | I think we ended up running whole disk (?) tests twice instead | 12:15 |
| dtantsur | iurygregory can confirm or deny | 12:15 |
| rpittau | ah | 12:15 |
| rpittau | so we're not running partition in master also | 12:16 |
| dtantsur | yeah | 12:16 |
| * dtantsur -> food | 12:16 | |
| rpittau | ok, uhmmmm | 12:16 |
| rpittau | yeah me too I guess, the food I mean, not the partition | 12:16 |
| * iurygregory looks | 12:17 | |
| janders | wrapping up here, see you tomorrow o/ | 12:18 |
| iurygregory | until https://review.opendev.org/#/c/741445/ lands tests are a bit blur | 12:18 |
| patchbot | patch 741445 - ironic-tempest-plugin - Update basic_ops - 13 patch sets | 12:18 |
| iurygregory | ok tempest seems happy on the new change | 12:19 |
| *** k_mouza has joined #openstack-ironic | 12:19 | |
| *** diurnalist has joined #openstack-ironic | 12:19 | |
| iurygregory | and on https://review.opendev.org/#/c/741886/ the failures seems to be timeout on ssh =( | 12:19 |
| patchbot | patch 741886 - ironic - Add tempest default_boot_option and altflavor - 4 patch sets | 12:19 |
| rpittau | bye janders :) | 12:20 |
| *** k_mouza has quit IRC | 12:24 | |
| *** diurnalist has quit IRC | 12:24 | |
| * TheJulia yawns | 12:32 | |
| TheJulia | good morning everyone | 12:33 |
| iurygregory | good morning TheJulia =) | 12:34 |
| dtantsur | TheJulia: g'morning | 12:36 |
| *** k_mouza has joined #openstack-ironic | 12:37 | |
| *** noama has joined #openstack-ironic | 12:39 | |
| *** k_mouza has quit IRC | 12:41 | |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Allow erase_devices_metadata to be used as a deploy step https://review.opendev.org/743272 | 12:42 |
| TheJulia | I'm getting a bad feeling about CI right now | 12:43 |
| rpittau | isn't that the usual feeling about CI? :) | 12:43 |
| *** ociuhandu has quit IRC | 12:57 | |
| *** ociuhandu has joined #openstack-ironic | 12:57 | |
| *** k_mouza has joined #openstack-ironic | 12:58 | |
| *** rcernin has joined #openstack-ironic | 12:59 | |
| *** diurnalist has joined #openstack-ironic | 13:01 | |
| *** k_mouza has quit IRC | 13:02 | |
| TheJulia | eh, I'd prefer to see it consistently work | 13:02 |
| *** rcernin has quit IRC | 13:03 | |
| *** diurnalist has quit IRC | 13:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: DNM test keystoneauth patches https://review.opendev.org/743530 | 13:06 |
| TheJulia | so v6 job failure with no route to host :( | 13:09 |
| dtantsur | folks, a small but pretty important deploy steps fix: https://review.opendev.org/#/c/743266/\ | 13:10 |
| patchbot | patch 743266 - ironic - Make the final deploy step validation actually fai... - 1 patch set | 13:10 |
| *** k_mouza has joined #openstack-ironic | 13:11 | |
| *** Goneri has joined #openstack-ironic | 13:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: POC deploy-time RAID https://review.opendev.org/741227 | 13:15 |
| *** k_mouza has quit IRC | 13:16 | |
| *** noama has quit IRC | 13:17 | |
| *** Lucas_Gray has quit IRC | 13:22 | |
| *** k_mouza has joined #openstack-ironic | 13:23 | |
| *** k_mouza has quit IRC | 13:27 | |
| *** Lucas_Gray has joined #openstack-ironic | 13:29 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Log IPv6 configuration status for tinycore https://review.opendev.org/743536 | 13:30 |
| TheJulia | so... downside of trying to run tinycore on v6 on rackspace | 13:33 |
| TheJulia | it seems we don't see the RA's | 13:34 |
| TheJulia | hence ^^^ | 13:34 |
| openstackgerrit | John Garbutt proposed openstack/networking-generic-switch master: WIP: Attempt to batch up cmds https://review.opendev.org/743283 | 13:37 |
| dtantsur | TheJulia: should we switch v6 to non-voting while we're debugging it? | 13:39 |
| dtantsur | I have a feeling we made it voting a bit too early | 13:39 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Shift dhcp package for DHCPv6 https://review.opendev.org/743539 | 13:45 |
| TheJulia | dtantsur: we likely should make it non-voting for the moment until we get the fix in, but it might be a couple days as we're well aware before we have it sorted | 13:45 |
| *** dougsz has quit IRC | 13:47 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Mark IPv6 job as non-voting to unblock the gate https://review.opendev.org/743542 | 13:48 |
| *** tzumainn has joined #openstack-ironic | 13:48 | |
| dtantsur | TheJulia: one comment ^^ | 13:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Mark IPv6 job as non-voting to unblock the gate https://review.opendev.org/743542 | 13:53 |
| TheJulia | dtantsur: done | 13:53 |
| TheJulia | I'd just single core approve it at this point | 13:53 |
| TheJulia | tbh | 13:53 |
| dtantsur | TheJulia: done exactly that | 13:54 |
| iurygregory | I was about to give _2 | 13:55 |
| dtantsur | iurygregory: you still can :) | 13:55 |
| iurygregory | hehehe | 13:55 |
| iurygregory | first I need about 1L of water | 13:55 |
| iurygregory | or maybe I will die | 13:55 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Shift dhcp package for DHCPv6 https://review.opendev.org/743539 | 13:56 |
| TheJulia | whyyyyyy do we have two copies of the dhcp pattern | 13:56 |
| iurygregory | I hate the fact I had to go out to pick up a package in the post office and it was 34 C outside | 13:56 |
| arne_wiebalck | Redfish Interop Profile for Ironic in 2 mins: https://cern.zoom.us/j/97073886481 | 13:58 |
| dtantsur | iurygregory: ugh :( | 13:58 |
| rpioso | arne_wiebalck: I am getting an error when trying to connect to the Redfish Interop Profiles Zoom meeting -- "This meeting ID is not valid. Please check and try again." I tried the one linked yesterday's meeting -- https://cern.zoom.us/j/97073886481 | 13:58 |
| rpioso | arne_wiebalck: I'm in \o/ | 13:59 |
| *** k_mouza has joined #openstack-ironic | 13:59 | |
| arne_wiebalck | \o/ | 13:59 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: DNM test keystoneauth patches https://review.opendev.org/743530 | 14:00 |
| TheJulia | is it being recorded | 14:02 |
| *** dougsz has joined #openstack-ironic | 14:03 | |
| *** k_mouza has quit IRC | 14:03 | |
| *** k_mouza has joined #openstack-ironic | 14:04 | |
| iurygregory | netboot should work fine for wholedisk + uefi right? | 14:08 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/train: Prevent un-needed iscsi cleanup https://review.opendev.org/743550 | 14:17 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/stein: Prevent un-needed iscsi cleanup https://review.opendev.org/743552 | 14:17 |
| dtantsur | iurygregory: I think so | 14:18 |
| iurygregory | I'm trying to understand why it fails | 14:18 |
| dtantsur | hmm, wait | 14:18 |
| iurygregory | ironic-tempest-ipa-partition-uefi-pxe_ipmitool, is mad | 14:19 |
| TheJulia | theoretically should but not officially supported | 14:19 |
| dtantsur | I seem to recall something about ipxe and sanboot not working with efi | 14:19 |
| TheJulia | or should not be officially supported because tenant network separation | 14:19 |
| iurygregory | ok, so it would make sense that all wholedisk we test with local and partition with netboot? | 14:20 |
| iurygregory | =) | 14:20 |
| iurygregory | or maybe I'm overthinking... | 14:20 |
| TheJulia | iurygregory: uhh... can we avoid that? | 14:20 |
| iurygregory | we can | 14:20 |
| TheJulia | \o/ | 14:20 |
| iurygregory | and we can also cover this specific case | 14:20 |
| iurygregory | to be local for this job only | 14:20 |
| *** diurnalist has joined #openstack-ironic | 14:21 | |
| iurygregory | by default I made tempest use default_boot_option as boot_option for the tests | 14:22 |
| TheJulia | I think that would be best... if someone is trying it and it worked and then doesn't work, we can handle it as a bug... it is just we should generally discourage network booting everything by default wherever possible | 14:22 |
| iurygregory | and since ironic-tempest-ipa-partition-uefi-pxe_ipmitool uses netboot it will try netboot for wholedisk | 14:22 |
| iurygregory | partition tests is set for netboot anyways so it wouldn't be a problem if we change the config of the job for local =) | 14:23 |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Enhance certificate verification for ilo harware type https://review.opendev.org/743490 | 14:25 |
| *** diurnalist has quit IRC | 14:26 | |
| * iurygregory will give a try seeting the option of the job | 14:29 | |
| TheJulia | umm... did we revert the eventlet monkey patch in ipa?!? | 14:30 |
| TheJulia | or am I like smoking something amazing and put a patch against the wrong project? | 14:30 |
| *** abdysn has quit IRC | 14:31 | |
| TheJulia | oh no | 14:31 |
| TheJulia | it is eventlet loading in general | 14:31 |
| TheJulia | nevermind! | 14:31 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 14:33 |
| dtantsur | Fatal Python error: GC object already tracked | 14:35 |
| dtantsur | Monday vibes | 14:35 |
| iurygregory | wow I never saw this one | 14:35 |
| dtantsur | learning new things every day | 14:36 |
| *** waleedm has quit IRC | 14:39 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Deprecate http_basic_username and http_basic_password in [json_rpc] https://review.opendev.org/742474 | 14:43 |
| *** diurnalist has joined #openstack-ironic | 14:49 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: set EVENTLET_NO_GREENDNS to 'yes' https://review.opendev.org/742011 | 14:49 |
| TheJulia | dtantsur: Umm.. where did you see that?!? | 14:50 |
| dtantsur | TheJulia: bifrost crashed with that on one of my patches ^_^ | 14:50 |
| dtantsur | hopefully transient, other jobs were not affected | 14:51 |
| dtantsur | on the bright side, I think my keystoneauth patches will make HTTP basic auth actually work with ironic ansible modules | 14:51 |
| dtantsur | https://review.opendev.org/743282 and its parent if anybody is interested | 14:51 |
| patchbot | patch 743282 - keystoneauth - Correct endpoint and major version discovery for n... - 3 patch sets | 14:51 |
| TheJulia | dtantsur: I was thinking they might when I looked at them but I wasn't 100% sure | 14:51 |
| TheJulia | its been a while since I've looked at the ansible modules themselves | 14:52 |
| dtantsur | some clean up is overdue nowadays. I hope sshnaidm will do it sooner than I get to it :-P | 14:52 |
| * sshnaidm will do sooner :) | 14:53 | |
| dtantsur | \o/ | 14:56 |
| dtantsur | sshnaidm: we need to find a way to kill ironic_url with fire. That would be a great start. | 14:56 |
| sshnaidm | dtantsur, what's the point of ironic url? To run w/o keystone? | 14:58 |
| dtantsur | sshnaidm: yes, but we should have used standard stuff | 14:58 |
| dtantsur | i.e. with {auth: {endpoint: ...}} or baremetal_endpoint_override | 14:58 |
| dtantsur | the problem is, currently ironic_url is even required | 14:59 |
| dtantsur | making me realize, I need to update https://review.opendev.org/#/c/742932/ with more logic | 15:00 |
| patchbot | patch 742932 - ansible-collections-openstack - Do not require ironic_url if cloud is provided - 2 patch sets | 15:00 |
| sshnaidm | dtantsur, do you have some links to read about it? I'd like to have some context.. | 15:02 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove locks before RPC bus is started https://review.opendev.org/740360 | 15:03 |
| TheJulia | rpittau: ^^^ addresses your nits. ^^ dtantsur we likely need to wait to +W until after the v6 job is non-voting | 15:03 |
| dtantsur | sshnaidm: honestly, not sure. it may be tribal knowledge :( | 15:07 |
| TheJulia | dtantsur: well, auth/endpoint dictionary structure was not solidified when we started down the path of writing ironic modules. I think the back-end code changed a couple times over the past five years to keep it working | 15:10 |
| dtantsur | likely so | 15:10 |
| TheJulia | it was all "keystone will magically solve it all | 15:10 |
| TheJulia | " | 15:10 |
| TheJulia | then os_cloud_config appeared | 15:11 |
| dtantsur | and then keystoneauth, and then openstacksdk :) | 15:11 |
| dtantsur | and here we are now, trying to understand how all this legacy works :D | 15:11 |
| TheJulia | whiskey? | 15:12 |
| dtantsur | I got a lot of fresh mint, and the weather is pretty warm. You know where it's heading, don't you? ;) | 15:12 |
| TheJulia | mojito | 15:12 |
| dtantsur | exactly | 15:13 |
| TheJulia | +2 | 15:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: POC deploy-time RAID https://review.opendev.org/741227 | 15:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Validate that the services are running after installation https://review.opendev.org/743569 | 15:54 |
| openstackgerrit | Merged openstack/ironic master: Make the final deploy step validation actually fail deploy https://review.opendev.org/743266 | 15:56 |
| openstackgerrit | Merged openstack/ironic-inspector stable/ussuri: Make inspector base CI job use 2 CPUs https://review.opendev.org/740747 | 15:56 |
| *** waleedm has joined #openstack-ironic | 15:56 | |
| *** rloo has quit IRC | 16:02 | |
| *** k_mouza has quit IRC | 16:02 | |
| *** rloo has joined #openstack-ironic | 16:03 | |
| *** rloo has quit IRC | 16:04 | |
| *** rloo has joined #openstack-ironic | 16:06 | |
| *** priteau has quit IRC | 16:08 | |
| *** lucasagomes has quit IRC | 16:18 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Make the final deploy step validation actually fail deploy https://review.opendev.org/743587 | 16:20 |
| *** k_mouza has joined #openstack-ironic | 16:23 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Validate that the services are running after installation https://review.opendev.org/743569 | 16:23 |
| *** Lucas_Gray has quit IRC | 16:26 | |
| *** k_mouza has quit IRC | 16:28 | |
| *** dougsz has quit IRC | 16:29 | |
| *** k_mouza has joined #openstack-ironic | 16:31 | |
| *** dtantsur is now known as dtantsur|afk | 16:32 | |
| dtantsur|afk | see you tomorrow! | 16:32 |
| TheJulia | goodnight | 16:32 |
| arne_wiebalck | bye everyone o/ | 16:33 |
| rpittau | bye! o/ | 16:34 |
| *** k_mouza has quit IRC | 16:36 | |
| *** k_mouza has joined #openstack-ironic | 16:37 | |
| *** rloo has quit IRC | 16:39 | |
| *** k_mouza has quit IRC | 16:42 | |
| *** k_mouza has joined #openstack-ironic | 16:42 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Validate that the services are running after installation https://review.opendev.org/743569 | 16:42 |
| *** ociuhandu has quit IRC | 16:45 | |
| *** k_mouza has quit IRC | 16:47 | |
| *** waleedm has quit IRC | 16:51 | |
| *** k_mouza has joined #openstack-ironic | 16:52 | |
| *** rloo has joined #openstack-ironic | 16:52 | |
| *** k_mouza has quit IRC | 16:57 | |
| iurygregory | yay good news https://review.opendev.org/741445 https://review.opendev.org/#/c/741886/ | 17:00 |
| patchbot | patch 741445 - ironic-tempest-plugin - Update basic_ops - 13 patch sets | 17:00 |
| patchbot | patch 741886 - ironic - Add tempest default_boot_option and altflavor - 5 patch sets | 17:00 |
| *** derekh has quit IRC | 17:00 | |
| iurygregory | dtantsur|afk, TheJulia rpittau this should cover partition/wholedisk testing, but let's review carefully to see if my changes makes sense | 17:00 |
| iurygregory | we will probably need to backport 741886 | 17:01 |
| *** k_mouza has joined #openstack-ironic | 17:05 | |
| openstackgerrit | Merged openstack/bifrost master: Use random passwords by default https://review.opendev.org/742203 | 17:05 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Allow erase_devices_metadata to be used as a deploy step https://review.opendev.org/743272 | 17:07 |
| *** uzumaki has joined #openstack-ironic | 17:08 | |
| *** k_mouza has quit IRC | 17:09 | |
| *** k_mouza has joined #openstack-ironic | 17:11 | |
| *** k_mouza has quit IRC | 17:16 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/bifrost master: DNM use baremetal_node_info from collections https://review.opendev.org/743599 | 17:22 |
| sshnaidm | dtantsur|afk, will test here ^ | 17:22 |
| *** k_mouza has joined #openstack-ironic | 17:28 | |
| *** k_mouza has quit IRC | 17:32 | |
| *** k_mouza has joined #openstack-ironic | 17:36 | |
| *** k_mouza has quit IRC | 17:41 | |
| *** k_mouza has joined #openstack-ironic | 17:45 | |
| *** k_mouza has quit IRC | 17:49 | |
| *** ociuhandu has joined #openstack-ironic | 17:54 | |
| *** k_mouza has joined #openstack-ironic | 17:54 | |
| *** ociuhandu has quit IRC | 17:58 | |
| *** k_mouza has quit IRC | 17:59 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Capture TypeError when the logger fails https://review.opendev.org/743606 | 18:02 |
| TheJulia | bfournie: ^^^ I mentioned this downstream. I'm not really sure where the actual exception is coming from but I'm thinking the least we could do is likely try and capture it | 18:03 |
| *** k_mouza has joined #openstack-ironic | 18:12 | |
| *** k_mouza has quit IRC | 18:16 | |
| *** k_mouza has joined #openstack-ironic | 18:20 | |
| *** k_mouza has quit IRC | 18:24 | |
| *** jamesden_ has joined #openstack-ironic | 18:27 | |
| *** k_mouza has joined #openstack-ironic | 18:30 | |
| *** jamesdenton has quit IRC | 18:30 | |
| *** k_mouza has quit IRC | 18:39 | |
| openstackgerrit | Merged openstack/ironic master: Deprecate http_basic_username and http_basic_password in [json_rpc] https://review.opendev.org/742474 | 19:14 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Capture TypeError when the logger fails https://review.opendev.org/743606 | 19:49 |
| openstackgerrit | Merged openstack/ironic master: Mark IPv6 job as non-voting to unblock the gate https://review.opendev.org/743542 | 19:54 |
| janders | good morning / good evening ironic o/ | 20:34 |
| *** Lucas_Gray has joined #openstack-ironic | 20:34 | |
| stevebaker | morning | 20:52 |
| *** gyee has joined #openstack-ironic | 21:02 | |
| *** uzumaki has quit IRC | 21:02 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/bifrost master: DNM use baremetal_node_info from collections https://review.opendev.org/743599 | 21:06 |
| zaneb | stevebaker, dtantsur: I posted a question on https://review.opendev.org/#/c/742474/3 about docs for that change | 21:20 |
| patchbot | patch 742474 - ironic - Deprecate http_basic_username and http_basic_passw... (MERGED) - 3 patch sets | 21:20 |
| zaneb | huge +1 for that change btw :) | 21:21 |
| stevebaker | zaneb: it look like dtantsur|afk's change sets auth_type based on auth_strategy https://review.opendev.org/#/c/742474/3/ironic/common/json_rpc/client.py | 21:26 |
| patchbot | patch 742474 - ironic - Deprecate http_basic_username and http_basic_passw... (MERGED) - 3 patch sets | 21:26 |
| zaneb | stevebaker: it defaults it to that | 21:26 |
| zaneb | stevebaker: which means you can just set auth_type directly in the config file | 21:26 |
| zaneb | and if you don't then it defaults to the auth strategy | 21:27 |
| zaneb | what's documented will work, but it's inconsistent with other sections of the config (i.e. [inspector]) | 21:27 |
| stevebaker | zaneb: yep agreed, client options are applied regardless of server options, lets document that | 21:30 |
| zaneb | ++ | 21:36 |
| openstackgerrit | Sagi Shnaidman proposed openstack/bifrost master: DNM use baremetal_node_info from collections https://review.opendev.org/743599 | 21:59 |
| *** rcernin has joined #openstack-ironic | 22:09 | |
| *** rcernin has quit IRC | 22:22 | |
| *** rh-jelabarre has quit IRC | 22:23 | |
| *** rh-jelabarre has joined #openstack-ironic | 22:24 | |
| *** rcernin has joined #openstack-ironic | 22:37 | |
| *** rcernin has quit IRC | 22:50 | |
| *** rcernin has joined #openstack-ironic | 22:50 | |
| *** tkajinam has joined #openstack-ironic | 22:52 | |
| *** zer0c00l has joined #openstack-ironic | 23:10 | |
| zer0c00l | TheJulia: Do we have any OS images (whole disk/partition) that can be used for testing with IPA? | 23:17 |
| TheJulia | zer0c00l: not stock ones we publish due to republished works restrictions. Best to grab at least the whole disk qcows from the distros themselves | 23:18 |
| TheJulia | Partition is fairly easy to extract or create from those.... Loopback devices and all | 23:19 |
| *** diurnalist has quit IRC | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!