Earlier  
Posted Nick Remark
#openstack-sdks - 2021-05-03
14:54:26 dtantsur yep, thanks
14:55:12 gtema_ wlcm
14:55:12 gtema_ wlcm
15:13:46 openstackgerrit Merged openstack/openstacksdk stable/train: bifrost-integration-tinyipa-ubuntu-xenial: make non-voting https://review.opendev.org/c/openstack/openstacksdk/+/788883
15:13:46 openstackgerrit Merged openstack/openstacksdk stable/train: bifrost-integration-tinyipa-ubuntu-xenial: make non-voting https://review.opendev.org/c/openstack/openstacksdk/+/788883
17:28:16 openstackgerrit Merged openstack/openstacksdk master: Increase RAM for the Ironic CI jobs https://review.opendev.org/c/openstack/openstacksdk/+/789283
17:28:16 openstackgerrit Merged openstack/openstacksdk master: Increase RAM for the Ironic CI jobs https://review.opendev.org/c/openstack/openstacksdk/+/789283
17:28:24 openstackgerrit Merged openstack/openstacksdk master: Implement driver vendor passthrough https://review.opendev.org/c/openstack/openstacksdk/+/785753
17:28:24 openstackgerrit Merged openstack/openstacksdk master: Implement driver vendor passthrough https://review.opendev.org/c/openstack/openstacksdk/+/785753
20:08:03 openstackgerrit Dirk Mueller proposed openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/789410
20:08:03 openstackgerrit Dirk Mueller proposed openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/789410
#openstack-sdks - 2021-05-04
07:21:47 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk stable/wallaby: Increase RAM for the Ironic CI jobs https://review.opendev.org/c/openstack/openstacksdk/+/789439
07:21:47 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk stable/wallaby: Increase RAM for the Ironic CI jobs https://review.opendev.org/c/openstack/openstacksdk/+/789439
09:20:56 openstackgerrit Merged openstack/openstacksdk stable/train: Switch nodepool test to containers https://review.opendev.org/c/openstack/openstacksdk/+/788416
09:20:56 openstackgerrit Merged openstack/openstacksdk stable/train: Switch nodepool test to containers https://review.opendev.org/c/openstack/openstacksdk/+/788416
09:23:14 openstackgerrit Merged openstack/openstacksdk master: statsd: use timedelta and pipeline https://review.opendev.org/c/openstack/openstacksdk/+/786991
09:23:14 openstackgerrit Merged openstack/openstacksdk master: statsd: use timedelta and pipeline https://review.opendev.org/c/openstack/openstacksdk/+/786991
09:25:50 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix the idempotent of compute_flavor module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/788236
09:25:51 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix the idempotent of compute_flavor module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/788236
12:22:23 sum12 Can I not specify stream=true for image download from the cli ?
12:22:23 sum12 Can I not specify stream=true for image download from the cli ?
15:12:06 openstackgerrit Dmitriy Rabotyagov proposed openstack/openstacksdk master: Adjust image _base_proxy condition https://review.opendev.org/c/openstack/openstacksdk/+/788739
15:12:06 openstackgerrit Dmitriy Rabotyagov proposed openstack/openstacksdk master: Adjust image _base_proxy condition https://review.opendev.org/c/openstack/openstacksdk/+/788739
15:28:59 mordred sum12: I think the cli should just DTRT as far as downloads go - I wouldn't expect it to ever not do stream=true
15:28:59 mordred sum12: I think the cli should just DTRT as far as downloads go - I wouldn't expect it to ever not do stream=true
17:02:57 sum12 * sum12 should pip install rather than dnf install
23:10:01 openstackgerrit Ian Wienand proposed openstack/openstacksdk stable/stein: Switch nodepool test to containers https://review.opendev.org/c/openstack/openstacksdk/+/788414
23:10:01 openstackgerrit Ian Wienand proposed openstack/openstacksdk stable/stein: Switch nodepool test to containers https://review.opendev.org/c/openstack/openstacksdk/+/788414
23:37:41 openstackgerrit Cyril Roelandt proposed openstack/python-openstackclient master: openstack image create: honor protection/visibility flags https://review.opendev.org/c/openstack/python-openstackclient/+/789649
23:37:41 openstackgerrit Cyril Roelandt proposed openstack/python-openstackclient master: openstack image create: honor protection/visibility flags https://review.opendev.org/c/openstack/python-openstackclient/+/789649
#openstack-sdks - 2021-05-05
02:45:35 openstackgerrit 江治林 proposed openstack/keystoneauth master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/keystoneauth/+/789699
02:46:28 openstackgerrit 江治林 proposed openstack/keystoneauth master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/keystoneauth/+/789699
02:46:28 openstackgerrit 江治林 proposed openstack/keystoneauth master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/keystoneauth/+/789699
06:58:31 openstackgerrit YuehuiLei proposed openstack/python-openstackclient master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/python-openstackclient/+/789710
06:58:31 openstackgerrit YuehuiLei proposed openstack/python-openstackclient master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/python-openstackclient/+/789710
09:41:58 frickler this looks like a regression in glance to me, not like an issue with osc https://zuul.opendev.org/t/openstack/build/5811986c8be64dab843a1871773a030d
09:41:58 frickler this looks like a regression in glance to me, not like an issue with osc https://zuul.opendev.org/t/openstack/build/5811986c8be64dab843a1871773a030d
11:01:09 openstackgerrit Merged openstack/openstacksdk stable/wallaby: Increase RAM for the Ironic CI jobs https://review.opendev.org/c/openstack/openstacksdk/+/789439
11:01:09 openstackgerrit Merged openstack/openstacksdk stable/wallaby: Increase RAM for the Ironic CI jobs https://review.opendev.org/c/openstack/openstacksdk/+/789439
11:34:59 openstackgerrit Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch project and project_info module to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/789793
11:49:42 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP add TripleO job on collections used in TripleO https://review.opendev.org/c/openstack/ansible-collections-openstack/+/789818
11:49:42 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP add TripleO job on collections used in TripleO https://review.opendev.org/c/openstack/ansible-collections-openstack/+/789818
12:54:12 frickler gtema: noonedeadpunk: actually https://review.opendev.org/c/openstack/openstacksdk/+/783849 is causing the above regression, seems it was a mistake to ignore the failing osc job, sadly logs have already expired
12:54:12 frickler gtema: noonedeadpunk: actually https://review.opendev.org/c/openstack/openstacksdk/+/783849 is causing the above regression, seems it was a mistake to ignore the failing osc job, sadly logs have already expired
12:54:37 frickler also not sure if you can find a quick fix or want to revert first
12:54:37 frickler also not sure if you can find a quick fix or want to revert first
12:57:06 gtema not that I understand why it is causing this
12:57:06 gtema not that I understand why it is causing this
12:57:18 noonedeadpunk this one https://review.opendev.org/c/openstack/openstacksdk/+/783849/2/openstack/image/v2/image.py
12:57:18 noonedeadpunk this one https://review.opendev.org/c/openstack/openstacksdk/+/783849/2/openstack/image/v2/image.py
12:57:55 noonedeadpunk so yeah, might be worth at least partially reverting it...
12:57:55 noonedeadpunk so yeah, might be worth at least partially reverting it...
12:58:38 noonedeadpunk or at least really quick fix
12:58:38 noonedeadpunk or at least really quick fix
12:59:26 gtema someone show me logs where it fails?
12:59:26 gtema someone show me logs where it fails?
12:59:36 gtema I was not really looking into the discussion recently
12:59:36 gtema I was not really looking into the discussion recently
12:59:57 noonedeadpunk It tries to ` tempest.lib.exceptions.CommandFailed: Command 'openstack --os-cloud=devstack-admin image list --tag my_tag --long -f json' returned non-zero exit status 1.`
12:59:57 noonedeadpunk It tries to ` tempest.lib.exceptions.CommandFailed: Command 'openstack --os-cloud=devstack-admin image list --tag my_tag --long -f json' returned non-zero exit status 1.`
13:01:48 gtema I need to see API call
13:01:48 gtema I need to see API call
13:02:52 noonedeadpunk well with patch I renamed option I gues :(((
13:02:52 noonedeadpunk well with patch I renamed option I gues :(((
13:04:30 noonedeadpunk I jsut kind of assumed, that it can't be used since it's not really working properly
13:04:30 noonedeadpunk I jsut kind of assumed, that it can't be used since it's not really working properly
13:05:05 openstackgerrit Dmitriy Rabotyagov proposed openstack/openstacksdk master: Revert tags option back to tag https://review.opendev.org/c/openstack/openstacksdk/+/789827
13:05:37 noonedeadpunk not super sure about that though ^
13:05:37 noonedeadpunk not super sure about that though ^
13:06:26 noonedeadpunk I don't think it even comes to API call
13:06:26 noonedeadpunk I don't think it even comes to API call
13:06:47 gtema noho, the fix should be in https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L613
13:06:47 gtema noho, the fix should be in https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L613
13:06:48 noonedeadpunk it fails just on running openstack as unknown option is passed
13:06:48 noonedeadpunk it fails just on running openstack as unknown option is passed
13:07:08 gtema compare with https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L323
13:07:08 gtema compare with https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L323
13:07:42 gtema this should be also constructing array of tags
13:07:42 gtema this should be also constructing array of tags
13:07:44 frickler hmm, for list-image, the filter is called "tag", but for show-image, a "tags" field is returned https://docs.openstack.org/api-ref/image/v2/index.html?expanded=list-images-detail,show-image-detail#list-images
13:07:44 frickler hmm, for list-image, the filter is called "tag", but for show-image, a "tags" field is returned https://docs.openstack.org/api-ref/image/v2/index.html?expanded=list-images-detail,show-image-detail#list-images
13:08:15 noonedeadpunk oh, well, I think you're right gtema
13:08:15 noonedeadpunk oh, well, I think you're right gtema
13:08:45 noonedeadpunk frickler: well, you can use --tag several times, right?
13:08:46 noonedeadpunk frickler: well, you can use --tag several times, right?
13:08:51 gtema noonedeadpunk - will you prepare change or should I?
13:08:51 gtema noonedeadpunk - will you prepare change or should I?
13:09:10 gtema yes - you should be able to have same interface of repeating multiple --tag
13:09:10 gtema yes - you should be able to have same interface of repeating multiple --tag
13:09:23 noonedeadpunk doesn't matter for me:)
13:09:23 noonedeadpunk doesn't matter for me:)
13:09:38 noonedeadpunk probably to merge things faster (in terms of self vote) I can push it
13:09:38 noonedeadpunk probably to merge things faster (in terms of self vote) I can push it
13:09:40 gtema if you have time - please, I'm pretty overloaded right now
13:09:40 gtema if you have time - please, I'm pretty overloaded right now
13:27:24 noonedeadpunk sure, testing it :)
13:27:24 noonedeadpunk sure, testing it :)
13:27:35 gtema thks
13:40:16 noonedeadpunk gtema: nah, I was right with https://review.opendev.org/c/openstack/openstacksdk/+/789827
13:40:16 noonedeadpunk gtema: nah, I was right with https://review.opendev.org/c/openstack/openstacksdk/+/789827

Earlier   Later