Tuesday, 2022-02-01

« Monday, 2022-01-31 Index Wednesday, 2022-02-02 »
opendevreviewMahnoor Asghar proposed openstack/ironic master: Create API documentation from docstrings  https://review.opendev.org/c/openstack/ironic/+/82720000:32
TheJuliaMahnoorAsghar: could you assign me as a reviewer to that?00:53
MahnoorAsgharTheJulia: Yes, added you as a reviewer00:55
TheJuliaThanks00:55
TheJuliaMy brain is basically out of computing power today00:55
TheJuliabut I'll try to dig through it tomorrow00:55
MahnoorAsgharThank you! I will try to look deeper00:58
opendevreviewMerged openstack/metalsmith master: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82715203:14
arne_wiebalckGood morning, Ironic!07:41
opendevreviewHarald Jensås proposed openstack/metalsmith master: Allow both 'network' and 'subnet' in NIC  https://review.opendev.org/c/openstack/metalsmith/+/82721908:51
dtantsurmorning ironic09:00
opendevreviewDmitry Tantsur proposed openstack/metalsmith stable/xena: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82723309:01
opendevreviewDmitry Tantsur proposed openstack/metalsmith stable/wallaby: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82723409:02
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: Use canonical device name for RAID device for ESP  https://review.opendev.org/c/openstack/ironic-python-agent/+/82703609:04
dtantsurthis needs urgent reviews please ^^^ arne_wiebalck, I had to fix a unit test09:04
dtantsuroh, lemme fix the message09:05
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: Use canonical device name for RAID device for ESP  https://review.opendev.org/c/openstack/ironic-python-agent/+/82703609:06
arne_wiebalckdtantsur: checking09:06
opendevreviewDmitry Tantsur proposed openstack/ironic stable/xena: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722209:08
opendevreviewDmitry Tantsur proposed openstack/ironic bugfix/19.0: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722409:09
opendevreviewDmitry Tantsur proposed openstack/ironic bugfix/18.1: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722509:09
opendevreviewDmitry Tantsur proposed openstack/ironic stable/wallaby: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722609:10
opendevreviewDmitry Tantsur proposed openstack/ironic stable/victoria: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722709:10
dtantsursigh, this is a lot of "fun"09:10
arne_wiebalckdtantsur: do we still need the rescan? I think this is mostly for partitions (which we rescan in the loop), and I think you said the rescan does not make any difference.09:12
dtantsurarne_wiebalck: I'd keep it just in case? at least udev settle is probably good to have09:13
dtantsurand partprobe allows us to loop until the device actually exists09:13
arne_wiebalckok09:13
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82722909:15
opendevreviewDmitry Tantsur proposed openstack/ironic-inspector master: Support listening on a Unix socket  https://review.opendev.org/c/openstack/ironic-inspector/+/82712409:23
opendevreviewDmitry Tantsur proposed openstack/ironic-inspector master: Support listening on a Unix socket  https://review.opendev.org/c/openstack/ironic-inspector/+/82712409:30
opendevreviewDmitry Tantsur proposed openstack/ironic stable/victoria: Avoid non-Stream CentOS, remove Grenade job  https://review.opendev.org/c/openstack/ironic/+/82722709:47
rpittaugood morning ironic! oi/09:56
dtantsurrpittau: o/09:59
rpittauhey dtantsur :)09:59
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Enable epel repository only when needed  https://review.opendev.org/c/openstack/bifrost/+/82644210:04
rpittau^ fixed release note10:04
opendevreviewMerged openstack/ironic master: Avoid non-Stream CentOS and temporary disable the standalone job  https://review.opendev.org/c/openstack/ironic/+/82712910:37
opendevreviewMerged openstack/ironic master: Add additional ramdisk tests  https://review.opendev.org/c/openstack/ironic/+/82650610:41
iurygregorygood morning Ironic o/10:49
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/19.0: Add additional ramdisk tests  https://review.opendev.org/c/openstack/ironic/+/82723711:28
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/18.1: Add additional ramdisk tests  https://review.opendev.org/c/openstack/ironic/+/82723811:28
opendevreviewMerged openstack/metalsmith stable/xena: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82723312:39
opendevreviewVerification of a change to openstack/metalsmith stable/wallaby failed: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82723412:39
opendevreviewMerged openstack/ironic-python-agent master: Use canonical device name for RAID device for ESP  https://review.opendev.org/c/openstack/ironic-python-agent/+/82703612:41
*** sshnaidm is now known as sshnaidm|afk13:06
opendevreviewDmitry Tantsur proposed openstack/ironic master: Re-enable the standalone job  https://review.opendev.org/c/openstack/ironic/+/82728713:41
dtantsurrpittau, arne_wiebalck ^^13:41
dtantsuralso arne_wiebalck, please check https://review.opendev.org/c/openstack/ironic-python-agent/+/82722913:41
dtantsur(to avoid further regressions)13:41
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/xena: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82728813:43
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.3: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82728913:44
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.1: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82729013:44
*** sshnaidm|afk is now known as sshnaidm13:54
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/wallaby: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82729113:54
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dib ipa jobs  https://review.opendev.org/c/openstack/bifrost/+/82729314:06
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dib ipa jobs  https://review.opendev.org/c/openstack/bifrost/+/82729314:07
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dib ipa jobs  https://review.opendev.org/c/openstack/bifrost/+/82729314:08
opendevreviewmitya-eremeev-2 proposed openstack/ironic master: Delete resource provider after node deletion.  https://review.opendev.org/c/openstack/ironic/+/82729514:11
dtantsurTheJulia: interesting ^^14:27
TheJuliainteresting14:28
opendevreviewVerification of a change to openstack/ironic master failed: Inspector: better error message on DiscoveryFailure  https://review.opendev.org/c/openstack/ironic/+/82554414:30
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Use debian current stable for dib  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82730114:32
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] build tinyipa on tinycore 13.x  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82713714:41
iurygregorygood morning TheJulia =)14:52
TheJuliadtantsur: I've left a review on it with some thoughts. I've tagged nobodycam as well since he started looking at something similar, but I don't think he was destroying the node out of ironic15:00
TheJuliabased on the sync run of nova-compute, this may not get picked up state change wise I think, which means the record could just get recreated.15:00
TheJuliaand then deleted later15:02
TheJuliain the next run15:02
opendevreviewMark Goddard proposed openstack/tenks master: Explicitly set boot_mode to legacy BIOS  https://review.opendev.org/c/openstack/tenks/+/82730415:06
opendevreviewMerged openstack/ironic-python-agent master: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82722915:32
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dib ipa jobs  https://review.opendev.org/c/openstack/bifrost/+/82729315:33
dtantsurTheJulia: if you have just a few seconds: https://review.opendev.org/c/openstack/ironic/+/82728715:39
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dib ipa jobs  https://review.opendev.org/c/openstack/bifrost/+/82729315:47
TheJuliaarne_wiebalck: we backported lazy loading right?15:52
dtantsurI think so, not sure how far16:11
arne_wiebalckTheJulia: yes, we did16:24
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dib ipa jobs  https://review.opendev.org/c/openstack/bifrost/+/82729317:01
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] build tinyipa on tinycore 13.x  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82713717:22
opendevreviewMerged openstack/ironic master: Re-enable the standalone job  https://review.opendev.org/c/openstack/ironic/+/82728717:23
arne_wiebalckbye everyone o/17:24
rpittau good night! o/17:30
opendevreviewMerged openstack/ironic master: Inspector: better error message on DiscoveryFailure  https://review.opendev.org/c/openstack/ironic/+/82554417:45
opendevreviewVerification of a change to openstack/ironic-python-agent stable/xena failed: Make the standalone job voting again  https://review.opendev.org/c/openstack/ironic-python-agent/+/82728818:28
opendevreviewDmitry Tantsur proposed openstack/ironic stable/xena: Inspector: better error message on DiscoveryFailure  https://review.opendev.org/c/openstack/ironic/+/82733418:29
opendevreviewDmitry Tantsur proposed openstack/ironic bugfix/19.0: Inspector: better error message on DiscoveryFailure  https://review.opendev.org/c/openstack/ironic/+/82733518:30
dtantsurAttributeError: module 'asyncio' has no attribute 'get_running_loop'18:32
dtantsurwtf18:32
iurygregorywoott18:32
dtantsurhmm, python 3.6 strikes again18:32
iurygregorydtantsur, where you saw the error?18:34
dtantsurhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_736/827124/5/check/ironic-inspector-tempest-managed-non-standalone/736a50f/controller/logs/ironic-bm-logs/node-0_no_ansi_2022-02-01-10:14:44_log.txt18:34
dtantsura job using DIB IPA18:35
dtantsuralso see the thread "[requirements] The version of zeroconf in upper constraints no longer supports python3.6"18:35
iurygregory<facepalm>18:35
opendevreviewVerification of a change to openstack/metalsmith stable/wallaby failed: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82723418:38
dtantsurhttps://review.opendev.org/c/openstack/requirements/+/82733818:41
iurygregory+118:46
dtantsurehhmm, of course I made a typo in versions....18:56
dtantsurokay, on to the next failing job18:58
dtantsurah, it's the same problem18:58
dtantsurTheJulia: https://zuul.openstack.org/builds?job_name=ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode seems to behave, should we consider re-enabling it?19:00
dtantsuriurygregory: hmm, we still run 2 tests per job, both with the same image type?19:07
iurygregory2 tests per job only?19:09
dtantsuriurygregory: well, on "standard" jobs, not standalone19:09
dtantsurtest_baremetal_server_ops_partition_image and test_baremetal_server_ops_wholedisk_image19:10
dtantsurboth seems to use the same image type in reality?19:10
iurygregoryright19:11
TheJuliahmm, dtantsur likely reenable I guess19:11
TheJuliamulti-node is good to test, but it can break fragilely at times19:11
iurygregorylet's re-enable after we do the release XD19:12
dtantsurfair enough19:12
iurygregorytime to grab more coffee and some cookies to eat :D19:16
opendevreviewDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Apply the correct image reference when booting an instance  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/82734019:20
dtantsuriurygregory: an idea ^^^19:20
dtantsursee you tomorrow folks o/19:21
opendevreviewMerged openstack/ironic stable/xena: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722219:22
iurygregorydtantsur, ack19:26
opendevreviewMerged openstack/ironic bugfix/19.0: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722419:33
opendevreviewMerged openstack/ironic bugfix/18.1: Avoid non-Stream CentOS  https://review.opendev.org/c/openstack/ironic/+/82722519:33
stevebaker[m]good morning19:41
iurygregorygood morning stevebaker[m] o/19:45
TheJuliao/ stevebaker[m] 19:48
stevebaker[m]TheJulia: can you spare a spoon to look at this zuul syntax error? https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/82619919:53
TheJuliastevebaker[m]: I get it finally20:01
* TheJulia ponders20:01
TheJuliaso the tl;dr is it is cross-referenced20:05
stevebaker[m]ah, the oldest stable builder branch is wallaby20:06
TheJuliaother project20:10
TheJuliabingo https://github.com/openstack/ironic-python-agent/blob/stable/train/zuul.d/project.yaml#L4820:11
stevebaker[m]TheJulia: oh my fix change the wrong job https://review.opendev.org/c/openstack/ironic-python-agent/+/826792/1/zuul.d/project.yaml20:12
stevebaker[m]or did it20:15
stevebaker[m]TheJulia: actually I think that fix is correct, and it will be needed on U and V also20:16
TheJuliaahh, but your change is on builder20:21
TheJuliathe cross-repo checks fail20:21
opendevreviewJulia Kreger proposed openstack/ironic-python-agent stable/train: Remove legacy centos7 build jobs  https://review.opendev.org/c/openstack/ironic-python-agent/+/82735820:25
TheJuliaweird, it didn't run on your base change20:27
stevebaker[m]I'm tempted to just approve ^^20:29
TheJuliaIt's long past eol right?20:29
iurygregoryhttps://wiki.centos.org/About/Product20:31
TheJuliaeh... 202420:32
TheJuliapractically though...20:32
iurygregoryyeah20:32
opendevreviewVerification of a change to openstack/metalsmith stable/wallaby failed: Avoid non-Stream CentOS 8, it is EOL  https://review.opendev.org/c/openstack/metalsmith/+/82723421:49
*** dking is now known as Guest144622:04
*** Guest1446 is now known as dking22:05
« Monday, 2022-01-31 Index Wednesday, 2022-02-02 »

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!