| opendevreview | Merged openstack/patrole master: Move to Xena testing runtime template https://review.opendev.org/c/openstack/patrole/+/788511 | 00:32 |
|---|---|---|
| opendevreview | Ade Lee proposed openstack/tempest master: WIP Add support for ecdsa keys https://review.opendev.org/c/openstack/tempest/+/807465 | 03:31 |
| opendevreview | Ade Lee proposed openstack/tempest master: Add skip for tests that wont work under fips https://review.opendev.org/c/openstack/tempest/+/810808 | 03:31 |
| opendevreview | Ade Lee proposed openstack/tempest master: DNM - Add temporary link to patched paramiko https://review.opendev.org/c/openstack/tempest/+/810809 | 03:31 |
| *** akekane_ is now known as abhishekk | 05:23 | |
| *** rpittau|afk is now known as rpittau | 06:41 | |
| *** jpena|off is now known as jpena | 07:07 | |
| opendevreview | Alfredo Moralejo proposed openstack/devstack master: [WIP] Support CentOS Stream 9 https://review.opendev.org/c/openstack/devstack/+/800903 | 07:40 |
| opendevreview | Merged openstack/patrole master: Add stable/wallaby job on master gate https://review.opendev.org/c/openstack/patrole/+/788746 | 08:17 |
| opendevreview | Alfredo Moralejo proposed openstack/devstack master: [WIP] Support CentOS Stream 9 https://review.opendev.org/c/openstack/devstack/+/800903 | 08:42 |
| opendevreview | Lee Yarwood proposed openstack/tempest master: zuul: Add a CentOS 8 stream job to integrated-gate-compute https://review.opendev.org/c/openstack/tempest/+/797614 | 10:52 |
| *** jpena is now known as jpena|lunch | 11:02 | |
| *** bhagyashris is now known as bhagyashris|rover | 11:09 | |
| *** dviroel|out is now known as dviroel | 11:13 | |
| opendevreview | yatin proposed openstack/devstack master: [WIP] Support CentOS Stream 9 https://review.opendev.org/c/openstack/devstack/+/800903 | 11:22 |
| *** jpena|lunch is now known as jpena | 11:52 | |
| opendevreview | OpenStack Release Bot proposed openstack/devstack stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/devstack/+/810893 | 12:53 |
| opendevreview | OpenStack Release Bot proposed openstack/devstack stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/devstack/+/810894 | 12:53 |
| frickler | yoctozepto: I added some comments on the bullseye patch, do you want to continue with that or should I take it over again? both options are fine for me, I just want to avoid duplicating efforts | 13:03 |
| opendevreview | Lee Yarwood proposed openstack/tempest master: WIP scenario: Migrate attached read-only multiattach volume https://review.opendev.org/c/openstack/tempest/+/810514 | 13:24 |
| yoctozepto | frickler: I'm drowning in tasks atm, you are very welcome to take it! :-) | 13:27 |
| frickler | yoctozepto: o.k., so I'll do some more runs with concurrency=2 and then try to finalize this | 13:45 |
| amodi | artom: lyarwood does it look good to merge now? https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/786979 | 14:17 |
| artom | amodi, I'm looking at it now | 14:18 |
| amodi | ack | 14:18 |
| artom | amodi, there, I think the domblklist helper can be improved | 14:26 |
| *** pojadhav is now known as pojadhav|out | 14:26 | |
| amodi | sure, /me checks | 14:30 |
| opendevreview | OpenStack Release Bot proposed openstack/grenade stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/grenade/+/810926 | 14:37 |
| opendevreview | OpenStack Release Bot proposed openstack/grenade stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/grenade/+/810927 | 14:37 |
| opendevreview | Archit Modi proposed openstack/whitebox-tempest-plugin master: Add Rbd direct download test https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/786979 | 14:46 |
| amodi | artom: ^ did implement it, only somewhat-of-an-issue is if I return None, the self.assertNotIn still would pass for 'rbd'. Do i add another assert to check if path exists? | 14:50 |
| amodi | if the source does not exist, ill get an UnboundLocalError, so it will anyway fail | 14:51 |
| artom | amodi, oh right, because assertNotIn('rbd', None) is True | 14:52 |
| amodi | yup | 14:52 |
| artom | I guess it's legit to assume get_server_blockdevice_path always returns something... | 14:53 |
| amodi | yeah ill remove returning None then, from get-server_blockpth | 14:53 |
| artom | I mean the thing is... if would still return None implicitly if there was programmer error | 14:54 |
| artom | IOW, you pass it 'vdda' or something and it doesn't find that line in the domblklist output | 14:55 |
| artom | So maybe it's more robust to assert not none, and then assert the actual value | 14:55 |
| amodi | artom: it will fail the test with Unboundlocalerror, source is not intiatied before the | 14:55 |
| amodi | i'd tried it and it had failed | 14:56 |
| amodi | UnboundLocalError: local variable 'source' referenced before assignment Ran 1 test in 93.815s FAILED (failures=1) | 14:57 |
| artom | amodi, oh right, my bad, we have to set source = None before the loop | 15:05 |
| artom | `source or None` doesn't work if `source` has never been init'ed | 15:05 |
| amodi | yup | 15:05 |
| amodi | artom: i guess then, i can assert this: assertIsNotNone(path) or assertNotIn('rbd', path) | 15:08 |
| amodi | intiate source to None also | 15:08 |
| artom | amodi, 'and' instead of 'or', surely? | 15:08 |
| amodi | oh yes, and | 15:08 |
| amodi | cool | 15:08 |
| artom | Yeah, that sounds good to me | 15:08 |
| opendevreview | Archit Modi proposed openstack/whitebox-tempest-plugin master: Add Rbd direct download test https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/786979 | 15:13 |
| opendevreview | Archit Modi proposed openstack/whitebox-tempest-plugin master: Add Rbd direct download test https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/786979 | 15:15 |
| yoctozepto | frickler: ack | 15:28 |
| opendevreview | Merged openstack/devstack stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/devstack/+/810893 | 16:06 |
| opendevreview | Merged openstack/devstack stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/devstack/+/810894 | 16:06 |
| *** jpena is now known as jpena|off | 16:11 | |
| clarkb | heads up a lot of devstack based jobs are retry limiting because something is trying to create the swapfile twice on the same host which leads to "mkswap: error: /root/swapfile is mounted; will not make swapspace" | 16:19 |
| clarkb | https://zuul.opendev.org/t/openstack/build/52fa83b18e1a476b80afc41617fe68f9/log/job-output.txt#1138 is an example of this. Note the controller laready had swap created at https://zuul.opendev.org/t/openstack/build/52fa83b18e1a476b80afc41617fe68f9/log/job-output.txt#923 | 16:19 |
| clarkb | fungi: ^ I was out yesterday, curious if you have any idea what might have triggered that | 16:22 |
| clarkb | also unrelated it looks like openstack health is dead now? I don't think we've started deleting any of the hosts around that, but this is why we'd like to. | 16:23 |
| fungi | clarkb: nope, nobody mentioned swap issues, no clue what that's about | 16:25 |
| fungi | and no we haven't deleted anything around openstack-health yet | 16:25 |
| fungi | something's probably just crashed again | 16:25 |
| fungi | or we filled up the subunit db or something | 16:26 |
| clarkb | ya I think the subunit workers are not putting new data into the database | 16:26 |
| clarkb | it warns of stale data when you load up the page | 16:26 |
| clarkb | looking at the rendered console the pre playbook is running twice | 16:39 |
| clarkb | I think configure-swap should ideally be idempotent, but this also points out what is likely a bug so maybe it is good that it isn't | 16:39 |
| fungi | is someone double-adding a pre-run stanza, like in a job which is already inheriting it? | 16:44 |
| clarkb | fungi: ya looking at the inheritance path I see some oddness with the feature/r1 branch | 16:44 |
| clarkb | I think that may be what is happening: we've got two branches adding the pre.yaml for some reason | 16:45 |
| clarkb | also xena seems to be in the mix https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_52f/800291/2/check/neutron-tempest-plugin-scenario-linuxbridge/52fa83b/zuul-info/inventory.yaml | 16:45 |
| fungi | what project has a feature/r1 branch? | 16:46 |
| clarkb | I think that may be openstacksdk? | 16:46 |
| clarkb | I'm not compeltely sure | 16:46 |
| clarkb | but I think that may be noise I want to say that has been stable for a bit. The real issue is likely related to the xena branch ending up in the inheritance tree there | 16:47 |
| clarkb | and we're running master and xena pre.yaml pre-run playbooks | 16:47 |
| fungi | oh, because of some project(s) having not yet branched for stable/xena? | 16:48 |
| clarkb | Maybe? though I doubt it? Seems like somehow the devstack-tempest job is finding devstack's master and xena configs. I wonder if devstack recently branched and has a branch directive for master that is now on xena | 16:49 |
| clarkb | yes that is exactly it | 16:49 |
| clarkb | https://opendev.org/openstack/devstack/src/branch/stable/xena/.zuul.yaml#L1-L7 that needs to be deleted I think | 16:50 |
| fungi | oh, ouch | 16:50 |
| clarkb | I'll push a change | 16:50 |
| fungi | happy to +2, i think i have core on devstack anyway | 16:50 |
| opendevreview | Clark Boylan proposed openstack/devstack stable/xena: Remove the zuul pragma to match master and feature/r1 https://review.opendev.org/c/openstack/devstack/+/810947 | 16:53 |
| clarkb | Basically all master (and maybe xena) devstack based jobs will fail until that merges. And possibly after that merges if that is an incomplete fix | 16:53 |
| clarkb | fungi: any idea on how we can tell when the xena branch was created? I'd like to write an email about this to openstack-discuss and having a rough time frame for when this started would be good. I guess some openstack/releases job creates it? | 16:54 |
| fungi | should have been a merge to openstack/releases | 16:55 |
| fungi | checking | 16:55 |
| fungi | looks like the most i can give on devstack (at least stable/xena) is cr +1 | 16:55 |
| fungi | clarkb: https://review.opendev.org/810864 Create stable/xena for devstack Submitted-at: Fri, 24 Sep 2021 12:47:26 +0000 | 16:57 |
| fungi | so a little over 4 hours | 16:57 |
| clarkb | https://zuul.opendev.org/t/openstack/build/ebe77aee0c104c9e9b917ff28a4731ee/log/job-output.txt#715 seems to be the exact time | 16:57 |
| clarkb | Alright I'll write that email now, link to the fix, and then go back to reviewing these zuul changes | 16:58 |
| *** dviroel is now known as dviroel|afk | 17:49 | |
| clarkb | I'm hoping others will be able to review and approve that change but I guess if it passes check testing I can go ahead and self approve it | 17:55 |
| opendevreview | Merged openstack/grenade stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/grenade/+/810926 | 18:17 |
| clarkb | a tempest test faield in the main tempest job for 810947 | 18:18 |
| clarkb | I guess I approve and recheck it anyway | 18:18 |
| gmann | clarkb: thanks, +2 | 18:20 |
| gmann | we need to add a note there to take care of it while cutting stable branch | 18:20 |
| fungi | gmann: in this case i think it's a one-off, but yes if that pragma is still there on master in six months you'll want to undo it on stable/yoga as well | 18:25 |
| fungi | so i agree, a code comment next to it may help, *if* people notice it/remember | 18:25 |
| gmann | yeah, I will also add it in QA release wiki page so that we do not forget it | 18:27 |
| fungi | good idea | 18:34 |
| clarkb | right depends on how long that feature/r1 branch will be around | 18:40 |
| *** dviroel|afk is now known as dviroel | 21:00 | |
| *** dviroel is now known as dviroel|out | 21:53 | |
| opendevreview | Ronelle Landy proposed openstack/tempest master: DNM - test python syntax fix https://review.opendev.org/c/openstack/tempest/+/810969 | 22:22 |
| opendevreview | Merged openstack/devstack stable/xena: Remove the zuul pragma to match master and feature/r1 https://review.opendev.org/c/openstack/devstack/+/810947 | 22:47 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/xena: Update stackrc for stable/xena branch https://review.opendev.org/c/openstack/devstack/+/810991 | 22:52 |
| opendevreview | Ghanshyam proposed openstack/grenade master: Update the master grenade setting for xena to master upgrade https://review.opendev.org/c/openstack/grenade/+/810992 | 22:52 |
| opendevreview | Ghanshyam proposed openstack/grenade stable/xena: Update grenade setting in stable/xena https://review.opendev.org/c/openstack/grenade/+/810994 | 22:56 |
| opendevreview | Ghanshyam proposed openstack/devstack master: Update DEVSTACK_SERIES to yoga https://review.opendev.org/c/openstack/devstack/+/810995 | 23:01 |
| opendevreview | Ghanshyam proposed openstack/devstack stable/xena: Cap max microversions for stable/xena https://review.opendev.org/c/openstack/devstack/+/810997 | 23:08 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add Tempest gate job for stable/xena https://review.opendev.org/c/openstack/tempest/+/810998 | 23:12 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add Tempest gate job for stable/xena https://review.opendev.org/c/openstack/tempest/+/810998 | 23:20 |
| gmann | kopecmartin: this is remaining setup for Qa projects of xena release https://review.opendev.org/q/topic:%22qa-xena-release%22+(status:open%20OR%20status:merged) | 23:21 |
| gmann | also updated our wiki page to remove the devstack-gate settings which we do not need anymore | 23:23 |
| gmann | neutron/volume/swift extensions cap for stable/xena is pending which I can do on monday | 23:23 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!