Friday, 2017-08-18

openstackgerritSean McGinnis proposed openstack/glance master: Separate out functional tests  https://review.openstack.org/47481600:05
*** masuberu has joined #openstack-glance00:17
*** masber has quit IRC00:20
*** mtanino has quit IRC01:03
*** zhurong has joined #openstack-glance01:11
*** lucasxu has joined #openstack-glance01:20
*** lucasxu has quit IRC01:22
*** zzzeek has quit IRC01:23
*** zzzeek has joined #openstack-glance01:24
*** r-daneel has quit IRC01:24
*** tonyb has quit IRC01:24
*** zzzeek has quit IRC01:25
*** tonyb has joined #openstack-glance01:25
*** zzzeek has joined #openstack-glance01:25
*** rydou has joined #openstack-glance01:27
*** rydou has quit IRC01:28
*** rydou has joined #openstack-glance01:29
*** rydou has quit IRC01:31
*** rydou has joined #openstack-glance01:32
*** masber has joined #openstack-glance01:34
*** masber has quit IRC01:34
*** masuberu has quit IRC01:34
*** markvoelker has joined #openstack-glance01:41
*** markvoelker has quit IRC02:15
*** amrith has joined #openstack-glance02:54
amrithhola rosmaita02:54
rosmaitahola02:54
amrithdidn't realize that my new bouncer had booted glance channel. just added it back02:55
amrithjust figured I'd say hello. nothing fun and interesting, tty tomorrow02:57
rosmaitasounds good! ttyl02:57
*** markvoelker has joined #openstack-glance03:12
*** mine0901 has joined #openstack-glance03:13
rydouhi rosmaita , pls review https://review.openstack.org/#/c/494349/ again, updated the commit msg, thx03:25
*** nicolasbock_ has joined #openstack-glance03:25
rosmaitarydou: thanks for the quick work on that03:26
rosmaitarydou: next time you cherry pick, use -x and git will put the cherry-pick info in the commit message for you03:27
rydourosmaita : OK, and can you help to review another bug-fix for glance-client https://review.openstack.org/#/c/491656/03:28
rosmaitarydou: i'll put that on my list for tomorrow (i am in new york time zone)03:30
rydourosmaita : OK thank you03:31
*** udesale has joined #openstack-glance03:42
*** markvoelker has quit IRC03:46
*** masber has joined #openstack-glance03:49
*** masber has quit IRC03:53
*** mkrcmari__ has joined #openstack-glance04:01
*** hoonetorg has joined #openstack-glance04:18
*** links has joined #openstack-glance04:23
openstackgerritOpenStack Proposal Bot proposed openstack/glance_store master: Updated from global requirements  https://review.openstack.org/49480904:40
*** markvoelker has joined #openstack-glance04:43
*** gyee has quit IRC04:45
openstackgerritBrian Rosmaita proposed openstack/glance master: Add 'tasks_api_access' policy  https://review.openstack.org/49473204:47
*** abhishekk has joined #openstack-glance04:50
*** abhishekk is now known as abhishekk|busy04:50
openstackgerritOpenStack Proposal Bot proposed openstack/python-glanceclient master: Updated from global requirements  https://review.openstack.org/49489404:51
*** adisky__ has joined #openstack-glance04:55
*** masber has joined #openstack-glance05:09
*** markvoelker has quit IRC05:16
openstackgerritBrian Rosmaita proposed openstack/glance master: Return 404 for import-info call  https://review.openstack.org/49454405:17
*** rosmaita has quit IRC05:19
*** ratailor has joined #openstack-glance05:25
*** zhurong has quit IRC05:38
*** zhurong has joined #openstack-glance05:41
*** abhishekk|busy is now known as abhishekk05:59
*** groen692 has joined #openstack-glance05:59
*** markvoelker has joined #openstack-glance06:13
*** pdeore has joined #openstack-glance06:33
*** rcernin has joined #openstack-glance06:41
*** markvoelker has quit IRC06:46
*** tesseract has joined #openstack-glance07:13
*** namnh has joined #openstack-glance07:18
*** namnh has quit IRC07:18
*** namnh has joined #openstack-glance07:19
*** namnh has quit IRC07:20
*** abhishekk is now known as abhishekk|lunch07:31
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: image-create-via-import fails with ValueError  https://review.openstack.org/49498207:31
*** pcaruana has quit IRC07:32
*** markvoelker has joined #openstack-glance07:43
*** e0ne has joined #openstack-glance07:46
*** e0ne has quit IRC07:53
*** pcaruana has joined #openstack-glance07:54
*** e0ne has joined #openstack-glance07:55
*** e0ne has quit IRC08:05
*** e0ne has joined #openstack-glance08:06
*** nicolasbock_ has quit IRC08:14
*** markvoelker has quit IRC08:16
*** abhishekk|lunch is now known as abhishekk08:23
*** masber has quit IRC08:26
*** mine0901 has quit IRC08:33
*** markvoelker has joined #openstack-glance09:14
*** rydou has quit IRC09:31
*** dalgaaf has quit IRC09:34
*** dalgaaf has joined #openstack-glance09:37
*** markvoelker has quit IRC09:47
*** pcaruana has quit IRC10:02
*** udesale has quit IRC10:04
*** pcaruana has joined #openstack-glance10:22
*** markvoelker has joined #openstack-glance10:44
*** zhurong has quit IRC11:01
*** markvoelker has quit IRC11:17
*** ratailor has quit IRC11:29
*** adisky__ has quit IRC11:31
*** mvk has joined #openstack-glance11:32
*** mkrcmari__ has quit IRC11:35
*** pdeore has quit IRC11:44
*** rosmaita has joined #openstack-glance11:49
*** gcb has quit IRC12:10
*** abhishekk has quit IRC12:11
*** rcernin has quit IRC12:21
*** markvoelker has joined #openstack-glance12:25
*** rcernin has joined #openstack-glance12:38
*** Dinesh_Bhor has quit IRC12:39
*** catintheroof has joined #openstack-glance12:40
*** catinthe_ has joined #openstack-glance12:41
*** catintheroof has quit IRC12:41
*** jaosorior has quit IRC13:05
*** lucasxu has joined #openstack-glance13:08
*** ejat has quit IRC13:16
*** ejat has joined #openstack-glance13:20
openstackgerritSean McGinnis proposed openstack/glance master: Separate out functional tests  https://review.openstack.org/47481613:26
*** mosulica has joined #openstack-glance13:33
openstackgerritSean McGinnis proposed openstack/glance master: Separate out functional tests  https://review.openstack.org/47481613:34
*** mtanino has joined #openstack-glance13:58
*** efried is now known as fried_rice14:06
*** lucasxu has quit IRC14:07
*** baojg has joined #openstack-glance14:12
openstackgerritMerged openstack/glance master: Fix 500 error from image-stage call  https://review.openstack.org/49403614:22
openstackgerritMerged openstack/glance master: Add 'api_image_import' type to task(s) schemas  https://review.openstack.org/49406314:22
*** MVenesio has joined #openstack-glance14:27
*** MVenesio has quit IRC14:28
*** MVenesio has joined #openstack-glance14:28
rosmaitamtreinish: can you answer kairat's question on https://review.openstack.org/#/c/494595/ ?14:53
*** dpawlik is now known as danpawlik_15:03
*** danpawlik_ is now known as dpawlik_15:03
*** mosulica has quit IRC15:07
*** links has quit IRC15:10
openstackgerritBrian Rosmaita proposed openstack/python-glanceclient master: Fix image-import call  https://review.openstack.org/49435215:12
*** udesale has joined #openstack-glance15:21
*** MVenesio has quit IRC15:25
*** gyee has joined #openstack-glance15:28
openstackgerritOpenStack Proposal Bot proposed openstack/glance master: Updated from global requirements  https://review.openstack.org/49480815:46
*** sapd has quit IRC15:55
*** aarefiev is now known as aarefiev_afk15:55
*** e0ne has quit IRC16:09
*** pcaruana has quit IRC16:12
*** tesseract has quit IRC16:12
*** rcernin has quit IRC16:12
*** abhishekk_ has joined #openstack-glance16:15
*** markvoelker has quit IRC16:16
*** groen692 has quit IRC16:17
openstackgerritOctave Orgeron proposed openstack/glance master: Alembic should use oslo_db facades  https://review.openstack.org/47766516:28
*** mvk_ has joined #openstack-glance16:36
*** mvk_ has quit IRC16:38
*** mvk has quit IRC16:39
*** mvk has joined #openstack-glance16:40
openstackgerritSean McGinnis proposed openstack/glance master: Separate out functional tests  https://review.openstack.org/47481616:41
*** mvk_ has joined #openstack-glance16:43
*** catinthe_ has quit IRC16:43
abhishekk_rosmaita: hi, please let me know if any thing is required16:45
*** catintheroof has joined #openstack-glance16:45
*** mvk has quit IRC16:46
rosmaitaabhishekk_ i do have something, looking ...16:47
rosmaitaabhishekk_ could use you restoring your +2 on https://review.openstack.org/#/c/494352/ when it passes tests16:48
abhishekk_Ok16:48
rosmaita(gate is getting kind of slow)16:48
abhishekk_I will keep look on that16:48
rosmaitathanks, saw your latest glanceclient patch, i +2d it16:49
abhishekk_Thank you, just saw that16:49
rosmaitanp, thanks for finding that16:49
abhishekk_As you mentioned Ihave applied all required glance and client patches for testing it16:50
openstackgerritMerged openstack/glance master: api-ref: update container_format, disk_format  https://review.openstack.org/49388016:50
openstackgerritMerged openstack/glance master: Add 'tasks_api_access' policy  https://review.openstack.org/49473216:50
rosmaitai guess the other thing to keep an eye on is the cherry picks16:50
rosmaitato stable/pike16:50
rosmaitai;m adding them to the etherpad16:50
abhishekk_Will look at those too16:51
rosmaitathanks ... what time is it where you are?16:52
abhishekk_10.30 pm now16:52
rosmaitagetting late ...16:53
abhishekk_No issue16:54
rosmaitawell, i don't want you to burn out!16:54
rosmaitai think i can merge all the cherry picks myself16:54
abhishekk_Tomorrow is a rest day, so I can stretch little extra16:55
rosmaitai appreciate your dedication!16:55
abhishekk_Its my duty :)16:55
*** abhishekk_ has quit IRC16:59
*** abhishekk has joined #openstack-glance17:00
*** lucasxu has joined #openstack-glance17:06
rosmaitaabhishekk i think it's just https://review.openstack.org/494352 ... the two remaining non-doc glance patches are ones where flaper87 had questions17:12
rosmaitaif he accepts my responses, he can +2 those17:12
rosmaitai think it's ok if the glanceclient release is a bit later than glance17:12
abhishekkgreat, hopefully he will +2 those patches17:13
rosmaitathe only broken stuff we know about is all around image import17:13
abhishekkyes17:13
abhishekkgate is very slow, so IMO we need to wait on glanceclient release17:14
rosmaitai agree17:14
rosmaitait will probably speed up tomorrow!17:14
abhishekkI am pretty sure that 494352 is just commit message change and it will pass the tests17:15
abhishekkIf required I will be available tomorrow17:15
rosmaitathanks ... hopefully not.  i'll shoot you an email if there's something that could use your attention17:16
rosmaitawe can always release monday ... today was really just a suggestion17:17
abhishekkDo we need to backport glanceclient patches to stable/pike?17:17
rosmaitagood question17:17
rosmaitai think maybe we do17:18
abhishekkok, if we decided to delay client release then I can backport them tomorrow morning my time17:19
rosmaitathat would be helpful ... not sure those will all be merged by then, though17:20
abhishekkI will backport them once merged (anytime tomorrow or day after tomorrow)17:20
rosmaitaok, cool, that will be great17:22
rosmaitasmcginnis: when you have a minute, i've got a question about the glanceclient and stable/pike17:22
abhishekkis flavio around, I guess he might be away now17:24
rosmaitanot sure what time zone he is in these days17:25
rosmaitai need to grab some lunch and walk the dog, will be away for half hour or so17:33
abhishekkok17:33
*** lucasxu has quit IRC17:48
*** mvk_ has quit IRC18:10
*** lucasxu has joined #openstack-glance18:20
*** catinthe_ has joined #openstack-glance18:33
*** catinth__ has joined #openstack-glance18:33
*** udesale has quit IRC18:34
*** catintheroof has quit IRC18:35
*** catinthe_ has quit IRC18:37
*** markvoelker has joined #openstack-glance19:05
*** lucasxu has quit IRC19:06
*** e0ne has joined #openstack-glance19:17
smcginnisrosmaita: I'm back now. What's up?19:25
rosmaitagot some release questions19:26
rosmaita(and now i'm trying to remember what they are)19:26
smcginnisHah! :)19:27
rosmaitai guess it's mostly about the glanceclient19:27
smcginnisI see some mention in the scrollback. You need a new client release?19:27
rosmaitayeah, partially bugfixes and partly compatibility changes to go with the v2.6 experimental api19:28
rosmaitaso, we have a stable/pike branch of glanceclient19:28
rosmaitado we do the same thing that we're doing with glance: make fixes in master, and then cherry-pick to stable/pike?19:28
smcginnisrosmaita: Yeah. So here's the maybe tricky part...19:28
smcginnisrosmaita: Since it's past client lib freeze and everything is branched, the only real constraint is you can no longer raise the minimum required version.19:29
smcginnisrosmaita: So if there's some functionality that must have the new client release, there's a risk that a deployment may have an older version because we can't raise that.19:29
smcginnisrosmaita: Other than that, if it's just making something available, as soon as the freeze is over (next week I think) you can backport to stable/pike and request a new release.19:30
smcginnisrosmaita: And the upper constraint in the requirements can be raised, that part is not a probelm.19:30
smcginnisrosmaita: It works just like any stable release at that point.19:31
rosmaitaok, another question ... we tagged glanceclient at 2.8.0 and that's where stable/pike was cut19:32
rosmaitawould the next release be 2.8.1?19:32
smcginnisrosmaita: Are there only bug fixes, or are there new features included?19:33
rosmaitajust bugfixes (pretty sure)19:33
smcginnisrosmaita: OK, in that case it makes sense to be a .1 release.19:34
*** greghayn1 has joined #openstack-glance19:34
*** greghaynes has quit IRC19:34
smcginnisrosmaita: You have the option to do a 2.9 since a new release hasn't been done off of master with that.19:34
*** greghayn1 has left #openstack-glance19:34
smcginnisrosmaita: But then after GA of Pike, it should only be .1, .2, etc.19:34
rosmaita(sorry to be dense, but i find this stuff confusing)19:35
rosmaitaso, we'll be putting bugfixes in master19:35
rosmaitathen do we tag master with 2.8.1?19:36
smcginnisrosmaita: No worries, it's definitely confusing until you've done it a few times. :)19:36
rosmaitaor do we cherry pick the bugfixes into stable/pike and make that 2.8.1 ?19:36
smcginnisrosmaita: So any patches should get merged to master first before backporting.19:36
smcginnisrosmaita: But yeah, they do need to be cherry-picked to stable/pike.19:37
smcginnisrosmaita: And the pike update would then be off of that stable/pike commit hash.19:37
smcginnisrosmaita: You can just do a new release off of master, but the limitation there is the stable/pike upper constraints would not be able to be raised to that. I think. I'm not even 100% on that point though.19:38
smcginnisrosmaita: But if it's needed for pike, probably better to do off of pike.19:38
rosmaitaok, makes sense19:39
rosmaitai keep forgetting about the upper constraints19:39
smcginnisrosmaita: Are these client side bugfixes, or client lib code used by the glance service?19:39
smcginnisIn other words, do we even need to raise upper constraints?19:39
rosmaitai think they're all client side19:40
smcginnisrosmaita: OK, then u-c might not be a concern. But I think there are some distro packages that would not include the bugfix release if it's not from the stable/pike code.19:41
smcginnisrosmaita: So I think you've got it. Merge fixes to master, backport to stable/pike, then request a release off of the stable branch hash and we should be good.19:41
rosmaitaok, so it sounds like we definitely want to update stable/pike code, even if we don't mess with u-c19:42
rosmaitagreat, thanks!19:42
smcginnisrosmaita: Yeah, I think so.19:42
smcginnisrosmaita: No problem, glad to help. It is a little confusing since there are so many moving parts.19:42
rosmaitano kidding!19:42
rosmaitaone last thing ... probably add a release note about the bugs fixed ?19:43
smcginnisrosmaita: If you think they are significant enough for end users to need to know about in reading through the release notes.19:43
smcginnisrosmaita: No requirement, but can be useful depending on the bug.19:43
rosmaitaok19:45
smcginnisrosmaita: Low priority, so I hesitate to even bring it up at this point and cause a distraction, but breaking out the functional tests separate from the unit tests now work.19:47
smcginnisrosmaita: https://review.openstack.org/47481619:47
rosmaitai noticed you were working on that19:47
rosmaitaglad it's working, thanks19:47
smcginnisrosmaita: Something to think about maybe in the near future. Makes it a little faster running ut validation locally.19:47
rosmaitai think it's a good idea, we can make the change after we've got all the import stuff cherry picked out of master19:48
smcginnisrosmaita: Cool, sounds good.19:48
rosmaitai appreciate your work, you've been a big part of the glance pike release19:49
smcginnisrosmaita: Thanks, I appreciate that. Just trying to help where I can.19:50
*** e0ne has quit IRC19:51
*** jdillaman has quit IRC19:51
*** abhishekk has quit IRC20:07
*** eastmde has joined #openstack-glance20:10
*** eastmde has quit IRC20:23
rosmaitasmcginnis: one more question if you're still around21:07
smcginnisrosmaita: Yep, still here21:08
rosmaitacool21:08
rosmaitaso, if i include some release notes for glance RC-2, do those go directly into stable/pike ?21:08
smcginnisrosmaita: I believe reno handles it right if you merge them in master and cherry pick them stable/pike.21:09
rosmaitaok, great21:09
rosmaitathanks!21:10
smcginnisAny time21:10
*** catintheroof has joined #openstack-glance21:22
*** catinth__ has quit IRC21:25
openstackgerritBrian Rosmaita proposed openstack/glance master: Add release note for Glance Pike RC-2  https://review.openstack.org/49543721:29
*** mtanino has quit IRC21:54
*** catintheroof has quit IRC21:57
*** rosmaita has quit IRC22:01
*** rosmaita has joined #openstack-glance22:07
openstackgerritBrian Rosmaita proposed openstack/glance master: Add release note for Glance Pike RC-2  https://review.openstack.org/49543722:10

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!