| *** dviroel|out is now known as dviroel | 11:19 | |
| ecsantos[m] | Hello folks. We (we = felipe_rodrigues) noticed that the Manila DevStack plugin does not honor MANILA_OPTGROUP_<backend>_backend_availabiltiy_zone options: https://opendev.org/openstack/manila/src/commit/02995d468ed5857c0eb10f92333f15a3496fc79a/devstack/plugin.sh#L147-L162 | 13:42 |
|---|---|---|
| ecsantos[m] | This configuration option is listed in the sample manila.conf: https://docs.openstack.org/manila/latest/configuration/shared-file-systems/samples/manila.conf.html | 13:42 |
| ecsantos[m] | Instead of retrieving this option for each backend (when present), a local variable zonenum gets incremented and that's set as the availability zone for each backend sequentially | 13:44 |
| gouthamr | ecsantos[m]: hmmm, yes | 13:44 |
| gouthamr | actually the code you linked is effectively overriding whatever was set | 13:45 |
| ecsantos[m] | Exactly | 13:45 |
| gouthamr | maybe you can iniget and verify if there's a value before the iniset here: https://opendev.org/openstack/manila/src/commit/02995d468ed5857c0eb10f92333f15a3496fc79a/devstack/plugin.sh#L159 | 13:45 |
| gouthamr | i.e., skip writing a value if there's one | 13:46 |
| gouthamr | but yes, a bug | 13:46 |
| ecsantos[m] | Yeah, I think the fix would consist of checking for the option for each backend and setting it, else use the increment | 13:47 |
| gouthamr | yes | 13:48 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Ceph set minimum client version https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/782624 | 14:20 |
| *** dviroel is now known as dviroel|lunch | 15:19 | |
| fzzf[m] | ecsantos: hi, thanks for help. I let zuul return log address by modify zuul code. I found in zuul. generate url have two ways. https://opendev.org/zuul/zuul/src/tag/4.6.0/zuul/model.py#L3039 . if set web-root and report_build_page, it would return build url.else it would use default url(in zuul is log_url) and log-url generate log address. I modify these code. ensure it alway generate log url, then zuul would return log address on | 15:41 |
| fzzf[m] | comment. | 15:41 |
| *** dviroel|lunch is now known as dviroel | 16:18 | |
| ecsantos[m] | fzzf: I think that also works, but you'd need to maintain a local fork of Zuul and keep it updated. Did overriding the zuul.log_url variable not work for you? | 16:37 |
| opendevreview | Douglas Viroel proposed openstack/manila stable/yoga: Define queues at project level https://review.opendev.org/c/openstack/manila/+/847441 | 16:43 |
| opendevreview | Douglas Viroel proposed openstack/manila stable/xena: Define queues at project level https://review.opendev.org/c/openstack/manila/+/847442 | 16:44 |
| opendevreview | Douglas Viroel proposed openstack/manila stable/wallaby: Define queues at project level https://review.opendev.org/c/openstack/manila/+/847443 | 16:45 |
| opendevreview | Douglas Viroel proposed openstack/manila stable/victoria: Define queues at project level https://review.opendev.org/c/openstack/manila/+/847444 | 16:47 |
| opendevreview | Douglas Viroel proposed openstack/manila stable/ussuri: Define queues at project level https://review.opendev.org/c/openstack/manila/+/847445 | 16:48 |
| opendevreview | Carlos Eduardo proposed openstack/manila stable/yoga: Use defined project config of manila-tempest-plugin https://review.opendev.org/c/openstack/manila/+/847400 | 17:27 |
| opendevreview | Douglas Viroel proposed openstack/manila stable/train: Define queues at project level https://review.opendev.org/c/openstack/manila/+/847595 | 17:41 |
| opendevreview | Carlos Eduardo proposed openstack/manila stable/yoga: Use defined project config of manila-tempest-plugin https://review.opendev.org/c/openstack/manila/+/847400 | 18:53 |
| opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: Change the set of jobs in the stable branches https://review.opendev.org/c/openstack/manila-tempest-plugin/+/847366 | 19:38 |
| opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: Change the set of jobs in the stable branches https://review.opendev.org/c/openstack/manila-tempest-plugin/+/847366 | 19:42 |
| opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: Change the set of jobs in the stable branches https://review.opendev.org/c/openstack/manila-tempest-plugin/+/847366 | 19:47 |
| *** dviroel is now known as dviroel|out | 20:50 | |
| opendevreview | Carlos Eduardo proposed openstack/manila stable/yoga: Use defined project config of manila-tempest-plugin https://review.opendev.org/c/openstack/manila/+/847400 | 21:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!