opendevreview | Steve Baker proposed openstack/sushy-tools master: Fix per-request memoization https://review.opendev.org/c/openstack/sushy-tools/+/840780 | 04:11 |
---|---|---|
opendevreview | Steve Baker proposed openstack/sushy-tools master: Document SUSHY_EMULATOR_ALLOWED_INSTANCES https://review.opendev.org/c/openstack/sushy-tools/+/840804 | 04:11 |
arne_wiebalck | Good morning, Ironic! | 06:08 |
rpittau | good morning ironic! o/ | 07:18 |
rpittau | what's better than start a Tuesday with a merge conflict? | 07:38 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Multipath Hardware path handling https://review.opendev.org/c/openstack/ironic-python-agent/+/837039 | 07:42 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/ironic-python-agent/+/841220 | 07:46 |
rpittau | lightning fast review ^ | 07:46 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Multipath Hardware path handling https://review.opendev.org/c/openstack/ironic-python-agent/+/837039 | 07:47 |
janders | good morning arne_wiebalck rpittau and Ironic o/ | 07:56 |
rpittau | hey janders :) | 07:57 |
opendevreview | Riccardo Pittau proposed openstack/sushy-tools master: Update jobs names https://review.opendev.org/c/openstack/sushy-tools/+/838664 | 08:47 |
kamlesh6808c | Good morning Ironic! | 08:49 |
dtantsur | could I get a 2nd +2 on https://review.opendev.org/c/openstack/sushy-tools/+/836795 https://review.opendev.org/c/openstack/sushy-tools/+/830157 https://review.opendev.org/c/openstack/sushy-tools/+/830598 please? | 09:48 |
kamlesh6808c | HI iurygreogory & Ironic team : Currently, I am working on raid cleaning step tempest implementation, This implementation is intended to work on physical baremetal servers . | 09:52 |
kamlesh6808c | In order to have full coverage of raid cleaning to test iDRAC's raid cleaning, I have been planning to have user input in tempest configuration file as json in string (ex . https://paste.openstack.org/show/bf0h7nEcoomy0m1bmv2C/) and get it validated with json schema. | 09:53 |
kamlesh6808c | My query is - Can we use json input as a string in config, validate and utilize inside raid_cleaning implementation of TC in ironic tempest plugin? | 09:53 |
opendevreview | Merged openstack/sushy-tools master: vmedia: keep the original URL in Image https://review.opendev.org/c/openstack/sushy-tools/+/836795 | 10:00 |
opendevreview | Merged openstack/sushy-tools master: Fewer mandatory methods in AbstractSystemsDriver https://review.opendev.org/c/openstack/sushy-tools/+/830157 | 10:05 |
opendevreview | Merged openstack/ironic-specs master: Fix Multiple top-level packages discovered in flat-layout https://review.opendev.org/c/openstack/ironic-specs/+/841162 | 10:30 |
opendevreview | Merged openstack/sushy stable/wallaby: Use Etag in precondition header when setting boot options https://review.opendev.org/c/openstack/sushy/+/840652 | 10:35 |
opendevreview | Merged openstack/sushy stable/wallaby: Attempt using Redfish settings resource to get URI https://review.opendev.org/c/openstack/sushy/+/840653 | 10:42 |
opendevreview | Merged openstack/sushy stable/wallaby: Follow up to Attempt using Redfish settings resource to get URI https://review.opendev.org/c/openstack/sushy/+/840654 | 10:43 |
opendevreview | Merged openstack/sushy stable/wallaby: Re-trying InsertMedia call with TransferProtocolType if required https://review.opendev.org/c/openstack/sushy/+/840655 | 10:47 |
iurygregory | good morning Ironic | 11:08 |
dtantsur | morning iurygregory | 11:09 |
janders | hey iurygregory o/ | 11:09 |
kamlesh6808c | hi iurygregory! | 11:10 |
arne_wiebalck | hey janders rpittau dtantsur kamlesh6808c o/ | 11:11 |
dtantsur | rpittau, iurygregory, could you check https://review.opendev.org/c/openstack/ironic-python-agent/+/840170 and https://review.opendev.org/c/openstack/ironic-lib/+/839949 please? | 11:13 |
dtantsur | this is for the multipath issue | 11:14 |
opendevreview | Merged openstack/ironic-tempest-plugin master: Remove six https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/841138 | 11:28 |
opendevreview | Merged openstack/ironic-inspector master: Fix CI https://review.opendev.org/c/openstack/ironic-inspector/+/840588 | 11:28 |
iurygregory | sure | 11:28 |
opendevreview | Merged openstack/virtualbmc master: Use zed jobs https://review.opendev.org/c/openstack/virtualbmc/+/840501 | 11:37 |
opendevreview | Merged openstack/networking-baremetal master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/networking-baremetal/+/840070 | 12:24 |
opendevreview | Riccardo Pittau proposed openstack/ironic-tempest-plugin master: Add yoga tests, stop testing victoria https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/840508 | 12:28 |
TheJulia | Good morning | 12:33 |
opendevreview | Merged openstack/networking-generic-switch master: Add support for switches running SONiC operating system https://review.opendev.org/c/openstack/networking-generic-switch/+/835912 | 12:43 |
dtantsur | morning TheJulia | 13:00 |
rpittau | good morning TheJulia :) | 13:00 |
dtantsur | rpittau: staging-drivers is such a mess.. I'll take your stestr addition into my patch | 13:01 |
rpittau | dtantsur: yeah, go ahead :/ | 13:02 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Upgrade from stable/yoga https://review.opendev.org/c/openstack/bifrost/+/839369 | 13:06 |
opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Do not try to guess EFI partition path by its number https://review.opendev.org/c/openstack/ironic-python-agent/+/840170 | 13:07 |
rpittau | iurygregory: the inspector CI fix has merged, if you have a minute today for the inspector release, otherwise let me know and I can take care of it :) | 13:22 |
iurygregory | rpittau, will do after the baremetal sig o/ | 13:22 |
iurygregory | if you want to push I will be happy to +1 | 13:23 |
rpittau | iurygregory: ack, I'll do it now | 13:23 |
iurygregory | ok o/ | 13:23 |
opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: CI: use pre-existing ssh key on multinode jobs https://review.opendev.org/c/openstack/networking-generic-switch/+/841265 | 13:24 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Multinode Grenade https://review.opendev.org/c/openstack/ironic/+/840530 | 13:26 |
TheJulia | I'm growing tired of multinode jobs | 13:29 |
TheJulia | Anyone have a dragon available for them? Preferable cute and that breaths fire | 13:30 |
iurygregory | I would love to have one also | 13:30 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent stable/xena: Use a pre-defined partition UUID to detect configdrive on GPT https://review.opendev.org/c/openstack/ironic-python-agent/+/840348 | 13:34 |
opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Fix CI https://review.opendev.org/c/openstack/python-ironicclient/+/841269 | 13:36 |
arne_wiebalck | Bare Metal SIG meeting in 25mins: iurygregory on xena/zed for Ironic ! | 13:36 |
TheJulia | \o/ | 13:37 |
TheJulia | For the first time in a long time, I don't have a conflicting meeting | 13:37 |
arne_wiebalck | \o/ | 13:37 |
rpittau | TheJulia: https://i.pinimg.com/originals/7a/61/b0/7a61b0f6fa8de54a352057db4852a10e.jpg | 13:38 |
TheJulia | rpittau: now the question is... how do we hire a dragon? | 13:38 |
rpittau | heh... how many gold coins, gems and magic itmes we have available? | 13:38 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.1: Use a pre-defined partition UUID to detect configdrive on GPT https://review.opendev.org/c/openstack/ironic-python-agent/+/840471 | 13:38 |
TheJulia | I... have non. But the RPG player in the house might... | 13:39 |
TheJulia | s/non./none./ | 13:39 |
rpittau | :) | 13:39 |
TheJulia | or I have Nan | 13:39 |
rpittau | lol | 13:39 |
dtantsur | rpittau: ImportError: cannot import name 'json' from 'werkzeug.wrappers' - what have we done to fix that? | 13:39 |
dtantsur | it's haunting bugfix branches on IPA | 13:40 |
rpittau | dtantsur: increase werkzeug version | 13:40 |
dtantsur | not acceptable for bugfix branches | 13:40 |
rpittau | errrr..... | 13:40 |
* dtantsur is curious if we're using the right u-c | 13:40 | |
rpittau | heh | 13:40 |
rpittau | I was going to say that | 13:40 |
dtantsur | we probably don't, lemme see | 13:41 |
rpittau | yep | 13:41 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.4: Use Yoga constraints for bugfix/8.4 https://review.opendev.org/c/openstack/ironic-python-agent/+/841290 | 13:42 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.3: Use Yoga constraints for bugfix/8.3 https://review.opendev.org/c/openstack/ironic-python-agent/+/841291 | 13:43 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.1: Use Xena constraints for bugfix/8.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/841293 | 13:44 |
dtantsur | rpittau: will hopefully help ^^ | 13:45 |
* rpittau writes down: remember to update u-c when creating bugfix branches | 13:45 | |
dtantsur | yup | 13:45 |
TheJulia | we should add that to the release docs | 13:46 |
rpittau | I have that open, I can update the page now | 13:48 |
TheJulia | dtantsur: replied to https://review.opendev.org/c/openstack/keystoneauth/+/841169 | 13:48 |
TheJulia | it is freaky that the pip resolver is freaking out on that repo | 13:48 |
rpittau | should I remove bifrost from the bugfix branches section at this point ? | 13:49 |
rpittau | since we're here | 13:50 |
dtantsur | yep | 13:50 |
TheJulia | +1 | 13:50 |
dtantsur | TheJulia: we could ask more people for their opinions on HTTP.. but I think dropping params is the way to go | 13:50 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Update bugfix section https://review.opendev.org/c/openstack/ironic/+/841294 | 13:54 |
rpittau | we will never forget again (probably) ^ | 13:54 |
opendevreview | Merged openstack/ironic-lib stable/wallaby: Use a pre-defined partition UUID to detect configdrive on GPT https://review.opendev.org/c/openstack/ironic-lib/+/839949 | 13:56 |
TheJulia | dtantsur: works for me :) either way basically things need to break if something is super funky | 13:57 |
TheJulia | and that way at least gives things a chance of working on redirect | 13:57 |
arne_wiebalck | Bare Metal SIG meeting in 2 mins! | 13:58 |
* dtantsur has a conflicting meeting | 13:59 | |
dtantsur | rpittau: the u-c change does not help. I wonder why only 3.9 is affected | 13:59 |
dtantsur | hmm, unconstrained on 3.9? https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L560-L561 | 14:00 |
rpittau | yeah that's where I was going too | 14:00 |
rpittau | kind of weird we have 3.6 and 3.9 but not 3.8 also | 14:01 |
* TheJulia noms breakfast and awaits awesome presentation | 14:01 | |
dtantsur | rpittau: I don't really understand why stable/yoga passes then | 14:03 |
dtantsur | or did we bump the version there? | 14:03 |
rpittau | dtantsur: we bumped the version in zed | 14:04 |
dtantsur | then I don't understand.. | 14:04 |
rpittau | I'm checking | 14:04 |
rpittau | dtantsur: in the latest stable/yoga job we have werkzeug 2.0.3, which does work, in bugfix/8.4 now we're instead using 2.1.2 which does not work | 14:10 |
dtantsur | how does it happen? u-c has 2.1 for 3.8+ and 2.0 for 3.6 | 14:10 |
rpittau | yep, trying to understand that | 14:10 |
dtantsur | ah, u-c for yoga only has 2.0 | 14:11 |
dtantsur | we may update the zuul conf to fetch yoga requirements | 14:11 |
rpittau | ah there you go | 14:11 |
dtantsur | I can do it after the meeting if you don't beat me to it | 14:11 |
rpittau | dtantsur: I don't remember if we can actually override the branch in a template, probably need to create a job based on that, I'll give that a try | 14:23 |
dtantsur | hmmmmmmmmm | 14:24 |
dtantsur | worth asking #opendev? | 14:24 |
rpittau | yeah | 14:24 |
arne_wiebalck | rpioso: we should take up the interop profiles again :) | 14:32 |
* iurygregory wondering if we should try to have a group dinner in Berlin :D (like we had in the mid cycle at CERN) | 14:46 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Only use findfs when looking up partitions by UUID https://review.opendev.org/c/openstack/ironic-python-agent/+/840499 | 14:47 |
dtantsur | iurygregory: we should. I wonder how many people will be there. | 14:47 |
rpittau | dtantsur: I'm going to hijack your ipa bugfix/8.4 patch to do a test | 14:48 |
dtantsur | rpittau, iurygregory, up for a quick review? https://review.opendev.org/c/x/ironic-staging-drivers/+/841248 | 14:48 |
dtantsur | sure | 14:48 |
iurygregory | should I send and email with a doodle to see? | 14:48 |
dtantsur | sounds good | 14:48 |
iurygregory | dtantsur, doing now | 14:48 |
iurygregory | oh we need to release ironic-staging-drivers for yoga... =X | 14:49 |
dtantsur | yeesssss | 14:49 |
dtantsur | completely missed that | 14:49 |
arne_wiebalck | thanks iurygregory for volunteering to present at today's SIG session! | 14:51 |
iurygregory | arne_wiebalck, np! | 14:51 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.4: Use Yoga constraints for bugfix/8.4 https://review.opendev.org/c/openstack/ironic-python-agent/+/841290 | 14:55 |
rpittau | mmm should I remove netboot instead of adding bios to the ironicclient tempest job ? | 15:03 |
dtantsur | rpittau: if possible - yes. netboot is going away. bios.. eventually as well :) | 15:05 |
rpittau | right | 15:05 |
rpittau | ok, let's change that then | 15:05 |
opendevreview | Riccardo Pittau proposed openstack/python-ironicclient master: Fix CI https://review.opendev.org/c/openstack/python-ironicclient/+/841269 | 15:06 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.4: Use Yoga constraints for bugfix/8.4 https://review.opendev.org/c/openstack/ironic-python-agent/+/841290 | 15:19 |
rpittau | ^ last test, then I'll just go for a new job based on the template and override the branch | 15:21 |
dtantsur | JFYI working on yoga for staging-drivers | 16:31 |
rpittau | I'll take the ipa fix again tomorrow, pragma does not work as expected | 16:41 |
rpittau | goodnight! o/ | 16:41 |
arne_wiebalck | bye everyone o/ | 16:42 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Ability to generate fake nodes https://review.opendev.org/c/openstack/bifrost/+/839796 | 16:44 |
dtantsur | JFYI https://review.opendev.org/c/x/ironic-staging-drivers/+/841309 will give us stable/yoga for staging drivers | 16:47 |
TheJulia | arne_wiebalck: w/r/t technology to enable remote forum discussions, that seems like it may be discussed among the foundation, the concern being that generally it has been very difficult for remote participation for such interactions. | 17:15 |
dtantsur | at the very least, will need a huge screen and a lot of mics | 17:34 |
dtantsur | see you tomorrow folks o/ | 17:34 |
JayF | TheJulia: counterpoint: it's very difficult for in-person participation at PTGs, too | 17:36 |
JayF | TheJulia: as someone with hearing problems, I have always had trouble participating in the PTG in person | 17:36 |
iurygregory | dtantsur, rpittau for the multipath issue in master the only missing bit is https://review.opendev.org/c/openstack/ironic-python-agent/+/837039 right? | 17:51 |
opendevreview | Vishal Manchanda proposed openstack/ironic-ui master: Use Horizon project template for nodejs jobs https://review.opendev.org/c/openstack/ironic-ui/+/841313 | 18:11 |
TheJulia | JayF: I am going to struggle a *lot* since I've got hearing damage too :( | 18:16 |
TheJulia | and won't be able to read lips | 18:16 |
JayF | I just don't want there to be some arbitrary threshold for "good enough" set by folks who think the in-person system is great | 18:16 |
JayF | because I've never really thought it was great tbh | 18:16 |
TheJulia | In person is more so about building cross-person context | 18:17 |
* TheJulia has been streaming videos on swordsmithing.... and may need to acquire a forge... | 18:17 | |
TheJulia | ... they may be occassionally distracting. | 18:25 |
opendevreview | Merged openstack/ironic stable/yoga: Exclude current conductor from offline_conductors https://review.opendev.org/c/openstack/ironic/+/839739 | 18:47 |
arne_wiebalck | TheJulia: right ... just got the reply that no remote participation is planned this time | 19:14 |
arne_wiebalck | rpioso: ^^ | 19:15 |
arne_wiebalck | the forum sessions will be added soon to the schedule | 19:15 |
TheJulia | arne_wiebalck: yeah, I talked to Jonathan about a half hour before that email was sent | 19:45 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Catch ismount not being handled https://review.opendev.org/c/openstack/ironic-python-agent/+/841333 | 19:47 |
stevebaker[m] | ah, I have a video to edit! | 20:09 |
* TheJulia wonders why multiple tests are failing for no reason on a backport | 20:25 | |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Ignore efi grub2-install failure https://review.opendev.org/c/openstack/ironic-python-agent/+/841275 | 21:16 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Ignore efi grub2-install failure https://review.opendev.org/c/openstack/ironic-python-agent/+/841275 | 21:30 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Catch ismount not being handled https://review.opendev.org/c/openstack/ironic-python-agent/+/841333 | 21:30 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Ignore efi grub2-install failure https://review.opendev.org/c/openstack/ironic-python-agent/+/841342 | 21:39 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Only mount the ESP if not yet mounted https://review.opendev.org/c/openstack/ironic-python-agent/+/841343 | 21:40 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Catch ismount not being handled https://review.opendev.org/c/openstack/ironic-python-agent/+/841344 | 21:40 |
opendevreview | Merged openstack/sushy-tools master: Add a fake system driver https://review.opendev.org/c/openstack/sushy-tools/+/830598 | 22:16 |
opendevreview | Merged openstack/sushy-tools master: Update jobs names https://review.opendev.org/c/openstack/sushy-tools/+/838664 | 23:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!