| opendevreview | Duc Truong proposed openstack/ironic master: Fix auth_protocol and priv_protocol for SNMP v3 https://review.opendev.org/c/openstack/ironic/+/875323 | 00:44 |
|---|---|---|
| arne_wiebalck | Good morning, Ironic! | 07:46 |
| rpittau | good morning ironic! o/ | 09:21 |
| rpittau | fungi: great news, I'm going to check asap :) | 09:21 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix enabling epel repo for rpm distributions https://review.opendev.org/c/openstack/bifrost/+/875929 | 11:04 |
| iurygregory | good morning Ironic | 11:19 |
| rpittau | JayF: I've added ironic-core to the virtualpdu-core and virtualpdu-release groups, so now we should all be able to approve virtualpdu patches | 11:19 |
| rpittau | I guess next steps would be to formalize the passage under ironic umbrella and move the repo under openstack organization | 11:19 |
| rpittau | hey iurygregory :) | 11:19 |
| * iurygregory just saw the virtualpdu email in openstack-discuss \o/ | 11:20 | |
| iurygregory | nice! | 11:20 |
| rpittau | with that said, if anyone has a moment please check this change to bring virtualpdu to Python 3 era :) https://review.opendev.org/c/x/virtualpdu/+/869983 | 11:21 |
| iurygregory | I checked yesterday :D | 11:21 |
| rpittau | and then we can request a release and move ironic CI to jammy :D | 11:21 |
| iurygregory | just added +2 | 11:21 |
| rpittau | thanks! | 11:21 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/876079 | 11:56 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 11:58 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 12:10 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Support batching up commands https://review.opendev.org/c/openstack/networking-generic-switch/+/743283 | 12:14 |
| samuelkunkel[m] | Hi | 12:48 |
| samuelkunkel[m] | is the blogpost about the arm deployment mentioned by the cern / arne_wiebalck already available? | 12:48 |
| samuelkunkel[m] | maybe I just did not see it | 12:48 |
| dtantsur | if it is, I want to see it as well :) | 12:49 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix enabling epel repo for rpm distributions https://review.opendev.org/c/openstack/bifrost/+/875929 | 12:57 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Migrate the inspector's /continue API https://review.opendev.org/c/openstack/ironic/+/875944 | 13:01 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 13:47 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-lib/+/876143 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-lib/+/876144 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/876145 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/876146 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/876147 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/876148 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/876149 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/876150 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/876151 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/876152 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/876153 | 14:13 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/sushy/+/876154 | 14:14 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/sushy/+/876155 | 14:14 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/sushy/+/876156 | 14:14 |
| opendevreview | Merged openstack/ironic master: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/874784 | 14:23 |
| TheJulia | good morning everyone | 14:28 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/875778 | 14:29 |
| TheJulia | rpittau: thanks for spotting that typo, revised ^ | 14:30 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/876161 | 14:33 |
| opendevreview | Merged openstack/python-ironic-inspector-client stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/876148 | 14:37 |
| opendevreview | Merged openstack/sushy stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/sushy/+/876154 | 14:46 |
| opendevreview | Merged openstack/metalsmith stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/876145 | 14:49 |
| opendevreview | Merged openstack/metalsmith master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/876147 | 14:49 |
| opendevreview | Merged openstack/python-ironicclient stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/876151 | 14:53 |
| opendevreview | Merged openstack/python-ironicclient stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/876152 | 14:53 |
| dtantsur | rpittau: could you follow-up on https://review.opendev.org/c/x/virtualpdu/+/869983 please? | 14:57 |
| rpittau | dtantsur: sure thing! thanks for the approval | 14:58 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/875778 | 15:02 |
| TheJulia | dtantsur: great catch on ^ | 15:03 |
| dtantsur | :) | 15:03 |
| opendevreview | Merged openstack/sushy stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/sushy/+/876155 | 15:03 |
| opendevreview | Merged openstack/metalsmith stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/metalsmith/+/876146 | 15:03 |
| opendevreview | Merged openstack/python-ironic-inspector-client master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/876150 | 15:04 |
| opendevreview | Merged openstack/ironic-lib master: Use direct writes when erasing the GPT https://review.opendev.org/c/openstack/ironic-lib/+/875931 | 15:04 |
| opendevreview | Merged openstack/ironic-lib stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-lib/+/876143 | 15:04 |
| opendevreview | Merged openstack/ironic-lib stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-lib/+/876144 | 15:04 |
| opendevreview | Merged openstack/python-ironicclient master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/876153 | 15:07 |
| opendevreview | Merged openstack/python-ironic-inspector-client stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/876149 | 15:24 |
| JayF | I need to do a prelude for Ironic release notes, right? | 15:26 |
| rpittau | dtantsur: follow up for vpdu https://review.opendev.org/c/x/virtualpdu/+/876169?forceReload=true | 15:27 |
| * JayF does a thing | 15:27 | |
| dtantsur | thx! | 15:29 |
| TheJulia | preludes are always good | 15:30 |
| rpittau | dtantsur: I left the catalog/messages still in setup.cfg, but I think we can remove them, I'll do another patch for that later | 15:31 |
| dtantsur | rpittau: I think we can safely remove all of the translation infra from it | 15:36 |
| rpittau | yeah | 15:36 |
| * dtantsur walk, brb | 15:54 | |
| opendevreview | Merged openstack/networking-generic-switch master: Support batching up commands https://review.opendev.org/c/openstack/networking-generic-switch/+/743283 | 15:56 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 16:23 |
| ebbex | is there a way to supply a json file to --config-drive ? i.e. openstack baremetal node deploy <node> --config-drive superduper_conf.json, it just feels so awful writing a string of json at the commandline. | 16:27 |
| TheJulia | ebbex: feels like your in use the client api territory. Is this a json payload to map to passing bulk json in on the api as https://docs.openstack.org/api-ref/baremetal/?expanded=change-node-provision-state-detail#id33 in the configdrive value? | 16:35 |
| *** xhku_ is now known as xhku | 16:37 | |
| TheJulia | ebbex: the code in the client only knows how to check if it is just a blob by the string | 16:39 |
| ebbex | Hmm, don't really want to be in that territory yet, are there any good examples on files to put in a directory you could supply for --config-drive ? I think I read that's a possibility, type `--config-drive /path/to/files` | 16:39 |
| TheJulia | ebbex: so... maybe --config-drive "$(cat superduper_conf.json)" ? | 16:39 |
| rpittau | good night! o/ | 16:40 |
| ebbex | i think if you just cat superduper_conf, you'd have to pipe it through sed to unescape the quotes (\") | 16:40 |
| TheJulia | a folder is a possibility, but I don't have any good examples | 16:41 |
| TheJulia | this seems like a "is this a file, is it a file of json" is a totally valid client patch | 16:41 |
| TheJulia | instead of trying to find a way to hack around this with bash | 16:41 |
| TheJulia | or $favoriteshell | 16:41 |
| TheJulia | I don't think it would expand if it is in single quotes, but I'm not sure if bash would expand it or not | 16:41 |
| TheJulia | JSON_CONTENT=$(cat /path/to/json) | 16:42 |
| TheJulia | --config-drive '${JSON_CONTENT}' might work and prevent expansion | 16:42 |
| ebbex | things in single-quotes wont expand in in bash i think | 16:45 |
| ebbex | * do variable expansion | 16:46 |
| ebbex | i'll look more into the directory approach first, tanks for the suggestions | 16:47 |
| JayF | can't you do @filename.json | 16:47 |
| jrosser | ebbex: i do similar in ansible with the command: module passing stuff like `"{{ node.raid_config | to_json }}"` | 16:47 |
| JayF | I thought you could @/path/to/file for --config-drive | 16:48 |
| JayF | it's possible that was a local change at a place I worked or the older client | 16:48 |
| TheJulia | ... this seems like a super simple thing to just add to the client | 16:49 |
| JayF | if what I said there doesn't exist; it should | 16:49 |
| TheJulia | given we already do file handling in the underlying client, we should just chekc any file if it is json too | 16:50 |
| TheJulia | a file is implied a whole pre-prepared binary image | 16:50 |
| JayF | oooh | 16:50 |
| ebbex | jrosser: thanks for the tip, I might do something like that when I get a bit further along :) | 16:55 |
| TheJulia | ebbex: fwiw, I believe referring to variables in the ${variable_name} style also helps prevent expansion. The added challenge though is that may prevent things for bash, but what about the underlying command line parser in python | 16:58 |
| jrosser | ebbex: i can share some ansible for provisioning baremetal nodes if you want some starting point | 16:58 |
| TheJulia | (the answer, dunno. I'd just check if a file is json first in python-ironicclient code and upload a patch :) | 16:58 |
| jrosser | but tbh i am quite surprised at how horrible it has turned out - very very far from the idempotence expected from ansible | 16:59 |
| ebbex | jrosser: yeah, I'd love to get some examples of configuration | 16:59 |
| jrosser | ebbex: this is `include_tasks:` per node https://paste.opendev.org/show/byKxnFyF9v0CD0fX54J8/ | 17:07 |
| ebbex | also, can I override the the kernel_params for a cloud-image? Specifically, I've supplied image_source to be AlmaLinux-8-GenericCloud-UEFI-8.7-20221111.x86_64.qcow2 and I want to override the `net.ifnames=0 biosdevname=0` part of the image. | 17:08 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add prelude for OpenStack 2023.1 Ironic release https://review.opendev.org/c/openstack/ironic/+/876195 | 17:13 |
| TheJulia | ebbex: most folks use diskimage-builder to repack the images (and also fix some semi-common issues) as well as make base changes, like such | 17:15 |
| ebbex | jrosser: cheers, you wouldn't happen to have a sample for the node.raid_confg aswell? | 17:18 |
| ebbex | TheJulia: yeah, I suppose I have some more work to get done :/ | 17:18 |
| TheJulia | ebbex: depends on needs :) Distro defaults are for like the 80% of users of the images | 17:19 |
| jrosser | can you use a regular qcow2 cloud image like that? | 17:21 |
| TheJulia | sometimes, depends on the image | 17:21 |
| TheJulia | some were not in such a shape, like original centos images back in the early centos8 days didn't properly handle UEFI | 17:22 |
| TheJulia | but they have since been fixed | 17:22 |
| TheJulia | most of the images are a single partition root with other partitions for /boot and /boot/efi | 17:23 |
| JayF | cloud images also might not include firmware for all possible hardwares, depending on *what* kind of cloud image you're using | 17:23 |
| TheJulia | the most common change is "do I pre-create a user" or "do I change how cloud-init will operate" | 17:24 |
| jrosser | ebbex: for a dell raid https://paste.opendev.org/show/bfGVibqpMbgqDkWfzy6E/ | 17:24 |
| jrosser | TheJulia: it is funny you mention cloud-init becasue this week we have nostly been victim to not having this https://github.com/canonical/cloud-init/commit/02202954c65a7a1cdb9b28703bd0af01edd0e091#diff-8e27dfc14b3ffceee1e29d3[…]0879febd47c578de04ea2dddR1265 | 17:25 |
| TheJulia | most are decent on the firmware, but definitely a case and some of those images can be absurd. We delete hundreds of megabytes of binary firmware blobs from IPA by default | 17:25 |
| TheJulia | oh, heh | 17:27 |
| TheJulia | I suspect other folks have not had the original patch which constrained it because I know people who have been doing metadata service usage w/physical metal for a while, but on centos/rh | 17:28 |
| jrosser | it seems that when we build an ubuntu user image with DIB its hardwired to the ec2 backend in cloud-init and 'just works' but when we build rocky9 similary it's not set up that way and fails | 17:28 |
| dtantsur | ebbex: I don't think I had problems with quotes and cat. lemme find a link | 17:29 |
| dtantsur | Cannot find a link, but I'm sure I've done it in the past | 17:30 |
| dtantsur | we could, of course, get explicit support in the client | 17:30 |
| dtantsur | which is complicated since we already accept an ISO | 17:33 |
| * dtantsur is curious if there is an easy way to detect an ISO file | 17:33 | |
| * dtantsur is about to write some scary code! | 17:35 | |
| jrosser | oh whilst i remember, is it possible to use ipmi serial-over-lan console when using the dell idrac driver? | 17:40 |
| dtantsur | jrosser: does not look so: https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/drac.py | 17:40 |
| dtantsur | I wonder why. Maybe nobody just thought about it | 17:41 |
| jrosser | it's like i want to say "this whole thing is idrac except console which is ipmitool" | 17:41 |
| dtantsur | yeah, this is what the console interface is for. but the idrac hardware type does not declare compatibility with ipmi interfaces | 17:41 |
| TheJulia | dtantsur: don't detect the iso file, we require it is also compressed afaik | 17:41 |
| TheJulia | so just look for gzip encoding | 17:42 |
| dtantsur | hmmmm | 17:42 |
| TheJulia | or first check for json | 17:42 |
| * TheJulia hides | 17:42 | |
| dtantsur | yeah, I don't want to end up doing json.loads() for a large ISO :D | 17:42 |
| TheJulia | ahh, trye | 17:42 |
| TheJulia | true | 17:42 |
| dtantsur | actually, I wonder if there is anything similar to the 'file' tool | 17:42 |
| dtantsur | in python | 17:42 |
| TheJulia | there might be | 17:42 |
| TheJulia | you only need to check the first few characters really | 17:43 |
| dtantsur | yeah, I may just check if configdrive[0] is b'{' | 17:43 |
| jlvillal | Hopefully easy question. Can I build aarch64 IPA images? Ultimate goal is trying to get Bifrost/Ironic working to be able to provision aarch64 hardware. But will run Bifrost/Ironic on x86_64. I'm assuming there are no pre-generated aarch64 IPA images. | 17:43 |
| TheJulia | anything that is far outside what we would expect in a json payload (even in utf-16) would be a strong hiint | 17:43 |
| TheJulia | s/hiint/hint/ | 17:43 |
| TheJulia | jlvillal: this came up on the openstack-discuss mailing list.... 2ish weeks ago | 17:44 |
| TheJulia | maybe 3 | 17:44 |
| dtantsur | jlvillal: wow, long time no see! | 17:44 |
| dtantsur | jlvillal: experimental images in https://tarballs.opendev.org/openstack/ironic-python-agent-builder/dib/files/ | 17:44 |
| dtantsur | not tested in the CI though | 17:44 |
| jlvillal | dtantsur, Hi! :) | 17:44 |
| jlvillal | TheJulia, Ah. Okay. I will check that out. | 17:44 |
| TheJulia | jlvillal: the tl'dr is you need to do the build from an arm machine to begin with | 17:44 |
| jlvillal | dtantsur, I will look at that too :) | 17:44 |
| TheJulia | but yeah, the experimental image | 17:44 |
| dtantsur | jlvillal: if you contribute multi-arch support for bifrost, I'll get another reason to buy you a drink the next time we meet | 17:44 |
| jlvillal | TheJulia, Oh I have plenty of ARM systems here. I work at Ampere ;) | 17:45 |
| dtantsur | wowowow nice! | 17:45 |
| jlvillal | dtantsur, hehe. I'll see what I can do :D | 17:45 |
| dtantsur | so we have actually chances of chatting with you more often again | 17:45 |
| TheJulia | dtantsur: I can also happily proxy that so you don't end up in the territory of owing tons of beer to jlvillal :) | 17:45 |
| dtantsur | TheJulia: https://pypi.org/project/filetype/ | 17:45 |
| TheJulia | jlvillal: awesome | 17:45 |
| TheJulia | so, I thought ampere had already sorted ironic based CI in their lab? | 17:46 |
| TheJulia | ... where is he whose name changes every few years hiding... | 17:46 |
| jlvillal | Possibly. My boss wants me to do a Proof-of-Concept using Bifrost/Ironic. So I am muddling my way through. I did get an x86_64 to be provisioned with Ubuntu. So progress is being made :) | 17:46 |
| TheJulia | jlvillal: ack, okay. It could also be a different group | 17:46 |
| jlvillal | TheJulia, I don't know about that. If they had I would like to know :) | 17:46 |
| jrosser | jlvillal: i have ampere systems here with ironic | 17:47 |
| jlvillal | Nice to see a lot of the gang still here :) | 17:47 |
| jlvillal | jrosser, Cool! Very cool. | 17:47 |
| jlvillal | jrosser, I'm assuming then you built your own IPA images then? | 17:48 |
| TheJulia | jlvillal: ping adam young :) | 17:48 |
| jrosser | yes i did, i had to find a free-tier VM somewhere to build on the right architecture to get bootstrapped | 17:48 |
| dtantsur | jrosser: you need an RPi for building images :) | 17:49 |
| * dtantsur is pondering the same | 17:49 | |
| jlvillal | jrosser, Excellent. Okay. I will try to do the same on a system here. Thanks! | 17:49 |
| jrosser | i think that was OK but not for a user image? one or the other failed on a pi | 17:49 |
| * TheJulia had to delineate the plural he whose name on irc changes every few years into separate people to remember | 17:49 | |
| jlvillal | jrosser, And very cool to see someone using Ampere systems in the wild. | 17:50 |
| jrosser | jlvillal: i have some notes here | 17:50 |
| jrosser | https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/867547/12/doc/source/configure-ironic-multiarch.rst | 17:50 |
| JayF | jrosser: jlvillal: Would you all be consumers of an upstream-published arm image (aarch64, I presume)? | 17:50 |
| jlvillal | jrosser, Thanks. | 17:51 |
| JayF | No promises; but it's on the list to discuss at bobcat ptg, and having specific customers for it would give it some boost in priority, I suspect | 17:51 |
| jrosser | for IPA? | 17:51 |
| jlvillal | JayF, I would if one existed. But will work on building my own. | 17:51 |
| JayF | jrosser: yep | 17:51 |
| dtantsur | I want to go as far as having a Bifrost CI job on arm64 | 17:51 |
| dtantsur | (not multi-arch, arm on arm) | 17:52 |
| JayF | that would be a requirement on the path to publishing an image | 17:52 |
| JayF | well, at least running an arm64 target | 17:52 |
| JayF | not neccessarily arm-on-arm but that would be cool | 17:52 |
| dtantsur | I also want(/need) to get a multi-arch metal3 | 17:52 |
| * JayF would prefer the metal3+sqlite job first, but that's just me | 17:52 | |
| jrosser | i think when i did this the experimental images for arm were not there? | 17:52 |
| JayF | jrosser: yep, we're considering adding them | 17:52 |
| jrosser | but i would have certainly started there if i could have done | 17:52 |
| JayF | jrosser: that's what I'm asking: if we picked that up as a task now, is it likely you / OSA would consume them? | 17:53 |
| jrosser | but things very rapidly turned into needing to enable a bunch of debug, so i had to build my own at that point anyway | 17:53 |
| dtantsur | I don't remember when we started publishing them. probably in the beginning of A | 17:53 |
| jrosser | i would really like to make an OSA CI job that actually tests ironic properly with virtualbmc | 17:53 |
| jrosser | becasue at the moment it's really just "did it deploy" and thats not good enough | 17:54 |
| jlvillal | TheJulia, Thanks for the pointer to the name. I just saw that in reading the back-scroll more thoroughly. I can't type and read at the same time... | 17:59 |
| TheJulia | jlvillal: no problem, I *think* he had to build ipxe binaries because he hit an actual bug somewhere along the way | 18:00 |
| TheJulia | I think that was the only hiccup he mentioned | 18:00 |
| jlvillal | Thanks! | 18:01 |
| opendevreview | Dmitry Tantsur proposed openstack/python-ironicclient master: [WIP] Accept configdrive as a JSON file https://review.opendev.org/c/openstack/python-ironicclient/+/876199 | 18:13 |
| dtantsur | TheJulia, ebbex, wdyt ^^^? | 18:13 |
| JayF | I think you're a freak of nature to churn out a WIP PR for that so fast | 18:15 |
| JayF | seriously impressive, if you can bottle that up and share it with me I'll take some :D | 18:16 |
| TheJulia | dtantsur: lgtm | 18:16 |
| TheJulia | well, reno and test needed :) | 18:16 |
| JayF | it looks pretty good to me | 18:16 |
| dtantsur | JayF: I'm pretty sure you can produce 30 lines of code within a similar timeframe, c'mon :) | 18:16 |
| dtantsur | it took me more to remember how to use timeit | 18:17 |
| JayF | dtantsur: depends on which 30 lines you want :) It would've taken me a while to get to the approach of looking for ' [{' and bailing if it's not that | 18:17 |
| JayF | translating "psuedocode" into actual-code is easy for me, coming up with what I want to do on a high level can be extremely difficult for me | 18:17 |
| dtantsur | JayF: speaking of fast typing.. I have logical error there | 18:21 |
| JayF | I'd like reviews on https://review.opendev.org/c/openstack/ironic/+/876195 -- release notes prelude | 18:22 |
| dtantsur | lemme finish this patch first | 18:29 |
| JayF | no real rush I think; but I'm not sure when it's due | 18:30 |
| opendevreview | Dmitry Tantsur proposed openstack/python-ironicclient master: Accept configdrive as a JSON file https://review.opendev.org/c/openstack/python-ironicclient/+/876199 | 18:35 |
| dtantsur | this should be it ^^^ | 18:35 |
| dtantsur | JayF: do you want to highlight the inventory API addition? | 18:36 |
| JayF | dtantsur: I was told none of that is user-facing at this point, it's not really any benefit to end users until it's done, right? | 18:36 |
| JayF | as opposed to the other things, which either expose anything at all to end users right now or improve their experience | 18:37 |
| dtantsur | JayF: it's in a bit of weird place. It's a new API, and it is working, but it is working by essentially proxying (with caching) to ironic-inspector | 18:38 |
| dtantsur | FWIW it's the same approach I'm proposing in https://review.opendev.org/c/openstack/ironic/+/875944 | 18:38 |
| JayF | dtantsur: I'm happy to include it then if you want to comment with some verbiage | 18:38 |
| dtantsur | JayF: thinking about it.. maybe we should not highlight the new API until we can boast a completely independent implementation of inspection | 18:39 |
| dtantsur | let's wait until then next release | 18:39 |
| JayF | that's sorta what I was thinking | 18:39 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/21.3: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876173 | 18:45 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/21.2: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876174 | 18:45 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/zed: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876175 | 18:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/21.0: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876176 | 18:47 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/20.2: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876177 | 18:47 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/yoga: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876178 | 18:47 |
| dtantsur | so many branches, so little time :) | 18:47 |
| dtantsur | see you tomorrow | 18:47 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/19.0: Do not move nodes to CLEAN FAILED with empty last_error https://review.opendev.org/c/openstack/ironic/+/876179 | 18:48 |
| dtantsur | (at least we'll see if the CI works on these branches, lol) | 18:48 |
| opendevreview | Ebbex proposed openstack/bifrost master: Use a more traditional ansible approach to include_vars https://review.opendev.org/c/openstack/bifrost/+/855806 | 18:53 |
| opendevreview | Ebbex proposed openstack/bifrost master: Refactor use of include_vars https://review.opendev.org/c/openstack/bifrost/+/855807 | 18:53 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add prelude for OpenStack 2023.1 Ironic release https://review.opendev.org/c/openstack/ironic/+/876195 | 18:53 |
| opendevreview | Merged openstack/sushy stable/yoga: Handle proper code_status in unit test https://review.opendev.org/c/openstack/sushy/+/868289 | 19:17 |
| JayF | rpittau: FYI; https://review.opendev.org/c/openstack/governance/+/876208 is the first of the chain of patches for us to adopt+rename that repo | 22:12 |
| JayF | the rename is likely not to happen for weeks due to ongoing infra activities and the release | 22:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!