| *** dsneddon has quit IRC | 00:02 | |
| *** dsneddon has joined #openstack-ironic | 00:06 | |
| *** dsneddon has quit IRC | 00:12 | |
| *** dsneddon has joined #openstack-ironic | 00:12 | |
| *** TxGirlGeek has joined #openstack-ironic | 00:18 | |
| *** kaifeng has quit IRC | 00:36 | |
| *** kaifeng has joined #openstack-ironic | 00:43 | |
| *** ricolin has joined #openstack-ironic | 00:48 | |
| *** rloo has quit IRC | 00:55 | |
| *** TxGirlGeek has quit IRC | 01:06 | |
| *** gyee has quit IRC | 01:45 | |
| *** ricolin_ has joined #openstack-ironic | 02:05 | |
| *** ijw_ has quit IRC | 02:06 | |
| *** dsneddon has quit IRC | 02:06 | |
| *** ricolin has quit IRC | 02:07 | |
| *** dsneddon has joined #openstack-ironic | 02:29 | |
| *** ijw has joined #openstack-ironic | 02:41 | |
| *** ijw has quit IRC | 02:47 | |
| *** mkrai has joined #openstack-ironic | 03:00 | |
| *** tzumainn has quit IRC | 03:12 | |
| *** rh-jelabarre has quit IRC | 03:12 | |
| *** TxGirlGeek has joined #openstack-ironic | 03:43 | |
| *** ijw has joined #openstack-ironic | 04:43 | |
| mkrai | good morning ironic o/ | 04:48 |
|---|---|---|
| *** ijw has quit IRC | 04:48 | |
| *** gregoryo has joined #openstack-ironic | 04:57 | |
| *** hwoarang has quit IRC | 05:33 | |
| *** hwoarang has joined #openstack-ironic | 05:34 | |
| *** TxGirlGeek has quit IRC | 05:41 | |
| *** dsneddon has quit IRC | 05:58 | |
| *** dsneddon has joined #openstack-ironic | 06:02 | |
| *** dsneddon has quit IRC | 06:07 | |
| *** etingof is now known as etingof|afk | 06:14 | |
| *** pcaruana has joined #openstack-ironic | 06:18 | |
| *** mkrai has quit IRC | 06:23 | |
| *** mkrai_ has joined #openstack-ironic | 06:24 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-python-agent-builder master: Fix the binary name in document https://review.opendev.org/698631 | 06:36 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui stable/stein: Imported Translations from Zanata https://review.opendev.org/698633 | 06:38 |
| *** jtomasek has joined #openstack-ironic | 06:41 | |
| *** ijw has joined #openstack-ironic | 06:45 | |
| *** dsneddon has joined #openstack-ironic | 06:49 | |
| *** ijw has quit IRC | 06:50 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Remove deprecated ironic-agent element https://review.opendev.org/698636 | 07:11 |
| *** gkadam has joined #openstack-ironic | 07:13 | |
| *** gkadam has quit IRC | 07:13 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-python-agent master: Remove deprecated ironic-agent element https://review.opendev.org/698637 | 07:14 |
| kaifeng | good morning mkrai and ironic o/ | 07:17 |
| mkrai_ | good morning kaifeng o/ | 07:18 |
| mkrai_ | kaifeng, Do you have the link for ironic mid-cycle handy? | 07:19 |
| kaifeng | mkrai_: there was discussion in last few meetings, but i don't remember there is a link to the event | 07:21 |
| mkrai_ | kaifeng, Ok I will check the logs | 07:21 |
| mkrai_ | I was looking at the etherpad but didn't find it | 07:21 |
| mkrai_ | But I remember there was some link for attendance confirmation | 07:22 |
| kaifeng | mkrai_: there was a registration link at CERN site, if that's you are looking for. | 07:22 |
| mkrai_ | kaifeng, yes yes | 07:22 |
| *** gregoryo has quit IRC | 07:22 | |
| kaifeng | i think this is the one: https://idico.cern.ch/event/863986 | 07:23 |
| kaifeng | s/idico/indico/ | 07:23 |
| mkrai_ | kaifeng, thanks :) | 07:26 |
| kaifeng | no big deal :) | 07:27 |
| arne_wiebalck | Good morning, ironic! | 07:36 |
| kaifeng | hey arne_wiebalck o/ | 07:38 |
| arne_wiebalck | Hey kaifeng and mkrai_ o/ | 07:38 |
| mkrai_ | good morning arne_wiebalck o/ | 07:56 |
| *** tesseract has joined #openstack-ironic | 07:59 | |
| arne_wiebalck | mkrai_: ping me if you need to know anything for the logistics of the mid-cycle | 08:06 |
| *** khansa has joined #openstack-ironic | 08:28 | |
| *** mkrai_ has quit IRC | 08:32 | |
| *** dsneddon has quit IRC | 08:32 | |
| *** khansa has quit IRC | 08:36 | |
| *** gudrutis2 has left #openstack-ironic | 08:39 | |
| *** gudrutis2 has joined #openstack-ironic | 08:39 | |
| *** mkrai_ has joined #openstack-ironic | 08:41 | |
| *** ijw has joined #openstack-ironic | 08:46 | |
| *** khansa has joined #openstack-ironic | 08:48 | |
| *** ijw has quit IRC | 08:51 | |
| *** tonyb has quit IRC | 08:56 | |
| *** tonyb has joined #openstack-ironic | 08:56 | |
| *** dsneddon has joined #openstack-ironic | 08:58 | |
| *** rpittau|afk is now known as rpittau | 09:00 | |
| rpittau | good morning ironic! o/ | 09:00 |
| *** dsneddon has quit IRC | 09:05 | |
| kaifeng | hi rpittau o/ | 09:07 |
| rpittau | hey kaifeng :) | 09:07 |
| khansa | hey rpittau ! Good morning o/ | 09:09 |
| rpittau | hey khansa :) | 09:09 |
| rpittau | khansa: how did the update go? :) | 09:09 |
| khansa | rpittau humm not very well :D , I postponed it to next weekend :D but sushy emulator works very very well :D | 09:12 |
| *** lucasagomes has joined #openstack-ironic | 09:13 | |
| rpittau | khansa: eheh, well glad to see the sushy emulator works fine :) | 09:14 |
| khansa | lol yes :D I am currently discovering and reading some cliff documentation, very interesting (y) | 09:18 |
| *** derekh has joined #openstack-ironic | 09:19 | |
| *** hwoarang has quit IRC | 09:23 | |
| *** hwoarang has joined #openstack-ironic | 09:26 | |
| *** goldyfruit_ has quit IRC | 09:29 | |
| *** iurygregory has joined #openstack-ironic | 09:31 | |
| iurygregory | good morning Ironic | 09:31 |
| openstackgerrit | Merged openstack/ironic master: Remove deprecated ironic-agent element https://review.opendev.org/698636 | 09:35 |
| *** dsneddon has joined #openstack-ironic | 09:40 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Fix the binary name in document https://review.opendev.org/698631 | 09:42 |
| openstackgerrit | Merged openstack/ironic-ui stable/stein: Imported Translations from Zanata https://review.opendev.org/698633 | 09:47 |
| *** dtantsur|afk is now known as dtantsur | 09:52 | |
| dtantsur | morning ironic | 09:53 |
| *** dougsz has joined #openstack-ironic | 09:59 | |
| *** mmethot has quit IRC | 10:03 | |
| iurygregory | morning dtantsur | 10:03 |
| *** mmethot has joined #openstack-ironic | 10:04 | |
| hjensas | dtantsur: 2019-11-28 (dtantsur) DIB job is broken: dhcp-all-interfaces times out on DHCP. Help needed. - Is this still an issue? | 10:07 |
| dtantsur | hjensas: we only got the new DIB in upper-constrains last night, so hard to tell | 10:08 |
| dtantsur | it was a problem yesterday | 10:08 |
| dtantsur | I can post a test patch to check | 10:09 |
| hjensas | ok, I hope the fix to the missing break in the retry loop that arne_wiebalck spotted fixes it. | 10:10 |
| dtantsur | me too | 10:11 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM testing the DIB job https://review.opendev.org/698674 | 10:11 |
| hjensas | dtantsur: thanks, I'll keep an eye on that check. | 10:11 |
| *** khansa has quit IRC | 10:19 | |
| *** mkrai_ has quit IRC | 10:22 | |
| *** dsneddon has quit IRC | 10:24 | |
| *** alexmcleod has joined #openstack-ironic | 10:32 | |
| *** mmethot has quit IRC | 10:34 | |
| *** khansa has joined #openstack-ironic | 10:34 | |
| *** dsneddon has joined #openstack-ironic | 10:40 | |
| *** mmethot has joined #openstack-ironic | 10:42 | |
| *** dsneddon has quit IRC | 10:46 | |
| *** ijw has joined #openstack-ironic | 10:47 | |
| *** goldyfruit_ has joined #openstack-ironic | 10:50 | |
| *** bobmel has joined #openstack-ironic | 10:51 | |
| *** ijw has quit IRC | 10:51 | |
| *** rcernin has quit IRC | 10:56 | |
| *** pcaruana has quit IRC | 11:03 | |
| *** khansa has quit IRC | 11:05 | |
| *** pcaruana has joined #openstack-ironic | 11:06 | |
| *** khansa has joined #openstack-ironic | 11:07 | |
| *** dtantsur is now known as dtantsur|brb | 11:09 | |
| *** goldyfruit_ has quit IRC | 11:13 | |
| *** dsneddon has joined #openstack-ironic | 11:13 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test multinode job https://review.opendev.org/698696 | 11:18 |
| *** dougsz has quit IRC | 11:24 | |
| *** sshnaidm|afk is now known as sshnaidm | 11:32 | |
| *** mkrai has joined #openstack-ironic | 11:35 | |
| *** dougsz has joined #openstack-ironic | 11:41 | |
| mkrai | morning iurygregory dtantsur|brb hjensas | 11:46 |
| openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Remove version check for encoding https://review.opendev.org/698701 | 11:48 |
| *** mmethot has quit IRC | 11:52 | |
| *** mmethot has joined #openstack-ironic | 11:52 | |
| *** mmethot has quit IRC | 11:58 | |
| *** belmoreira has joined #openstack-ironic | 12:01 | |
| iurygregory | morning mkrai o/ | 12:02 |
| *** khansa has quit IRC | 12:04 | |
| *** mkrai has quit IRC | 12:04 | |
| *** mkrai_ has joined #openstack-ironic | 12:04 | |
| *** khansa has joined #openstack-ironic | 12:06 | |
| *** mmethot has joined #openstack-ironic | 12:09 | |
| *** dsneddon has quit IRC | 12:28 | |
| *** Goneri has quit IRC | 12:31 | |
| *** sshnaidm has quit IRC | 12:37 | |
| *** sshnaidm has joined #openstack-ironic | 12:38 | |
| *** ijw has joined #openstack-ironic | 12:48 | |
| *** ijw has quit IRC | 12:52 | |
| *** bfournie has quit IRC | 12:57 | |
| *** mkrai_ has quit IRC | 12:57 | |
| *** dtantsur|brb is now known as dtantsur | 12:58 | |
| *** dsneddon has joined #openstack-ironic | 13:00 | |
| *** ijw has joined #openstack-ironic | 13:00 | |
| dtantsur | hjensas: we're hitting a python 2 problem now. this is solvable! | 13:02 |
| iurygregory | kill py2 and you will solve the problem \o/ | 13:04 |
| *** rh-jelabarre has joined #openstack-ironic | 13:04 | |
| dtantsur | I'm trying to see if we can do it without switching to centos 8 | 13:10 |
| iurygregory | good luck o/ | 13:10 |
| *** belmorei_ has joined #openstack-ironic | 13:11 | |
| *** belmorei_ has quit IRC | 13:13 | |
| *** belmoreira has quit IRC | 13:13 | |
| *** belmoreira has joined #openstack-ironic | 13:13 | |
| *** sshnaidm has quit IRC | 13:18 | |
| *** FlorianFa has joined #openstack-ironic | 13:22 | |
| *** mmethot has quit IRC | 13:24 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Avoid grub2-install when on UEFI boot mode https://review.opendev.org/696914 | 13:27 |
| *** mmethot has joined #openstack-ironic | 13:30 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Replace CentOS 7 images with CentOS 8 https://review.opendev.org/696383 | 13:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Replace CentOS 7 images with CentOS 8 https://review.opendev.org/696383 | 13:38 |
| openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Remove version check for encoding https://review.opendev.org/698701 | 13:41 |
| *** mkrai has joined #openstack-ironic | 13:42 | |
| *** Goneri has joined #openstack-ironic | 13:45 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Increasing BUILD_TIMEOUT value for multinode job https://review.opendev.org/698720 | 13:49 |
| *** bfournie has joined #openstack-ironic | 13:50 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Increasing BUILD_TIMEOUT value for multinode job https://review.opendev.org/698720 | 13:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: DNM test the DIB fix https://review.opendev.org/698722 | 13:55 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test multinode job https://review.opendev.org/698696 | 13:56 |
| *** pcaruana has quit IRC | 13:57 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test multinode job https://review.opendev.org/698696 | 14:01 |
| *** ijw has quit IRC | 14:04 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: DNM test the DIB fix https://review.opendev.org/698722 | 14:04 |
| hjensas | dtantsur: ok, thats good news. | 14:05 |
| dtantsur | and IPA-builder is broken by DIB again :( | 14:06 |
| rpittau | dtantsur: I think there's another patch needed before centos8 will work -> https://review.opendev.org/698448 | 14:07 |
| patchbot | patch 698448 - diskimage-builder - Update bindep for RHEL/CentOS 8 - 2 patch sets | 14:07 |
| dtantsur | rpittau: and another https://review.opendev.org/#/c/697211/ | 14:08 |
| patchbot | patch 697211 - diskimage-builder - Set correct python version for non-chroot scripts - 8 patch sets | 14:08 |
| rpittau | yeah, I saw the discussion on the dib channel | 14:09 |
| dtantsur | ah, got it | 14:09 |
| *** hjensas has quit IRC | 14:11 | |
| *** dsneddon has quit IRC | 14:13 | |
| TheJulia | ugh | 14:16 |
| * TheJulia ponders another latte | 14:16 | |
| dtantsur | rpittau: oh, https://review.opendev.org/#/c/698448/ won't be needed because we only use CentOS 8 inside of the image, outside it's Ubuntu | 14:16 |
| patchbot | patch 698448 - diskimage-builder - Update bindep for RHEL/CentOS 8 - 2 patch sets | 14:16 |
| dtantsur | morning TheJulia | 14:16 |
| rpittau | dtantsur: cool, right! | 14:16 |
| *** aedc has joined #openstack-ironic | 14:17 | |
| rpittau | good morning TheJulia :) | 14:17 |
| iurygregory | good morning TheJulia | 14:20 |
| gudrutis2 | Hi all . How do you debug your unit tests? I have something like ` tox -e py3 -- ironic_inspector.test.unit.test_rules.TestApplyActions.test_apply `, but I can't run PDB | 14:24 |
| efried | gudrutis2: I use remote-pdb | 14:25 |
| efried | I'm pretty sure there are ways to do it with regular pdb, but I've never taken the time to figure them out. | 14:25 |
| gudrutis2 | Can I just print it the values somehow then? | 14:26 |
| dtantsur | insert prints everywhere \o/ | 14:26 |
| efried | gudrutis2: Here's an article about how to use it (it's oriented around debugging processes under systemd control, but the principle is the same) https://docs.openstack.org/devstack/latest/systemd.html#using-remote-pdb | 14:26 |
| dtantsur | this is what I do :) | 14:26 |
| efried | remote-pdb is easy, works *almost* exactly like regular pdb | 14:26 |
| iurygregory | gudrutis2, tox -r -vv -epy3 -- ironic_inspector.test.unit.test_rules.TestApplyActions.test_apply | 14:28 |
| iurygregory | this you will run a specific test, if you want to add prints to it feel free but don't push changes with prints | 14:28 |
| rpittau | print ftw \o/ | 14:29 |
| iurygregory | _\m/ | 14:29 |
| *** dsneddon has joined #openstack-ironic | 14:30 | |
| gudrutis2 | dtantsur: you are right, simple print() works, just needed to find in the logs | 14:35 |
| *** dsneddon has quit IRC | 14:35 | |
| *** pcaruana has joined #openstack-ironic | 14:36 | |
| iurygregory | gudrutis2, if you only run the test you want to debug it helps to find the prints | 14:40 |
| iurygregory | if you run all tests I can only say "good luck" =) | 14:40 |
| gudrutis2 | efried: thanks for the tip, but isn't remote-pdb it more for service debugging, not unit test. That is what I understood from reading? | 14:40 |
| efried | gudrutis2: It's for whatever. Service debugging is what that article is oriented toward, but you can (and I do, frequently) use it for tox. | 14:41 |
| efried | Because you have to do special things to make regular pdb work with tox. | 14:42 |
| efried | (I think, but am not sure, that you can bypass tox, activate the venv, and run stestr manually, and that'll pay attention to pdb, but I haven't tried that lately) | 14:42 |
| * efried tries... | 14:42 | |
| efried | gudrutis2: Okay, yes, you can, but you have to use the -n flag (--no-discover) to make it single-thread. | 14:44 |
| efried | otherwise you still get BdbQuit | 14:45 |
| efried | gudrutis2: so for future reference, you can: | 14:47 |
| efried | $ source .tox/{env}/bin/activate | 14:47 |
| efried | $ stestr run -n exact.path.to.module.TestClass.test_case | 14:47 |
| efried | ...having injected your usual | 14:47 |
| efried | import pdb; pdb.set_trace() | 14:47 |
| efried | where you want to break | 14:47 |
| gudrutis2 | Thanks, it works | 14:49 |
| efried | \o/ | 14:49 |
| gudrutis2 | should we update the documentation you gave to me? | 14:51 |
| *** tzumainn has joined #openstack-ironic | 14:53 | |
| efried | gudrutis2: Not that doc, as it's still oriented toward services. But it would be cool to find another place to put that information. | 14:54 |
| efried | I thought there was a doc somewhere that talked about tox and stestr and stuff. Maybe I'm thinking of an internal email from a zillion years ago. mtreinish might know? | 14:55 |
| *** hjensas has joined #openstack-ironic | 14:56 | |
| *** ricolin_ is now known as ricolin | 14:57 | |
| efried | gudrutis2: I'm not finding anything via obvious google searches. (At least nothing recent; there's an old superuser article that still talks about using testtools to get at pdb.) | 14:58 |
| efried | there's this wiki page https://wiki.openstack.org/wiki/Testing | 14:58 |
| efried | We could update that; but it would be way better to find a home on docs.o.o somewhere instead. | 14:58 |
| efried | Could as the folks in #openstack-doc for a suitable place to port that wiki page | 14:59 |
| efried | ask* | 14:59 |
| *** henriqueof has quit IRC | 15:08 | |
| *** dsneddon has joined #openstack-ironic | 15:08 | |
| *** henriqueof1 has joined #openstack-ironic | 15:08 | |
| *** dtantsur is now known as dtantsur|brb | 15:10 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: Fix use of urlparse.urljoin https://review.opendev.org/698734 | 15:16 |
| gudrutis2 | efried: yeah I was a bit confused that we have 2 different documentations. | 15:27 |
| efried | gudrutis2: wiki is "old"; docs.o.o is "new". | 15:27 |
| efried | as people a) care and b) have time, we're trying to port content that's actually used/useful from the former to the latter. | 15:28 |
| gudrutis2 | ok, now I know, but there is no statement that wiki is discontinued in title page :) | 15:29 |
| efried | gudrutis2: Another good thing to bring up in -doc. I'm sure they would welcome some help cleaning up things like this. | 15:30 |
| gudrutis2 | efried: ok, I will ask | 15:30 |
| efried | and it's not technically "discontinued", because we haven't ported everything over. | 15:30 |
| efried | it's just, if there's a docs.o.o page for $topic, it's preferred. And it would be nice to chop the corresponding wiki page and make it redirect/link to the docs.o.o page, so if you notice that, you should feel free to do so. | 15:31 |
| gudrutis2 | efried: Well, lets call it deprecated in this case | 15:31 |
| efried | updating the wiki is "easier" than updating something on docs.o.o -- no peer review :) | 15:31 |
| *** dsneddon has quit IRC | 15:32 | |
| gudrutis2 | efried: lol, next time if I need to fabricate some functionality, I will know where to go :) | 15:33 |
| efried | it's not a total wild west; your name is still attached to your edits :P | 15:34 |
| gudrutis2 | Pff, as if it is totally hard and impossible to fake... | 15:35 |
| TheJulia | brraaaainnnnsss | 15:41 |
| gudrutis2 | efried: asked in #openstack-doc, I think you saw it | 15:42 |
| *** ijw has joined #openstack-ironic | 15:45 | |
| *** TxGirlGeek has joined #openstack-ironic | 15:45 | |
| *** ijw has quit IRC | 15:50 | |
| *** dtantsur|brb is now known as dtantsur | 15:52 | |
| *** hwoarang has quit IRC | 15:52 | |
| gudrutis2 | TheJulia: oh hi o\ | 15:53 |
| * TheJulia is a brain eating meeting running zombie at this time | 15:53 | |
| *** hwoarang has joined #openstack-ironic | 15:54 | |
| TheJulia | "retrospective..... braiiinnsss..... retrospective.... brainsss...." | 15:54 |
| * TheJulia hopes everyone is amused | 15:54 | |
| dtantsur | :D | 15:54 |
| dtantsur | TheJulia: I'd be afraid of you if I had any brains left :) | 15:55 |
| gudrutis2 | TheJulia: I am not sure if everyone is amused, but for sure I am confused... | 15:55 |
| dtantsur | confused is a normal state when working on openstack | 15:56 |
| TheJulia | gudrutis2: sorry, I'm just really tried since I've had lots of meetings, hence the zombie reference... and I'm about to run my internal team's retrospective in a few minutes | 15:56 |
| TheJulia | This may be one of those weeks where I go ahead and take a nap after my meetings | 15:57 |
| dtantsur | ++ | 15:58 |
| *** ijw has joined #openstack-ironic | 16:01 | |
| *** hjensas has quit IRC | 16:04 | |
| *** gyee has joined #openstack-ironic | 16:05 | |
| *** irclogbot_2 has quit IRC | 16:14 | |
| *** irclogbot_2 has joined #openstack-ironic | 16:15 | |
| gudrutis2 | TheJulia: perfectly understandable. I wish you good rest, Praise The Lord :) | 16:16 |
| *** dsneddon has joined #openstack-ironic | 16:19 | |
| *** Lucas_Gray has joined #openstack-ironic | 16:32 | |
| *** TxGirlGeek has quit IRC | 16:45 | |
| *** tesseract has quit IRC | 16:48 | |
| *** lucasagomes has quit IRC | 16:52 | |
| *** TxGirlGeek has joined #openstack-ironic | 16:52 | |
| *** alexmcleod has quit IRC | 16:56 | |
| dtantsur | rpioso: hey! derekh is hitting this line https://opendev.org/x/sushy-oem-idrac/src/branch/master/sushy_oem_idrac/asynchronous.py#L48 do you know if it's actually required? | 17:00 |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic-tempest-plugin master: Remove use of deprecated config options for volume API selection https://review.opendev.org/698763 | 17:02 |
| *** efried is now known as efried_afk | 17:08 | |
| *** jtomasek has quit IRC | 17:08 | |
| *** ociuhandu has joined #openstack-ironic | 17:10 | |
| *** ociuhandu has quit IRC | 17:10 | |
| *** ociuhandu has joined #openstack-ironic | 17:10 | |
| *** iurygregory has quit IRC | 17:11 | |
| *** Lucas_Gray has quit IRC | 17:12 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP: Decompose core deploy step https://review.opendev.org/698309 | 17:12 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP: In-band deploy steps https://review.opendev.org/698768 | 17:12 |
| openstackgerrit | Mark Goddard proposed openstack/ironic-python-agent master: WIP: In-band deploy steps https://review.opendev.org/698770 | 17:14 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP: In-band deploy steps https://review.opendev.org/698768 | 17:14 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Import importlib directly https://review.opendev.org/698771 | 17:16 |
| rpioso | Good morning, ironicers! | 17:23 |
| rpioso | dtantsur: Do you have more context? | 17:26 |
| dtantsur | rpioso: derekh does. I think he's trying to deploy with virtual media and hits this issue. | 17:26 |
| rpioso | derekh, dtantsur: Which HW model and iDRAC firmware version encountered that exception? Do you have a traceback? Information about the workflow which encountered it would be helpful, too. | 17:28 |
| derekh | rpioso: hi, ya I'm trying out idrac-redfish-virtual-media , but hit that error when ironic is trying to set the boot device , power management seems to work | 17:28 |
| * rpioso is preparing to do the same very soon. | 17:28 | |
| rpioso | derekh: Awesome timing :-) | 17:29 |
| *** sziviani has quit IRC | 17:29 | |
| derekh | rpioso: PowerEdge R640 , 3.36.36.36 | 17:29 |
| rpioso | derekh: Did you notice if the iDRAC had a pending, scheduled job immediately following that exception? | 17:30 |
| derekh | rpioso: I didn't but I'll keep an look out for it next time around | 17:31 |
| * rpioso retrieves the iDRAC fw version from his R740xd. | 17:32 | |
| *** iurygregory has joined #openstack-ironic | 17:32 | |
| rpioso | derekh: I'm running 3.30.30.30. However, please note I haven't performed that workflow, yet, so ... | 17:33 |
| derekh | rpioso: ok | 17:33 |
| rpioso | derekh: There's a fresh iDRAC firmware release 4.00.00.00. It was released yesterday. | 17:36 |
| dtantsur | this version looks like something one should immediately put in production :D | 17:36 |
| JayF | 4.redflag.redflag.redflag | 17:37 |
| JayF | lol | 17:37 |
| dtantsur | hehe | 17:37 |
| *** dtantsur is now known as dtantsur|afk | 17:37 | |
| *** ociuhandu has quit IRC | 17:37 | |
| dtantsur|afk | have a good night | 17:37 |
| *** ociuhandu has joined #openstack-ironic | 17:39 | |
| derekh | rpioso: I've updated sushy_task_poll_period=5 and the GET got a HTTP 200 instead of a 202, and it looks like the boot deploy is continuing | 17:40 |
| derekh | rpioso: so Location header just missing from HTTP 202's maybe | 17:40 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test multinode job https://review.opendev.org/698696 | 17:45 |
| *** ociuhandu has quit IRC | 17:46 | |
| *** ociuhandu has joined #openstack-ironic | 17:46 | |
| rpioso | derekh: Interesting ... Your hypothesis sounds reasonable. I wonder if it was the response from the original POST or subsequent GET which was missing the location header. A traceback would help. | 17:48 |
| *** rpittau is now known as rpittau|afk | 17:49 | |
| rpittau|afk | good night! o/ | 17:49 |
| rpioso | derekh: We shouldn't have to tweak that sush_task_poll_period to make it work, although it's a workaround for now. | 17:49 |
| rpioso | rpittau|afk: Nite o/ | 17:49 |
| derekh | rpioso: looks like it was the GET https://goodsquishy.com/upload/098a88afe88383724e8a | 17:50 |
| *** ijw has quit IRC | 17:53 | |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Restrict ability to change owner on provisioned node https://review.opendev.org/698078 | 17:53 |
| *** TxGirlGeek has quit IRC | 17:59 | |
| derekh | rpioso: gotta run, ping me tomorrow if you want any info, I'm happy for the moment with the workaround | 18:00 |
| * TheJulia finally feels over with meetings for the day and somewhat refreshed | 18:00 | |
| *** derekh is now known as derekh_afk | 18:00 | |
| TheJulia | JayF: lol | 18:02 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Stop using six library https://review.opendev.org/696669 | 18:02 |
| etingof|afk | derekh_afk, interestingly, I did not run into this timing thing... | 18:11 |
| rpioso | derekh_afk: Yeah, it's the GET of the location URI in the POST's response. | 18:11 |
| rpioso | etingof|afk: o/ | 18:12 |
| TheJulia | It may be BMC cpu load related | 18:12 |
| TheJulia | or just slightly different version related | 18:12 |
| *** mkrai has quit IRC | 18:12 | |
| rpioso | etingof|afk: Did you exercise the same iDRAC fw version, 3.36.36.36, as derekh_afk? I seem to recall you installed 3.34.34.34. | 18:13 |
| * rpioso wonders if the response from a GET of the Task Monitor must include a location header for 202 . The DMTF Redfish spec is very clear the POST must, and it contains the URI of the Task Monitor. | 18:18 | |
| *** igordc has joined #openstack-ironic | 18:21 | |
| *** ricolin has quit IRC | 18:24 | |
| *** ociuhandu has quit IRC | 18:25 | |
| *** ociuhandu has joined #openstack-ironic | 18:26 | |
| derekh_afk | or because my ironic had lower latency to the BMC, anyways latest deploy worked , will be trying to make it IPv6 later.... | 18:27 |
| *** TxGirlGeek has joined #openstack-ironic | 18:30 | |
| *** ociuhandu has quit IRC | 18:31 | |
| TheJulia | oh, that could definitely be. I know etingof's setup has some huge latency | 18:36 |
| TheJulia | (which also is good, but yeah...) | 18:37 |
| etingof|afk | rpioso, o/ my fw is 3.34.34.34 | 18:38 |
| * rpioso is relieved he remembered ;-) | 18:39 | |
| etingof|afk | TheJulia, my setup stretches the cloud across the face of the Earth | 18:39 |
| rpioso | etingof|afk: Do you recall if the GET returned a 202? | 18:40 |
| * rpioso is willing to bet it didn't. | 18:40 | |
| *** rmcall has joined #openstack-ironic | 18:40 | |
| etingof|afk | I do not remember already :( | 18:41 |
| * etingof|afk is short of memory | 18:41 | |
| * rpioso vaguely recalls etingof's description of the network config to access real hardware :-) | 18:41 | |
| openstackgerrit | Merged openstack/ironic master: Add a CI job to legacy boot over Redfish virtual media https://review.opendev.org/693017 | 18:43 |
| TheJulia | etingof|afk: so... your doing edge deployment ;) | 18:43 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-tempest-plugin master: Avoid referenced before assignment in retry_on_conflict https://review.opendev.org/698785 | 18:44 |
| rpioso | derekh_afk, etingof|afk, TheJulia: My rig has exceedingly low latency. When I test vmedia boot, I'll start with 3.30.30.30 and see if it trips over the missing location header. I'll also try to contact mraineri. Meanwhile, ... | 18:45 |
| * etingof|afk likes the other name for his suffering | 18:45 | |
| rpioso | bdodd: WeWhen the response from a GET of a TaskMonitor URI has status code 202, must it contain a location header? | 18:48 |
| *** ociuhandu has joined #openstack-ironic | 18:49 | |
| rpioso | bdodd: We're discussing an issue ^^^ with an async operation POSTed to an iDRAC. | 18:49 |
| rpioso | s/WeWhen/When/ | 18:50 |
| * rpioso curses his uncooperative keyboard. | 18:50 | |
| *** rloo has joined #openstack-ironic | 18:50 | |
| *** ociuhandu has quit IRC | 18:51 | |
| bdodd | rpioso: Yes any response with a 202 status must have a Location header. | 18:53 |
| *** gmann is now known as gmann_afk | 18:53 | |
| rpioso | bdodd: Wouldn't the Location header's value be the same URI that was just GET? | 18:55 |
| rpioso | Seems sort of redundant. | 18:56 |
| bdodd | Yes, it should be the same, so if you saved it initially you wouldn't need to re-read the Location header | 18:57 |
| rpioso | bdodd: Thank you :-) | 18:57 |
| bdodd | rpioso yw! | 18:58 |
| *** ociuhandu has joined #openstack-ironic | 18:59 | |
| TheJulia | I _thought_ that you could get back a different location header in the reply as the indicator of where to go next with the redfish API, or am I thinking of just the ?eventing? stuffs | 19:00 |
| *** ociuhandu has quit IRC | 19:03 | |
| *** dougsz has quit IRC | 19:08 | |
| *** ociuhandu has joined #openstack-ironic | 19:09 | |
| *** ociuhandu has quit IRC | 19:14 | |
| bdodd | TheJulia I can't find any language in the spec that the task monitor URI for a given async operation should or could change. | 19:20 |
| *** iurygregory has quit IRC | 19:22 | |
| *** iurygregory has joined #openstack-ironic | 19:23 | |
| *** chihhsin has quit IRC | 19:27 | |
| *** ociuhandu has joined #openstack-ironic | 19:30 | |
| *** ociuhandu has quit IRC | 19:34 | |
| *** efried_afk is now known as efried | 19:38 | |
| *** mmethot has quit IRC | 19:40 | |
| *** dougsz has joined #openstack-ironic | 19:43 | |
| rpioso | bdodd, etingof|afk, TheJulia: From my read of the spec, it's not crystal clear Location header must be returned by the task monitor with 202. The first paragraph of https://www.dmtf.org/sites/default/files/standards/documents/DSP0266_1.8.0.pdf, p. 123, is in the context of the response to a request for a long-running operation (POST, in our case) which started at the end of p. 122. All other spec discussions of | 19:52 |
| rpioso | Location header similarly seem to be in the context of the response to the original request for a long-running operation. | 19:52 |
| openstackgerrit | Merged openstack/ironic master: Add owner to allocations and create relevant policies https://review.opendev.org/696707 | 19:53 |
| *** iurygregory has quit IRC | 20:08 | |
| bdodd | rpioso See the table on pages 56 and 57. It makes a general statement about the requirements for a 202 response. | 20:09 |
| bdodd | I do agree that the "Asynchronous operations" section of the spec could be more clear about Location header behavior in the the case of the GET on the Task Monitor URI. | 20:12 |
| *** ijw has joined #openstack-ironic | 20:14 | |
| *** pcaruana has quit IRC | 20:19 | |
| *** dougsz has quit IRC | 20:21 | |
| rpioso | bdodd: The description of 202 in that table seems to focus on the original request -- "The request has been accepted for processing, but the processing has not been completed. The Location header shall be set to the URI of a Task Monitor that can later be queried to determine the status of the operation. A representation of the Task resource may be included in the response body." | 20:21 |
| rpioso | "The request has been accepted for processing". Doh! Of course, the POST caused that earlier. | 20:23 |
| rpioso | "but the processing has not been completed". That applies to the POST and GETs of the Task Monitor. | 20:24 |
| rpioso | "The Location header shall ..." Seems redundant for GETs of the Task Monitor, since there's no provision in the spec for changing its URI. | 20:25 |
| rpioso | "A representation of the Task resource may be included in the response body." Only applies to the POST. | 20:25 |
| rpioso | However, I suppose the word 'may' could cover that. | 20:27 |
| *** belmoreira has quit IRC | 20:27 | |
| bdodd | Yes, I think the Task resource could also be included in the body of the 202 response for the task monitor GET. | 20:29 |
| rpioso | That does make sense. | 20:31 |
| bdodd | Conceptually, the task monitor behaves like this: (1) while the task is still in process, the response is like the initial response (202 status, Location header, optional Retry-After header, optional Task resource in body) | 20:33 |
| * rpioso finds that description crystal clear :-) | 20:35 | |
| bdodd | (2) after the task is completed the response is the same as if the initial response completed synchronously | 20:35 |
| bdodd | That's my understanding from the spec and past discussions with Raineri and company. | 20:37 |
| rpioso | I find (2) pretty well described toward the bottom of p. 123. | 20:38 |
| bdodd | Agreed | 20:39 |
| rpioso | Thanks, again, for clarifying. Given all that, it appears we likely have a service implementation bug. | 20:40 |
| rpioso | Btw, your description of (2) is crisper :-) | 20:40 |
| bdodd | :-) | 20:41 |
| TheJulia | bdodd: thanks, I'm likely just mixing stuff up in my brain. :( | 20:44 |
| bdodd | TheJulia I understand. There is a lot of "stuff"! :-) | 20:45 |
| *** derekh_afk has quit IRC | 20:48 | |
| TheJulia | Indeed | 20:49 |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: Fix use of urlparse.urljoin https://review.opendev.org/698734 | 20:52 |
| *** Goneri has quit IRC | 21:27 | |
| *** khansa has quit IRC | 21:34 | |
| *** TxGirlGeek has quit IRC | 22:03 | |
| *** TxGirlGeek has joined #openstack-ironic | 22:04 | |
| *** henriqueof1 has quit IRC | 22:08 | |
| *** gmann_afk is now known as gmann | 22:29 | |
| openstackgerrit | Merged openstack/ironic master: Update nova os-server-external-events response logic https://review.opendev.org/698273 | 22:30 |
| *** rh-jelabarre has quit IRC | 22:30 | |
| *** rmcall has quit IRC | 23:08 | |
| *** rmcall has joined #openstack-ironic | 23:08 | |
| *** bobmel has quit IRC | 23:09 | |
| *** dmellado has quit IRC | 23:09 | |
| *** zbitter has joined #openstack-ironic | 23:10 | |
| *** zaneb has quit IRC | 23:10 | |
| *** irclogbot_2 has quit IRC | 23:11 | |
| *** dmellado has joined #openstack-ironic | 23:11 | |
| *** irclogbot_1 has joined #openstack-ironic | 23:12 | |
| *** TxGirlGeek has quit IRC | 23:14 | |
| *** bfournie has quit IRC | 23:29 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!