| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-06-17 | |||
| 10:41:34 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Switch hostaggregate module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792435 | |
| 10:52:00 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Dns zone info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/795752 | |
| 10:52:07 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Dns zone info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/795752 | |
| 10:57:53 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Only apply necessary changes to subnets https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770121 | |
| 11:20:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Add check_mode attribute to OpenstackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/796261 | |
| 11:20:07 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch port_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792420 | |
| 11:20:12 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch federation_idp_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792440 | |
| 11:20:29 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch floating_ip module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792437 | |
| 11:20:35 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch federation_mapping_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792438 | |
| 11:59:39 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch federation_mapping module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792439 | |
| 12:05:34 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch KeystoneFederationProtocolInfo module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792426 | |
| 13:06:25 | opendevreview | Merged openstack/python-openstackclient master: volume: Allow more versions https://review.opendev.org/c/openstack/python-openstackclient/+/792950 | |
| 14:06:10 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch group_assignment module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792436 | |
| 14:11:17 | opendevreview | Jan Hartkopf proposed openstack/ansible-collections-openstack master: fix update on empty list of allowed address pairs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/796847 | |
| 15:06:37 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch catalog_service module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792447 | |
| 15:53:36 | diablo_rojo | Are we meeting today? | |
| 15:54:05 | gtema | dunno, calendar says yes. But as of now I do not feel someone has need for that | |
| 15:54:10 | gtema | I'm here anyway | |
| 15:54:13 | diablo_rojo | Me neither.. | |
| 15:54:20 | diablo_rojo | I would also love a nap lol | |
| 15:54:34 | diablo_rojo | Since I have been in back to back meetings for ~3 hours | |
| 15:54:46 | diablo_rojo | and after this one they go on for another ~3 hours.. | |
| 15:54:50 | gtema | oh poor diablo_rojo | |
| 15:55:01 | diablo_rojo | Yes. Woe is me. | |
| 15:55:06 | stephenfin | yeah, I'm good to skip also | |
| 15:55:12 | stephenfin | nothing urgent for m | |
| 15:55:13 | stephenfin | *me | |
| 15:55:22 | gtema | oki guys | |
| 15:55:26 | diablo_rojo | I also ran about 6.5k this morning. | |
| 15:55:36 | diablo_rojo | with sprints.. before my meetings. | |
| 15:55:41 | gtema | :) | |
| 15:55:44 | diablo_rojo | Feel bad for me yet? ;) | |
| 15:56:25 | gtema | bad? - no. It's all for good, isnt it? | |
| 15:57:34 | diablo_rojo | I suppose. | |
| 15:57:41 | diablo_rojo | The meetings part is debatable | |
| 15:58:02 | gtema | yeah, I meant more your sport part | |
| 15:58:59 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch lb_pool module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792423 | |
| 15:59:01 | diablo_rojo | Yes. That is self inflicted. | |
| 16:10:58 | gouthamr | kafilat: o/ | |
| 16:11:07 | kafilat | Hi | |
| 16:11:19 | gouthamr | hey gtema: kafilat and i are looking to get stuff into the r1 branch | |
| 16:11:32 | gtema | great | |
| 16:11:34 | gouthamr | gtema: but, the nodepool job's holding us up | |
| 16:11:35 | gouthamr | :) | |
| 16:11:40 | gouthamr | i see your patch to disable it: https://review.opendev.org/c/openstack/openstacksdk/+/796027 | |
| 16:11:56 | gouthamr | were we looking to get that merged? | |
| 16:12:08 | gtema | yah, wanted to double check whether nodepool was fixed in the meanwhile | |
| 16:12:17 | gouthamr | asking because folks usually ignore patches with a -1 on them | |
| 16:12:48 | gouthamr | awesome! i'll follow it through, btw, diablo_rojo and gtema: kafilat is our new outreachy intern | |
| 16:12:49 | gtema | stephenfin: have something against https://review.opendev.org/c/openstack/openstacksdk/+/796027? | |
| 16:13:19 | kafilat | Hello | |
| 16:13:21 | gouthamr | she's going through onboarding right now; but she's refreshing the openstacksdk patches we have open and addressing review comments from gtema and stephenfin | |
| 16:13:39 | stephenfin | gtema: nope, lgtm | |
| 16:13:47 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch lb_member module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792424 | |
| 16:13:51 | opendevreview | Merged openstack/ansible-collections-openstack master: Switch lb_listener module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/792425 | |
| 16:14:09 | gtema | cool, thks | |
| 16:14:17 | stephenfin | actually | |
| 16:14:23 | stephenfin | comment left | |
| 16:14:28 | stephenfin | should we make it non-voting instead | |
| 16:14:34 | stephenfin | depends on whether we plan to ever fix it | |
| 16:14:44 | stephenfin | if not, perhaps we should just remove it altogether? | |
| 16:14:54 | gtema | well, it's not we, but infra folks | |
| 16:15:03 | gtema | I know they are trying | |
| 16:15:16 | stephenfin | maybe make it non-voting for now so? | |
| 16:15:18 | gtema | my plan is to turn it back on before going to master | |
| 16:15:27 | stephenfin | oh | |
| 16:15:31 | gtema | makes no sense - it just wastes resources | |
| 16:15:39 | stephenfin | okay, fair | |
| 16:25:03 | gouthamr | ++ | |
| 16:50:29 | melwitt | stephenfin: just to confirm, there isn't any way to specify arbitrary fields to show from the server list response in osc right? like the --fields thing in novaclient | |
| 16:55:57 | stephenfin | melwitt: is the '-c'/'--column' option what you're looking for? | |
| 16:56:09 | stephenfin | that's not passed through to the server though | |
| 16:56:29 | stephenfin | so you can only use it to show/hide things the server returns in the default response | |
| 16:56:50 | stephenfin | (I know neutron let's you request different fields at a server level) | |
| 16:58:42 | melwitt | stephenfin: yeah I tried -c but it doesn't seem to let you specify like host_status, for example | |
| 16:58:42 | stephenfin | try 'Host Status' instead | |
| 16:58:42 | melwitt | it omits the column when I try that | |
| 16:58:42 | melwitt | or rather, it doesn't show up | |
| 16:59:02 | melwitt | $ openstack server list -c "Host Status" | |
| 16:59:02 | melwitt | No recognized column names in ['Host Status']. Recognized columns are ('ID', 'Name', 'Status', 'Networks', 'Image', 'Flavor'). | |
| 17:00:07 | melwitt | and with --long, | |
| 17:00:09 | melwitt | No recognized column names in ['Host Status']. Recognized columns are ('ID', 'Name', 'Status', 'Task State', 'Power State', 'Networks', 'Image Name', 'Image ID', 'Flavor Name', 'Flavor ID', 'Availability Zone', 'Host', 'Properties'). | |
| 17:00:33 | melwitt | just wanted to make sure I wasn't missing something if there's a way to pick out any field from the response | |
| 17:01:25 | stephenfin | okay, yeah, we need to do something like https://github.com/openstack/python-openstackclient/commit/311f4130d2c59638074531fa59e67783c2571e91 for that so | |
| 17:02:28 | melwitt | gotcha, thanks | |
| 17:03:02 | stephenfin | how we do the whole column handling in OSC/cliff is fairly flaky, unfortunately | |
| 17:05:08 | opendevreview | Jeremy Stanley proposed openstack/osc-lib master: Correct the tox option for skipping sdist generation https://review.opendev.org/c/openstack/osc-lib/+/796910 | |
| 17:08:29 | opendevreview | Jeremy Stanley proposed openstack/python-openstackclient master: Correct the tox option for skipping sdist generation https://review.opendev.org/c/openstack/python-openstackclient/+/796915 | |
| 17:10:00 | opendevreview | Merged openstack/ansible-collections-openstack master: Dns zone info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/795752 | |
| 17:10:15 | opendevreview | Jeremy Stanley proposed openstack/service-types-authority master: Correct the tox option for skipping sdist generation https://review.opendev.org/c/openstack/service-types-authority/+/796917 | |
| #openstack-sdks - 2021-06-18 | |||
| 11:16:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: setup.cfg: Drop underscores https://review.opendev.org/c/openstack/python-openstackclient/+/797058 | |
| 11:16:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add hyphenated commands https://review.opendev.org/c/openstack/python-openstackclient/+/797059 | |
| 16:52:03 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume attachment *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/792951 | |
| 16:52:09 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume message *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/781637 | |
| 16:56:09 | opendevreview | Merged openstack/python-openstackclient master: volume: Add missing 'volume backup *' options https://review.opendev.org/c/openstack/python-openstackclient/+/794640 | |
| 18:02:37 | opendevreview | Merged openstack/python-openstackclient master: tests: Rename 'FakeType' -> 'FakeVolumeType' https://review.opendev.org/c/openstack/python-openstackclient/+/794641 | |
| 18:05:41 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume group *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/794642 | |
| #openstack-sdks - 2021-06-21 | |||
| 13:09:40 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume group type *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/794643 | |
| 13:09:46 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume group snapshot *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/794644 | |
| 13:20:32 | opendevreview | Merged openstack/python-openstackclient master: volume: Add more missing 'volume backup *' options https://review.opendev.org/c/openstack/python-openstackclient/+/795828 | |