| janders | good morning Ironic o/ | 00:01 |
|---|---|---|
| *** pmannidi is now known as pmannidi|brb | 00:08 | |
| *** pmannidi|brb is now known as pmannidi | 00:34 | |
| *** pmannidi is now known as pmannidi|brb | 01:24 | |
| opendevreview | Steve Baker proposed openstack/ironic stable/train: Fixes the issue that instance bond port can't get IP address https://review.opendev.org/c/openstack/ironic/+/782543 | 02:15 |
| *** pmannidi|brb is now known as pmannidi|Lunch | 03:27 | |
| *** pmannidi|Lunch is now known as pmannidi | 04:15 | |
| *** pmannidi is now known as pmannidi|brb | 04:39 | |
| iurygregory | good morning janders and Ironic o/ | 05:51 |
| *** pmannidi|brb is now known as pmannidi | 06:06 | |
| *** pmannidi is now known as pmannidi|brb | 06:22 | |
| *** rpittau|afk is now known as rpittau | 07:29 | |
| rpittau | good morning ironic! o/ | 07:29 |
| dtantsur | morning ironic | 07:30 |
| iurygregory | morning rpittau dtantsur o/ | 07:30 |
| janders | good morning iurygregory rpittau dtantsur and Ironic o/ | 07:35 |
| iurygregory | o/ | 07:35 |
| rpittau | hey iurygregory, janders :) | 07:37 |
| rpittau | my internet connection is terribly unstable today | 07:37 |
| rpittau | if anyone has a minute please check -> https://review.opendev.org/c/openstack/python-ironicclient/+/802324 | 07:38 |
| dtantsur | +A | 07:38 |
| iurygregory | rpittau, I will take a look now | 07:38 |
| dtantsur | too late, hahahahahahaha!!1 | 07:38 |
| rpittau | oO | 07:38 |
| * dtantsur is evil this morning | 07:38 | |
| iurygregory | dtantsur is awake for sure :D | 07:39 |
| rpittau | good morning evil dtantsur :) | 07:39 |
| dtantsur | :D | 07:39 |
| iurygregory | evil is implied :D | 07:39 |
| iurygregory | we are all evil :D | 07:40 |
| dtantsur | yay! | 07:41 |
| rpittau | there's difference between evil and crazy :D | 07:49 |
| dtantsur | not in my case | 07:50 |
| iurygregory | lol | 07:55 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix regression in ramdisk deploy kernel parameters https://review.opendev.org/c/openstack/ironic/+/802437 | 08:45 |
| opendevreview | Merged openstack/python-ironicclient master: Fix for missing fonts in PDF jobs https://review.opendev.org/c/openstack/python-ironicclient/+/802324 | 08:49 |
| opendevreview | Merged openstack/ironic master: Minor updates to anaconda doc https://review.opendev.org/c/openstack/ironic/+/802314 | 08:49 |
| dtantsur | would really appreciate a review on https://review.opendev.org/c/openstack/ironic/+/802437 | 08:52 |
| iurygregory | dtantsur, I will add to my list to look today | 08:56 |
| dtantsur | thx! | 08:56 |
| iurygregory | playing with Go atm so I don't want to do a context switch now hehe | 08:56 |
| dtantsur | oh, I can imagine | 08:57 |
| dtantsur | cenne: when debugging something unrelated, noticed that https://opendev.org/openstack/ironic/src/branch/master/ironic/conductor/utils.py#L1474 misses the node argument. Could you please fix it? | 09:03 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Update deprecated pxe_append_params -> kernel_append_params https://review.opendev.org/c/openstack/bifrost/+/802439 | 09:05 |
| * dtantsur -> grocery, brb | 09:06 | |
| cenne | dtantsur: on it | 09:10 |
| opendevreview | cenne proposed openstack/ironic master: Fixes missing argument for log format string https://review.opendev.org/c/openstack/ironic/+/802448 | 09:55 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost bugfix/11.1: Update .gitreview for bugfix/11.1 https://review.opendev.org/c/openstack/bifrost/+/802449 | 09:57 |
| cenne | thanks rpittau | 10:23 |
| rpittau | np | 10:24 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Remove optional attributes from insert_media https://review.opendev.org/c/openstack/sushy/+/802452 | 10:37 |
| opendevreview | Jacob Anders proposed openstack/sushy master: [WIP] Remove optional attributes from insert_media https://review.opendev.org/c/openstack/sushy/+/802452 | 10:38 |
| opendevreview | Merged openstack/python-ironicclient master: Add support for 'boot_mode' and 'secure_boot' node resource fields https://review.opendev.org/c/openstack/python-ironicclient/+/802026 | 11:34 |
| cenne | dtantsur: Regarding the failed (HPE .., ZuuL) builds https://review.opendev.org/c/openstack/ironic/+/802448 | 11:43 |
| cenne | Is that a server issue? | 11:43 |
| cenne | I am confused with both zuul and HPE not being happy. | 11:44 |
| dtantsur | cenne: I looked at the arm64 failures, they're a known issue with DB migration tests | 11:46 |
| dtantsur | the HPE failure (at least one of them) seems to be a problem with their ironic configuration: https://hpproliant.github.io/hpeproliant.github.io//logs/48/802448/1/check/hpe-gen9-dsvm-ironic-standalone-wholedisk-ilo-fw-update-direct-python3/953bb3b/job-output.json | 11:47 |
| dtantsur | in any case, if the main CI does not pass, your patch will not get merged :) | 11:47 |
| cenne | Oh cool. Thanks. | 11:50 |
| opendevreview | Merged openstack/bifrost bugfix/11.1: Update .gitreview for bugfix/11.1 https://review.opendev.org/c/openstack/bifrost/+/802449 | 12:16 |
| janders | see you tomorrow Ironic o/ | 12:45 |
| iurygregory | bye janders o/ | 12:47 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Increase version of hacking and pycodestyle https://review.opendev.org/c/openstack/sushy/+/802491 | 13:04 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Support credentials for virtual media https://review.opendev.org/c/openstack/sushy-tools/+/802493 | 13:08 |
| TheJulia | good morning | 13:12 |
| iurygregory | good morning TheJulia =) | 13:12 |
| cenne | Good morning TheJulia | 13:12 |
| dtantsur | morning TheJulia | 13:12 |
| opendevreview | Merged openstack/ironic master: Fixes missing argument for log format string https://review.opendev.org/c/openstack/ironic/+/802448 | 13:16 |
| iurygregory | dtantsur, TheJulia I've added a few comments with some ideas to the OpenInfra slides o/ I can work on it if we agree that it makes sense =) | 13:16 |
| dtantsur | cenne: could you cherry-pick this ^^^ to bugfix/18.1 now by using `git cherry-pick -x`? | 13:16 |
| cenne | dtantsur: okay | 13:17 |
| iurygregory | cherry-pick in gerrit UI won't work? :D | 13:17 |
| dtantsur | iurygregory: a bit of CLI practice won't hurt :) | 13:17 |
| iurygregory | true | 13:18 |
| dtantsur | but yeah, it probably will | 13:18 |
| dtantsur | my point here is to highlight the importance of -x | 13:18 |
| iurygregory | ++ | 13:18 |
| dtantsur | "The username to be used when accessing the URI specified by the Image parameter", says the Redfish schema. Let us try to guess if it means HTTP basic or digest auth :) | 13:19 |
| iurygregory | :D | 13:19 |
| iurygregory | maybe both? | 13:19 |
| dtantsur | it's actually possible | 13:20 |
| dtantsur | mraineri: if you're around ^^ | 13:20 |
| dtantsur | (this is re https://redfish.dmtf.org/schemas/v1/VirtualMedia.v1_3_0.json) | 13:20 |
| opendevreview | cenne proposed openstack/ironic bugfix/18.1: Fixes missing argument for log format string https://review.opendev.org/c/openstack/ironic/+/802523 | 13:28 |
| cenne | okay done! (I think..) | 13:29 |
| TheJulia | has anyone seen the name of the truck which ran me over? | 13:29 |
| TheJulia | well, the plate number to be precise? | 13:29 |
| iurygregory | woot?! | 13:31 |
| cenne | Thanks dtantsur. Using cherry-pick for the first time. | 13:35 |
| dtantsur | it's good to know it, a very important tool in the git workflow whenever support branches are involved | 13:36 |
| cenne | ack | 13:44 |
| mraineri | dtantsur: it really depends on what the remote file server is using for authentication | 13:45 |
| dtantsur | mraineri: so it should react normally for either request from the server? | 13:50 |
| mraineri | I would expect it to | 13:50 |
| dtantsur | okay, cool | 13:50 |
| dtantsur | oh lol, and just after this conversation I see https://www.monkeyuser.com/2021/fill-in-the-gaps/ | 13:51 |
| dtantsur | the timing is perfect :D | 13:51 |
| iurygregory | dtantsur, ++ | 13:53 |
| iurygregory | OMG | 13:53 |
| iurygregory | I didn't know MonkeyUser.com tks for sharing | 13:54 |
| dtantsur | oh, this is worth knowing | 13:54 |
| dtantsur | iurygregory: https://classicprogrammerpaintings.com/ is another recommendation | 13:55 |
| iurygregory | dtantsur, The code reviewer OMG | 13:55 |
| iurygregory | I can't stop laughing now | 13:56 |
| dtantsur | :) | 13:57 |
| rpittau | monkeyuser is the best, almost at xkcd level | 14:02 |
| dtantsur | yep. a bit less philosophical | 14:03 |
| * dtantsur sighs and starts refactoring bifrost | 14:04 | |
| opendevreview | cenne proposed openstack/ironic master: Add api endpoint for setting boot_mode on managed node https://review.opendev.org/c/openstack/ironic/+/800084 | 14:17 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Fixes missing argument for log format string https://review.opendev.org/c/openstack/ironic/+/802523 | 14:22 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Move Nginx code to a new role bifrost-nginx-install https://review.opendev.org/c/openstack/bifrost/+/802532 | 14:35 |
| dtantsur | the promised refactoring ^^ (I haven't touched keystone yet) | 14:40 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Move Nginx code to a new role bifrost-nginx-install https://review.opendev.org/c/openstack/bifrost/+/802532 | 14:57 |
| opendevreview | cenne proposed openstack/ironic master: Add api endpoint for setting boot_mode on managed node https://review.opendev.org/c/openstack/ironic/+/800084 | 15:11 |
| TheJulia | brraaaaaaiiinnsss | 16:24 |
| *** sshnaidm is now known as sshnaidm|afk | 16:34 | |
| rpittau | good rainy night! o/ | 16:40 |
| *** rpittau is now known as rpittau|afk | 16:40 | |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Clean up released features/items https://review.opendev.org/c/openstack/ironic-specs/+/801650 | 16:43 |
| TheJulia | forgot the toc entries :( | 16:44 |
| TheJulia | stevebaker: first of the backports to ussuri for vmedia permission seems happier with cI now, rechecked the second | 17:51 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/train: Make test-setup.sh compatible with mysql8 https://review.opendev.org/c/openstack/ironic/+/802460 | 17:59 |
| TheJulia | stevebaker: looks like mysql8 is breaking stable/train tox jobs :\ | 18:00 |
| TheJulia | Patch ^^^ backported also went to ussuri for same reason | 18:00 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/train: Make test-setup.sh compatible with mysql8 https://review.opendev.org/c/openstack/ironic/+/802460 | 18:30 |
| * TheJulia does the branch silently broke dance | 18:36 | |
| opendevreview | Julia Kreger proposed openstack/ironic stable/train: Make test-setup.sh compatible with mysql8 https://review.opendev.org/c/openstack/ironic/+/802460 | 18:52 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/train: Set driver lib tox jobs to utilize openstack-tox https://review.opendev.org/c/openstack/ironic/+/802603 | 18:58 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/victoria: Use openstack-tox for ironic-tox-unit-with-driver-libs https://review.opendev.org/c/openstack/ironic/+/802462 | 19:01 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/ussuri: Use openstack-tox for ironic-tox-unit-with-driver-libs https://review.opendev.org/c/openstack/ironic/+/802463 | 19:02 |
| TheJulia | Okay, that *should* fix things and prevent the version break later on | 19:03 |
| opendevreview | Merged openstack/ironic master: Fix regression in ramdisk deploy kernel parameters https://review.opendev.org/c/openstack/ironic/+/802437 | 19:07 |
| TheJulia | stevebaker: I guess on a plus side, I *think* we can invoke dnsmasq entirely by commandline which makes the puppet changes pointless | 19:15 |
| TheJulia | at least, for now | 19:15 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic bugfix/18.1: Fix regression in ramdisk deploy kernel parameters https://review.opendev.org/c/openstack/ironic/+/802464 | 19:19 |
| stevebaker | morning | 19:26 |
| stevebaker | TheJulia: yeah it might be nice not to have to manage another conf file | 19:26 |
| TheJulia | yeah | 19:53 |
| TheJulia | I guess it would definitely torpedo someone trying to use something like packstack on rhel9 | 20:25 |
| TheJulia | . o O ( maybe that is a good thing? ) | 20:26 |
| stevebaker | TheJulia: You could still package a unit service file which specifies all dnsmasq arguments, populated from env variables defined in /etc/sysconfig/foo. But then I suppose puppet needs to manage *that* | 20:36 |
| TheJulia | not really since the container is supposed to be launched with a direct process | 20:36 |
| TheJulia | I mean, we could, as long as the wrapper is there for process launching | 20:36 |
| stevebaker | I mean for the packstack case, not the tripleo/container one | 20:37 |
| TheJulia | oh yeah | 20:41 |
| TheJulia | but... we shouldn't be encouraging continuted packstack use | 20:42 |
| TheJulia | at least, that is my opinion | 20:42 |
| TheJulia | so it *looks* like it may make sense to get the packaging in because it should get pulled in... in theory | 20:44 |
| TheJulia | becasue there seems to be a non-visible direct connection between the manfiest executing to build the containers | 20:44 |
| TheJulia | like the pxe container has the conductor stuffs in it... it seems. | 20:44 |
| TheJulia | which would explain a lot and should get the dependencies in place | 20:45 |
| TheJulia | stevebaker: https://review.opendev.org/c/openstack/ironic/+/802603 <-- If you wouldn't mind before the end of the day so we can merge stuff down to train again | 21:14 |
| stevebaker | approved | 21:17 |
| stevebaker | TheJulia: here is the image building config for the ironic-pxe image https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/tcib/base/os/ironic-base/ironic-pxe/ironic-pxe.yaml | 21:22 |
| stevebaker | TheJulia: I think the directory structure implies this is also included https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/tcib/base/os/ironic-base/ironic-base.yaml | 21:22 |
| TheJulia | wow that is burried | 21:24 |
| TheJulia | thanks! | 21:24 |
| stevebaker | container building and running has already been completely rewritten since my time :) | 21:25 |
| NobodyCam | hehehe I've heard of self documenting code but this is not what I was expecting: https://docs.openstack.org/ironic/latest/_modules/ironic/drivers/modules/storage/cinder.html | 21:55 |
| JayF | https://docs.openstack.org/ironic/latest/_modules/ironic/drivers/modules/storage/external.html | 22:08 |
| JayF | it looks like a lot of modules are getting dropped directly into our docs | 22:08 |
| JayF | weird | 22:08 |
| opendevreview | Ghanshyam proposed openstack/networking-generic-switch stable/train: DNM: testing tempest change https://review.opendev.org/c/openstack/networking-generic-switch/+/802630 | 22:10 |
| opendevreview | Merged openstack/ironic stable/train: Set driver lib tox jobs to utilize openstack-tox https://review.opendev.org/c/openstack/ironic/+/802603 | 22:35 |
| *** pmannidi|brb is now known as pmannidi|AFK | 23:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!