Wednesday, 2014-03-05

zaitcevcreiht: ok writing todo: "gurumed: read eventlet code, find what danpb missed re. finding greenthreads"00:00
creihthehe00:00
creihtI'm poking a bit00:00
creihtI'm also a little worried that crawling through all of gc.get_objects() could pause things for quite a bit00:01
creihton a prod system00:01
creihtthough if the thing is hung anyways, may not be a big deal00:19
creihtand it looks like there isn't a straightforward way to crawl all the current greenlets00:19
*** miurahr has quit IRC00:19
*** changbl has quit IRC00:24
*** miurahr has joined #openstack-swift00:31
*** miurahr has quit IRC00:31
*** shri has quit IRC00:33
creihtzaitcev: ok so some experimentation seems to indicate that if the greenlet has been created, but hasn't run yet then it will show that behavior00:34
zaitcevcreiht: but the tracebacks are not empty, they all end in some kind of idling like listening00:35
creihtzaitcev: an interesting change might be to ignore if greenlet.dead == True00:36
zaitcevactually hmmm.... I'll look at id(gt.gt_traceback), maybe they are all the same :-)00:36
creihtzaitcev: well it it showing the stack of the thing that is checking to see if there is a stack because there is not stack yet00:36
creihtof course the dead thing wouldn't solve the above problem, but would make sure that it wouldn't show any dead threads00:37
creihtor maybe at least print that value out in the report00:37
creihtoh00:38
creihtzaitcev: if gr_frame == None then it hasn't done anything yet so has no stack to print00:38
creihtso it is printing the stack of the parent I guess00:38
creihtso if gr_frame == None print out that there is a thread but hasn't ben run yet or some other message like that00:39
creihtdid all of that make sense?00:40
creihttime for me to step away now, but I think that will clean it up a bit00:41
zaitcevyes, I saved that line about None00:43
creihtcool00:44
*** miurahr has joined #openstack-swift00:51
*** miurahr has quit IRC00:53
*** Dharmit has joined #openstack-swift00:54
*** tdasilva has left #openstack-swift00:57
*** csd_ has joined #openstack-swift00:57
*** csd has quit IRC01:00
*** vu has quit IRC01:08
openstackgerritA change was merged to openstack/swift: Don't lard up InternalClient with extra middleware  https://review.openstack.org/7704201:09
*** csd_ has quit IRC01:16
zaitcevhttps://review.openstack.org/51613 -- gives an interesting 404 (transparency with a button), supposed to be the review to add EC support to policies01:38
*** nosnos has joined #openstack-swift01:56
*** catintheroof has joined #openstack-swift02:00
catintheroofHi guys, one quick question, having the object-replication service just one object-server.conf on most of the deploys, the intention is for the service to go one device at a time an look for the partitions to replicate? i mean, we have multiple .conf for every device, and replication is running at the same time for every device, but y dont want that behaviour, so just confirming how the replication should work02:05
*** haomaiwang has joined #openstack-swift02:15
*** haomaiwang has quit IRC02:20
*** haomaiwa_ has joined #openstack-swift02:21
*** catintheroof has quit IRC02:27
openstackgerritSamuel Merritt proposed a change to openstack/swift: Speed up failing InternalClient requests  https://review.openstack.org/7807002:43
*** changbl has joined #openstack-swift02:50
*** haomaiw__ has joined #openstack-swift03:01
*** haomaiwa_ has quit IRC03:04
openstackgerritZhang Hua proposed a change to openstack/swift: Add profiling middleware in Swift  https://review.openstack.org/5327003:18
openstackgerritZhang Hua proposed a change to openstack/swift: Add profiling middleware in Swift  https://review.openstack.org/5327003:27
*** Edward-Zhang has joined #openstack-swift03:34
*** chandan_kumar has joined #openstack-swift03:42
*** miurahr has joined #openstack-swift03:46
*** chandan_kumar has quit IRC03:46
*** Edward-Zhang has quit IRC03:50
*** miurahr has quit IRC03:58
*** gyee has quit IRC04:03
*** Edward-Zhang has joined #openstack-swift04:39
*** ppai has joined #openstack-swift04:45
openstackgerritZhang Hua proposed a change to openstack/swift: Add profiling middleware in Swift  https://review.openstack.org/5327004:47
*** Midnightmyth has joined #openstack-swift05:03
*** chandankumar_ has quit IRC05:28
*** miurahr has joined #openstack-swift05:31
*** miurahr has quit IRC05:35
*** nshaikh has joined #openstack-swift05:37
*** chandan_kumar has joined #openstack-swift05:38
*** miurahr has joined #openstack-swift05:45
*** miurahr has left #openstack-swift05:46
*** Midnightmyth has quit IRC05:53
*** miurahr has joined #openstack-swift05:59
*** miurahr has left #openstack-swift06:00
*** haomaiw__ has quit IRC06:11
*** bvandenh has joined #openstack-swift06:15
*** sungju has quit IRC06:16
*** madhuri has quit IRC06:19
*** madhuri has joined #openstack-swift06:19
*** miurahr has joined #openstack-swift06:34
*** basha has joined #openstack-swift06:35
*** zaitcev has quit IRC06:38
*** ppai has quit IRC06:45
*** miurahr has quit IRC06:45
*** ppai has joined #openstack-swift07:02
*** Edward-Zhang has quit IRC07:02
*** saju_m has joined #openstack-swift07:02
*** haomaiwang has joined #openstack-swift07:18
*** basha has quit IRC07:56
*** yuanz has joined #openstack-swift08:08
*** nosnos has quit IRC08:11
*** yuan has quit IRC08:11
*** openstackgerrit has quit IRC08:11
*** creiht has quit IRC08:11
*** gholt has quit IRC08:11
*** saju_m has quit IRC08:11
*** ppai has quit IRC08:11
*** clarkb has quit IRC08:11
*** chandan_kumar has quit IRC08:11
*** mrsnivvel has quit IRC08:11
*** Slidey has quit IRC08:11
*** joearnold has quit IRC08:11
*** cschwede- has quit IRC08:11
*** Kim-Chi-San has quit IRC08:11
*** tanee-away has quit IRC08:11
*** haomaiwang has quit IRC08:11
*** Dharmit has quit IRC08:11
*** bada has quit IRC08:11
*** krtaylor has quit IRC08:11
*** erlon has quit IRC08:11
*** peluse has quit IRC08:11
*** swifterdarrell has quit IRC08:11
*** sileht has quit IRC08:11
*** sudorandom has quit IRC08:11
*** ondergetekende has quit IRC08:11
*** akscram has quit IRC08:11
*** grapsus_ has quit IRC08:11
*** Anju has quit IRC08:11
*** mkerrin has quit IRC08:11
*** otherjon has quit IRC08:11
*** Alex_Gaynor has quit IRC08:11
*** mtreinish has quit IRC08:11
*** occupant has quit IRC08:11
*** wer has quit IRC08:11
*** CrackerJackMack has quit IRC08:11
*** briancline has quit IRC08:11
*** bvandenh has quit IRC08:11
*** wkelly has quit IRC08:11
*** ccorrigan has quit IRC08:11
*** saschpe has quit IRC08:11
*** annegentle has quit IRC08:11
*** acoles has quit IRC08:11
*** sfineberg has quit IRC08:11
*** tristanC has quit IRC08:11
*** pberis has quit IRC08:11
*** cropalato has quit IRC08:11
*** wayneeseguin has quit IRC08:11
*** acorwin has quit IRC08:11
*** jokke_ has quit IRC08:11
*** zackmdavis has quit IRC08:11
*** jeblair has quit IRC08:11
*** greghaynes has quit IRC08:11
*** nshaikh has quit IRC08:11
*** redbo has quit IRC08:11
*** judd7 has quit IRC08:11
*** notmyname has quit IRC08:11
*** MooingLemur has quit IRC08:11
*** rahmu has quit IRC08:11
*** dosaboy has quit IRC08:11
*** bsdkurt has quit IRC08:11
*** torgomatic has quit IRC08:11
*** minnear has quit IRC08:11
*** glange has quit IRC08:11
*** hugokuo has quit IRC08:11
*** ctennis has quit IRC08:11
*** anderstj has quit IRC08:11
*** alpha_ori has quit IRC08:11
*** mandarine has quit IRC08:11
*** early has quit IRC08:11
*** ryao has quit IRC08:11
*** anticw has quit IRC08:11
*** pconstantine has quit IRC08:11
*** j_king has quit IRC08:11
*** dfg has quit IRC08:11
*** mordred has quit IRC08:11
*** kragniz has quit IRC08:11
*** otoolee has quit IRC08:11
*** StevenK has quit IRC08:11
*** EmilienM has quit IRC08:11
*** pandemicsyn has quit IRC08:11
*** chmouel has quit IRC08:11
*** Diddi has quit IRC08:11
*** omame has quit IRC08:11
*** zigo has quit IRC08:11
*** mhu has quit IRC08:11
*** russellb has quit IRC08:11
*** Anticimex has quit IRC08:11
*** zanc has quit IRC08:11
*** fbo_away has quit IRC08:11
*** ChanServ has quit IRC08:11
*** madhuri has quit IRC08:11
*** mjseger has quit IRC08:11
*** ekarlso has quit IRC08:11
*** Dieterbe has quit IRC08:11
*** physcx has quit IRC08:11
*** JelleB has quit IRC08:11
*** mlanner has quit IRC08:11
*** amandap has quit IRC08:11
*** therve has quit IRC08:11
*** portante has quit IRC08:11
*** luisbg has quit IRC08:11
*** ahale has quit IRC08:11
*** rpedde has quit IRC08:11
*** clayg has quit IRC08:11
*** miurahr has joined #openstack-swift08:26
*** nosnos has joined #openstack-swift08:28
*** yuan has joined #openstack-swift08:28
*** nosnos has quit IRC08:29
*** yuan has quit IRC08:29
*** miurahr has quit IRC08:33
*** yuanz has quit IRC08:41
*** nacim_ has joined #openstack-swift08:53
*** basha has joined #openstack-swift09:02
*** openstackgerrit has joined #openstack-swift09:02
*** nosnos_ has joined #openstack-swift09:02
*** creiht_ has joined #openstack-swift09:02
*** 1JTAAJOSR has joined #openstack-swift09:02
*** saju_m has joined #openstack-swift09:02
*** ppai has joined #openstack-swift09:02
*** madhuri has joined #openstack-swift09:02
*** bvandenh has joined #openstack-swift09:02
*** chandan_kumar has joined #openstack-swift09:02
*** Dharmit has joined #openstack-swift09:02
*** otoolee has joined #openstack-swift09:02
*** gholt has joined #openstack-swift09:02
*** bada has joined #openstack-swift09:02
*** redbo has joined #openstack-swift09:02
*** wkelly has joined #openstack-swift09:02
*** judd7 has joined #openstack-swift09:02
*** mrsnivvel has joined #openstack-swift09:02
*** ccorrigan has joined #openstack-swift09:02
*** annegentle has joined #openstack-swift09:02
*** krtaylor has joined #openstack-swift09:02
*** ekarlso has joined #openstack-swift09:02
*** erlon has joined #openstack-swift09:02
*** saschpe has joined #openstack-swift09:02
*** acoles has joined #openstack-swift09:02
*** sfineberg has joined #openstack-swift09:02
*** mjseger has joined #openstack-swift09:02
*** clarkb has joined #openstack-swift09:02
*** peluse has joined #openstack-swift09:02
*** swifterdarrell has joined #openstack-swift09:02
*** tristanC has joined #openstack-swift09:02
*** sileht has joined #openstack-swift09:02
*** sudorandom has joined #openstack-swift09:02
*** ondergetekende has joined #openstack-swift09:02
*** Dieterbe has joined #openstack-swift09:02
*** physcx has joined #openstack-swift09:02
*** JelleB has joined #openstack-swift09:02
*** greghaynes has joined #openstack-swift09:02
*** notmyname has joined #openstack-swift09:02
*** dickson.freenode.net sets mode: +vvvv gholt redbo swifterdarrell notmyname09:02
*** MooingLemur has joined #openstack-swift09:02
*** pberis has joined #openstack-swift09:02
*** cropalato has joined #openstack-swift09:02
*** mlanner has joined #openstack-swift09:02
*** amandap has joined #openstack-swift09:02
*** wayneeseguin has joined #openstack-swift09:02
*** therve has joined #openstack-swift09:02
*** Slidey has joined #openstack-swift09:02
*** jokke_ has joined #openstack-swift09:02
*** zackmdavis has joined #openstack-swift09:02
*** jeblair has joined #openstack-swift09:02
*** akscram has joined #openstack-swift09:02
*** grapsus_ has joined #openstack-swift09:02
*** Anju has joined #openstack-swift09:02
*** joearnold has joined #openstack-swift09:02
*** cschwede- has joined #openstack-swift09:02
*** Kim-Chi-San has joined #openstack-swift09:02
*** tanee-away has joined #openstack-swift09:02
*** mandarine has joined #openstack-swift09:02
*** acorwin has joined #openstack-swift09:02
*** briancline has joined #openstack-swift09:02
*** CrackerJackMack has joined #openstack-swift09:02
*** wer has joined #openstack-swift09:02
*** occupant has joined #openstack-swift09:02
*** mtreinish has joined #openstack-swift09:02
*** Alex_Gaynor has joined #openstack-swift09:02
*** otherjon has joined #openstack-swift09:02
*** mkerrin has joined #openstack-swift09:02
*** rahmu has joined #openstack-swift09:02
*** StevenK has joined #openstack-swift09:02
*** dosaboy has joined #openstack-swift09:02
*** bsdkurt has joined #openstack-swift09:02
*** EmilienM has joined #openstack-swift09:02
*** pandemicsyn has joined #openstack-swift09:02
*** torgomatic has joined #openstack-swift09:02
*** minnear has joined #openstack-swift09:02
*** glange has joined #openstack-swift09:02
*** hugokuo has joined #openstack-swift09:02
*** ctennis has joined #openstack-swift09:02
*** anderstj has joined #openstack-swift09:02
*** alpha_ori has joined #openstack-swift09:02
*** portante has joined #openstack-swift09:02
*** early has joined #openstack-swift09:02
*** ryao has joined #openstack-swift09:02
*** anticw has joined #openstack-swift09:02
*** pconstantine has joined #openstack-swift09:02
*** j_king has joined #openstack-swift09:02
*** dfg has joined #openstack-swift09:02
*** dickson.freenode.net sets mode: +vvvv torgomatic glange portante dfg09:02
*** mordred has joined #openstack-swift09:02
*** kragniz has joined #openstack-swift09:02
*** chmouel has joined #openstack-swift09:02
*** Diddi has joined #openstack-swift09:02
*** omame has joined #openstack-swift09:02
*** zigo has joined #openstack-swift09:02
*** mhu has joined #openstack-swift09:02
*** russellb has joined #openstack-swift09:02
*** Anticimex has joined #openstack-swift09:02
*** luisbg has joined #openstack-swift09:02
*** ahale has joined #openstack-swift09:02
*** zanc has joined #openstack-swift09:02
*** rpedde has joined #openstack-swift09:02
*** fbo_away has joined #openstack-swift09:02
*** ChanServ has joined #openstack-swift09:02
*** clayg has joined #openstack-swift09:02
*** dickson.freenode.net sets mode: +ov ChanServ clayg09:02
*** bvandenh has quit IRC09:02
*** fbo_away is now known as fbo09:05
*** nshaikh has joined #openstack-swift09:05
*** tanee-away is now known as tanee09:05
*** tanee is now known as tanee-away09:05
openstackgerritFabien Boucher proposed a change to openstack/python-swiftclient: Allow get_account and get_container to return an iterator  https://review.openstack.org/7484509:14
*** mlipchuk has joined #openstack-swift09:16
*** saju_m has quit IRC09:19
*** psharma has joined #openstack-swift09:25
*** tanee-away is now known as tanee09:30
*** saju_m has joined #openstack-swift09:34
*** jamieh has joined #openstack-swift09:42
*** psharma has quit IRC09:47
*** Trixboxer has joined #openstack-swift10:03
*** bada has quit IRC10:22
*** miurahr has joined #openstack-swift10:25
*** miurahr has quit IRC10:30
*** jamieh has quit IRC10:37
*** jamieh has joined #openstack-swift10:38
*** miurahr has joined #openstack-swift10:45
*** mkollaro has joined #openstack-swift10:49
*** 1JTAAJOSR has quit IRC10:49
*** bvandenh has joined #openstack-swift10:51
*** bvandenh has quit IRC10:59
*** foexle has joined #openstack-swift11:02
*** miurahr has quit IRC11:17
*** chandan_kumar has quit IRC11:25
*** nshaikh has quit IRC11:32
openstackgerritChristian Schwede proposed a change to openstack/swift: Add option to log container metadata  https://review.openstack.org/6728211:42
*** psharma has joined #openstack-swift11:43
*** fbo is now known as fbo_away11:44
*** mtreinish_ has joined #openstack-swift11:48
*** mrsnivvel has quit IRC11:50
*** bada has joined #openstack-swift11:52
*** jamieh_ has joined #openstack-swift11:55
*** grapsus__ has joined #openstack-swift11:55
*** Alex_Gaynor_ has joined #openstack-swift11:55
*** mtreinish has quit IRC11:55
*** akscram has quit IRC11:56
*** mtreinish_ is now known as mtreinish11:56
*** Alex_Gaynor has quit IRC11:56
*** jamieh has quit IRC11:56
*** wer has quit IRC11:56
*** briancline has quit IRC11:56
*** grapsus_ has quit IRC11:57
*** briancline has joined #openstack-swift11:57
*** Dharmit has quit IRC11:57
*** ondergetekende has quit IRC11:57
*** ondergetekende has joined #openstack-swift11:57
*** Alex_Gaynor_ is now known as Alex_Gaynor11:57
*** wer has joined #openstack-swift11:58
*** tdasilva has joined #openstack-swift12:00
*** basha has quit IRC12:10
*** chandan_kumar has joined #openstack-swift12:11
*** dmsimard has joined #openstack-swift12:27
*** psharma has quit IRC12:44
*** bvandenh has joined #openstack-swift12:53
*** miurahr has joined #openstack-swift12:56
*** ppai has quit IRC13:03
*** cschwede- has quit IRC13:06
*** tsnider has joined #openstack-swift13:11
*** fbo_away is now known as fbo13:13
*** tsnider has left #openstack-swift13:13
*** tdasilva has left #openstack-swift13:23
*** tongli has joined #openstack-swift13:37
*** nosnos_ has quit IRC13:45
*** vuvu has joined #openstack-swift13:45
*** vu_ has joined #openstack-swift13:45
*** krtaylor has quit IRC13:46
*** akscram has joined #openstack-swift13:48
*** nosnos has joined #openstack-swift13:48
*** nosnos has quit IRC13:49
*** nosnos has joined #openstack-swift13:50
*** nosnos has quit IRC13:54
*** vu_ has quit IRC13:55
*** vuvu has quit IRC13:55
*** vuvu has joined #openstack-swift13:56
*** vu_ has joined #openstack-swift13:56
*** vu_ has quit IRC14:03
*** vuvu has quit IRC14:04
*** vu_ has joined #openstack-swift14:04
*** vuvu has joined #openstack-swift14:07
*** vu_ has quit IRC14:08
*** vu_ has joined #openstack-swift14:09
*** Dharmit has joined #openstack-swift14:09
*** vuvu has quit IRC14:12
*** vu_ has quit IRC14:14
*** saju_m has quit IRC14:15
*** vu_ has joined #openstack-swift14:19
*** vuvu has joined #openstack-swift14:19
*** vuvu has quit IRC14:19
*** vu_ has quit IRC14:19
*** vuvu has joined #openstack-swift14:20
*** vu_ has joined #openstack-swift14:20
*** vuvu has quit IRC14:25
*** vu_ has quit IRC14:25
*** vuvu has joined #openstack-swift14:34
*** vu_ has joined #openstack-swift14:34
*** krtaylor has joined #openstack-swift14:40
*** zaitcev has joined #openstack-swift14:47
*** ChanServ sets mode: +v zaitcev14:47
*** bvandenh has quit IRC14:48
*** dmsimard1 has joined #openstack-swift14:49
*** creiht_ is now known as creiht14:51
*** ChanServ sets mode: +v creiht14:51
*** dmsimard has quit IRC14:51
*** hurricanerix has joined #openstack-swift15:13
*** basha has joined #openstack-swift15:14
*** tdasilva has joined #openstack-swift15:14
*** Dharmit has quit IRC15:33
mjsegerdoes anyone know why swift has to be started as root?15:37
luisbgmjseger, I run it from my user in my box15:40
mjsegerhmm...15:40
luisbgmjseger, if all the related files in /etc/swift/ are read/writeable by the user I don't see why you would need root15:41
mjsegerare you saying you can start it w/o sudo?15:41
mjsegermight there be some things that might not work as expected?15:41
ahalei'd guess it starts as root so proxies can bind to low ports and then drop privs later15:41
luisbghttp://docs.openstack.org/developer/swift/development_saio.html15:42
luisbgstartmain started as a user15:42
mjsegerthe big problem with doing this the /proc/pid/io structures aren't readable by non-root processes, a real negative for monitoring tools that can't run as root15:42
luisbgmjseger, what distribution is this?15:44
mjsegerubuntu15:45
*** basha has quit IRC15:45
mjsegerthe newer kernels introduced a security patch that only allows the owners or root to access process I/O stats15:46
luisbgmjseger, so you answered your own question ;) if you use ubuntu's process i/o stats for swift, you need to be root15:46
luisbgAFAIK fedora uses something different15:46
*** Dharmit has joined #openstack-swift15:47
mjsegerbut my point is swift actually does run as the swift user, but since it starts out as root you lose access to those stats, hence the question of why it has to start as root.15:48
mjsegerthe reason I care is I want ot be able to get data like this for our production servers but can only do so as root, something that's disallowed - http://paste.openstack.org/show/72632/15:50
mjsegera real nifty way to get a summary of how much cpu/io each swift process is doing15:50
*** chandan_kumar has quit IRC15:51
*** vuvu has quit IRC15:51
*** vu_ has quit IRC15:51
*** chandan_kumar has joined #openstack-swift15:57
*** kun_huang has joined #openstack-swift15:58
*** vu_ has joined #openstack-swift16:02
*** vuvu has joined #openstack-swift16:02
*** mkollaro has quit IRC16:02
*** jergerber has joined #openstack-swift16:04
*** vu_ has quit IRC16:06
*** vuvu has quit IRC16:06
*** Midnightmyth has joined #openstack-swift16:06
*** asselin_wfh has joined #openstack-swift16:07
*** asselin_wfh has left #openstack-swift16:07
*** vu_ has joined #openstack-swift16:08
*** vuvu has joined #openstack-swift16:08
*** vu_ has quit IRC16:08
*** vuvu has quit IRC16:08
*** vuvu has joined #openstack-swift16:08
*** vu_ has joined #openstack-swift16:08
*** vuvu has quit IRC16:13
*** vu_ has quit IRC16:13
*** vu_ has joined #openstack-swift16:14
*** vuvu has joined #openstack-swift16:14
*** dmsimard has joined #openstack-swift16:15
*** dmsimard1 has quit IRC16:17
*** vuvu has quit IRC16:18
*** vu_ has quit IRC16:19
*** vuvu has joined #openstack-swift16:21
*** vu_ has joined #openstack-swift16:21
notmynamegood morning16:24
creihtis it lunch time yet?16:24
creiht:)16:24
luisbgcreiht, hahahaha16:34
luisbgnotmyname, around?16:34
notmynameluisbg: what's up?16:35
creihtso I'm good now with the profiling middleware16:35
creihthttps://review.openstack.org/#/c/53270/16:35
creihtif someone else wouldn't mind taking a look at it16:35
creihtit would be highly appreciated16:35
creiht;)16:35
luisbgnotmyname, could we get the two commits that I have pending in gerrit through? there are conflicting changes bound to happen16:36
luisbgand I want to stop seeing those while I move to other things :)16:36
notmynamehttps://review.openstack.org/#/c/74578/ and https://review.openstack.org/#/c/76630/16:37
luisbgnotmyname, yes, those :)16:38
luisbgcreiht, looking16:39
luisbgcreiht, I wish I could see the entire diff in one page in gerrit, is there such a way?16:39
luisbgs/such a thing/a way16:39
notmynamethere's a tiny gitweb link16:39
luisbgnotmyname, oooh! it was well hidden16:40
luisbgmuch nicer16:40
luisbgcreiht, extensive unit testing, nice :)16:41
*** foexle has quit IRC16:42
creihtluisbg: I didn't write it :)16:43
creihtbut should be handy16:43
luisbgcreiht, Zhang Hua16:43
openstackgerritChristian Schwede proposed a change to openstack/swift: Add option to log container metadata  https://review.openstack.org/6728216:45
*** gdrudy has joined #openstack-swift16:45
*** Dharmit has quit IRC16:46
*** gyee has joined #openstack-swift16:47
notmynamecreiht: thanks for the review. I'm looking at the other one16:50
creihtcool16:50
creihtI'll skip that one then :)16:50
*** piyush has joined #openstack-swift16:51
openstackgerritConstantine Peresypkin proposed a change to openstack/swift: account to account copy implementation  https://review.openstack.org/7215716:55
luisbgcreiht, thanks! Ñ=16:56
luisbgoops, wrong keyboard layout16:56
luisbgmeant, :)16:56
creihthah16:56
luisbgnotmyname, thanks John!17:00
luisbgany low hanging fruit bugs I could assign to myself and play with?17:01
notmynameluisbg: torgomatic mentioned something to me yesterday about a few simple things he had come across. otherwise, you may want to work with mjseger about the small file request latencies he's been poking at17:06
luisbgnotmyname, I've been following msjeger's benchmarking work17:06
luisbgwill talk to him to see if his intention is to raise and fix, or just to raise to make the community aware17:07
luisbgtorgomatic, could you tell me more about these few simple things you have come across? ^ (when you are around)17:07
luisbgnotmyname, thanks for the suggestions :)17:08
*** chandan_kumar has quit IRC17:16
*** krtaylor has quit IRC17:23
*** Trixboxer has quit IRC17:28
*** krtaylor has joined #openstack-swift17:30
*** openstackgerrit has quit IRC17:32
torgomaticluisbg: I'm as curious as you are17:47
notmyname:-)17:47
luisbgtorgomatic, hahahaha, wondering what notmyname is talking about?17:47
* torgomatic has been working on other things recently, so has no context17:47
notmynameluisbg: I probably mis-heard or mis-remembered. let's not bother torgomatic so that he can keep working on the last bit of storage policies :-)17:48
torgomatichehe, "last" :D17:49
luisbgheh, 60% of the time, it is the last time17:51
*** nacim_ has quit IRC17:58
*** kun_huang has quit IRC17:58
claygnotmyname: he's not working on *just* storage policies - did you see that critical fix to remove all definitions of old-style clases!?17:59
notmynamewhy is it 10am and I already want a strong drink?18:00
claygdid someone say wsgi!?18:00
notmynameor CORS?18:00
notmynameoh, reminder that we aren't having a swift team meeting today. skip until next week18:01
*** fbo is now known as fbo_away18:07
pelusenotmyname:"  is that so we can drink instead?18:08
*** tanee is now known as tanee-away18:10
*** openstackgerrit has joined #openstack-swift18:26
*** tong_ has joined #openstack-swift18:31
*** shri1 has joined #openstack-swift18:32
*** keving1 has joined #openstack-swift18:33
*** tongli has quit IRC18:33
*** csd has joined #openstack-swift18:36
*** kun_huang has joined #openstack-swift18:49
portantenotmyname: thanks for the reminder for no team meeting today18:50
openstackgerritpaul luse proposed a change to openstack/swift: Add Storage Policy Support to ssync  https://review.openstack.org/6534718:53
*** early has quit IRC18:58
*** kun_huang has quit IRC18:59
*** openstackgerrit has quit IRC19:02
*** early has joined #openstack-swift19:04
*** jamieh_ has quit IRC19:22
portantetorgomatic: looking at https://review.openstack.org/71704, "Store policy index in container_stat table"19:29
portanteDo you have a minute for some questions?19:29
*** vuvu has quit IRC19:39
*** vu_ has quit IRC19:39
zaitcevI rememeber that one, what's up?19:45
portanteI am trying to understand the details behind the commit message, and why we need to store the policy index per object in that table19:48
creihtclayg: I dropped in a suggestion for the proxy refactor19:51
claygportante: I think we're not going to have the policy index per object in the container in the first pass - but it would have been great, but maybe simpler without it19:52
claygcreiht: i hope you mean in the form of a push ;)  j/k - thanks19:52
claygcreiht: you saw there was a depdendent patch too right?19:53
portanteclayg: I am confused by the commit message and will go back and re-read to see how I arrived at that19:53
claygportante: i'm calling in the boss to set us stright - stay on the line for torgomatic19:55
torgomatichm... yeah, that one's commit message is probably somewhat wrong now... I used to have a plan19:55
portante;019:55
portante:)19:55
portanteand now?19:55
torgomaticbut anyway, the thing is that the semantics of X-Storage-Policy-Index on replication are different from the metadata19:56
torgomaticnamely, metadata keeps neweest, but X-SPI keeps oldest19:56
torgomaticso I didn't want to keep this thing in with the metadata, as that was just asking for trouble19:56
creihtclayg: no didn't see that part19:56
dmsimardCeilometer with swift question.. I have ceilometer in my proxy-server pipeline and I get meters for bytes in, bytes out and API calls. There seems to be more meters than that - something to do with a pollster ? Am I missing something ? Ref: http://docs.openstack.org/developer/ceilometer/measurements.html#object-storage-swift19:56
creihtclayg: cool, I'll look at that real quick19:57
creihtclayg: re the diff, I can submit a push if you are cool with it19:57
torgomaticportante: in any case, this makes it much faster to get at the SPI for things, and it makes it possible to use in SQL triggers and such19:57
creihtjust didn't want to stomp over your changes19:58
claygcreiht: could you ping redbo to chime in on your gist as well - I think it's about the same so if you guys both like that one better...19:58
creihtoh redbo made one as well?19:58
claygcreiht: no he was just just like "i don't like that part"19:58
creihtahh19:58
creihtk19:58
claygcreiht: yeah no worries - thanks19:59
portantetorgomatic: I read what you wrote, and it makes some sense to me, but I must be missing or not truly understanding because I still ask myself, but why do we need a column for the index?19:59
claygportante: i.e. per object instead of only in the stats table?20:00
creihtclayg: I'll go ahead and push and see what redbo thinks20:00
creihtstill easy to change to something else20:00
creihtor revert20:00
claygcreiht: right on!20:00
torgomaticportante: the comment talks (incorrectly) about stuff in the object table, which I don't think I want anymore, or at least not now20:01
torgomaticthe code only puts it in the container_stat table20:01
portanteyes20:01
torgomaticso, I definitely need to go make the commit message less wrong20:02
torgomaticportante: but does it make sense to put the new thing in container_stat?20:03
portanteclayg, torgomatic this line of questioning seems to mean I don't really understand the container stat table, so lemme go look a bit deeper20:03
torgomatick20:03
zaitcevwait, IIRC there was a moment in that patch when policy index cannot be gotten from configuration... Som kind of replication or whatnot.20:04
zaitcevstat table is just a bunch of meta, as I understand. It only has 1 row.20:04
torgomaticyes, one row only20:04
zaitcevso column header is key, content is value20:04
portanteah, I see the fetchone() on line 286, so that matches what I had hoped this was doing now20:05
portantegreat20:05
*** dmsimard has quit IRC20:26
*** fbo_away is now known as fbo20:46
portantenotmyname, torgomatic, clayg, creiht: one of the concerns I have with the feature/ec branch is that our unit test coverage is uncomfortably low from my perspective20:48
portantethat is to say, on master, I believe our unit test coverage is too low, and taking in the feature/ec work is likely to introduce a problem that might otherwise have been caught be better unit test coverage20:49
portanteI realize that I have not been involved for about a good month here, so I am not intending this to be a critique of the work, but rather pointing out a liability that I think we can collectively solve by raising the bar on the coverage20:50
claygreset20:56
claygheh20:56
*** shri1 has quit IRC21:02
*** vu has joined #openstack-swift21:05
*** vu has quit IRC21:06
*** vu has joined #openstack-swift21:07
*** shri has joined #openstack-swift21:10
*** vu has quit IRC21:11
portantetorgomatic, peluse, clayg, notmyname: regarding SP reviews, looks like there is a chain of patches that include a WIP and an OUTDATED in the change set, so I'll wait until that is resolved before continuing21:52
portanteworking on acc/cnt backends now21:52
portantezaitcev: ^^^21:52
torgomaticportante: yeah... the start of that patch chain is probably reviewable, but the end... not so much21:52
notmynameportante: the first 3 are good21:53
notmynameoh wait, I think torgomatic merged it21:53
portanteI reviewed those, so that we have a 2 company +2 on them21:54
zaitcevumm guys. SP is on its own branch. It can't possibly be not buildable21:54
zaitcevit's isolated from merges into master, with someone (Paul?) rebasing it once in a while21:54
zaitcevBranchfeature/ec21:54
portantezaitcev: what problem are you seeing with tit?21:56
portanteit/21:56
portanteit?21:56
zaitcevI'm not21:56
portanteoh, okay21:56
zaitcevbut you said that rest of SP patches are outdated. How is that possible?21:56
portantefrom gerrit saying that the given patch set to review is based on a patch set that is no longer the latest in a dependent review21:56
pelusezaitcev:  yes, we keep it up to date with master once per week (either me or torgomatic)21:57
peluseportante:  sw your question about ssync and the upgrade story and I can take that check out and assume index 0 if not included - you are right I wasn't thikning of the case where a rolling upgrade what happening21:58
torgomaticzaitcev: someone pushes chain A -> B -> C -> D; someone else fixes bug in A creating A' and pushes A' -> B' -> C' but doesn't update D; now D is outdated21:59
portantepeluse: I think there are two cases to consider22:00
portanteold ssync to new ssync, new ssync to old ssync22:00
portantein the case of old to new, accepting the default should be okay,22:01
portantebut I am not sure about  new to old22:01
*** fbo is now known as fbo_away22:01
portantethe new ssync sender should never be sending a not-default storage policy to an old receiver22:01
portantebut not sure how to detect that off the top of my head22:02
portanteperhaps gholt could weigh in here ...22:03
torgomaticwhat I'm doing for db replicator is, if ∃ only SP 0, send old-format sync command (a 7-tuple)22:03
torgomaticotherwise, send new-format sync command (an 8-tuple, with storage policy index the last one)22:03
portantewhat is that backwards "E" thing mean?22:04
torgomaticso it'll break if you define new storage policies before you're fully upgraded, but so will everything else22:04
torgomaticoh, sorry, "there exists"22:04
portanteah, thanks22:04
torgomaticit tends to creep into things I say when I've been taking paper notes22:04
portantecool that you can type it22:04
portantetorgomatic: so that means we leave it up to the admin to ensure the entire environment is post-SP before they create new ones, and we don't have much in the way of code to help detect a bad situation22:06
portantejust document it22:06
peluseso for ssync I think I just need the receivee to not puke if there's no index.22:06
torgomaticportante: yup22:06
portanteseems okay, so peluse, just not puking would be good22:07
torgomaticthere's precedent; if you haven't upgraded all your proxies but you push out configs referencing a new middleware, stuff blows up22:07
portanteagreed22:07
portantein the case of ssync, I think we can detect the mismatch, though22:08
peluseportante:  say a little more about that...22:09
portantenew ssync send message to old ssync, which old ssync will die with a 500, it looks like22:10
peluseyes, you mean because of the raise I have there now right?22:11
portanteso the sender could detect the 500 from the receiver and emit a log message warning the user that it does appear that all nodes have been upgraded22:11
portantepeluse: actually, in the case of new to old, it is the old code that will 500 there, because the split() ends up with more items that the assignment is to22:12
portanteif I understand the code correctly22:12
peluseahhh... yes22:12
pelusethat is correct sir22:12
peluseportante:  I think I can catch that at the split before it 500's and log a warn there can't I?22:13
portantewell22:14
portantecan we change the "old" code?22:14
peluseI was thikning old to new again, not new to old... but that would be a neat trick :)22:15
dfgwow! neat! one of the gate tests is actually turning up a real problem!22:15
portantekewl22:15
portantedfg: what is the problem?22:15
dfgwith my cors stuff. who runs: check-swift-dsvm-functional22:16
portantehmm, those are supposed to be the regular functional tests, no?22:16
dfgor where can I see the test code for that22:16
portantegot a log to look at?22:17
notmynameya, those are normal .functests, but running in devstack22:17
dfgoh- swif tfunc tests don't work? thats not nearly as fun...22:17
dfghaha- they are our tests. I just didn't run func tests with some setting not turned on or something. dern it. oh well.22:18
peluseportante:  so for "old to new" I can keep the check I have but just log a warn instead of raising an error as it should be possible to complete the operation (only pol 0 can be valid from old->new) - whatcha think?22:18
portanteI wonder what that setting is, or if it is keystone22:18
portantepeluse: I would just default to 0, as it could never be the case that the new code would fail to send an spi22:19
portanteand you can detect old to new by a split that is short22:19
dfghmph- i thought it was some interoperability stuff for another system with requirements on swift- but its just me not running func tests with the default proxy-server.conf oh well.22:22
portantebummer, dfg, got all up and out of my chair over that one! :)22:24
dfgyou'll learn. nobody around here listens to anything i ever say22:25
peluseportante:  cool, on the rcv side I can jsut delete the checks I have now as the obj server PUT and DELETE methods will default to 0 already if the pol idx header is missing22:25
portanteyes, I think so22:26
*** Midnightmyth has quit IRC22:28
*** tong_ has quit IRC22:30
*** tdasilva has left #openstack-swift22:35
*** bada_ has joined #openstack-swift22:51
*** bada has quit IRC22:53
*** vu_ has joined #openstack-swift22:56
*** vu has joined #openstack-swift22:56
*** Midnightmyth has joined #openstack-swift23:00
*** openstack has joined #openstack-swift23:03
portantetorgomatic: in-process func tests are fun, huh?23:06
portanteI did not want to make a ton of changes to how we run the func tests, so I keyed off the presence of the test.conf and swift.conf files to make it work23:07
portantemight be a bad assumption in general, but on my SAIO I don't use tox, and on my development box I use tox, and don't have any system-wide swift configuration23:08
torgomaticportante: ah; I do all my testing (automated and manual) on my SAIO, so I have a swift.conf there23:09
portanterats23:09
torgomaticmaybe an environment variable or something?23:09
portantefrom within in tox, or also for ./.functests type runs?23:10
portantethe other downside to this is the static way the pipeline is constructed23:10
portanteI did not try to solve that23:10
portantebut the upside of seeing coverage for functional tests seems to be big for me23:11
portantecertainly convenient for developers23:11
torgomaticoh, I'd certainly like to see coverage in functional tests, if only so I can combine that with unit-test coverage to see what's completely missing23:17
portantegreat, so to enable it, we can also check for an environment variable23:19
portantedoes that seem like it would avoid the problem you ran into?23:19
portantetorgomatic:23:19
torgomaticportante: it sure does23:19
portantegotta name that ya like, I am not fond of naming things23:20
* torgomatic has nothing23:22
*** jergerber has quit IRC23:24
*** bada_ has quit IRC23:25
*** bada has joined #openstack-swift23:25
*** hurricanerix has quit IRC23:34
*** piyush has quit IRC23:34
*** sungju has joined #openstack-swift23:45
*** sungju has quit IRC23:50
*** sungju has joined #openstack-swift23:52
*** sungju has quit IRC23:52
*** sungju has joined #openstack-swift23:52

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