| opendevreview | Ghanshyam proposed openstack/grenade master: Update the master grenade setting https://review.opendev.org/c/openstack/grenade/+/932017 | 00:38 |
|---|---|---|
| melwitt | is there some kind of trick for getting a grenade job to use a lib from git? I have Depends-On a openstack/os-traits patch and I tried to use required-projects: openstack/os-traits but it's not pulling down the patch | 02:25 |
| opendevreview | OpenStack Proposal Bot proposed openstack/devstack master: Updated from generate-devstack-plugins-list https://review.opendev.org/c/openstack/devstack/+/932481 | 02:50 |
| gmann | melwitt: was Depends-On on the old node (branch) ? | 03:27 |
| gmann | if it is of master then grenade still pull old node os-traits from original source | 03:28 |
| melwitt | gmann: no it wasn't. ok, so it doesn't pull os-traits again when it sets up the new node? | 03:28 |
| melwitt | or when it upgrades to new | 03:30 |
| gmann | melwitt: as grenade does not run the devstack on new node, it just pull new code of target (new) branch | 03:32 |
| melwitt | gmann: ah.. ok. thanks for explaining, that must be why it's not working | 03:33 |
| gmann | if you add depends-on of stable/2024.2 and it should pull the required os-traits of stable/2024.2 backport and then upgrade to master one | 03:33 |
| gmann | melwitt: also current grenade job are not yet setup for stable/2024.2 -> master upgrade. it still test the stable/2024.1 --> master | 03:34 |
| gmann | melwitt: my change still not merged as its base change of running grenade on venv is needed https://review.opendev.org/c/openstack/grenade/+/932017/2 | 03:34 |
| gmann | because this will also impact your testing | 03:34 |
| melwitt | gmann: gotcha, thanks for the heads up | 03:35 |
| gmann | maybe depends-on 932017 should work as I am left with ironic fix only | 03:35 |
| opendevreview | Merged openstack/devstack master: Updated from generate-devstack-plugins-list https://review.opendev.org/c/openstack/devstack/+/932481 | 03:41 |
| melwitt | gmann: fyi, looks like os-traits doesn't have stable branches https://opendev.org/openstack/os-traits | 04:04 |
| gmann | ah, right | 04:04 |
| melwitt | although I see there's a variable I can use OS_TRAITS_BRANCH | 04:04 |
| melwitt | https://opendev.org/openstack/devstack/src/branch/master/stackrc#L564 | 04:06 |
| gmann | yeah, TARGET_BRANCH is always 'master' for branchles project | 04:06 |
| gmann | melwitt: this is the change right? https://review.opendev.org/c/openstack/nova/+/932153/4 | 04:08 |
| melwitt | gmann: yes | 04:08 |
| melwitt | I'm wondering if I set LIBS_FROM_GIT and OS_TRAITS_BRANCH if it might work | 04:09 |
| melwitt | in the job definition | 04:09 |
| gmann | melwitt: https://zuul.opendev.org/t/openstack/build/3769f40288644d7b878e176670a4ead2/log/controller/logs/old/devstacklog.txt#3599 | 04:12 |
| melwitt | oh huh, I see it reference the os-traits patch here https://zuul.opendev.org/t/openstack/build/3769f40288644d7b878e176670a4ead2/log/controller/logs/old/devstacklog.txt#3619 | 04:16 |
| melwitt | and there's an error at the end AttributeError: module 'setuptools.build_meta' has no attribute 'get_requires_for_build_editable'. Did you mean: 'get_requires_for_build_sdist'? | 04:17 |
| melwitt | it was grenade-skip-level-always that did not pull the os-traits patch. but nova-grenade-multinode did | 04:20 |
| gmann | melwitt: ohk, so this is exact error i am fixing it in https://review.opendev.org/c/openstack/grenade/+/930507 | 04:20 |
| gmann | can you make this as depends-on | 04:20 |
| melwitt | sure. thanks. sorry I get confused by this | 04:20 |
| gmann | melwitt: i faced the setuptool error when I was setting up the grenade for current master and then I have to move greande to virtual env first | 04:26 |
| melwitt | gmann: gotcha. yeah I was confused why it wasn't failing for anyone else 😆 | 04:28 |
| gmann | and it fail on multinode job only and maybe not many project test the greande on multinode | 04:30 |
| opendevreview | Ghanshyam proposed openstack/grenade master: Enable GLOBAL_VENV in grenade https://review.opendev.org/c/openstack/grenade/+/930507 | 05:14 |
| opendevreview | melanie witt proposed openstack/whitebox-tempest-plugin master: Add tests for Nova image multi-backend https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/931717 | 07:08 |
| opendevreview | melanie witt proposed openstack/whitebox-tempest-plugin master: Add tests for Nova image multi-backend https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/931717 | 07:47 |
| opendevreview | melanie witt proposed openstack/whitebox-tempest-plugin master: Add tests for Nova image multi-backend https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/931717 | 07:49 |
| opendevreview | melanie witt proposed openstack/whitebox-tempest-plugin master: Add tests for Nova image multi-backend https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/931717 | 15:45 |
| opendevreview | melanie witt proposed openstack/whitebox-tempest-plugin master: Add tests for Nova image multi-backend https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/931717 | 18:14 |
| opendevreview | melanie witt proposed openstack/whitebox-tempest-plugin master: Add tests for Nova image multi-backend https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/931717 | 19:38 |
| gmann | melwitt: you can even use the 932017 as depends-on to have grenade jobs testing the latest setting, commented here - https://review.opendev.org/c/openstack/nova/+/932153/comments/a1ee79b8_7be68371 | 19:56 |
| gmann | I am trying to get those merged soon (ironic, octavia fix are in gate but heat fix not yet reviewed ) | 19:56 |
| melwitt | gmann: I did :) it's the only way it will work heh | 19:56 |
| melwitt | thanks for that | 19:56 |
| opendevreview | Ghanshyam proposed openstack/grenade master: Update the master grenade setting https://review.opendev.org/c/openstack/grenade/+/932017 | 19:57 |
| gmann | melwitt: I think you depends-on on 930507 which fix the setuptool error but 932017 ^^ setup the grenade testing for stable/2024.2 -> master | 19:58 |
| gmann | with 932017, you will get the right result for stable/2024.2 -> master upgrade testing | 19:59 |
| gmann | current greande job in your change test stable/2024.1 -> master (old setting) | 19:59 |
| melwitt | gmann: oh, hah you're right. the numbers looked too similar to me 😝 | 19:59 |
| gmann | :) | 19:59 |
| *** gmann is now known as gmann_afk | 20:58 | |
| opendevreview | melanie witt proposed openstack/devstack master: lib/nova: Allow specifying NOVA_SERVICE_DOWN_TIME https://review.opendev.org/c/openstack/devstack/+/932571 | 21:22 |
| *** gmann_afk is now known as gmann_ | 21:41 | |
| *** gmann_ is now known as gmann | 21:41 | |
| gmann | dansmith: clarkb: grenade GLOBAL_VENV is ready, from depends-on only heat fix is left to merge but we can review greande one in parallel https://review.opendev.org/c/openstack/grenade/+/930507 | 21:42 |
| gmann | dansmith: clarkb and this one also which update the grenade testing setting for current master https://review.opendev.org/c/openstack/grenade/+/932017/3 | 21:42 |
| gmann | kopecmartin: ^^ | 21:42 |
| dansmith | gmann: got 'em | 23:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!