Wednesday, 2018-04-11

*** yamahata has joined #openstack-release00:07
*** annabelleB has quit IRC00:08
*** zhurong has joined #openstack-release00:21
*** annabelleB has joined #openstack-release00:27
prometheanfirebnemec: you got that patch in quick00:41
*** yamamoto has joined #openstack-release00:43
*** dave-mccowan has joined #openstack-release00:46
*** odyssey4me has quit IRC00:47
*** odyssey4me has joined #openstack-release00:47
*** yamamoto has quit IRC00:48
*** mriedem has quit IRC00:49
*** annabelleB has quit IRC00:50
*** kumarmn has joined #openstack-release01:04
*** rosmaita has quit IRC01:04
*** kumarmn has quit IRC01:08
*** harlowja has quit IRC01:14
*** kumarmn has joined #openstack-release01:16
*** iyamahat has quit IRC01:24
*** iyamahat has joined #openstack-release01:24
*** ykarel|away has joined #openstack-release01:34
*** hongbin has joined #openstack-release01:40
*** yamamoto has joined #openstack-release01:44
*** yamamoto has quit IRC01:50
*** ykarel|away has quit IRC01:54
*** zhurong has quit IRC02:00
*** kumarmn has quit IRC02:11
*** claudiub has quit IRC02:19
*** ricolin has joined #openstack-release02:34
*** ianychoi has quit IRC02:35
*** armax has quit IRC02:40
*** yamamoto has joined #openstack-release02:47
*** yamamoto has quit IRC02:52
*** dave-mccowan has quit IRC02:56
*** armax has joined #openstack-release03:10
*** kumarmn has joined #openstack-release03:19
*** ianychoi has joined #openstack-release03:19
*** kumarmn has quit IRC03:24
*** udesale has joined #openstack-release03:34
*** yamamoto has joined #openstack-release03:48
*** yamamoto has quit IRC03:53
*** hongbin has quit IRC03:59
*** Qiming_ is now known as Qiming04:02
*** harlowja has joined #openstack-release04:23
*** sree has joined #openstack-release04:26
*** udesale has quit IRC04:26
*** udesale has joined #openstack-release04:45
*** udesale has quit IRC04:46
*** udesale has joined #openstack-release04:47
*** yamamoto has joined #openstack-release04:50
*** yamamoto has quit IRC04:55
*** d0ugal has quit IRC05:16
*** armax has quit IRC05:20
*** d0ugal has joined #openstack-release05:27
*** harlowja has quit IRC05:35
*** yamamoto has joined #openstack-release05:52
*** armax has joined #openstack-release05:54
*** ykarel|away has joined #openstack-release05:56
*** yamamoto has quit IRC05:57
*** ykarel|away is now known as ykarel06:03
*** sree has quit IRC06:11
*** sree has joined #openstack-release06:12
*** sree has quit IRC06:13
*** sree has joined #openstack-release06:13
*** armax has quit IRC06:20
*** armax has joined #openstack-release06:20
*** sree has quit IRC06:22
*** sree has joined #openstack-release06:22
*** sree has quit IRC06:24
*** sree has joined #openstack-release06:25
*** armax has quit IRC06:25
*** sree has quit IRC06:26
*** sree has joined #openstack-release06:27
*** jhesketh_ is now known as jhesketh06:29
*** sree has quit IRC06:31
*** radeks has joined #openstack-release06:33
*** sree has joined #openstack-release06:43
*** sree has quit IRC06:48
*** yamamoto has joined #openstack-release06:53
*** alexchadin has joined #openstack-release06:53
*** yamamoto has quit IRC06:59
*** sree has joined #openstack-release07:05
*** yamamoto has joined #openstack-release07:08
*** amoralej|off is now known as amoralej07:08
*** zhurong has joined #openstack-release07:10
*** sree has quit IRC07:10
*** shardy has joined #openstack-release07:14
*** claudiub has joined #openstack-release07:29
*** iyamahat has quit IRC07:29
*** gcb has joined #openstack-release07:30
*** ykarel is now known as ykarel|lunch07:31
*** yamamoto has quit IRC07:49
*** ifat_afek has joined #openstack-release07:49
*** pkovar has joined #openstack-release07:52
*** jpich has joined #openstack-release07:58
*** sree has joined #openstack-release07:58
*** sree has quit IRC08:08
*** zhurong has quit IRC08:08
*** lucas-afk is now known as lucasagomes08:11
*** pcaruana has joined #openstack-release08:13
*** yamamoto has joined #openstack-release08:13
*** yamamoto has quit IRC08:15
*** yamamoto has joined #openstack-release08:19
*** ykarel|lunch is now known as ykarel08:21
openstackgerritStephen Finucane proposed openstack/reno master: Integrate a setuptools command  https://review.openstack.org/47503308:26
*** electrofelix has joined #openstack-release08:40
*** radeks has quit IRC08:41
*** ianychoi has quit IRC08:42
*** radeks has joined #openstack-release08:42
*** zhurong has joined #openstack-release08:50
*** yamamoto has quit IRC08:54
*** yamamoto has joined #openstack-release08:56
*** yamamoto has quit IRC08:58
*** yamamoto has joined #openstack-release08:58
*** armax has joined #openstack-release09:18
openstackgerritThierry Carrez proposed openstack/releases master: Add dib-utils to aclfixer.py exception list  https://review.openstack.org/56032609:21
*** armax has quit IRC09:23
*** ifat_afek has quit IRC09:27
openstackgerritMerged openstack/releases master: Oslo releases for 2018-4-10  https://review.openstack.org/56014909:39
*** yamamoto has quit IRC09:43
*** yamamoto has joined #openstack-release09:44
*** ianychoi has joined #openstack-release09:46
*** alexchadin has quit IRC09:47
*** alexchadin has joined #openstack-release09:48
*** alexchadin has quit IRC09:48
*** alexchadin has joined #openstack-release09:49
*** alexchadin has quit IRC09:49
*** alexchadin has joined #openstack-release09:50
*** alexchadin has quit IRC09:50
*** alexchadin has joined #openstack-release09:50
*** alexchadin has quit IRC09:51
*** alexchadin has joined #openstack-release09:51
*** alexchadin has quit IRC09:51
*** alexchadin has joined #openstack-release09:52
*** alexchadin has quit IRC09:56
*** sree has joined #openstack-release10:05
*** sree has quit IRC10:09
*** sree has joined #openstack-release10:15
*** alexchadin has joined #openstack-release10:23
*** shardy is now known as shardy_afk10:36
*** pkovar has quit IRC10:38
*** alexchadin has quit IRC10:45
*** alexchadin has joined #openstack-release10:50
*** e0ne has joined #openstack-release10:54
*** alexchadin has quit IRC11:03
*** shardy_afk is now known as shardy11:04
*** e0ne has quit IRC11:04
*** pkovar has joined #openstack-release11:06
*** yamamoto has quit IRC11:18
*** e0ne has joined #openstack-release11:20
*** armax has joined #openstack-release11:20
*** kumarmn has joined #openstack-release11:20
*** armax has quit IRC11:24
*** kumarmn has quit IRC11:25
*** yamamoto has joined #openstack-release11:25
*** yamamoto has quit IRC11:30
*** yamamoto has joined #openstack-release11:34
*** yamamoto has quit IRC11:39
*** amoralej is now known as amoralej|lunch12:01
*** edmondsw has joined #openstack-release12:08
*** rosmaita has joined #openstack-release12:11
*** odyssey4me has quit IRC12:14
*** odyssey4me has joined #openstack-release12:14
*** alexchadin has joined #openstack-release12:15
*** yamamoto has joined #openstack-release12:21
dhellmanntonyb : keeping up with nodes like that is a pain, and that's why I prefer to produce rst programmatically and then call nested_parse_with_titles() https://github.com/sphinx-contrib/datatemplates/blob/master/sphinxcontrib/datatemplates/directive.py#L8112:22
*** yamamoto has quit IRC12:25
*** gcb has quit IRC12:27
*** kumarmn has joined #openstack-release12:31
*** yamamoto has joined #openstack-release12:34
*** dave-mccowan has joined #openstack-release12:34
*** kumarmn has quit IRC12:53
*** amoralej|lunch is now known as amoralej13:01
*** bobh has joined #openstack-release13:01
*** mriedem has joined #openstack-release13:06
*** kumarmn has joined #openstack-release13:09
*** kumarmn has quit IRC13:09
*** kumarmn has joined #openstack-release13:09
*** armax has joined #openstack-release13:21
*** radeks has quit IRC13:21
*** radeks has joined #openstack-release13:22
*** armax has quit IRC13:26
*** gcb has joined #openstack-release13:33
*** esberglu has joined #openstack-release13:35
*** alexchadin has quit IRC13:44
*** hongbin has joined #openstack-release13:47
mriedemprometheanfire: so should i be expecting to see an reqs update proposal bot change for https://review.openstack.org/#/c/559367/ in nova?13:50
*** udesale has quit IRC13:51
*** ttx has quit IRC13:55
*** ttx has joined #openstack-release13:57
*** udesale has joined #openstack-release14:00
dhellmannmriedem : the bot is turned off, so no14:07
dhellmannalthough maybe prometheanfire was going to do those by hand14:07
mriedemok "2. push uncapped eventlet out via requirements updates to all consumers" wasn't clear to me14:10
mriedembut i guess we need https://review.openstack.org/#/c/560420/ then14:10
dhellmannyeah, I'm not sure what he had in mind there14:10
mriedemdhellmann: jichen said he asked you14:10
dhellmannhmm14:10
dhellmannah, yeah.14:11
mriedemso a g-r change was made which is going to blow up the world before any of the other projects changes were in place?14:11
dhellmannin parallel with the eventlet change we made a change that means all dependency specifications must match the global rules when any are changed14:11
dhellmannbefore we only checked the dependency spec being updated14:11
mriedemoh14:11
mriedemwhich is https://review.openstack.org/#/c/523387/32/requirements.txt14:11
dhellmannso if the requirements files aren't touched, everythying is fine14:11
mriedemyeah it is in that patch14:11
dhellmannbut if anything is touched the whole file has to pass the tests14:12
dhellmannand I think they were running into an error because of eventlet when they added or changed something else14:12
mriedemok, i guess i don't know if i should merge the eventlet uncap on its own, or tell them to include it in the zvm patch that changes requirements.txt14:12
dhellmannif it passes the tests you can merge it on its own14:12
mriedemi assume this is a foregone conclusion and we'll just need jichen's patch14:12
*** pkovar has quit IRC14:13
mriedemok yeah so far passing unit and functional tests, so tempest should be fine14:13
dhellmannyeah, the surprise might be the check-requirements job14:13
mriedemyeah it was, i didn't try to follow the ML thread14:13
dhellmannlet's see if it passes with just the eventlet change14:14
dhellmannif not I'll help you get things squared away so future changes are easier14:14
*** iyamahat has joined #openstack-release14:19
*** bobh has quit IRC14:25
*** iyamahat has quit IRC14:29
*** iyamahat has joined #openstack-release14:30
*** mlavalle has joined #openstack-release14:30
openstackgerritBrian Rosmaita proposed openstack/releases master: Release python-glanceclient 2.11.0 (rocky)  https://review.openstack.org/56044814:37
*** annabelleB has joined #openstack-release14:45
*** bobh has joined #openstack-release14:57
*** armax has joined #openstack-release15:01
openstackgerritMerged openstack/releases master: oslo.service 1.31.0  https://review.openstack.org/56016315:02
*** ykarel is now known as ykarel|away15:05
prometheanfiredhellmann: I'm going to have to, which sucks :|15:14
dhellmannprometheanfire : how many projects have the cap in place?15:16
prometheanfireI need to do a codesearch/grep15:17
prometheanfirehttp://codesearch.openstack.org/?q=eventlet&i=nope&files=.*requirements.*&repos=15:17
dhellmannprometheanfire : http://paste.openstack.org/show/718945/15:19
prometheanfiremistral barbican blazar cinder compute-hyperv congress cyborg desinate ec2api futurist gve-api glance glance_store (glare) hacking heat heat-tempest-plugin ironic ironic-inspector ironic-lib ironic-python-agent karbor kingbird kuryr-kubernetes magnum manila mixmatch mogan monasca-agent monasca-api monasca-events-api monasca-log-api murano murano-agent networking-cisco networking-hyperv15:22
prometheanfirenetworking-mlnx networking-powervm  networking-sfc neutron15:22
prometheanfireheh, yep15:22
prometheanfirethat one of tony's tools?15:22
dhellmannit's a wrapper around beagle15:23
dhellmannbeagle search \15:23
dhellmann        --ignore-case \15:23
dhellmann        --file '(.*requirement.*|.*constraint.*|setup.cfg|tox.ini)' \15:23
dhellmann        ${1}'[><=! ]'15:23
prometheanfireah15:23
prometheanfireI'd also include setup.py15:24
prometheanfirebut ya, useful15:24
dhellmannyeah15:24
prometheanfireI need to package that or install it via venv still15:24
prometheanfireadded to the todo list15:24
prometheanfireso, brb, getting caffeine15:25
dhellmannI ran it with 'eventlet' and then piped that to grep '<' to find the ones with a cap15:25
prometheanfireyep15:25
strigaziprometheanfire: dhellmann what is the above list? o.service users?15:26
dhellmannstrigazi : projects that have eventlet capped15:26
dhellmannwe turned off the bot that syncs dependency settings before we removed the eventlet cap so we need to update those by hand15:26
strigaziwe need to inc version? to which version?15:27
dhellmannstrigazi : we need to remove the <X.Y settings15:27
dhellmannwe shouldn't need to change anything else15:27
strigazidhellmann: ok, thanks15:27
*** iyamahat has quit IRC15:29
strigazispeaking of eventlet, I don't know how to test better this fix in eventlet https://github.com/eventlet/eventlet/issues/172#issuecomment-379421165 it is the compatibility issue with threading (on my local setup with magnum works). I'll talk to oslo but I thought you might be interested.15:32
*** yamahata has quit IRC15:33
*** bobh has quit IRC15:36
*** bobh_ has joined #openstack-release15:36
*** ykarel|away has quit IRC15:38
*** edmondsw has quit IRC15:39
*** annabelleB has quit IRC15:45
*** iyamahat has joined #openstack-release15:45
*** lucasagomes is now known as lucas-afk15:59
dhellmannprometheanfire : incoming patches for most of those repos: https://review.openstack.org/#/q/topic:uncap-eventlet+(status:open+OR+status:merged)16:00
dhellmannit's going to take a few minutes for the script to submit them all16:00
dhellmannand that only handles the easy cases; there are a few special cases that need to be done by hand16:00
*** ykarel|away has joined #openstack-release16:01
prometheanfiredhellmann: oh, you got around to it before me, nice (hound is now packaged in gentoo btw)16:01
dhellmannhound or beagle?16:01
prometheanfires/hound/beagle16:01
dhellmannyeah16:01
dhellmannthese look like special cases: daisycloud-core  taskflow  terracotta  tricircle  trove  valence  vitrage  vmware-nsx  vmware-nsxlib  zun16:02
dhellmannthe sed command I ran against requirements.txt and test-requirements.txt failed for one reason or another16:02
*** annabelleB has joined #openstack-release16:02
prometheanfirehttps://pypi.python.org/pypi/cliff16:03
prometheanfirethat seems really out of date16:03
dhellmannout of date how?16:03
smcginnisThere are a few projects that already have patches out there to uncap eventlet.16:04
smcginnisWell, at least Cinder does. :)16:04
*** yamahata has joined #openstack-release16:04
prometheanfiredhellmann: 2.4.1 is what shows up for me there16:05
prometheanfire2.11.0 is out16:05
*** harlowja has joined #openstack-release16:05
prometheanfiremaybe I'm looking at the wrong pypi?16:05
dhellmannprometheanfire : https://pypi.org/project/cliff/16:05
prometheanfireoh, that's pypi.python.org16:05
prometheanfireya16:05
prometheanfiredhellmann: beagle depends on cliff, beagle states support for py36, cliff does not state support for py36, so the dependency is broken16:07
dhellmannprometheanfire : cliff supports python 316:08
prometheanfireincluding 3.0?16:08
*** bobh_ has quit IRC16:09
dhellmannI think I have all of those uncap patches proposed now16:11
dhellmannprometheanfire : if you want more detail in the metadata please just propose a patch16:11
jlvillalWe (Ironic) were wondering if some of the blacklisted Python packages should be put into upper-constraints/lower-constraints in stable branches. For example 'pycodestyle'16:12
jlvillalThere was a new release of pycodestyle which broke our master, but also broke our stable branches.16:12
dhellmannjlvillal : that's a question for prometheanfire but I think we expected projects to pin those versions in their test-requirements.txt file16:13
prometheanfirejlvillal: dhellmann is correct, those pins (preferably a range vs ===) go in test-requires16:14
jlvillaldhellmann, So the projects are supposed to do that when they make the stable branch?16:15
dhellmannjlvillal : ideally you would set up the cap on master and then it would carry over to the stable branch, but since that didn't happen I guess you'll need to update the stable branch now16:15
jlvillaldhellmann, Hmm. I'm thinking of situation where you want to let it float on master. And fix as needed. But on stable it shouldn't break once released.16:16
dhellmannjlvillal : those things are not typically the sorts of things we want to let float. They tend to introduce breaking changes at inopportune times.16:17
prometheanfirejlvillal: new releases that break things (and cause the 'need' for a cap) are a pain, we have capped those things in the past16:17
dhellmannI'm a little surprised the caps aren't already in place but maybe we were relying on hacking to do that16:17
prometheanfirethat's my assumption16:17
jlvillaldhellmann, prometheanfire: Thanks. Yeah 'pycodestyle' was never in the test-requirements file.16:18
dhellmannis hacking listed? maybe we expected hacking to cap pycodestyle?16:18
dhellmannwe should write this down when we figure it out. :-)16:18
jlvillalhacking!=0.13.0,<0.14,>=0.12.0 # Apache-2.016:19
prometheanfirepycodestyle isn't part of hacking16:19
dhellmannisn't pycodestyle the replacement for pep8?16:28
dhellmannyeah, it's the renamed pep816:28
prometheanfireyep16:28
dhellmannhmm, and hacking doesn't depend on pep816:29
dhellmannoh, flake816:29
dhellmannwhich installs pep816:29
*** masayukig has joined #openstack-release16:29
*** sree has quit IRC16:29
dhellmannso where is pycodestyle coming from?16:29
smcginnisIsn't that the "new" pep8?16:30
prometheanfireya, because of the cap16:30
prometheanfirewhere it's coming from I don't know though16:30
smcginnisOh, I see you just said that. :)16:31
dhellmannit's listed in the ironic test-requirements.txt file with a cap16:31
dhellmannat least on master16:31
*** jpich has quit IRC16:31
dhellmanntime for lunch, bbiab16:31
prometheanfiredhellmann: cya16:32
*** udesale has quit IRC16:35
*** ykarel|away has quit IRC16:43
clarkbdhellmann: prometheanfire: my pep8 tox log says "Collecting pycodestyle (from flake8-import-order>=0.13->-r /home/clark/src/openstack/ironic/test-requirements.txt (line 20))"16:45
clarkbappears that ironic's use of flake8-import-order may be to blame16:46
prometheanfireclarkb: ya, they may either need a cap on that or pycodestyle16:48
*** electrofelix has quit IRC16:53
*** iyamahat has quit IRC16:59
*** iyamahat has joined #openstack-release16:59
*** e0ne has quit IRC16:59
*** sree has joined #openstack-release17:03
rosmaitacould someone eyeball the .yaml in https://review.openstack.org/#/c/560448/ ? it's failing all 3 tests due to "TypeError: unorderable types: NoneType() < str()" so i'm thinking some kind of syntax edge case, but the yaml looks ok to me17:06
*** harlowja has quit IRC17:07
rosmaitasmcginnis ^^17:07
*** sree has quit IRC17:07
smcginnisHmm17:07
rosmaitai'm hoping that i did something stupid and it will be easy to spot17:08
smcginnisrosmaita: It might be something from a recent change in our validation scripts.17:09
rosmaitasmcginnis: i just compared it to http://git.openstack.org/cgit/openstack/releases/tree/deliverables/queens/python-glanceclient.yaml17:10
rosmaitado i need a branches stanza? or is that only for when the rocky branch is actually cut ?17:11
smcginnisrosmaita: No, not until we actually need a branch.17:11
smcginnisrosmaita: It's actually blowing up on parsing the repos that are part of the deliverable.17:12
smcginnisThe good thing is I can reproduce it locally.17:12
rosmaitaok17:12
rosmaitasounds like a good time for me to go to lunch17:13
rosmaita:)17:13
smcginnis;)17:13
smcginnisrosmaita: Oh, I think I see. Checking something...17:15
smcginnisrosmaita: Extra "-" in there.17:15
openstackgerritSean McGinnis proposed openstack/releases master: Release python-glanceclient 2.11.0 (rocky)  https://review.openstack.org/56044817:16
*** annabelleB has quit IRC17:19
dhellmannwell spotted17:20
smcginnisThat one was hard to spot.17:21
*** annabelleB has joined #openstack-release17:24
*** ekcs has joined #openstack-release17:25
rosmaitasmcginnis thanks!  that extra '-' looks so natural there!17:27
*** sree has joined #openstack-release17:29
*** yamamoto has quit IRC17:29
openstackgerritDoug Hellmann proposed openstack/releases master: specify required fields for projects  https://review.openstack.org/56057117:30
dhellmannsmcginnis, rosmaita : ^^ would have caught the error17:30
*** kumarmn has quit IRC17:32
*** sree has quit IRC17:33
*** kumarmn has joined #openstack-release17:35
*** kumarmn has quit IRC17:35
*** kumarmn has joined #openstack-release17:37
dhellmannprometheanfire : whee, it looks like most of those uncap patches are failing17:39
prometheanfirelol17:39
prometheanfirelooks closer to 50:5017:40
dhellmannprobably because of the new check-requirements change that enforces consistency on all dependencies when any changes17:40
prometheanfirebut ya, a bunch17:40
prometheanfireya17:40
*** ykarel|away has joined #openstack-release17:49
*** yamamoto has joined #openstack-release17:49
*** e0ne has joined #openstack-release17:50
*** yamamoto has quit IRC17:50
*** yamamoto has joined #openstack-release17:50
openstackgerritJay Bryant proposed openstack/releases master: Add Cinder Dates for Rocky  https://review.openstack.org/56057217:50
*** harlowja has joined #openstack-release17:52
*** yamamoto has quit IRC17:57
*** edmondsw has joined #openstack-release17:59
*** kumarmn has quit IRC18:02
openstackgerritMerged openstack/releases master: Release python-glanceclient 2.11.0 (rocky)  https://review.openstack.org/56044818:07
*** kumarmn has joined #openstack-release18:10
*** annabelleB has quit IRC18:14
*** iyamahat has quit IRC18:14
*** sree has joined #openstack-release18:14
*** yamamoto has joined #openstack-release18:16
*** iyamahat has joined #openstack-release18:16
*** annabelleB has joined #openstack-release18:17
*** yamamoto has quit IRC18:18
*** sree has quit IRC18:19
*** radeks has quit IRC18:21
*** radeks has joined #openstack-release18:22
*** ykarel|away has quit IRC18:24
*** ricolin has quit IRC18:24
*** yamamoto has joined #openstack-release18:25
*** claudiub has quit IRC18:31
*** yamamoto has quit IRC18:34
*** rosmaita has quit IRC18:35
*** yamamoto has joined #openstack-release18:38
openstackgerritJay Bryant proposed openstack/releases master: Add Cinder Dates for Rocky  https://review.openstack.org/56057218:38
*** annabelleB has quit IRC18:39
*** yamamoto has quit IRC18:40
*** annabelleB has joined #openstack-release18:45
*** tenobreg has joined #openstack-release18:55
*** pcaruana has quit IRC18:55
*** tellesnobrega has quit IRC18:56
*** tenobreg is now known as tellesnobrega18:56
openstackgerritMerged openstack/releases master: Add Cinder Dates for Rocky  https://review.openstack.org/56057219:07
*** kumarmn has quit IRC19:22
*** kumarmn has joined #openstack-release19:26
*** annabelleB has quit IRC19:29
*** iyamahat has quit IRC19:35
*** annabelleB has joined #openstack-release19:36
*** annabelleB_ has joined #openstack-release19:40
*** annabelleB has quit IRC19:41
*** annabelleB_ is now known as annabelleB19:41
*** kumarmn has quit IRC19:44
*** iyamahat has joined #openstack-release19:44
*** kumarmn has joined #openstack-release19:45
*** kumarmn has quit IRC19:46
*** kumarmn has joined #openstack-release19:46
*** edmondsw has quit IRC19:47
*** kumarmn has quit IRC19:51
*** kumarmn has joined #openstack-release19:52
*** kumarmn_ has joined #openstack-release19:54
*** kumarmn_ has quit IRC19:54
*** kumarmn_ has joined #openstack-release19:54
*** kumarmn has quit IRC19:56
*** bobh has joined #openstack-release19:58
*** claudiub has joined #openstack-release20:02
*** ying_zuo_ has joined #openstack-release20:20
*** dirk__ has joined #openstack-release20:21
*** kmalloc_ has joined #openstack-release20:21
*** dirk has quit IRC20:24
*** dirk__ is now known as dirk20:24
prometheanfirejungleboyj: is this a valid fail for the new oslo-versionedobjects and cinder? http://logs.openstack.org/46/560346/1/check/cross-cinder-py27/074d9aa/testr_results.html.gz20:24
*** kmalloc_ has quit IRC20:25
smcginnisThat's... odd.20:25
*** kmalloc_ has joined #openstack-release20:25
*** rosmaita has joined #openstack-release20:25
prometheanfiresmcginnis: yep20:26
*** kmalloc_ is now known as notmorgan20:26
tonybdhellmann: Thanks for the pointer, I'll look into that.20:26
*** notmorgan has quit IRC20:26
*** notmorgan has joined #openstack-release20:26
*** notmorgan has quit IRC20:26
*** notmorgan has joined #openstack-release20:26
smcginnisprometheanfire: It could be an OVO thing. I wonder if something has changed in there to explicitly covert values to unicode.20:26
*** fanzhang_ has joined #openstack-release20:27
prometheanfireya20:27
*** kmalloc has quit IRC20:27
*** andreaf_ has joined #openstack-release20:27
*** tobberydberg_ has joined #openstack-release20:27
prometheanfirehttps://github.com/openstack/oslo.versionedobjects/compare/1.32.0...1.33.020:27
prometheanfirehttps://github.com/openstack/oslo.versionedobjects/compare/1.32.0...1.33.0#diff-594e9ad3a9f264d03a2f35571f8970ceR36720:27
smcginnisMaybe this: https://github.com/openstack/oslo.versionedobjects/commit/b1d0b5d886afef8c08330bf3c2291e180aa1f53420:27
*** notmorgan is now known as kmalloc20:27
prometheanfireyep20:27
prometheanfirethe cinder test probably should be updated (preferably) but I'm neither team20:28
*** johnthetubaguy_ has joined #openstack-release20:28
*** fanzhang has quit IRC20:28
*** mordred has quit IRC20:28
*** andreaf has quit IRC20:28
*** johnsom has quit IRC20:28
*** ying_zuo has quit IRC20:28
*** tobberydberg has quit IRC20:28
*** johnthetubaguy has quit IRC20:28
*** ying_zuo_ is now known as ying_zuo20:28
smcginnisOdd though that the test expects a string but gets a different string back. Seems like OVO should just be using six.text_type there and not forcing unicode.20:29
*** johnsom has joined #openstack-release20:29
prometheanfiremaybe that's the better answer20:29
*** andreaf_ is now known as andreaf20:29
*** mordred has joined #openstack-release20:30
clarkbrelease team (and fungi for infra input) is there any reason to not go ahead and aprpove https://review.openstack.org/#/c/557730/5 at this point?20:38
clarkbsmcginnis: ^ you might want to send mail to the dev list about what that means?20:38
smcginnisclarkb: Yeah, I can send a notice.20:39
clarkbsmcginnis: ok I'll give it a few more minutes for others to object but otherwise plan to approve it20:39
*** annabelleB has quit IRC20:40
*** annabelleB has joined #openstack-release20:45
smcginnisclarkb: Sent20:47
clarkbsmcginnis: do you want to wait for feedback before I approve or should I just do it now?20:47
fungiclarkb: lgtm but did not approve20:47
*** yamamoto has joined #openstack-release20:52
smcginnisclarkb: I think we can just go ahead.20:52
smcginnisclarkb: There may be one or two that are caught off guard by it, but we've stated this is the official policy, so we should enforce it.20:52
clarkbok approving20:53
*** jtomasek has quit IRC20:54
*** edmondsw has joined #openstack-release20:54
*** sree has joined #openstack-release20:55
*** yamamoto has quit IRC20:56
*** edmondsw has quit IRC20:59
*** sree has quit IRC21:00
corvussmcginnis, dhellmann: i'd like to restart zuul shortly; can you avoid approving releases for a bit?21:11
smcginniscorvus: ack, works for me.21:13
dhellmanncorvus : ack21:15
jungleboyjsmcginnis: prometheanfire  Back.21:18
jungleboyjSo, did you guys decide that the test case needs to be updated?21:18
smcginnisjungleboyj: I have a fix put up against OVO: https://review.openstack.org/#/c/560639/21:19
prometheanfirejungleboyj: ya, I think this is the best comment on the mater21:20
prometheanfire15:29 <          smcginnis > Odd though that the test expects a string but gets a different string back. Seems like OVO should just be using six.text_type there and not forcing unicode.21:20
jungleboyjsmcginnis:  Thank you.21:21
smcginnis;)21:21
*** esberglu has quit IRC21:47
*** radeks has quit IRC22:10
*** kumarmn_ has quit IRC22:10
*** kumarmn has joined #openstack-release22:10
*** kumarmn has quit IRC22:15
*** mlavalle has quit IRC22:19
*** e0ne has quit IRC22:20
-openstackstatus- NOTICE: zuul was restarted to updated to the latest code; you may need to recheck changes uploaded or approvals added between 21:30 and 21:4522:30
*** annabelleB has quit IRC22:31
*** mriedem is now known as mriedem_afk22:36
*** kumarmn has joined #openstack-release22:37
*** bobh has quit IRC22:40
*** bobh has joined #openstack-release22:42
*** edmondsw has joined #openstack-release22:43
*** d0ugal has quit IRC22:43
*** yamamoto has joined #openstack-release22:46
*** edmondsw has quit IRC22:48
*** d0ugal has joined #openstack-release22:48
*** yamamoto has quit IRC22:50
*** yamamoto has joined #openstack-release22:51
*** hongbin has quit IRC22:53
*** yamamoto has quit IRC22:56
*** yamamoto has joined #openstack-release22:57
*** kumarmn has quit IRC23:01
*** yamamoto has quit IRC23:01
*** yamamoto has joined #openstack-release23:03
*** sree has joined #openstack-release23:11
*** sree has quit IRC23:16
*** kumarmn has joined #openstack-release23:21
*** kumarmn has quit IRC23:31
*** claudiub has quit IRC23:35
*** yamamoto has quit IRC23:47
*** kumarmn has joined #openstack-release23:51

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