Monday, 2017-09-25

*** catintheroof has joined #openstack-oslo00:19
*** catintheroof has quit IRC01:37
*** catintheroof has joined #openstack-oslo01:38
*** yamamoto has joined #openstack-oslo01:54
*** catintheroof has quit IRC01:57
*** zhouyaguo has joined #openstack-oslo02:25
openstackgerritMerged openstack/oslo.db master: Add new foreign key utility function to utils  https://review.openstack.org/49224102:54
openstackgerritRajath Agasthya proposed openstack/oslo.messaging master: Fix default value of RPC dispatcher access_policy  https://review.openstack.org/50045603:25
*** chhavi has joined #openstack-oslo04:05
*** Dinesh_Bhor has joined #openstack-oslo04:14
*** links has joined #openstack-oslo04:25
*** yamamoto_ has joined #openstack-oslo06:36
*** yamamoto has quit IRC06:36
*** rcernin has joined #openstack-oslo06:41
*** pcaruana has joined #openstack-oslo07:06
*** yamamoto_ has quit IRC07:14
*** yamamoto has joined #openstack-oslo07:16
*** tesseract has joined #openstack-oslo07:23
*** chhavi has quit IRC07:28
*** yamamoto_ has joined #openstack-oslo07:33
*** yamamoto has quit IRC07:36
*** yamamoto_ has quit IRC07:41
*** ralonsoh has joined #openstack-oslo07:46
*** shardy has joined #openstack-oslo07:55
*** yamamoto has joined #openstack-oslo08:10
*** e0ne has joined #openstack-oslo08:25
*** mikal_ has joined #openstack-oslo08:30
*** mikal has quit IRC08:32
*** yamamoto has quit IRC08:32
*** finucannot is now known as stephenfin08:47
*** lpetrut has joined #openstack-oslo09:20
*** pbourke has quit IRC09:22
*** pbourke has joined #openstack-oslo09:23
*** jaosorior has joined #openstack-oslo09:28
*** yamamoto has joined #openstack-oslo09:40
*** chhavi has joined #openstack-oslo09:43
*** yamamoto has quit IRC10:00
*** nicolasbock has joined #openstack-oslo10:02
*** nicolasbock has quit IRC10:12
*** nicolasbock has joined #openstack-oslo10:24
*** gcb has joined #openstack-oslo10:33
*** zhouyaguo has quit IRC10:40
*** zhouyaguo has joined #openstack-oslo10:40
*** pblaho has joined #openstack-oslo10:51
*** gcb has quit IRC10:53
*** sdague has joined #openstack-oslo10:59
*** ps_jadhav has joined #openstack-oslo11:04
*** pooja_jadhav has quit IRC11:07
*** clayton has quit IRC11:25
*** clayton has joined #openstack-oslo11:30
*** yamamoto has joined #openstack-oslo11:30
*** raildo has joined #openstack-oslo11:37
*** pblaho has quit IRC11:44
*** ps_jadhav is now known as pooja_jadhav11:45
dimsdhellmann : good morning. is https://review.openstack.org/#/c/491783/ ok now?11:53
*** yamamoto has quit IRC12:00
*** eck`gone is now known as eck`12:02
*** pblaho has joined #openstack-oslo12:08
*** MVenesio has joined #openstack-oslo12:11
*** sambetts|afk is now known as sambetts12:13
*** yamamoto has joined #openstack-oslo12:18
*** yamamoto has quit IRC12:33
*** ansmith has joined #openstack-oslo12:34
*** boden has joined #openstack-oslo12:36
*** gordc has joined #openstack-oslo12:40
openstackgerritmelissaml proposed openstack/oslo.utils master: Cleanup test-requirements  https://review.openstack.org/50708212:44
openstackgerritmelissaml proposed openstack/oslo.log master: Cleanup test-requirements  https://review.openstack.org/50708412:46
openstackgerritmelissaml proposed openstack/oslo.config master: Cleanup test-requirements  https://review.openstack.org/50708612:48
openstackgerritmelissaml proposed openstack/oslo.rootwrap master: Cleanup test-requirements  https://review.openstack.org/50708812:50
openstackgerritmelissaml proposed openstack/oslo.vmware master: Cleanup test-requirements  https://review.openstack.org/50708912:50
*** yamamoto has joined #openstack-oslo12:52
*** janzian has joined #openstack-oslo12:52
*** kgiusti has joined #openstack-oslo12:59
*** gcb has joined #openstack-oslo13:10
*** links has quit IRC13:23
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: External Policy hook should support SSL  https://review.openstack.org/49178313:25
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: [WIP] Make http/https check rules as stevedore extensions  https://review.openstack.org/50709813:25
*** lbragstad has joined #openstack-oslo13:28
*** dave-mccowan has joined #openstack-oslo13:41
openstackgerritDirk Mueller proposed openstack/oslo.db master: Drop MySQL-python dependency from oslo.db  https://review.openstack.org/50583513:45
*** chhavi has quit IRC13:46
*** zxy has joined #openstack-oslo13:49
*** zxy has quit IRC13:49
*** zxy has joined #openstack-oslo13:50
openstackgerritThomas Bechtold proposed openstack/stevedore master: Remove duplicate optional requirement  https://review.openstack.org/50710813:51
openstackgerritThomas Bechtold proposed openstack/stevedore master: Move reno to optional docs requirements  https://review.openstack.org/50710913:52
gcbweekly meeting in 3 minutes at #openstack-meeting-313:57
*** beekneemech is now known as bnemec14:02
*** superdan is now known as dansmith14:02
*** haypo has joined #openstack-oslo14:09
haypodhellmann, gcb : would you mind to look again to my JSON to_primitive() minor enhancement? https://review.openstack.org/#/c/503993/ it doesn't change the default behaviour, it only adds a small parameter to fix oslo.log JSON Formatter14:11
dhellmannhaypo : we're in the meeting in #openstack-meeting-3, want to join us and mention it there?14:11
haypodhellmann: i joined the channel14:12
openstackgerritVictor Stinner proposed openstack/oslo.log master: JSONFormatter convert unserializable with repr()  https://review.openstack.org/50400414:20
openstackgerritVictor Stinner proposed openstack/oslo.serialization master: jsonutils.to_primitive(): add fallback parameter  https://review.openstack.org/50399314:22
*** thomas__ has joined #openstack-oslo14:41
thomas__ping dhellmann :)14:41
dhellmannhi, thomas__14:42
dhellmannsquashing those 2 commits together isn't complicated, but if a problem comes up I thought it would be better to be here to talk about it14:42
dhellmannto start, you'll want to make sure you have the latest version of the master branch of oslo.policy. something like:14:43
dhellmanngit checkout master && git pull14:43
dhellmannthen you want to check out a copy of my patch with : git review -d 50678514:43
dhellmannthat should create a branch with a name starting with "review"14:44
dhellmannthe branch will include my patch as the most recent patch, and your patch as its parent14:44
thomas__how do you obtain that number 506785 ?14:44
dhellmannthat is the review number, from the URL for my patch14:44
dhellmannhttps://review.openstack.org/#/c/506785/14:45
thomas__OK14:45
dhellmannafter you have that branch, you rebase it on master with "git rebase -i master"14:45
dhellmannthat should open your editor, and list both patches14:45
dhellmannchange the "pick" instruction to "s" for squash, save the buffer, and quit your editor14:45
*** takedakn has joined #openstack-oslo14:46
dhellmanngit should then ask you to modify the commit message in your editor and it should fill in the buffer with both of the existing messages14:46
dhellmannmake sure you save the Change-Id line from *your* patch and *remove* the one from my patch14:46
dhellmannthen update the rest of the text of the message, save, and exit14:47
dhellmannthen resubmit the patch with "git review" like you normally would14:47
dhellmannwhen you've done that, I will abandon my patch14:47
*** ihrachys has joined #openstack-oslo14:48
thomas__when I write "s" and quit my editor, it says "Cannot 'squash' without a previous commit"14:48
dhellmannleave "pick" on your patch and change it to "s" on my patch14:49
dhellmannsorry, I wasn't clear14:49
openstackgerritMerged openstack-dev/cookiecutter master: Switch to use stestr as the default test runner  https://review.openstack.org/50577214:51
dhellmannthomas__ : I need to step away for a minute, but I will be right back.14:53
*** gcb has quit IRC14:55
dhellmannback14:57
thomas__dhellmann: in my editor I have to write something like this ?15:01
thomas__pick 0a11ae8 make it easier to reuse the invocation logic for check objects \nsquash15:01
dhellmannthis might be easier if we share screens so I can see what you're seeing -- I may have summarized what happens too much15:02
dhellmannlet me set up a thing for that, I will give you a URL in a minute15:02
*** zxy has quit IRC15:03
*** links has joined #openstack-oslo15:08
openstackgerritMerged openstack/oslo.context master: Rename deprecated context params  https://review.openstack.org/49830515:11
*** links has quit IRC15:17
*** rajathagasthya has joined #openstack-oslo15:17
openstackgerritMerged openstack/oslo.middleware master: Remove method set_latent  https://review.openstack.org/45372315:21
*** e0ne has quit IRC15:23
*** links has joined #openstack-oslo15:29
*** takedakn has quit IRC15:32
*** pcaruana has quit IRC15:33
openstackgerritThomas Duval proposed openstack/oslo.policy master: Modification to add additional information in the HTTPCheck request.  https://review.openstack.org/49846715:33
openstackgerritStephen Finucane proposed openstack-dev/pbr master: builddoc: Use '[sphinx_build] builders' with Sphinx < 1.6  https://review.openstack.org/48162015:33
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Deprecate support for Sphinx < 1.6  https://review.openstack.org/48162115:34
*** pcaruana has joined #openstack-oslo15:34
*** thomas__ has quit IRC15:36
*** aselius has joined #openstack-oslo15:44
*** rajathagasthya has quit IRC15:45
*** rcernin has quit IRC15:48
*** pooja_jadhav has quit IRC16:03
*** rajathagasthya has joined #openstack-oslo16:07
*** Dinesh_Bhor has quit IRC16:07
*** links has quit IRC16:10
openstackgerritMerged openstack/oslo.messaging master: Fix wrong transport warnings in functional tests  https://review.openstack.org/50681316:13
*** pcaruana has quit IRC16:15
openstackgerritMerged openstack/stevedore master: Remove duplicate optional requirement  https://review.openstack.org/50710816:16
*** lpetrut has quit IRC16:16
*** chhavi has joined #openstack-oslo16:29
*** ralonsoh has quit IRC16:33
openstackgerritMerged openstack/stevedore master: Move reno to optional docs requirements  https://review.openstack.org/50710916:38
chhaviharlowja: have a query on taskflow, is there a way to discard the retry of the mainflow if there is a specific exception in one of the flow, rever method only handles the subflow, and still perform retries. The control does not comes to on_falure. Is there anything specific which needs to be taken care to handle this scenario.16:41
*** chhavi has quit IRC16:42
*** pblaho has quit IRC16:42
*** chhavi has joined #openstack-oslo16:43
*** chhavi has quit IRC16:43
*** lpetrut has joined #openstack-oslo17:00
*** harlowja has joined #openstack-oslo17:20
*** kgiusti has quit IRC17:27
*** kgiusti has joined #openstack-oslo17:30
*** tesseract has quit IRC17:53
*** ihrachys has quit IRC17:57
*** ihrachys has joined #openstack-oslo17:57
*** sambetts is now known as sambetts|afk18:07
openstackgerritDoug Hellmann proposed openstack/oslo.policy master: Modification to add additional information in the HTTPCheck request.  https://review.openstack.org/49846718:09
*** rcernin has joined #openstack-oslo18:11
*** lpetrut has quit IRC18:11
*** kgiusti has quit IRC18:14
*** pblaho has joined #openstack-oslo18:32
*** kgiusti has joined #openstack-oslo18:33
*** eck` has quit IRC18:34
*** raildo has quit IRC18:34
*** eck` has joined #openstack-oslo18:37
*** raildo has joined #openstack-oslo18:42
openstackgerritDoug Hellmann proposed openstack/oslo.serialization master: add more tests for serializing unserializable objects  https://review.openstack.org/50723318:45
*** dave-mccowan has quit IRC18:56
*** lpetrut has joined #openstack-oslo19:09
*** e0ne has joined #openstack-oslo19:12
*** lpetrut has quit IRC19:14
*** lifeless has quit IRC19:14
*** lifeless has joined #openstack-oslo19:15
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: External Policy hook should support SSL  https://review.openstack.org/49178319:22
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: [WIP] Make http/https check rules as stevedore extensions  https://review.openstack.org/50709819:22
*** e0ne has quit IRC19:22
*** dave-mccowan has joined #openstack-oslo19:24
dhellmanndims : what's ^^ for?19:28
dimsdhellmann : i was trying to see if we could let folks add their own rule checks like we added the HttpsCheck19:29
dimsexperiment19:30
dhellmanndims : deployers? because apps can already do that19:30
dhellmannbut yeah, I like the idea19:30
*** dave-mccowan has quit IRC19:32
*** dave-mccowan has joined #openstack-oslo19:33
dimsthanks! i mean someone writes a new rule check, publishes a library and deployers can just use that new check19:35
dimsdhellmann : i have a funny error in the docs run for https://review.openstack.org/#/c/491783/ will ping you when i see the CI go red or warning shows up19:36
dhellmanndims : ok. it works for me locally using "tox -e docs" on a checkout of your patch19:37
dimsdhellmann : ah i forgot to switch warning-is-error=1 back on before updating19:38
dimscan you please try with that? (setup.cfg)19:39
dhellmannthe setup.cfg I have shows warning-is-error=119:40
dhellmannoh, wait, I may be looking at the wrong patch19:40
dhellmannhmm, no, it's 49178319:41
dimssorry i i pasted the wrong one. https://review.openstack.org/#/c/507098/2/setup.cfg19:43
dhellmannok, trying that one19:43
dimshere's what i see with warning-as-error set to 0 - http://paste.openstack.org/show/621865/19:44
dhellmanndims : running sphinx-build directly gives me the traceback19:45
dhellmannlooking at _checks I see it calling _init_extensions() on import; which imports external; which imports _checks -- cycle19:45
dhellmannsince _init_extensions() is being run before the rest of the _checks module is initialized, external can't find what it needs19:46
dhellmanntry moving the extension registration stuff into the enforcer class?19:46
dhellmanni.e., do it at runtime19:46
dhellmannthat makes it easier to deal with mocking for tests anyway19:47
dimsack will try that19:47
dimsthanks dhellmann19:47
*** jdandrea_ has joined #openstack-oslo19:49
*** e0ne has joined #openstack-oslo19:55
*** prometheanfire has joined #openstack-oslo20:00
prometheanfirehas anyone looked into updating versionedobjects for the new iso8601? https://bugs.launchpad.net/cinder/+bug/171548620:01
openstackLaunchpad bug 1715486 in oslo.versionedobjects "py35 tests failing due to iso8601 module change" [Undecided,In progress] - Assigned to zhangyangyang (zhangyangyang)20:01
*** raildo has quit IRC20:03
*** rcernin has quit IRC20:11
*** eck` has quit IRC20:14
*** eck`gone has joined #openstack-oslo20:17
*** eck`gone is now known as eck`20:17
*** e0ne has quit IRC20:18
openstackgerritVictor Stinner proposed openstack/oslo.serialization master: jsonutils.to_primitive(): add fallback parameter  https://review.openstack.org/50399320:23
*** kgiusti has left #openstack-oslo20:29
*** nicolasbock has quit IRC20:35
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: [WIP] Make http/https check rules as stevedore extensions  https://review.openstack.org/50709820:39
*** ansmith has quit IRC20:50
-openstackstatus- NOTICE: The infra team is bringing Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information.20:50
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: External Policy hook should support SSL  https://review.openstack.org/49178321:26
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.policy master: [WIP] Make http/https check rules as stevedore extensions  https://review.openstack.org/50709821:26
*** jdandrea_ has quit IRC21:28
*** MVenesio has quit IRC21:37
*** MVenesio has joined #openstack-oslo21:37
*** MVenesio has quit IRC21:41
*** dave-mccowan has quit IRC21:43
*** shardy has quit IRC21:47
*** janzian has quit IRC21:50
*** john51 has joined #openstack-oslo21:59
*** john51_ has quit IRC22:01
*** gordc has quit IRC22:17
*** sai has quit IRC22:25
*** sai has joined #openstack-oslo22:25
*** dave-mccowan has joined #openstack-oslo22:44
*** dave-mcc_ has joined #openstack-oslo23:05
*** dave-mccowan has quit IRC23:07
*** ansmith has joined #openstack-oslo23:13
*** sdague has quit IRC23:39
*** rajathagasthya has quit IRC23:46
openstackgerritMerged openstack/oslo.versionedobjects master: Add a TimestampedObject mixin  https://review.openstack.org/49591323:47

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