| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-10 | |||
| 19:52:42 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 22:33:17 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 22:34:51 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 23:28:32 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 23:36:34 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 23:41:59 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| #openstack-sdks - 2022-01-11 | |||
| 02:33:28 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 02:36:30 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 06:05:04 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 06:25:02 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Fix volume set functional test https://review.opendev.org/c/openstack/python-openstackclient/+/824049 | |
| 14:02:01 | frickler | gtema: I would think ^^ is good to go with three passes now or are you waiting for other reviews? | |
| 14:02:26 | gtema | +A'ed | |
| 14:02:36 | gtema | was waiting to be sure you are done | |
| 14:02:41 | gtema | thnks a lot | |
| 14:20:50 | opendevreview | Ashraf Hasson proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 15:06:07 | opendevreview | Merged openstack/python-openstackclient master: Fix volume set functional test https://review.opendev.org/c/openstack/python-openstackclient/+/824049 | |
| 15:49:00 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 15:50:15 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 17:00:19 | opendevreview | Ashraf Hasson proposed openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 17:14:49 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 17:17:52 | diablo_rojo | gtema, around? Got 30 min in 13 min? :D | |
| 17:18:07 | gtema | uhm, okay | |
| 17:18:21 | gtema | was just going to close for today ;-) | |
| 17:21:33 | diablo_rojo | How about for 15 min in 9 min then? :) | |
| 17:21:34 | diablo_rojo | Better? | |
| 17:21:38 | diablo_rojo | Plus a beer in Berlin? | |
| 17:21:46 | gtema | no problems | |
| 17:21:52 | gtema | beer in Berlin sounds like a plan | |
| 17:22:11 | diablo_rojo | Just thought it might be good to have you meet the new OSU interns. | |
| 17:22:22 | gtema | sure | |
| 17:23:04 | diablo_rojo | We should do a team dinner in Berlin. | |
| 17:23:18 | gtema | :) | |
| 17:31:50 | diablo_rojo | stephenfin, going to be able to join us today? | |
| 17:49:08 | stephenfin | yeah, let's use Zoom going forward | |
| 17:49:58 | gtema | yupp, clearly better than what we had last time | |
| 17:50:30 | diablo_rojo | Enterprise software will do that sometimes unfortunately. | |
| 17:50:45 | diablo_rojo | gtema, I see you on the invite as a no already? | |
| 17:51:03 | diablo_rojo | I tried removing you and readding you but I dont think it sent a notification.. | |
| 17:51:13 | gtema | oh, I actually only declined 2 occurrences in Dec | |
| 17:52:33 | diablo_rojo | Hmmm | |
| 17:53:16 | diablo_rojo | I sent an email notification for next week's occurance. | |
| 17:53:22 | diablo_rojo | Maybe that will trigger something? | |
| 17:53:32 | gtema | let's see | |
| 17:55:33 | gtema | yey, I was able to recover it | |
| #openstack-sdks - 2022-01-12 | |||
| 08:35:16 | opendevreview | mbu proposed openstack/cliff stable/victoria: Add conflict_handler parameter as attribut in Command class https://review.opendev.org/c/openstack/cliff/+/824304 | |
| 08:36:24 | opendevreview | mbu proposed openstack/cliff stable/ussuri: Add conflict_handler parameter as attribut in Command class https://review.opendev.org/c/openstack/cliff/+/824305 | |
| 08:36:44 | opendevreview | mbu proposed openstack/cliff stable/train: Add conflict_handler parameter as attribut in Command class https://review.opendev.org/c/openstack/cliff/+/824306 | |
| 08:37:42 | opendevreview | mbu proposed openstack/cliff stable/train: Add conflict_handler parameter as attribut in Command class https://review.opendev.org/c/openstack/cliff/+/824306 | |
| 10:58:26 | opendevreview | Merged openstack/ansible-collections-openstack master: Add compute services list module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/823868 | |
| 11:13:53 | opendevreview | Merged openstack/ansible-collections-openstack master: Add Neutron RBAC modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/799896 | |
| 13:24:09 | opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | |
| #openstack-sdks - 2022-01-13 | |||
| 02:11:59 | headphoneJames | I did another pass at adding system role to sdk. I integrated feedback from the initial patch and built it on the feature/r1 branch. | |
| 02:12:10 | headphoneJames | https://review.opendev.org/c/openstack/openstacksdk/+/824470/ | |
| 12:46:29 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release 1.6.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/824559 | |
| 13:42:48 | opendevreview | Merged openstack/ansible-collections-openstack master: Release 1.6.0 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/824559 | |
| 17:13:12 | ralonsoh | stephenfin, hi! do you have a couple of mins for https://review.opendev.org/c/openstack/openstacksdk/+/804979? | |
| 17:13:19 | ralonsoh | thanks in advance! | |
| 17:22:19 | outbrito | Howdy folks! | |
| 17:22:53 | outbrito | gtema, I just got some time to check on your comment on this patch: https://review.opendev.org/c/openstack/openstacksdk/+/777801/3#message-1d4c80ba284da4feaf2da6497fa8fa5db833024f | |
| 17:23:02 | outbrito | Do you know what is the plan to merge those changes on feature/r1 on master? | |
| 19:14:21 | gtema | R1 to master merge is in progress. I need to analyse test failures which are most likely due to merge issues | |
| 19:55:10 | frickler | gtema: is that for https://review.opendev.org/c/openstack/openstacksdk/+/823961 ? I saw that, but was wondering if that was just an intermediate test. will you postpone the cache rework, then? | |
| 19:55:35 | frickler | I can help looking at the failures if that's the only thing left | |
| 20:13:28 | opendevreview | Merged openstack/openstacksdk master: Add "security_group_ids" to Port's query parameters https://review.opendev.org/c/openstack/openstacksdk/+/804979 | |
| 20:13:31 | opendevreview | Merged openstack/python-openstackclient master: Add --security-group to port list https://review.opendev.org/c/openstack/python-openstackclient/+/815506 | |
| 20:14:33 | outbrito | gtema, nice, so I'll abandon that change and rebase the other ones | |
| 21:19:12 | frickler | gtema: stephenfin: finding 1: https://review.opendev.org/c/openstack/openstacksdk/+/813233 is broken, allow_get is still being used in osc_lib. it is strange that the osc test only fails for flavors, because all resources seem to have "name" appearing twice in keys() after that | |
| 21:22:11 | frickler | reverting that and adding original_name to hidden_columns for flavors fixes the osc unit tests for me | |
| 21:22:36 | frickler | tomorrow I'll take a look at the tenant_id things | |
| 21:24:00 | frickler | this is the reference in osc-lib fwiw https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/utils/__init__.py#L726 | |
| #openstack-sdks - 2022-01-14 | |||
| 05:28:24 | gtema | Thks frickler | |
| 05:30:43 | gtema | I also experienced merge issues (lost parts). Especially on such huge merge this can also easily happen. Would also try to check today | |
| 06:13:36 | 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:55:34 | gtema | Yeah, thanks | |
| 14:56:05 | 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 | |
| 16:39:18 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Skip original_name from flavor info https://review.opendev.org/c/openstack/python-openstackclient/+/824733 | |
| 16:41:25 | 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 | |
| 17:00:06 | 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:01:14 | 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:50 | gtema | oh frickler, I was also on that right now :) | |
| 17:02:05 | gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/824733 is what you also changed | |
| 17:02:26 | gtema | and for network it will be a little bigger patch - need to clean more network resources - I am on it now | |
| 17:12:06 | 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:28 | gtema | sure | |
| 17:13:05 | 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:38 | gtema | additionally flavor failures are voting, network non voting (may help merging pain) | |
| 17:14:38 | 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:47 | gtema | cool | |
| 17:53:42 | 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:56:05 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | |
| 18:39:20 | 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 | |
| 19:15:27 | 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 | |
| 21:28:39 | gtema | Cool, thks | |
| #openstack-sdks - 2022-01-15 | |||
| 14:23:11 | opendevreview | Merged openstack/openstacksdk master: Revert "Add "security_group_ids" to Port's query parameters" https://review.opendev.org/c/openstack/openstacksdk/+/824751 | |
| 15:00:14 | opendevreview | Merged openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| #openstack-sdks - 2022-01-17 | |||
| 08:16:03 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Ensure flavor attributes are unique https://review.opendev.org/c/openstack/python-openstackclient/+/824865 | |
| 08:17:15 | thomasb06 | gtema: Hello, my script is ready. You only need one commit per service? | |
| 08:17:26 | gtema | yes, please | |
| 08:17:35 | thomasb06 | let's g | |
| 08:17:38 | thomasb06 | *go | |