| opendevreview | JaromÃr Wysoglad proposed openstack/devstack master: Fix neutron empty string check https://review.opendev.org/c/openstack/devstack/+/914440 | 06:38 |
|---|---|---|
| tosky | hi, I've noticed that the image-feature-enabled.os_glance_reserved option can be switched to True by default, or even removed, because all glance versions post wallaby supports that feature: https://opendev.org/openstack/tempest/src/branch/master/tempest/config.py#L697 | 10:31 |
| tosky | what would it be the best way to proceed? Move to true or just remove? | 10:31 |
| tkajinam | tosky, this is what I proposed some time ago https://review.opendev.org/c/openstack/tempest/+/906462 | 10:37 |
| tkajinam | probably we can continue the discussion in it ? | 10:38 |
| tosky | oh, sure, nice! | 10:39 |
| tosky | or maybe do it on by one? It looks like the other option is more controversial | 10:41 |
| tkajinam | yeah | 10:41 |
| tkajinam | I was hoping to get some opinions about the other options from glance folks, so am "stealing" this chance for it :-P | 10:42 |
| tkajinam | if glance folks can confirm the other one can be also removed then we can proceed with the current. if not then I can update the patch to cover only os_glance_reserved. | 10:42 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/2023.1: Fix manila jobs on stable/2023.1 branch https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/909055 | 12:53 |
| tosky | tkajinam: glance feedback (I asked a few people directly) is that it would be better to split the patch into two | 14:45 |
| tkajinam | tosky, ok. let me update my patch then | 14:57 |
| opendevreview | Takashi Kajinami proposed openstack/tempest master: Deprecate os_glance_reserved https://review.opendev.org/c/openstack/tempest/+/906462 | 15:06 |
| opendevreview | Takashi Kajinami proposed openstack/tempest master: Deprecate os_glance_reserved https://review.opendev.org/c/openstack/tempest/+/906462 | 15:07 |
| tkajinam | tosky, ^^^ | 15:07 |
| tkajinam | abhishekk, I'll create a separate change for image_import. fyi | 15:08 |
| opendevreview | Takashi Kajinami proposed openstack/tempest master: Deprecate import_image https://review.opendev.org/c/openstack/tempest/+/915729 | 15:17 |
| opendevreview | Takashi Kajinami proposed openstack/tempest master: Deprecate import_image https://review.opendev.org/c/openstack/tempest/+/915729 | 15:17 |
| opendevreview | Luigi Toscano proposed openstack/tempest master: WIP Enable snapshot deletion with deps on ceph https://review.opendev.org/c/openstack/tempest/+/782392 | 15:41 |
| sean-k-mooney | tosky: gmann i belive this is what is required to do a release of the whitebox-tempest-plugin upstream but we apprently have never done that since it became a deliverable of the qa team | 17:02 |
| sean-k-mooney | https://review.opendev.org/c/openstack/releases/+/915751 | 17:02 |
| sean-k-mooney | those ^ look reasonabel to people | 17:02 |
| sean-k-mooney | https://github.com/openstack/governance/blob/master/reference/projects.yaml#L2254-L2257 | 17:03 |
| sean-k-mooney | i guess its tecnially release-management: none | 17:03 |
| sean-k-mooney | i tought it was independent | 17:03 |
| sean-k-mooney | tosky_: you were asking for the new tag ^ for https://review.rdoproject.org/r/c/openstack/whitebox-tempest-plugin-distgit/+/52480 | 17:04 |
| sean-k-mooney | should i first propose a seperate patch to the governance repo to make it release-modle independnet | 17:06 |
| sean-k-mooney | release modle None is not technial a thing defiend in https://github.com/openstack/releases/blob/master/doc/source/reference/release_models.rst | 17:06 |
| sean-k-mooney | https://github.com/openstack/releases/blob/master/doc/source/reference/release_models.rst#independent i belive is what we orginally intended | 17:06 |
| sean-k-mooney | the other way to create a tag would be for myself or artom to push a signed tag manulaly | 17:09 |
| *** atmark_ is now known as atmark | 17:16 | |
| gmann | sean-k-mooney: you do not need to add release model in governance. Most of the (i think all) tempest plugin are cycle-with-intermediary which is same as Tempest. I release them most of the time to match the Tempest versions | 17:19 |
| gmann | sean-k-mooney: but if you want whitebox-tempest-plugin to be in independent release it is totally fine. | 17:19 |
| sean-k-mooney | artom: ^ | 17:20 |
| gmann | sean-k-mooney: commented on the change, that is only needed, once you fix that I will +1 | 17:20 |
| sean-k-mooney | hum i dont think he is here ill get him to join and readmack | 17:20 |
| gmann | sure, and cycle-with-intermediary means as you know we need to have a at least one release per cycle. if this plugins does not have much to release every cycle I will suggest independent release model is ok | 17:21 |
| sean-k-mooney | so far its just been used directly form git mostly | 17:22 |
| sean-k-mooney | so we didnt need release or tags | 17:22 |
| gmann | k, tag/release is helpful for un-supported branches testing (unmaintained branches now). otherwise yes Tempest and all plugins are used form master to test master as well as all supported releases | 17:23 |
| sean-k-mooney | we still have some python 2 support that we want to drop and wtih tripleo going way we will likely want to clean up some of the tripleo specific support | 17:26 |
| sean-k-mooney | i have not had time to check with artom why this was desired now | 17:27 |
| sean-k-mooney | so i dont know if they wanted to tag before looking at that clean up | 17:27 |
| sean-k-mooney | or if it was for some other reason | 17:27 |
| gmann | ack | 17:29 |
| abhishekk | tkajinam, ack | 18:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!