Thursday, 2017-05-25

*** mingyu has quit IRC00:11
*** NM has joined #openstack-swift00:13
*** Sukhdev_ has quit IRC00:16
*** alenavolk has quit IRC00:48
*** klrmn has quit IRC01:12
*** _JZ__ has joined #openstack-swift01:14
*** jeffli has joined #openstack-swift01:16
*** _JZ_ has quit IRC01:18
*** links has joined #openstack-swift01:30
claygi'm going to look at patch 44824001:36
patchbothttps://review.openstack.org/#/c/448240/ - swift - Enable per policy proxy config options01:36
*** zhengyin has joined #openstack-swift01:41
zaitcevhttp://paste.openstack.org/raw/610610/01:42
zaitcevyou've got to be kidding01:42
*** chsc has joined #openstack-swift01:43
*** tovin07_ has joined #openstack-swift01:51
*** JimCheung has quit IRC01:52
claygkota_: do we have a patch to fixup http://saio:8080/v1/AUTH_test/doc/overview_erasure_code.html#global-ec01:52
*** JimCheung has joined #openstack-swift01:52
claygoh sorry :\01:52
clayghttps://docs.openstack.org/developer/swift/overview_erasure_code.html#global-ec how bout (cc kota_ )01:52
claygredbo: can you help out zaitcev ?01:54
*** JimCheung has quit IRC01:56
redboit probably needs go 1.801:58
zaitcevgolang-1.7.5-2.fc25.x86_6401:58
zaitcevBut... why? This used to work on 1.7 and the code didn't change since you wrote it in 2014.01:59
zaitcevIt's this construct01:59
zaitcevtype HummingbirdServer struct {01:59
zaitcevhttp.Server01:59
*** vint_bra has joined #openstack-swift01:59
redbohttp.Server grew a Shutdown method for gracefully shutting down in 1.802:00
zaitcevoh02:00
*** NM has quit IRC02:02
kota_back to online02:05
*** zhurong has joined #openstack-swift02:05
kota_clayg: not yet but good call02:06
*** notmyname has quit IRC02:08
*** klrmn has joined #openstack-swift02:09
claygnice work redbo!02:12
jeffliquit02:12
*** jeffli has quit IRC02:12
*** notmyname has joined #openstack-swift02:13
*** ChanServ sets mode: +v notmyname02:13
*** alenavolk has joined #openstack-swift02:32
*** lucasxu has joined #openstack-swift02:33
*** itlinux has joined #openstack-swift02:37
*** itlinux has quit IRC03:03
*** lucasxu has quit IRC03:06
*** lucasxu has joined #openstack-swift03:08
*** _JZ__ has quit IRC03:09
zaitcev        Error:          Expected nil, but got: &errors.errorString{s:"Error setting up logger: Unable to create logger: open /var/log/swift/accountreplicator.log: no such file or directory"}03:12
zaitcev(no extra help needed, I'm just amazed that apparently everyone runs "go test -cover ./..." as root nowadays)03:12
*** alenavolk has quit IRC03:15
*** lucasxu has quit IRC03:18
claygzaitcev: in a container - you're always root03:18
*** vint_bra has quit IRC03:21
*** Sukhdev has joined #openstack-swift03:32
claygomg I should run my saio with write_affinity *all the time* i *love* handoffs!03:35
claygand replication 😋03:35
claygOMG I have *three* different timestamps for my EC object right now - and *all* of my servers are running - why did you guys not tell me how much fun this wAS!03:39
*** kei_yama has quit IRC04:00
*** kei_yama has joined #openstack-swift04:01
*** psachin has joined #openstack-swift04:05
*** klrmn has quit IRC04:05
*** zhurong has quit IRC04:08
*** Sukhdev has quit IRC04:11
claygOMG i was in less and then something happened and I was in nano!?04:11
claygs'ok - I used stackoverflow to figure out how to exit - https://stackoverflow.blog/2017/05/23/stack-overflow-helping-one-million-developers-exit-vim/04:11
*** zhurong has joined #openstack-swift04:16
*** winggundamth has joined #openstack-swift04:27
*** alenavolk has joined #openstack-swift04:34
*** klrmn has joined #openstack-swift04:54
*** gyee has quit IRC05:03
*** chsc has quit IRC05:07
*** klrmn has quit IRC05:07
*** zhengyin has quit IRC05:11
*** zhengyin has joined #openstack-swift05:11
*** skudlik has joined #openstack-swift05:20
claygacoles: kota_: I spent the afternoon with patch 448240 - I +A'd it05:21
patchbothttps://review.openstack.org/#/c/448240/ - swift - Enable per policy proxy config options05:21
claygnice work!05:22
kota_clayg: !05:22
clayg:\05:22
claygdid i miss something!?05:22
kota_clayg: no no, much appreciated05:22
claygoh, my pleasure - the thanks goes to you two (and timburke) - you all worked really hard on that feature05:23
claygand that one guy that did that one patch way back in the day?  where'd he go?  Thanks to him too05:23
*** _JZ_ has joined #openstack-swift05:23
kota_clayg: but I'm also on that patch and maybe less than 400 Lines rest to get the end of the patch :P05:23
claygOH NO!05:23
kota_but currently i didn't find issues to prevent the merge05:24
claygkota_: http://eavesdrop.openstack.org/irclogs/%23openstack-swift/latest.log.html#t2017-05-25T01:36:4005:24
clayg:\05:24
kota_i just am willing to understand whole even it was merged05:24
claygexcellent!05:25
claygi'm going to need to take off05:25
kota_yeah, it's your late time05:25
kota_probably some of my comments will be included into timburke's follow up05:26
*** cshastri has joined #openstack-swift05:36
*** NareshA has joined #openstack-swift05:46
NareshAHi folks05:46
NareshAswift stat giving 503 service unavailable error05:47
*** alenavolk has quit IRC05:48
NareshAwhich is the right place I can get instant help to debug my issue?06:04
*** Dinesh_Bhor has quit IRC06:06
*** Dinesh_Bhor has joined #openstack-swift06:11
*** rcernin has joined #openstack-swift06:18
*** pcaruana has joined #openstack-swift06:20
openstackgerritMerged openstack/swift master: Enable per policy proxy config options  https://review.openstack.org/44824006:34
*** wer has quit IRC06:44
*** _JZ_ has quit IRC07:16
*** oshritf has joined #openstack-swift07:29
*** zhurong has quit IRC07:30
*** oshritf has quit IRC07:32
*** oshritf has joined #openstack-swift07:33
*** oshritf has quit IRC07:36
*** oshritf has joined #openstack-swift07:39
*** zhurong has joined #openstack-swift07:45
openstackgerritKota Tsuyuzaki proposed openstack/swift master: Follow up for affinity config per policy  https://review.openstack.org/46792007:49
kota_timburke: perhaps, https://review.openstack.org/467920 can be picked to your follow up. That is for minor things I found in the reviews.07:54
patchbotpatch 467920 - swift - Follow up for affinity config per policy07:54
*** oshritf has quit IRC07:56
acolesgood morning07:57
*** oshritf has joined #openstack-swift08:03
*** oshritf has quit IRC08:05
acolesNareshA: start by looking in the proxy server logs08:10
*** sheel has joined #openstack-swift08:13
NareshAacoles: thank you. I figured out the error. Its working now.08:15
acolesNareshA: great!08:15
*** oshritf has joined #openstack-swift08:16
*** NareshA has quit IRC08:16
*** oshritf has quit IRC08:18
*** joeljwright has joined #openstack-swift08:33
*** ChanServ sets mode: +v joeljwright08:33
*** oshritf has joined #openstack-swift08:38
*** oshritf has quit IRC08:41
*** zhurong has quit IRC08:54
*** zhurong has joined #openstack-swift09:04
*** oshritf has joined #openstack-swift09:13
openstackgerritMerged openstack/swift master: Clean up how PatchPolicies works  https://review.openstack.org/46036209:20
*** zhurong has quit IRC09:35
*** mvk_ has quit IRC09:41
*** adriant has quit IRC09:58
*** tovin07_ has quit IRC10:00
*** mvk has joined #openstack-swift10:09
*** sheel has quit IRC10:22
*** mingyu has joined #openstack-swift10:48
openstackgerritAlistair Coles proposed openstack/swift master: Add link from policies overview to per-policy proxy-server conf  https://review.openstack.org/46799410:49
*** mingyu has quit IRC10:53
*** kei_yama has quit IRC11:23
*** zhengyin has quit IRC11:36
openstackgerritAlistair Coles proposed openstack/swift master: Update Global EC docs with reference to composite rings  https://review.openstack.org/46801111:48
acolestimburke: I looked over https://review.openstack.org/#/c/450491 and it all looks sane but I did have one question. If the answer is trivial then I would not object to the patch being squashed  with parent.12:08
patchbotpatch 450491 - swift - fixup! Add container sharding to Swift containers12:08
*** NM has joined #openstack-swift12:22
*** esnyder has quit IRC12:52
*** links has quit IRC12:54
openstackgerritOpenStack Proposal Bot proposed openstack/swift master: Updated from global requirements  https://review.openstack.org/8873613:01
*** oshritf has quit IRC13:06
*** chlong has quit IRC13:06
*** oshritf has joined #openstack-swift13:09
*** klamath has joined #openstack-swift13:09
*** klamath has quit IRC13:09
*** klamath has joined #openstack-swift13:10
*** psachin has quit IRC13:13
*** oshritf has quit IRC13:14
*** lucasxu has joined #openstack-swift13:16
*** lucasxu has quit IRC13:27
*** lucasxu has joined #openstack-swift13:27
*** lucasxu has quit IRC13:33
*** lucasxu has joined #openstack-swift13:34
*** oshritf has joined #openstack-swift13:40
*** lucasxu has quit IRC13:43
*** lucasxu has joined #openstack-swift13:43
*** oshritf has quit IRC13:49
*** oshritf has joined #openstack-swift13:53
*** oshritf has quit IRC14:14
*** _JZ_ has joined #openstack-swift14:14
*** oshritf has joined #openstack-swift14:16
*** oshritf has quit IRC14:23
*** oshritf has joined #openstack-swift14:25
*** chlong has joined #openstack-swift14:27
*** vint_bra has joined #openstack-swift14:27
*** lucasxu has quit IRC14:39
*** kestrels has joined #openstack-swift14:40
*** gyee has joined #openstack-swift14:50
*** lucasxu has joined #openstack-swift14:56
*** NM has left #openstack-swift14:59
*** cshastri has quit IRC15:06
openstackgerritTim Burke proposed openstack/swift master: fixup! Replace slowdown option with *_per_second option  https://review.openstack.org/46780115:19
openstackgerritTim Burke proposed openstack/swift master: Log deprecation warnings for run_pause  https://review.openstack.org/46806515:27
*** _JZ_ has quit IRC15:28
*** pcaruana has quit IRC15:38
*** pcaruana has joined #openstack-swift15:39
notmynamegood morning15:39
notmynamehttps://review.openstack.org/#/c/448240/15:42
patchbotpatch 448240 - swift - Enable per policy proxy config options (MERGED)15:42
notmynameit landed!15:42
notmynamethanks kota_ clayg timburke and acoles15:42
*** jrichli has quit IRC15:47
*** jrichli has joined #openstack-swift15:49
*** joeljwright1 has joined #openstack-swift15:50
*** joeljwright has quit IRC15:53
*** oshritf has quit IRC15:54
openstackgerritTim Burke proposed openstack/swift master: Log deprecation warnings for account_white/blacklist  https://review.openstack.org/46807715:54
*** joeljwright1 has quit IRC15:56
*** lucasxu has quit IRC16:03
*** lucasxu has joined #openstack-swift16:03
mwheckmannhello. Can someone elaborate on what the "post_quorum_timeout" does exactly? The doc isn't super clear. Specifically, I'm interested in the effect of quorum and response time back to clients.16:04
mwheckmannfor writes especially16:04
*** alenavolk has joined #openstack-swift16:08
*** chsc has joined #openstack-swift16:09
*** chsc has joined #openstack-swift16:09
openstackgerritTim Burke proposed openstack/swift master: Log deprecation warning for allow_versions  https://review.openstack.org/46808416:11
mwheckmannwhile I'm at asking questions, here's another one: when using write_affinity, a DELETE verb on an object will return a 404 if said object has been replicated to all it's primaries yet. I'm getting end users who are complaining about this behaviour. I get that it's likely returning the 404 since it hasn't found the object in all it's primary locations, but is there anything that can be done16:12
mwheckmannabout that? Of course, the delete actually still works in the eventually consistent sense: The obect dissapears from the container listing but is still GET'able until the handoffs have been cleared16:13
mwheckmannof course I meant if "..said object *hasn't* been replicated.."16:13
acolesmwheckmann: re post_quorum_timeout, assume the PUT is for a 3 replicas policy, IIRC once 2 replicas have been successfully PUT, we have a quorum and could return success to client right then. But then the 3rd replica write would get terminated, so instead the proxy waits a little longer to see if the 3rd replica write completes. But if the 3rd write is really slow then the proxy returns to client after post_quorum_timeout16:15
acoles.16:15
claygI'm not really sure how the last laggard request is closed/cleaned up.16:18
acolesmaybe 'terminated' was wrong word - the slow write's response wouldn't be considered in evaluating the client response16:20
mwheckmannacoles, clayg:hmm. interesting. I currently have post_quorum_timeout set to 0.5 sec and my experience with it is that it always waits for all replicas to finish. This is when I tested on a 6 replica globally replicated  ring (3 regions) and when combined with write_affinity + write_affinity_node_count=3 it seems to always wait for all remote PUTs to finish. i.e since quorum for 6 replicas = 3,16:20
mwheckmannI would expect it to simple write to 3 local locations and that's it. Replicatos would handle the rest.16:20
*** _JZ_ has joined #openstack-swift16:20
mwheckmannbasically I want to try to limit to 1 single local handoff location instead of 4 handoffs. (re: the issue I'm having with DELETEs).16:22
*** alenavolk has quit IRC16:26
*** pcaruana has quit IRC16:30
*** JimCheung has joined #openstack-swift16:30
*** pcaruana has joined #openstack-swift16:30
*** alenavolk has joined #openstack-swift16:31
openstackgerritTim Burke proposed openstack/swift master: Remove deprecated vm_test_mode option  https://review.openstack.org/46809916:36
*** chsc has quit IRC16:37
*** alenavolk has quit IRC16:38
*** rcernin has quit IRC16:40
openstackgerritTim Burke proposed openstack/swift master: Require that known-bad EC schemes be deprecated  https://review.openstack.org/46810516:56
timburkeso much house-cleaning this morning!16:56
*** mvk has quit IRC16:57
tdasilvatimburke: great! thank you!!17:01
claygtimburke: can you patch the vm_test_mode param in http://programmerthoughts.com/openstack/swift-on-pi/ as well - i know it came up recently on the ML as a document that was being used to guide new user/contributors through that configuration.  ;)17:11
timburkeclayg: i think i know a guy that can help with that17:13
claygok, i'll try to do some googles and track down the rest of them for you - but think of how much better swift will be once we don't have that deprecation warning in there anymore!17:14
claygsorry forgot the ;)17:14
timburkeclayg: think of how much better swift will be once we don't have this deprecation warning: https://github.com/openstack/swift/blob/2.14.0/swift/common/middleware/copy.py#L278-L28317:15
timburkeif we establish a history of not following through on deprecations, we'll *never* be able to remove them17:16
tdasilvatimburke: when are we removing that one???? :D17:17
timburketdasilva: i've got my calendar reminder set for 2018-02-2617:18
tdasilvawe should have asked operators last summit17:18
timburkestarted warning in ocata (2.13.0), when that's eol, i won't have any qualms17:19
tdasilvaI hope ops show up at PTG, otherwise I have a feeling we won't see them until Vancouver17:20
*** lucasxu has quit IRC17:20
*** Renich has joined #openstack-swift17:20
*** klrmn has joined #openstack-swift17:28
*** chsc has joined #openstack-swift17:36
*** chsc has joined #openstack-swift17:36
openstackgerritTim Burke proposed openstack/swift master: Require that known-bad EC schemes be deprecated  https://review.openstack.org/46810517:42
*** oshritf has joined #openstack-swift17:58
timburkeclayg: happy now? https://github.com/notmyname/swift_on_pi/pull/317:59
claygtrying to make clayg is non-goal - at best you can aim to make him "less unhappy" - but it's a looser's game18:00
*** oshritf has quit IRC18:07
tdasilvaclayg, kota_, timburke, notmyname we haven't done a libec+pyeclib release in a while and there's some good stuff there since last release, wdyt???18:22
*** lucasxu has joined #openstack-swift18:25
openstackgerritMerged openstack/liberasurecode master: Allow to disable optimizations for portability  https://review.openstack.org/46776118:28
*** JimCheung has quit IRC18:30
clayg@tdasilva love it!  new libec+pyeclib let's do it!18:31
openstackgerritThiago da Silva proposed openstack/pyeclib master: add six to test-requirements  https://review.openstack.org/46812518:32
tdasilvaclayg: looking through outstanding patches...18:32
clayg@timburke jokes aside - you're right about "neglected and forgotten" being a bad deprecation "strategy" for things like post-as-copy - for things like the isa-l configs I'm *incredibly* happy to see better maintainers than I keeping their eye on the ball - we *really should* try to push operators that may be unaware to do the right thing.  I think a ML post18:34
claygto openstack-operators about that change is *also* a good idea - KUDOS18:34
claygfor stuff like vm_test_mode or some other rando config option with a few lines of compatibility shim - meh - #1 bigger fish #2 you can't fix *all* the internets rando tutorials - it's not unreasonable to find something someone wrote about say... mysql in the early 2000's and I can *still* copy and paste it into my linux machine word for word and *most* stuff18:36
claygstill works.18:36
claygI know "2 years EOL'd" *seems* like a long time now, but it's not...18:36
claygfor stuff that is mainly just a burden to maintainers - and esp if that burden isn't that high - I value the smooth experience of the user/operator much higher than my own personal comfort18:37
clayg$0.0218:38
*** aselius has joined #openstack-swift18:40
*** Sukhdev has joined #openstack-swift18:43
mwheckmann18:55
*** pcaruana has quit IRC19:09
*** alenavolk has joined #openstack-swift19:10
*** mvk has joined #openstack-swift19:11
openstackgerritTim Burke proposed openstack/swift master: Remove tempauth docs from deployment guide  https://review.openstack.org/46695319:35
*** jamielennox is now known as jamielennox|away19:56
*** Sukhdev has quit IRC19:57
openstackgerritTim Burke proposed openstack/swift master: Remove deprecated vm_test_mode option  https://review.openstack.org/46809920:03
*** alenavolk has quit IRC20:04
*** chlong has quit IRC20:15
*** lucasxu has quit IRC20:21
*** lucasxu has joined #openstack-swift20:22
*** chlong has joined #openstack-swift20:33
*** Renich has quit IRC20:36
*** chlong has quit IRC20:37
*** chlong has joined #openstack-swift20:38
*** alenavolk has joined #openstack-swift20:40
*** skudlik has quit IRC20:40
*** vint_bra has quit IRC21:01
*** vint_bra has joined #openstack-swift21:02
*** itlinux has joined #openstack-swift21:05
*** chlong has quit IRC21:11
openstackgerritTim Burke proposed openstack/swift master: Improve time_check output  https://review.openstack.org/46816821:20
*** kestrels has quit IRC21:27
claygtimburke: I +A'd patch 466953 - is that cool?!21:36
patchbothttps://review.openstack.org/#/c/466953/ - swift - Remove tempauth docs from deployment guide21:36
timburkelove it!21:36
timburkethanks clayg :-)21:36
claygzaitcev: maybe I should have asked you first :\21:36
notmynametimburke: thanks for updating the priority reviews page21:37
notmynamehttps://wiki.openstack.org/wiki/Swift/PriorityReviews <<-- has links to the current global ec follow-up patches21:37
*** alenavolk has quit IRC21:37
*** alenavolk has joined #openstack-swift21:38
*** alenavolk has quit IRC21:38
*** lucasxu has quit IRC21:39
*** itlinux has quit IRC21:42
mattoliverauMorning, I'm back from a short vacation, sorry I missed the meeting yesterday21:54
timburkegood morning mattoliverau!21:54
zaitcevclayg: it's fine, we still have etc/proxy-server.conf-sample, so we do not totally lose the documentation.22:07
claygzaitcev: yeah totally - it was a copy paste not a cut22:07
*** itlinux has joined #openstack-swift22:22
notmynamehow are we supposed to model durability of global ec policies? if you've got 5+3 with 2x replication, how does the 2x replication factor into the durability calculation? it feels wrong that it would be equivalent to 10+6, but I don't know22:25
timburkenotmyname: i've previously played around with a probabilistic approach in https://gist.github.com/tipabu/b614587d2e978df8438c0250cf353ebc -- if you don't care about timings (as i did when looking at duplication vs increased parities) and are happy to assume that `k` unique frags => can reconstruct, you can probably simplify it a bit, not even make an ECDriver, and speed it up a whole lot in the process22:32
notmynameinteresting22:33
timburkethen i used https://gist.github.com/tipabu/454d8859d997110ba0acbca36ab6b7ec to make graphs of # failures across the cluster vs. P[can_reconstruct]22:34
notmynamekota_ and cschwede_ had worked on http://notmyname.github.io/swift-durability-calculator/ previously (that's my fork of charz's for of kota_'s fork of cschwede_'s fork)22:34
timburkeyou could tweak it to have the y axis be log-scale, to simplify the "how many 9s" question22:35
*** jamielennox|away is now known as jamielennox22:36
*** vint_bra has quit IRC22:36
timburkeyeah, i debated about a more-pure-math approach, but figured trying to properly account for duplicates would leave me much less satisfied with my correctness22:38
*** itlinux has quit IRC23:07
notmynamewhat's quorum size on a replicated ec policy?23:27
timburkenotmyname: n_data * duplication_factor https://github.com/openstack/swift/blob/master/swift/common/storage_policy.py#L60723:30
notmynamethanks23:30
timburkethough i'm not really sure that's a *good* choice...23:31
*** chsc has quit IRC23:32
mattoliverauI thought you'd make it the same as it means things will still work in a region split brain scenario.. but I havent really thought about it hard yet cause no coffee ;)23:32
timburkeer, (n_data + some per-ec-scheme value) * dup_factor23:33
notmynameyeah, and are they unique fragments? or per region?23:33
timburkeyeah, that's what i'm thinking right now too... i think it was done to be defensive?23:33
notmynameor?23:33
mattoliverauOr do we only care about access is split brain not continuing to put23:33
timburkeas i recall, the putter will preferentially send out unique frags, at least to handoffs23:34
timburkeie if both primaries for frag 1 are down and only one of the ones for frag 2, frag 1 is *definitely* getting prioritized for handoffs23:35
*** _JZ_ has quit IRC23:43
*** alenavolk has joined #openstack-swift23:47
*** klamath has quit IRC23:48
*** alenavolk has quit IRC23:51

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