Friday, 2018-11-02

*** phuongnh has joined #openstack-oslo00:48
*** mattgo has quit IRC01:08
*** njohnston has quit IRC01:27
*** phuongnh has quit IRC01:32
openstackgerritwangxiyuan proposed openstack/oslo.policy master: Add policy-upgrade tool  https://review.openstack.org/61390601:51
*** cfriesen has quit IRC01:55
openstackgerritwangxiyuan proposed openstack/oslo.policy master: Add policy-upgrade tool  https://review.openstack.org/61390601:57
*** hoangcx has joined #openstack-oslo02:05
*** bobh has joined #openstack-oslo02:37
*** phuongnh has joined #openstack-oslo02:53
*** bobh has quit IRC02:57
*** ducnv has joined #openstack-oslo03:00
openstackgerritMerged openstack/oslo.policy master: Unit test for CLI  https://review.openstack.org/61435603:18
*** chhagarw has joined #openstack-oslo03:42
*** chhavi__ has joined #openstack-oslo04:08
*** ducnv_ has joined #openstack-oslo04:08
*** ducnv has quit IRC04:09
*** chhagarw has quit IRC04:11
*** toabctl has quit IRC04:13
*** toabctl has joined #openstack-oslo04:14
*** threestrands has quit IRC05:40
*** phuongnh has quit IRC06:07
*** phuongnh has joined #openstack-oslo06:07
*** njohnston has joined #openstack-oslo06:25
openstackgerritVieri proposed openstack/oslo-specs master: Update min tox version to 2.0  https://review.openstack.org/61504807:01
openstackgerritVieri proposed openstack/osprofiler master: Update min tox version to 2.0  https://review.openstack.org/61505207:04
*** pcaruana has joined #openstack-oslo07:20
*** mattgo has joined #openstack-oslo07:29
openstackgerritVieri proposed openstack-dev/oslo-cookiecutter master: Update min tox version to 2.0  https://review.openstack.org/61508007:46
*** hoangcx has quit IRC07:49
*** hberaud has quit IRC08:11
*** hberaud has joined #openstack-oslo08:11
*** hberaud has quit IRC08:12
*** hberaud has joined #openstack-oslo08:12
*** slaweq has joined #openstack-oslo08:17
openstackgerritwangxiyuan proposed openstack/oslo.policy master: Add policy-upgrade tool  https://review.openstack.org/61390608:22
*** cdent has joined #openstack-oslo09:04
*** cdent has quit IRC09:06
*** cdent has joined #openstack-oslo09:10
*** e0ne has joined #openstack-oslo09:51
openstackgerritMerged openstack-dev/oslo-cookiecutter master: Update min tox version to 2.0  https://review.openstack.org/61508009:59
openstackgerritMerged openstack/oslo-specs master: Update min tox version to 2.0  https://review.openstack.org/61504810:00
*** phuongnh has quit IRC10:20
*** ducnv_ has quit IRC10:24
*** chhavi__ has quit IRC10:41
*** chhavi__ has joined #openstack-oslo10:41
*** stephenfin is now known as finucannot10:44
*** mattgo has quit IRC11:05
*** dave-mccowan has joined #openstack-oslo11:30
*** lpetrut has joined #openstack-oslo11:32
*** raildo has joined #openstack-oslo11:45
*** cdent has quit IRC12:06
*** pbourke has quit IRC12:10
*** pbourke has joined #openstack-oslo12:10
*** lpetrut has quit IRC12:13
openstackgerritJuan Antonio Osorio Robles proposed openstack/oslo.policy master: Add ability to pass in target data for the oslopolicy-checker  https://review.openstack.org/61331312:14
openstackgerritJuan Antonio Osorio Robles proposed openstack/oslo.policy master: WIP: Create OPA check  https://review.openstack.org/61422412:14
jaosoriorAny chance of getting this merged https://review.openstack.org/#/c/614223/3 ? :D12:14
openstackgerritJuan Antonio Osorio Robles proposed openstack/oslo.policy master: WIP: Create OPA check  https://review.openstack.org/61422412:31
jaosoriorthanks dhellmann!12:37
*** chhavi__ has quit IRC12:38
dhellmannjaosorior : np, thank *you*12:43
dhellmannjaosorior : will you be in Berlin for the Summit?12:52
*** kgiusti has joined #openstack-oslo12:53
jaosoriordhellmann: yes12:53
jaosoriordhellmann: will you?12:53
jaosoriordhellmann: this is finally a summit that is close to me :D12:53
dhellmannyes, are you interested in finding some time to sit down and talk about policy plugins?12:53
*** raildo has quit IRC12:53
dhellmannheh12:53
jaosoriordhellmann: sure!12:54
*** raildo has joined #openstack-oslo12:54
jaosoriordhellmann: So, I'm actually leaning more towards creating a custom check instead of a driver (like I had initiallty proposed)12:54
jaosoriorit's not great IMO, but it works12:54
dhellmanninteresting12:54
dhellmannI think there's value in doing the driver thing, I just think it's going to take a bit more thinking to get the API right12:55
jaosoriordhellmann: so, yeah, for Open Policy Agent, here is the WIP patch https://review.openstack.org/61422412:55
dhellmannmaybe we could meet for lunch one day12:55
jaosoriordhellmann: sure! I'm up for that. I'll be there all week12:55
dhellmannah, yeah, I have that starred12:55
dhellmannI'm in the board meeting all day monday, and tuesday is the contributor awards lunch, but I'm free wed and thu12:56
jaosoriordhellmann: both days work for me. Tuesday is my busiest day, and after that I'm free too.12:56
dhellmanncool, how about we say wed then and if something happens we can fall back to thurs12:56
dhellmannI'll put it on my calendar to block out the time, and we can figure out a good meeting place once we're on site12:57
jaosoriorlets do that then12:57
dhellmannI'll set up a meeting invitation12:58
*** bobh has joined #openstack-oslo12:59
dhellmannjaosorior : is there anyone else coming that we should include?13:01
jaosoriordhellmann: Moisés Guimaraes is coming as well.13:01
dhellmanncool, I'll add him13:01
jaosoriorlbragstad: around?13:20
openstackgerritJuan Antonio Osorio Robles proposed openstack/oslo.policy master: WIP: Add test case for overridding both old and deprecated policy names  https://review.openstack.org/61518313:22
*** d0ugal has quit IRC13:40
*** lpetrut has joined #openstack-oslo13:42
dhellmannbnemec : what's the plan for the config mapping patch series? Some of the cleanup items are approved but the base isn't. Do we have enough of that ready to land that we can go ahead?13:45
dhellmannhttps://review.openstack.org/#/c/526314/3713:45
lbragstadjaosorior yup13:55
openstackgerritMerged openstack/oslo.policy master: Pass in policy name as part of the oslopolicy-check check call  https://review.openstack.org/61422314:06
jaosoriorlbragstad: so, I was reviewing your patch https://review.openstack.org/#/c/614195/ and found a weird case here: https://review.openstack.org/#/c/615183/   Do you think that's a valid case?14:09
bnemecdhellmann: There's one patch at the end of the series that I have not figured out yet: https://review.openstack.org/#/c/60306014:12
dhellmannyeah14:12
bnemecI think they combined a fix and a refactoring, but I'm still not sure exactly what was behaving incorrectly.14:12
lbragstadjaosorior you mean if an operator is overriding both the old and the new policy names?14:12
dhellmannthat code is really messy14:12
dhellmannI think they're using loops where dicts would work in several cases14:13
*** dansmith is now known as SteelyDan14:13
lbragstadjaosorior in your opinion, is the weird part that they have conflicting values?14:13
lbragstadjaosorior in that case, i think it would be reasonable for the code to assume the new override as opposed to the old override14:21
lbragstadespecially since the service relies on the new name instead of the old name14:21
*** lpetrut has quit IRC14:26
jaosoriorlbragstad: so, from what Icould tell, with that test, the valid policy does not get the overriden value. It gets the old one.14:27
*** d0ugal has joined #openstack-oslo14:27
lbragstadjaosorior does that test pass?14:28
openstackgerritSean McGinnis proposed openstack/oslo.privsep master: Use threads to process target function  https://review.openstack.org/59355614:29
lbragstadjaosorior huh - doesn't look like it14:30
lbragstadat least not when i run it locally14:30
*** lpetrut has joined #openstack-oslo14:30
openstackgerritMerged openstack/oslo.messaging stable/ocata: Revert "rabbit: Don't prefetch when batch_size is set"  https://review.openstack.org/56712114:34
jaosoriorlbragstad: it doesn't pass, which is why I poked you. I thought it should pass.14:37
*** lpetrut has quit IRC14:37
lbragstadi must be missing a case somewhere then14:37
lbragstadi would expect that test to pass, since that seems like the most reasonable behavior14:38
*** cfriesen has joined #openstack-oslo14:52
openstackgerritMerged openstack/osprofiler master: Update min tox version to 2.0  https://review.openstack.org/61505215:11
*** dave-mccowan has quit IRC15:25
*** dave-mccowan has joined #openstack-oslo15:26
*** sum12 has quit IRC15:46
*** sum12 has joined #openstack-oslo15:48
*** jaosorior has quit IRC15:58
*** bnemec is now known as beekneemech16:00
*** sum12 has quit IRC16:16
*** sum12 has joined #openstack-oslo16:16
*** pcaruana has quit IRC16:17
openstackgerritMerged openstack/oslo.config master: Document when sample_default is in use  https://review.openstack.org/60735316:24
*** e0ne has quit IRC16:33
openstackgerritChris Friesen proposed openstack/oslo.concurrency master: Add support for fair locks  https://review.openstack.org/47397316:40
*** cdent has joined #openstack-oslo17:29
openstackgerritHervé Beraud proposed openstack/oslo.service stable/rocky: Fix stop of loopingcall  https://review.openstack.org/61526317:50
openstackgerritHervé Beraud proposed openstack/oslo.service stable/rocky: Use eventlet Event for loopingcall events  https://review.openstack.org/61448917:53
openstackgerritBen Nemec proposed openstack/oslo.config master: Clean up grammar and spelling in driver docs  https://review.openstack.org/61526517:55
beekneemechHmm, oslo.upgradecheck doesn't appear in https://docs.openstack.org/stein/projects.html17:56
openstackgerritMerged openstack/oslo.service stable/rocky: Stop asserting on Eventlet internals  https://review.openstack.org/61459018:02
-openstackstatus- NOTICE: OpenStack infra's mirror nodes stopped accepting connections on ports 8080, 8081, and 8082. We will notify when this is fixed and jobs can be rechecked if they failed to communicate with a mirror on these ports.18:09
*** cdent has quit IRC18:24
*** pcaruana has joined #openstack-oslo18:27
*** e0ne has joined #openstack-oslo18:34
*** chhavi__ has joined #openstack-oslo18:34
*** chhavi__ has quit IRC18:39
-openstackstatus- NOTICE: The firewall situation with ports 8080, 8081, and 8082 on mirror nodes has been resolved. You can recheck jobs that have failed to communicate to the mirrors on those ports now.18:54
*** bobh has quit IRC19:04
*** e0ne has quit IRC19:05
*** pcaruana has quit IRC19:08
*** bobh has joined #openstack-oslo19:25
*** lpetrut has joined #openstack-oslo19:34
*** finucannot is now known as stephenfin19:35
openstackgerritMerged openstack/oslo.config master: Add support for looking in environment for config  https://review.openstack.org/58585019:58
*** kgiusti has left #openstack-oslo19:59
*** cdent has joined #openstack-oslo20:24
*** lpetrut has quit IRC20:35
*** bobh has quit IRC20:59
*** bobh has joined #openstack-oslo21:03
*** bobh has quit IRC21:09
*** cdent has quit IRC21:13
*** njohnston has quit IRC21:16
*** njohnston has joined #openstack-oslo21:17
*** raildo has quit IRC21:19
*** dave-mccowan has quit IRC21:35
openstackgerritMerged openstack/oslo.config master: Clean up grammar and spelling in driver docs  https://review.openstack.org/61526521:38
openstackgerritLance Bragstad proposed openstack/oslo.policy master: Make upgrades more robust with policy overrides  https://review.openstack.org/61419521:46
*** bobh has joined #openstack-oslo22:44
*** bobh has quit IRC22:50
*** ianychoi has quit IRC23:02
*** shardy_ has quit IRC23:18
*** shardy has quit IRC23:18
*** ianychoi has joined #openstack-oslo23:55

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