| *** rloo has quit IRC | 00:30 | |
| *** mcarden has joined #openstack-ironic | 00:48 | |
| *** xinliang has joined #openstack-ironic | 00:57 | |
| xinliang | Hello morning, one quick question if I deploy ironic with external dhcp server then how to set the dhcp_provider? neutrion or none? | 01:00 |
|---|---|---|
| *** tzumainn has quit IRC | 01:28 | |
| openstackgerrit | yuanliu proposed openstack/ironic master: If the [conductor]XXX_timeout is less than 0，disable periodic task https://review.opendev.org/723795 | 01:40 |
| openstackgerrit | yuanliu proposed openstack/ironic master: If the [conductor]XXX_timeout is less than 0，disable periodic task https://review.opendev.org/723795 | 01:46 |
| openstackgerrit | Merged openstack/ironic master: Update grenade job https://review.opendev.org/725360 | 01:48 |
| *** Qianbiao has joined #openstack-ironic | 01:51 | |
| *** Qianbiao is now known as QIANBIAO | 01:51 | |
| openstackgerrit | Verification of a change to openstack/networking-baremetal failed: BUILD_TIMEOUT is not needed https://review.opendev.org/720016 | 02:00 |
| *** jdandrea has quit IRC | 02:43 | |
| *** mkrai has joined #openstack-ironic | 02:55 | |
| *** xinliang has quit IRC | 03:41 | |
| *** mkrai has quit IRC | 04:06 | |
| *** mkrai has joined #openstack-ironic | 04:09 | |
| *** k_mouza has joined #openstack-ironic | 04:12 | |
| *** k_mouza has quit IRC | 04:16 | |
| *** stevebaker_ has joined #openstack-ironic | 04:26 | |
| *** stevebaker has quit IRC | 04:26 | |
| *** QIANBIAO has quit IRC | 05:07 | |
| *** QIANBIAO has joined #openstack-ironic | 05:07 | |
| *** HagunKim has joined #openstack-ironic | 05:09 | |
| *** uzumaki has quit IRC | 05:19 | |
| *** mkrai has quit IRC | 05:47 | |
| *** mkrai has joined #openstack-ironic | 05:48 | |
| *** gyee has quit IRC | 06:07 | |
| *** ociuhandu has joined #openstack-ironic | 06:17 | |
| arne_wiebalck | Good morning, ironic! | 06:52 |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 07:08 |
| *** ociuhandu has quit IRC | 07:19 | |
| *** xinliang has joined #openstack-ironic | 07:35 | |
| *** rpittau|afk is now known as rpittau | 07:47 | |
| rpittau | good morning ironic! o/ | 07:47 |
| *** gokhani has joined #openstack-ironic | 07:47 | |
| iurygregory | morning rpittau o/ | 07:51 |
| rpittau | hey iurygregory :) | 07:52 |
| iurygregory | we didn't have much lucky in inspector =( | 07:53 |
| arne_wiebalck | hey iurygregory and rpittau o/ | 07:53 |
| rpittau | hey arne_wiebalck :) | 07:53 |
| rpittau | still not passing gate ? | 07:53 |
| rpittau | ah non-standalone failed us! | 07:54 |
| iurygregory | rpittau, not because of grenade (well it failed only once I think) other jobs failling -.-' | 07:54 |
| iurygregory | ironic-inspector-non-standalone-tempest =( | 07:54 |
| *** TheJulia has quit IRC | 07:56 | |
| *** mnasiadka has quit IRC | 07:56 | |
| *** Anticimex has quit IRC | 07:56 | |
| *** sziviani has quit IRC | 07:56 | |
| *** TheJulia has joined #openstack-ironic | 08:01 | |
| *** mnasiadka has joined #openstack-ironic | 08:01 | |
| *** Anticimex has joined #openstack-ironic | 08:01 | |
| *** sziviani has joined #openstack-ironic | 08:01 | |
| *** lucasagomes has joined #openstack-ironic | 08:06 | |
| *** ociuhandu has joined #openstack-ironic | 08:20 | |
| gokhani | hi dtantsur|afk I solved my tftp issue and it is now reacheable from ironic node. but now it stuck at downloading NBP file. it can not download NBP file. what can be the reason for this problem ? I can see traffic on tftp server . tftp logs are in http://paste.openstack.org/show/793247/ and dhcp server logs are in | 08:36 |
| gokhani | http://paste.openstack.org/show/793254/. I can see pxelinux.0 file under the /tftpboot directory but ironic can not download it. | 08:36 |
| *** alexmcleod has joined #openstack-ironic | 08:49 | |
| *** livelace has joined #openstack-ironic | 08:53 | |
| *** jtomasek has joined #openstack-ironic | 08:59 | |
| *** k_mouza has joined #openstack-ironic | 09:07 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf https://review.opendev.org/716889 | 09:15 |
| *** livelace has quit IRC | 09:16 | |
| *** dtantsur|afk is now known as dtantsur | 09:18 | |
| dtantsur | gokhani: TFTP was configured by openstack-ansible, right? could you double-check it against https://docs.openstack.org/ironic/latest/install/configure-pxe.html ? | 09:19 |
| dtantsur | good morning, ironic | 09:19 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: redfish: handle hardware that is unable to set persistent boot https://review.opendev.org/725239 | 09:26 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [Trivial] Fix awkward mocking in redfish boot tests https://review.opendev.org/725918 | 09:26 |
| gokhani | dtantsur, yes it vas configured by openstack-ansible. ok ı will check it. | 09:26 |
| dtantsur | rpittau: hey, could you check https://review.opendev.org/#/c/725239/ please? | 09:26 |
| patchbot | patch 725239 - ironic - redfish: handle hardware that is unable to set per... - 9 patch sets | 09:26 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Build tinyipa on focal https://review.opendev.org/725799 | 09:28 |
| iurygregory | good morning dtantsur o/ | 09:32 |
| dtantsur | people who know or care about inspector: do we have a single reason not to deprecate the iptables PXE filter? | 09:32 |
| dtantsur | asking before I propose it on the ML | 09:32 |
| iurygregory | do we have an idea how many people uses the filter? | 09:33 |
| iurygregory | we can try to also ask in our meeting on monday | 09:34 |
| dtantsur | it's the default now, but the dnsmasq PXE filter allows HA configuration | 09:35 |
| openstackgerrit | Merged openstack/metalsmith master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721658 | 09:35 |
| dtantsur | OoO defaults to dnsmasq https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/ironic/ironic-inspector-container-puppet.yaml#L255 | 09:36 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [docs] remove the horribly outdated version support matrix https://review.opendev.org/726071 | 09:38 |
| iurygregory | we have 3 options iptables, dnsmasq noop | 09:39 |
| rpittau | default is still iptables in inspector, but I think it makes sense to just move it to dnsmasq | 09:39 |
| *** mkowalski[b] has quit IRC | 09:40 | |
| iurygregory | ++ | 09:40 |
| iurygregory | does dnsmasq covers all scenarios where iptables would be used? | 09:40 |
| dtantsur | I *think* so (I wish Milan was around for context) | 09:41 |
| *** irclogbot_1 has quit IRC | 09:41 | |
| *** irclogbot_3 has joined #openstack-ironic | 09:42 | |
| dtantsur | at least TripleO has been using it for a couple of releases | 09:42 |
| dtantsur | hjensas: hey, do you have opinion on ^^^? | 09:42 |
| *** jtomasek has quit IRC | 09:43 | |
| *** sshnaidm|afk is now known as sshnaidm | 09:44 | |
| *** jtomasek has joined #openstack-ironic | 09:45 | |
| *** QIANBIAO is now known as qianbiao|afk | 09:56 | |
| *** qianbiao|afk has quit IRC | 09:56 | |
| iurygregory | dtantsur, for stable-queens I think I have and idea that would allow us to check-out the plugin | 09:56 |
| * iurygregory checks the jobs in stable-queens | 09:56 | |
| *** mkowalski[b] has joined #openstack-ironic | 09:58 | |
| iurygregory | http://paste.openstack.org/show/793257/ | 09:59 |
| dtantsur | iurygregory: that's probably what is required. can you update rpittau's patch to try it? | 10:00 |
| iurygregory | dtantsur, sure I can give a try | 10:00 |
| iurygregory | what version we want from our plugin? | 10:00 |
| *** xinliang has quit IRC | 10:00 | |
| dtantsur | eeehhhmmmmm | 10:00 |
| iurygregory | XD | 10:00 |
| iurygregory | I'm looking here to see if I can figure out | 10:01 |
| dtantsur | it clearly worked a few months ago | 10:01 |
| iurygregory | so maybe 1.5.0 or 1.5.1 | 10:02 |
| rpittau | did we change default driver for the jobs ? | 10:02 |
| rpittau | so before 2.0.0 | 10:02 |
| dtantsur | I was about to propose 1.5.1, yes | 10:02 |
| rpittau | ok | 10:02 |
| dtantsur | Sep 2019, I think stable/queens worked back then\ | 10:02 |
| iurygregory | you going to update rpittau? | 10:02 |
| rpittau | iurygregory: no, go ahead, you can even push a different patch as I guess with the cap my changes are not needed | 10:03 |
| iurygregory | rpittau, ok I will give a try | 10:03 |
| *** jtomasek has quit IRC | 10:04 | |
| *** rpittau is now known as rpittau|bbl | 10:10 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Silence debug messages from oslo_messaging https://review.opendev.org/725789 | 10:11 |
| *** jtomasek has joined #openstack-ironic | 10:12 | |
| gokhani | dtantsur, do ı need to install also networking-baremetal plugin and agent ? In my environment I am using linuxbridge | 10:13 |
| dtantsur | gokhani: ideally, you should. they're not strictly required, unless you're using routed networks. | 10:14 |
| gokhani | dtantsur, ok ı will install it and continue step step :) | 10:17 |
| *** jtomasek has quit IRC | 10:17 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/queens: Pin ironic-tempest-plugin version https://review.opendev.org/726083 | 10:18 |
| dtantsur | gokhani: if you encounter missing or incorrect setup, please do let the openstack-ansible folks know | 10:18 |
| iurygregory | let's see how it goes | 10:18 |
| *** jtomasek has joined #openstack-ironic | 10:18 | |
| dtantsur | we don't have constant contacts with them, so this sort of feedback is valuable | 10:18 |
| dtantsur | thanks iurygregory! | 10:18 |
| iurygregory | dtantsur, np o/ fingers crossed | 10:18 |
| iurygregory | ok seems i need to add "name" before the project -.-' | 10:20 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/queens: Pin ironic-tempest-plugin version https://review.opendev.org/726083 | 10:20 |
| *** derekh has joined #openstack-ironic | 10:21 | |
| hjensas | dtantsur: iurygregory: regarding inspector filter's. Seems we don't have much issues with the dnsmasq filtering. So switching the default makes sense. Deprecating iptables however not so good, as use with other DHCP server software will break. | 10:29 |
| dtantsur | it's a good point, I wonder if anybody is using ironic-inspector with other DHCP server.. | 10:30 |
| *** ociuhandu has quit IRC | 10:32 | |
| *** ociuhandu has joined #openstack-ironic | 10:33 | |
| hjensas | hard to know, probably not. but we never stop being surprised of how someone quietly uses software in some corner. | 10:34 |
| iurygregory | truth | 10:35 |
| *** k_mouza has quit IRC | 10:35 | |
| iurygregory | sounds like a good idea change the default at least =) | 10:35 |
| dtantsur | hjensas: thanks! do you by chance remember why we needed to create the dnsmasq filter? | 10:36 |
| dtantsur | I understand it scales much better, but I don't remember if there were other arguments | 10:36 |
| *** ociuhandu has quit IRC | 10:38 | |
| iurygregory | the checkout seems to work let's see if the tests will run ok =) | 10:40 |
| hjensas | When using a dhcp relay we cannot filter on the MAC because it would be the realy servers mac address. | 10:40 |
| dtantsur | hjensas: so, how does the dnsmasq filter handle it? | 10:41 |
| hjensas | dtantsur: dnsmasq get's the mac from the dhcp payload. | 10:42 |
| dtantsur | ah, nice. Thanks for refreshing my memory! | 10:42 |
| *** ociuhandu has joined #openstack-ironic | 10:42 | |
| *** k_mouza has joined #openstack-ironic | 10:47 | |
| *** mkrai has quit IRC | 10:48 | |
| *** k_mouza has quit IRC | 10:58 | |
| *** ociuhandu has quit IRC | 11:04 | |
| *** ociuhandu has joined #openstack-ironic | 11:04 | |
| *** k_mouza has joined #openstack-ironic | 11:04 | |
| *** k_mouza has quit IRC | 11:05 | |
| *** k_mouza has joined #openstack-ironic | 11:05 | |
| *** ociuhandu has quit IRC | 11:10 | |
| *** livelace has joined #openstack-ironic | 11:38 | |
| openstackgerrit | Merged openstack/ironic master: Remove testscenarios, testrepository and Babel from test-requirements https://review.opendev.org/723871 | 11:43 |
| bfournie | dtantsur: good morning, so for https://review.opendev.org/725954, should we just make the retry limit configurable? If we change command_retry_timeout to a lower default it still won't reflect how long command is taking - i.e. its taking 2 minutes when command_retry_limit is 60 seconds, although maybe that doesn't matter | 11:44 |
| patchbot | patch 725954 - ironic - Limit the number of ipmitool retries - 2 patch sets | 11:44 |
| iurygregory | stable/queens green again \o/ | 11:49 |
| iurygregory | bfournie, make it configurable would be good ++ | 11:49 |
| iurygregory | dtantsur, it worked \o/ https://review.opendev.org/726083 | 11:51 |
| patchbot | patch 726083 - ironic (stable/queens) - Pin ironic-tempest-plugin version - 2 patch sets | 11:51 |
| *** rh-jelabarre has joined #openstack-ironic | 11:55 | |
| *** alexmcleod has quit IRC | 12:03 | |
| *** alexmcleod has joined #openstack-ironic | 12:04 | |
| dtantsur | iurygregory: nice job! | 12:10 |
| iurygregory | o/ tks | 12:10 |
| dtantsur | bfournie: command_retry_limit defines the number of retries | 12:10 |
| dtantsur | the 2 mins vs 60 mins issue is the ipmitool internal problem from our perspective | 12:10 |
| dtantsur | I quite regret having a timeout instead of a number of retries, but I'm not sure it's worth the deprecation dance | 12:11 |
| *** rpittau|bbl is now known as rpittau | 12:11 | |
| iurygregory | I think the jobs in ironic-inspector had a meeting and they decided that they should fail in one after the other -.- | 12:14 |
| dtantsur | cooperating! | 12:14 |
| iurygregory | yeah | 12:14 |
| iurygregory | hey it's your turn to fail now, SURE! | 12:14 |
| dtantsur | we need to limit the number of jobs for real | 12:14 |
| iurygregory | yeah | 12:15 |
| dtantsur | I wonder why the inspector jobs are seemingly less stable than the ironic ones | 12:16 |
| dtantsur | I mean, we have fewer jobs on the former.. | 12:16 |
| iurygregory | yeah | 12:17 |
| iurygregory | and we are still in bionic | 12:17 |
| iurygregory | I'm just waitting for the nightmare with focal | 12:17 |
| iurygregory | =) | 12:17 |
| dtantsur | oh boy | 12:17 |
| dtantsur | iurygregory: look, what if we merge the "managed" job into the non-standalone one? | 12:17 |
| dtantsur | no, the other way around | 12:18 |
| iurygregory | non-standalone in managed | 12:18 |
| dtantsur | let's make the managed job run in split mode? | 12:18 |
| iurygregory | can be | 12:18 |
| iurygregory | I can push a patch for that | 12:18 |
| dtantsur | thanks, would be grat | 12:18 |
| dtantsur | * great | 12:18 |
| iurygregory | let me just grab some coffe =) | 12:18 |
| dtantsur | coffee++ | 12:19 |
| dtantsur | I switched to decaf some time ago, but because the world is in flames, I've started adding strawberry baileys recently. | 12:20 |
| dtantsur | I hope it still counts as healthier? | 12:20 |
| iurygregory | O.o wait | 12:20 |
| iurygregory | strawberry + coffe? | 12:20 |
| dtantsur | is it the first time you notice I'm weird? :D | 12:20 |
| rpittau | dtantsur, iurygregory, I think the instability is related mainly in the different times of execution, we're adding complexity and preocesses with inspection and that adds a lot of time, adding variables where we can tweak the timeout I think it will help in the future | 12:21 |
| iurygregory | nah you are not weird | 12:21 |
| dtantsur | rpittau: yep. but it doesn't prevent us from having fewer jobs | 12:21 |
| iurygregory | rpittau, having the timeouts configurables is a good approach | 12:21 |
| bfournie | dtantsur: ok, the only thing with just using command_retry_timeout is its also used with kill_on_timeout in addition to deriving the number of retries https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L497 | 12:21 |
| jroll | dtantsur: I can get behind baileys, but I think caffeine is probably healthier than alcohol :P | 12:21 |
| jroll | morning all :) | 12:21 |
| iurygregory | dtantsur, when I was at the university i had vodka + strawberry yogurt XD | 12:22 |
| dtantsur | morning jroll | 12:22 |
| dtantsur | iurygregory: sounds about right | 12:22 |
| dtantsur | bfournie: ugh. that is not perfect. | 12:23 |
| iurygregory | my friends were: WTF dude?! XD | 12:23 |
| rpittau | dtantsur: no, I'm totally for reducing the jobs, just saying that is one more way to help prevent failures | 12:23 |
| iurygregory | morning jroll | 12:23 |
| jroll | \o | 12:23 |
| iurygregory | caffeine with alcohol is better =) | 12:23 |
| dtantsur | ... and this is not even Friday yet | 12:24 |
| iurygregory | thats why stouts are good =D | 12:24 |
| rpittau | iurygregory: we call that "caffè corretto" :) | 12:24 |
| rpittau | dtantsur: technically for iurygregory and me is friday :D | 12:24 |
| dtantsur | ah, right | 12:24 |
| bfournie | dtantsur: yeah I'm not sure that kill_on_timeout is working as we expect, I don't see it killing ipmitool in this case even though we've exceeded that time /me investigates | 12:24 |
| iurygregory | yeah | 12:24 |
| iurygregory | for me is friday XD | 12:24 |
| dtantsur | rpittau: Germans don't celebrate victory day for obvious reasons :D | 12:24 |
| jroll | hmm, iurygregory says I should add whisky to this coffee? I guess I must | 12:24 |
| dtantsur | bfournie: I *think* kill_on_timeout is off by default | 12:25 |
| rpittau | dtantsur: I don't blame them, Italians also don't :D | 12:25 |
| iurygregory | jroll, some people mix whisky with beer so why not | 12:25 |
| jroll | indeed | 12:25 |
| *** ociuhandu has joined #openstack-ironic | 12:25 | |
| dtantsur | I'd go for rum with coffee then | 12:25 |
| dtantsur | bfournie: mmm, [ipmi]kill_on_timeout is True by default | 12:25 |
| bfournie | dtantsur: yeah | 12:26 |
| rpittau | we put "abba ardente" (warm water) in coffee sometimes :) | 12:26 |
| dtantsur | kind of what people call americano? | 12:26 |
| rpittau | it's not literally warm water | 12:26 |
| dtantsur | ah! | 12:26 |
| rpittau | :) | 12:26 |
| dtantsur | gotta like euphemisms | 12:26 |
| jroll | neat :) | 12:27 |
| dtantsur | black tea with brandy is a more Russian thing | 12:27 |
| dtantsur | ... and brown sugar if you like it sweet | 12:28 |
| *** ociuhandu has quit IRC | 12:28 | |
| iurygregory | wouldnt be white sugar? O.o | 12:28 |
| *** ociuhandu has joined #openstack-ironic | 12:28 | |
| dtantsur | why? | 12:29 |
| iurygregory | brown sugar is less sweet I would say, or requires lager ammounts compare to the white sugar (but it's healthier) | 12:29 |
| dtantsur | well, it may be my thing. it's the only case when I put any sugar in tea. | 12:29 |
| bfournie | dtantsur: checking why that kill didn't happen, but anyway my thought was that if we shorten to timeout to affect the num_retries it will also affect this kill_on_timeout functionality, so configurable retry_limit could be useful | 12:30 |
| dtantsur | bfournie: are we really out of options to make ipmitool NOT retry on cipher suite unavailability? | 12:31 |
| iurygregory | if it provides an specific exception we can handle to not retry I would say | 12:31 |
| dtantsur | bfournie: I don't mind having retry_limit, but then we need to figure out how it's going to work with command_retry_timeout | 12:31 |
| bfournie | dtantsur: haven't heard much from the ipmitool guy except questioning why we do so many rertries | 12:32 |
| dtantsur | iurygregory: we cannot, it's inside ipmitool | 12:32 |
| iurygregory | ouch | 12:32 |
| dtantsur | bfournie: we may be playing on a safe side | 12:32 |
| iurygregory | we can't check the result? | 12:32 |
| dtantsur | iurygregory: we can, after 2 minutes | 12:32 |
| bfournie | dtantsur: I think it could be used just an upper bound | 12:32 |
| dtantsur | bfournie: not sure I get it, how? | 12:33 |
| dtantsur | you mean, for killing ipmitool? | 12:33 |
| dtantsur | sure, but there is backward compatibility. people might have changed the timeout and expect it to affect the number of retries | 12:34 |
| dtantsur | if we introduce a separate option for retries, we need to be careful with such cases | 12:34 |
| * dtantsur -> tea (without brandy, unfortunately) | 12:34 | |
| openstackgerrit | Merged openstack/networking-baremetal master: BUILD_TIMEOUT is not needed https://review.opendev.org/720016 | 12:36 |
| *** Goneri has joined #openstack-ironic | 12:37 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Add timeout and retries to JSON RPC client https://review.opendev.org/725883 | 12:38 |
| bfournie | iurygregory: I meant use retries=command_retry_timeout/interval as we do now, and just not to exceed the new retry_limit, so any changes to timeout would still be used. The retries wouldn't affect the kill_on_timeout | 12:38 |
| bfournie | sorry meant for dtantsur ^ | 12:40 |
| iurygregory | bfournie, this would reduce the number of failures, but we probably want to not retry if the error is persistent (if we can of course) | 12:41 |
| iurygregory | if the command fails and the output would tell that it was because of the cipher we can avoid running retry I think | 12:41 |
| bfournie | iurygregory: these are just the retries passed to ipmitool, if there is a peristent connection failure for example ipmitool will not retry | 12:42 |
| iurygregory | oh | 12:42 |
| iurygregory | gotcha | 12:42 |
| bfournie | iurygregory: we're not doing the retry, ipmitool is | 12:42 |
| iurygregory | ok I thought we have the controll inside the python code =) | 12:43 |
| bfournie | iurygregory: unfortunately no | 12:43 |
| iurygregory | ok so that is a bit more complicated D: | 12:43 |
| dtantsur | bfournie: it won't cancel out the backward compatibility issue that will prevent this change from being backportable.. | 12:46 |
| dtantsur | i.e. we'll have to default retry_limit=None (meaning, do as before) | 12:46 |
| dtantsur | I'm not against what you propose, just trying to think how to make it most useful | 12:46 |
| bfournie | dtantsur: yeah, we could also do rety_limit=timeout/interval as default, so there would be no change | 12:47 |
| * dtantsur thinks | 12:47 | |
| bfournie | dtantsur: doesn't like adding this extra config but trying to make it useful for users | 12:48 |
| dtantsur | bfournie: let's do what you propose (with a new option) and let's change the option to something smaller on master only (in a separate change with its own release note) | 12:49 |
| dtantsur | then we can consider what to do with tripleo/downstream | 12:49 |
| bfournie | dtantsur: ok sounds good, should we backport rety_limit=timeout/interval as default in Train and keep it smaller by default in master? | 12:50 |
| dtantsur | bfournie: exactly (hence two patches) | 12:51 |
| bfournie | dtantsur: good, thanks | 12:51 |
| dtantsur | bfournie: np, thank you for looking into it. I still believe we need to solve it with ipmitool people. Whatever we use as a new timeout, it's not great to hit it literally every time. | 12:52 |
| dtantsur | we can cut it 3x, but it's still 40 seconds | 12:53 |
| *** jdandrea has joined #openstack-ironic | 12:53 | |
| bfournie | dtantsur: yeah I think so to, there is definitely an ipmitool issue there | 12:54 |
| dtantsur | bfournie: I suggest we also add an option to specify a cipher suite explicitly (if we don't have one yet) | 12:54 |
| dtantsur | wanna look into that or should I? | 12:54 |
| dtantsur | (asking because it'll probably conflict with your patch) | 12:55 |
| bfournie | dtantsur: good point, we don't. HP works fine with CS 3, but supposedly others don't. I will look into this | 12:55 |
| dtantsur | great, thanks! | 12:55 |
| bfournie | dtantsur: it would have to be separate, no backport for that I would think | 12:56 |
| dtantsur | yeah, there are two sane suites in existence, 3 and some high number | 12:56 |
| bfournie | dtantsur: 17? | 12:56 |
| dtantsur | bfournie: it would have to bee separate and I can make a case to backport that even though it looks like a feature | 12:56 |
| dtantsur | yeah, 17 | 12:56 |
| dtantsur | "My hardware does not work without specifying a suite explicitly" is a strong enough reason for backport | 12:56 |
| iurygregory | ^ agree | 12:57 |
| dtantsur | (and something I'm trying to make official in the proposed release model) | 12:57 |
| iurygregory | "Why does Ironic hate me?" "It doesn't hate you, the hardware hates us all!" | 12:57 |
| dtantsur | +++ | 12:58 |
| iurygregory | best thing I saw yesterday =D | 12:58 |
| dtantsur | yeah :) I seriously want a t-shirt with that. | 12:58 |
| iurygregory | ^ yup | 12:58 |
| dtantsur | I'd also send it to all people who assume they can make it simpler than ironic | 12:58 |
| dtantsur | (esp. by Just Rewriting It In Go (tm)) | 12:58 |
| iurygregory | and we can have the bear doing a heart XD | 12:58 |
| dtantsur | I was thinking about a bear and a redfish eating each other | 12:59 |
| dtantsur | a bit darker theme, but we can have variations :D | 12:59 |
| iurygregory | hehehehe | 12:59 |
| iurygregory | one fish in each hand | 12:59 |
| iurygregory | one trying to bite the other | 12:59 |
| iurygregory | hehehehe | 12:59 |
| dtantsur | yep | 12:59 |
| * dtantsur wonders if any recent hardware still enables cipher 0 | 13:01 | |
| dtantsur | cipher 3, I think, is AES256+MD5, cipher 17 is AES256+SHA256(?) | 13:02 |
| TheJulia | good morning | 13:11 |
| rpioso | Good morning, ironic | 13:11 |
| rpioso | TheJulia: o/ | 13:11 |
| *** uzumaki has joined #openstack-ironic | 13:14 | |
| dtantsur | morning TheJulia | 13:15 |
| dtantsur | we've declared Friday and started discussing drinks already | 13:15 |
| dtantsur | also morning rpioso | 13:15 |
| rpittau | good morning TheJulia and rpioso :) | 13:15 |
| uzumaki | morning ironic! o/ | 13:16 |
| dtantsur | hi uzumaki | 13:16 |
| rpittau | hey uzumaki :) | 13:17 |
| uzumaki | hey dtantsur ! how's it oing? | 13:17 |
| dtantsur | see above :D | 13:17 |
| TheJulia | dtantsur: already?!? | 13:17 |
| uzumaki | rpittau, hola! how are you? | 13:17 |
| TheJulia | dtantsur: should I even ask why? | 13:17 |
| TheJulia | can I at least have coffee first | 13:17 |
| TheJulia | ? | 13:17 |
| uzumaki | dtantsur, all I see is the message about declaring friday as something, and what's that? | 13:17 |
| TheJulia | #amillionquestions | 13:17 |
| dtantsur | It was mostly without context (although the CI is misbehaving) | 13:17 |
| rpittau | uzumaki: all's good, it's technically-friday :) | 13:18 |
| dtantsur | uzumaki: not Friday as something, but today as Friday :) | 13:18 |
| dtantsur | TheJulia: I think I started it when coffee was last mentioned with: $ I switched to decaf some time ago, but because the world is in flames, I've started adding strawberry baileys recently. | 13:18 |
| uzumaki | oh, really? what timezone are you? here in Pakistan it's thursday evening | 13:18 |
| dtantsur | uzumaki: iurygregory and rpittau are in a timezone that has a public holiday tomorrow :) | 13:19 |
| TheJulia | I'm in US Pacific, so just getting up | 13:19 |
| dtantsur | (May 8th, Victory day) | 13:19 |
| dtantsur | not quite a victory day here though :D | 13:19 |
| iurygregory | good morning TheJulia o/ | 13:19 |
| uzumaki | dtantsur, that's so cool! holidays are always great | 13:19 |
| rpittau | I'm stealing French victory day :P | 13:19 |
| uzumaki | dtantsur, lol | 13:19 |
| iurygregory | morning rpittau, uzumaki o/ | 13:19 |
| TheJulia | rpittau: ++ | 13:19 |
| iurygregory | and I'm stealing Czech Victory day XD | 13:19 |
| TheJulia | rpittau: iurygregory: so no planning for you guys tomorrow? | 13:19 |
| rpittau | iurygregory: morning! :D | 13:19 |
| uzumaki | morning iurygregory ! o/ how's it going? you're going to be celebrating this victory day as well? | 13:19 |
| iurygregory | TheJulia, I can show up without problems | 13:20 |
| iurygregory | only for the planning ofc =) | 13:20 |
| TheJulia | I don't want to take away from celebration of a holiday or time away from the evil computers | 13:20 |
| *** ociuhandu has quit IRC | 13:20 | |
| *** ociuhandu_ has joined #openstack-ironic | 13:20 | |
| iurygregory | uzumaki, I have nothing to celebrate hehehehe XD since I'm from Brazil lol | 13:20 |
| dtantsur | move planning to Monday? | 13:20 |
| dtantsur | sorry, we shouldn't be discussing downstream meetings here | 13:20 |
| TheJulia | https://giphy.com/gifs/evil-hacker-hEMF9k5UHh2U0 | 13:20 |
| iurygregory | yeah | 13:21 |
| rpittau | move Mondays ? | 13:21 |
| uzumaki | iurygregory, but you're in CZ, don't they share this holiday with other european countries? | 13:21 |
| dtantsur | not with Germany for sure | 13:21 |
| iurygregory | ^ yeah | 13:21 |
| uzumaki | dtantsur, ah well, that's too bad | 13:21 |
| TheJulia | I'll poke bfournie downstream | 13:21 |
| iurygregory | I'm not even sure how was history here in Czech Republic | 13:21 |
| dtantsur | iurygregory: Czechia was on the right side of the conflict | 13:22 |
| iurygregory | dtantsur, gotcha | 13:22 |
| dtantsur | I don't remember when exactly it was freed, presumably early spring 1945 | 13:22 |
| dtantsur | or late winter | 13:23 |
| * uzumaki thinks it's clever how dtantsur said "right side of the conflict" ;) | 13:23 | |
| TheJulia | question sent | 13:23 |
| TheJulia | now, caffination must occur | 13:23 |
| dtantsur | uzumaki: there is a wide consensus nowadays which side was right :) | 13:23 |
| dtantsur | thanks TheJulia | 13:23 |
| uzumaki | i just like the pun xD dtantsur | 13:23 |
| dtantsur | :) | 13:24 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Add stable/ussuri job on master gate https://review.opendev.org/725728 | 13:25 |
| dtantsur | when is the next SPUC by the way? | 13:26 |
| TheJulia | i don't konw | 13:26 |
| TheJulia | I've been insanely busy and trying to battle off the daemons of depression | 13:27 |
| dtantsur | TheJulia: I hope you succeeded, these daemons are insanely fierce | 13:27 |
| iurygregory | TheJulia, you can do it o/ | 13:27 |
| TheJulia | I'm fairly strong, just... yeah | 13:27 |
| dtantsur | there is a mild consensus for this Friday https://doodle.com/poll/2q5zmv3g6uy2475e | 13:28 |
| TheJulia | it doesn't help there are like a million hanging threads in personal life | 13:28 |
| iurygregory | and if you need help you can ask us \o/ | 13:28 |
| dtantsur | ++ | 13:28 |
| *** HagunKim has quit IRC | 13:29 | |
| iurygregory | ajya, you had some problems in your backport to stable/queens if I do remember, after https://review.opendev.org/#/c/726083/ is merged you can put a recheck on your patches | 13:29 |
| patchbot | patch 726083 - ironic (stable/queens) - Pin ironic-tempest-plugin version - 2 patch sets | 13:29 |
| TheJulia | I guess the 9am winidow will work, I'm fairly sure they won't need me on the one conflict | 13:30 |
| dtantsur | TheJulia: 9am is 2pm UTC? | 13:30 |
| dtantsur | (the doodle is in local time for everyone) | 13:30 |
| TheJulia | uhh | 13:30 |
| TheJulia | 4pm UTC | 13:31 |
| *** tzumainn has joined #openstack-ironic | 13:31 | |
| dtantsur | ah, the one with 4 votes? | 13:31 |
| TheJulia | well, if we move the downstream meeting to next week | 13:32 |
| TheJulia | then we can start an hour earlier | 13:32 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Change ownership of tce info dir https://review.opendev.org/726119 | 13:32 |
| dtantsur | ++ I like the idea | 13:32 |
| dtantsur | asking people to work on the holiday is one thing, to wear a silly hat and chat - another | 13:32 |
| TheJulia | bob's moving the downstream meeting | 13:32 |
| dtantsur | cool. so, 3pm UTC? could you respond to the thread? | 13:33 |
| TheJulia | on the ml? | 13:33 |
| dtantsur | yeah, I've just responded there, so it should be on the top of your inbox | 13:33 |
| TheJulia | k | 13:33 |
| * iurygregory completely forgot about the II SPUC <O> | 13:34 | |
| * TheJulia gives the emails coffee to wkae it up | 13:34 | |
| dtantsur | iurygregory: if this ever repeats, we'll have to remove you from the position of the SPUC Manager! *shakes his finger* | 13:35 |
| rpittau | heh I won't be there tomorrow for the SPUC :( | 13:35 |
| dtantsur | :D | 13:35 |
| iurygregory | =O | 13:35 |
| dtantsur | rpittau: :( | 13:35 |
| iurygregory | I was a manager?! | 13:35 |
| iurygregory | no no I'm only an engineer =P | 13:35 |
| TheJulia | lol | 13:35 |
| dtantsur | iurygregory: this IT field, you just blink and they've already given you responsibilities, amiright? | 13:35 |
| TheJulia | lol | 13:35 |
| iurygregory | hehehehe | 13:35 |
| uzumaki | what's SPUC? | 13:36 |
| iurygregory | dtantsur, you are correct! | 13:36 |
| uzumaki | (that question might put me below the engineer in the heirarchy xD) | 13:36 |
| dtantsur | uzumaki: http://lists.openstack.org/pipermail/openstack-discuss/2020-March/013521.html | 13:36 |
| iurygregory | uzumaki, Sanity Preservation Un-Conference | 13:36 |
| dtantsur | a way to re-confirm that we're still humans, at least visually and vocally | 13:37 |
| uzumaki | I love that xD | 13:37 |
| uzumaki | wouldn't be surprised if iurygregory was the manager though xD | 13:37 |
| iurygregory | I'm the manager | 13:38 |
| dtantsur | uzumaki: 3pm UTC may be a bit late for you.. but if you can make it, stop by tomorrow | 13:38 |
| dtantsur | (dunno if we have a video call link yet) | 13:38 |
| uzumaki | sure, I'll try to stop by! | 13:38 |
| iurygregory | uzumaki, I was basically helping TheJulia =) | 13:38 |
| uzumaki | iurygregory, now that's too modest | 13:39 |
| iurygregory | uzumaki, well that's the truth =) since TheJulia sent the email with the whole idea =) | 13:40 |
| iurygregory | I only sent an email with a doodle to get a timeslot for the II SPUC =P | 13:40 |
| uzumaki | alright, humble human, I shall accept your explanation hereforth | 13:40 |
| dtantsur | Julia is a Principal SPUC Architect | 13:40 |
| iurygregory | ^ this | 13:41 |
| * dtantsur now suspects uzumaki is a cat | 13:41 | |
| iurygregory | LOL | 13:41 |
| *** jhesketh has quit IRC | 13:43 | |
| *** jtomasek has quit IRC | 13:45 | |
| TheJulia | heh | 13:46 |
| *** jhesketh has joined #openstack-ironic | 13:47 | |
| *** rloo has joined #openstack-ironic | 13:47 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Merge jobs https://review.opendev.org/726121 | 13:47 |
| TheJulia | so 2pm is 7 am for me, I can start at 8 am but I have a 7 am team leads call | 13:47 |
| * iurygregory forgot git review in the terminal... -.-' | 13:47 | |
| TheJulia | so if you guys are cool with starting without me! | 13:47 |
| dtantsur | TheJulia: I think we can start at 3pm, most people voted yes for 3-4 | 13:47 |
| dtantsur | then whoever can stay will stay | 13:47 |
| iurygregory | yeah | 13:48 |
| * dtantsur is not sure he'll last more than an hour anyway | 13:48 | |
| openstackgerrit | Merged openstack/ironic stable/queens: Pin ironic-tempest-plugin version https://review.opendev.org/726083 | 13:49 |
| openstackgerrit | Merged openstack/ironic-inspector master: [docs] remove the horribly outdated version support matrix https://review.opendev.org/726071 | 13:49 |
| iurygregory | yay stable/queens fixed \o/ | 13:50 |
| iurygregory | dtantsur, I had a patch for Ironic to rename the job XD forgot to send | 13:51 |
| iurygregory | and also the job is non-voting in ironic... | 13:51 |
| dtantsur | iurygregory: dunno, I'd just keep the name.. | 13:51 |
| dtantsur | we cannot encode every fact there anyway | 13:52 |
| dtantsur | but as you wish | 13:52 |
| iurygregory | I have the feeling that the name of the job should give an idea of what it does .-. | 13:52 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Change the name of the inspector job https://review.opendev.org/726124 | 13:54 |
| dtantsur | iurygregory: okay. but it seems that we have to do a 3-way rename or we'll upset zuul | 13:56 |
| dtantsur | it may not like if we remove a job from under it, even a non-voting one | 13:56 |
| iurygregory | oh | 13:56 |
| iurygregory | =O | 13:56 |
| iurygregory | I wasn't aware of that I thougt the job would only TIMED_OUT | 13:56 |
| iurygregory | and since it's non-voting we wouldn't get -1 from CI | 13:57 |
| dtantsur | definitely not, but I wondered if it can SKIP a job | 13:57 |
| dtantsur | it seems that not, see the failure on your ironic patch | 13:57 |
| dtantsur | let's play it safe and create a temporary duplicate? | 13:57 |
| iurygregory | yeah | 13:57 |
| iurygregory | I will rebase the inspector-pacth in 33min if CI merges 725514 | 13:58 |
| iurygregory | since I will have merge conflict | 13:58 |
| dtantsur | wfm | 13:59 |
| * iurygregory goes back to figure out the cinder deprecation =) | 13:59 | |
| TheJulia | dtantsur: sounds good to me | 14:00 |
| dtantsur | TheJulia: wanna reply with the updated time and your bjeans link? | 14:01 |
| TheJulia | yup, doing that now | 14:01 |
| bfournie | dtantsur: not quite clear to me how this poll for kiil_on_timeout is supposed to work but we're not killing these ipmtool sessions because poll isn't set - https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L396 | 14:09 |
| *** ociuhandu_ has quit IRC | 14:09 | |
| *** ociuhandu has joined #openstack-ironic | 14:10 | |
| *** gokhani has quit IRC | 14:13 | |
| *** ociuhandu has quit IRC | 14:15 | |
| dtantsur | OUCH | 14:19 |
| *** uzumaki has quit IRC | 14:20 | |
| iurygregory | ofc it will fail -.-' ironic-inspector-non-standalone-tempest tks! | 14:20 |
| dtantsur | iurygregory: is it also timeouts? | 14:21 |
| iurygregory | Details: Node c93d27d2-fe9c-4197-88ef-8c9058c6470d reached failure state inspect failed while waiting for provision_state=['manageable']. Error: None | 14:21 |
| iurygregory | looks like | 14:21 |
| dtantsur | "Error: None", I've wanted to look into it for a while.. | 14:21 |
| dtantsur | you can check the inspector logs to be sure | 14:22 |
| iurygregory | yeah doing now | 14:22 |
| *** ociuhandu has joined #openstack-ironic | 14:24 | |
| dtantsur | iurygregory: do you have a link to the tempest output? last_error should work, I don't get why it doesn't it | 14:27 |
| iurygregory | last_error null | 14:27 |
| iurygregory | .-. | 14:27 |
| dtantsur | exactly this I don't understand.. | 14:28 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Fix grenade job https://review.opendev.org/725514 | 14:28 |
| *** ociuhandu has quit IRC | 14:29 | |
| iurygregory | the test fails after 3min it started | 14:29 |
| *** ociuhandu has joined #openstack-ironic | 14:29 | |
| dtantsur | ehhmm, yeah | 14:29 |
| dtantsur | and ironic-inspector output has error:null, why is it failed in ironic? O__o | 14:30 |
| dtantsur | Error contacting ironic-inspector for inspection of node c93d27d2-fe9c-4197-88ef-8c9058c6470d: HttpException: HttpException: 502: Server Error for url: http://158.69.64.186/baremetal-introspection/v1/introspection/c93d27d2-fe9c-4197-88ef-8c9058c6470d, error was encountered while trying to use an ErrorDocument to handle the request.: Apache/2.4.29 (Ubuntu) Server at 158.69.64.186 Port 80: | 14:31 |
| dtantsur | Bad Gateway: response from an upstream server.: Additionally, a 202 Accepted: 502 Bad Gateway: The proxy server received an invalid: openstack.exceptions.HttpException: HttpException: 502 | 14:31 |
| iurygregory | https://6294497186881d7ba137-7f38bc6e36f1263ac9d91d77e56fe624.ssl.cf5.rackcdn.com/725514/4/gate/ironic-inspector-non-standalone-tempest/6a16fe9/controller/logs/ironic-bm-logs/node-0_console_log.txt | 14:31 |
| iurygregory | wut :O | 14:31 |
| dtantsur | something is broken in inspector isefl | 14:31 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Change ownership of tce info dir https://review.opendev.org/726119 | 14:32 |
| dtantsur | there are actually two things problematic here: 1. inspector returns 502, 2. we don't retry it | 14:32 |
| dtantsur | wtf is this https://zuul.opendev.org/t/openstack/build/6a16fe9e33a340b691fe70e10778afb8/log/controller/logs/screen-ironic-inspector-api.txt#846 | 14:32 |
| rpittau | wasn't the job passing ? | 14:32 |
| iurygregory | rpittau, it's passing | 14:32 |
| iurygregory | when it wants to | 14:32 |
| dtantsur | stupid rabbit | 14:33 |
| rpittau | that looks like rabbit failing hard | 14:33 |
| * dtantsur asks on #openstack-oslo | 14:34 | |
| iurygregory | dtantsur, funny that when the job works there is also this problem | 14:36 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/12eacd0dba41423b8a6a343734517b02/log/controller/logs/screen-ironic-inspector-api.txt | 14:36 |
| iurygregory | =) | 14:36 |
| *** dsneddon has quit IRC | 14:36 | |
| dtantsur | okay, so it's not a root cause | 14:39 |
| dtantsur | we need to understand why we're seeing HTTP 502 there and why it's not retried | 14:40 |
| dtantsur | I wonder if we can ignore failures while checking inspection status | 14:41 |
| dtantsur | worst case it will eventually time out | 14:41 |
| dtantsur | thoughts? | 14:41 |
| iurygregory | can be something where we are running the job? | 14:42 |
| *** tkajinam has quit IRC | 14:43 | |
| iurygregory | check was running in rax, gate in ovh | 14:44 |
| rpittau | dtantsur: have you seen the DEBUG msg in inspector conductor about iptables ? | 14:47 |
| dtantsur | nope, link? | 14:49 |
| rpittau | https://6294497186881d7ba137-7f38bc6e36f1263ac9d91d77e56fe624.ssl.cf5.rackcdn.com/725514/4/gate/ironic-inspector-non-standalone-tempest/6a16fe9/controller/logs/screen-ironic-inspector-conductor.txt | 14:49 |
| *** jtomasek has joined #openstack-ironic | 14:49 | |
| jroll | boo, someone booked a meeting on top of the SPUC, I was going to come this time :( | 14:51 |
| rpittau | dtantsur: it seems the introspection starts btw | 14:53 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/6a16fe9e33a340b691fe70e10778afb8/log/controller/logs/apache/ipxe_access_log.txt | 14:53 |
| iurygregory | we should be able to see more things no? | 14:54 |
| dtantsur | iurygregory: that's for ipxe only | 14:54 |
| dtantsur | I think inspector itself is run via uwsgi | 14:54 |
| rpittau | yeah | 14:54 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/12eacd0dba41423b8a6a343734517b02/log/controller/logs/apache/ipxe_access_log.txt | 14:54 |
| iurygregory | this one is when the job is working | 14:54 |
| *** dking has joined #openstack-ironic | 14:55 | |
| iurygregory | finally got +1 from CI in the 1st patch of the chain https://review.opendev.org/#/c/687910/ | 14:59 |
| patchbot | patch 687910 - ironic - Add `network_data` field to ironic node object - 31 patch sets | 14:59 |
| iurygregory | \o/ | 14:59 |
| *** jtomasek has quit IRC | 15:10 | |
| JayF | TheJulia: jroll: There's another SPUC? When? | 15:16 |
| dtantsur | JayF: tomorrow 3pm UTC | 15:16 |
| JayF | so in 23 hours and 45 minutes, yeah? | 15:17 |
| dtantsur | I believe so | 15:17 |
| JayF | oh crap, I have the same meeting as jroll, damn | 15:17 |
| JayF | someone download snap camera and wear a virtual funny hat for me | 15:17 |
| dtantsur | ++ iurygregory is a SPUC Manager, he has to | 15:18 |
| *** mkowalski[b] has left #openstack-ironic | 15:23 | |
| *** livelace has quit IRC | 15:26 | |
| jroll | lol | 15:27 |
| *** gyee has joined #openstack-ironic | 15:32 | |
| *** livelace has joined #openstack-ironic | 15:34 | |
| *** priteau has joined #openstack-ironic | 15:45 | |
| dtantsur | rpittau: https://review.opendev.org/#/c/725357/ may (just may) be related to the inspector sadness | 15:55 |
| patchbot | patch 725357 - ironic-inspector - Monkey patch original current_thread _active - 2 patch sets | 15:55 |
| rpittau | ah | 15:56 |
| rpittau | we need grenade back though first | 15:57 |
| dtantsur | yeah | 15:57 |
| rpittau | or we exclude grenade, merge it, reenable grenade? (piracy) | 15:58 |
| *** lucasagomes has quit IRC | 16:08 | |
| *** diurnalist has joined #openstack-ironic | 16:12 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic master: Limit the number of ipmitool retries https://review.opendev.org/725954 | 16:16 |
| iurygregory | dtantsur, you think https://review.opendev.org/725357 would help? | 16:16 |
| patchbot | patch 725357 - ironic-inspector - Monkey patch original current_thread _active - 2 patch sets | 16:16 |
| iurygregory | we can deactivate grenade, merge it and I can re-enable grenade in my patch | 16:17 |
| TheJulia | rpittau: I'll +2 it if there is somehow a pirate flag attached | 16:17 |
| TheJulia | ascii pirate flag?!? | 16:18 |
| rpittau | arrrrrr | 16:18 |
| dtantsur | we may need to disable both jobs and start re-enabling them as they get fixed.. | 16:18 |
| rpittau | piracy it is then | 16:18 |
| iurygregory | LOL | 16:18 |
| TheJulia | it happens, sadly | 16:19 |
| iurygregory | at least looking at the zuul results in the patch the non-standalone had 0 failures | 16:19 |
| iurygregory | so maybe it helps and we don't need to put non-voting for it | 16:20 |
| dtantsur | I think what will help is that we should stop failing inspection if we cannot verify its status | 16:21 |
| dtantsur | at least not immediately | 16:21 |
| dtantsur | I'm planning on a patch tomorrow but wouldn't mind if somebody did it today | 16:22 |
| iurygregory | that makes sense to me | 16:22 |
| iurygregory | I would help but I need to drop to do the dinner and tomorrow is holiday .-. | 16:23 |
| iurygregory | rpittau, are you pushing the patch to put the jobs non-voting ? | 16:23 |
| rpittau | iurygregory: yeah | 16:24 |
| iurygregory | rpittau, ok o/ | 16:24 |
| dtantsur | no worries, I'll take care of it tomorrow | 16:24 |
| dtantsur | what's so wrong with the CI today? post failures, timeouts, transient failures | 16:27 |
| dtantsur | I guess it's a sign for me to call it a day | 16:28 |
| *** uzumaki has joined #openstack-ironic | 16:28 | |
| *** dtantsur is now known as dtantsur|afk | 16:28 | |
| dtantsur|afk | take care | 16:28 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Temporary disbling grenade and non-standalone-tempest https://review.opendev.org/726198 | 16:29 |
| rpittau | dtantsur|afk, iurygregory, TheJulia ^ | 16:29 |
| dtantsur|afk | +2 and leaving for real | 16:29 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Revert disable grenade and non-standalone-tempest https://review.opendev.org/726200 | 16:31 |
| iurygregory | I was going to say it should have voting:false | 16:31 |
| iurygregory | .-. | 16:31 |
| iurygregory | because at least we would run with non-voting and we would see results | 16:31 |
| iurygregory | rpittau, ^ | 16:31 |
| rpittau | iurygregory: yeah, I thought the same while pressing enter, too quick | 16:32 |
| rpittau | I'll change that now | 16:32 |
| iurygregory | rpittau, tks! | 16:32 |
| iurygregory | I will +2 and will keep and eye to approve | 16:32 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Temporary disbling grenade and non-standalone-tempest https://review.opendev.org/726198 | 16:33 |
| iurygregory | awesome! | 16:34 |
| rpittau | I put the revert on top, we should be fine | 16:34 |
| rpittau | bye everyone! o/ | 16:34 |
| *** rpittau is now known as rpittau|afk | 16:34 | |
| *** diurnalist has quit IRC | 16:35 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in node methods https://review.opendev.org/725919 | 16:35 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id in port and volume https://review.opendev.org/725930 | 16:36 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Allow to pass global request id for remaining objects https://review.opendev.org/725941 | 16:36 |
| vdrok | hey folks, if someone still willing to review some easy changes ^^ :) chain starts at https://review.opendev.org/725661. it does not look pretty, but conforms to what we had already so that backwards compatibilty is not broken. maybe it's worth to allow passing arbitrary headers in each method, not sure, so opinions welcome | 16:38 |
| patchbot | patch 725661 - python-ironicclient - Add global_request_id into the base client methods - 3 patch sets | 16:38 |
| *** sshnaidm is now known as sshnaidm|afk | 16:46 | |
| *** uzumaki has quit IRC | 16:51 | |
| *** derekh has quit IRC | 16:53 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Change ownership of tce info dir https://review.opendev.org/726119 | 16:55 |
| arne_wiebalck | bye everyone o/ | 17:08 |
| *** ociuhandu has quit IRC | 17:19 | |
| *** gmann is now known as gmann_afk | 17:25 | |
| *** priteau has quit IRC | 17:25 | |
| *** alexmcleod has quit IRC | 17:49 | |
| *** ociuhandu has joined #openstack-ironic | 17:57 | |
| *** ociuhandu has quit IRC | 18:03 | |
| *** ociuhandu has joined #openstack-ironic | 18:11 | |
| *** ociuhandu has quit IRC | 18:27 | |
| *** ociuhandu has joined #openstack-ironic | 18:32 | |
| *** uzumaki has joined #openstack-ironic | 18:32 | |
| *** ociuhandu has quit IRC | 18:39 | |
| *** ociuhandu has joined #openstack-ironic | 18:56 | |
| *** gmann_afk is now known as gmann | 19:06 | |
| *** ociuhandu has quit IRC | 19:07 | |
| *** ociuhandu has joined #openstack-ironic | 19:10 | |
| *** ociuhandu has quit IRC | 19:15 | |
| *** yaawang has quit IRC | 19:16 | |
| *** k_mouza has quit IRC | 19:16 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Silence debug messages from oslo_messaging https://review.opendev.org/725789 | 19:27 |
| openstackgerrit | Merged openstack/ironic-inspector master: Fix grenade job https://review.opendev.org/725514 | 19:28 |
| openstackgerrit | Merged openstack/ironic-inspector master: Temporary disbling grenade and non-standalone-tempest https://review.opendev.org/726198 | 19:28 |
| iurygregory | wtf CI O.o | 19:37 |
| iurygregory | how you merged Fix grenade before we disabled? O.o | 19:37 |
| iurygregory | *magic* | 19:37 |
| *** livelace has quit IRC | 19:48 | |
| *** dsneddon has joined #openstack-ironic | 19:56 | |
| clarkb | iurygregory: they are enqueued in order, it is determinstic | 19:58 |
| clarkb | at 10:28AM clarkb local time https://review.opendev.org/725514 was enqueued to the gate. https://review.opendev.org/726198 was enqueued at 11:20AM | 19:59 |
| patchbot | patch 725514 - ironic-inspector - Fix grenade job (MERGED) - 4 patch sets | 19:59 |
| patchbot | patch 726198 - ironic-inspector - Temporary disbling grenade and non-standalone-tempest (MERGED) - 2 patch sets | 19:59 |
| clarkb | this meant the fix was ahead of the disable change in the queue | 19:59 |
| *** ociuhandu has joined #openstack-ironic | 19:59 | |
| *** dsneddon has quit IRC | 20:00 | |
| *** k_mouza has joined #openstack-ironic | 20:05 | |
| *** yaawang has joined #openstack-ironic | 20:32 | |
| *** livelace has joined #openstack-ironic | 20:39 | |
| *** gokhani has joined #openstack-ironic | 20:42 | |
| *** stevebaker_ has quit IRC | 20:59 | |
| *** stevebaker has joined #openstack-ironic | 20:59 | |
| *** gokhani has quit IRC | 20:59 | |
| iurygregory | clarkb, gotcha =) | 21:08 |
| *** ociuhandu has quit IRC | 21:13 | |
| *** ociuhandu has joined #openstack-ironic | 21:14 | |
| *** livelace has quit IRC | 21:14 | |
| *** ociuhandu has quit IRC | 21:19 | |
| *** ociuhandu has joined #openstack-ironic | 21:20 | |
| *** ociuhandu has quit IRC | 21:36 | |
| *** ociuhandu has joined #openstack-ironic | 21:36 | |
| *** ociuhandu has quit IRC | 21:42 | |
| TheJulia | stevebaker: you around? | 22:03 |
| openstackgerrit | Merged openstack/ironic-inspector master: Monkey patch original current_thread _active https://review.opendev.org/725357 | 22:04 |
| stevebaker | TheJulia: hey! | 22:16 |
| TheJulia | so I was looking at addding basic auth to ironic's api | 22:26 |
| *** rh-jelabarre has quit IRC | 22:27 | |
| TheJulia | and kind of wondering if you had some cycles to see what it might take to add native basic auth to ironic's api. I can see pluses to that, and I apache is just easier, but yeah | 22:28 |
| *** hoonetorg has quit IRC | 22:34 | |
| stevebaker | TheJulia: oh yeah that sounds fun. What would the credential store be? | 22:40 |
| *** hoonetorg has joined #openstack-ironic | 22:46 | |
| *** tkajinam has joined #openstack-ironic | 22:49 | |
| TheJulia | stevebaker: I really like the idea of just being able to read from a file using the same format as apache/nginx | 22:53 |
| NobodyCam | that would be kinda cool | 22:54 |
| stevebaker | TheJulia: yep cool, I'm on it. | 22:54 |
| TheJulia | stevebaker: tagged you on a downstream jira, I don't remember if I've created an epic yet, but I think the key is just first step, think through it and get a basic idea together | 22:55 |
| TheJulia | also... that makes it much easier to navigate our security differences around the ramdisk endpoint | 22:55 |
| *** kbaegis has joined #openstack-ironic | 22:56 | |
| stevebaker | TheJulia: ramdisk endpoint? is this a thing that exists? | 23:00 |
| TheJulia | yeah, /v1/ramdisk | 23:01 |
| TheJulia | so /v1/ramdisk/lookup and /v1/ramdisk/heartbeat | 23:01 |
| stevebaker | TheJulia: ah, spec says /v1/lookup, /v1/heartbeat https://docs.openstack.org/api-ref/baremetal/#utility | 23:03 |
| TheJulia | oh, it is ramdisk.py | 23:11 |
| TheJulia | so yeah, your right about the actual endpoints | 23:14 |
| TheJulia | in the rest api, just brains | 23:14 |
| *** spotz has quit IRC | 23:16 | |
| *** k_mouza has quit IRC | 23:18 | |
| *** dsneddon has joined #openstack-ironic | 23:18 | |
| stevebaker | TheJulia: cool, so the ramdisk paths are public currently, but they would be covered by basic auth? | 23:23 |
| TheJulia | stevebaker: no, they can't be | 23:27 |
| TheJulia | because the ramdisk still needs to be able to call home | 23:27 |
| stevebaker | TheJulia: ok, I'll do some research | 23:35 |
| *** kbaegis has quit IRC | 23:41 | |
| *** rloo has quit IRC | 23:51 | |
| *** gyee has quit IRC | 23:54 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!