Tuesday, 2023-07-18

opendevreviewJacob Anders proposed openstack/ironic master: [DNM] Retry connecting vmedia through a DVD device if available - alternate approach.  https://review.opendev.org/c/openstack/ironic/+/88874601:17
rpittaugood morning ironic! o/06:51
opendevreviewMerged openstack/bifrost master: Make inspector.ipxe respect inspector_debug  https://review.opendev.org/c/openstack/bifrost/+/88766708:21
opendevreviewMerged openstack/ironic master: Don't actually heartbeat with sqlite  https://review.opendev.org/c/openstack/ironic/+/88850609:59
opendevreviewVerification of a change to openstack/ironic master failed: Firmware Interface  https://review.opendev.org/c/openstack/ironic/+/88527610:57
opendevreviewVerification of a change to openstack/ironic master failed: Firmware Interface  https://review.opendev.org/c/openstack/ironic/+/88527612:23
opendevreviewMerged openstack/ironic master: Stop splitting installation docs per distros  https://review.opendev.org/c/openstack/ironic/+/88853112:45
iurygregorygood morning Ironic12:54
opendevreviewJulia Kreger proposed openstack/ironic master: Retry SQLite DB write failures due to locks  https://review.opendev.org/c/openstack/ironic/+/88850013:14
iurygregoryrpittau, tks for the review and the recheck13:20
TheJuliaJust a rebase ^^^13:20
iurygregoryTheJulia, ack, I will be looking at it today o/13:21
TheJuliahopefully it will log a retry today13:21
TheJulia:)13:21
opendevreviewJulia Kreger proposed openstack/ironic-python-agent master: Log the number of bytes downloaded  https://review.opendev.org/c/openstack/ironic-python-agent/+/88772913:34
rpittauiurygregory: no problem, long due :)13:35
opendevreviewVerification of a change to openstack/ironic master failed: add default conductor group capability  https://review.opendev.org/c/openstack/ironic/+/85570513:43
iurygregoryoh wow we still have openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/d5b61d6d277b42ddb4be5fa965219a3d : TIMED_OUT in 41m 01s :O14:07
JayFyeah we reduced frequency but didn't erradicate14:08
TheJuliastill in test_walk_versions interestingly enough14:47
TheJuliaJayF: locally, do you have mysql/mariadb and postgres running?14:47
TheJuliaso the full tests run?14:47
* TheJulia asks because your using py310 according to her memory banks14:48
JayFthey did locally when I was testing them14:49
JayFI don't have that env immediately at hand14:49
* JayF is transitioning some stuff in his homelab tonight and doesn't have access to all his usual stuff14:49
JayFit has passed before, though, for 100% sure14:50
TheJuliaoh absolutely has14:50
JayFI've never, not once, reproduced the hang locally14:50
JayFeven when trying to load my system while they are running14:51
TheJuliaI managed to a couple times at the house, but not recently14:52
TheJuliaI *will* be back down at the house Wednesday/Thursday14:53
JayFI'm building a windows computer tonight, dedicated for gaming, so my old desktop (16c/32t + 64gb of ram) becomes my new test hypervisor14:54
JayFwhich should be able to persistently hold things like ... a unit test running machine with mysql :D 14:54
TheJulia\o/14:54
TheJuliaI *relaly* need to update my home machine14:55
TheJuliareally14:55
JayFI got one of these little cases so my desktop can actually be on the desk top14:55
JayFhttps://estore.in-win.com/products/poc14:55
TheJulia... Post separation "I'm going to the computer store" build has a lot of attachment.14:56
JayFand I may have a few hundred stickers from etsy coming to stickerbomb the case with post-build14:56
JayFtime for new attachments. Like attaching hundreds of stickers about random stuff to a computer case14:56
TheJuliaheh14:56
JayFattachments such as that can be loosened with alcohol, fun fact ;)14:56
TheJuliaI don't drink like I once did early days in my time as a stacker14:57
JayFI *can't* drink like I once did, lol14:57
TheJulia... (I don't think any of us *can*)14:57
JayFI have a specific reason beyond "I'm too old for this" LOL14:59
TheJuliadon't we all? :)15:04
TheJuliain some way/shape/form15:04
opendevreviewLucas Alvares Gomes proposed openstack/ironic master: DNM Enable OVN  https://review.opendev.org/c/openstack/ironic/+/88508715:32
TheJuliaand metal3 passes again with the retry logc15:53
TheJuliaI'm just not convinced tenacity logs15:53
TheJuliaor to be more precise that before_sleep works15:53
rpittaubye everyone, see you tomorrow! o/15:59
TheJuliao/16:04
iurygregorybye rpittau o/16:12
opendevreviewJulia Kreger proposed openstack/ironic-python-agent master: WIP: preserve/handle  https://review.opendev.org/c/openstack/ironic-python-agent/+/88879418:03
opendevreviewJulia Kreger proposed openstack/ironic-python-agent master: WIP: preserve/handle config drives on 4k block devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/88879418:03
JayFTheJulia: ...do ISO-based configdrives hold any advantage over FAT formatted ones, generally18:03
TheJuliaSecurity wise, they are better but if someone gets to the level of access where they can reformat them, the game is up regardless18:04
TheJuliajust a "slightly" higher barrier18:04
JayFyeah, and FAT doesn't have a r/o flag18:04
JayFbest we could do is attr +r all the files18:04
TheJuliayup18:05
JayFI'm trying to remember, I'm fairly sure we used FAT formatted ones downstream at OnMetal18:06
JayFbecause I knew the trick to make cloud-init re-read the metadata, which was very useful for code testing cycles lol18:06
TheJuliaaround then,  nova has a bug noted w/r/t libvirt that mede sense for fat filesystems18:06
TheJuliaanyway, stevebaker[m] does the idea ^ make sense based upon the discussion we had yesterday w/r/t the bug in launchpad?18:09
TheJuliahttps://review.opendev.org/c/openstack/ironic/+/888500 has been remarkably green18:11
TheJuliaalmost, uncanny18:11
clarkbthere is a minor device ordering/naming thing that iso doesn't have a problem with in linux that fat did. I would have to dig up the ancient glean commit records to remmber what it was though18:58
TheJuliainteresting!19:07
TheJuliawe will likely need to sanity check glean code anyway in the grand scheme of things, since yay for drivers only supporting 4k blocks19:08
* TheJulia tablefilips19:08
JayFI mean, can you mention the hardware?19:08
JayFWhat kind of weird hardware won't do <4k block sizes?19:08
JayFprobably some super duper mcserious raid controller that costs more than my house?19:08
TheJuliaa lot of drivers will magically do the full read, and then re-write the larger 1 or 4k block19:10
TheJuliabut they don't really *have to* when the driver needs to be do low level io stuffs19:10
TheJuliaI don't know the hardware off the top of my head, but I'm aware some of the magical storage cluster hardware offerings do stuff liek that19:11
TheJuliaapparently https://www.spinics.net/lists/linux-scsi/msg147868.html19:15
JayFif I ever have to subscribe to linux-scsi mailing list, I quit ;) 19:16
TheJuliaheh19:19
TheJuliaJayF: I suspect a release note would be good on https://review.opendev.org/c/openstack/sushy/+/88813120:02
opendevreviewJay Faulkner proposed openstack/sushy master: Requests must always have a read/connect timeout  https://review.opendev.org/c/openstack/sushy/+/88813120:05
JayFTheJulia: there has always been a release note there, IDK what you're talking about, what kind of no-good PTL would forget to document a fix <.< >.>20:05
JayFlol20:05
TheJulialolz20:05
JayFTheJulia: I'm assuming you agree with me that is backportable?20:09
TheJuliayes20:09
opendevreviewJay Faulkner proposed openstack/sushy stable/2023.1: Requests must always have a read/connect timeout  https://review.opendev.org/c/openstack/sushy/+/88864920:09
TheJuliaI was going to ask, and then saw you already noted it20:09
JayFYeah, going back one step until master lands20:09
JayFwill take it back slowly 20:09
TheJulialooks like the ipa change to at least test-mount worked20:14
TheJulia... not sure there is a way we could fully test it20:14
opendevreviewMerged openstack/ironic master: Add additional logging on iLO power failure  https://review.opendev.org/c/openstack/ironic/+/88554920:56
opendevreviewMerged openstack/sushy master: Requests must always have a read/connect timeout  https://review.opendev.org/c/openstack/sushy/+/88813121:23
opendevreviewVerification of a change to openstack/ironic master failed: Revert "Fix IRONIC_IMAGE_NAME=non-existent-image"  https://review.opendev.org/c/openstack/ironic/+/88696023:07

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