| *** gyee has quit IRC | 00:33 | |
| *** rcernin has quit IRC | 00:34 | |
| *** rcernin has joined #openstack-ironic | 00:39 | |
| *** Lucas_Gray has quit IRC | 01:04 | |
| *** yaawang has quit IRC | 01:33 | |
| *** yaawang has joined #openstack-ironic | 01:33 | |
| *** yaawang has quit IRC | 01:59 | |
| *** yaawang has joined #openstack-ironic | 02:02 | |
| *** rh-jelabarre has quit IRC | 02:20 | |
| *** Goneri has quit IRC | 02:41 | |
| *** mkrai has joined #openstack-ironic | 02:54 | |
| *** uzumaki has joined #openstack-ironic | 03:50 | |
| *** ricolin has joined #openstack-ironic | 04:00 | |
| TheJulia | JayF: I _think_ ? | 04:20 |
|---|---|---|
| TheJulia | JayF: One thing worth considering is about the security around that, but that seems entirely navigable | 04:21 |
| TheJulia | Hey, I think dmitry would appreciate one more review on https://review.opendev.org/#/c/740304/ | 04:23 |
| patchbot | patch 740304 - ironic-inspector - Do not try to set local_gb to -1 when the matched ... - 2 patch sets | 04:23 |
| *** diurnalist has quit IRC | 04:23 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Change non-tinyipa jobs to use multiple cores https://review.opendev.org/741577 | 04:25 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Enforce autospec in test_deployments https://review.opendev.org/741408 | 04:36 |
| openstackgerrit | Merged openstack/bifrost master: selinux: allow nginx to read symbolic links https://review.opendev.org/741491 | 04:44 |
| *** rcernin has quit IRC | 04:47 | |
| *** rcernin has joined #openstack-ironic | 04:57 | |
| *** yaawang has quit IRC | 05:04 | |
| *** yaawang has joined #openstack-ironic | 05:05 | |
| *** rcernin has quit IRC | 05:12 | |
| *** rcernin has joined #openstack-ironic | 05:20 | |
| *** yaawang has quit IRC | 05:45 | |
| *** yaawang has joined #openstack-ironic | 05:46 | |
| rpittau | good morning ironic! o/ | 06:17 |
| rpittau | Happy Friday :) | 06:17 |
| *** rcernin has quit IRC | 06:19 | |
| janders | good morning rpittau o/ | 06:21 |
| rpittau | hey janders :) | 06:25 |
| rpittau | is zuul drunk? https://review.opendev.org/741408 | 06:26 |
| patchbot | patch 741408 - ironic - Enforce autospec in test_deployments - 2 patch sets | 06:26 |
| janders | it's Friday arvo here... | 06:28 |
| janders | I'm sure there's a lot of drunk people around town | 06:29 |
| janders | maybe Zuul teamed up with some? | 06:29 |
| rpittau | mmm interesting, I think a patch merged without autospec and made that test fail :) | 06:31 |
| rpittau | interesting for the timing | 06:31 |
| *** rcernin has joined #openstack-ironic | 06:32 | |
| *** yaawang has quit IRC | 06:32 | |
| *** yaawang has joined #openstack-ironic | 06:33 | |
| rpittau | and I approved that change, so I'm to blame :/ | 06:35 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Add missing agent RAID compatibility for ilo5 and idrac https://review.opendev.org/740975 | 06:38 |
| *** rcernin has quit IRC | 06:46 | |
| *** diurnalist has joined #openstack-ironic | 06:47 | |
| *** diurnalist has quit IRC | 06:51 | |
| *** rcernin has joined #openstack-ironic | 06:53 | |
| *** janders has quit IRC | 06:57 | |
| *** yaawang has quit IRC | 07:04 | |
| *** yaawang has joined #openstack-ironic | 07:05 | |
| *** rcernin has quit IRC | 07:06 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_deployments https://review.opendev.org/741408 | 07:07 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_notification_utils https://review.opendev.org/741419 | 07:08 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in some api tests https://review.opendev.org/741468 | 07:08 |
| iurygregory | good morning rpittau and Ironic | 07:08 |
| rpittau | hey iurygregory :) | 07:08 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 07:10 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic-python-agent-builder master: Fixes DIB_IPA_CERT certificate copy issue https://review.opendev.org/741585 | 07:13 |
| *** bdodd_ has joined #openstack-ironic | 07:14 | |
| *** bdodd has quit IRC | 07:15 | |
| *** mkrai has quit IRC | 07:21 | |
| *** uzumaki has quit IRC | 07:30 | |
| *** rcernin has joined #openstack-ironic | 07:34 | |
| *** rcernin has quit IRC | 07:42 | |
| *** rcernin has joined #openstack-ironic | 07:46 | |
| *** rcernin has quit IRC | 07:50 | |
| *** mkrai has joined #openstack-ironic | 07:52 | |
| *** diurnalist has joined #openstack-ironic | 07:58 | |
| *** Lucas_Gray has joined #openstack-ironic | 08:03 | |
| *** diurnalist has quit IRC | 08:03 | |
| *** tkaprol has joined #openstack-ironic | 08:05 | |
| *** lucasagomes has joined #openstack-ironic | 08:07 | |
| *** ociuhandu has joined #openstack-ironic | 08:08 | |
| *** janders has joined #openstack-ironic | 08:08 | |
| *** k_mouza has joined #openstack-ironic | 08:09 | |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Prometheus-exporter to use DevStack's neutron"-legacy" module https://review.opendev.org/741477 | 08:10 |
| *** janders has quit IRC | 08:10 | |
| iurygregory | lucasagomes, we should be fine with the stable branche (I assume) or we should backport to stable branches? =) | 08:12 |
| iurygregory | rpittau, I'm wondering what is EAFP =) | 08:29 |
| *** derekh has joined #openstack-ironic | 08:30 | |
| rpittau | easier to ask for forgiveness than permission, it's one of the Python principles | 08:30 |
| rpittau | https://docs.python.org/3/glossary.html#term-eafp | 08:30 |
| *** janders has joined #openstack-ironic | 08:32 | |
| iurygregory | wow I never heard this one =) | 08:32 |
| iurygregory | thanks! | 08:32 |
| lucasagomes | iurygregory, I think it should be fine because I don't think the neutron team will delete the "new" module from the stable branches in devstack | 08:33 |
| iurygregory | lucasagomes, if they do we will go after you \o/ | 08:33 |
| iurygregory | XD | 08:33 |
| iurygregory | because you have good beers in your country | 08:34 |
| lucasagomes | iurygregory, loool | 08:34 |
| lucasagomes | fair :D | 08:34 |
| iurygregory | yeah | 08:35 |
| iurygregory | dtantsur|afk, rpittau https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_635/741105/3/check/ironic-lib-bios-ipmi-iscsi-src/6351eae/controller/logs/tempest_log.txt it works \o/ | 08:39 |
| iurygregory | partition_image_ref = 7635e722-8aaa-4edf-8233-b613e4024c6f | 08:39 |
| iurygregory | whole_disk_image_ref = 8d7972d5-7824-4bc1-a4d5-5e07a07a76af | 08:39 |
| rpittau | great :) | 08:39 |
| iurygregory | both id's are showing in the requests \o/ | 08:39 |
| *** aedc has joined #openstack-ironic | 08:53 | |
| *** tkaprol has quit IRC | 08:54 | |
| *** uzumaki has joined #openstack-ironic | 09:01 | |
| *** diurnalist has joined #openstack-ironic | 09:10 | |
| *** diurnalist has quit IRC | 09:14 | |
| *** dtantsur|afk is now known as dtantsur | 09:19 | |
| dtantsur | morning ironic | 09:19 |
| dtantsur | iurygregory: w00t, great news! | 09:19 |
| iurygregory | dtantsur, yup! \o/ | 09:20 |
| rpittau | hey dtantsur :) | 09:20 |
| uzumaki | morning dtantsur! \o/ how's it going? | 09:20 |
| iurygregory | sorry for the trouble =) | 09:20 |
| dtantsur | iurygregory: no worries at all, happens with all of us. left a couple of nits, will vote when I see all results. | 09:21 |
| dtantsur | uzumaki: hey-hey, not so bad. Friday after all! you? | 09:21 |
| iurygregory | you can check the results on the ironic-lib patch | 09:21 |
| iurygregory | =) | 09:21 |
| iurygregory | https://review.opendev.org/#/c/741105/ | 09:21 |
| patchbot | patch 741105 - ironic-lib - Update ironic-lib jobs - 3 patch sets | 09:21 |
| uzumaki | dtantsur, noiceeee.. It's been hot as hell here.. now it's raining so nicely.. it's a bit chilly and the gusts of wind blowing.. so I'm really glad \o/ | 09:21 |
| janders | good morning dtantsur o/ | 09:25 |
| *** rcernin has joined #openstack-ironic | 09:29 | |
| dtantsur | janders: thank you for your comments on the install doc! I agree that the advanced part needs more love, we'll iterate on it eventually. Feed free to submit any fixes | 09:32 |
| janders | no worries, happy to help | 09:32 |
| janders | would you like me to submit as a new patch, or a comment to yours? | 09:33 |
| janders | I suppose the former, given it's already merged? | 09:33 |
| dtantsur | janders: correct | 09:35 |
| janders | I might do this while it's still fresh in my mind.. an easy fix for a Friday evening | 09:36 |
| *** vmud213 has joined #openstack-ironic | 09:42 | |
| ajya | Hi Ironic, happy Friday! Are there any known issues with IPA and multiple NICs? I'm observing intermittent connectivity issues during IPA running when network does not start, after PXE boot has succeeded. | 09:42 |
| iurygregory | dtantsur, passing directly the dict didn't work on my tests (since it should turn into the kwargs for the functions) | 09:44 |
| iurygregory | the assert self.image_ref is not None worked fine =) | 09:45 |
| dtantsur | iurygregory: image_id=self.image_ref is strictly identical to pass it as a dict.. | 09:46 |
| dtantsur | ajya: seen that, never got to the bottom | 09:46 |
| dtantsur | iurygregory: also some bad news from the run: we now need to assert boot_option on the test level | 09:47 |
| dtantsur | which.. I'm not sure how to do | 09:47 |
| dtantsur | but we don't support boot_option:local (the default) for cirros partition image | 09:48 |
| iurygregory | well | 09:48 |
| *** rcernin has quit IRC | 09:48 | |
| iurygregory | we can set the boot_option (local/netboot) | 09:48 |
| iurygregory | on tempest | 09:48 |
| dtantsur | you may need to update the flavor though.. (not sure) | 09:50 |
| janders | ajya: any chance portfast may play a role in this? I've seen issues like this on nodes connected to switches without portfast. | 09:50 |
| iurygregory | oh, we would need to add an option | 09:50 |
| dtantsur | could anyone please review https://review.opendev.org/#/c/740217/ and https://review.opendev.org/#/c/740224/ ? | 09:50 |
| patchbot | patch 740217 - bifrost - Get rid of ci_testing variable in roles - 1 patch set | 09:50 |
| patchbot | patch 740224 - bifrost - Add git_url_root to override root for all reposito... - 1 patch set | 09:50 |
| dtantsur | iurygregory: *ideally* we need to keep the default boot option for whole disk images.. or have 3 tests?.. | 09:51 |
| ajya | dtantsur: is this smth that started to appear recently or that has been there for a while? | 09:51 |
| dtantsur | ajya: for a good while | 09:52 |
| iurygregory | dtantsur, well for wholedisk we can use the default from the job and on the partition we need to just make sure we have netboot | 09:52 |
| ajya | janders: I'll take a look | 09:52 |
| dtantsur | yup | 09:52 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: selinux: allow nginx to read symbolic links https://review.opendev.org/741608 | 09:52 |
| janders | ajya: what happens then is on poweron the ports go up and stay up for long enough for the switch to finish its STP business, so ports are up for PXE. Then, after the server boots the ports go down and when the interfaces are being brought up, the switch doesn't finish STP scanning in time to make it before DHCP timeout. | 09:52 |
| *** tkajinam has quit IRC | 09:52 | |
| ajya | dtantsur: ok, I started to see this recently with new servers that have different setup, then it must be the setup | 09:52 |
| iurygregory | makes sense to me I will update the patch adding the option, let me just do some tests | 09:52 |
| dtantsur | janders: we even have a FAQ item on it :) https://docs.openstack.org/ironic/latest/admin/troubleshooting.html#dhcp-during-pxe-or-ipxe-is-inconsistent-or-unreliable | 09:53 |
| *** aedc has quit IRC | 09:54 | |
| janders | dtantsur yeah that's the one. I once spent a week with the network admin figuring out what da... we had port-channels in the mix as well which was obscuring the picture even more | 09:54 |
| *** aedc has joined #openstack-ironic | 09:54 | |
| openstackgerrit | Jakub Libosvar proposed openstack/ironic master: Explicitly set jobs to ML2/OVS https://review.opendev.org/739945 | 09:58 |
| *** yaawang has quit IRC | 10:02 | |
| *** yaawang has joined #openstack-ironic | 10:04 | |
| *** vmud213 has quit IRC | 10:06 | |
| *** dtantsur is now known as dtantsur|brb | 10:07 | |
| *** uzumaki has quit IRC | 10:10 | |
| *** ricolin has quit IRC | 10:11 | |
| iurygregory | dtantsur|brb, I think we only need to update the boot_option on the node.. | 10:13 |
| iurygregory | rpittau, ^ | 10:13 |
| iurygregory | nothing to do with the flavor *I think* | 10:13 |
| *** yaawang has quit IRC | 10:14 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 10:16 | |
| *** yaawang has joined #openstack-ironic | 10:21 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 10:23 |
| *** uzumaki has joined #openstack-ironic | 10:24 | |
| janders | dtantsur|brb I've expanded the advanced/playbook section with the info we discussed. | 10:26 |
| janders | I welcome your feedback. I may likely missed some doco etiquette - let me know what needs fixing up and I will submit a revised version on Monday | 10:27 |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 10:29 |
| janders | fixed up a couple of obvious issues | 10:31 |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 10:34 |
| openstackgerrit | Merged openstack/ironic stable/ussuri: Allow deleting nodes with a broken driver https://review.opendev.org/741460 | 10:38 |
| openstackgerrit | Merged openstack/ironic stable/train: Allow deleting nodes with a broken driver https://review.opendev.org/741461 | 10:38 |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Expand advanced install instructions (path info) https://review.opendev.org/741611 | 10:38 |
| janders | ..and a couple more. Sorry about the noise, should have run tox earlier :) | 10:38 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Make inspector base CI job use 2 CPUs https://review.opendev.org/740746 | 10:43 |
| *** mkrai has quit IRC | 10:53 | |
| openstackgerrit | Merged openstack/metalsmith master: Metalsmith to use DevStack's neutron"-legacy" module https://review.opendev.org/741476 | 10:54 |
| openstackgerrit | Merged openstack/metalsmith master: Set min version of tox to 3.2.1 https://review.opendev.org/741203 | 10:54 |
| *** uzumaki has quit IRC | 10:54 | |
| janders | wrapping up for the day - have a great weekend all! | 10:55 |
| janders | o/ | 10:55 |
| janders | see you next week | 10:55 |
| iurygregory | bye janders have a great weekend | 10:59 |
| *** lucasagomes is now known as lucas-hungry | 11:00 | |
| iurygregory | I think this would be enough to handle the boot_option http://paste.openstack.org/show/796041/ , thoughts? | 11:00 |
| openstackgerrit | Merged openstack/bifrost stable/ussuri: selinux: allow nginx to read symbolic links https://review.opendev.org/741608 | 11:09 |
| *** vmud213 has joined #openstack-ironic | 11:33 | |
| *** diurnalist has joined #openstack-ironic | 11:45 | |
| *** diurnalist has quit IRC | 11:50 | |
| *** lucas-hungry is now known as lucasagomes | 12:00 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:07 | |
| *** dtantsur|brb is now known as dtantsur | 12:10 | |
| dtantsur | iurygregory: let's give it a try | 12:11 |
| rpittau | iurygregory: it should work | 12:14 |
| dtantsur | assuming nova does not wipe capabilities - yes | 12:15 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Make skip_install a normal boolean variable https://review.opendev.org/741627 | 12:27 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 12:35 |
| iurygregory | let's see how it goes | 12:35 |
| *** vmud213 has quit IRC | 12:38 | |
| * iurygregory fingers crossed and back to read docs ... | 12:40 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Ironic to use DevStack's neutron"-legacy" module https://review.opendev.org/741256 | 12:46 |
| *** diurnalist has joined #openstack-ironic | 12:57 | |
| openstackgerrit | Jakub Libosvar proposed openstack/ironic master: Explicitly set jobs to ML2/OVS https://review.opendev.org/739945 | 12:58 |
| *** diurnalist has quit IRC | 13:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow disabling retries in AgentClient.get_command_statuses https://review.opendev.org/741632 | 13:03 |
| TheJulia | good morning | 13:04 |
| iurygregory | good morning TheJulia =) | 13:05 |
| dtantsur | morning TheJulia | 13:06 |
| rpittau | good morning TheJulia :) | 13:06 |
| dtantsur | I was about to say "you're early", but no, you're not. it's time flying too fast :( | 13:07 |
| iurygregory | yeah | 13:07 |
| iurygregory | because it's friday | 13:07 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Agent power interface https://review.opendev.org/735591 | 13:07 |
| TheJulia | heh | 13:12 |
| TheJulia | I can go back to bed if you guys would like | 13:12 |
| TheJulia | I wouldn't mind just sleeping today away | 13:12 |
| iurygregory | you can XD | 13:12 |
| rpittau | mmm sleep.... | 13:12 |
| iurygregory | I will save my question for monday | 13:12 |
| iurygregory | =D | 13:12 |
| TheJulia | spill it :) | 13:13 |
| iurygregory | did you have coffee at least? (before asking the real one on pm) | 13:13 |
| TheJulia | I do have coffee | 13:14 |
| TheJulia | heh | 13:14 |
| TheJulia | and I'm about half way through the first glass of it | 13:14 |
| *** k_mouza has quit IRC | 13:17 | |
| *** rloo has joined #openstack-ironic | 13:23 | |
| *** Lucas_Gray has quit IRC | 13:25 | |
| *** Lucas_Gray has joined #openstack-ironic | 13:29 | |
| dtantsur | TheJulia: just do it! :) | 13:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Enable metadata cleaning by default https://review.opendev.org/741638 | 13:44 |
| dtantsur | finally ^^^ | 13:44 |
| iurygregory | dtantsur, seems like the job didn't like the idea to update the boot_option | 13:47 |
| dtantsur | :( | 13:47 |
| iurygregory | https://ca768874132bdf15c4e3-b818c7a6c3b5b0701ea16d988cbfb8d4.ssl.cf1.rackcdn.com/741105/3/check/ironic-lib-bios-ipmi-direct-src/6356889/testr_results.html | 13:47 |
| iurygregory | or maybe I did something wrong on the change | 13:47 |
| dtantsur | I think you did, lemme see | 13:48 |
| dtantsur | iurygregory: this is a price you pay for excessive usage of **kwargs :-P | 13:48 |
| iurygregory | yeah | 13:48 |
| iurygregory | =X | 13:49 |
| dtantsur | left a comment | 13:49 |
| iurygregory | or I can go and add a paramter for it XD | 13:49 |
| dtantsur | ++ | 13:49 |
| iurygregory | I was thinking about the same thing from your comment lol | 13:49 |
| iurygregory | =O | 13:49 |
| dtantsur | great minds think alike :) | 13:51 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 13:54 |
| * iurygregory fingers crossed | 13:54 | |
| *** Goneri has joined #openstack-ironic | 13:57 | |
| *** diurnalist has joined #openstack-ironic | 14:08 | |
| *** k_mouza has joined #openstack-ironic | 14:12 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Account for power interfaces that cannot power on https://review.opendev.org/741650 | 14:13 |
| *** diurnalist has quit IRC | 14:13 | |
| *** aedc_ has joined #openstack-ironic | 14:28 | |
| *** aedc has quit IRC | 14:31 | |
| *** k_mouza has quit IRC | 14:31 | |
| *** diurnalist has joined #openstack-ironic | 14:37 | |
| *** k_mouza has joined #openstack-ironic | 14:42 | |
| openstackgerrit | Merged openstack/ironic master: Decompose the core deploy step on iscsi and ansible deploy https://review.opendev.org/731180 | 14:46 |
| openstackgerrit | Merged openstack/ironic stable/ussuri: Software RAID: don't try to set local_gb to MAX https://review.opendev.org/739488 | 14:47 |
| openstackgerrit | Merged openstack/ironic master: Add json and param parsing to args https://review.opendev.org/704489 | 14:47 |
| *** aedc_ has quit IRC | 14:49 | |
| dtantsur | mmm, bifrost CI does not account for cleaning, does it? | 14:50 |
| dtantsur | iurygregory: btw, do you plan on finishing switching bifrost to the new collection? | 14:51 |
| iurygregory | dtantsur, I do | 14:52 |
| iurygregory | it's on my list but is a bit low priority atm | 14:52 |
| dtantsur | k | 14:53 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Enable metadata cleaning by default https://review.opendev.org/741638 | 14:53 |
| TheJulia | dtantsur: I don't think bifrost defaults to cleaning when running testing | 14:53 |
| dtantsur | TheJulia: well, it does, with the patch above :) | 14:53 |
| TheJulia | I think in part because I also bricked a few hundred dollars worth of sata drives | 14:53 |
| dtantsur | the problem is, test-bifrost.yaml does not wait for cleaning to finish, so deleting nodes breaks | 14:55 |
| *** Lucas_Gray has quit IRC | 15:03 | |
| *** bnemec is now known as beekneemech | 15:10 | |
| dking | During inspection, will local_link_connection attempt LLDP if it cannot get a DCHP? | 15:11 |
| dking | I've been trying to figure out why it's not recording any LLDP information for my second interface. If I boot into a server successfully, I can get LLDP packets, but there's currently an issue with DHCP on that interface, and I was wondering whether that might be the cause. | 15:13 |
| *** Lucas_Gray has joined #openstack-ironic | 15:13 | |
| dtantsur | I don't think these are necessary related, but I'm not 100% sure | 15:17 |
| *** ociuhandu_ has joined #openstack-ironic | 15:20 | |
| dking | I wanted to troubleshoot more directly, but for some reason, my IPA image isn't letting me log in, even though I've confirmed that the ramdisk has the devuser. So, I'm hoping that if I resolve the one issue maybe it will resolve the other. I gave ipa-lldp-timeout a value of 120.0, and that didn't seem to help. | 15:20 |
| *** ociuhandu has quit IRC | 15:24 | |
| *** ociuhandu_ has quit IRC | 15:24 | |
| *** gyee has joined #openstack-ironic | 15:26 | |
| *** hwoarang has quit IRC | 15:27 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow disabling retries in AgentClient.get_command_statuses https://review.opendev.org/741632 | 15:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Account for power interfaces that cannot power on https://review.opendev.org/741650 | 15:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add agent power interface https://review.opendev.org/735591 | 15:32 |
| iurygregory | seems like I need to use add instead of replace LOL | 15:34 |
| iurygregory | {'error_message': '{"faultcode": "Client", "faultstring": "Couldn\'t apply patch \'{\'path\': \'/instance_info/capabilities\', \'op\': \'replace\', \'value\': {\'boot_option\': \'netboot\'}}\'. Reason: can\'t replace non-existent object \'capabilities\'", "debuginfo": null}'} | 15:34 |
| dtantsur | yup | 15:34 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 15:35 |
| * iurygregory fingers crossed again | 15:35 | |
| dtantsur | TheJulia: with functional cleaning: https://review.opendev.org/#/c/741638/ \o/ | 15:35 |
| patchbot | patch 741638 - bifrost - Enable metadata cleaning by default - 2 patch sets | 15:35 |
| iurygregory | \o/ | 15:35 |
| dking | So, it's not in local_link_connection, or even lldp_basic. Does anybody know what code should be collecting the LLDP information? Is it part of code somewhere for validate_interfaces? | 15:41 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Add missing agent RAID compatibility for ilo5 and idrac https://review.opendev.org/740975 | 15:41 |
| dtantsur | dking: https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/hardware.py#L966 | 15:44 |
| dking | dtantsur: Thank you. I'm hoping that with the logging on, I'll be able to get a little closer. It looks like it just doesn't have any LLDP info at all for the second interface. | 15:47 |
| openstackgerrit | Bill Dodd proposed openstack/ironic master: WIP: Add Redfish RAID management to Ironic https://review.opendev.org/730000 | 15:47 |
| iurygregory | woot Redfish RAID management =O | 15:54 |
| * iurygregory liked the number of the change =D | 15:54 | |
| *** derekh has quit IRC | 15:55 | |
| *** dking has quit IRC | 16:00 | |
| *** lucasagomes has quit IRC | 16:02 | |
| *** k_mouza has quit IRC | 16:02 | |
| dtantsur | heh | 16:04 |
| *** k_mouza has joined #openstack-ironic | 16:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Document fast-track and the agent power interface https://review.opendev.org/741668 | 16:10 |
| dtantsur | TheJulia: note, this ^^ includes fast-track docs (they seem to be missing) | 16:10 |
| openstackgerrit | Bill Dodd proposed openstack/ironic master: WIP: Add Redfish RAID management to Ironic https://review.opendev.org/730000 | 16:12 |
| *** dking has joined #openstack-ironic | 16:13 | |
| dtantsur | on this positive note I'm calling it a day. have a great weekend! | 16:13 |
| *** dtantsur is now known as dtantsur|afk | 16:13 | |
| *** k_mouza has quit IRC | 16:15 | |
| *** yolanda has quit IRC | 16:39 | |
| *** Lucas_Gray has quit IRC | 16:56 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Fixes DIB_IPA_CERT certificate copy issue https://review.opendev.org/741585 | 17:01 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Tempest job to use the inherited devstack_services value https://review.opendev.org/741474 | 17:16 |
| iurygregory | yay the changes on the ironic-tempest-plugin are working \o/ | 17:25 |
| iurygregory | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d0c/741105/3/check/ironic-lib-bios-ipmi-iscsi-src/d0c0547/testr_results.html | 17:25 |
| iurygregory | bye everyone o/ have a great weekend =) | 17:27 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Follow-up on blocking port deletions https://review.opendev.org/741039 | 17:37 |
| TheJulia | with everyone going for the weekend, urge to take a nap rising | 17:50 |
| TheJulia | :) | 17:50 |
| *** vdrok has quit IRC | 18:07 | |
| TheJulia | dtantsur|afk: I could have sworn I wrote someting at some point, but life is a blur. Thanks! | 18:07 |
| *** vdrok has joined #openstack-ironic | 18:08 | |
| *** NobodyCam has quit IRC | 18:12 | |
| JayF | TheJulia: I am jealous of your ability to take naps. (I often have the time, but just can't sleep during the day even if I'm exhausted). Go take a nap so I can nap vicariously loil | 18:12 |
| *** NobodyCam has joined #openstack-ironic | 18:12 | |
| *** TheJulia has quit IRC | 18:22 | |
| *** TheJulia has joined #openstack-ironic | 18:23 | |
| *** johnsom has quit IRC | 18:24 | |
| *** johnsom has joined #openstack-ironic | 18:24 | |
| TheJulia | JayF: I barely can, but even just disconneting and closing the visual input controllers down can improve headspace and energy level | 19:24 |
| *** johnthetubaguy has quit IRC | 19:56 | |
| *** johnthetubaguy has joined #openstack-ironic | 19:58 | |
| *** johnthetubaguy has quit IRC | 20:05 | |
| openstackgerrit | Merged openstack/bifrost master: Set min version of tox to 3.2.1 https://review.opendev.org/741193 | 20:05 |
| *** johnthetubaguy has joined #openstack-ironic | 20:08 | |
| *** Lucas_Gray has joined #openstack-ironic | 20:11 | |
| *** johnthetubaguy has quit IRC | 20:14 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Ironic to use DevStack's neutron"-legacy" module https://review.opendev.org/741256 | 20:22 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Explicitly set jobs to ML2/OVS https://review.opendev.org/739945 | 20:22 |
| * TheJulia needs a caffiene iv | 20:50 | |
| *** rpittau has quit IRC | 21:12 | |
| *** rpittau has joined #openstack-ironic | 21:13 | |
| *** buhman has quit IRC | 21:36 | |
| *** buhman has joined #openstack-ironic | 21:37 | |
| *** bfournie has quit IRC | 21:37 | |
| *** gyee has quit IRC | 21:40 | |
| *** gyee has joined #openstack-ironic | 21:45 | |
| *** melwitt is now known as jgwentworth | 21:49 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: New configuration parameter to use ipmitool retries https://review.opendev.org/741496 | 22:05 |
| *** rh-jelabarre has quit IRC | 22:06 | |
| *** k_mouza has joined #openstack-ironic | 22:17 | |
| *** rloo has quit IRC | 22:18 | |
| *** k_mouza has quit IRC | 22:22 | |
| TheJulia | I'm starting to wonder if we need to somehow adjust our job timeouts for just RAX hosts | 22:29 |
| clarkb | rax hosts are where / is smaller (the rest of the disk is in /opt beacuse that is where devstack goes) | 22:32 |
| clarkb | or maybe devstack goes there because the rest of the disk space is /opt... | 22:32 |
| clarkb | (just pointing that out as I know there have been space constraint issues in the past | 22:32 |
| *** ociuhandu has joined #openstack-ironic | 22:39 | |
| *** ociuhandu has quit IRC | 22:44 | |
| TheJulia | clarkb: Well, I'm thinking more in terms of CPU, since we're seeing ~300 seconds to decompress the ramdisk on some of those hosts. | 22:45 |
| clarkb | are you doing it in a pipe to glance client? | 22:46 |
| clarkb | I forget who it was but doing that in a pipe to glanceclient made things really slow for them. Doing it as a separate decompress then upload made it happier | 22:46 |
| TheJulia | clarkb: VM firing up the ramdisk image | 22:51 |
| TheJulia | so CPU time actually spent decompressing the file in memory for the ramdisk to actually boot | 22:51 |
| clarkb | gotcha | 22:51 |
| clarkb | and it would be doing that within qemu on rax | 22:51 |
| clarkb | which is probably unsurprisingly slow | 22:51 |
| clarkb | I'm not sure how to make that better without a smaller ramdisk (or not compressing it?) | 22:52 |
| TheJulia | I'm basically dealing with a strong desire to use a non-specialty ramdisk which makes it super hard to do it fast | 22:52 |
| TheJulia | I'm kind of just thinking we need to detect rax hosts and adjust a few key tunables so we don't timeout | 22:53 |
| *** gyee has quit IRC | 23:01 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Set min version of tox to 3.2.1 https://review.opendev.org/741190 | 23:14 |
| *** diurnalist has quit IRC | 23:16 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Auto extend the timeout for RAX hosts https://review.opendev.org/741791 | 23:19 |
| *** diurnalist has joined #openstack-ironic | 23:29 | |
| *** diurnalist has quit IRC | 23:38 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add some tuning documentation https://review.opendev.org/739783 | 23:39 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add some tuning documentation https://review.opendev.org/739783 | 23:40 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram https://review.opendev.org/726483 | 23:41 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: DNM/Test Force Insufficent memory in CI https://review.opendev.org/741519 | 23:49 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: DNM/Test Force Insufficent memory in CI https://review.opendev.org/741519 | 23:49 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add some tuning documentation https://review.opendev.org/739783 | 23:49 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram https://review.opendev.org/726483 | 23:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add some tuning documentation https://review.opendev.org/739783 | 23:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: DNM/Test Force Insufficent memory in CI https://review.opendev.org/741519 | 23:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!