Wednesday, 2017-05-03

*** harlowja has quit IRC00:04
*** bobh has quit IRC00:09
*** bobh has joined #openstack-release00:13
*** dave-mccowan has quit IRC00:13
*** bobh has quit IRC00:21
*** bobh has joined #openstack-release00:24
*** bobh has quit IRC00:30
*** mriedem has quit IRC00:40
*** dave-mccowan has joined #openstack-release00:47
*** bobh has joined #openstack-release00:56
*** bobh has quit IRC00:58
*** yanyanhu has joined #openstack-release01:25
*** yolkfull__ has joined #openstack-release01:29
*** mriedem has joined #openstack-release01:36
*** zhurong has joined #openstack-release01:50
*** yamahata has joined #openstack-release01:58
*** zhurong has quit IRC02:07
*** flwang2 has joined #openstack-release02:20
*** flwang has quit IRC02:21
openstackgerritEmilien Macchi proposed openstack/releases master: Release openstacklient 3.10.0  https://review.openstack.org/46195902:33
EmilienMdtroyer: ^02:33
dtroyerEmilienM: thanks, but not quite yet, I'm still chasing a thing or two that really needs to get fixed02:34
EmilienMdtroyer: ok - can you confirm that this release will help with https://bugs.launchpad.net/tripleo/+bug/1687706 ?02:35
openstackLaunchpad bug 1687706 in tripleo "__init__() got an unexpected keyword argument 'url'" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)02:35
EmilienMthis bug is blocking us to promote the packages in tripleo CI :(02:35
dtroyeryeah, it should address that02:36
*** dave-mccowan has quit IRC02:36
dtroyerFWIW, I've been trying to do this release for over a week now, and NNED to get it done before going to Boston, so basically tomorrow02:36
EmilienMok02:36
EmilienMdtroyer: i'll pin the package in RDO to the latest commit in the meantime02:37
EmilienMd930b043eef262f3eb34489f01b9856077dc2dc402:37
dtroyerthe timing of the novaclient 8.0 stuff set me back 2 weeks and that's what I've been focused on02:37
EmilienMI hope it will work fine with latest tag of osc-lib02:37
dtroyerit should, we bumped g-r min to the current osc-lib and os-client-config, both of which the next osc need02:38
EmilienMok02:38
EmilienMthx for the infos02:38
dtroyerok, I just saw stevemar merged the last two func test fixes I had, so there is just one more thing I need to look at02:39
EmilienMdtroyer: feel free to take over my release patch, I won't abandon it02:39
dtroyerit'll need to be updated, I'll do that when its time02:39
*** yamahata has quit IRC02:40
EmilienMoki02:43
*** armax has quit IRC02:48
*** hongbin has quit IRC02:59
*** gcb has quit IRC02:59
*** armax has joined #openstack-release03:05
*** ricolin has joined #openstack-release03:12
*** gcb has joined #openstack-release03:12
*** bobh has joined #openstack-release03:13
*** bobh has quit IRC03:16
*** armax has quit IRC03:19
*** bobh has joined #openstack-release03:21
*** mriedem has quit IRC03:25
*** armax has joined #openstack-release03:35
*** zhurong has joined #openstack-release03:37
*** zhurong has quit IRC03:40
*** bobh has quit IRC03:41
stevemaryeah, EmilienM, dtroyer is super close to getting everything fixed03:45
EmilienMcool !03:47
*** armax has quit IRC04:27
*** armax has joined #openstack-release04:28
*** zhurong has joined #openstack-release04:37
*** armax has quit IRC04:38
*** yamahata has joined #openstack-release04:40
*** yamahata has quit IRC04:55
*** udesale has joined #openstack-release04:58
*** udesale has quit IRC05:00
*** udesale has joined #openstack-release05:01
*** gcb has quit IRC05:10
*** gcb has joined #openstack-release05:27
*** pcaruana has joined #openstack-release06:27
*** eranrom has joined #openstack-release06:56
*** shardy has joined #openstack-release07:00
*** bauwser is now known as bauzas07:12
*** zhurong has quit IRC07:24
*** tmorin has joined #openstack-release07:34
*** pbandark has joined #openstack-release07:34
*** Qiming has quit IRC07:36
*** dtantsur|afk is now known as dtantsur07:40
*** Qiming has joined #openstack-release07:42
*** zhurong has joined #openstack-release07:42
*** amoralej|off is now known as amoralej07:48
*** oanson has quit IRC07:52
*** oanson has joined #openstack-release07:53
*** alexchadin has joined #openstack-release08:09
*** lucas-afk is now known as lucasagomes08:26
*** witek has quit IRC08:26
*** electrofelix has joined #openstack-release08:28
openstackgerritMerged openstack/releases master: Release magnum 3.2.0  https://review.openstack.org/46047808:40
openstackgerritMerged openstack/releases master: Release magnum 2.1.0  https://review.openstack.org/46048108:41
*** zhurong has quit IRC08:47
*** zhurong has joined #openstack-release09:01
*** tmorin has quit IRC09:13
*** Jeffrey4l has quit IRC09:25
*** Jeffrey4l has joined #openstack-release09:27
*** zhurong has quit IRC09:41
*** udesale has quit IRC09:45
*** alexchadin has quit IRC09:48
*** sdague has joined #openstack-release09:55
*** tmorin has joined #openstack-release09:59
*** tmorin has left #openstack-release10:01
*** gcb has quit IRC10:04
*** ricolin has quit IRC10:07
*** ricolin has joined #openstack-release10:08
*** zhurong has joined #openstack-release10:19
*** oanson has quit IRC10:19
*** oanson has joined #openstack-release10:21
*** zhurong has quit IRC10:30
*** yanyanhu has quit IRC10:40
*** tmorin1 has joined #openstack-release10:47
*** witek has joined #openstack-release10:50
*** yolkfull__ has quit IRC11:00
*** alexchadin has joined #openstack-release11:00
*** alexchadin has quit IRC11:14
*** alexchadin has joined #openstack-release11:14
*** lucasagomes is now known as lucas-hungry11:28
*** alexchadin has quit IRC11:29
*** alexchadin has joined #openstack-release11:34
*** tellesnobrega has joined #openstack-release11:35
openstackgerritDean Troyer proposed openstack/releases master: Release python-openstacklient 3.10.0  https://review.openstack.org/46195911:40
dtroyerEmilienM: ^^^^ thanks11:41
EmilienMwoot11:42
*** pcaruana has quit IRC11:53
ttx+2d11:54
openstackgerritMerged openstack/releases master: Release python-openstacklient 3.10.0  https://review.openstack.org/46195911:57
*** pcaruana has joined #openstack-release11:57
*** wolverineav has joined #openstack-release11:58
*** zhurong has joined #openstack-release12:02
*** Qiming has quit IRC12:06
*** tmorin1 has quit IRC12:08
*** tmorin has joined #openstack-release12:08
*** Qiming has joined #openstack-release12:20
*** askb has quit IRC12:24
*** alexchadin has quit IRC12:34
*** lucas-hungry is now known as lucasagomes12:42
*** mriedem has joined #openstack-release12:51
dimsttx : dhellmann : o/12:55
ttxdims: o/12:55
*** gcb has joined #openstack-release12:56
*** claudiub has joined #openstack-release13:02
*** alexchadin has joined #openstack-release13:06
dhellmanndims, ttx: o/13:06
*** tmorin has quit IRC13:08
*** askb has joined #openstack-release13:15
*** zhurong has quit IRC13:28
*** dtantsur is now known as dtantsur|brb13:35
*** gcb has quit IRC13:40
*** gcb has joined #openstack-release13:40
*** gcb has quit IRC13:43
*** gcb has joined #openstack-release13:44
andymccrmorning releases team. im having an issue with the reno gate for OSA - TL;DR some duplicate renos merged in newton (the id at the end was the same, due to a bulk patch across multiple repos), that's since been fixed, but now the "fix patch" causes failures on ocata/master reno gates13:46
andymccrit looks due to the way it iterates through commits (it tries to perform the delete for the rename patch, before processing the commit that adds the original duplicate renos - so fails)13:46
*** mriedem has quit IRC13:48
*** mriedem has joined #openstack-release13:51
*** armax has joined #openstack-release13:51
*** zhurong has joined #openstack-release14:02
*** bobh has joined #openstack-release14:06
*** openstackgerrit has quit IRC14:18
*** dave-mccowan has joined #openstack-release14:25
*** amotoki has joined #openstack-release14:26
*** askb has quit IRC14:36
ttxandymccr: do you have pointers to the failed tests ? /cc dhellmann14:40
andymccrttx: sure do! the failure is here: http://logs.openstack.org/04/461004/5/check/gate-openstack-ansible-releasenotes/e9001b0/console.html#_2017-05-03_08_59_24_65095314:41
*** udesale has joined #openstack-release14:43
ttxwow, that exceeds my limited reno debug capabilities. And I don't have much time this week to improve those capabilities :) But maybe piece of cake for dhellmann14:47
*** zhurong has quit IRC14:52
*** dtantsur|brb is now known as dtantsur14:56
*** gcb has quit IRC15:02
*** gcb has joined #openstack-release15:03
*** gcb has quit IRC15:10
*** hongbin has joined #openstack-release15:13
*** gcb has joined #openstack-release15:17
*** udesale has quit IRC15:22
dhellmannandymccr, ttx : I'll take a look15:23
*** alexchadin has quit IRC15:24
andymccrdhellmann: thanks! I dug a bit, so i think i understand the reason its failing, but im not too sure how to best fix that - appreciate the help15:27
dhellmannandymccr : this is definitely a case I didn't anticipate with big random numbers to provide unique filenames :-)15:28
andymccrdhellmann: edge cases are always more fun!15:29
dhellmannandymccr: I think I understand what happened here, please verify: there are a bunch of reno files in one commit but that contain different content. is that right?15:40
andymccrdhellmann: yes, there are 5 renos in https://github.com/openstack/openstack-ansible/commit/179668188a18c0f183136220fb6ab14e593a32f5 that have the same ID but different content (its basically the same feature for a few different role repositories)15:42
*** Rockyg has joined #openstack-release15:42
dhellmannok, got it15:42
andymccrusually that doesnt merge because it checks for duplicate ids (so not sure how it did merge this time!)15:42
dhellmannI wonder why that patch didn't fail in this same way :-/15:42
andymccrthe gate for stable/ocata was then failing because of duplicates - so i adjusted them: https://github.com/openstack/openstack-ansible/commit/121fb00c82b62be4585f16b95b4fcb67495d83f115:43
andymccrbut yeah now its failing as its not able to delete :)15:43
*** tmorin has joined #openstack-release15:43
dhellmannyeah15:44
dhellmannok, still digging15:44
dhellmannI think I just need to make that loop smarter15:44
dhellmannI suppose I should ignore those files15:44
andymccri figured it should loop through the other way and that way it would action the "add" first, and then the "delete" afterwards. but couldnt quite figure out how to do that :)15:46
dhellmannthe problem isn't that there's an add, it's that there are several files with the same uid being added15:46
dhellmannthat should have barfed15:46
andymccrcould the uid not be based on a hash of the filename?15:47
*** openstackgerrit has joined #openstack-release15:59
openstackgerritDoug Hellmann proposed openstack/reno master: deal with non-unique UIDs  https://review.openstack.org/46219915:59
dhellmannandymccr : see if ^^ produces the output you expect locally, please15:59
dhellmannnow I need to figure out how to protect against this case in the future without breaking your build again15:59
andymccrdhellmann: i'll give that a test -thanks for digging into that16:00
dhellmannandymccr : maybe a better fix is to just ignore those files and emit a warning that they're being ignored. that leaves them unhandled, and they can then be renamed16:02
dhellmannandymccr : I guess the patch as it stands now might fail by emitting the notes twice, so that's the thing to check for16:03
dhellmannyeah, I'm seeing dupes16:04
*** tmorin has quit IRC16:10
andymccrdhellmann: how can you tell the end result? im seeing deletes for the 5 notes16:10
andymccrthe test passed - but yeah if its passing /w duplicates thats not ideal16:11
dhellmannandymccr : I ran "tox -e releasenotes" in openstack-ansible then .tox/releasenotes/bin/pip uninstall reno then installed reno with that patch then ran tox -e releasenotes and looked at the output16:12
andymccrdhellmann: ok so im looking at the same - but i can see "note deleted in 121fb00c82b62be4585f16b95b4fcb67495d83f1" x 5 which im guessing is the removal of teh duplicate from https://github.com/openstack/openstack-ansible/commit/121fb00c82b62be4585f16b95b4fcb67495d83f116:13
*** ekcs has joined #openstack-release16:13
andymccror at least thats how im reading it16:13
openstackgerritDoug Hellmann proposed openstack/reno master: deal with non-unique UIDs  https://review.openstack.org/46219916:13
dhellmannandymccr : yeah, it's going to report on everything it does but you have to look at the actual html output to see the duplicate content16:14
andymccrahh i see16:14
dhellmannthe new version of the patch eliminates the duplicates in the output16:14
andymccrdhellmann: ahh ok tested that - yeah the new one doesnt have duplicates in the html output.16:17
andymccrdhellmann: thanks again. that LGTM.16:19
dhellmannandymccr : great16:19
dhellmannI wonder if that warning is enough16:20
dhellmannideally the add would have broken things16:20
*** lucasagomes is now known as lucas-afk16:21
andymccrdhellmann: agreed. im not too sure how that didnt flag as duplicates (it did on the ocata gate). I guess the warning just exists for the case where the duplicates were somehow added - but allows them to be deleted.16:22
dhellmannright, if I throw an error there you'll never be able to fix your build16:23
*** pcaruana has quit IRC16:24
*** shardy has quit IRC16:28
*** tellesnobrega has quit IRC16:40
*** dtantsur is now known as dtantsur|afk16:40
*** armax has quit IRC16:43
*** tellesnobrega has joined #openstack-release16:46
*** harlowja has joined #openstack-release16:48
*** armax has joined #openstack-release16:51
dhellmannandymccr : did you open a bug for this that I can reference in these patches?16:55
openstackgerritDoug Hellmann proposed openstack/reno master: deal with non-unique UIDs  https://review.openstack.org/46219917:01
openstackgerritDoug Hellmann proposed openstack/reno master: modify the change aggregation api  https://review.openstack.org/46222017:01
openstackgerritDoug Hellmann proposed openstack/reno master: do not allow multiple files with the same UID  https://review.openstack.org/46222117:01
*** ricolin has quit IRC17:04
dhellmanndims, ttx, AJaegar: it would be good to get this series reviewed and released soon to unblock andymccr and the OSA team17:06
*** zul has quit IRC17:26
*** Rockyg has quit IRC17:27
*** zul has joined #openstack-release17:29
*** amoralej is now known as amoralej|off17:29
dimsack dhellmann17:37
*** mtreinish has quit IRC17:46
dhellmanndims : I'm going to add a release note there -- I keep forgetting to add them in reno :-)17:46
dimsLOL :)17:47
openstackgerritDoug Hellmann proposed openstack/reno master: do not allow multiple files with the same UID  https://review.openstack.org/46222117:49
*** hongbin has quit IRC17:55
*** pbandark has quit IRC18:03
*** mtreinish has joined #openstack-release18:05
*** mriedem is now known as mriedem_away18:19
*** david-lyle has quit IRC18:24
*** dklyle has joined #openstack-release18:24
EmilienMdhellmann: hey, we have a bunch of release notes that have same ideas, but not for same project. For example, it's highly possible that some renos have been copy/pasted across projects. Is it a big deal?18:32
dhellmannEmilienM : that case is fine18:32
EmilienMok18:33
dhellmannthe issue is the way reno tries to figure out if a file has been renamed within a given repository18:33
dhellmannsmcginnis : can you unapprove https://review.openstack.org/462250 for now?18:34
dhellmannI need to test something -- it seems to be deleting the old note18:35
smcginnisdhellmann: Not just renaming?18:35
dhellmannsmcginnis : because the "sameness" detection is based on the UID, not the full filename, renaming one of those files makes reno think both were deleted18:36
smcginnisHuh. Well, good luck with that. :)18:36
dhellmannyeah18:36
-openstackstatus- NOTICE: Gerrit on review.openstack.org is being restarted to accomodate a memory leak in Gerrit. Service should return shortly.18:53
*** electrofelix has quit IRC18:55
*** hongbin has joined #openstack-release18:59
*** hongbin has quit IRC19:06
*** hongbin has joined #openstack-release19:07
openstackgerritTerry Wilson proposed openstack/releases master: Release ovsdbapp 0.4.0  https://review.openstack.org/46227319:24
*** otherwiseguy has joined #openstack-release19:35
openstackgerritDoug Hellmann proposed openstack/reno master: add a lint command  https://review.openstack.org/46227619:36
openstackgerritDoug Hellmann proposed openstack/reno master: add a lint command  https://review.openstack.org/46227619:38
*** mriedem_away is now known as mriedem19:49
openstackgerritDoug Hellmann proposed openstack/reno master: add a lint command  https://review.openstack.org/46227619:53
*** otherwiseguy has quit IRC20:17
*** dave-mccowan has quit IRC20:41
*** askb has joined #openstack-release21:01
*** hongbin has quit IRC21:12
*** hongbin has joined #openstack-release21:12
*** bobh has quit IRC21:15
*** otherwiseguy has joined #openstack-release21:28
*** wolverineav has quit IRC22:07
*** wolverineav has joined #openstack-release22:08
*** wolverineav has quit IRC22:15
*** abadger2000 has joined #openstack-release22:25
*** harlowja has quit IRC22:28
*** abadger1999 has quit IRC22:28
*** d0ugal has quit IRC22:28
*** d0ugal has joined #openstack-release22:53
*** sdague has quit IRC23:02
*** bobh has joined #openstack-release23:04
*** csomerville has joined #openstack-release23:04
*** hongbin has quit IRC23:08
*** cody-somerville has quit IRC23:09
*** bobh has quit IRC23:11
*** dklyle has quit IRC23:23
*** harlowja has joined #openstack-release23:31
*** gcb has quit IRC23:42

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