| *** rcastillo|rover_ is now known as rcastillo | 03:56 | |
| kamlesh6808c | Good morning Ironic! | 05:48 |
|---|---|---|
| opendevreview | Vanou Ishii proposed openstack/ironic master: [WIP] Deal with iRMC virtual media incompatibility https://review.opendev.org/c/openstack/ironic/+/823790 | 06:21 |
| kubajj | Good morning Ironic! | 07:10 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 08:23 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 08:25 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 09:03 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 09:32 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [WIP] Deal with iRMC virtual media incompatibility https://review.opendev.org/c/openstack/ironic/+/823790 | 09:47 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 10:25 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 10:25 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 10:39 |
| opendevreview | SONG SHUKUN proposed openstack/ironic stable/yoga: Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/850553 | 10:45 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 10:47 |
| opendevreview | SONG SHUKUN proposed openstack/ironic stable/yoga: Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/850553 | 10:57 |
| iurygregory | good morning Ironic | 11:03 |
| opendevreview | Merged openstack/sushy stable/ussuri: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/854804 | 12:40 |
| Nisha_Agarwal | iurygregory, hi | 13:06 |
| Nisha_Agarwal | dtantsur, hi ping | 13:06 |
| iurygregory | Nisha_Agarwal, hi | 13:07 |
| iurygregory | Dmitry is out today | 13:07 |
| Nisha_Agarwal | iurygregory, i raised proliantutils patch for some issue today and realized that proliantutils is broken due to updated sushy version | 13:07 |
| Nisha_Agarwal | and today is the last day for client ibraries release | 13:07 |
| Nisha_Agarwal | libraries* | 13:07 |
| iurygregory | oh wow | 13:08 |
| Nisha_Agarwal | the test cases are failing | 13:08 |
| iurygregory | can you send me the link for the patch so I can look at it? | 13:08 |
| Nisha_Agarwal | I dont think it is breaking on the functionality part | 13:08 |
| Nisha_Agarwal | https://review.opendev.org/c/x/proliantutils/+/855646 | 13:08 |
| iurygregory | so, sushy is not a client lib (it was released last week...) | 13:09 |
| iurygregory | client libs we have are python-ironicclient and the inspector one | 13:09 |
| Nisha_Agarwal | and proliantutils come under client library? | 13:09 |
| Nisha_Agarwal | or not? | 13:09 |
| Nisha_Agarwal | it is used by ironic ilo and ilo5 drivers | 13:10 |
| iurygregory | proliantutils we use as driver-requirements in ironic only no? | 13:11 |
| Nisha_Agarwal | Yes | 13:11 |
| iurygregory | let me double check | 13:11 |
| iurygregory | so, a few things I noticed you have py36 in proliantutils | 13:11 |
| iurygregory | this is not supported in Zed anymore | 13:11 |
| iurygregory | we dropped in the begin of the cycle | 13:11 |
| Nisha_Agarwal | i am actually addressing the comments for https://review.opendev.org/c/openstack/ironic/+/804486 | 13:12 |
| Nisha_Agarwal | means? | 13:12 |
| Nisha_Agarwal | ok | 13:12 |
| Nisha_Agarwal | py36 has to be removed? | 13:12 |
| Nisha_Agarwal | i can do that | 13:12 |
| iurygregory | yeah, you will need to update the jobs running on proliantutils gate =) | 13:12 |
| Nisha_Agarwal | ok | 13:12 |
| iurygregory | let me see the comments in the ironic patch to understand a bit more | 13:13 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/xena: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852802 | 13:24 |
| Nisha_Agarwal | iurygregory, if we release proliantutils by early next week and i update ironic patch https://review.opendev.org/c/openstack/ironic/+/804486 with correct proliantutils version, will https://review.opendev.org/c/openstack/ironic/+/804486 be considered for zed release | 13:36 |
| Nisha_Agarwal | ? | 13:36 |
| iurygregory | Nisha_Agarwal, let me check the dates | 13:36 |
| iurygregory | Nisha_Agarwal, yeah we should have time to merge it =) | 13:38 |
| Nisha_Agarwal | iurygregory, Thanks | 13:39 |
| iurygregory | I think the failures in the py39 job is valid in https://review.opendev.org/c/x/proliantutils/+/855646 , you probably need to update the unit tests in proliantutils and the zuul job to run zed tests | 13:40 |
| Nisha_Agarwal | iurygregory, Yes we are looking into it...and that happened i guess due to some update in sushy | 13:40 |
| iurygregory | yeah maybe is related to https://opendev.org/openstack/sushy/commit/8d55d51284288ef7cc3dad4dce00fe0155b1e6f1 | 13:41 |
| Nisha_Agarwal | iurygregory, may be ....it looks like very old commit(6 months back)...not sure | 13:43 |
| Nisha_Agarwal | iurygregory, it was passing earlier so i dont think this commit is related to our test case failure issue | 13:46 |
| iurygregory | humm let me see if I can find something | 13:47 |
| TheJulia | yay for over sleeping a little bit | 13:56 |
| opendevreview | Merged openstack/sushy stable/wallaby: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/854806 | 13:57 |
| TheJulia | Good morning everyone! | 13:59 |
| iurygregory | yay! | 14:04 |
| iurygregory | good morning TheJulia =) | 14:04 |
| TheJulia | I at least feel mostly rested... | 14:04 |
| TheJulia | Mostly. | 14:04 |
| TheJulia | iurygregory: https://storyboard.openstack.org/#!/story/2010267 | 14:12 |
| TheJulia | I've been thinking of ^^^ for the last couple of days, any thoughts? | 14:14 |
| Nisha_Agarwal | iurygregory, where is the zuul gates patch raised? | 14:17 |
| TheJulia | Nisha_Agarwal: the zuul config? | 14:18 |
| Nisha_Agarwal | TheJulia, hi....we need to remove the py36 gate for proliantutils so that any patch for proliantutils can merge in...and that is upstream gates....i am not able to recollect the project for raising this patch | 14:20 |
| JayF | Those changes are in the repo now, iirc | 14:20 |
| JayF | https://opendev.org/x/proliantutils/src/branch/master/.zuul.yaml#L1 | 14:21 |
| TheJulia | yeah, the vast majority of projects have it in repo, in a zuul.d folder or .zuul.yaml file | 14:21 |
| TheJulia | ++ | 14:21 |
| Nisha_Agarwal | TheJulia, JayF Thanks for the pointer | 14:22 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [WIP] Deal with iRMC virtual media incompatibility https://review.opendev.org/c/openstack/ironic/+/823790 | 14:25 |
| * TheJulia attempts to wake up | 14:27 | |
| kubajj | As my affiliation with CERN comes to an end next week, could I get a review for this TheJulia: https://review.opendev.org/c/openstack/ironic-python-agent/+/853182 ? The skipping RAIDs change needs to be changed (just slightly in test_hardware.py) and I would like to get it to a final form before leaving as I will have way less time for Ironic afterwards. | 14:32 |
| JayF | kubajj: too bad, it's been nice seeing your changes upstream. Feel free to stick around and say hello occassionally even if you won't be paid to contribute | 14:33 |
| kubajj | JayF: I think I will stick around. I will just be quite busy during the coming weeks as I need to go back from Switzerland and then start the academic year. | 14:34 |
| TheJulia | kubajj: fwiw, it is missing a release note, but I just workflowed it | 14:34 |
| JayF | oh yeah, good call, I shoulda seen that | 14:35 |
| JayF | kubajj: ah, makes sense. Good luck with your studies :) | 14:35 |
| TheJulia | if you could write something as a separte patch, or revise the current one really quickly, that would be awesome | 14:35 |
| TheJulia | kubajj: ^^ | 14:35 |
| JayF | we are both here so a quick release note won't slow down the landing | 14:35 |
| kubajj | TheJulia: I will try to write something up now | 14:36 |
| TheJulia | kubajj: Also, good luck on your studies :) Sorry for not getting back to that one after the mdadm raid utility weirdness it was failing on previously | 14:36 |
| kubajj | TheJulia: how fast do I need to add the release note (like if it is 5 minutes or more) | 14:39 |
| JayF | you have like an hour before the gate will land it | 14:40 |
| JayF | you can also set workflow -1 yourself if you want it in this patch and wanna guarantee it won't land | 14:40 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: Create RAIDs with volume name https://review.opendev.org/c/openstack/ironic-python-agent/+/853182 | 14:44 |
| kubajj | That should be the release note added ^^ | 14:45 |
| JayF | lookin | 14:49 |
| JayF | kubajj: can you add a period at the end of the sentence real quick? It's a nit, but you're here and it should just take a sec... | 14:51 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: Create RAIDs with volume name https://review.opendev.org/c/openstack/ironic-python-agent/+/853182 | 14:52 |
| iurygregory | kubajj, thanks for all your contributions and good luck on your academic year =) o/ | 14:52 |
| opendevreview | Merged openstack/sushy-oem-idrac master: Add OEM storage controller extension https://review.opendev.org/c/openstack/sushy-oem-idrac/+/851893 | 14:55 |
| JayF | kubajj: thanks, added my +2 so hopefully someone *cough* can land it | 14:58 |
| ajya | happy Friday, Ironic | 15:06 |
| ajya | can this backport https://review.opendev.org/c/openstack/ironic/+/852726 get reviewed and this chain of backports workflowed - https://review.opendev.org/q/Id66b87309dd26a2a165b35ac1d81580e4605d629 Thanks | 15:07 |
| iurygregory | ajya, done | 17:14 |
| TheJulia | sorry, been troubleshooting an environment | 17:33 |
| TheJulia | JayF: kubajj: w/r/t https://review.opendev.org/c/openstack/ironic-python-agent/+/852999 I've got a couple things. I'm going to at least add the task ID to the commit message and +2+A since otherwise it looks good to me | 17:44 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: SoftwareRAID: Enable skipping RAIDS https://review.opendev.org/c/openstack/ironic-python-agent/+/852999 | 17:45 |
| JayF | sgtm | 17:50 |
| kubajj | TheJulia: I will add the test for get volume name of RAID device with the logic that checks whether device on the skip list actually exists. | 18:23 |
| opendevreview | Merged openstack/ironic bugfix/20.2: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/852726 | 18:36 |
| opendevreview | Merged openstack/ironic bugfix/20.2: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852727 | 18:44 |
| opendevreview | Merged openstack/bifrost master: Fix TFTP read access for enforcing SELinux https://review.opendev.org/c/openstack/bifrost/+/849246 | 19:23 |
| opendevreview | Merged openstack/ironic-python-agent master: Create RAIDs with volume name https://review.opendev.org/c/openstack/ironic-python-agent/+/853182 | 19:26 |
| opendevreview | Merged openstack/bifrost master: make sure we quote extra_kernel_options https://review.opendev.org/c/openstack/bifrost/+/852877 | 19:31 |
| opendevreview | Merged openstack/ironic stable/xena: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852802 | 21:57 |
| opendevreview | Jay Faulkner proposed openstack/sushy stable/train: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/855555 | 22:06 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: SoftwareRAID: Enable skipping RAIDS https://review.opendev.org/c/openstack/ironic-python-agent/+/852999 | 22:08 |
| opendevreview | Jay Faulkner proposed openstack/sushy stable/victoria: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/855702 | 22:12 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Update releasenote for proper formatting https://review.opendev.org/c/openstack/ironic/+/855704 | 22:21 |
| JayF | ^ very minor formatting fix needed for a reno | 22:22 |
| TheJulia | LGTM, Thanks! | 22:24 |
| * TheJulia *really* needs to legacy policy bits | 22:32 | |
| JayF | TheJulia: what does that mean in english? | 22:34 |
| JayF | TheJulia: e.g. if you put it in a story and it's smaller than a breadbox it might magically get done lol | 22:34 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: add default conductor group capability https://review.opendev.org/c/openstack/ironic/+/855705 | 22:43 |
| TheJulia | JayF: ack, to do next week. also it is intertwined with all the rbac stuffs | 22:44 |
| TheJulia | Okay, I'm going to call it, my brain has filed a request for sleep() | 22:45 |
| opendevreview | Merged openstack/ironic master: Update releasenote for proper formatting https://review.opendev.org/c/openstack/ironic/+/855704 | 22:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!