| opendevreview | Merged openstack/governance master: Drop private classifier https://review.opendev.org/c/openstack/governance/+/991343 | 03:11 |
|---|---|---|
| opendevreview | Merged openstack/governance master: pti: Rework Python guide https://review.opendev.org/c/openstack/governance/+/988438 | 03:11 |
| opendevreview | Merged openstack/governance master: pti: Update with references to dependency groups https://review.opendev.org/c/openstack/governance/+/988439 | 03:11 |
| opendevreview | Merged openstack/governance master: pti: Update with references to bindep profiles https://review.opendev.org/c/openstack/governance/+/989178 | 03:11 |
| opendevreview | Merged openstack/governance master: pti: Insist on minimal pyproject.toml https://review.opendev.org/c/openstack/governance/+/989193 | 03:11 |
| toabctl | hey. https://review.opendev.org/c/openinfra/python-tempestconf/+/991768 has a chain of commits to make python-tempestconf CI happy again . tosky are you the right person to ask for a review? this is my first openstack commit since 6 years or so. I hope I did everything correct ... | 09:34 |
| tosky | toabctl: I am one of people with commit access, yes, thanks | 09:42 |
| tosky | toabctl: but all of the changes needed at the same time? I see some fixes for services and at the same time real fixes for the CI, can the chain be broken? | 09:43 |
| tosky | ideally the CI should be fixed first without adding anything else | 09:43 |
| toabctl | tosky: it's really a chain of broken things. | 11:44 |
| toabctl | but let me try to reorder to make the CI happy first and then do the bug fixing... | 11:46 |
| tosky | toabctl: thanks - and each commit must be green | 11:53 |
| toabctl | tosky: if each commit must be green, then I have to squash commits. because there is multiple things broken with current CI. do you want me to do that? | 11:55 |
| toabctl | tosky: https://review.opendev.org/c/openinfra/python-tempestconf/+/991843 is the single commit changeset that does fix the CI | 12:26 |
| tosky | toabctl: thanks, quickly here " Since tempest change b78b60c5a" can you please reference the change-id instead of the commit, as it was done for "devstack change Iddd45e08"? | 12:35 |
| toabctl | tosky: done | 12:38 |
| toabctl | tosky: voting jobs are green now. ready for a review. I'll create a follow up with the bug fix when this change got merged. | 13:42 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.13 and will be offline starting at 1600 UTC. We have allocated an hour for the outage window lasting until 1700 UTC. https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/thread/UIFRQR34WYMF3KDMNQFMO222DIK3HTUD/ | 15:01 | |
| toabctl | tosky: https://review.opendev.org/c/openinfra/python-tempestconf/+/991669 is now the follow-up bugfix (why I started to look into the whole thing) . would be great if you could have a look at that one, too | 15:05 |
| tosky | toabctl: thanks again. If you can share it, do you (your organization) actively use python-tempestconf? | 15:11 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.13 and will be offline momentarily. We have allocated an hour for the outage window lasting until 1700 UTC. https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/thread/UIFRQR34WYMF3KDMNQFMO222DIK3HTUD/ | 16:00 | |
| toabctl | tosky: I do use it together with tempest for testing. isn't that the way how things should be tested nowadays? | 16:21 |
| toabctl | kolla-ansilbe is also using it. | 16:22 |
| tosky | well, unless you have an installer which can deploy tempest.conf from the real configuration used to deploy the whole thing :D | 16:32 |
| tosky | (as devstack does) | 16:32 |
| tosky | the tempest.conf generated by python-tempestconf is always meant to be a starting point which should be checked, fixed and extended | 16:33 |
| toabctl | tosky: thx for the reviews. can we also get a new release of python-tempestconf? what's the workflow for that? | 16:47 |
| tosky | at the moment, it's me pushing a release tag :D | 16:56 |
| toabctl | thx! | 17:39 |
| *** skandix29 is now known as skandix2 | 22:10 | |
| *** mrunge_ is now known as mrunge | 23:39 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!