| opendevreview | yatin proposed openstack/devstack master: [DNM] Check osprofiler https://review.opendev.org/c/openstack/devstack/+/926169 | 03:54 |
|---|---|---|
| opendevreview | yatin proposed openstack/devstack master: [DNM] Check osprofiler with fix https://review.opendev.org/c/openstack/devstack/+/926169 | 06:14 |
| opendevreview | Pranali Deore proposed openstack/tempest master: Add tests for new add location api of glance https://review.opendev.org/c/openstack/tempest/+/891026 | 06:53 |
| opendevreview | yatin proposed openstack/devstack master: [DNM] Check osprofiler with fix https://review.opendev.org/c/openstack/devstack/+/926169 | 07:52 |
| opendevreview | Pranali Deore proposed openstack/tempest master: Add tests for new add location api of glance https://review.opendev.org/c/openstack/tempest/+/891026 | 10:49 |
| ykarel | frickler, dansmith is this known that https://review.opendev.org/c/openstack/devstack/+/925425 broke rocky and 9-stream jobs? | 13:12 |
| ykarel | ERROR:generate:Command 'qemu-img create -f qed standard.qed 10M' failed with 1: b"qemu-img: standard.qed: Unknown file format 'qed'\n" | 13:13 |
| ykarel | failed in the patch itself https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7fc/925425/5/check/devstack-platform-rocky-blue-onyx/7fcc25b/job-output.txt | 13:15 |
| ykarel | ok found a patch from dansmith https://review.opendev.org/c/openstack/os-test-images/+/926162 which seems to resolve it | 13:18 |
| opendevreview | yatin proposed openstack/devstack master: [DNM] Check el9 os-test-images fix https://review.opendev.org/c/openstack/devstack/+/926208 | 13:20 |
| ykarel | validating in ^ | 13:20 |
| dansmith | ykarel: yeah, sorry, I'll approve that now on the assumption that it does | 13:32 |
| dansmith | ykarel: rocky is non-voting on devstack and has been fail-y in the recent past which is why it wasn't noticed | 13:32 |
| ykarel | dansmith, yeap those jobs were recently fixed | 13:33 |
| dansmith | I know, I reviewed those fixes, so my bad | 13:34 |
| ykarel | dansmith, testpatch failed with TypeError: not enough arguments for format string | 13:53 |
| ykarel | so would need another fix | 13:54 |
| dansmith | guh | 13:54 |
| dansmith | oh in the effing log message :/ | 13:55 |
| ykarel | should now be impacting non centos jobs too :) | 13:55 |
| ykarel | okk no it's in if so it's safe | 13:55 |
| dansmith | right | 13:56 |
| dansmith | I was waiting to merge this so I could test yesterday and ran out of time, | 13:56 |
| dansmith | and since rocky was non-voting I didn't think it'd break anyone | 13:56 |
| dansmith | ykarel: https://review.opendev.org/c/openstack/os-test-images/+/926211 | 13:58 |
| dansmith | can you re-point your test at that? | 13:58 |
| ykarel | yes sure | 13:58 |
| dansmith | ykarel: if you have a rocky/centos box handy to run "tox" on that, that might be fasdter | 13:59 |
| opendevreview | yatin proposed openstack/devstack master: [DNM] Check el9 os-test-images fix https://review.opendev.org/c/openstack/devstack/+/926208 | 13:59 |
| ykarel | done ^ | 13:59 |
| ykarel | but have 9-stream so can try tox locally | 13:59 |
| dansmith | ack, if you can confirm it runs and doesn't create the qed image, that should be good enough actually | 14:01 |
| ykarel | yes locally it passed that step and failed later at:- Unsupported option '-udf'\nxorriso | 14:02 |
| dansmith | orly, the mkiofs doesn't support udf? | 14:03 |
| ykarel | https://paste.opendev.org/show/biIpGy7WrW1ohOqqE4Cm/ | 14:03 |
| dansmith | jeez | 14:04 |
| dansmith | sean-k-mooney: didn't you make some comment about udf not being supported on some mkisofs? maybe in the unit test or something? | 14:04 |
| ykarel | i am having xorriso-1.5.4-4.el9.x86_64 | 14:05 |
| dansmith | works on my fedora, mkisofs-3.02-a09.2.fc30.x86_64 | 14:06 |
| dansmith | xorriso-1.5.6-1.fc37.x86_64 | 14:06 |
| dansmith | do you also have mkisofs package? | 14:06 |
| sean-k-mooney | dansmith: technially yes but very very very old version | 14:07 |
| sean-k-mooney | mkisofs is geniso... on some systems | 14:07 |
| sean-k-mooney | genisoimge perhaps | 14:07 |
| ykarel | dansmith, /usr/bin/mkisofs is provided by xorriso itself | 14:08 |
| sean-k-mooney | but yes that why i was checkign the help text and skiping if that didnt fied -udf | 14:08 |
| sean-k-mooney | is this on centos9 | 14:09 |
| dansmith | okay yeah genisoimage on my debian machine, xorriso on fedora | 14:09 |
| dansmith | but my fedora seems to be fine creating the udf | 14:09 |
| sean-k-mooney | that less impornat by the way they dedector does nto actully care | 14:09 |
| sean-k-mooney | udf extended the header but its backwards compatible | 14:10 |
| sean-k-mooney | so i just tested it for compelte ness | 14:10 |
| dansmith | yeah I know, it's just annoying the difference | 14:10 |
| sean-k-mooney | yep | 14:10 |
| dansmith | ykarel: updated the review, can you try again? | 14:11 |
| ykarel | dansmith, also need to add that check in iso-with-qcow2-in-system | 14:16 |
| dansmith | we don't need udf for that one | 14:16 |
| dansmith | and if I add that check it will not confirm that on centos | 14:17 |
| dansmith | so let me just un-UDF it | 14:17 |
| ykarel | ack | 14:18 |
| dansmith | done | 14:18 |
| ykarel | locally green now generate: OK (0.60=setup[0.05]+cmd[0.55] seconds) | 14:19 |
| dansmith | okay, I'll just ninja this in then | 14:19 |
| dansmith | thanks for your help, sorry for the trouble | 14:19 |
| ykarel | np, thx for the quick fixes | 14:19 |
| dansmith | ykarel: I assume rocky is voting for someone? | 14:20 |
| ykarel | me not sure, in neutron periodic jobs we run with 9-stream | 14:22 |
| dansmith | okay | 14:22 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: DNM: Baseline testing for libvirt offline CPU socket bug https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/926219 | 15:45 |
| ykarel | dansmith, jfyi both 9-stream and rocky jobs passing now | 15:53 |
| dansmith | ykarel: cool, thanks | 15:53 |
| *** artom_ is now known as arotm | 16:04 | |
| *** arotm is now known as artom | 16:04 | |
| *** gibi_ is now known as gibi | 16:27 | |
| opendevreview | Rajat Dhasmana proposed openstack/devstack master: Add config options for optimized upload volume https://review.opendev.org/c/openstack/devstack/+/909841 | 19:37 |
| opendevreview | Rajat Dhasmana proposed openstack/devstack master: Configure cinder service token https://review.opendev.org/c/openstack/devstack/+/910473 | 19:37 |
| kopecmartin | hi there, i'm back from my PTO | 19:52 |
| kopecmartin | gmann: when you have a minute, would you check this please https://review.opendev.org/c/openstack/tempest/+/924785 , thanks | 19:52 |
| gmann | kopecmartin: welcome back :) | 19:52 |
| gmann | sure, checking | 19:52 |
| kopecmartin | i was back already yesterday but i've just realized that i forgot to turn my znc bot for irc on :D | 19:53 |
| gmann | ohk which is good :P | 19:57 |
| gmann | kopecmartin: I have one for you https://review.opendev.org/c/openstack/tempest/+/925290 | 19:57 |
| * kopecmartin checking | 20:03 | |
| opendevreview | Dan Smith proposed openstack/tempest master: Handle glance with fixed ISO handling https://review.opendev.org/c/openstack/tempest/+/926238 | 21:43 |
| opendevreview | Merged openstack/tempest master: Add early failure detection in devstack-tempest-ipv6 https://review.opendev.org/c/openstack/tempest/+/925290 | 21:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!