*** gyee has quit IRC | 00:21 | |
*** NobodyCam_ has joined #openstack-ironic | 00:52 | |
*** NobodyCam has quit IRC | 00:52 | |
*** NobodyCam_ is now known as NobodyCam | 00:52 | |
*** Goneri has quit IRC | 00:59 | |
*** mugsie has quit IRC | 01:00 | |
*** mugsie has joined #openstack-ironic | 01:04 | |
*** xinliang has joined #openstack-ironic | 01:13 | |
xinliang | Good morning all. | 01:16 |
---|---|---|
*** JamesBenson has joined #openstack-ironic | 01:26 | |
*** JamesBenson has quit IRC | 01:30 | |
*** Qianbiao has joined #openstack-ironic | 01:44 | |
*** JamesBenson has joined #openstack-ironic | 02:06 | |
*** JamesBenson has quit IRC | 02:10 | |
*** JamesBenson has joined #openstack-ironic | 02:21 | |
*** JamesBenson has quit IRC | 02:25 | |
*** xinliang has quit IRC | 02:27 | |
*** Qianbiao has quit IRC | 02:35 | |
*** rcernin has quit IRC | 02:38 | |
*** Qianbiao has joined #openstack-ironic | 02:50 | |
*** xinliang has joined #openstack-ironic | 03:02 | |
*** ayoung has quit IRC | 03:03 | |
*** rcernin has joined #openstack-ironic | 03:06 | |
*** rcernin has quit IRC | 03:26 | |
openstackgerrit | likui proposed openstack/ironic-python-agent master: Remove the unused coding style modules https://review.opendev.org/759792 | 03:38 |
*** rcernin has joined #openstack-ironic | 03:47 | |
*** rcernin has quit IRC | 03:47 | |
*** rcernin has joined #openstack-ironic | 03:47 | |
openstackgerrit | Merged openstack/ironic master: Python 3.9: base64.{en,de}codestring function is removed https://review.opendev.org/758663 | 03:59 |
*** zaneb has quit IRC | 04:14 | |
*** zaneb has joined #openstack-ironic | 04:15 | |
*** xinliang has quit IRC | 04:16 | |
*** tzumainn has quit IRC | 04:41 | |
openstackgerrit | Kafilat Adeleke proposed openstack/sushy master: Adds sushy library overview https://review.opendev.org/757907 | 06:27 |
*** abdysn has joined #openstack-ironic | 06:51 | |
arne_wiebalck | Good morning, ironic! | 06:58 |
Qianbiao | Morning arne_wiebalck | 07:08 |
arne_wiebalck | Hey Qianbiao o/ | 07:08 |
Qianbiao | Hey o/ | 07:10 |
*** fgofurov has joined #openstack-ironic | 07:13 | |
*** dtantsur|afk is now known as dtantsur | 07:20 | |
dtantsur | morning ironic | 07:20 |
Qianbiao | morning dtantsur | 07:26 |
fgofurov | Good morning everyone! dtantsur, we have +2 for adoption patch, and I see there are some checks failed, is there anything needs to be done to get that patch merged? | 07:40 |
dtantsur | fgofurov: which patch do you have in mind? | 07:40 |
fgofurov | sorry, this one: https://review.opendev.org/#/c/759245/ | 07:41 |
patchbot | patch 759245 - ironic - Add timeout to image operations in the direct deploy - 1 patch set | 07:41 |
dtantsur | I'll check once I finish something here | 07:41 |
fgofurov | Sure, thanks dtantsur | 07:41 |
*** rpittau|afk is now known as rpittau | 07:42 | |
rpittau | good morning ironic! o/ | 07:42 |
fgofurov | morning rpittau | 07:42 |
rpittau | hey fgofurov :) | 07:42 |
rpittau | dtantsur: I'm afraid I was right about pymemcache -> https://review.opendev.org/759711 | 07:44 |
patchbot | patch 759711 - ironic-inspector - [DNM] test older version of pymemcache - 2 patch sets | 07:44 |
*** anuradha1904 has joined #openstack-ironic | 07:45 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: increase cleaning timeout and tie it to PXE boot timeout https://review.opendev.org/759824 | 07:46 |
dtantsur | rpittau: ugh :( good catch! | 07:48 |
rpittau | it seems the problem is on tooz side | 07:48 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/victoria: Fix database migrations and disable the non-standalone job https://review.opendev.org/759825 | 07:49 |
dtantsur | fgofurov: https://review.opendev.org/#/c/759245/ just needs a 2nd review from a core team member. rpittau maybe? | 07:51 |
patchbot | patch 759245 - ironic - Add timeout to image operations in the direct deploy - 1 patch set | 07:51 |
rpittau | yep, I was looking at that | 07:51 |
fgofurov | thanks rpittau | 07:55 |
rpittau | np | 07:55 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ussuri: Fix database migrations and disable the non-standalone job https://review.opendev.org/759826 | 07:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/train: Fix database migrations and disable the non-standalone job https://review.opendev.org/759827 | 07:58 |
rpittau | TheJulia: I will include the alloc memory increase in my ipa change :/ | 07:58 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 08:04 |
*** sdanni has quit IRC | 08:10 | |
iurygregory | good morning Ironic o/ | 08:21 |
rpittau | hey iurygregory :) | 08:22 |
*** rcernin has quit IRC | 08:23 | |
dtantsur | iurygregory: hey, was bureaucracy defeated yesterday? :) | 08:34 |
*** tosky has joined #openstack-ironic | 08:36 | |
iurygregory | dtantsur, it went well, but it took like 2hrs.. | 08:38 |
iurygregory | to go to the office and come back | 08:38 |
iurygregory | at least on the end of november I can travel again =) | 08:39 |
dtantsur | heh, good. I hope my appointment in December is not cancelled because of covid | 08:39 |
rpittau | yay for bureaucracy, my passport expired and I don't know when I will be able to have a new one again \o/ | 08:39 |
iurygregory | wow | 08:41 |
iurygregory | I'm planning in travel to Brazil and stay some time, covid will be more crazy during winter here in Europe.. (I think) | 08:41 |
rpittau | iurygregory: that's a great strategy :) | 08:42 |
iurygregory | rpittau, yeah and at least I will be able to see my family =) | 08:42 |
dtantsur | rpittau: I received my refreshed passport 1 (one) day before the initial lockdown | 08:44 |
rpittau | heh, same for my id card, but for the passport I needed a new separate appointment and it's currently impossible to get one | 08:45 |
Qianbiao | thanks for review dtantsur | 08:47 |
Qianbiao | and I need one more workflow for https://review.opendev.org/#/c/752006/ and https://review.opendev.org/#/c/757606/ | 08:48 |
patchbot | patch 752006 - ironic (stable/ussuri) - opt: Enhance old stable branches to use latest pyt... - 8 patch sets | 08:48 |
patchbot | patch 757606 - ironic (stable/train) - opt: Enhance old stable branches to use latest pyt... - 2 patch sets | 08:48 |
Qianbiao | best strategy is stay home before vaccine is made up. be careful folks. | 08:49 |
Qianbiao | ty rpittau | 08:50 |
rpittau | yw :) | 08:50 |
iurygregory | dtantsur, do we still need functional tests for sushy-tools? https://storyboard.openstack.org/#!/story/1751133 | 08:53 |
dtantsur | iurygregory: I don't really think so | 08:57 |
iurygregory | we already run in the CI so I think we are fine ... | 08:58 |
dtantsur | yeah | 09:00 |
*** xinliang has joined #openstack-ironic | 09:03 | |
*** lucasagomes has joined #openstack-ironic | 09:11 | |
*** ociuhandu has joined #openstack-ironic | 09:19 | |
openstackgerrit | Merged openstack/bifrost master: Stop using deprecated iPXE parameters https://review.opendev.org/759458 | 09:20 |
openstackgerrit | Kafilat Adeleke proposed openstack/sushy master: Adds sushy library features https://review.opendev.org/759834 | 09:25 |
openstackgerrit | Merged openstack/bifrost stable/victoria: Force local connection for localhost https://review.opendev.org/759188 | 09:32 |
openstackgerrit | Verification of a change to openstack/python-ironic-inspector-client failed: Set safe version of hacking https://review.opendev.org/758764 | 09:50 |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Set safe version of hacking https://review.opendev.org/758764 | 09:54 |
*** uzumaki has joined #openstack-ironic | 10:00 | |
*** k_mouza has joined #openstack-ironic | 10:46 | |
fgofurov | Hello ironic! We are seeing some unexpected behavior in our testing environment, that makes image writing unreliable. The problem is that, even when IPA is unable to write the disk image to the physical disk, it reports success to Ironic. | 10:53 |
openstackgerrit | Xinliang Liu proposed openstack/ironic-specs master: Add PXE boot from iSCSI volume scenario https://review.opendev.org/759839 | 10:54 |
dtantsur | fgofurov: make sure you have https://opendev.org/openstack/ironic-python-agent/commit/420ebc0d73ccfa7248e93881c58a0e45fda0e718 | 10:54 |
fgofurov | dtantsur thanks :) so does it mean you do not have to see the logs? I was trying to put them in http://paste.openstack.org/ | 10:56 |
*** k_mouza has quit IRC | 10:56 | |
dtantsur | fgofurov: if this patch fixes your problem - no :) if it doesn't please let us know. | 10:57 |
fgofurov | and opened a story: https://storyboard.openstack.org/#!/story/2008287 | 10:57 |
*** k_mouza has joined #openstack-ironic | 10:57 | |
fgofurov | wow, incredible, thanks a lot once again, I will do come back if problem still exists | 10:58 |
*** k_mouza has quit IRC | 11:02 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Add timeout to image operations in the direct deploy https://review.opendev.org/759245 | 11:05 |
*** xinliang has quit IRC | 11:05 | |
dtantsur | sigh | 11:06 |
*** k_mouza has joined #openstack-ironic | 11:10 | |
fgofurov | dtantsur hmm, zuul checks have failed on other patch :) thanks for recheck | 11:10 |
dtantsur | yeah, the multinode job is not the most stable in the word.. | 11:10 |
fgofurov | :) | 11:11 |
*** dsneddon has quit IRC | 11:16 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: increase cleaning timeout and tie it to PXE boot timeout https://review.opendev.org/759824 | 11:22 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Move the multinode grenade job to the experimental pipeline https://review.opendev.org/759847 | 11:33 |
dtantsur | iurygregory: FYI ^^^ | 11:33 |
dtantsur | TheJulia: ^^ | 11:34 |
iurygregory | dtantsur, ++ makes sense to me =) | 11:34 |
rpittau | CI today is particularly upsetting | 11:44 |
*** dsneddon has joined #openstack-ironic | 11:45 | |
*** fgofurov has quit IRC | 11:48 | |
*** fgofurov has joined #openstack-ironic | 11:52 | |
*** rh-jelabarre has joined #openstack-ironic | 11:53 | |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add support to expose oem_vendors from resource links https://review.opendev.org/755323 | 11:55 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add support to expose oem_vendors from resource links https://review.opendev.org/755323 | 11:57 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add support to expose oem_vendors from resource links https://review.opendev.org/755323 | 11:59 |
*** dsneddon has quit IRC | 12:08 | |
*** dsneddon has joined #openstack-ironic | 12:10 | |
*** dsneddon has quit IRC | 12:15 | |
*** dsneddon has joined #openstack-ironic | 12:16 | |
*** abdysn has quit IRC | 12:21 | |
*** dsneddon has quit IRC | 12:21 | |
*** dsneddon has joined #openstack-ironic | 12:22 | |
*** dtantsur is now known as dtantsur|brb | 12:34 | |
*** Qianbiao has quit IRC | 12:41 | |
*** Goneri has joined #openstack-ironic | 12:47 | |
*** Qianbiao has joined #openstack-ironic | 12:57 | |
*** rloo has joined #openstack-ironic | 13:08 | |
*** dsneddon has quit IRC | 13:09 | |
*** thiagop has joined #openstack-ironic | 13:10 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Tear down raid sets https://review.opendev.org/757141 | 13:11 |
*** dsneddon has joined #openstack-ironic | 13:15 | |
*** paras333_ has joined #openstack-ironic | 13:19 | |
*** akiselev has joined #openstack-ironic | 13:37 | |
*** Qianbiao has quit IRC | 13:38 | |
*** Qianbiao has joined #openstack-ironic | 13:39 | |
*** tzumainn has joined #openstack-ironic | 13:41 | |
*** akiselev has quit IRC | 13:56 | |
*** xinliang has joined #openstack-ironic | 13:59 | |
TheJulia | PTG starting | 14:02 |
janders | \o/ | 14:02 |
openstackgerrit | Merged openstack/ironic master: Fixes empty physical_network is not guarded https://review.opendev.org/759567 | 14:02 |
openstackgerrit | Verification of a change to openstack/ironic failed: Remove the support to create port in API https://review.opendev.org/759546 | 14:02 |
openstackgerrit | Verification of a change to openstack/ironic failed: Add node name to ironic-conductor ramdisk log filename https://review.opendev.org/759549 | 14:02 |
dtantsur|brb | janders, bfournie, we're in the austin room | 14:03 |
dtantsur|brb | (I saw you in kilo which is wrong) | 14:03 |
*** dtantsur|brb is now known as dtantsur | 14:03 | |
*** Qianbiao has quit IRC | 14:03 | |
rloo | anyone that wants to join ironic PTG: https://www.openstack.org/ptg/rooms/austin | 14:03 |
TheJulia | #ironic url https://www.openstack.org/ptg/rooms/austin | 14:04 |
*** Qianbiao has joined #openstack-ironic | 14:04 | |
janders | oops... split brain condition? | 14:04 |
janders | (I was in another room) | 14:04 |
*** sshnaidm|rover has quit IRC | 14:05 | |
dtantsur | janders: incorrect link from yesterday | 14:05 |
*** sshnaidm has joined #openstack-ironic | 14:05 | |
*** sshnaidm is now known as sshnaidm|rover | 14:06 | |
*** stendulker has joined #openstack-ironic | 14:08 | |
openstackgerrit | Xinliang Liu proposed openstack/ironic-specs master: Add PXE boot from iSCSI volume scenario https://review.opendev.org/759839 | 14:10 |
openstackgerrit | Anuradha proposed openstack/ironic master: Changes to add exception in "default_interface()" function https://review.opendev.org/756682 | 14:16 |
openstackgerrit | Verification of a change to openstack/ironic failed: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/752006 | 14:23 |
openstackgerrit | Zane Bitter proposed openstack/ironic master: json-rpc: surround IPv6 address with [] in conductor URL https://review.opendev.org/759877 | 14:26 |
iurygregory | zaneb, good catch ^ =) | 14:30 |
zaneb | I hate that url syntax | 14:31 |
iurygregory | yeah =( | 14:35 |
dtantsur | zaneb: good catch, did you run pep8 on the patch? I'm not sure about it. | 14:38 |
zaneb | I thought I had | 14:38 |
* zaneb checks again | 14:38 | |
dtantsur | I think it shouldn't pass :) but maybe it does | 14:39 |
zaneb | nope, you're right | 14:40 |
*** dsneddon has quit IRC | 14:40 | |
zaneb | my editor must have decided to mess with that line even though I copied and pasted it from a block that was indented correctly, sigh | 14:40 |
openstackgerrit | Zane Bitter proposed openstack/ironic master: json-rpc: surround IPv6 address with [] in conductor URL https://review.opendev.org/759877 | 14:41 |
*** dsneddon has joined #openstack-ironic | 14:41 | |
openstackgerrit | Verification of a change to openstack/ironic failed: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/757606 | 14:42 |
iurygregory | pep8 =D | 14:42 |
*** dsneddon has quit IRC | 14:47 | |
*** dsneddon has joined #openstack-ironic | 14:51 | |
*** dsneddon has quit IRC | 14:56 | |
xinliang | dtantsur, TheJulia Hi, when you have time please take a look at this RFE: https://storyboard.openstack.org/#!/story/2008023 | 15:01 |
xinliang | want to know your opinions about this RFE :) | 15:02 |
JayF | I think tinyipa jobs are broken in IPA? https://zuul.opendev.org/t/openstack/build/e647f2fee8ea4aab8229297f717c44c1 | 15:13 |
rpittau | JayF: the fix is https://review.opendev.org/757812 | 15:14 |
patchbot | patch 757812 - ironic-python-agent - Run dib ipa src jobs on ubuntu focal - 7 patch sets | 15:14 |
JayF | rpittau: thanks! | 15:15 |
rpittau | np :) | 15:21 |
openstackgerrit | Zane Bitter proposed openstack/ironic master: json-rpc: surround IPv6 address with [] in conductor URL https://review.opendev.org/759877 | 15:22 |
*** pmannidi has quit IRC | 15:26 | |
*** pmannidi has joined #openstack-ironic | 15:26 | |
*** pmannidi has quit IRC | 15:32 | |
*** gyee has joined #openstack-ironic | 15:34 | |
*** Qianbiao has quit IRC | 15:37 | |
openstackgerrit | Merged openstack/ironic-inspector stable/victoria: Fix database migrations and disable the non-standalone job https://review.opendev.org/759825 | 15:39 |
*** happy-contrib-43 has joined #openstack-ironic | 15:40 | |
*** pmannidi has joined #openstack-ironic | 15:40 | |
happy-contrib-43 | Good morning, I'm adding a means to load flask blueprints, how would you guys like it tracked, what kind of testing do you want done, and what unit tests added? | 15:42 |
*** early has quit IRC | 16:00 | |
*** early has joined #openstack-ironic | 16:01 | |
*** uzumaki has quit IRC | 16:01 | |
*** dsneddon has joined #openstack-ironic | 16:06 | |
*** xinliang has quit IRC | 16:07 | |
*** tosky_ has joined #openstack-ironic | 16:07 | |
*** tosky has quit IRC | 16:08 | |
*** JamesBenson has joined #openstack-ironic | 16:12 | |
dtantsur | happy-contrib-43: hi, I don't really get the context. what exactly are you trying to do? | 16:19 |
*** stendulker has quit IRC | 16:34 | |
TheJulia | I'd like to propose we make standalone jobs non-voting until we get tempest and ironic conductor fixes in | 16:44 |
TheJulia | otherwise we're going to have an endless chicken/egg scenario between tempest and ironic | 16:44 |
TheJulia | and I guess ironic-python-agent | 16:45 |
*** dsneddon has quit IRC | 16:45 | |
rpittau | I guess we can't avoid it | 16:46 |
*** dsneddon has joined #openstack-ironic | 16:46 | |
TheJulia | I have fond an issue where we end up failing a deploy in the conductor when we try to do the standalone cleaning | 16:46 |
TheJulia | I'm working on a patch now, it will need testing and I need to double check a few code paths since I am adding a new exception | 16:47 |
TheJulia | but that should at least handle half the conundrum. The standalone raid cleaning fix, once it finally passes could likely just work, but it hit the same issue with the conductor mid-stream deployment on the last job | 16:47 |
JayF | +2 | 16:48 |
*** lucasagomes has quit IRC | 16:48 | |
happy-contrib-43 | dtantsur: sushy-emulator did not support sessions which we needed to work with rvmc. We added a fake session route and added it as a flask blueprint to avoid changes to the sushy-emulator directly. We expanded checking for our singular blueprint to adding a check to read all blueprints in a directory for sushy-emulator. | 16:50 |
*** dsneddon has quit IRC | 16:53 | |
*** dsneddon has joined #openstack-ironic | 16:55 | |
*** tosky_ is now known as tosky | 16:56 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Handle agent still doing the prior command https://review.opendev.org/759906 | 16:58 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Handle agent still doing the prior command https://review.opendev.org/759906 | 16:58 |
TheJulia | consider ^^^ a wip, needs tests, but I put all the info in the commit message to hopefully grok the issue and why | 16:59 |
openstackgerrit | Merged openstack/ironic-inspector master: Run bifrost integration job on focal https://review.opendev.org/758199 | 16:59 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 16:59 |
*** dsneddon has quit IRC | 17:01 | |
*** dsneddon has joined #openstack-ironic | 17:02 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Mark standalone job non-voting/remove from gate https://review.opendev.org/759909 | 17:07 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: The standalone job at present has a high chance of failure due to two separate things occuring: https://review.opendev.org/759910 | 17:09 |
TheJulia | changes for IPA + Ironic to just get them "un-wedged" | 17:09 |
TheJulia | The noted tempest change should be fine and pass once the ironic change merges, at which point we should be able to press the revert button | 17:09 |
*** ricolin has quit IRC | 17:10 | |
*** aedc_ has joined #openstack-ironic | 17:10 | |
*** aedc has quit IRC | 17:11 | |
iurygregory | TheJulia, ack | 17:15 |
iurygregory | if no other core is around I will put +W after zuul gives +1 | 17:16 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Mark standalone job non-voting/remove from gate https://review.opendev.org/759910 | 17:16 |
TheJulia | iurygregory: fixed commit message of ^^^ | 17:16 |
dtantsur | happy-contrib-43: why not add support for sessions directly to sushy-emulator? | 17:17 |
dtantsur | it sounds useful to me | 17:17 |
rloo | iurygregory: you mean https://review.opendev.org/#/c/759910/? | 17:17 |
patchbot | patch 759910 - ironic - Mark standalone job non-voting/remove from gate - 2 patch sets | 17:17 |
iurygregory | rloo, yeah =) | 17:17 |
rloo | iurygregory: done. if CI is happy with it though :) | 17:18 |
iurygregory | rloo, yup =) | 17:18 |
TheJulia | \o/ | 17:24 |
*** k_mouza has quit IRC | 17:25 | |
rpittau | of course what we need mostly failed... | 17:29 |
TheJulia | the ipa patch? | 17:30 |
rpittau | yeah | 17:30 |
TheJulia | I looked at it earlier, it was just standalone then | 17:30 |
rpittau | yep | 17:30 |
TheJulia | yup, standalone | 17:30 |
TheJulia | and still configured with software raid | 17:31 |
rpittau | I will rebase on top of yours | 17:31 |
TheJulia | so my ironic-tempest-plugin change should fix that | 17:31 |
rpittau | mmmm | 17:31 |
TheJulia | we're leaving the node dirty | 17:32 |
TheJulia | but this is why I put up changes to mark standalone non-voting for now | 17:32 |
rpittau | yeah, ok I see it, I will rebase on that until we have the final fix | 17:32 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 17:33 |
rpittau | aaaand I wish a good rest of the day to everyone, stay safe o/ | 17:34 |
*** rpittau is now known as rpittau|afk | 17:34 | |
*** ociuhandu_ has joined #openstack-ironic | 17:35 | |
happy-contrib-43 | dtantsur: our current solution is more of a stub to fake a session. While that is sufficient to address our needs it isn't a complete solution. I believe proper session support would involve a database and user authentication. | 17:37 |
*** ociuhandu has quit IRC | 17:38 | |
*** ociuhandu_ has quit IRC | 17:40 | |
*** dsneddon has quit IRC | 17:42 | |
*** dsneddon has joined #openstack-ironic | 17:43 | |
*** JamesBenson has quit IRC | 17:46 | |
*** uzumaki has joined #openstack-ironic | 17:47 | |
*** JamesBenson has joined #openstack-ironic | 18:03 | |
*** dsneddon has quit IRC | 18:04 | |
*** dsneddon has joined #openstack-ironic | 18:06 | |
*** dtantsur is now known as dtantsur|afk | 18:13 | |
*** dsneddon has quit IRC | 18:16 | |
*** dsneddon has joined #openstack-ironic | 18:16 | |
*** dsneddon has quit IRC | 18:29 | |
*** dsneddon has joined #openstack-ironic | 18:39 | |
*** akiselev has joined #openstack-ironic | 18:40 | |
*** dsneddon has quit IRC | 18:44 | |
*** dsneddon has joined #openstack-ironic | 18:46 | |
*** dsneddon has quit IRC | 18:51 | |
*** ociuhandu has joined #openstack-ironic | 19:13 | |
TheJulia | so far so good with my agent is busy fix, almost done with unit testing since basically we had the same logic elsewhere *facepalm* | 19:26 |
*** dsneddon has joined #openstack-ironic | 19:38 | |
*** dsneddon has quit IRC | 19:44 | |
openstackgerrit | Leo McGann proposed openstack/ironic-specs master: Add security interface spec https://review.opendev.org/576718 | 20:02 |
*** melwitt has joined #openstack-ironic | 20:07 | |
*** ociuhandu has quit IRC | 20:15 | |
*** ociuhandu has joined #openstack-ironic | 20:16 | |
*** rpittau|afk is now known as rpittau | 20:18 | |
rpittau | TheJulia: I just realized, I can't rebase on top of yours, your won't pass without mine :/ | 20:18 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 20:19 |
rpittau | I rebased again on top of master, hoping will pass | 20:20 |
rpittau | another solution would be disabling the 2 jobs failing in your patch and re-enabling them in mine | 20:20 |
rpittau | ^ TheJulia | 20:20 |
TheJulia | rpittau: ugh | 20:20 |
TheJulia | true | 20:20 |
TheJulia | whatever moves us forward right now | 20:20 |
rpittau | it's sad but I think we should disable ipa-tempest-bios-ipmi-direct-src and ipa-tempest-bios-ipmi-iscsi-src in yours, I can rebase again on top of that re-enabling them and we should be fine, just need to re-enable standalone afterwards | 20:22 |
TheJulia | AHHH I see what happened | 20:23 |
rpittau | :/ | 20:24 |
TheJulia | agent go tchanged to emit a 409, we had some code previously to handle a 400, and it further broke | 20:24 |
TheJulia | rpittau: if you want to propose that then go ahead and I'll approve it since it functionally changes the patch | 20:24 |
rpittau | TheJulia: ok, I'll make the changes now | 20:25 |
TheJulia | as long as we have a clean revertable change I'm happy, but I'm also in whatever path works mode | 20:25 |
rpittau | TheJulia: for simplicity I will work directly on your patch, if that's ok, since standalone it's already disabled there | 20:26 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Mark standalone job non-voting/remove from gate https://review.opendev.org/759909 | 20:29 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 20:30 |
TheJulia | wfm | 20:30 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 20:31 |
rpittau | ok done | 20:32 |
TheJulia | k, I'm going to dig into the logs to make sure we don't have anything else obvious in the agent communication and that should get ironic happier overall. The majority of jobs passed so a relatively good sign. | 20:33 |
TheJulia | err, looks like multinode in general failed for unrelated reasons | 20:39 |
TheJulia | and grenade looks like it kind of just literally hit a slow node :( | 20:42 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Handle agent still doing the prior command https://review.opendev.org/759906 | 20:48 |
TheJulia | Okay, tagged some ironic-cores on ^^ | 20:53 |
openstackgerrit | Verification of a change to openstack/ironic failed: Move the multinode grenade job to the experimental pipeline https://review.opendev.org/759847 | 20:57 |
*** gouthamr has quit IRC | 20:58 | |
*** logan- has quit IRC | 20:58 | |
*** gouthamr has joined #openstack-ironic | 20:58 | |
*** gouthamr has quit IRC | 20:59 | |
*** gouthamr has joined #openstack-ironic | 20:59 | |
*** logan- has joined #openstack-ironic | 21:01 | |
rpittau | I'll check the CI changes for IPA later | 21:03 |
rpittau | o/ | 21:03 |
*** rpittau is now known as rpittau|afk | 21:03 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Tear down raid sets https://review.opendev.org/757141 | 21:14 |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Tear down raid sets https://review.opendev.org/757141 | 21:16 |
TheJulia | Okay, that should be better... I hope | 21:17 |
*** fgofurov has quit IRC | 21:29 | |
*** akiselev has quit IRC | 21:38 | |
*** aedc_ has quit IRC | 21:42 | |
*** Goneri has quit IRC | 21:48 | |
*** sshnaidm|rover is now known as sshnaidm|afk | 21:50 | |
*** thiagop has quit IRC | 22:01 | |
*** k_mouza has joined #openstack-ironic | 22:08 | |
*** dsneddon has joined #openstack-ironic | 22:20 | |
*** uzumaki has quit IRC | 22:23 | |
*** ociuhandu has quit IRC | 22:39 | |
*** happy-contrib-43 has quit IRC | 22:53 | |
*** rcernin has joined #openstack-ironic | 23:02 | |
TheJulia | Looks like infra is making changes w/r/t rax due to performance issues | 23:14 |
TheJulia | Once that is in, I guess we should be able to recheck | 23:14 |
*** dsneddon has quit IRC | 23:34 | |
*** dsneddon has joined #openstack-ironic | 23:34 | |
*** Goneri has joined #openstack-ironic | 23:38 | |
*** dsneddon has quit IRC | 23:39 | |
*** dsneddon has joined #openstack-ironic | 23:44 | |
*** tzumainn has quit IRC | 23:50 | |
*** tosky has quit IRC | 23:52 | |
*** anuradha1904 has quit IRC | 23:54 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!