Thursday, 2017-01-05

*** janonymous has joined #openstack-swift00:10
*** tdasilva has quit IRC00:16
openstackgerritMatthew Treinish proposed openstack/swift: Use pbr console_script entrypoints for bin/ scripts  https://review.openstack.org/41677600:17
openstackgerritMatthew Treinish proposed openstack/swift: Use pbr console_script entrypoints for bin/ scripts  https://review.openstack.org/41677600:30
dimsmtreinish : ack thanks00:38
mtreinishdims: although timburke brought up a good point the scripts still get their shebang adjusted with setuptools in the current setup.cfg00:44
mtreinishthe reason it's failing for us now with py35 is because lib/swift calls SWIFT_DIR/bin/* manually instead of the installed ones00:44
*** manous has quit IRC00:55
*** jamielennox is now known as jamielennox|away01:02
*** winggundamth has joined #openstack-swift01:09
*** winggundamth has quit IRC01:17
*** jamielennox|away is now known as jamielennox01:21
*** winggundamth has joined #openstack-swift01:23
*** garyj has quit IRC01:24
*** furlongm has quit IRC01:36
*** furlongm has joined #openstack-swift01:36
*** bkopilov has quit IRC01:41
*** bkopilov has joined #openstack-swift01:52
* kota_ is back to online at his office01:59
*** klrmn has quit IRC02:05
*** vinurs has joined #openstack-swift02:06
*** JimCheung has left #openstack-swift02:08
*** tqtran has quit IRC02:31
*** trananhkma has quit IRC02:34
*** bkopilov has quit IRC03:04
*** tonyb has quit IRC03:05
*** treyd has quit IRC03:05
*** sgundur has quit IRC03:05
*** acoles_ has quit IRC03:05
*** timss has quit IRC03:05
*** mminesh has quit IRC03:05
*** charz has quit IRC03:05
*** verto has quit IRC03:05
*** andymccr has quit IRC03:05
*** lxsli has quit IRC03:05
*** mminesh` has joined #openstack-swift03:05
*** tonyb has joined #openstack-swift03:05
*** verto_ has joined #openstack-swift03:05
*** lxsli has joined #openstack-swift03:05
*** andymccr_ has joined #openstack-swift03:05
*** JimCheung has joined #openstack-swift03:05
*** timss has joined #openstack-swift03:05
*** mminesh` is now known as mminesh03:05
*** charz has joined #openstack-swift03:06
*** sgundur has joined #openstack-swift03:06
*** treyd has joined #openstack-swift03:06
*** acoles_ has joined #openstack-swift03:06
*** ChanServ sets mode: +v acoles_03:06
*** acoles_ is now known as acoles03:06
*** JimCheung has quit IRC03:10
jrichlianyone else only getting some of the email notifications for patches, but not all that they used to get?03:32
*** klrmn has joined #openstack-swift03:37
*** links has joined #openstack-swift03:40
*** Jeffrey4l has quit IRC04:06
*** Jeffrey4l has joined #openstack-swift04:07
*** tqtran has joined #openstack-swift04:27
*** psachin has joined #openstack-swift04:29
*** tqtran has quit IRC04:30
*** bkopilov has joined #openstack-swift04:30
*** furlongm has quit IRC04:35
*** furlongm has joined #openstack-swift04:36
*** ppai has joined #openstack-swift05:10
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873605:15
*** ksameersrk has joined #openstack-swift05:17
*** jamielennox is now known as jamielennox|away05:23
ksameersrkHi All, I was eager to know the data-flow of an object on Openstack Swift PUT command. I know a highlevel overview from various posts, which I am not sure of. Client Initiates PUT command specifying the object path on local storage to that on swift cloud. The Object is transferred to proxy-server over HTTP request. If (REPLICAS = 3), then three primary nodes are found out using Ring Algorithm. Then the object is transfer05:24
ksameersrk*from proxy server. Then after majority successful ACK (two here), the client is sent back ACK. I will be happy if someone can confirm the above sequence of steps is correct. My follow questions are What happens when there is NO ACK from the 3rd node? How is rsync used, when there is node failure?05:25
*** tqtran has joined #openstack-swift05:30
*** jamielennox|away is now known as jamielennox05:30
*** Jeffrey4l_ has joined #openstack-swift05:31
*** tqtran has quit IRC05:32
*** Jeffrey4l has quit IRC05:34
*** dfflanders has quit IRC05:41
*** SkyRocknRoll has joined #openstack-swift05:46
*** ksameersrk has quit IRC06:11
*** chsc has joined #openstack-swift06:17
*** chsc has quit IRC06:21
*** McMurlock1 has joined #openstack-swift06:52
*** pcaruana has joined #openstack-swift07:05
*** tesseract has joined #openstack-swift07:09
*** ChubYann has quit IRC07:16
*** rcernin has joined #openstack-swift07:17
*** hseipp has joined #openstack-swift07:42
*** openstackgerrit has quit IRC07:50
*** zhugaoxiao has quit IRC07:54
*** zhugaoxiao has joined #openstack-swift07:54
donaghksameersrk: your sequence is mostly correct. Except, if the obj node is down before you start the transaction, the proxy will notice and send the object to yet another node (called the handoff). So in that case you end up with 3 replicas on disk. Your sequence is correct when the obj node dies in the middle of the transaction.07:59
donaghWhen the node is restored/rebooted/fixed, the object replicators will use rsync to copy the data from one of the other replicas.08:02
donaghThis could be one of the primary copies or a handoff - does not matter .. the first to notice will fix up the missing replica08:03
donaghThe handoff node will then delete its copy once it knows the originally-down node has a good copy08:03
*** rledisez has joined #openstack-swift08:17
rledisezhello kota_, are you around?08:20
*** ianstanfield has quit IRC08:46
*** oshritf has joined #openstack-swift08:48
*** openstackgerrit has joined #openstack-swift08:53
openstackgerritChristian Schwede proposed openstack/swift: Add support to increase object ring partition power  https://review.openstack.org/33729708:53
*** cbartz has joined #openstack-swift09:07
*** asettle_ has joined #openstack-swift09:08
*** cbartz has quit IRC09:08
*** andymccr_ is now known as andymccr09:10
*** abhitechie has joined #openstack-swift09:30
*** tqtran has joined #openstack-swift09:32
*** openstackgerrit has quit IRC09:33
*** svp has joined #openstack-swift09:33
*** tqtran has quit IRC09:33
*** svp is now known as Guest6000109:34
Guest60001hi, i wanted to know the role of rsync in object replication09:35
*** ianstanfield has joined #openstack-swift09:42
kota_rledisez: here09:47
*** ianstanfield has quit IRC09:47
*** Guest60001 has quit IRC09:48
rledisezkota_: hello :)09:49
kota_rledisez: hello, hapy new year :-)09:49
rledisezthx, happy new year to you too :)09:49
rledisezin barcelona we had nice discussion about https://review.openstack.org/#/c/287057/09:49
patchbot_patch 287057 - swift - WIP: Swift Automated Tiering09:49
rledisezi think it was with tdasilva-09:50
kota_rledisez: sure09:50
rledisezi see that the review did not evolved much since09:50
rledisezwe are very interrested in this feature, so we would like to offer you help to make it alive again09:50
rledisezif that's ok with you09:50
kota_ah, it looks like the last updated is... Oct!?!?!?09:51
kota_i saw the main author, m_kazuhiro, for that pach was working recently09:52
rledisezyeah :) maybe you don't have time to work on this, or something is blocking you (we saw that few changes were needed about the devices scanning thing)09:52
rledisezoh, ok09:52
kota_he sits down next door. I will poke him to update09:52
rledisezok, thx :)09:52
kota_i think he has progress with tdasilva, iirc, he had telconf with him the end of the last year.09:53
*** mvk has quit IRC09:53
rledisezok. i would be interested to be in the loop if possible. i can have a colleague work on that for few weeks if needed. we really want this to be upstream ;)09:54
kota_probably, m_kazhiro has diff since the patch set 50 and, if you prefer to review that patch, i think it's nice to happen after updating.09:54
kota_sure09:54
*** m_kazuhiro has joined #openstack-swift09:58
m_kazuhirogood morning.09:58
rledisezm_kazuhiro: hello :)09:59
m_kazuhirorledisez: hello!10:00
rledisezm_kazuhiro: as i was telling kota_, we have serious interest in having https://review.openstack.org/#/c/287057/ merged upstream10:01
patchbot_patch 287057 - swift - WIP: Swift Automated Tiering10:01
rledisezm_kazuhiro: so, if we can help in any way, we will be happy to do so10:01
rledisezm_kazuhiro: I have a developper that can work almost full time on this for few weeks if it can help10:02
*** sheel has joined #openstack-swift10:03
m_kazuhirorledisez: Thank you for your suggestion. In current status, code review on gerrit will help me.10:05
m_kazuhirorledisez: There will be small changes on tiering patch for comments in the last summit. But I think we can start reviewing.10:08
rledisezm_kazuhiro: ok, i'll take time to see that when the review will be updated10:08
*** geaaru has joined #openstack-swift10:09
m_kazuhirorledisez: Thank you. I will push local change on tiering patch to gerrit today.10:11
*** openstackstatus has quit IRC10:13
*** openstack has joined #openstack-swift10:16
*** openstackstatus has joined #openstack-swift10:16
*** ChanServ sets mode: +v openstackstatus10:16
*** openstackgerrit has joined #openstack-swift10:20
openstackgerritFalk Reimann proposed openstack/swift: Preserve trailing slash in domain remap  https://review.openstack.org/41692410:20
*** manous has joined #openstack-swift10:21
*** mvk has joined #openstack-swift10:25
m_kazuhirorledisez: Are you still here?10:36
rledisezm_kazuhiro: yes10:37
m_kazuhirorledisez: I have an additional request on my tiering patch.10:38
m_kazuhirorledisez: One of my largest task to merge automated tiering is making document.10:39
m_kazuhirorledisez: But I'm not good at English, so the document task will be hard. Therefore, I want support to make the document.10:42
rledisezm_kazuhiro: sure. if you write a first draft, I can have a look at it to improve (my english is not perfect too, but I'll do my best ;))10:45
*** furlongm has quit IRC10:48
*** furlongm_ has joined #openstack-swift10:48
m_kazuhirorledisez: Thank you. I will make the first draft.10:51
*** jordanP has joined #openstack-swift10:55
*** sams-gleb has joined #openstack-swift10:57
*** sams-gleb has joined #openstack-swift10:57
*** kei_yama has quit IRC11:12
*** furlongm has joined #openstack-swift11:12
*** furlongm_ has quit IRC11:13
*** nikivi has joined #openstack-swift11:15
*** abhitechie has quit IRC11:17
*** vint_bra has joined #openstack-swift11:17
*** nikivi has quit IRC11:20
*** vint_bra has quit IRC11:21
*** asettle_ has quit IRC11:22
*** asettle_ has joined #openstack-swift11:23
*** asettle_ is now known as asettle11:25
*** nikivi has joined #openstack-swift11:28
*** psachin has quit IRC11:43
*** ianstanfield has joined #openstack-swift11:43
*** timss has quit IRC11:44
*** psachin has joined #openstack-swift11:44
*** timss has joined #openstack-swift11:44
*** JimCheung has joined #openstack-swift11:45
*** ianstanfield has quit IRC11:48
*** svp has joined #openstack-swift11:49
*** svp is now known as Guest6309211:49
Guest63092hi11:50
*** JimCheung has quit IRC11:50
Guest63092hi11:51
*** abhitechie has joined #openstack-swift11:52
Guest63092hi, i wanted to know the role of rsync in object replication11:52
Guest63092once the proxy server creates a copy of the object on the 3 nodes(assuming replication factor = 3), where is rsync used?11:53
Guest63092is it used if there is a failure of a node during copy or in other scenarios as well ?11:54
*** Raymii has quit IRC11:55
*** glb has quit IRC11:55
*** mtreinish has quit IRC11:55
*** sileht has quit IRC11:55
*** htruta has quit IRC11:55
*** topol has quit IRC11:55
*** balajir has quit IRC11:55
*** mahatic_ has quit IRC11:55
*** m3m0_ has quit IRC11:55
*** dosaboy has quit IRC11:55
*** Anticimex has quit IRC11:55
*** dosaboy has joined #openstack-swift11:55
*** topol_ has joined #openstack-swift11:55
*** Raymii has joined #openstack-swift11:55
*** htruta has joined #openstack-swift11:55
*** mahatic has joined #openstack-swift11:56
*** glb has joined #openstack-swift11:56
*** mtreinish has joined #openstack-swift11:56
*** sileht has joined #openstack-swift11:56
*** balajir has joined #openstack-swift11:56
*** m3m0 has joined #openstack-swift11:58
*** serverascode has quit IRC11:59
*** Guest63092 has quit IRC11:59
*** serverascode has joined #openstack-swift12:01
*** nikivi has quit IRC12:04
*** nikivi has joined #openstack-swift12:06
*** links has quit IRC12:13
*** bkopilov has quit IRC12:20
*** links has joined #openstack-swift12:25
acolesrsync is used whenever the replication process detects that replica is missing from a primary node. Replicas may be missing because the original put from proxy failed, or because a disk later failed, or because a disk error corrupted the file and the auditor quarantined the bad file.12:29
openstackgerritFalk Reimann proposed openstack/swift: Preserve trailing slash in domain remap  https://review.openstack.org/41692412:33
openstackgerritKazuhiro MIYAHARA proposed openstack/swift: WIP: Swift Automated Tiering  https://review.openstack.org/28705712:36
*** catintheroof has quit IRC12:38
*** m_kazuhiro has quit IRC12:42
*** catintheroof has joined #openstack-swift12:44
*** psachin has quit IRC12:48
*** vinsh_ has quit IRC12:51
*** psachin has joined #openstack-swift12:55
*** links has quit IRC12:55
*** catintheroof has quit IRC12:57
*** catintheroof has joined #openstack-swift12:58
*** catintheroof has quit IRC13:02
*** catintheroof has joined #openstack-swift13:04
*** links has joined #openstack-swift13:07
*** catintheroof has quit IRC13:11
*** Anticimex has joined #openstack-swift13:25
*** abhitechie has quit IRC13:27
*** d0ugal has quit IRC13:34
*** d0ugal has joined #openstack-swift13:35
*** klamath has joined #openstack-swift13:37
*** klamath has quit IRC13:37
*** klamath has joined #openstack-swift13:38
*** ianstanfield has joined #openstack-swift13:45
*** catintheroof has joined #openstack-swift13:47
*** nikivi has quit IRC13:48
*** ianstanfield has quit IRC13:50
*** cdelatte has joined #openstack-swift13:53
jrichlimathiasb, mahatic, acoles: I see tdasilva- is away, and he hasn't accepted the invite for the upcoming video conf.13:56
*** etienneme has joined #openstack-swift13:58
acolesI can imagine he is just pulling into the parking lot :)13:58
tdasilva-good morning13:59
mahaticyay he is here13:59
jrichlioh, godd.  tdasilva- o/13:59
tdasilva-sorry i'm late13:59
mahatictdasilva-: good morning13:59
jrichlis/godd/good/13:59
acolesI made an etherpad to capture some thoughts https://etherpad.openstack.org/p/swift_keymaster13:59
mahatictdasilva-: you're not late, we were looking out early ;)13:59
tdasilva-i'm in the meeting here: https://bluejeans.com/93393558514:01
acolesme too14:01
jrichliim working on it14:02
*** links has quit IRC14:02
jrichlii thought this was already installed, but it seemed to think it had to install.  launcher is taking awhile14:03
acolesI'm spinning on "Connecting"14:04
mathiasbacoles: we saw your chat14:04
acolesk14:04
mathiasbacoles: but now you've dropped14:06
*** caiobrentano has quit IRC14:09
*** caiobrentano has joined #openstack-swift14:10
*** bkopilov has joined #openstack-swift14:10
*** vinurs has quit IRC14:11
*** vinurs has joined #openstack-swift14:11
*** vinurs has quit IRC14:12
*** vinurs has joined #openstack-swift14:12
*** caiobrentano has quit IRC14:14
*** caiobrentano has joined #openstack-swift14:16
*** vinurs has quit IRC14:17
*** _JZ_ has joined #openstack-swift14:18
*** Jeffrey4l_ has quit IRC14:21
openstackgerritChristian Schwede proposed openstack/swift: Add support to increase object ring partition power  https://review.openstack.org/33729714:23
*** _JZ__ has joined #openstack-swift14:25
*** nikivi has joined #openstack-swift14:31
*** Jeffrey4l_ has joined #openstack-swift14:35
jrichliwe lost you acoles14:36
*** wer has quit IRC14:42
*** wer has joined #openstack-swift14:43
*** vinurs has joined #openstack-swift14:45
*** Jeffrey4l_ has quit IRC14:52
*** chlong has joined #openstack-swift15:05
*** Jeffrey4l_ has joined #openstack-swift15:05
*** psachin has quit IRC15:06
*** sams-gleb has quit IRC15:15
*** cnf has quit IRC15:15
*** sams-gleb has joined #openstack-swift15:15
*** cnf has joined #openstack-swift15:16
*** sams-gleb has quit IRC15:20
openstackgerritFalk Reimann proposed openstack/swift: CNAME lookup within storage domain  https://review.openstack.org/41701015:23
*** tdasilva- is now known as tdasilva15:23
*** caiobrentano has quit IRC15:23
*** ChanServ sets mode: +v tdasilva15:23
*** caiobrentano has joined #openstack-swift15:27
*** vint_bra has joined #openstack-swift15:28
tdasilvaacoles, jrichli, mathiasb, mahatic: added a link of the recording to the etherpad15:29
jrichlitdasilva: great, thanks!15:30
*** sams-gleb has joined #openstack-swift15:32
*** catintheroof has quit IRC15:32
*** Jeffrey4l_ has quit IRC15:36
acolestdasilva: thanks15:38
*** d0ugal has quit IRC15:38
*** vinurs has quit IRC15:41
*** d0ugal has joined #openstack-swift15:41
*** sgundur_ has joined #openstack-swift15:44
*** ianstanfield has joined #openstack-swift15:46
*** mvk has quit IRC15:47
*** Jeffrey4l_ has joined #openstack-swift15:48
*** vinsh has joined #openstack-swift15:49
*** ppai has quit IRC15:50
*** ianstanfield has quit IRC15:50
*** chsc has joined #openstack-swift15:53
*** cbartz has joined #openstack-swift16:00
*** oshritf has quit IRC16:00
*** rcernin has quit IRC16:07
acolesjrichli, mathiasb, mahatic, tdasilva: updated etherpad https://etherpad.openstack.org/p/swift_keymaster16:07
jrichliacoles: thanks for recording results16:08
zaitcevGuys, anyone does some kind of function tests with Hummingbird? I found that it's impossible to tell it to use my test /etc/swift/swift.conf. Seems impossible, but the code just uses configLocations and that's it.16:09
zaitcevI'm going to add another flag16:10
*** sgundur_ has quit IRC16:15
openstackgerritTakashi Kajinami proposed openstack/swift: Pass logger instances to AccountBroker/ContainerBroker  https://review.openstack.org/29587516:20
*** manous has quit IRC16:20
*** tesseract has quit IRC16:21
*** catintheroof has joined #openstack-swift16:32
*** manous has joined #openstack-swift16:34
*** sgundur_ has joined #openstack-swift16:46
*** pcaruana has quit IRC16:58
*** ianstanfield has joined #openstack-swift16:59
*** tqtran has joined #openstack-swift17:01
*** manous has quit IRC17:03
*** tqtran has quit IRC17:03
*** garyj has joined #openstack-swift17:03
*** winggundamth has quit IRC17:05
notmynamegood morning17:11
garyjg'morning17:11
*** McMurlock1 has quit IRC17:20
*** LiberalCarrot has joined #openstack-swift17:22
*** LiberalCarrot has left #openstack-swift17:22
*** manous has joined #openstack-swift17:25
*** hseipp has quit IRC17:25
*** sgundur_ has quit IRC17:28
*** sgundur_ has joined #openstack-swift17:29
notmynamethis has some good info on the openstack forum in boston http://superuser.openstack.org/articles/openstack-forum/17:33
*** sams-gleb has quit IRC17:33
*** sams-gleb has joined #openstack-swift17:34
*** sams-gleb has quit IRC17:35
*** sams-gleb has joined #openstack-swift17:35
garyjnotmyname: thanks17:40
*** sgundur_ has quit IRC17:44
*** sgundur_ has joined #openstack-swift17:44
*** LiberalSquash has joined #openstack-swift17:45
*** LiberalSquash has left #openstack-swift17:45
*** JimCheung has joined #openstack-swift17:55
*** AnxiousGarlic has joined #openstack-swift17:58
*** AnxiousGarlic has left #openstack-swift17:58
notmynameacoles: jrichli: et al. thanks for the notes on the etherpad17:59
*** rledisez has quit IRC17:59
*** jordanP has quit IRC18:04
*** manous has quit IRC18:05
*** iwaniwan has joined #openstack-swift18:12
iwaniwanIf i use django swift client, can i upload file more than 5 gb with auto segmentation?18:14
claygacoles: you need to get that line about "review is colloboration not judgement" into the review guidelines -> https://github.com/openstack/swift/blob/master/REVIEW_GUIDELINES.rst18:15
notmynameiwaniwan: if the django client does the segmentation, then sure (in that sense, it's not any different than any other client)18:15
clayg^ that was some poetical shit man18:15
*** manous has joined #openstack-swift18:16
jrichliwow, did I miss a good dev mailing list thread again?  i'll go l ook18:19
notmynamejrichli: https://www.youtube.com/watch?v=kXsIfpnThFA18:20
jrichlinotmyname: lol, good use of that one18:20
claygnotmyname: you're so jaded18:21
*** cbartz has quit IRC18:21
*** iwaniwan has quit IRC18:22
*** ChubYann has joined #openstack-swift18:23
openstackgerritJohn Dickinson proposed openstack/swift: added a quote  https://review.openstack.org/41712918:24
notmynameclayg: there ^18:24
notmynamealternatively, "the UK is sleeping, land code referencing acoles!"18:24
claygheh, we'll we should probably give him a chance to correct my probably flawed retelling of that quote - but yeah - that's the general idea.18:25
MooingLemuris there a published spec for the X-Storage-Pass/X-Storage-User/X-Auth-User/X-Auth-Key on whether those headers should be percent-escaped?  swauth seems to unescape them, but ftpcloudfs (via python swiftclient) does not escape input.18:26
acolesclayg: notmyname: sometimes I just watch in silence18:26
clayg... when he's *sleeping*18:26
claygacoles: the doug crockford quote seems to just indent?18:26
notmynameMooingLemur: all header values should be url-quoted UTF818:26
claygI like the idea that the author of "javascript the good parts" and alister coles are of similar stature in the eyes of swift reviewers18:27
acolesLOL18:28
timburkenotmyname: ...we have func tests that definitely *don't* do that18:28
acoleswhat does that say about the judgement of swift reviewers? :D18:28
notmynametimburke: well, I'm not sure I actually just said that correctly. mostly, if unicode comes up, I'm confused on what to do where/when18:28
timburkearen't we all :-(18:28
claygi guess as I understand it Mr. jslint has got himself in a bit of hot water these last few years with SJW's for a few poor taste comments made in jest :\18:29
claygacoles would never do that18:29
notmynamemaybe I was thinking of header keys18:29
acolesclayg: maybe we could co-author "Swift: the good parts"18:29
clayglol @ unicode == confused18:29
* acoles waits for obvious jokes on that one18:30
*** sgundur_ has quit IRC18:30
claygacoles: it'd be A SHORT BOOK LOL!18:30
MooingLemurnotmyname: we found that the issue comes up specifically when someone tries to set % inside of a password.  They can't set %, but they can set as %25, but then the input to swiftclient's get_auth_1_0 must contain %25 in the key rather than % itself.18:30
* acoles didn;t need to wait long18:30
claygrofl18:30
* clayg wipes tears18:30
claygacoles: happy new year btw18:30
timburkeacoles: step 1: everything is ascii. step 2: just talk directly to the storage nodes. rip out all that middleware stuff18:30
notmynameclayg: oh hey. you can finally have a stable irc client! https://irssi.org/2017/01/05/irssi-1.0.0-released/18:31
MooingLemurhaha18:31
acolesclayg: you too18:31
claygi got my swift 2.11 release out (just in time for 2.12) and got distracted by ec reconstructor inefficiencies - think i see the end on that deal and need to push some stuff upstream - so you may get stuck with a bit more of me for a few weeks until the next tire fire18:31
timburkeclayg: is that when there'll be another -dev golang ML thread or something?18:32
timburkewhy does there *have* to be a next tire fire?18:33
notmynametimburke: ...implying it could be the *same* tire fire? ;-)18:33
claygtimburke: those are more like indigestion18:34
*** sgundur_ has joined #openstack-swift18:35
*** EmilienM has joined #openstack-swift18:37
EmilienMhello Swift experts :-) happy new year first18:37
notmynamehello EmilienM18:37
EmilienMI might need some help to understand a failure we have in TripleO CI, in HA environement (3 controllers), with Glance deployed using Swift backend18:37
EmilienMthe error happens when we try to upload and list an image in Glance18:38
EmilienMthe error in Glance is here: http://logs.openstack.org/74/416974/2/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/6f2c9a7/logs/overcloud-controller-0/var/log/glance/api.txt.gz#_2017-01-05_17_30_11_24318:38
EmilienMand you can see the trace in Swift:18:38
EmilienMhttp://logs.openstack.org/74/416974/2/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/6f2c9a7/logs/overcloud-controller-0/var/log/swift/swift.txt.gz#_Jan__5_17_30_1118:38
EmilienMwe were wondering if it could be the "node_timeout" value (30s in our setup) that might be too low (our image is very very small though...)18:39
EmilienMnote: this problem is inconsistent and doesn't happen all the times (maybe 10%)18:39
EmilienMbefore jumping on launchpad and file a bug, do you have any thoughts?18:39
*** sgundur_ has quit IRC18:41
notmynameEmilienM: I wonder if there's some sort of deadlock going on. looks like both sides errored on a read18:42
EmilienMif you're wondering since when we have this problem, we use this logstash query: build_name: *tripleo-ci* AND build_status: FAILURE AND message: "Error finding address for"18:43
EmilienMand we think it's something that merged in Swift between Dec 25th and January 3rd18:44
notmynameheh, there were literally zero commits landed between dec 25 and jan 418:44
*** mvk has joined #openstack-swift18:45
notmynamehttps://github.com/openstack/swift/commits/master18:45
EmilienMwait sorry, let me check the exact dates, it might be a bit before18:45
*** sgundur_ has joined #openstack-swift18:46
EmilienMok it was 2318:46
EmilienMcould it be related then? https://github.com/openstack/swift/commit/cbb99f97b02857dd638b0d61f5e19b584037b25f18:47
EmilienMI don't think so...18:47
timburkewhat happened on the client side? it looks like they ended the chunked transfer improperly, then glance sent an extra newline or something?18:47
*** caiobren_ has joined #openstack-swift18:48
*** garyj has quit IRC18:48
EmilienMtimburke: http://logs.openstack.org/74/416974/2/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/6f2c9a7/console.html#_2017-01-05_17_30_08_67006518:48
EmilienMthis is what we have for client output18:48
*** caiobrentano has quit IRC18:49
*** asettle has quit IRC18:49
notmynameEmilienM: it's hard to tell without more details about what the client sent on the wire. looks like (from the swift-side error) that chunked transfer encoding was being used and the client didn't follow that protocol. unfortunately, all of that is handled in the wsgi server (eventlet) instead of the wsgi app (swift) so it's really hard to figure out what's going on from the app side. (that is18:52
notmyname, it's eventlet that's doing the chunked encoding parsing and then sending the app a stream of data)18:52
EmilienMnotmyname: ok, would it be useful if I provide more debug on what clients runs and the output of what happens?18:53
notmynameEmilienM: ultimately what we all need to look at is the loop in the client that's sending chunks of data on the wire18:54
notmynameI'd hope this is done in glance, but it could be the glance code that calls some sdk that uses some http library, etc18:54
acolesgood night18:56
*** acoles is now known as acoles_18:57
EmilienMnotmyname: ok, let me add --debug to the CLIs18:57
EmilienMnotmyname: so I can provide more logs on what glanceclient is doing. Sounds good?18:57
notmynameok18:58
timburkenotmyname: well, there are two errors: there's the client <-> glance error (which looks like an unexpected socket close? like, glance expected to be able to read more data but then couldn't?) and there's the glance <-> swift error, which it seems like requests might emit an extra newline as we unwind from the IO error18:58
timburkeEmilienM: yeah, i'd expect a --debug around https://github.com/openstack-infra/tripleo-ci/blob/master/scripts/tripleo.sh#L739 to be helpful18:58
EmilienMtimburke: you got it!18:58
notmynametimburke: oh, that sounds interesting (if requests is emitting an extra newline)18:58
EmilienMI was about doing that. Cool thanks folks !18:58
timburkemaybe because it's trying to re-use the connection?18:59
EmilienMI'll come back here with more debug18:59
timburkeit totally makes sense for glance to try to re-use swiftclient connections under normal operation. but if the client hangs up before the upload completes... i think glance has to just close it and grab a new one on the next request19:01
EmilienMso I proposed https://review.openstack.org/417134 but I hope we'll hit this bug, as like I said, it happens randomly19:01
patchbot_patch 417134 - openstack-infra/tripleo-ci - DNM: add --debug to glance image upload19:01
timburke(really, requests should as part of the error handling during read)19:01
notmynameEmilienM: ...meaning that you'll "recheck" until it does?19:02
EmilienMnotmyname: yes, no choice :(19:02
notmynameany chance of being able to wrap it in that --run-until-failure thing that testr has and do it locally?19:03
timburkei wonder if https://review.openstack.org/#/c/120866/ would help isolate the problem? or just paper over the glance <-> swift half to the point that it becomes really hard to spot :-/19:03
patchbot_patch 120866 - glance_store - swift store: Add upload recovery19:03
*** manous has quit IRC19:04
*** rcernin has joined #openstack-swift19:11
*** oshritf has joined #openstack-swift19:12
*** manous has joined #openstack-swift19:17
*** nikivi has quit IRC19:45
*** nikivi has joined #openstack-swift19:50
*** chlong has quit IRC20:03
*** _JZ_ has quit IRC20:17
*** ukaynar has joined #openstack-swift20:22
*** sgundur_ has quit IRC20:28
*** _JZ_ has joined #openstack-swift20:42
timburkeoh! i see better now: glance <-> swift is fine, it's the proxy-server <-> object-server connection that goes a bit sideways. proxy logs the 499, closes down the putters, and when the object-server goes to read, it throws the error20:45
*** catintheroof has quit IRC20:45
*** xlucas has joined #openstack-swift20:48
*** oshritf has quit IRC20:49
*** sgundur_ has joined #openstack-swift20:50
timburketorgomatic: you wrote https://github.com/eventlet/eventlet/commit/c3ce3ee -- do you happen to remember whether we ended up *doing* anything with that? we supposedly still work with eventlet 0.17.4 (which predates that); do we actually catch & handle ValueErrors on read somewhere?20:50
zaitcevOooh crap. Looks like Hummingbird does not work with go 1.7. Lots of panic: inconsistent import:20:59
openstackgerritTim Burke proposed openstack/swift: Prevent traceback in object-server on client disconnect  https://review.openstack.org/41718721:07
*** xlucas has quit IRC21:07
*** sams-gleb has quit IRC21:13
torgomatictimburke: not to my knowledge; I think I wanted to handle that in __call__ somewhere, but then never got around to bumping our eventlet dependency past 0.17.4 (or whenever that merged)21:17
timburketorgomatic: the EC path looks great! ^^^ should fix the replicated case21:19
*** cdelatte has quit IRC21:19
*** silor has joined #openstack-swift21:20
notmynametimburke: might be good to explain what you told me for EmilienM, so he's not spending time looking in the wrong place21:25
tdasilvatimburke: hi21:28
*** silor1 has joined #openstack-swift21:29
*** silor has quit IRC21:29
*** silor1 is now known as silor21:29
tdasilvatimburke: going through your comments on symlink and got puzzled by your comment on SLO. It seems like you tested a manifest that has a symlink as a segment21:29
timburkeEmilienM: basically, the swift log was a red herring; the proxy handled everything just fine. the object server didn't handle the proxy's disconnect gracefully, but that didn't impact glance at all. the issue is still the IOError in glance's logs, and that seems to be caused by an unexpected client disconnect21:29
timburketdasilva: yup!21:29
tdasilvatimburke: but doesnt slo have a requirement that segments must have a least 1 byte in size?21:29
tdasilvasymlinks must be 021:29
EmilienMtimburke: ok. What could cause this disconnection? network infra? or something in the client?21:30
timburketdasilva: i created an SLO with normal segments, then copied the segment elsewhere and replaced it with a symlink to the new location. downloading the SLO would still work just fine (SLO is left of symlink, so the segment gets dereferenced just fine); but COPY or POST (under post-as-copy) would fail21:32
tdasilvatricksy21:33
timburkeEmilienM: yes? not entirely sure, really. could be code, could just be transient network troubles21:34
EmilienMmhh21:35
*** Jeffrey4l_ has quit IRC21:35
timburketdasilva: yes :-) but as i recall from torgomatic's original planning of the thing, it should all totally work21:35
timburkewhen i related it to notmyname, his reaction was "we haven't gotten rid of post-as-copy yet?" :P21:36
tdasilvatimburke: yes, i agree, just trying to figure out how to make it totally work now ;)21:36
notmyname:-)21:36
tdasilvaand yeah, my first question was similar21:36
tdasilvawhen can we get rid of post-as-copy?21:36
claygtdasilva: do it right now!21:37
tdasilvaclayg: it sure would make our lives simpler21:38
claygwe did a release with support for post_as_copy = false that makes sense - this next os release is the time to make post_as_copy = false the default and emit a deprecation warning if you have the wrong value set!21:39
timburketdasilva: we'd still have to worry about COPY, though21:39
notmynameyep. clayg took the words right out of my mouth21:39
notmynameand furthermore, when we did the original post_as_copy=true release, we said...21:40
notmynameWhile the default still is21:40
notmyname      `object_post_as_copy` as True, the plan is to change the default21:40
notmyname      to False and then deprecate post-as-copy functionality in later21:40
notmyname      releases.21:40
tdasilvayeah, i think that was me ;)21:40
tdasilvahehe21:40
notmynamethat was waaay back in 2.7.0 (Mar 25, 2016)21:40
tdasilvabut would we ever remove it all together?21:40
notmynametdasilva: yes! delete it! from orbit! with fire!21:41
claygif it's been ~6mo since people have been running a version of swift that calls them basically nazi's for using the wrong setting and no one complained at us I think we can safely assume it's ok to rip it out without making anyone too sad21:41
claygit's not like it has a client facing behavior chagne where we can't really know what they're doing21:42
claygif you love post-as-copy so much - DON'T UPGRADE21:42
notmynamewe need a release where we change the default and warn if it's set to true21:42
tdasilvadoes hummingbird have support for post yet?21:42
claygno21:42
claygtdasilva: you read the latest interview with taylor rhodes from RAX?21:42
tdasilvanotmyname: swift release or OS release?21:42
claygtdasilva: which ever, it's wall time for majority of swift operators21:43
notmynametdasilva: OS release. so we need to do it asap in swift and include it in the ...what are we on? ocata? release21:43
tdasilvanotmyname: right21:44
tdasilvaclayg: did not, link?21:44
claygpotatoe21:44
notmynamethen we drop it after the ocata release, asap, and we're good to go21:44
claygrobert is your fathers brother21:44
notmynameclaybot got stuck answering riddles21:45
*** Jeffrey4l_ has joined #openstack-swift21:48
*** sgundur_ has left #openstack-swift21:49
jrichliha!21:52
openstackgerritTim Burke proposed openstack/swift: Emit deprecation warnings when running POST-as-COPY  https://review.openstack.org/41720922:00
timburkenotmyname: or do you want me to go ahead and change the default, too?22:00
notmynametimburke: yes, please. there will be a handful of docs changes needed too22:01
notmynameand thanks :-)22:01
timburkehmmm... we should consider doing something similar for the allow_versions setting in container-server.conf22:04
tdasilvatimburke: you are on fire!22:06
tdasilvadon't stop22:06
*** darrenc is now known as darrenc_afk22:07
openstackgerritPete Zaitcev proposed openstack/swift: Add the -s flag to specify hummingbird.conf  https://review.openstack.org/41721622:15
*** silor has quit IRC22:25
*** darrenc_afk is now known as darrenc22:25
openstackgerritMerged openstack/swift: added a quote  https://review.openstack.org/41712922:30
pdardeauwhat's the backstory on the acoles quote?22:34
notmynamepdardeau: mostly it's what clayg said: acoles is really smart22:35
notmynameit's something he said in a talk and in conversation either in barcelona or before, when we were all talking about contributions, the community, reviews, etc. it stuck out to me (and apparently also to clayg)22:36
pdardeauoic, thx for elaborating. i knew the really smart part already22:36
notmynamethis is an interesting discussion https://github.com/golang/go/issues/1851722:37
jrichlipdardeau: there is a link in a comment to the exact place in the the talk if you visit the review22:43
openstackgerritTim Burke proposed openstack/swift: Default object_post_as_copy to False  https://review.openstack.org/41720922:43
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873622:44
pdardeaujrichli: nice! thx for mentioning. i'll watch :)22:44
timburkehmm... should we just drop object_post_as_copy from the manpages entirely? i'm comfortable doing that for the saio example conf, but less so with the manpages. at the same time, the full explanation seems longer than what should be in a manpage...22:54
notmynametimburke: no, it's still a thing, even if there's a warning. I wouldn't remove docs on how it works until it actually isn't a thing any more22:56
notmyname(IMO)22:56
*** si1ver has joined #openstack-swift22:57
timburkenotmyname: the manpage currently only describes what happens when you set it to false :-/ since that'll be the default...22:58
timburkeand it seems silly to start adding *more* documentation about how post-as-copy works when we don't believe there's a reason to enable it22:58
notmynametimburke: ah23:00
notmynamehmm...23:00
timburkesimilar issue in the deployment guide...23:00
notmynameFYI a feature idea someone at swiftstack on the customer-facing side would like to see https://blueprints.launchpad.net/swift/+spec/human-readable-tempurl-timestamp23:03
*** manous has quit IRC23:15
*** _JZ_ has quit IRC23:18
*** kei_yama has joined #openstack-swift23:22
*** manous has joined #openstack-swift23:29
*** geaaru has quit IRC23:29
*** _JZ__ has quit IRC23:31
claygtimburke: anywhere you find the docs lacking just say literally "Deprecated" - not "what's the difference" or "this is the behavior if a or b" just "don't use this option"23:34
claygtimburke: if there's existing doco - just make sure it reflects the new/correct default23:34
claygif someone encounters the deprecated option but still thinks they need to change it - they know where to find us23:35
claygthe bigger issue is getting people currently running the default to not freak out when all of their POSTs are faster and more consistent after they upgrade!23:35
*** chsc has quit IRC23:38
*** JimCheung has quit IRC23:41
*** JimCheung has joined #openstack-swift23:41
*** JimCheung has quit IRC23:45
*** klamath has quit IRC23:45
*** mvk has quit IRC23:57
*** manous has quit IRC23:59

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