| TheJulia | iurygregory: much appreciated | 00:02 |
|---|---|---|
| *** ociuhandu has joined #openstack-ironic | 00:04 | |
| * TheJulia rechecks the single lone outtstanding patch :( | 00:05 | |
| TheJulia | mirror returned 403s on packages | 00:06 |
| TheJulia | :( | 00:06 |
| *** uzumaki has joined #openstack-ironic | 00:12 | |
| *** ociuhandu has quit IRC | 00:12 | |
| *** cdearborn has quit IRC | 00:40 | |
| *** k_mouza has joined #openstack-ironic | 00:44 | |
| *** threestrands has joined #openstack-ironic | 00:45 | |
| *** k_mouza has quit IRC | 01:08 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: CI: Temporary disable grenade job for Ussuri release https://review.opendev.org/724205 | 01:15 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add ironic-python-agent-builder to grenade projects https://review.opendev.org/724206 | 01:20 |
| TheJulia | I guess we'll see ^^^ | 01:21 |
| TheJulia | because grenade sadness | 01:21 |
| TheJulia | dtantsur|afk: yeah, they changed the grenade config defaults and now things are trying to pull stable/ussuri | 01:32 |
| * TheJulia sighs | 01:32 | |
| *** tzumainn has quit IRC | 01:36 | |
| *** ociuhandu has joined #openstack-ironic | 01:50 | |
| *** Qianbiao has joined #openstack-ironic | 01:50 | |
| *** ociuhandu has quit IRC | 02:03 | |
| *** rh-jelabarre has quit IRC | 02:29 | |
| *** rh-jelabarre has joined #openstack-ironic | 02:30 | |
| *** gyee has quit IRC | 02:52 | |
| *** mkrai has joined #openstack-ironic | 02:55 | |
| *** uzumaki has quit IRC | 02:56 | |
| *** uzumaki has joined #openstack-ironic | 02:56 | |
| *** ianychoi_ has joined #openstack-ironic | 03:08 | |
| *** ianychoi has quit IRC | 03:10 | |
| *** ociuhandu has joined #openstack-ironic | 03:39 | |
| *** jdandrea has quit IRC | 03:41 | |
| *** ociuhandu has quit IRC | 03:53 | |
| *** rh-jelabarre has quit IRC | 03:59 | |
| TheJulia | fun factoid, we have 6300 unit tests | 04:00 |
| *** threestrands has quit IRC | 04:17 | |
| *** mkrai has quit IRC | 04:31 | |
| *** mkrai has joined #openstack-ironic | 04:32 | |
| *** ociuhandu has joined #openstack-ironic | 04:39 | |
| *** ociuhandu has quit IRC | 04:48 | |
| *** gryf has joined #openstack-ironic | 05:07 | |
| *** bnemec has quit IRC | 05:15 | |
| *** mkrai_ has joined #openstack-ironic | 05:37 | |
| *** mkrai has quit IRC | 05:39 | |
| *** uzumaki has quit IRC | 05:42 | |
| *** Qianbiao has quit IRC | 05:43 | |
| *** mkrai_ has quit IRC | 05:51 | |
| *** mkrai has joined #openstack-ironic | 05:51 | |
| *** mkrai has quit IRC | 05:53 | |
| *** mkrai has joined #openstack-ironic | 05:53 | |
| *** ociuhandu has joined #openstack-ironic | 06:01 | |
| *** Qianbiao has joined #openstack-ironic | 06:07 | |
| *** ociuhandu has quit IRC | 06:15 | |
| openstackgerrit | yuanliu proposed openstack/ironic master: If the CONF.conductor.XXX_timeout is less than 0，disable periodic task https://review.opendev.org/723795 | 06:29 |
| *** rpittau|afk is now known as rpittau | 06:32 | |
| rpittau | good morning ironic! o/ | 06:32 |
| *** ociuhandu has joined #openstack-ironic | 06:36 | |
| arne_wiebalck | Good morning, rpittau and ironic! | 06:38 |
| rpittau | hey arne_wiebalck :) | 06:39 |
| openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Set flake8-import-order version to 0.17.1 https://review.opendev.org/721657 | 06:40 |
| openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721653 | 06:46 |
| *** dustinc has quit IRC | 06:53 | |
| *** ociuhandu has quit IRC | 06:59 | |
| openstackgerrit | Riccardo Pittau proposed openstack/networking-baremetal master: Convert networking-baremetal job to dib https://review.opendev.org/718697 | 07:00 |
| *** chihhsin has quit IRC | 07:03 | |
| *** chihhsin has joined #openstack-ironic | 07:07 | |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/723823 | 07:10 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Temporary add per-file-ignores to flake8 tests https://review.opendev.org/723886 | 07:13 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix 3rd party driver mocks and related tests https://review.opendev.org/724080 | 07:14 |
| iurygregory | good morning ironic rpittau arne_wiebalck o/ | 07:15 |
| rpittau | hey iurygregory :) | 07:16 |
| *** yolanda has joined #openstack-ironic | 07:19 | |
| zigo | Can someone help me with Construct 2.10? I can't find which is the replacement for the Embedded() method/function. | 07:22 |
| zigo | The incompatibility prevents me from uploading the package to Debain... | 07:23 |
| iurygregory | zigo, if I got right there is no replacement.... | 07:24 |
| zigo | iurygregory: So what should be done then? | 07:24 |
| iurygregory | I'm looking at the code and i will give a try to update the constraints and change code | 07:24 |
| zigo | What was the function for? | 07:24 |
| iurygregory | I think at that time it was something necessary for the declaration | 07:26 |
| *** mkrai has quit IRC | 07:26 | |
| iurygregory | now it's not | 07:27 |
| *** mkrai has joined #openstack-ironic | 07:27 | |
| iurygregory | https://construct.readthedocs.io/en/latest/basics.html#nesting-and-embedding | 07:27 |
| *** rcernin has quit IRC | 07:27 | |
| *** ociuhandu has joined #openstack-ironic | 07:29 | |
| arne_wiebalck | iurygregory: o/ | 07:33 |
| *** mkrai has quit IRC | 07:42 | |
| *** mkrai has joined #openstack-ironic | 07:45 | |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Stop configuring install_command in tox. https://review.opendev.org/720793 | 07:47 |
| *** mkrai has quit IRC | 08:01 | |
| *** mkrai_ has joined #openstack-ironic | 08:01 | |
| *** lucasagomes has joined #openstack-ironic | 08:16 | |
| iurygregory | zigo, just fyi seems it was not only Embedded how went way, some other class got renamed | 08:27 |
| zigo | Oh ... :/ | 08:27 |
| iurygregory | String was renamed to PaddedString | 08:28 |
| iurygregory | from 2.8 to 2.9 | 08:28 |
| iurygregory | wget: unable to resolve host address ‘tarballs.openstack.org’ \o/ | 08:37 |
| iurygregory | Tks Zuul! | 08:37 |
| *** alexmcleod has joined #openstack-ironic | 08:46 | |
| *** ociuhandu has quit IRC | 08:50 | |
| *** ociuhandu has joined #openstack-ironic | 08:50 | |
| *** ociuhandu has quit IRC | 08:50 | |
| *** ociuhandu has joined #openstack-ironic | 08:53 | |
| *** k_mouza has joined #openstack-ironic | 09:02 | |
| *** mkrai has joined #openstack-ironic | 09:07 | |
| *** mkrai_ has quit IRC | 09:08 | |
| *** mkrai has quit IRC | 09:15 | |
| *** mkrai_ has joined #openstack-ironic | 09:15 | |
| ajya | Hi, thanks for previous help with getting stein CI working. There are still issues with stable/queen. Getting error "No valid host was found. Reason: No conductor service registered which supports driver fake-hardware." Looking at the ironic.conf in CI run, there is only ipmi enabled. The same job is also in rocky. Having taken a look at successful rocky run, I see that rocky has | 09:17 |
| ajya | fake-hardware enabled. Why is it missing in queens? | 09:17 |
| ajya | One of the jobs failing is ironic-tempest-dsvm-ipa-partition-pxe_ipmitool-tinyipa-python3 | 09:18 |
| rpittau | ajya: can you point to the patch link please ? | 09:23 |
| *** dtantsur|afk is now known as dtantsur | 09:24 | |
| ajya | rpittau: both of them - as far as I could see, the failures are the same https://review.opendev.org/#/c/721712/ https://review.opendev.org/#/c/721811/ | 09:25 |
| patchbot | patch 721712 - ironic (stable/queens) - Fix SpanLength calculation for DRAC RAID configura... - 1 patch set | 09:25 |
| patchbot | patch 721811 - ironic (stable/queens) - Fix issue where server fails to reboot - 1 patch set | 09:25 |
| dtantsur | morning ironic | 09:25 |
| dtantsur | new day, new failed CI | 09:25 |
| iurygregory | dtantsur, yeah =) | 09:26 |
| rpittau | good morning dtantsur :) | 09:26 |
| iurygregory | start the day with wget: unable to resolve host address ‘tarballs.openstack.org’ \o/ | 09:26 |
| iurygregory | starting* | 09:26 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add ironic-python-agent-builder to grenade projects https://review.opendev.org/724206 | 09:27 |
| rpittau | ajya: the problem might be in the tempest plugin for ironic as it adds fake-hardware as driver | 09:27 |
| dtantsur | we also have grenade hard-blocked by https://review.opendev.org/#/c/724206/ | 09:27 |
| patchbot | patch 724206 - ironic - Add ironic-python-agent-builder to grenade projects - 2 patch sets | 09:27 |
| dtantsur | rpittau: ^^^ | 09:27 |
| dtantsur | mgoddard, iurygregory, are we ready to release and branch ironic-prometheus-exporter? | 09:30 |
| iurygregory | dtantsur, YES | 09:30 |
| iurygregory | \o/ | 09:30 |
| iurygregory | sorry for the caps | 09:31 |
| dtantsur | nice! | 09:33 |
| dtantsur | also iurygregory, could I get an urgent review of https://review.opendev.org/724233 please? | 09:33 |
| patchbot | patch 724233 - puppet-ironic - Explicitly install inspector-dnsmasq package - 1 patch set | 09:33 |
| iurygregory | dtantsur, sure! | 09:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/ussuri: Remove testresources and testscenarios from test-requirements https://review.opendev.org/724267 | 09:37 |
| ajya | rpittau: so have to check if there have been recent changes to tempest plugin that could impact queens? | 09:38 |
| *** k_mouza has quit IRC | 09:40 | |
| *** uzumaki has joined #openstack-ironic | 09:44 | |
| *** k_mouza has joined #openstack-ironic | 09:44 | |
| rpittau | ajya: the problem is that fake-hardware is added as support, but not enabled driver | 09:44 |
| mgoddard | dtantsur: I'm busy until lunchtime, feel free to update the review in gerrit | 09:47 |
| *** ociuhandu has quit IRC | 09:47 | |
| *** ociuhandu has joined #openstack-ironic | 09:48 | |
| uzumaki | morning ironic! p/ | 09:49 |
| *** ociuhandu has quit IRC | 09:53 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Get rid of the oslo.serialization dependency https://review.opendev.org/723868 | 09:53 |
| dtantsur | mgoddard: ack | 09:54 |
| rpittau | dtantsur: I'm afraid queens backports will never pass if we don't add fake-hardware to IRONIC_ENABLED_DRIVERS | 10:12 |
| dtantsur | rpittau: fake-hardware cannot go to IRONIC_ENABLED_DRIVERS | 10:12 |
| *** ociuhandu has joined #openstack-ironic | 10:12 | |
| dtantsur | do you mean fake to IRONIC_ENABLED_DRIVERS or fake-hardware to IRONIC_ENABLED_HARDWARE_TYPES | 10:12 |
| dtantsur | ?\ | 10:12 |
| dtantsur | mgoddard: I also intend to release virtualbmc and sushy-tools | 10:12 |
| rpittau | dtantsur: sorry, pre-lunch confusion, IRONIC_ENABLED_HARDWARE_TYPES | 10:13 |
| dtantsur | do you know why it's required? | 10:13 |
| rpittau | dtantsur: I see we're requiring that in ironic-tempest-plugin | 10:14 |
| dtantsur | that's possible | 10:14 |
| rpittau | it's in supported_drivers | 10:14 |
| rpittau | so for example this is failing https://review.opendev.org/#/c/721712/ | 10:15 |
| patchbot | patch 721712 - ironic (stable/queens) - Fix SpanLength calculation for DRAC RAID configura... - 1 patch set | 10:15 |
| *** uzumaki has quit IRC | 10:25 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix 3rd party driver mocks and related tests https://review.opendev.org/724080 | 10:48 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/queens: [DNM] test CI https://review.opendev.org/724277 | 10:50 |
| *** rpittau is now known as rpittau|bbl | 10:55 | |
| openstackgerrit | Mahendra Kamble proposed x/sushy-oem-idrac master: Export System Configuration https://review.opendev.org/710327 | 10:59 |
| *** ociuhandu has quit IRC | 11:06 | |
| *** ociuhandu has joined #openstack-ironic | 11:07 | |
| openstackgerrit | Merged openstack/virtualbmc master: Set flake8-import-order version to 0.17.1 https://review.opendev.org/721657 | 11:10 |
| *** ociuhandu has quit IRC | 11:13 | |
| openstackgerrit | Merged openstack/sushy-tools master: Stop configuring install_command in tox. https://review.opendev.org/720807 | 11:19 |
| *** Qianbiao has quit IRC | 11:27 | |
| *** mkrai_ has quit IRC | 11:29 | |
| *** mkrai has joined #openstack-ironic | 11:29 | |
| *** derekh has joined #openstack-ironic | 11:30 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add ironic-python-agent-builder to grenade projects and use netboot https://review.opendev.org/724206 | 11:43 |
| dtantsur | another attempt ^^ | 11:43 |
| *** rh-jelabarre has joined #openstack-ironic | 11:45 | |
| *** gokhani has joined #openstack-ironic | 11:47 | |
| *** Qianbiao has joined #openstack-ironic | 11:55 | |
| *** uzumaki has joined #openstack-ironic | 12:01 | |
| *** ociuhandu has joined #openstack-ironic | 12:08 | |
| *** Goneri has joined #openstack-ironic | 12:22 | |
| *** derekh has quit IRC | 12:26 | |
| *** rloo has joined #openstack-ironic | 12:26 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/724302 | 12:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724303 | 12:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/ussuri https://review.opendev.org/724304 | 12:27 |
| openstackgerrit | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Add Python3 victoria unit tests https://review.opendev.org/724305 | 12:27 |
| *** rpittau|bbl is now known as rpittau | 12:31 | |
| *** rloo has quit IRC | 12:37 | |
| *** rloo has joined #openstack-ironic | 12:37 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix 3rd party driver mocks and related tests https://review.opendev.org/724080 | 12:42 |
| *** ociuhandu has quit IRC | 12:44 | |
| *** uzumaki has quit IRC | 12:45 | |
| *** ociuhandu has joined #openstack-ironic | 12:45 | |
| *** ociuhandu has quit IRC | 12:49 | |
| openstackgerrit | Merged openstack/ironic-inspector-specs master: Cleanup py27 support https://review.opendev.org/724105 | 12:53 |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Update master for stable/ussuri https://review.opendev.org/724304 | 12:54 |
| *** ociuhandu has joined #openstack-ironic | 12:59 | |
| TheJulia | good morning | 13:09 |
| TheJulia | dtantsur: any luck? | 13:09 |
| rpittau | good morning TheJulia :) | 13:09 |
| dtantsur | TheJulia: waiting for the CI now | 13:09 |
| dtantsur | and good morning | 13:10 |
| dtantsur | also, looking into why the SNMP job is so dead | 13:10 |
| TheJulia | sigh | 13:10 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/queens: [DNM] test CI https://review.opendev.org/724277 | 13:10 |
| *** derekh has joined #openstack-ironic | 13:12 | |
| TheJulia | ugh, all sorts of failures | 13:12 |
| dtantsur | mmm, regressed in https://review.opendev.org/#/c/710278/ presumably | 13:14 |
| patchbot | patch 710278 - ironic - Default IRONIC_RAMDISK_TYPE to dib (MERGED) - 19 patch sets | 13:14 |
| dtantsur | no, wrong, returned back in https://review.opendev.org/#/c/718607/ | 13:16 |
| patchbot | patch 718607 - ironic - Change default ram value (MERGED) - 1 patch set | 13:16 |
| dtantsur | ah, got it, it's probably https://review.opendev.org/#/c/492114/ | 13:17 |
| patchbot | patch 492114 - ironic - Change [deploy]/default_boot_option to local (MERGED) - 34 patch sets | 13:17 |
| dtantsur | it's a whole disk job, but it cannot use boot_option=local since it doesn't have a mgmt interface | 13:17 |
| * dtantsur fixes | 13:17 | |
| *** jdandrea has joined #openstack-ironic | 13:21 | |
| *** cdearborn has joined #openstack-ironic | 13:21 | |
| TheJulia | ugh | 13:21 |
| derekh | TheJulia: dtantsur IPv6 job is here when ye have a chance https://review.opendev.org/#/c/724087/4 | 13:23 |
| patchbot | patch 724087 - ironic - Add IPv6 ci Job - 4 patch sets | 13:23 |
| dtantsur | thanks derekh | 13:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: repair the SNMP job and make it voting. https://review.opendev.org/724320 | 13:24 |
| iurygregory | morning TheJulia o/ | 13:24 |
| derekh | TheJulia: I opened a new review as your one had 3 jobs and I only got one working, so most of it is from your WIP patch with my changes, without 2 of the jobs and soem of the DEBUG stuff | 13:26 |
| *** mkrai has quit IRC | 13:28 | |
| dtantsur | hmm, the redfish-vmedia-inspector job is annoyingly unstable: https://zuul.openstack.org/builds?job_name=ironic-inspector-tempest-partition-bios-redfish-vmedia | 13:32 |
| rpittau | dtantsur: that might be just a problem with timeout value | 13:37 |
| dtantsur | a similar UEFI job seems to pass more often though.. | 13:37 |
| arne_wiebalck | FWIW: we've split our deployment into conductor groups today, seems to have gone ok! | 13:38 |
| iurygregory | arne_wiebalck, congrats! | 13:39 |
| iurygregory | \o/ | 13:39 |
| arne_wiebalck | 10 groups, ~500 nodes each | 13:39 |
| iurygregory | graphs or it didn't happen =P | 13:39 |
| arne_wiebalck | https://twitter.com/ArneWiebalck/status/1255477097990107141 | 13:40 |
| iurygregory | *-* *-* *-* | 13:40 |
| rpittau | I'm not seeing specific errors and the timeout is "just" 720, maybe increasing it a little will help, it seems sometimes it takes longer than 12 minutes | 13:40 |
| iurygregory | nice | 13:40 |
| dtantsur | arne_wiebalck: \o/ | 13:40 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: exclude the inspection abort tests from the inspector-redfish-vmedia job https://review.opendev.org/724323 | 13:40 |
| rpittau | arne_wiebalck: that looks really cool :) | 13:40 |
| dtantsur | rpittau: which timeout value is it? | 13:41 |
| arne_wiebalck | task for the advanced Ironicer: spot the typo I made! | 13:41 |
| rpittau | both BUILD_TIMEOUT and IRONIC_TEMPEST_BUILD_TIMEOUT | 13:41 |
| arne_wiebalck | iurygregory: dtantsur rpittau thanks | 13:43 |
| *** mkrai has joined #openstack-ironic | 13:44 | |
| *** tzumainn has joined #openstack-ironic | 13:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: CI: exclude the inspection abort tests from the inspector-redfish-vmedia job https://review.opendev.org/724323 | 13:44 |
| dtantsur | rpittau: ^^ | 13:44 |
| iurygregory | we have tests that are taking more than 720 sec? | 13:44 |
| rpittau | iurygregory: lot of them | 13:44 |
| iurygregory | ouch D: | 13:44 |
| rpittau | in most cases it's because of the switch to the centos8 ramdisk | 13:45 |
| rpittau | it takes a while to load and run ipa | 13:45 |
| *** pas-ha has joined #openstack-ironic | 13:46 | |
| * TheJulia needs some super strong coffee today | 13:51 | |
| TheJulia | iurygregory: 400-500+ seconds just to boot a larger dib ramdisk | 13:51 |
| dtantsur | rpittau, arne_wiebalck, could you check https://review.opendev.org/#/c/724206/ please? it has passed the grenade job | 13:51 |
| patchbot | patch 724206 - ironic - Add ironic-python-agent-builder to grenade project... - 3 patch sets | 13:51 |
| dtantsur | TheJulia: switching to master->master testing did break us, but not in the way you expected :) | 13:52 |
| rpittau | dtantsur: done | 13:52 |
| iurygregory | well so let's increase =) | 13:52 |
| TheJulia | dtantsur: yeah, I noticed :( | 13:52 |
| TheJulia | dtantsur: I think by that point I rechecked my patch to just disable grenade so we can merge the final patch and move in in life | 13:53 |
| dtantsur | heh | 13:53 |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/724302 | 13:54 |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724303 | 13:54 |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Add Python3 victoria unit tests https://review.opendev.org/724305 | 13:54 |
| *** bnemec has joined #openstack-ironic | 13:56 | |
| *** stendulker_ has joined #openstack-ironic | 14:06 | |
| openstackgerrit | Verification of a change to openstack/python-ironicclient failed: Check import order https://review.opendev.org/721193 | 14:14 |
| *** tkajinam has joined #openstack-ironic | 14:19 | |
| *** priteau has joined #openstack-ironic | 14:23 | |
| *** lucasagomes has quit IRC | 14:25 | |
| *** lucasagomes has joined #openstack-ironic | 14:27 | |
| -openstackstatus- NOTICE: Zuul had to be restarted, all changes submitted or approved between 14:00 UTC to 14:30 need to be rechecked, we queued already those running at 14:00 | 14:34 | |
| iurygregory | all days of this week can be considered Monday.... | 14:35 |
| *** zbitter is now known as zaneb | 14:44 | |
| *** gokhani has quit IRC | 14:47 | |
| *** ociuhandu has quit IRC | 14:52 | |
| *** trident has quit IRC | 15:04 | |
| *** trident has joined #openstack-ironic | 15:05 | |
| *** ociuhandu has joined #openstack-ironic | 15:15 | |
| TheJulia | iurygregory: ++ | 15:19 |
| TheJulia | indeed | 15:19 |
| *** mkrai has quit IRC | 15:25 | |
| iurygregory | the new construct version is returning extra fields after doing the parser >.< 'switch_chassis_id': {'_io': <_io.BytesIO object at 0x7fbcb9136360>, 'value': '11:22:33:aa:bb:cc'} | 15:28 |
| rpittau | iurygregory: that's how it's supposed to work, using Embedded was never recommended | 15:29 |
| rpittau | it doesn't look like the change is trivial though | 15:29 |
| iurygregory | rpittau, yeah, I was just checking if it's possible to avoid including or not =) | 15:29 |
| iurygregory | and I also saw things being parsed to Container (value) instead of value only | 15:30 |
| iurygregory | rpittau, so for the ones with _io info it should be ok to just update tests right? | 15:35 |
| iurygregory | or we want to remove that data? | 15:35 |
| openstackgerrit | Merged openstack/ironic-inspector master: Hacking: enforce usage of autospec=True in tests https://review.opendev.org/723859 | 15:41 |
| *** stendulker_ has quit IRC | 15:48 | |
| *** gyee has joined #openstack-ironic | 15:53 | |
| *** uzumaki has joined #openstack-ironic | 15:58 | |
| *** Qianbiao has quit IRC | 16:01 | |
| *** ociuhandu has quit IRC | 16:07 | |
| *** ociuhandu has joined #openstack-ironic | 16:08 | |
| *** ociuhandu has quit IRC | 16:13 | |
| TheJulia | rechecked the grenade change, everything was non-voting that failed but zuul still said it failed | 16:28 |
| TheJulia | :( | 16:29 |
| rpittau | iurygregory: I'm not sure we should modify any test to be honest | 16:29 |
| dtantsur | TheJulia: mmm? https://review.opendev.org/#/c/724206/ passed the check and is now in the gate pipeline | 16:32 |
| patchbot | patch 724206 - ironic - Add ironic-python-agent-builder to grenade project... - 3 patch sets | 16:32 |
| TheJulia | dtantsur: look at the times... | 16:32 |
| TheJulia | 3 minutes to pass CI? :) | 16:32 |
| TheJulia | zuul is magic! | 16:32 |
| TheJulia | all hail the mighty zuul magic | 16:32 |
| dtantsur | TheJulia: the passed build was not triggered by you :) it was triggered by infra when they restarted misbehaving zuul :) | 16:33 |
| iurygregory | zuul is using some drugs I'm sure | 16:33 |
| iurygregory | 3min?! | 16:33 |
| dtantsur | (recheck is ignored when a build is running) | 16:33 |
| iurygregory | in 3hrs it gives timeout for our jobs u.u' | 16:34 |
| TheJulia | the prior message is Verified-1... | 16:34 |
| TheJulia | before the recheck | 16:34 |
| dtantsur | TheJulia: please see my explanation above | 16:34 |
| dtantsur | I've been following the process | 16:34 |
| dtantsur | in the previous V-1 some jobs failed with EXCEPTION because of zuul problems and the built was retriggered by infra | 16:34 |
| dtantsur | it happened with a few other builds as well | 16:35 |
| dtantsur | (it was completely opaque for somebody who was not following zuul status like me) | 16:35 |
| TheJulia | it only picked up into the gate queue 8 minutes ago, so it seems like it got confused, and upon the infra issues, it posted V-1 even though it passed | 16:35 |
| * TheJulia suspects we're on different wavelenghts or something today | 16:35 | |
| dtantsur | probably | 16:35 |
| dtantsur | anyway, it's in the gate queue now. fingers crossed! | 16:36 |
| TheJulia | I get what your saying, I just looked, and re-looked at the times and yeah | 16:36 |
| TheJulia | anyway, indeed ++ | 16:36 |
| *** dtantsur is now known as dtantsur|afk | 16:36 | |
| dtantsur|afk | g'night everyone | 16:36 |
| TheJulia | goodnight dtantsur|afk | 16:36 |
| rpittau | goodnight! o/ | 16:37 |
| *** rpittau is now known as rpittau|afk | 16:37 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/queens: [DNM] test CI https://review.opendev.org/724277 | 16:37 |
| arne_wiebalck | bye everyone o/ | 16:47 |
| *** tkajinam has quit IRC | 16:57 | |
| *** uzumaki has quit IRC | 17:00 | |
| *** lucasagomes has quit IRC | 17:01 | |
| *** ociuhandu has joined #openstack-ironic | 17:12 | |
| *** ociuhandu has quit IRC | 17:17 | |
| *** priteau has quit IRC | 17:32 | |
| *** priteau has joined #openstack-ironic | 17:35 | |
| *** priteau has quit IRC | 17:40 | |
| *** k_mouza has quit IRC | 17:55 | |
| *** ociuhandu has joined #openstack-ironic | 18:00 | |
| openstackgerrit | Merged openstack/ironic master: Add ironic-python-agent-builder to grenade projects and use netboot https://review.opendev.org/724206 | 18:28 |
| *** ociuhandu has quit IRC | 18:31 | |
| *** ociuhandu has joined #openstack-ironic | 18:33 | |
| *** Lucas_Gray has joined #openstack-ironic | 18:33 | |
| *** derekh has quit IRC | 18:36 | |
| *** lucasagomes has joined #openstack-ironic | 18:37 | |
| *** ociuhandu has quit IRC | 18:37 | |
| *** iurygregory has quit IRC | 18:38 | |
| TheJulia | dtantsur|afk: thanks for issuing the recheck | 18:54 |
| *** lucasagomes has quit IRC | 18:55 | |
| *** hjensas has quit IRC | 19:05 | |
| *** ociuhandu has joined #openstack-ironic | 19:12 | |
| *** alexmcleod has quit IRC | 19:18 | |
| *** ociuhandu has quit IRC | 19:25 | |
| *** ociuhandu has joined #openstack-ironic | 19:26 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/queens: [DNM] test CI https://review.opendev.org/724277 | 19:28 |
| *** ociuhandu has quit IRC | 19:32 | |
| *** yolanda has quit IRC | 19:36 | |
| *** priteau has joined #openstack-ironic | 19:44 | |
| openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721653 | 19:47 |
| *** priteau has quit IRC | 19:51 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Check import order https://review.opendev.org/721193 | 20:57 |
| *** ociuhandu has joined #openstack-ironic | 21:01 | |
| *** ociuhandu has quit IRC | 21:06 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Hacking: enforce usage of autospec=True in tests https://review.opendev.org/723861 | 21:18 |
| *** rcernin has joined #openstack-ironic | 21:32 | |
| *** rcernin has quit IRC | 21:33 | |
| *** rcernin has joined #openstack-ironic | 21:34 | |
| NobodyCam | Good AfterNoon Ironic folks! | 21:35 |
| *** k_mouza has joined #openstack-ironic | 21:55 | |
| TheJulia | good afternoon NobodyCam | 21:56 |
| NobodyCam | :) 👋 hi TheJulia | 21:57 |
| *** k_mouza has quit IRC | 21:59 | |
| *** hamzy_ has joined #openstack-ironic | 22:11 | |
| *** hamzy has quit IRC | 22:13 | |
| TheJulia | how goes it NobodyCam ? | 22:18 |
| NobodyCam | it goes well ... so far ;) | 22:20 |
| NobodyCam | and how about with you? | 22:20 |
| TheJulia | the day is quickly coming to an end :) | 22:20 |
| NobodyCam | oh ya | 22:20 |
| *** Lucas_Gray has quit IRC | 22:23 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Don't break UEFI install with older IPAs https://review.opendev.org/720781 | 22:35 |
| *** ociuhandu has joined #openstack-ironic | 22:36 | |
| *** tkajinam has joined #openstack-ironic | 22:51 | |
| *** ociuhandu has quit IRC | 22:58 | |
| *** tzumainn has quit IRC | 23:32 | |
| *** ociuhandu has joined #openstack-ironic | 23:33 | |
| *** ociuhandu has quit IRC | 23:53 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!