Friday, 2022-08-26

opendevreviewAbhishek Kekane proposed openstack/glance master: Add documentation for glance-download plugin  https://review.opendev.org/c/openstack/glance/+/85392805:53
opendevreviewAbhishek Kekane proposed openstack/glance master: [glance-download] Add missing config options to config list  https://review.opendev.org/c/openstack/glance/+/85450105:53
opendevreviewAbhishek Kekane proposed openstack/glance master: Refresh Glance example configs for zed milestone 3  https://review.opendev.org/c/openstack/glance/+/85449505:53
opendevreviewAbhishek Kekane proposed openstack/glance master: DNM: Test glance-download with tempest  https://review.opendev.org/c/openstack/glance/+/85260905:54
opendevreviewMerged openstack/glance master: Fix a deprecation warning about threading.Thread  https://review.opendev.org/c/openstack/glance/+/85387409:39
opendevreviewErno Kuvaja proposed openstack/glance master: Remove 'glance-download' from default import methods  https://review.opendev.org/c/openstack/glance/+/85477211:30
opendevreviewErno Kuvaja proposed openstack/python-glanceclient master: Bump default pagesize  https://review.opendev.org/c/openstack/python-glanceclient/+/85477311:50
rosmaitajokke_: left a comment for you on https://review.opendev.org/c/openstack/glance/+/85477212:37
rosmaitagood catch, btw12:37
jokke_rosmaita: good points, thanks! Will respin it in a minute12:43
rosmaitaty12:43
opendevreviewErno Kuvaja proposed openstack/glance master: Remove 'glance-download' from default import methods  https://review.opendev.org/c/openstack/glance/+/85477212:46
jokke_rosmaita: added little comment on the top of that test as well to clarify it's testing just te defaults12:46
jokke_darn review priority is not sticky anymore ... need to figure that out12:47
rosmaitayeah, i'm not sure it ever was tbh12:49
jokke_oh12:50
jokke_kk12:50
jokke_thanks for quick review ;)12:50
rosmaitajokke_: https://review.opendev.org/c/openstack/project-config/+/85478113:51
opendevreviewPranali Deore proposed openstack/python-glanceclient master: Add support for glance-download import method  https://review.opendev.org/c/openstack/python-glanceclient/+/85478413:53
*** pdeore is now known as pdeore|afk13:54
*** amorin_ is now known as amorin14:00
jokke_rosmaita: the tests are failing with """Exception: Unrecognized key in line: "copyCondition = changekind:REWORK""""14:28
jokke_So that has something gerrit does not like14:29
rosmaitajokke_: could be 'change-kind', the gerrit docs were ambiguous14:31
rosmaitahmmm ... doesn't like that either14:53
jokke_abhishekk: \o15:55
abhishekko/15:55
abhishekkjokke_, o/15:56
opendevreviewErno Kuvaja proposed openstack/glance master: Remove 'glance-download' from default import methods  https://review.opendev.org/c/openstack/glance/+/85477215:57
jokke_There was actual pep8 failure ... my vim plugins messes the indents in some cases and I tend to forget to double check those in these very quick respins15:58
abhishekkack15:58
abhishekki am operating from mobile, so overlooked it as other failure15:59
abhishekkgood catch 16:00
opendevreviewMerged openstack/glance master: Remove 'glance-download' from default import methods  https://review.opendev.org/c/openstack/glance/+/85477219:43

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!