| opendevreview | Merged openstack/ironic-python-agent stable/train: Reduce logging verbosity when collecting logs https://review.opendev.org/c/openstack/ironic-python-agent/+/796218 | 00:52 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 02:28 |
| *** pmannidi is now known as pmannidi|brb | 04:11 | |
| *** pmannidi|brb is now known as pmannidi | 04:54 | |
| iurygregory | good morning janders and Ironic o/ | 06:44 |
| arne_wiebalck | Good morning janders iurygregory and Ironic! | 06:44 |
| iurygregory | morning arne_wiebalck o/ | 06:44 |
| *** rpittau|afk is now known as rpittau | 07:39 | |
| rpittau | good morning ironic! o/ | 07:40 |
| iurygregory | morning rpittau o/ | 07:40 |
| rpittau | hey iurygregory :) | 07:40 |
| rpittau | JayF, TheJulia, I like the idea of having a container for that, maybe using podman and quay we may bypass some limitations of docker ? | 07:43 |
| iurygregory | you mean the download limit? (if I recall open source projects can avoid that, but I know tripleO had to move from docker upstream so maybe we should do something in the style that tripleO does) | 07:50 |
| dtantsur | morning ironic | 08:09 |
| * dtantsur had to take an unexpected PTO yesterday | 08:10 | |
| dtantsur | could someone please review https://review.opendev.org/c/openstack/ironic/+/799820 and https://review.opendev.org/c/openstack/ironic/+/799696 before we do bugfix releases? | 08:12 |
| cenne | good morning dtantsur. | 08:37 |
| cenne | morning ironic | 08:38 |
| dtantsur | o/ | 08:41 |
| iurygregory | morning dtantsur and cenne =) | 09:22 |
| cenne | morning iurygregory o/ | 09:23 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Add network step https://review.opendev.org/c/openstack/ironic-python-agent/+/799687 | 09:36 |
| opendevreview | Merged openstack/ironic master: Clean up images when ejecting an ISO with Redfish https://review.opendev.org/c/openstack/ironic/+/799820 | 10:20 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/wallaby: Clean up images when ejecting an ISO with Redfish https://review.opendev.org/c/openstack/ironic/+/800392 | 10:22 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/18.0: Clean up images when ejecting an ISO with Redfish https://review.opendev.org/c/openstack/ironic/+/800394 | 10:22 |
| opendevreview | Merged openstack/ironic master: Bring boot_iso/deploy_iso handling in iLO closer to Redfish https://review.opendev.org/c/openstack/ironic/+/799696 | 10:33 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Sanity check object version changes https://review.opendev.org/c/openstack/ironic/+/800252 | 12:07 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/18.0: Add missing BIOSSettings version mapping for older releases https://review.opendev.org/c/openstack/ironic/+/800625 | 12:08 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/wallaby: Add missing BIOSSettings version mapping for older releases https://review.opendev.org/c/openstack/ironic/+/800626 | 12:10 |
| dtantsur | cenne: something I completely forgot: we need to add support for new fields in the official client: https://opendev.org/openstack/python-ironicclient | 12:12 |
| dtantsur | and to https://opendev.org/openstack/openstacksdk/ which is already familiar to you :) | 12:12 |
| dtantsur | since the fields are read-only, there is not much to add, only a few places where the fields are listed and where the maximum supported API versions is specified. | 12:14 |
| dtantsur | see e.g. https://opendev.org/openstack/python-ironicclient/commit/a98cf2223230447a50f27f29fb77b5d235bb30fc (this one is not read-only) | 12:14 |
| dtantsur | TheJulia: FYI ubuntu doesn't seem to have snponly.efi, nor does openSUSE | 12:32 |
| dtantsur | I think we should at the very least fix our documentation | 12:33 |
| TheJulia | Oh ffs | 12:34 |
| dtantsur | even fedora doesn't seem to... | 12:34 |
| dtantsur | maybe it's only a RHEL/CentOS thing to ship it? | 12:34 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Update requirements https://review.opendev.org/c/openstack/ironic/+/800630 | 12:39 |
| dtantsur | TheJulia: filed https://bugzilla.redhat.com/show_bug.cgi?id=1981799 for Fedora. If you have some cases to back the proposal, please add them | 12:40 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix iPXE docs: snponly is not always available https://review.opendev.org/c/openstack/ironic/+/800632 | 12:44 |
| dtantsur | TheJulia: rewording ^^^ | 12:44 |
| rpittau | dtantsur: interesting, ubuntu 20.10 and 21.04 have snponly.efi in the ipxe pkg, but 20.04 doesn't | 12:44 |
| dtantsur | rpittau: yeah, people slowly start realising they need it | 12:44 |
| rpittau | heh I guess :/ | 12:45 |
| dtantsur | that's why I don't exclude Ubuntu completely in my docs patch | 12:45 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Remove unnecessary defaults from keystone https://review.opendev.org/c/openstack/bifrost/+/800633 | 12:46 |
| *** ricolin_ is now known as ricolin | 12:48 | |
| TheJulia | dtantsur: I could have sworn ubuntu does as well... guess only partially. | 13:00 |
| TheJulia | I guess all testing with ipxe and uefi on those platforms is with VMs as well.... | 13:00 |
| TheJulia | dtantsur: likely not virtualization folks but yeah, fedora should as well. It is *really* surprising they don't.... | 13:04 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 13:04 |
| dtantsur | janders: I've left some comments ^^ I suspect they're not on the previous patchset now | 13:05 |
| janders | thank you @dtantsur! :) | 13:07 |
| janders | the last patchset was mostly around splitting existing tests (and a couple small things) so the comments will still be 100% relevant | 13:09 |
| TheJulia | I feel like the ironicbaremetal blog needs a "why ipxe is important" post | 13:09 |
| janders | will go through these tomorrow | 13:09 |
| TheJulia | o/ janders | 13:09 |
| janders | hey @TheJulia o/ | 13:09 |
| janders | HexChat noob question: how do I properly tag people? :) | 13:09 |
| janders | (I dpm | 13:09 |
| janders | (I don't think ^ worked) | 13:10 |
| dtantsur | for me every mention will do | 13:10 |
| TheJulia | same | 13:10 |
| dtantsur | I suspect most clients are set this way | 13:10 |
| opendevreview | Merged openstack/ironic bugfix/18.0: Add missing BIOSSettings version mapping for older releases https://review.opendev.org/c/openstack/ironic/+/800625 | 13:10 |
| dtantsur | (I have dtantsur highlighted even if my current nick is not dtantsur) | 13:10 |
| TheJulia | dtantsur++ | 13:13 |
| janders | cool! thanks @TheJulia @dtantsur | 13:13 |
| janders | see you tomorrow Ironic o/ | 13:14 |
| dtantsur | g'night janders | 13:20 |
| TheJulia | odds of me waking up this morning? | 13:24 |
| dtantsur | -2 | 13:26 |
| TheJulia | :( | 13:26 |
| arne_wiebalck | Bare metal SIG meeting in 30 mins featuring TheJulia with an intro to Bifrost ! | 13:30 |
| * arne_wiebalck helps waking up TheJulia :-D | 13:30 | |
| TheJulia | heh | 13:30 |
| TheJulia | and of course, I spilled my water glass shifting because of light coming in the window | 13:39 |
| TheJulia | As a reminder, I believe we decided we would put on the review jam this morning because of the bare metal sig meeting | 13:50 |
| rpittau | TheJulia: would the review jam be at 2 or 3 pm UTC? | 13:57 |
| arne_wiebalck | Bare metal SIG meeting starts in about 1 minute! https://etherpad.opendev.org/p/bare-metal-sig | 13:58 |
| trandles | nice talk TheJulia | 14:24 |
| TheJulia | trandles: thanks | 14:24 |
| TheJulia | rpittau: it would have bene 2-3 pm utc | 14:24 |
| rpittau | thanks! | 14:24 |
| TheJulia | trandles: I wanted to keep it high level and introduce the concepts, kind of like a teaser of what can be done | 14:25 |
| trandles | I still haven't used bifrost-cli so the intro was good. | 14:25 |
| dtantsur | it's relatively new | 14:27 |
| TheJulia | rpittau: do you have patches we need to jam on? | 14:28 |
| rpittau | TheJulia: not at the moment, at least not from me, thanks :) | 14:29 |
| TheJulia | does anyone? I have a half hour free and I can hold the 5pm UTC review jam | 14:30 |
| dtantsur | oh ffs, the bifrost upgrade job still doesn't do what it's supposed to do..... | 15:14 |
| rpittau | dtantsur: have you seen my patch for the ansible part btw ? | 15:17 |
| dtantsur | ah, no. commented. | 15:19 |
| dtantsur | this is not my problem though: my problem is that ironic's source code is not updated. even though it is. | 15:19 |
| TheJulia | uhh.. hmm | 15:20 |
| rpittau | oO | 15:21 |
| dtantsur | I've seen this problem once, I hoped it went away... but it didn't | 15:26 |
| dtantsur | I don't know what to do other than declaring bifrost upgrades not supported... | 15:26 |
| TheJulia | is the install actually occuring? It did a long time ago because I broke my db and on upgrade the db upgrade failed as one would expect | 15:27 |
| dtantsur | it's a normal bifrost run, so I guess so? | 15:27 |
| dtantsur | I mean, there is no specific upgrade code | 15:27 |
| *** mgoddard- is now known as mgoddard | 15:27 | |
| TheJulia | ... umm... weird | 15:28 |
| TheJulia | could it be a nature of the CI path override? | 15:28 |
| TheJulia | well, there is the dbsync upgrade in the script | 15:29 |
| dtantsur | Successfully uninstalled ironic-16.0.4.dev38\nSuccessfully installed ironic-18.0.1.dev105 | 15:31 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Perform upgrade by default when installing ansible https://review.opendev.org/c/openstack/bifrost/+/800461 | 15:32 |
| dtantsur | the version is correct. but then the old ironic is launched from *somewhere* | 15:32 |
| arne_wiebalck | TheJulia: thanks again for the bifrost intro earlier today, was well received I believe! | 15:33 |
| * arne_wiebalck had to join another meeting immediately after | 15:33 | |
| dtantsur | it even gets correct new entry points, but not new code... | 15:34 |
| TheJulia | dtantsur: ugh *blink* *blink* | 15:36 |
| dtantsur | exactly | 15:36 |
| dtantsur | (it's not helping that we're attempting a skip-version upgrade, but this is easily fixable) | 15:38 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: CI: upgrade from Wallaby https://review.opendev.org/c/openstack/bifrost/+/800671 | 15:40 |
| dtantsur | I'm afraid this ^^ will hide the problem again.. | 15:40 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 15:42 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Allow configuring enabled vendor interfaces https://review.opendev.org/c/openstack/bifrost/+/799837 | 15:42 |
| opendevreview | Merged openstack/ironic stable/wallaby: Add missing BIOSSettings version mapping for older releases https://review.opendev.org/c/openstack/ironic/+/800626 | 15:52 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector master: Update requirements https://review.opendev.org/c/openstack/ironic-inspector/+/800680 | 16:14 |
| rpittau | good (rainy) night! o/ | 16:23 |
| *** rpittau is now known as rpittau|afk | 16:23 | |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 16:25 |
| dtantsur | o/ | 16:58 |
| JayF | o/ | 16:58 |
| JayF | we doing a review jam? | 16:58 |
| -opendevstatus- NOTICE: Depends-On using https://review.opendev.org URLs are currently not working. This was due to a config change in Zuul that we are reverting and will be restarting Zuul to pick up. | 17:41 | |
| TheJulia | my current call from before is still running :\ | 17:51 |
| janders | good morning Ironic o/ | 20:40 |
| opendevreview | Merged openstack/sushy-tools master: Update min version of tox to use allowlist https://review.opendev.org/c/openstack/sushy-tools/+/796400 | 21:25 |
| opendevreview | Merged openstack/ironic master: Update requirements https://review.opendev.org/c/openstack/ironic/+/800630 | 21:27 |
| opendevreview | Merged openstack/ironic bugfix/18.0: Clean up images when ejecting an ISO with Redfish https://review.opendev.org/c/openstack/ironic/+/800394 | 21:27 |
| opendevreview | Merged openstack/sushy master: Update min version of tox to use allowlist https://review.opendev.org/c/openstack/sushy/+/796399 | 21:32 |
| opendevreview | Merged openstack/ironic stable/wallaby: Clean up images when ejecting an ISO with Redfish https://review.opendev.org/c/openstack/ironic/+/800392 | 21:52 |
| opendevreview | Merged openstack/ironic master: Fix iPXE docs: snponly is not always available https://review.opendev.org/c/openstack/ironic/+/800632 | 21:52 |
| opendevreview | Verification of a change to openstack/ironic failed: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic/+/800465 | 21:55 |
| NobodyCam | Good Afternoon Ironic folks! | 22:21 |
| opendevreview | Merged openstack/ironic-python-agent master: Burn-in: Add network step https://review.opendev.org/c/openstack/ironic-python-agent/+/799687 | 22:46 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Use get_service_clients framework with basic Secure RBAC https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/797521 | 22:46 |
| opendevreview | Merged openstack/ironic master: Sanity check object version changes https://review.opendev.org/c/openstack/ironic/+/800252 | 22:46 |
| NobodyCam | oh did I miss the operator feedback session :( | 23:14 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!