| rpittau | good morning ironic! o/ | 06:55 |
|---|---|---|
| ajya | Good morning, anyone looked at why this job has started failing - https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py36&project=openstack/sushy-tools ? Before I try to do that. | 07:34 |
| rpittau | ajya: it looks like an incompatibility between py3.6 and the coverage version in upper-constraints, not sure we can do much, I'll check if we can cap that in some way | 07:52 |
| rpittau | ajya: removing the lower bound breaks the requirements check btw | 07:53 |
| ajya | yes, there is one patch that tries to fix it along with other changes - https://review.opendev.org/c/openstack/sushy-tools/+/844284 | 07:54 |
| rpittau | yep, that needs to be reverted | 07:55 |
| ajya | I left the comment about it there | 07:55 |
| rpittau | ack | 07:55 |
| ajya | upper-constraints 2 weeks ago removed python3.6 specific line - coverage===6.2;python_version=='3.6' | 07:59 |
| rpittau | ajya: coverage removed support for python3.6 in 6.3, upper-constraints now uses 6.4.1, in yoga it was capped to 6.2 for py3.6 | 07:59 |
| ajya | after 6.2 coverage dropped 3.6 support | 07:59 |
| rpittau | yes because they dropped support for py3.6 | 07:59 |
| rpittau | zed does not support 3.6 at all | 07:59 |
| ajya | yeah, is there any way to fix this in sushy-tools without dropping 3.6.. | 07:59 |
| rpittau | heh if we need toi keep retro compatibility we have to find a way | 08:00 |
| rpittau | maybe a custom job | 08:00 |
| rpittau | let me see, we're not the only project keeping some stuff with py3.6 | 08:00 |
| ajya | ok, thanks | 08:00 |
| dtantsur | it may be a question for #openstack-requirements | 08:01 |
| dtantsur | we're not the only ones with branchless projects | 08:01 |
| rpittau | dtantsur: yep, IW as going to ask there :) | 08:01 |
| dtantsur | keeping in mind, their recommendation may end up dropping 3.6 and capping sushy-tools/vbmc on stable branches | 08:01 |
| rpittau | at some point we'll have to | 08:02 |
| rpittau | more and more libs are dropping py3.6 | 08:02 |
| dtantsur | yeah. we may issue the last release and Just Do It | 08:02 |
| rpittau | heh :/ | 08:06 |
| rpittau | it goes without saying that removing u-c the tests pass perfectly :/ | 08:39 |
| rpittau | dtantsur: ehmmm apparently projects with py36 are using separate py36 constraints, e.g. https://opendev.org/openstack/swift/src/branch/master/py36-constraints.txt | 08:42 |
| rpittau | how bad do we want to keep py36 support? | 08:43 |
| dmellado | does that even work? I couldn't even install on cs8 | 08:52 |
| dmellado | so... | 08:52 |
| dmellado | I'd say burn in hell xD | 08:52 |
| rpittau | always ready to splash napalm on something | 08:54 |
| dtantsur | rpittau: I don't think keeping sushy-tools or vbmc working on 3.6 is critical. | 09:37 |
| dtantsur | I hoped we could do it.. but if it takes additional effort, rather nope. | 09:37 |
| rpittau | dtantsur: I agree, flamethrower it is then | 09:37 |
| *** tosky_ is now known as tosky | 09:42 | |
| iurygregory | yay for jetlag (6:48...) | 09:48 |
| iurygregory | morning Ironic o/ | 09:48 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-tools master: Remove support for Python 3.6 and 3.7 https://review.opendev.org/c/openstack/sushy-tools/+/845914 | 09:52 |
| rpittau | hey iurygregory :) | 09:52 |
| iurygregory | hey hey rpittau =) | 10:00 |
| opendevreview | Riccardo Pittau proposed openstack/virtualbmc master: Remove support for Python 3.6 and 3.7 https://review.opendev.org/c/openstack/virtualbmc/+/845915 | 10:07 |
| opendevreview | Fiona Allen proposed openstack/ironic-inspector master: Removed blacklisted to blocked https://review.opendev.org/c/openstack/ironic-inspector/+/845919 | 10:41 |
| dtantsur | rpittau: do you plan on a couple of releases before we remove the support? | 11:12 |
| dtantsur | so that we can cap the stable branches to them? | 11:12 |
| dmellado | iurygregory: jetlag is AWESOME | 11:19 |
| dmellado | for me, getting 'back to east' is the worse ever | 11:19 |
| dmellado | xD | 11:19 |
| iurygregory | dmellado, NOOOOOOOOO =D | 11:19 |
| dmellado | getting back from Vancouver was... challenging xD | 11:20 |
| iurygregory | agree it would be good to pin some releases to support py36/py37 in sushy-tools | 11:20 |
| dmellado | +1 | 11:20 |
| iurygregory | dmellado, next year you can tell me =P | 11:20 |
| dmellado | lol | 11:21 |
| dmellado | a release note would be awesome, in any case | 11:21 |
| rpittau | dmellado: I realized I forgot the release note right after the push, I will add them | 11:33 |
| rpittau | dtantsur: yeah, I'll propose releases for both before merging the py36 drop | 11:34 |
| rpittau | even though if both of them are broken in py3.6 :) | 11:35 |
| *** tosky_ is now known as tosky | 11:55 | |
| dtantsur | rpittau: the code is not broken if used with the right constraints? | 12:07 |
| rpittau | dtantsur: no, I forgot to add "tests" in the phrase | 12:07 |
| dmellado | rpittau: heh, no worries! | 12:51 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-lib master: Resolve symlinks when comparing device names https://review.opendev.org/c/openstack/ironic-lib/+/845986 | 13:06 |
| dtantsur | a good idea that came out of the today's discussion ^^ | 13:06 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-lib master: Resolve symlinks when comparing device names https://review.opendev.org/c/openstack/ironic-lib/+/845986 | 13:07 |
| dtantsur | hmm, I need to think about implications for inspector.. | 13:08 |
| *** tosky__ is now known as tosky | 13:17 | |
| TheJulia | o/ | 13:26 |
| TheJulia | dtantsur: I don't think device names would change.. but I feel like the unit tests don't really provide a solid view into it | 13:27 |
| TheJulia | truthfully :( | 13:27 |
| *** tosky is now known as Guest2169 | 13:29 | |
| *** tosky__ is now known as tosky | 13:29 | |
| rpittau | TheJulia, dtantsur, iurygregory, the issue with swift in the grenade job seems not moving forward, definitely need advice as I doubt it depends on us, please check discussion in https://review.opendev.org/c/openstack/devstack/+/845675 | 13:48 |
| iurygregory | rpittau, ack | 13:49 |
| *** tosky__ is now known as tosky | 13:58 | |
| TheJulia | rpittau: ugh, yeah https://review.opendev.org/c/openstack/python-swiftclient/+/845157 is goign to be needed | 13:58 |
| TheJulia | hidden usage in python-swiftclient | 13:59 |
| rpittau | oh! thanks TheJulia :) | 13:59 |
| TheJulia | realistically, they shouldn't have pulled support out | 13:59 |
| rpittau | same thought | 13:59 |
| TheJulia | at least, before fixing tempurl client support | 13:59 |
| TheJulia | I can't seem to login right now, andI'm out of spoons to deal with things after traveling | 13:59 |
| TheJulia | so feel free to copy/paste my thoughts | 14:00 |
| TheJulia | I can chime in tomorrow, my immediate focus is tryign to get an appt with the eye doctor | 14:00 |
| rpittau | TheJulia: no worries and thanks again, I linked the swiftclient patch to the devstack backport and +1ed it | 14:01 |
| ameya49 | iurygregory: Hi! Hope you guys had a good time in Berlin :) | 14:19 |
| ameya49 | iurygregory : Requesting to go through few of these patches for review : https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/838462 and https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/843590. Can we please consider these patches for review this week | 14:22 |
| iurygregory | ameya49, Berlin was awesome =) | 14:24 |
| iurygregory | sure I'll add to the list | 14:24 |
| ameya49 | Thanks a ton! :) | 14:24 |
| rpittau | good night! o/ | 16:10 |
| iurygregory | rpioso, hey o/ in the R740 even with latest FW we still see the issue with the clear_job_queue, this is the output we have from redfish https://BMC/redfish/v1/Managers/iDRAC.Embedded.1 https://paste.opendev.org/show/baV3qJJmVCYE3NSutzds/ | 16:23 |
| iurygregory | The attribute Links/Oem/Dell/DellJobService is missing from the resource /redfish/v1/Managers/iDRAC.Embedded.1' | 16:23 |
| opendevreview | Samuel Zuk proposed openstack/ironic master: First implementation of the Redfish proxy for Ironic node power states. https://review.opendev.org/c/openstack/ironic/+/845639 | 16:57 |
| rpioso | ajya: ^^^ re: clear_job_queue issue reported by iurygregory. | 17:39 |
| cbouchar_ | dtantsur: Would you happen to have aarch template file to reference? | 18:22 |
| cbouchar_ | dtantsur: for use as grub.cfg / pxe_config_template_by_arch | 18:29 |
| cbouchar_ | dtantsur: I'm asking because 'default' or 'label' is an error in grub. Not sure of substitute if any. Doing PXE not iPXE BTW. | 18:35 |
| janders | rpioso ajya when I added support for verify steps, I put decorators around clear_job_queue, reset_idrac and known_good_state so this will run on any machine with a drac driver: https://review.opendev.org/c/openstack/ironic/+/804032/20/ironic/drivers/modules/drac/management.py. Is it expected to work only on certain iDRAC generations? Should this | 21:24 |
| janders | code only run conditionally, when Links/Oem/Dell/DellJobService? | 21:24 |
| janders | (this is further to iurygregory 's earlier comment) | 21:25 |
| janders | Sample error from an iDRAC8 with latest firmware: https://paste.openstack.org/show/bCDEhKlNTsQOY6Z1VhrT/ ( iurygregory likely already shared this with you) | 21:26 |
| janders | It would be great to hear your insights rpioso ajya | 21:27 |
| opendevreview | Steve Baker proposed openstack/metalsmith master: metalsmith list handle missing port https://review.opendev.org/c/openstack/metalsmith/+/846067 | 21:47 |
| iurygregory | janders, yeah I'm start to wonder if only iDRAC9 supports clear_job_queue | 21:56 |
| iurygregory | maybe we should try to add something that would check if we can find the Links/Oem/Dell/DellJobService =( | 21:57 |
| iurygregory | and if we get this response (ok we can ignore errors when running clear_job_queue) | 21:58 |
| opendevreview | Steve Baker proposed openstack/ironic master: Allocation candidates prefer matching name https://review.opendev.org/c/openstack/ironic/+/845856 | 22:33 |
| janders | iurygregory thinking along the same lines - just wondering if verify steps should handle it or the clear_job_queue step specifically. I think the latter, cause it can also be a clean/deploy step so if we fix it in the step itself we will fix it for all the workflows | 23:30 |
| janders | for now I am trying to reproduce this on bifrost, having moderate success | 23:30 |
| iurygregory | janders, I would say this would need to go in the clear_job_queue directly (like you said, it's also a clean/deploy step) | 23:31 |
| opendevreview | Steve Baker proposed openstack/ironic master: Docs: replace nova cli calls with openstack https://review.opendev.org/c/openstack/ironic/+/846072 | 23:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!