opendevreview | Andrey Kurilin proposed openstack/rally master: Update python versions https://review.opendev.org/c/openstack/rally/+/917487 | 08:06 |
---|---|---|
opendevreview | Andrey Kurilin proposed openstack/rally master: Update python versions https://review.opendev.org/c/openstack/rally/+/917487 | 09:16 |
opendevreview | Andrey Kurilin proposed openstack/rally master: Update python versions https://review.opendev.org/c/openstack/rally/+/917487 | 09:23 |
opendevreview | Andrey Kurilin proposed openstack/rally master: Update python versions https://review.opendev.org/c/openstack/rally/+/917487 | 09:25 |
opendevreview | Andrey Kurilin proposed openstack/rally master: Update python versions https://review.opendev.org/c/openstack/rally/+/917487 | 09:29 |
opendevreview | Merged openstack/rally master: Update python versions https://review.opendev.org/c/openstack/rally/+/917487 | 13:01 |
frickler | andreykurilin: while youre here, do you have any idea whats going wrong in https://review.opendev.org/c/openstack/rally-openstack/+/935821 ? | 15:05 |
andreykurilin | frickler: it is simple. There are `type` and `name` options available in `api_version` context. Passing ‘block-storage’ as cinder service name is incorrect as there is no such service. | 15:14 |
andreykurilin | I’ll leave a comment to the change | 15:14 |
frickler | andreykurilin: iiuc this is in fact the new service name that devstack uses. or should it be type instead of name? | 15:19 |
andreykurilin | frickler: I see only ‘cinder’ service name now https://zuul.opendev.org/t/openstack/build/dbf0fe7dbbfc4eb490339afff8a6a048/log/job-output.txt#21411 | 15:21 |
andreykurilin | And it refers to default configuration specified for cinder client in Rally - https://github.com/openstack/rally-openstack/blob/master/rally_openstack/common/osclients.py#L508 | 15:23 |
andreykurilin | The only point of using api_versions context was to check ability to use non-default service entity from catalog. As there is a single entity that matches current default configuration- nothing to check and the test case can be dropped | 15:24 |
opendevreview | Jon Bernard proposed openstack/rally-openstack master: Remove explicit cinder v3 endpoint test https://review.opendev.org/c/openstack/rally-openstack/+/935821 | 15:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!