| *** artom__ is now known as artom | 02:07 | |
| gtema | Thks frickler | 05:28 |
|---|---|---|
| gtema | I also experienced merge issues (lost parts). Especially on such huge merge this can also easily happen. Would also try to check today | 05:30 |
| frickler | gtema: with https://review.opendev.org/c/openstack/openstacksdk/+/824674 you can now see the osc-unit failures on the r1 branch already. I can try to do the same for the osc-functional job, then we could clean up the issues on the feature branch directly | 06:13 |
| gtema | Yeah, thanks | 06:55 |
| *** holser is now known as holser_ | 12:35 | |
| *** Guest9440 is now known as diablo_rojo_phone | 14:01 | |
| *** holser_ is now known as holser | 14:12 | |
| *** holser is now known as holser_ | 14:24 | |
| opendevreview | Nicolas Belouin proposed openstack/python-openstackclient master: Add trustor and trustee filtering to trusts list https://review.opendev.org/c/openstack/python-openstackclient/+/824721 | 14:56 |
| *** holser_ is now known as holser | 15:58 | |
| opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Skip original_name from flavor info https://review.opendev.org/c/openstack/python-openstackclient/+/824733 | 16:39 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Revert "Add "security_group_ids" to Port's query parameters" https://review.opendev.org/c/openstack/openstacksdk/+/824751 | 16:41 |
| opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | 17:00 |
| opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Fixes for the upcoming major release of the SDK https://review.opendev.org/c/openstack/python-openstackclient/+/824737 | 17:01 |
| gtema | oh frickler, I was also on that right now :) | 17:01 |
| gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/824733 is what you also changed | 17:02 |
| gtema | and for network it will be a little bigger patch - need to clean more network resources - I am on it now | 17:02 |
| frickler | gtema: ah, I submitted without checking. I also pushed https://review.opendev.org/c/openstack/openstacksdk/+/824741 to test my change with the r1 branch, will wait for results there before abandoning my change again | 17:12 |
| gtema | sure | 17:12 |
| gtema | I think it is worse having 2 patches for cleaning up flavors and network stuff in sdk. Especially network is a bigger one | 17:13 |
| gtema | additionally flavor failures are voting, network non voting (may help merging pain) | 17:13 |
| frickler | gtema: yeah, I'm fine with that, I was mainly wanting to see now whether my local testing would also be confirmed by the CI | 17:14 |
| gtema | cool | 17:14 |
| opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Get rid of tenant_id in the network commands https://review.opendev.org/c/openstack/python-openstackclient/+/824746 | 17:53 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | 17:56 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | 18:39 |
| frickler | gtema: the revert together with your osc patches passes osc unit+func https://review.opendev.org/c/openstack/openstacksdk/+/824741 , with that I'm out for now, feel free to ping if anything else comes up | 19:15 |
| gtema | Cool, thks | 21:28 |
| *** holser is now known as holser_ | 22:29 | |
| *** holser_ is now known as holser | 23:05 | |
| *** holser is now known as holser_ | 23:15 | |
| *** holser_ is now known as holser | 23:52 | |
| *** holser is now known as holser_ | 23:52 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!