| opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | 01:11 |
|---|---|---|
| frickler | gtema: could you add https://review.opendev.org/c/openstack/python-openstackclient/+/864497 to your review list please? | 08:19 |
| gtema | ack | 08:19 |
| gtema | frickler - we were talking with you about naming: "force" is a bad idea from my pov | 08:20 |
| frickler | gtema: so what would you prefer instead? "--no-confirm"? | 08:24 |
| gtema | yes, maybe, or --assume-yes or --auto-approve (like in TF) | 08:26 |
| frickler | the other option would be to make the whole thing non-interactive and just have a "--dry-run" option for the timid | 08:27 |
| gtema | hmm, this is too dangerous in my eyes | 08:32 |
| frickler | o.k., so I'll amend to use --auto-approve | 08:34 |
| gtema | cool, thanks | 08:35 |
| opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add auto-approve option to project cleanup https://review.opendev.org/c/openstack/python-openstackclient/+/864497 | 08:43 |
| frickler | seems I totally missed 1.0.0, was there a good party? | 09:10 |
| gtema | frickler - a party was sadly in mind only | 10:27 |
| lajoskatona | gtema, frickler: just a note, sdk 1.0.0 is still not in use, as u-c bump is not merged: https://review.opendev.org/c/openstack/requirements/+/872282 | 10:56 |
| frickler | lajoskatona: yes, though we actually have 1.0.1 now. but requirements CI is broken as well as kuryr like you mentioned | 11:00 |
| frickler | you could try to depend your neutronclient patch on the u-c bump, though | 11:01 |
| lajoskatona | frickler: yeah, but anyway we can't merge (and release) it without having correct u-c.txt | 11:03 |
| opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Include "security_groups" to "Port" query parameters https://review.opendev.org/c/openstack/openstacksdk/+/873946 | 16:46 |
| opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Include "security_groups" to "Port" query parameters https://review.opendev.org/c/openstack/openstacksdk/+/873946 | 16:51 |
| opendevreview | Elod Illes proposed openstack/cliff master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/cliff/+/873979 | 19:27 |
| opendevreview | Elod Illes proposed openstack/osc-lib master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/osc-lib/+/873986 | 19:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!