| rosmaita | fyi, i will be raising the minima in requirements for os-brick release: http://lists.openstack.org/pipermail/openstack-discuss/2022-February/027192.html | 13:24 |
|---|---|---|
| rosmaita | abhishekk: jokke_: croelandt: dansmith: ^^ | 13:24 |
| abhishekk | ade_lee, failure continues, can we make the job non-voting for now (better alternative than to revert it back to non-fips job)? | 13:51 |
| * abhishekk having sore eyes, so will not be able to much around today | 14:15 | |
| ade_lee | abhishekk, yes please. we don't want to hold things up | 14:35 |
| ade_lee | abhishekk, I'm going to put up a review where we test with centos + non-fips to see if its fips or centos related | 14:35 |
| abhishekk | ade_lee, cool | 14:36 |
| abhishekk | after that we can decide what to do | 14:36 |
| ade_lee | abhishekk, do you want me to put up the review to make the job non-voting in the meantime? | 14:37 |
| abhishekk | sounds good | 14:38 |
| ade_lee | ok - will do | 14:38 |
| abhishekk | thank you | 14:44 |
| opendevreview | Ade Lee proposed openstack/glance master: Make FIPS job non-voting https://review.opendev.org/c/openstack/glance/+/828892 | 15:02 |
| opendevreview | Ade Lee proposed openstack/glance master: DNM - Testing centos and fips https://review.opendev.org/c/openstack/glance/+/828893 | 15:02 |
| ade_lee | abhishekk, ^^ | 15:02 |
| abhishekk | ade_lee, thanks | 15:02 |
| abhishekk | jokke_, rosmaita, dansmith, croelandt please have a look, Make FIPS job non-voting - https://review.opendev.org/c/openstack/glance/+/828892 | 17:00 |
| abhishekk | I need to step away due to sore eyes :/ | 17:00 |
| dansmith | rosmaita: see ^ | 17:50 |
| dansmith | I hate to just make it non-voting, vs maybe re-introducing the non-fips version of it | 17:51 |
| rosmaita | i was focusing on 'cinder' and not on 'fips' ... i agree, it would be good to have some kind of voting with an import job | 17:54 |
| rosmaita | though we do have the tempest-integrated-storage-import | 17:54 |
| rosmaita | i guess that one doesn't use multistore? | 17:55 |
| dansmith | no I think it does, it's the cinderness in that job that we'd lose | 17:58 |
| dansmith | by the way, pulled out at least one crash type failure from one of the tests in your patch: | 17:58 |
| dansmith | https://zuul.opendev.org/t/openstack/build/2d8b2290c1944a319dff6ed4e31da5b3/log/controller/logs/libvirt/libvirt/qemu/instance-00000073_log.txt#155 | 17:58 |
| dansmith | that's definitely either out of memory crash, or qemu/kernel/os crash behavior | 17:59 |
| dansmith | so not openstack, cinder, etc | 17:59 |
| dansmith | rosmaita: if you're good with re-enabling the non-fips cinder and making the fips cinder job non-voting, I'll push up that rev on top of abhi's change and we can merge | 18:03 |
| dansmith | croelandt: you around? | 18:04 |
| ade_lee | dansmith, rosmaita definitely sees to be centos rather than fips related -- https://review.opendev.org/c/openstack/glance/+/828893 | 18:10 |
| dansmith | ade_lee: right, I'm sure it's centos, just using -fips as a tag :) | 18:10 |
| rosmaita | i'm ok with saying, not cinder, not fips ... it's centos! | 18:11 |
| rosmaita | dansmith: i will look for your update to abhishek's patch when i get back from lunch | 18:11 |
| opendevreview | Dan Smith proposed openstack/glance master: Make FIPS job non-voting https://review.opendev.org/c/openstack/glance/+/828892 | 18:12 |
| dansmith | rosmaita: ^ | 18:12 |
| croelandt | dansmith: need a +2 on "make FIPS jobs non-voting"? | 18:31 |
| dansmith | croelandt: well, sanity check and then a +2 but yeah :) | 18:31 |
| dansmith | croelandt: although abhi responded positively, so pretty safe bet I think | 18:31 |
| croelandt | glance-multistore-cinder-import <- so you got that job from an earlier commit, right? | 18:32 |
| dansmith | yeah that's what it was before we fipsified it | 18:33 |
| croelandt | yeah so it should be pretty safe | 18:34 |
| opendevreview | Merged openstack/glance master: Make FIPS job non-voting https://review.opendev.org/c/openstack/glance/+/828892 | 20:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!