| *** pmannidi is now known as pmannidi|brb | 00:58 | |
| opendevreview | Steve Baker proposed openstack/ironic master: WIP use packaged grub efi for network boot https://review.opendev.org/c/openstack/ironic/+/806808 | 01:06 |
|---|---|---|
| *** pmannidi|brb is now known as pmannidi | 01:34 | |
| *** pmannidi is now known as pmannidi|Lunch | 02:20 | |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 02:31 |
| *** pmannidi|Lunch is now known as pmannidi | 04:03 | |
| opendevreview | Steve Baker proposed openstack/ironic master: WIP use packaged grub efi for network boot https://review.opendev.org/c/openstack/ironic/+/806808 | 04:13 |
| iurygregory | good morning Ironic, TGIF! | 06:09 |
| tonyb | stevebaker: Ahh okay that's helpful | 06:13 |
| * tonyb has no idea why irssi just decided to show me those replies :/ ? | 06:15 | |
| janders_ | hey iurygregory o/ | 06:15 |
| *** janders_ is now known as janders | 06:19 | |
| janders | ^ had a comms dropout earlier :/ | 06:20 |
| iurygregory | hey janders o/ | 06:22 |
| * tonyb wonders if janders is *the* Janders ;p ? | 06:28 | |
| janders | tonyb: janders is *the* Janders indeed | 06:31 |
| tonyb | \o/ | 06:31 |
| janders | o/ | 06:32 |
| janders | hey hey | 06:32 |
| tonyb | janders: how've you been? Doing okay in these crazy times? | 06:32 |
| iurygregory | hey tonyb o/ how are you doing? | 06:32 |
| janders | tonyb: we're doing well, thank you :) How have you been? | 06:33 |
| tonyb | iurygregory: hey! Doing okay all things considered you? | 06:33 |
| tonyb | janders: great. Yeah it's been a bumpy ride but I think I'm doing better now | 06:33 |
| arne_wiebalck | Good morning, Ironic! | 06:34 |
| iurygregory | doing good :D (if I remove the weather factor \o/) | 06:34 |
| janders | tonyb: that's great to hear! | 06:34 |
| iurygregory | good morning arne_wiebalck o/ | 06:34 |
| arne_wiebalck | Hey iurygregory o/ | 06:34 |
| dtantsur | morning ironic | 07:22 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Support HttpHeaders in create_subscription https://review.opendev.org/c/openstack/ironic/+/806859 | 07:23 |
| iurygregory | good morning dtantsur | 07:23 |
| dtantsur | folks, could I get some reviews on https://review.opendev.org/c/openstack/ironic/+/807105 please? | 07:50 |
| dtantsur | we're facing a problem that is too hard to debug without it | 07:50 |
| iurygregory | dtantsur, ack I probably missed because didn't have the hashtag | 07:55 |
| dtantsur | iurygregory: have you seen the comment from hberaud on https://review.opendev.org/c/openstack/requirements/+/805611 ? | 07:55 |
| iurygregory | dtantsur, I just saw | 07:55 |
| dtantsur | re hashtag: the patch is very new | 07:55 |
| iurygregory | dtantsur, ou can add the hashtag if you think it's something important that we need to take a look (even if it's new and you didn't ask during the upstream meeting) =) | 07:56 |
| dtantsur | ah, ack | 07:56 |
| iurygregory | WHAT?! No clean steps found <O> ?! | 07:57 |
| dtantsur | yep | 08:11 |
| dtantsur | janders: https://github.com/metal3-io/baremetal-operator/issues/959#issuecomment-912348291 | 08:11 |
| dtantsur | iurygregory: this is because of https://review.opendev.org/c/openstack/ironic/+/807187 | 08:11 |
| iurygregory | I see | 08:12 |
| janders | dtantsur: ACK - and thanks! :) | 08:12 |
| dtantsur | janders: added a 2nd comment explaining the problem #2. Please double-check me on my findings. | 08:14 |
| janders | dtantsur: ACK | 08:17 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 09:13 |
| iurygregory | CI will explode :D | 09:14 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix upgrade logic to allow for bundled changes https://review.opendev.org/c/openstack/ironic/+/803532 | 09:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 09:24 |
| iurygregory | I said CI will explode and we have two patches that failed to merge :D | 09:24 |
| iurygregory | OMG | 09:25 |
| janders | pick your words carefully iurygregory! :) | 09:25 |
| iurygregory | janders, I was talking about my change to test oslo.db 11.0.0 :D | 09:25 |
| iurygregory | but... it's life | 09:25 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Expand the driver contributor documentation https://review.opendev.org/c/openstack/ironic/+/807284 | 09:53 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 09:55 |
| janders | I'm trying to do some RAID related testing using bifrost, VMs and sushy-emulator. I'm getting this error while trying to configure RAID: | 10:16 |
| janders | https://paste.opendev.org/show/808574/ | 10:16 |
| janders | am I running into a sushy-emulator limitation or do I need to put in some additional config in place? | 10:17 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix upgrade logic to allow for bundled changes https://review.opendev.org/c/openstack/ironic/+/803532 | 10:18 |
| dtantsur | janders: I think the patch to manage enabled_raid_interfaces in bifrost hasn't merged et | 10:39 |
| dtantsur | you need to fix ironic.conf manually | 10:39 |
| iurygregory | I was sure I've reviewed this patch | 11:03 |
| * iurygregory looks again | 11:03 | |
| opendevreview | test proposed openstack/ironic master: Modify the definition of owner in parameter.yaml https://review.opendev.org/c/openstack/ironic/+/807290 | 11:05 |
| iurygregory | WAIT... WHAT?! | 11:05 |
| iurygregory | no luck trying to fix the oslo.db problem .-. | 11:08 |
| opendevreview | Merged openstack/ironic master: Improve edge-case debugging for deployment and cleaning https://review.opendev.org/c/openstack/ironic/+/807105 | 11:10 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/18.1: Improve edge-case debugging for deployment and cleaning https://review.opendev.org/c/openstack/ironic/+/807262 | 11:11 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/wallaby: Improve edge-case debugging for deployment and cleaning https://review.opendev.org/c/openstack/ironic/+/807263 | 11:11 |
| iurygregory | zzzeek, hey o/ I'm trying to fix the problem where the insert is not bound to a Connection or Engine in ironic I've changed a test to see but still shows the same failure https://review.opendev.org/c/openstack/ironic/+/807278/2/ironic/tests/unit/db/sqlalchemy/test_migrations.py https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d30/807278/2/check/openstack-tox-py36/d300195/t | 11:11 |
| iurygregory | estr_results.html | 11:11 |
| iurygregory | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d30/807278/2/check/openstack-tox-py36/d300195/testr_results.html for the full log url | 11:11 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add and document high-level helpers for async steps https://review.opendev.org/c/openstack/ironic/+/807295 | 11:29 |
| opendevreview | test proposed openstack/ironic master: Modify the definition of owner https://review.opendev.org/c/openstack/ironic/+/807302 | 12:02 |
| opendevreview | Merged openstack/bifrost master: Allow configuring enabled raid interfaces https://review.opendev.org/c/openstack/bifrost/+/806123 | 12:04 |
| janders | see you next week Ironic o/ | 12:28 |
| janders | have a great weekend everyone | 12:28 |
| iurygregory | bye janders o/ | 12:30 |
| TheJulia | good morning | 13:15 |
| TheJulia | didn't exception handling for duplicate rows change at some point? | 13:17 |
| TheJulia | rpioso: good morning | 13:17 |
| rpioso | TheJulia: Good Friday morning :) | 13:18 |
| opendevreview | Merged openstack/ironic master: Fix upgrade logic to allow for bundled changes https://review.opendev.org/c/openstack/ironic/+/803532 | 13:20 |
| opendevreview | Merged openstack/ironic-python-agent master: Trivial: better debugging in list_all_block_devices https://review.opendev.org/c/openstack/ironic-python-agent/+/806344 | 13:20 |
| TheJulia | rpioso: got time in a few minutes? | 13:22 |
| iurygregory | good morning TheJulia =) | 13:22 |
| rpioso | TheJulia: Yes, I do. | 13:22 |
| dtantsur | does anyone understand the PDF docs and can explain why it fails? https://zuul.opendev.org/t/openstack/build/64ac6aac984544068c7b85f53e93c3d4/log/sphinx-build-pdf.log | 13:28 |
| dtantsur | keeping in mind that answer "noone" will result in me disabling the PDF build | 13:28 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 13:31 |
| TheJulia | dtantsur: aside from latex warnings being thrown on every single link? :( | 13:32 |
| dtantsur | I assume they alone don't cause failures? | 13:32 |
| dtantsur | why are we even doing it? at least a part of the resulting pdf is garbage | 13:33 |
| TheJulia | I *think* all warnings are failures on pdf as well | 13:33 |
| iurygregory | I think so | 13:33 |
| TheJulia | the pdf stuff looked good originally :\ | 13:33 |
| TheJulia | is there still a community docs team? | 13:33 |
| TheJulia | rpioso: https://meet.google.com/jyp-ygdf-rqh | 13:33 |
| iurygregory | dtantsur, I'm testing your patch locally to see if I can find the issue | 13:34 |
| dtantsur | iurygregory: thank you! | 13:35 |
| dtantsur | that being said, I'm personally for removing pdf support | 13:35 |
| dtantsur | I *think* it complains about the code snippet in the docs | 13:35 |
| dtantsur | which may be cause by some rendering issues | 13:37 |
| iurygregory | well, I can't run tox -e pdf-docs locally =X | 13:37 |
| dtantsur | \o/ | 13:38 |
| dtantsur | missing deps? for me it requires some latex bits (that I don't really want to install.. sigh) | 13:38 |
| iurygregory | let me look at the changes you did in the files :D | 13:38 |
| iurygregory | nope, it runs and get stuck with Latex warnings and a lot of random characters :D | 13:38 |
| dtantsur | sweet | 13:38 |
| dtantsur | if you check https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_64a/807295/1/check/openstack-tox-docs/64ac6aa/pdf/doc-ironic.pdf, it starts being complete garbage once it gets to the news docs | 13:39 |
| dtantsur | I suspect it doesn't like a large Python snippet.. which I don't want to remove just because someone introduced a stupid feature to everyone's jobs... | 13:39 |
| dtantsur | actually no, it starts being garbage in the OS profiler docs | 13:40 |
| dtantsur | I guess my addition just hits some threshold | 13:40 |
| opendevreview | Verification of a change to openstack/ironic master failed: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 13:42 |
| dtantsur | yep, everything goes sideways after the image in the profiling docs. maybe we can just remove it? | 13:43 |
| dtantsur | (assuming we think the pdf build is a stupid idea, which I don't) | 13:43 |
| prometheanfire | dtantsur: the explicit sqlalchemy conversion, would tracking https://review.opendev.org/c/openstack/ironic/+/807278 be right? | 13:44 |
| prometheanfire | or would tracking another review be better? | 13:45 |
| dtantsur | prometheanfire: I assume so, but ask iurygregory better | 13:45 |
| prometheanfire | iurygregory: so.... :D | 13:45 |
| dtantsur | iurygregory, TheJulia, I'll try removing the offending images from the OS profiler docs. | 13:47 |
| iurygregory | prometheanfire, well, I'm trying to fix :D | 13:47 |
| iurygregory | no luck so far... | 13:47 |
| iurygregory | because I can't reproduce the issue locally... | 13:47 |
| iurygregory | if you have any tips :D | 13:48 |
| iurygregory | dtantsur, funny thing https://review.opendev.org/c/openstack/ironic/+/807295/1/doc/source/contributor/deploy-steps.rst I don't see "Clean steps basics" section in the master file O.o https://raw.githubusercontent.com/openstack/ironic/master/doc/source/contributor/deploy-steps.rst | 13:49 |
| iurygregory | oh ok | 13:49 |
| dtantsur | mmm? | 13:49 |
| iurygregory | there is a another patch you have.. | 13:49 |
| dtantsur | yeah, it's from the parent patch | 13:50 |
| iurygregory | I was looking at the changes to add from master to see how it goes | 13:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Remove images from the OS profiler docs https://review.opendev.org/c/openstack/ironic/+/807326 | 13:50 |
| iurygregory | and I was like "Woot?" | 13:50 |
| dtantsur | heh | 13:51 |
| zzzeek | iurygregory: i replied | 13:51 |
| iurygregory | zzzeek, tks | 13:52 |
| prometheanfire | iurygregory: not sure about reproducing locally, but you can reproduce in gate at least? | 13:52 |
| iurygregory | prometheanfire, yeah in gate it reproduces =) I will update the test based on zzzeek feedback :D | 13:53 |
| iurygregory | fingers crossed | 13:53 |
| prometheanfire | h,, k | 13:53 |
| opendevreview | test proposed openstack/ironic master: Modify the definition of owner in parameter.yaml https://review.opendev.org/c/openstack/ironic/+/807332 | 13:55 |
| TheJulia | rpioso: ajya: try https://meet.google.com/krd-kchc-ojk | 13:56 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 13:59 |
| iurygregory | now it should fail on other test (fingers crossed) | 14:00 |
| cenne | dtantsur: re your call for outreachy mentors; Do you think i could be a co-mentor? | 14:10 |
| cenne | If it's possible I'd like to think about it further and see if I can | 14:10 |
| dtantsur | cenne: it's an interesting question. I think you could try, but note that it's a quite significant time commitment. | 14:11 |
| dtantsur | and will require you to learn ironic deeper, all in your free time, unless you find a position that allows you to work on ironic | 14:11 |
| cenne | yes realized it'd be a big time commitment, that's why wanted to ask first if it seemed possible. | 14:13 |
| cenne | Oh, didn't think about the second part, haha. | 14:13 |
| dtantsur | yeah, that's the trickiest part. I assume you'll be busy with other things in your life the next half a year :) | 14:14 |
| dtantsur | but that's for you to understand and decide | 14:14 |
| cenne | Also, wanted to say... that torrent stuff is really interesting! | 14:14 |
| * cenne expresses interest about wanting to work on it, if it's not already alloted, provided there's some funding. | 14:14 | |
| cenne | Have only glanced over the spec on mobile (and the project-insanity link). Seems cool. | 14:14 |
| dtantsur | I guess I have the same answer: it will require months of work time. You need to evaluate if you'll be able to realistically do it. | 14:15 |
| cenne | Okay, I'll try reading through the spec proposal more thoroughly to get a better idea. | 14:18 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 14:23 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Remove images from the OS profiler docs https://review.opendev.org/c/openstack/ironic/+/807326 | 14:35 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add and document high-level helpers for async steps https://review.opendev.org/c/openstack/ironic/+/807295 | 14:35 |
| arne_wiebalck | FYI, https://bugs.launchpad.net/nova/+bug/1884217 may need some input from Ironic. | 14:42 |
| iurygregory | zzzeek, I've tested the approach you mentioned but still got the same failure =( the some CI jobs are still running but this is the log from the cover job | 14:42 |
| iurygregory | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_0c6/807278/5/check/openstack-tox-cover/0c69f9e/testr_results.html | 14:42 |
| zzzeek | iurygregory: there were a lot of places that had the problem, looking | 14:45 |
| iurygregory | yeah, but the test failed in the one I changed =( (at least I was expecting to work and see a failure in a different test) | 14:46 |
| * iurygregory wondering if I need to change all tests ... | 14:46 | |
| zzzeek | iurygregory: replied, you missed part of my change | 14:47 |
| iurygregory | zzzeek, oh ok o/ | 14:47 |
| iurygregory | oh I see! | 14:48 |
| * iurygregory facepalm | 14:48 | |
| zzzeek | :) | 14:48 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 14:50 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix RAID steps for non-RAID in idrac-redfish https://review.opendev.org/c/openstack/ironic/+/804112 | 14:54 |
| iurygregory | zzzeek, \o/ yay now the problem is in another test :D | 15:01 |
| iurygregory | going to try to fix all tests that have the same logic | 15:02 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/807326/ fixes the build for my patch | 15:02 |
| dtantsur | and also makes the PDF readable again | 15:02 |
| dtantsur | we may keep an eye on the PDF situation though.. it seems like we have no indication that it breaks until it breaks too much | 15:03 |
| iurygregory | \o/ | 15:10 |
| * iurygregory adds more coffee to his mug, so he can keep fixing the unit tests to not break oslo.db | 15:11 | |
| TheJulia | rpioso: word has it power state sync being turned on has at least resulted in success \o/ | 15:29 |
| rpioso | TheJulia: Great news! | 15:33 |
| rpioso | TheJulia: Thank you! \o/ | 15:34 |
| TheJulia | rpioso: I've requested another sosreport be supplied to us so we can at least dig through and see if we're seeing follow-up behavior on power state changes | 15:36 |
| rpioso | TheJulia: Cool! Should be interesting. We are continuing to dig into our findings, which, as you know, are somewhat different. | 15:45 |
| iurygregory | zzzeek, quick question if you are still around, this type of line should also be changed " nodes.select(nodes.c.uuid == data['uuid']).execute().first()" since it has the execute call ? | 15:46 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add uWSGI role and use systemd instead of emperor mode https://review.opendev.org/c/openstack/bifrost/+/806106 | 16:01 |
| arne_wiebalck | bye everyone, have a good weekend o/ | 16:26 |
| TheJulia | o/ | 16:26 |
| *** hjensas is now known as hjensas|afk | 16:29 | |
| dtantsur | was looking into switching ironic-api in bifrost from oslo.service to uWSGI (like in devstack), and suddenly started wondering if it's a good idea at all | 16:43 |
| dtantsur | uWSGI+Nginx is definitely a more stable combination, and we know oslo.service+eventlet to cause issues with TLS.. | 16:43 |
| dtantsur | .. but on the other hand, we have synchronous API pass-thrus to BMCs, which, in case of IPMI, can take minutes to respond. | 16:43 |
| dtantsur | so there is no safe processors+threads combination that cannot be DOSed with an API access very trivially | 16:44 |
| dtantsur | I guess I should spend my brain on something else on Friday evening :) | 16:47 |
| dtantsur | have a nice weekend folks! | 16:47 |
| TheJulia | heh | 16:48 |
| TheJulia | have a wonderful weekend dtantsur | 16:48 |
| TheJulia | and there are definitely trade-offs :( | 16:48 |
| dtantsur | TheJulia: maybe on Monday we should chat about stopping having synchronous requests in the API.. maybe.. | 16:51 |
| dtantsur | okay, now leaving for real | 16:51 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: [DNM] Testing fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 16:53 |
| TheJulia | dtantsur: holiday in the states on Monday | 17:01 |
| opendevreview | Merged openstack/ironic master: Allow initial versions to not be created yet https://review.opendev.org/c/openstack/ironic/+/804003 | 17:26 |
| TheJulia | woot | 17:35 |
| iurygregory | \o/ | 17:42 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 18:43 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 19:02 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Include ironic/tests in irrelevant-files https://review.opendev.org/c/openstack/ironic-inspector/+/807394 | 19:14 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 19:16 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 19:44 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 20:12 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 20:31 |
| TheJulia | stevebaker: so... looks like rhel9 grub builds only put a single efi binary in palce... to /boot | 20:37 |
| iurygregory | at least after 13 patches I finally got the sqlalchemy test_migrations in MySQL to work... now I need to figure out the failure in PostgreSQL \o/ | 20:49 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Facilitate asset copy for bootloader ops https://review.opendev.org/c/openstack/ironic/+/806815 | 20:49 |
| TheJulia | iurygregory: ugh :( | 20:49 |
| iurygregory | TheJulia, the error is the best part :D oslo_db.exception.DBError: (psycopg2.errors.InFailedSqlTransaction) current transaction is aborted, commands ignored until end of transaction block | 20:51 |
| TheJulia | oh jeeze | 20:51 |
| TheJulia | time for debug logging in sqlalchemy? | 20:52 |
| iurygregory | the best part is that I couldn't reproduce the issue locally, so I'm looking at CI logs to see if I can find something | 20:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add release note upgrade version check handling change https://review.opendev.org/c/openstack/ironic/+/807403 | 20:55 |
| TheJulia | I guess I need to do cycle highlights | 21:08 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Implements node history: database https://review.opendev.org/c/openstack/ironic/+/768009 | 21:09 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 21:10 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix to unblock oslodb 11.0.0 https://review.opendev.org/c/openstack/ironic/+/807278 | 21:41 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Record node history and manage events in db https://review.opendev.org/c/openstack/ironic/+/803292 | 21:58 |
| TheJulia | iurygregory: https://review.opendev.org/c/openstack/releases/+/807408 | 22:20 |
| TheJulia | Well, this is cute: https://github.com/openstack/ironic/pull/10 | 22:29 |
| tonyb | stevebaker: So I don't think I can set it to false '.. node unset --automated-clean' sets it to None, '.. node set --automated-clean' sets it to True | 22:34 |
| TheJulia | hmm, never used that so I don't remember mechanics around it but I thought explicit set to false was an option | 23:38 |
| TheJulia | tonyb: you mentioned elsewhere you were getting the wrong file handed out by dhcp.... I can't help but wonder if puppet-ironic is handing out the wrong templates or you need additional configuraiton. Because of the config model, it shims a lot of stuff into not great/ultimately incorrect defaults | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!