| opendevreview | Steve Baker proposed openstack/bifrost master: Use distro conf loader_file_paths to copy ipxe/grub files https://review.opendev.org/c/openstack/bifrost/+/818709 | 01:20 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 01:20 |
| opendevreview | Harald Jensås proposed openstack/metalsmith master: Set binding:host_id on VIFs https://review.opendev.org/c/openstack/metalsmith/+/819743 | 01:20 |
| opendevreview | Harald Jensås proposed openstack/metalsmith master: Set binding:host_id on VIFs https://review.opendev.org/c/openstack/metalsmith/+/819743 | 01:21 |
| opendevreview | Harald Jensås proposed openstack/metalsmith master: Set binding:host_id on VIFs https://review.opendev.org/c/openstack/metalsmith/+/819743 | 01:23 |
| hgy_ | Good morning, Ironic! | 01:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 04:34 |
| hgy | Good afternoon, ironic! | 07:14 |
| arne_wiebalck | Good morning hgy and Ironic! | 07:27 |
| hgy | Good morning arne_wiebalck! | 07:31 |
| muellerbe | good morning arne_wiebalck o/ | 07:31 |
| arne_wiebalck | hey hgy and muellerbe o/ | 07:31 |
| janders | hey hgy arne_wiebalck muellerbe and Ironic o/ | 07:41 |
| muellerbe | hey janders o/ | 07:41 |
| rpittau | good morning ironic! o/ | 07:53 |
| muellerbe | good morning rpittau o/ | 07:54 |
| rpittau | hey muellerbe :) | 07:57 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Clarify driver load error message https://review.opendev.org/c/openstack/ironic/+/819678 | 08:12 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic/+/819764 | 08:44 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic/+/819764 | 08:58 |
| dtantsur | good morning folks | 08:59 |
| dtantsur | we may need to make ironic-tox-unit-with-driver-libs, there seems to be a breakage in proliantutils | 09:25 |
| dtantsur | stendulker: FYI ^^^ | 09:25 |
| dtantsur | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_3cc/819729/1/check/ironic-tox-unit-with-driver-libs/3cc5240/testr_results.html | 09:25 |
| dtantsur | * make <job> non-voting I mean | 09:25 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/bifrost/+/819780 | 09:41 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-inspector/+/819781 | 09:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-lib master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-lib/+/819782 | 09:44 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/819783 | 09:46 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-python-agent/+/819784 | 09:47 |
| opendevreview | Piotr Parczewski proposed openstack/bifrost master: Add support for root filesystem UUID customisation https://review.opendev.org/c/openstack/bifrost/+/819785 | 09:48 |
| opendevreview | Merged openstack/ironic stable/wallaby: Clarify driver load error message https://review.opendev.org/c/openstack/ironic/+/819588 | 09:51 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/metalsmith/+/819786 | 09:51 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-python-agent/+/819784 | 09:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-lib master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-lib/+/819782 | 09:54 |
| dtantsur | proliantutils fix is https://review.opendev.org/c/x/proliantutils/+/819787 | 09:55 |
| dtantsur | do we still want to make the job temporary non-voting? | 09:55 |
| rpittau | let's give it some time ? unless we have something urgent to merge | 09:59 |
| opendevreview | Riccardo Pittau proposed openstack/networking-baremetal master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/networking-baremetal/+/819792 | 10:02 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/networking-generic-switch/+/819793 | 10:04 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/python-ironicclient/+/819794 | 10:05 |
| opendevreview | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/819796 | 10:07 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/sushy/+/819797 | 10:08 |
| opendevreview | Riccardo Pittau proposed openstack/virtualbmc master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/virtualbmc/+/819798 | 10:09 |
| mnasiadka | Good morning | 10:15 |
| ajya | dtantsur: about idrac virtual media boot - during integration testing found that issue is still there for AMD systems :(. Checking with firmware team and based on that will see if need to wait for the fix to proceed with the patch. | 10:26 |
| arne_wiebalck | hey janders o/ | 10:27 |
| dtantsur | ajya: :( | 10:29 |
| opendevreview | Merged openstack/ironic stable/xena: Clarify driver load error message https://review.opendev.org/c/openstack/ironic/+/819587 | 11:43 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Stop exposing JSON RPC to the whole network https://review.opendev.org/c/openstack/bifrost/+/819649 | 12:09 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 12:10 |
| iurygregory | good morning Ironic o/ | 12:12 |
| dtantsur | hey-hey iurygregory! how is Brazil? | 12:13 |
| iurygregory | hey dtantsur o/, well very hot... =X | 12:13 |
| dtantsur | as expected :D | 12:13 |
| iurygregory | 28 feels like 32 :D | 12:13 |
| iurygregory | yeah | 12:13 |
| iurygregory | well it will be worst in January :D | 12:13 |
| dtantsur | it's not going to perfect in January here either :) | 12:27 |
| iurygregory | yeah, anything urgent that needs attention today? (I'm on PTO today, but tomorrow and Thursday I will be working :D) | 12:33 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Stop exposing JSON RPC to the whole network https://review.opendev.org/c/openstack/bifrost/+/819649 | 12:41 |
| dtantsur | iurygregory: don't work on PTO :) | 12:42 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 12:42 |
| iurygregory | :D | 12:49 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 13:19 |
| *** sshnaidm|afk is now known as sshnaidm | 13:29 | |
| opendevreview | Merged x/sushy-oem-idrac master: Fix moved tests https://review.opendev.org/c/x/sushy-oem-idrac/+/816641 | 13:44 |
| opendevreview | Merged openstack/metalsmith master: Set binding:host_id on VIFs https://review.opendev.org/c/openstack/metalsmith/+/819743 | 13:53 |
| opendevreview | Riccardo Pittau proposed x/sushy-oem-idrac master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/x/sushy-oem-idrac/+/819883 | 14:10 |
| TheJulia | good morning | 14:12 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Add destructive field flag to export configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/816815 | 14:16 |
| rpittau | good morning TheJulia :) | 14:20 |
| TheJulia | my brain doesn't want to wake up today | 14:29 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/xena: Set binding:host_id on VIFs https://review.opendev.org/c/openstack/metalsmith/+/819831 | 14:51 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/wallaby: Set binding:host_id on VIFs https://review.opendev.org/c/openstack/metalsmith/+/819832 | 14:51 |
| dtantsur | morning TheJulia | 15:20 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Terminate TLS on Nginx https://review.opendev.org/c/openstack/bifrost/+/819640 | 15:26 |
| dtantsur | oslo_messaging.exceptions.MessagingTimeout: Timed out waiting for a reply to message ID | 15:40 |
| dtantsur | is caused by my all-in-one patch, even though it doesn't change anything on this code path | 15:40 |
| dtantsur | I don't even... | 15:41 |
| dtantsur | and of course it cannot be reproduced locally | 15:41 |
| dtantsur | rpittau: a thought: if adding Stream 9 support to Bifrost is not crazily complex, we need to backport it to Xena. Then if Yoga breaks Stream 8, we still have an upgrade path. | 15:58 |
| rpittau | dtantsur: it doesn't look too crazy, it installs and run fine, just need to understand what's failing and why :/ | 16:00 |
| dtantsur | "what's failing and why" is sometimes the key question :) | 16:01 |
| rpittau | I expressed myself poorly, I know what's failing, I don't know why! :D | 16:05 |
| rpittau | but I'll remove the WIP from the patch because it seems good on installation and configuration | 16:05 |
| rpittau | well, ok, I'll leave WIP until the job is passing to avoid rerunnig the entire CI | 16:14 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 16:14 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 16:51 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM try to repair RPC https://review.opendev.org/c/openstack/ironic/+/819903 | 16:51 |
| rpittau | good night! o/ | 17:09 |
| dtantsur | folks, especially TheJulia, the whiteboard now has draft Yoga themes: https://etherpad.opendev.org/p/IronicWhiteBoard | 17:09 |
| NobodyCam | Top of the Not Monday Morning Ironic'ers | 17:12 |
| dtantsur | Not So Much Monday :) | 17:13 |
| dtantsur | good morning NobodyCam :) | 17:13 |
| NobodyCam | Hey hey dtantsur :) | 17:13 |
| NobodyCam | arne_wiebalck would love to see what your thoughts on https://review.opendev.org/c/openstack/nova/+/813897 are.. | 17:24 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/819783 | 17:35 |
| opendevreview | Merged openstack/sushy master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/sushy/+/819797 | 18:06 |
| opendevreview | Merged openstack/metalsmith master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/metalsmith/+/819786 | 18:19 |
| dtantsur | o/ | 18:21 |
| opendevreview | Merged openstack/ironic-python-agent master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/ironic-python-agent/+/819784 | 18:24 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Update jobs for yoga cycle https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/819321 | 18:24 |
| opendevreview | Merged openstack/python-ironicclient master: Test python 3.6 for distributions compatibility https://review.opendev.org/c/openstack/python-ironicclient/+/819794 | 18:24 |
| TheJulia | blarg drain | 18:47 |
| TheJulia | err, brain | 18:47 |
| stevebaker[m] | morning | 19:42 |
| janders | good morning Ironic o/ | 21:04 |
| arne_wiebalck | NobodyCam: this issues has hit us as well, to a level where we removed all that rebalance logic from our deployment a while ago | 21:07 |
| arne_wiebalck | NobodyCam: TheJulia: the hash ring rebalancing is a tricky area and I would not trust my review without testing it IRL tbh | 21:15 |
| arne_wiebalck | NobodyCam: TheJulia: testing it, however, is not that obvious either (in our deployment) | 21:16 |
| NobodyCam | oh interesting approach | 21:16 |
| NobodyCam | that was my concern I was thinking about a logic flag that could switch between old and new logic, but that could make it much more complicated | 21:18 |
| arne_wiebalck | NobodyCam: TheJulia: at the moment we're quite ok with the 1 n-c per conductor group setup, but I talked to Belmiro about it today, actually, and we will have a look if we can test it somehow ... | 21:18 |
| TheJulia | o'rly removed it all | 21:18 |
| arne_wiebalck | NobodyCam: TheJulia: I will leave a comment supporting the comments already made by you, Chris, and Arun confirming that this is an issue | 21:19 |
| arne_wiebalck | TheJulia: yes, caused too many issues | 21:19 |
| TheJulia | Yeah, I think we can all agree it is a pain point, just wanting to make sure we don't make things worse | 21:19 |
| TheJulia | sounds like things were pretty bad for you all at cern | 21:19 |
| arne_wiebalck | TheJulia: but, IIRC, this was before conductor groups | 21:19 |
| TheJulia | to excise it out... | 21:19 |
| arne_wiebalck | well, I think Belmiro had to clean up database a couple of times, and at some point we had to take a decision :) | 21:20 |
| arne_wiebalck | with a few thousand nodes it becomes painful | 21:21 |
| TheJulia | yeah, the manual db is what we want to avoid | 21:21 |
| * NobodyCam agrees | 21:21 | |
| TheJulia | we have a client that had to do it because they did some other work and ended up "migrating" to new controllers whilst also upgrading | 21:21 |
| NobodyCam | explode-y-head emoji | 21:22 |
| arne_wiebalck | the problem to test for us is to backport this to Stein/Train, then have a conductor group with a few hundred (prod?) instances, and then switch n-c on and off ... sounds like asking for trouble :) | 21:30 |
| arne_wiebalck | while the instances will probably be safe, we may need to clean up in the db if things do not work out | 21:31 |
| TheJulia | absolutely does sound like trouble | 21:37 |
| TheJulia | yeah | 21:38 |
| TheJulia | Well, it is all data points | 21:38 |
| arne_wiebalck | right ... we'll have a look at what we can do! | 21:53 |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 22:29 |
| opendevreview | Julia Kreger proposed openstack/python-ironicclient master: WIP: node history support https://review.opendev.org/c/openstack/python-ironicclient/+/819929 | 23:11 |
| *** pmannidi is now known as pmannidi|AFK | 23:19 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!