Friday, 2020-04-24

*** stephenfin has quit IRC00:37
*** stephenfin has joined #openstack-oslo00:43
*** hemna has quit IRC01:10
*** hemna has joined #openstack-oslo01:11
*** hemna has quit IRC01:50
*** hemna has joined #openstack-oslo01:50
*** hemna has quit IRC02:17
*** hemna has joined #openstack-oslo02:18
*** rcernin has quit IRC06:27
*** rcernin has joined #openstack-oslo06:29
*** rpittau|afk is now known as rpittau07:17
*** tosky has joined #openstack-oslo07:33
*** rcernin has quit IRC07:35
*** ralonsoh has joined #openstack-oslo08:01
*** tkajinam has quit IRC08:24
*** yoctozepto has quit IRC10:09
openstackgerritHervé Beraud proposed openstack/oslo.serialization master: Add release notes links to doc index  https://review.opendev.org/72051510:10
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add 'openstackdocs_auto_version' config option  https://review.opendev.org/68268110:11
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add 'openstackdocs_auto_name' config option  https://review.opendev.org/68268210:11
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Merge, rather than overwrite, LaTeX settings  https://review.opendev.org/68268310:11
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add package prefixes for all config options  https://review.opendev.org/68268410:11
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: trivial: Add whitespace  https://review.opendev.org/68268510:11
*** rpittau is now known as rpittau|bbl10:12
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add 'openstackdocs_auto_version' config option, path checking  https://review.opendev.org/68268111:05
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add 'openstackdocs_auto_name' config option  https://review.opendev.org/68268211:05
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Merge, rather than overwrite, LaTeX settings  https://review.opendev.org/68268311:05
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add package prefixes for all config options  https://review.opendev.org/68268411:05
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: trivial: Add whitespace  https://review.opendev.org/68268511:05
*** AJaeger has joined #openstack-oslo11:42
AJaegerstephenfin, I propose to get https://review.opendev.org/719513 in for openstackdocstheme as well and then do a new release with your changes - perhaps directly after ussuri release?11:43
*** ansmith has joined #openstack-oslo11:50
*** rpittau|bbl is now known as rpittau12:13
*** raildo has joined #openstack-oslo12:20
*** kgiusti has joined #openstack-oslo12:26
stephenfinsounds good to me12:43
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add 'openstackdocs_auto_name' config option  https://review.opendev.org/68268212:48
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Merge, rather than overwrite, LaTeX settings  https://review.opendev.org/68268312:48
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: Add package prefixes for all config options  https://review.opendev.org/68268412:48
openstackgerritStephen Finucane proposed openstack/openstackdocstheme master: trivial: Add whitespace  https://review.opendev.org/68268512:48
AJaegerstephenfin: all green from my side ;)12:55
stephenfinAJaeger: Great, thanks!13:01
openstackgerritSean McGinnis proposed openstack/automaton master: Add py38 package metadata  https://review.opendev.org/72268713:24
openstackgerritSean McGinnis proposed openstack/futurist master: Add py38 package metadata  https://review.opendev.org/72269413:24
openstackgerritSean McGinnis proposed openstack/mox3 master: Add py38 package metadata  https://review.opendev.org/72270513:25
openstackgerritSean McGinnis proposed openstack/oslo.concurrency master: Add py38 package metadata  https://review.opendev.org/72271813:25
openstackgerritSean McGinnis proposed openstack/oslo.context master: Add py38 package metadata  https://review.opendev.org/72271913:25
openstackgerritSean McGinnis proposed openstack/oslo.db master: Add py38 package metadata  https://review.opendev.org/72272013:25
openstackgerritSean McGinnis proposed openstack/oslo.i18n master: Add py38 package metadata  https://review.opendev.org/72272113:25
openstackgerritSean McGinnis proposed openstack/oslo.log master: Add py38 package metadata  https://review.opendev.org/72272213:25
openstackgerritSean McGinnis proposed openstack/oslo.messaging master: Add py38 package metadata  https://review.opendev.org/72272313:25
openstackgerritSean McGinnis proposed openstack/oslo.middleware master: Add py38 package metadata  https://review.opendev.org/72272413:25
openstackgerritSean McGinnis proposed openstack/oslo.policy master: Add py38 package metadata  https://review.opendev.org/72272513:25
openstackgerritSean McGinnis proposed openstack/oslo.privsep master: Add py38 package metadata  https://review.opendev.org/72272613:25
openstackgerritSean McGinnis proposed openstack/oslo.reports master: Add py38 package metadata  https://review.opendev.org/72272713:25
openstackgerritSean McGinnis proposed openstack/oslo.rootwrap master: Add py38 package metadata  https://review.opendev.org/72272813:25
openstackgerritSean McGinnis proposed openstack/oslo.serialization master: Add py38 package metadata  https://review.opendev.org/72272913:25
openstackgerritSean McGinnis proposed openstack/oslo.service master: Add py38 package metadata  https://review.opendev.org/72273013:25
openstackgerritSean McGinnis proposed openstack/oslotest master: Add py38 package metadata  https://review.opendev.org/72273113:25
openstackgerritSean McGinnis proposed openstack/oslo.upgradecheck master: Add py38 package metadata  https://review.opendev.org/72273213:25
openstackgerritSean McGinnis proposed openstack/oslo.versionedobjects master: Add py38 package metadata  https://review.opendev.org/72273313:25
openstackgerritSean McGinnis proposed openstack/oslo.vmware master: Add py38 package metadata  https://review.opendev.org/72273413:26
openstackgerritSean McGinnis proposed openstack/osprofiler master: Add py38 package metadata  https://review.opendev.org/72273513:26
openstackgerritSean McGinnis proposed openstack/taskflow master: Add py38 package metadata  https://review.opendev.org/72277813:27
*** bnemec is now known as beekneemech13:54
*** iurygregory has quit IRC13:55
*** iurygregory has joined #openstack-oslo13:56
AJaegersmcginnis: do you have a script for that? ;)13:57
smcginnisJust an ad hoc combination of sed and some old fashioned bash. ;)14:01
AJaeger;)14:02
AJaegerThat's how I would have done it - was wondering whether you had something more sophisticated :)14:02
smcginnisNah. Key ingredient actually was beagle: https://pypi.org/project/beagle/14:04
smcginnisso beagle, grep, cut, sed, and some git. And duct tape.14:04
AJaegerwasn't aware of beagle...14:06
*** smcginnis has quit IRC14:08
*** smcginnis has joined #openstack-oslo14:10
*** moguimar has joined #openstack-oslo14:28
moguimaro/14:30
moguimarstephenfin, could you take a look on this topic later?14:30
moguimarhttps://review.opendev.org/#/q/topic:new-with-revert-and-fix+(status:open+OR+status:merged)14:30
stephenfinsure14:31
moguimarI'd like to help hberaud getting the pymemcache_pool backend merged14:31
moguimarI'm reviewing it right now14:31
hberaudthanks14:32
moguimarhberaud, did you keep the same options from memcache_pool to pymemcache_pool? Do I need to just change the backend and that's it?14:39
hberaudmoguimar: IIRC yep you just have to choose the right backend, like this => https://review.opendev.org/#/c/711220/34/.zuul.yaml14:40
hberaudI also added 2 new options => https://review.opendev.org/#/c/711220/34/oslo_cache/_opts.py14:41
moguimarI posted some comments on that file14:41
moguimarregarding to the oslo.config env values14:42
hberaudmoguimar: libraries name things differently so I prefered dedicated options to avoid confusing14:42
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding the new pymemcache_pool backend  https://review.opendev.org/71122014:50
moguimarhberaud, you have other oslo.config env values in that file, I wonder why is everything passing in the memcache_pool job14:51
moguimarmaybe it is due to default values14:52
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding functional tests  https://review.opendev.org/71247914:52
moguimarbut I don't know oslo.cache options that much yet14:52
hberaudI updated previous commit to add the missing "_" to other jobs14:52
hberauds/commit/commits14:53
moguimaryep14:53
moguimarjust saw it14:53
moguimarthanks14:53
moguimarhberaud, can you rebase https://review.opendev.org/#/c/711220 ?14:54
hberaudyep I'll14:54
moguimarack14:54
hberaudI've some things to adapt too14:54
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Refactoring functional tests based on standard usages  https://review.opendev.org/71822314:58
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding functional tests for memcache_pool backend  https://review.opendev.org/71817614:58
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding the new pymemcache_pool backend  https://review.opendev.org/71122014:58
hberaudmoguimar: done ^^14:58
moguimargreat!14:58
hberaudmoguimar: my main concern about the pymemcache backend is on how he manage dead servers compared to python-memcached backend15:01
hberaudmoguimar: python-memcached have public attributes which allow users to tweak things within the returned client15:03
hberaudmoguimar: especially the list of dead hosts15:03
moguimarby dead host you mean an unreachable memcached server, right?15:03
hberaudmoguimar: yep15:04
moguimarack15:04
hberaudmoguimar: and in oslo we use it to keep our backend running => https://github.com/openstack/oslo.cache/blob/master/oslo_cache/_memcache_pool.py#L24915:04
hberaudmoguimar: I mean we tweak these attribute to avoid to stop execution15:05
hberaudmoguimar: pymemcache manage this kind of thing differently and I'm not sure which pathway we could use to keep identical behavior between backends15:06
hberaudmoguimar: I did tests by swithching backends with => https://github.com/4383/oslo.labs and I observed different behavior on this point15:08
* moguimar didn't know about oslo.labs15:09
moguimarhberaud++15:09
hberaudmoguimar: in your futur functional tests we should surround this kind of things15:09
hberaudto ensure to keep the same behavior15:09
hberaudmoguimar: oslo.labs is a personal project to help me to debug things15:10
moguimarah nice15:11
moguimarI didn't check who was the 4383 xD15:11
hberaudmoguimar: to observe this behavior the test is easier... run 2-3 memcache servers, use oslo.cache with memcache_pool and now stop the 3 servers and observe results, after that you just have to do the same things with the new backend pymemcache_pool and you will see the difference during the execution15:13
hberauds/easier/easy/15:14
openstackgerritSean McGinnis proposed openstack/automaton master: Bump default tox env from py37 to py38  https://review.opendev.org/72282415:26
openstackgerritSean McGinnis proposed openstack/futurist master: Bump default tox env from py37 to py38  https://review.opendev.org/72282915:27
openstackgerritSean McGinnis proposed openstack/mox3 master: Bump default tox env from py37 to py38  https://review.opendev.org/72283815:28
openstackgerritSean McGinnis proposed openstack/oslo.concurrency master: Bump default tox env from py37 to py38  https://review.opendev.org/72285215:30
openstackgerritSean McGinnis proposed openstack/oslo.context master: Bump default tox env from py37 to py38  https://review.opendev.org/72285315:30
openstackgerritSean McGinnis proposed openstack/oslo.db master: Bump default tox env from py37 to py38  https://review.opendev.org/72285515:30
openstackgerritSean McGinnis proposed openstack/oslo.i18n master: Bump default tox env from py37 to py38  https://review.opendev.org/72285615:30
openstackgerritSean McGinnis proposed openstack/oslo.log master: Bump default tox env from py37 to py38  https://review.opendev.org/72285715:30
openstackgerritSean McGinnis proposed openstack/oslo.messaging master: Bump default tox env from py37 to py38  https://review.opendev.org/72285815:31
openstackgerritSean McGinnis proposed openstack/oslo.middleware master: Bump default tox env from py37 to py38  https://review.opendev.org/72285915:31
openstackgerritSean McGinnis proposed openstack/oslo.policy master: Bump default tox env from py37 to py38  https://review.opendev.org/72286015:31
openstackgerritSean McGinnis proposed openstack/oslo.privsep master: Bump default tox env from py37 to py38  https://review.opendev.org/72286115:31
openstackgerritSean McGinnis proposed openstack/oslo.reports master: Bump default tox env from py37 to py38  https://review.opendev.org/72286215:31
openstackgerritSean McGinnis proposed openstack/oslo.rootwrap master: Bump default tox env from py37 to py38  https://review.opendev.org/72286315:31
openstackgerritSean McGinnis proposed openstack/oslo.serialization master: Bump default tox env from py37 to py38  https://review.opendev.org/72286415:31
openstackgerritSean McGinnis proposed openstack/oslo.service master: Bump default tox env from py37 to py38  https://review.opendev.org/72286515:32
openstackgerritSean McGinnis proposed openstack/oslotest master: Bump default tox env from py37 to py38  https://review.opendev.org/72286615:32
openstackgerritSean McGinnis proposed openstack/oslo.upgradecheck master: Bump default tox env from py37 to py38  https://review.opendev.org/72286715:32
openstackgerritSean McGinnis proposed openstack/oslo.versionedobjects master: Bump default tox env from py37 to py38  https://review.opendev.org/72286815:32
openstackgerritSean McGinnis proposed openstack/oslo.vmware master: Bump default tox env from py37 to py38  https://review.opendev.org/72286915:32
openstackgerritSean McGinnis proposed openstack/osprofiler master: Bump default tox env from py37 to py38  https://review.opendev.org/72287015:32
AJaegersmcginnis: that change and the other one should be part of the release bot for "opening up victoria" ;)15:36
smcginnisI thought about that, but I think it would be a little trickier to automate.15:37
openstackgerritSean McGinnis proposed openstack/taskflow master: Bump default tox env from py37 to py38  https://review.opendev.org/72291015:37
smcginnisWe would need the script to be able to understand what python versions are in the release, where right now, we just need to look up the next release name and update the job template to use the new name.15:38
smcginnisBut maybe there can be a way to figure that out. I agree, it would be nice to make that automatic.15:38
AJaegerwould be nice indeed ;)15:39
smcginnisOtherwise someone needs to remember to do this every time we raise the max python version for a cycle.15:39
AJaegeryep15:39
smcginnisThis default tox target isn't as important, but the metadata piece needs to be done at some point for sure.15:39
AJaegeryes, for sure.15:39
AJaegerOr add a release test for victoria: Reject release, no py38 metadata ;)15:40
smcginnisHmmmm... :)15:40
* AJaeger is evil today :)15:40
smcginnisI like that idea, but I wonder if we can force that. I'll have to think about that.15:41
smcginnisThat could be how we make this all work. We can add a data file that captures the TC defined runtimes into a yaml file that can be read. Then using that data file, that can drive validation checks and the new branch automation.15:42
hberaud+1 to be evil :p15:42
hberaudit could be an huge achievement to proceed like this15:43
hberaudwe could save lot of time15:43
hberaudand it could help us to drive by priority/goal and reusable in each cycle15:44
smcginnis++15:46
smcginnisAdded notes to next release team meeting to discuss, if I don't get to it before then.15:47
*** rpittau is now known as rpittau|afk16:41
*** logan- has quit IRC17:30
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding functional tests  https://review.opendev.org/71247917:40
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Refactoring functional tests based on standard usages  https://review.opendev.org/71822317:40
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding functional tests for memcache_pool backend  https://review.opendev.org/71817617:40
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Adding the new pymemcache_pool backend  https://review.opendev.org/71122017:40
*** logan- has joined #openstack-oslo17:42
*** ralonsoh has quit IRC17:52
openstackgerritMerged openstack/futurist master: Add py38 package metadata  https://review.opendev.org/72269418:09
openstackgerritMerged openstack/osprofiler master: Add py38 package metadata  https://review.opendev.org/72273519:11
openstackgerritMerged openstack/osprofiler master: Bump default tox env from py37 to py38  https://review.opendev.org/72287019:11
*** ansmith_ has joined #openstack-oslo19:26
*** ansmith has quit IRC19:30
*** mgagne has joined #openstack-oslo19:52
*** toabctl has quit IRC20:00
-openstackstatus- NOTICE: The Zuul project gating service is reporting new patches in merge conflict erroneously due to a configuration error, fix in progress20:20
*** kgiusti has left #openstack-oslo21:05
*** raildo has quit IRC21:07
openstackgerritBen Nemec proposed openstack/oslo-specs master: Remove graduation-template.rst  https://review.opendev.org/72304221:49
openstackgerritBen Nemec proposed openstack/oslo-specs master: Add details about the different templates  https://review.opendev.org/72304321:49
openstackgerritBen Nemec proposed openstack/oslo-specs master: Add contributor doc policy  https://review.opendev.org/72304421:49
-openstackstatus- NOTICE: the This Zuul outage was taken as an opportunity to perform an impromptu maintenance for changing our service deployment model; any merge failures received from Zuul between 19:40 and 20:20 UTC were likely in error and those changes should be rechecked; any patches uploaded between 20:55 and 22:45 UTC were missed entirely by Zuul and should also be rechecked to get fresh test results22:57
*** tosky has quit IRC23:14
*** geguileo has quit IRC23:31

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