Tuesday, 2014-03-18

*** dosaboy_ has quit IRC00:14
*** dosaboy has joined #openstack-swift00:15
*** matsuhashi has joined #openstack-swift00:27
*** bsdkurt has quit IRC00:29
*** bsdkurt has joined #openstack-swift00:30
*** early has quit IRC00:37
*** early has joined #openstack-swift00:43
*** Dharmit has joined #openstack-swift00:46
*** shri has quit IRC00:48
*** mkollaro has quit IRC00:58
*** fifieldt has quit IRC01:02
*** Dharmit has quit IRC01:26
*** nosnos has joined #openstack-swift01:31
*** jergerber has joined #openstack-swift01:37
*** nosnos has quit IRC01:38
*** nosnos has joined #openstack-swift01:38
*** Dharmit has joined #openstack-swift01:49
*** Dieterbe has quit IRC02:07
*** Dieterbe has joined #openstack-swift02:14
*** haomai___ has quit IRC02:28
*** haomaiwang has joined #openstack-swift02:28
wernotmyname: just noted your Baader-Meinhof post  lolz.02:44
*** matsuhashi has quit IRC03:14
*** matsuhashi has joined #openstack-swift03:20
*** jergerber has quit IRC03:21
*** matsuhashi has quit IRC03:30
*** matsuhashi has joined #openstack-swift03:33
openstackgerritClay Gerrard proposed a change to openstack/swift: Make backend container requests use the same X-Timestamp  https://review.openstack.org/8115803:40
*** matsuhashi has quit IRC03:42
claygAm I doing my commit messages wrong?  how come bug #1292158 isn't linked to https://review.openstack.org/#/c/80073/ ???03:51
*** nosnos has quit IRC03:58
*** madhuri_ has joined #openstack-swift04:06
openstackgerritClay Gerrard proposed a change to openstack/swift: move cors-test-page to literal include  https://review.openstack.org/8116804:09
madhuri_Hi, who actually deletes the object file?04:13
*** Anju1 has left #openstack-swift04:14
*** Anju1 has joined #openstack-swift04:16
claygthat's wierd Anju1 was just asking that - obj.diskfile.hash_cleanup_listdir04:16
Anju1clayg : when a object is deleted, its corresponding entry in object table is as (noetag, 1). So whats the use of this entry?04:26
Anju1because object is deleted looking for its .ts file.04:26
claygwhat entry?  in the container listing?  I mean the "noetag" thing is just there to fill in the column for deleted objects - who's "looking" for the .ts file?04:27
Anju1I mean the usage of  database entry of deleted object in container database04:29
claygoh, you mean when do the deleted entries in the database get deleted?  yeah that's after reclaim_age04:30
claygthey have to hang around so container replicaton can get all the listings stright04:30
claygit's like a "tombstone row"04:30
*** chandan_kumar has joined #openstack-swift04:31
claygwoooo common.db.DatabaseBroker.reclaim04:31
Anju1clayg:  yeah   :) :)04:33
Anju1clayg  In case i don't want to use replication then how the reclaim would be call04:38
claygI think that's the guy that does it - you'd have to cook something up04:40
Anju1clayg : thank you  :)04:42
madhuri_clayg: Why status of an account if its deleted or not checked twice?04:48
madhuri_clayg: https://github.com/openstack/swift/blob/master/swift/account/server.py#L10004:48
madhuri_clayg: https://github.com/openstack/swift/blob/master/swift/account/server.py#L8004:48
*** ppai has joined #openstack-swift04:52
claygyeah that code pisses me off04:53
*** matsuhashi has joined #openstack-swift04:55
*** nshaikh has joined #openstack-swift04:57
claygmadhuri_: but here's some history -> https://bugs.launchpad.net/swift/+bug/118860904:57
*** haomaiw__ has joined #openstack-swift04:57
*** nosnos has joined #openstack-swift04:59
*** haomaiwang has quit IRC05:01
*** gyee has quit IRC05:11
madhuri_clayg: Thank you :)05:33
*** zaitcev has quit IRC05:43
*** psharma has joined #openstack-swift06:30
*** matsuhashi has quit IRC06:30
*** saju_m has joined #openstack-swift06:44
*** saju_m has quit IRC06:45
*** saju_m has joined #openstack-swift06:46
*** matsuhashi has joined #openstack-swift06:49
*** haomaiw__ has quit IRC07:43
*** haomaiwang has joined #openstack-swift07:43
*** haomaiw__ has joined #openstack-swift07:48
*** haomaiwang has quit IRC07:51
*** saschpe has joined #openstack-swift08:23
openstackgerritChristian Schwede proposed a change to openstack/python-swiftclient: Make bin/swift testable part 1  https://review.openstack.org/7648708:28
*** PradeepChandani has joined #openstack-swift08:29
*** saschpe has quit IRC08:29
*** saschpe_ has joined #openstack-swift08:35
*** mlipchuk has joined #openstack-swift08:37
*** mkerrin1 has quit IRC08:43
openstackgerritChristian Schwede proposed a change to openstack/python-swiftclient: Make bin/swift testable part 2  https://review.openstack.org/7648908:47
*** saschpe_ has quit IRC08:47
openstackgerritChristian Schwede proposed a change to openstack/python-swiftclient: Make bin/swift testable part 1  https://review.openstack.org/7648708:48
*** mkerrin has joined #openstack-swift08:49
*** saju_m has quit IRC08:50
openstackgerritClay Gerrard proposed a change to openstack/swift: probetest for the object expirer  https://review.openstack.org/8120508:50
*** nacim has joined #openstack-swift08:51
*** saschpe_ has joined #openstack-swift08:51
openstackgerritJenkins proposed a change to openstack/swift: Updated from global requirements  https://review.openstack.org/8120708:52
*** haomaiw__ has quit IRC08:55
*** mmcardle has joined #openstack-swift08:55
*** saju_m has joined #openstack-swift08:56
*** nshaikh has quit IRC09:01
*** tanee-away is now known as tanee09:10
*** tanee is now known as tanee-away09:10
*** haomaiwa_ has joined #openstack-swift09:13
*** Dharmit has quit IRC09:32
*** saschpe_ has quit IRC09:34
*** bada has joined #openstack-swift09:44
*** Trixboxer has joined #openstack-swift09:46
*** jamieh has joined #openstack-swift09:48
*** jamieh is now known as Guest8345609:48
*** haomaiw__ has joined #openstack-swift09:55
*** haomaiwa_ has quit IRC09:56
*** nosnos_ has joined #openstack-swift10:02
*** nosnos has quit IRC10:05
*** matsuhashi has quit IRC10:25
*** bvandenh has joined #openstack-swift10:31
*** nshaikh has joined #openstack-swift10:35
*** mkollaro has joined #openstack-swift10:44
*** tanee-away is now known as tanee10:47
*** mkollaro has quit IRC10:57
*** Dharmit has joined #openstack-swift11:04
*** ppai has quit IRC11:15
*** Dharmit is now known as d8911:18
*** d89 has quit IRC11:20
*** d89 has joined #openstack-swift11:20
*** nosnos_ has quit IRC11:30
*** nosnos has joined #openstack-swift11:31
*** ppai has joined #openstack-swift11:33
*** d89 has quit IRC11:35
*** d89 has joined #openstack-swift11:36
*** mandarine has quit IRC11:51
*** d89 has quit IRC11:55
*** JuanManuelOlle has joined #openstack-swift12:07
*** tusharsg has quit IRC12:08
*** creiht has quit IRC12:09
*** tusharsg has joined #openstack-swift12:09
*** creiht has joined #openstack-swift12:10
*** ChanServ sets mode: +v creiht12:10
*** bada has quit IRC12:10
*** bada has joined #openstack-swift12:11
*** d89 has joined #openstack-swift12:12
*** mkollaro has joined #openstack-swift12:17
*** bada_ has joined #openstack-swift12:35
*** bada has quit IRC12:39
*** ppai has quit IRC12:48
*** mkollaro has quit IRC13:01
*** mkollaro1 has joined #openstack-swift13:01
openstackgerritTushar Gohad proposed a change to openstack/swift: EC: add new storage policy type, add PyEClib req  https://review.openstack.org/6648213:06
*** tdasilva has joined #openstack-swift13:10
*** changbl has quit IRC13:14
*** psharma has quit IRC13:23
*** mmcardle has quit IRC13:25
openstackgerritpaul luse proposed a change to openstack/swift: Prevent object with storage policy mismatch from being updated in a container  https://review.openstack.org/7973113:27
*** mmcardle has joined #openstack-swift13:54
*** nosnos has quit IRC13:56
*** mkollaro has joined #openstack-swift13:56
*** mkollaro1 has quit IRC13:59
*** dmsimard has joined #openstack-swift14:03
*** byeager has joined #openstack-swift14:22
*** jergerber has joined #openstack-swift14:27
*** bsdkurt has quit IRC14:41
*** bsdkurt has joined #openstack-swift14:42
*** zaitcev has joined #openstack-swift14:45
*** ChanServ sets mode: +v zaitcev14:45
*** piyush1 has joined #openstack-swift14:49
*** zaitcev has quit IRC14:56
*** chandan_kumar has quit IRC15:02
*** chandan_kumar has joined #openstack-swift15:04
*** changbl has joined #openstack-swift15:12
*** Guest83456 has quit IRC15:20
*** jamieh has joined #openstack-swift15:20
*** jamieh is now known as Guest6230815:21
*** saju_m has quit IRC15:23
*** tanee has quit IRC15:37
*** gyee has joined #openstack-swift15:38
*** saju_m has joined #openstack-swift15:40
*** tanee-away has joined #openstack-swift15:42
*** tanee-away is now known as tanee15:42
luisbgtorgomatic, thanks for keeping the test suite offline friendly15:43
*** rupsky has joined #openstack-swift15:56
*** zaitcev has joined #openstack-swift15:57
*** ChanServ sets mode: +v zaitcev15:57
*** saju_m has quit IRC15:57
*** rupsky has quit IRC15:58
*** krtaylor has quit IRC16:04
*** csd has joined #openstack-swift16:06
*** nshaikh has quit IRC16:16
*** mkollaro has quit IRC16:22
*** nacim has quit IRC16:27
*** byeager has quit IRC16:30
*** byeager has joined #openstack-swift16:31
*** chandankumar_ has joined #openstack-swift16:34
*** byeager has quit IRC16:35
*** chandan_kumar has quit IRC16:37
*** d89 has quit IRC16:42
*** bada__ has joined #openstack-swift16:45
*** shri has joined #openstack-swift16:46
*** bada_ has quit IRC16:48
*** piyush1 has quit IRC16:57
*** JelleB is now known as a1|away17:00
*** byeager has joined #openstack-swift17:01
*** mlipchuk has quit IRC17:04
*** byeager has quit IRC17:10
*** piyush1 has joined #openstack-swift17:14
*** piyush1 has quit IRC17:25
*** piyush1 has joined #openstack-swift17:26
*** piyush2 has joined #openstack-swift17:27
*** piyush1 has quit IRC17:30
*** bada has joined #openstack-swift17:33
*** Guest62308 has quit IRC17:34
*** bada__ has quit IRC17:35
*** jamieh has joined #openstack-swift17:37
*** jamieh is now known as Guest9931517:38
*** haomaiw__ has quit IRC17:45
*** mmcardle has quit IRC17:47
*** Trixboxer has quit IRC17:54
*** lpabon has joined #openstack-swift17:58
*** gyee has quit IRC17:59
notmynamelooks like distro packagers are wanting to put a freeze on openstack projects adding or updating dependencies until the Icehouse release17:59
*** gyee has joined #openstack-swift17:59
*** tanee has quit IRC18:00
* torgomatic loves big slow time-based release cycles18:00
torgomaticno, wait, not love... the other one18:01
notmynameso the question is: what outstanding patches do we have that are proposing new dependencies and are likely to land in swift or swiftclient in the next few weeks18:01
torgomaticI think swiftclient is okay; the only dependency-related thing I see is making keystoneclient non-optional18:02
notmynameya. (I'm not sure how I feel about that yet). but that's not net-new for openstack, so it should be a noop from that perspective18:03
notmynamethe migration middleware has some optional dependencies, but nothing added by default18:03
notmyname(yay graceful degradation)18:03
*** shakamunyi has joined #openstack-swift18:04
*** byeager has joined #openstack-swift18:04
notmynamethere's some swiftclient stuff proposed that adds six as a dependency18:05
torgomaticyeah, that's kind of necessary for py3 compatibility18:06
torgomaticI mean, we could write a ton of conditional imports ourselves, but that's crappy18:06
*** tanee-away has joined #openstack-swift18:10
*** tanee-away is now known as tanee18:10
claygtorgomatic: are we going to merge master into ec now that the probe-test changes have landed?  I can't really write probetests with out the Manager now :\18:17
torgomaticclayg: sure, we can merge that any time18:17
torgomaticwell, modulo the gate being slow and flaky18:18
openstackgerritpaul luse proposed a change to openstack/swift: Add Storage Policy Support to Account HEAD  https://review.openstack.org/7374718:18
*** shakamunyi has quit IRC18:18
*** piyush2 has left #openstack-swift18:25
*** tanee is now known as tanee-away18:26
*** shakamunyi has joined #openstack-swift18:27
*** Guest99315 is now known as jamie_h18:29
*** judd7 has quit IRC18:38
*** haomaiwang has joined #openstack-swift18:40
*** tusharsg has quit IRC18:45
*** tusharg has joined #openstack-swift18:45
openstackgerritgro qez proposed a change to openstack/python-swiftclient: Decode HTTP responses, fixes bug #1282861  https://review.openstack.org/7804318:47
*** shakamunyi has quit IRC18:49
claygtorgomatic: aw man you should have done the timestamp on 404's on master - everybody loves that one!18:54
claygpeluse: I wanna take a stab at that 409 patch see if I communicate what I'm thinking - and see if you like it?18:59
*** mkollaro has joined #openstack-swift19:02
*** krtaylor has joined #openstack-swift19:03
openstackgerritSamuel Merritt proposed a change to openstack/swift: Store policy index in container_stat table  https://review.openstack.org/7170419:14
*** saju_m has joined #openstack-swift19:18
*** byeager has quit IRC19:18
*** byeager has joined #openstack-swift19:19
clayg^ so that one looks good to me19:21
claygtorgomatic: were you able to get that probetest passing?  Did that help or not really?19:21
torgomaticclayg: I couldn't get it passing on the branch, but I think I kind of see what was going on19:22
openstackgerritSamuel Merritt proposed a change to openstack/swift: remove debugging print  https://review.openstack.org/8134319:22
*** byeager has quit IRC19:24
*** gyee has quit IRC19:26
claygtorgomatic: yeah I had to merge master in to use the new probetest stuff19:28
claygtorgomatic: thanks19:28
torgomaticclayg: yup19:28
torgomaticI've got a master -> ec merge running tests now; should be proposed soon19:28
claygyay!19:28
pelusewheee :)19:29
peluseclayg:  I'm rebasing 409 patch now, after I make sure its up to snuff I have a question on one of your comments19:29
claygpeluse: so if a put/delete_object comes into a container without X-Storage-Policy-Index header, should that update use the default storage policy index - or 0?  Only old servers should be making these requests without the storage policy headers...19:43
openstackgerritSamuel Merritt proposed a change to openstack/swift: Merge branch 'master' into feature/ec  https://review.openstack.org/8134519:46
*** mlipchuk has joined #openstack-swift19:48
*** jamie_h has quit IRC19:51
openstackgerritSamuel Merritt proposed a change to openstack/swift: Remove some debugging prints from tests  https://review.openstack.org/8135319:54
*** csd has quit IRC19:59
peluseclayg:  yes, I think so20:00
pelusepolicy 0, not default I mean20:00
peluseclayg: about to post latest 409 with all of your comments addressed (well, attempts anyway)20:01
*** byeager has joined #openstack-swift20:02
peluseglayg:  but unless I'm reading the latest from torgomatic wrt your question, delete is using default when not supplied, not pol 0... (also in mine since I just rebased)20:04
openstackgerritpaul luse proposed a change to openstack/swift: Prevent object with storage policy mismatch from being updated in a container  https://review.openstack.org/7973120:05
claygpeluse: yeah I think that may be an error20:08
peluseclayg:  highlights the need already on trello for some more real functinoal tests :-)20:10
peluseclayg:  one small note on the 409 patch, I removed the policy_idx from DatabaseBroker init as you suggested but still setup the pending element there for the other brokers.  In ContainerBroker I overwrite it with a policy based pending file name....20:12
peluseclayg:  other than that the rest is self-explanantory and I really dig how it came out with that partial thing you suggested, way cool.20:13
*** csd has joined #openstack-swift20:21
notmynameI love how a .mailmap change is causing so many tests to break20:24
*** gyee has joined #openstack-swift20:30
luisbgnotmyname, .mailmap change?20:33
notmynamehttps://review.openstack.org/#/c/81027/ has been checked 4 times now20:35
claygpeluse: torgomatic: are the containerstat and container-put-object-409 patches rebased on the latest merge of master?20:43
torgomaticclayg: mine aren't yet20:45
pelusemine both have dependencies on torgomatics20:45
pelusetorgomatic:  I can go on a rebase binge if you want20:46
torgomaticpeluse: let me get mine in order real quick20:47
pelusetorgomatic:  cool enough20:47
notmynameFYI, the openstack-wide meeting is starting in 9 minute in #openstack-meeting20:52
*** JuanManuelOlle has quit IRC20:55
*** mlipchuk has quit IRC21:00
claygnotmyname: you got it covered for us?21:00
notmynamealways :-)21:01
notmynameclayg: did you see my question earlier about dependencies?21:01
notmynamethat was for this upcoming meeting21:01
*** jamieh has joined #openstack-swift21:02
*** jamieh is now known as Guest274921:03
claygnotmyname: yeah that's fine21:06
*** Guest2749 has quit IRC21:07
*** byeager has quit IRC21:11
*** lpabon has quit IRC21:18
openstackgerritGreg Lange proposed a change to openstack/swift: Fix TestGreenThreadSafeIterator test  https://review.openstack.org/8136921:18
openstackgerritSamuel Merritt proposed a change to openstack/swift: Store policy index in container_stat table  https://review.openstack.org/7170421:22
notmynameFYI dependency freeze next tuesday (openstack-wide)21:27
notmynamebut please don't merge things to swift that break generic centos/precise installs anyway :-)21:28
*** krtaylor has quit IRC21:31
pelusethere goes my 'break generic centos/precise' patch that I was planning on pushing for review later today :)21:34
*** saju_m has quit IRC21:42
openstackgerritSamuel Merritt proposed a change to openstack/swift: Put X-Timestamp in object 404 responses  https://review.openstack.org/8072921:45
openstackgerritSamuel Merritt proposed a change to openstack/swift: Make the container replicator notice misplaced objects  https://review.openstack.org/8073121:45
openstackgerritSamuel Merritt proposed a change to openstack/swift: Add object-reconciler daemon  https://review.openstack.org/8073021:45
openstackgerritSamuel Merritt proposed a change to openstack/swift: Fix changing of storage policy index  https://review.openstack.org/7253621:45
openstackgerritSamuel Merritt proposed a change to openstack/swift: Make object updater reconcile on 409s  https://review.openstack.org/8073221:46
notmynameif you have design summit sessions for swift, please submit them to http://summit.openstack.org21:57
torgomaticpeluse: okay, I think all my stuff is rebased now21:57
torgomaticwhat a pain; git-review won't let you submit something that depends on an in-flight merge, so you'll have to push to gerrit using plain git21:57
torgomatic(Gerrit's fine with it; only git-review has trouble.)21:57
pelusetorgomatic:  what fun.  about to git review one of mine rebased on yours and so far it looks OK...21:58
pelusetorgomatic:  never mind, big rebase mess... will ahve to take care of mine in the morn, running out of time today....22:13
torgomaticpeluse: hopefully the merge will land by then; I'll check on it this evening and kick it if necessary22:14
pelusesweet.... thx22:18
notmynameah. so pypy checks will always fail it seems22:21
notmynamebecause https://bugs.launchpad.net/openstack-ci/+bug/129056222:21
notmynamewhich means that either we wait for that to be fixed or we apply the patch at the bottom of the bug comments22:22
notmynameie http://paste.openstack.org/show/73701/22:22
torgomaticnotmyname: seems like a reasonable patch to apply while folks sort things out22:33
claygwtf is git review doing?  I'm all rebased on 9bea546 so why is it trying to apply 2abb829a5 "make cores better" if that's already in my history?22:46
claygmerged in via 78db7c23e (also in my history)22:46
*** dmsimard has quit IRC22:48
claygw.t.f - it like lost the merge commit22:48
clarkbclayg: it does do a rebase to look for conflicts22:49
clarkbclayg: and it does it with -p I wonder if that is te problem?22:50
peluseclayg:  it was doing that to me too - my understanding from torgomatic is that its because the merge master to EC is still in flight so git review doesn't work right (see his notes above)  I plan to wait til it lands to make life easier (plus have to leave in 5 min)22:51
claygwow, ok git is too smart for me22:54
claygtorgomatic: so do you just push gerrit ref/changes/suffix/change or do you have to make up a patset id too?22:58
torgomaticclayg: git push gerrit HEAD:refs/for/feature/ec/witty-topic-name-goes-here23:05
torgomaticit'll give you a patchset ID if you don't already have one23:05
torgomaticor however many you need23:05
openstackgerritClay Gerrard proposed a change to openstack/swift: Container respond Confict on object updates with wrong Storage-Policy-Index  https://review.openstack.org/7973123:06
claygpeluse: feel free to push right over that when ec is merged up, I think there's some things that work out looking ok wrt handling missing requested_storage_policy_index on obj PUT/DELETE - but I can reapply them later, some of the other stuff is more astetic23:11
claygtorgomatic: I ended up doing something maybe a little different with tests, sorta like the schema migrations but for storage policies23:15
claygtorgomatic: peluse: anyway, i'm going to look at 71704 again23:16
*** changbl has quit IRC23:24
*** krtaylor has joined #openstack-swift23:31

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