| kopecmartin | tosky: if the manila plugin pin is the only issue with this https://review.opendev.org/c/openinfra/python-tempestconf/+/853517 i'd say, let's merge, we can always get back to that, what do you think? | 09:07 |
|---|---|---|
| kopecmartin | i need this patch in refstack-client and it's more complicated to cherry pick everything than just merge this | 09:07 |
| tosky | kopecmartin: ook | 09:09 |
| tosky | kopecmartin: I still think that the manila devstack plugin shouldn't install the manila-tempest-plugin | 09:09 |
| tosky | should someone else vote too, or could that get a single approver? | 09:10 |
| kopecmartin | thanks tosky, single approve is ok | 09:13 |
| kopecmartin | i can +w if you want | 09:13 |
| tosky | done | 09:13 |
| kopecmartin | thank you | 09:15 |
| opendevreview | Merged openinfra/ansible-role-refstack-client master: Change default ssh_key_type to ecdsa https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/846109 | 09:46 |
| opendevreview | Martin Kopec proposed openinfra/ansible-role-refstack-client master: Update default guideline to 2022.06 https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/850877 | 09:49 |
| opendevreview | Martin Kopec proposed openinfra/ansible-role-refstack-client master: Exclude compute and object programs from defaults https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/853619 | 09:49 |
| opendevreview | Merged openinfra/ansible-role-refstack-client master: Update default guideline to 2022.06 https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/850877 | 10:04 |
| opendevreview | Merged openinfra/ansible-role-refstack-client master: Exclude compute and object programs from defaults https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/853619 | 10:04 |
| opendevreview | Martin Kopec proposed openinfra/python-tempestconf master: DNM Testing out packstack-centos-9 job https://review.opendev.org/c/openinfra/python-tempestconf/+/854172 | 11:26 |
| opendevreview | Merged openinfra/python-tempestconf master: Drop py3.6/7, T, U jobs and fix neutron discovery https://review.opendev.org/c/openinfra/python-tempestconf/+/853517 | 12:21 |
| opendevreview | Martin Kopec proposed openinfra/refstack-client master: Update tempest version and drop U and T jobs https://review.opendev.org/c/openinfra/refstack-client/+/846370 | 12:57 |
| gouthamr | o/ tosky | 13:46 |
| gouthamr | re: <tosky> kopecmartin: I still think that the manila devstack plugin shouldn't install the manila-tempest-plugin | 13:46 |
| gouthamr | where do yo see this happening? | 13:46 |
| * gouthamr was looking at logs on https://review.opendev.org/c/openinfra/python-tempestconf/+/853517 and is confused | 13:46 | |
| tosky | gouthamr: I've tried to summarize the reasoning in the comments in that review | 13:52 |
| tosky | if I understand it correctly, if the manila devstack plugin installs manila-tempest-plugin, it follows the upper-constraints which contains a specific version of tempest, older than master, and that doesn't work | 13:53 |
| gouthamr | ah | 13:54 |
| gouthamr | this might be solved with a devstack variable | 13:54 |
| gouthamr | "MANILA_INSTALL_TEMPEST_PLUGIN_SYSTEMWIDE=false" | 13:54 |
| gouthamr | we were doing the wrong thing by installing the tempest plugin *within* manila's devstack plugin until wallaby | 13:55 |
| gouthamr | but we excised that bit after, and added a variable to prevent installation in wallaby and earlier branches | 13:55 |
| tosky | uh, and in fact the affected branches are wallaby and victoria | 13:56 |
| tosky | that should solve it, thanks | 13:56 |
| gouthamr | ++ | 13:56 |
| opendevreview | Martin Kopec proposed openinfra/ansible-role-refstack-client master: Pin manila-tempest-plugin in W and V CI jobs https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/854211 | 14:09 |
| tosky | kopecmartin: have you seen ^^? | 14:09 |
| tosky | we may not need the pinning | 14:09 |
| * kopecmartin reading | 14:10 | |
| kopecmartin | where should we set that? job definition as a devstack var? | 14:12 |
| opendevreview | Goutham Pacha Ravi proposed openinfra/python-tempestconf master: [CI] Fix manila tempest plugin issue in W/V https://review.opendev.org/c/openinfra/python-tempestconf/+/854213 | 14:17 |
| gouthamr | i think so, kopecmartin ^ | 14:17 |
| opendevreview | Martin Kopec proposed openinfra/ansible-role-refstack-client master: Uncomment previously broken jobs https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/854214 | 14:18 |
| kopecmartin | gouthamr: ok, i'll try that | 14:19 |
| kopecmartin | thanks | 14:19 |
| opendevreview | Martin Kopec proposed openinfra/refstack-client master: Update tempest version and drop U and T jobs https://review.opendev.org/c/openinfra/refstack-client/+/846370 | 14:38 |
| opendevreview | Martin Kopec proposed openinfra/python-tempestconf master: DNM Testing out packstack-centos-9 job https://review.opendev.org/c/openinfra/python-tempestconf/+/854172 | 14:53 |
| opendevreview | Merged openinfra/ansible-role-refstack-client master: Pin manila-tempest-plugin in W and V CI jobs https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/854211 | 19:09 |
| opendevreview | Martin Kopec proposed openinfra/ansible-role-refstack-client master: Uncomment previously broken jobs, remove V, U ones https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/854214 | 19:20 |
| opendevreview | Merged openinfra/refstack-client master: Update tempest version and drop U and T jobs https://review.opendev.org/c/openinfra/refstack-client/+/846370 | 20:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!