| opendevreview | Merged openstack/devstack stable/2026.1: Update devstack stable/2026.1 branch setting https://review.opendev.org/c/openstack/devstack/+/981544 | 01:25 |
|---|---|---|
| *** ykarel__ is now known as ykarel | 04:36 | |
| pdeore | Can gmaan or anyone else from qa team review this https://review.opendev.org/c/openstack/tempest/+/891026 and help to merge it please ? (it has one +1 from glance core) | 06:44 |
| *** gibi_ is now known as gibi | 08:12 | |
| opendevreview | Pranali Deore proposed openstack/tempest master: Add tests for new add location api of glance https://review.opendev.org/c/openstack/tempest/+/891026 | 08:50 |
| *** vhari_ is now known as vhari | 10:21 | |
| opendevreview | Pranali Deore proposed openstack/tempest master: Add tests for new add location api of glance https://review.opendev.org/c/openstack/tempest/+/891026 | 13:13 |
| opendevreview | Pranali Deore proposed openstack/tempest master: Add tests for new add location api of glance https://review.opendev.org/c/openstack/tempest/+/891026 | 13: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 | 15:06 |
| gmaan | frickler: yes, thanks for updating and review | 15:13 |
| gmaan | pdeore_: ack, will check | 15:14 |
| gmaan | dansmith: grenade setup changes for master, stable/2026.1 are ready https://review.opendev.org/c/openstack/grenade/+/981624 https://review.opendev.org/c/openstack/grenade/+/981625 | 15:50 |
| dansmith | looks like frickler beat me (again) | 16:00 |
| frickler | dansmith: why, those could still use a second +2 and approval? | 16:15 |
| dansmith | oh sorry I saw two votes and thought you +2+A'd them | 16:16 |
| opendevreview | Merged openstack/grenade master: Update the master grenade setting https://review.opendev.org/c/openstack/grenade/+/981624 | 17:25 |
| opendevreview | Merged openstack/grenade stable/2026.1: Update grenade setting in stable/2026.1 https://review.opendev.org/c/openstack/grenade/+/981625 | 17:25 |
| gmaan | sean-k-mooney: I left comment in the cyborg grenade job change, that duplicates a lot of base settings which are handled by greande itself https://review.opendev.org/c/openstack/cyborg/+/979802 | 19:02 |
| gmaan | there are easy way to append the extra setting specific to project via job so that project specific grenade jobs do not need to duplicate the base setting and their maintenance on every release | 19:03 |
| sean-k-mooney | gmaan: cak thanks.im really not that familar with the greande jobs itslef in terms of its defaults | 19:04 |
| sean-k-mooney | gmaan: doesnt the parent grenade job add openstack/grenade in required-projects | 19:05 |
| gmaan | np!, I commented in change about how the settings can be done via job variables | 19:05 |
| sean-k-mooney | gmaan: so regarding https://review.opendev.org/c/openstack/cyborg/+/979802/25/devstack/upgrade/devstack.local.conf.base-grenade ectra | 19:05 |
| sean-k-mooney | orgianlly didnt want to include them at all but chandan was addign them for non ci usage | 19:06 |
| gmaan | sean-k-mooney: yes, required-projects one is taken care by grenade job | 19:06 |
| sean-k-mooney | i.e. to simplfy running grenade locally | 19:06 |
| sean-k-mooney | that was one of the things i was unsuer of we shoudl do ro not | 19:06 |
| gmaan | Still, for non-CI usage, you still need the GitHub repo to clone, right? so cyborg readme can mentioned to add extra settings in grenade rc file | 19:07 |
| sean-k-mooney | right that is waht i was orginally thinking too | 19:07 |
| sean-k-mooney | i just didnt really knwo how to express that in the review becuase its been years since i manually ran greande | 19:07 |
| sean-k-mooney | as in the last time i did that i was still at intel | 19:08 |
| gmaan | I mean those can be shown as example in readme but controlling grenade jobs via those extra file will give more maintenance to cyborg | 19:08 |
| sean-k-mooney | oh ya those extra files were never intended ot be used in teh jobs | 19:08 |
| sean-k-mooney | so if we can instea djus thave a quick readme that expalins what to set and how to use grenade locally that woudl be my prefence | 19:09 |
| sean-k-mooney | so basicly update https://review.opendev.org/c/openstack/cyborg/+/979802/25/devstack/upgrade/README.rst | 19:09 |
| gmaan | better to keep it simple and add local run things by appending cyborg setting in exosting grenade files | 19:09 |
| sean-k-mooney | yep | 19:10 |
| gmaan | sean-k-mooney: I will suggest to add that as a separate effort and by testing it locally. and thios change only for running greande testing in upstream | 19:10 |
| sean-k-mooney | ack ya spliting it also makes sense | 19:14 |
| opendevreview | Tanvi Nautiyal proposed openstack/tempest master: Add new test case for Swift account_quota https://review.opendev.org/c/openstack/tempest/+/955051 | 20:08 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Remove skip check for vtpm live migration https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/981411 | 20:28 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Add vtpm_live_migration parameter to config https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/981411 | 21:00 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: [DNM] Addressing vtpm_live_migration parameter downstream https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/981411 | 21:45 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!