Monday, 2014-03-10

*** _bluev has quit IRC00:32
*** miurahr has joined #openstack-swift00:32
*** miurahr has quit IRC00:54
*** Dharmit has joined #openstack-swift00:54
*** Dharmit has quit IRC00:57
*** Dharmit has joined #openstack-swift00:57
*** _bluev has joined #openstack-swift01:03
*** _bluev has quit IRC01:07
*** swills has joined #openstack-swift01:31
*** nosnos has joined #openstack-swift01:37
*** nosnos has quit IRC01:38
*** nosnos has joined #openstack-swift01:40
openstackgerritpaul luse proposed a change to openstack/swift: Add Storage Policy Support to ssync  https://review.openstack.org/6534701:43
*** _bluev has joined #openstack-swift02:04
*** haomaiwa_ has joined #openstack-swift02:06
*** haomaiwang has quit IRC02:06
*** _bluev1 has joined #openstack-swift02:07
*** _bluev has quit IRC02:09
*** _bluev1 has quit IRC02:12
*** haomaiwang has joined #openstack-swift02:41
*** haomaiwa_ has quit IRC02:45
*** kun_huang has joined #openstack-swift02:45
*** kun_huang has quit IRC02:46
*** kun_huang has joined #openstack-swift02:58
*** _bluev has joined #openstack-swift03:08
*** _bluev has quit IRC03:13
*** Edward-Zhang has joined #openstack-swift03:46
*** mtreinish has quit IRC04:01
*** mtreinish has joined #openstack-swift04:03
*** Edward-Zhang has quit IRC04:05
*** madhuri has quit IRC04:08
*** mtreinish has quit IRC04:08
*** madhuri has joined #openstack-swift04:08
*** _bluev has joined #openstack-swift04:09
madhuriportante: ping?04:11
*** _bluev has quit IRC04:13
*** mtreinish has joined #openstack-swift04:15
*** haomaiwa_ has joined #openstack-swift04:33
*** haomaiwang has quit IRC04:34
*** chandan_kumar has joined #openstack-swift04:43
*** chandan_kumar has quit IRC04:52
*** Edward-Zhang has joined #openstack-swift04:55
*** saju_m has joined #openstack-swift04:59
*** chandan_kumar has joined #openstack-swift05:01
*** _bluev has joined #openstack-swift05:09
*** _bluev has quit IRC05:14
*** ppai has joined #openstack-swift05:23
*** _bluev has joined #openstack-swift05:38
*** nosnos has quit IRC05:39
*** nosnos has joined #openstack-swift05:39
*** saju_m has quit IRC05:54
*** saju_m has joined #openstack-swift05:55
*** nshaikh has joined #openstack-swift05:59
*** saju_m has quit IRC06:00
*** saju_m has joined #openstack-swift06:02
*** rostomyan has quit IRC06:07
*** nshaikh has left #openstack-swift06:14
*** navid__ has joined #openstack-swift06:14
*** navid__ is now known as nshaikh06:14
*** nshaikh has quit IRC06:45
*** nosnos_ has joined #openstack-swift06:51
*** nosnos has quit IRC06:51
*** nosnos has joined #openstack-swift06:56
*** psharma has joined #openstack-swift06:57
*** nosnos_ has quit IRC06:58
*** nosnos has quit IRC07:05
*** nosnos has joined #openstack-swift07:08
*** nosnos has quit IRC07:12
*** nosnos has joined #openstack-swift07:12
*** davidhadas has quit IRC07:16
*** _bluev has quit IRC07:19
*** matsuhashi has joined #openstack-swift07:39
*** occup4nt has joined #openstack-swift07:49
*** _bluev has joined #openstack-swift07:50
*** _bluev1 has joined #openstack-swift07:52
*** _bluev has quit IRC07:52
*** occupant has quit IRC07:52
*** _bluev1 has quit IRC07:56
*** davidhadas has joined #openstack-swift08:09
*** saju_m has quit IRC08:22
*** foexle has joined #openstack-swift08:24
*** saju_m has joined #openstack-swift08:28
*** davidhadas has quit IRC08:31
*** davidhadas has joined #openstack-swift08:31
*** mlipchuk has joined #openstack-swift08:37
*** Trixboxer has joined #openstack-swift08:46
*** bada_ has joined #openstack-swift08:47
*** matsuhashi has quit IRC08:50
*** saju_m has quit IRC08:50
*** bada has quit IRC08:51
*** matsuhashi has joined #openstack-swift08:54
*** nacim has joined #openstack-swift08:57
*** saju_m has joined #openstack-swift09:04
*** mmcardle has joined #openstack-swift09:10
*** saju_m has quit IRC09:10
*** saju_m has joined #openstack-swift09:11
*** nacim has quit IRC09:22
*** _bluev has joined #openstack-swift09:23
*** bada__ has joined #openstack-swift09:27
*** bada_ has quit IRC09:30
*** fbo_away is now known as fbo09:31
*** Midnightmyth has joined #openstack-swift09:39
*** Edward-Zhang has quit IRC09:46
*** SkyRocknRoll has joined #openstack-swift10:05
*** SkyRocknRoll has joined #openstack-swift10:05
*** mkollaro has joined #openstack-swift10:14
*** Dharmit has quit IRC10:17
*** Slidey has quit IRC10:30
*** Slidey has joined #openstack-swift10:32
*** davidhadas has quit IRC10:34
*** davidhadas has joined #openstack-swift10:36
*** miurahr has joined #openstack-swift10:36
*** miurahr has left #openstack-swift10:37
*** Midnightmyth_ has joined #openstack-swift10:48
*** Midnightmyth has quit IRC10:50
*** miurahr has joined #openstack-swift10:58
*** Midnightmyth_ has quit IRC10:59
*** miurahr has quit IRC11:00
*** saju_m has quit IRC11:05
*** miurahr has joined #openstack-swift11:06
*** tdasilva has left #openstack-swift11:07
*** miurahr has quit IRC11:09
*** miurahr has joined #openstack-swift11:13
*** miurahr has quit IRC11:16
*** otoolee has joined #openstack-swift11:26
*** kun_huang has quit IRC11:54
*** gvernik has joined #openstack-swift12:01
*** ppai has quit IRC12:01
*** mkollaro has quit IRC12:04
*** gvernik has quit IRC12:06
*** mkollaro has joined #openstack-swift12:09
*** matsuhashi has quit IRC12:11
*** bvandenh has joined #openstack-swift12:13
*** matsuhashi has joined #openstack-swift12:13
*** Midnightmyth has joined #openstack-swift12:16
*** ppai has joined #openstack-swift12:19
*** SkyRocknRoll has quit IRC12:32
*** ppai has quit IRC12:45
*** bada_ has joined #openstack-swift12:45
*** bada__ has quit IRC12:46
*** gvernik has joined #openstack-swift12:47
*** bada has joined #openstack-swift12:48
*** bada_ has quit IRC12:51
*** bada has quit IRC12:54
*** psharma has quit IRC12:56
*** ppai has joined #openstack-swift12:58
*** bada has joined #openstack-swift13:00
*** bada has quit IRC13:08
*** Anju1 has joined #openstack-swift13:21
*** nacim has joined #openstack-swift13:25
*** bada has joined #openstack-swift13:26
*** rustlebee is now known as russellb13:26
*** mmcardle has quit IRC13:26
*** gvernik has quit IRC13:29
*** gvernik has joined #openstack-swift13:30
*** mmcardle has joined #openstack-swift13:31
*** matsuhashi has quit IRC13:32
*** nosnos has quit IRC13:38
*** gvernik has quit IRC13:39
*** madhuri has quit IRC13:49
*** madhuri has joined #openstack-swift13:49
madhuriportante: ping?13:49
*** gvernik has joined #openstack-swift13:53
*** gvernik has quit IRC13:58
portantemadhuri: pong, finally in at work14:03
*** kun_huang has joined #openstack-swift14:05
*** mlipchuk has quit IRC14:18
peluseportante:  I did lots of upgrade testing/mixed version testing this weekend....14:21
*** bada_ has joined #openstack-swift14:22
peluseall looks good.  Had to make one small tweak to ssync patch but other than that no issues14:22
*** bada has quit IRC14:22
*** piyush has joined #openstack-swift14:22
portantepeluse: great, I'll review it again if you think that will be helpful14:23
portantepropbably still best to have gholt weight in as well14:23
peluse4 node cluster, 1 proxy 3 SN.  Did 1.10-->EC and 1.13-->EC, starting with proxy each time and doing one node at a time.  Running IO during the whole set of tests.  Did the 1.13 testing with both rsync and ssync.  Tested sync via deleting files on both the old node and new nodes and watched the files get replaced.  Very cool14:23
peluseAlso tested what shouldn't work (using a policy before the whole cluster is ready) and it got errors where expected but none fatal.14:24
*** bada__ has joined #openstack-swift14:25
peluseyup, gholt said he'd look a while back so I'll check w/him a bit later today as well.  Appreciate the review, thanks14:25
*** ppai has quit IRC14:25
*** odyssey4me has joined #openstack-swift14:26
*** tongli has joined #openstack-swift14:27
*** bada_ has quit IRC14:27
*** miurahr has joined #openstack-swift14:29
*** mlipchuk has joined #openstack-swift14:30
*** dmsimard has joined #openstack-swift14:31
luisbgmorning14:37
*** mkerrin has quit IRC14:38
*** judd7 has joined #openstack-swift14:49
*** nacim has quit IRC14:50
*** nacim has joined #openstack-swift14:50
openstackgerritJenkins proposed a change to openstack/swift: Updated from global requirements  https://review.openstack.org/7559614:53
*** bvandenh has quit IRC15:06
*** corvus is now known as jeblair15:07
*** miurahr has quit IRC15:13
tongliHi, anyone here noticed this problem, when swift is enabled in devstack env. when you run stack.sh, it fails on glance.15:14
tongliany idea?15:14
*** occup4nt has quit IRC15:32
*** bada_ has joined #openstack-swift15:32
*** madhuri__ has joined #openstack-swift15:35
madhuri__Hi, anybody around?15:36
*** bada__ has quit IRC15:36
luisbgmadhuri__, yes, just ask :)15:37
madhuri__luisbg: can you please take a look?15:38
madhuri__luisbg: https://review.openstack.org/#/c/78530/15:38
madhuri__luisbg: I am new it and I have no idea why build failed?15:38
luisbgmadhuri__, let me check15:39
madhuri__luisbg: thank you15:39
luisbgmadhuri__, have you read https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures ?15:39
madhuri__luisbg: no15:39
luisbgmadhuri__, there are some unrelated bugs in OpenStack that you can sometimes trigger with a commit15:40
madhuri__luisbg: Is there any problem in patch?15:41
luisbgso if you see a build failing for something unrelated to your commit, the usual process is to go through these list of unrelated bugs (listed in the URL I gave you)15:41
madhuri__luisbg: or am I doing anything wrong?15:41
luisbgmadhuri__, looking, I just wanted to let you know about this first15:41
*** Midnightmyth has quit IRC15:41
madhuri__luisbg: Ok, thanks15:41
luisbgmadhuri__, do you run tox in your development box? does this test you are adding pass?15:42
madhuri__luisbg: It passed15:42
luisbgmadhuri__, so you are probably hitting one of this: http://status.openstack.org/rechecks/15:42
luisbgit sometimes happens with changes that involve touching the tests15:43
luisbgit has happened to me before :( I am sorry15:43
madhuri__luisbg: What should I do now?15:43
luisbgmadhuri__, http://logs.openstack.org/30/78530/4/check/check-grenade-dsvm/070b2dc/console.html.gz#_2014-03-07_10_35_56_85415:44
luisbgmadhuri__, I see that you have rechecked it a few times, this won't help15:44
*** ben_roeder has joined #openstack-swift15:45
luisbgmadhuri__, you need to do "recheck bug ####" in gerrit, where the number of the bug is the *transient* bug, and not yours15:45
luisbgthe bug listed here in the rechecks URL15:45
ben_roederdoes anyone know why does python swift client use shuffle https://github.com/openstack/python-swiftclient/blob/master/bin/swift#L459 and here https://github.com/openstack/python-swiftclient/blob/master/bin/swift#L48815:47
*** madhuri_ has joined #openstack-swift15:47
madhuri_luisbg: I am not able to find the bug no. :(15:47
*** madhuri__ has quit IRC15:48
luisbgmadhuri_, give me a minute and I will check for you15:48
madhuri_luisbg: thank you :)15:48
luisbgmadhuri_, np15:49
*** zaitcev has joined #openstack-swift15:52
*** ChanServ sets mode: +v zaitcev15:52
*** foexle has quit IRC15:57
*** _bluev has quit IRC16:03
*** madhuri_ has quit IRC16:05
*** _bluev has joined #openstack-swift16:08
*** davidhadas has quit IRC16:08
*** anbu has joined #openstack-swift16:15
*** anbu is now known as Guest315316:16
*** Guest3153 has quit IRC16:17
*** Guest3153 has joined #openstack-swift16:17
*** Guest3153 has quit IRC16:18
*** anbu_ has joined #openstack-swift16:19
*** gyee has joined #openstack-swift16:19
*** pberis has quit IRC16:23
anbu_ondergetekende, I understand that you drafted this blueprint https://blueprints.launchpad.net/swift/+spec/swift-proxy-caching16:26
anbu_ondergetekende, are you working on it?16:26
*** SkyRocknRoll__ has joined #openstack-swift16:28
otooleecreiht: Did you get a chance to look at https://review.openstack.org/#/c/75881/ yet?16:33
*** pberis has joined #openstack-swift16:39
*** anbu_ has quit IRC16:40
*** Midnightmyth has joined #openstack-swift16:45
claygdfg: hi!16:55
claygdfg: gholt: torgomatic: so did everything get figured out with the [ds]lo override auth stuff?16:56
*** davidhadas has joined #openstack-swift16:58
notmynamegood morning16:59
notmynamehappy time change to those who observe it17:00
notmynamepeluse: does this mean you are back on mountain time?17:00
notmynamereminder that openstack meetings are scheduled at UTC, which doesn't do DST. so therefore, our swift team meeting is still at 1900UTC (ie noon Pacific, 3 Eastern)17:01
*** kun_huang has quit IRC17:03
*** tanee-away is now known as tanee17:05
*** lpabon has joined #openstack-swift17:09
*** odyssey4me has quit IRC17:16
openstackgerritAndreas Jaeger proposed a change to openstack/python-swiftclient: Improve help strings  https://review.openstack.org/7914417:17
dfgclayg: we were just waiting on those func tests that just got merged right?17:22
claygdfg: i was out on friday, so not sure - just sort of catching up17:23
claygi do remember there was some fuctests - but i was thinking that the gate will probably need to have it's pipeline updated before they pass?17:23
swifterdarrellotherjon: zackmdavis: ya, I had a non-pushed tag:  * [new tag]         ss-release-1.13.0.1 -> ss-release-1.13.0.117:23
torgomaticclayg: I think so; there's just one functional test left to land, and that's slowly working its way through the gate17:23
swifterdarrell(nevermind)17:24
dfgclayg: oh ok. my merge got merged, then torgomatic added a patch for func tests. so pretty sure it should be taked care of17:24
claygwhoot!17:24
claygi guess... i mean it's more like PHEW!17:24
torgomaticclayg: I changed the devstack pipeline to be correct, and that landed this morning, so now the functest is going again17:24
claygtorgomatic: oh nice, good work17:25
claygso back to regularlly scheduled reconciler work today?17:25
torgomaticclayg: yeah, that's the plan17:26
*** piyush has quit IRC17:33
*** madhuri_ has joined #openstack-swift17:41
madhuri_luisbg: are you around?17:41
luisbgmadhuri_, sorry, didn't saw the related transient bug17:42
luisbgmadhuri_, maybe somebody else in the channel can find it17:42
madhuri_luisbg: thanku17:45
luisbgno problem17:45
peluse notmyname:  no, AZ is now on CA time so same as you guys17:48
pelusewell, I should say you switched to AZ time to be correct :)17:48
notmynamepeluse: ah ok, I wend the wrong way ;-)17:48
notmynames/d/t17:48
*** mlipchuk has quit IRC17:48
*** gyee has quit IRC17:49
pelusegave portante an update earlier on IRC if you didn't see it - good testing this week on rolling upgrade from 1.10-->ec branch as well as 1.13-->ec branch17:49
pelusethis weekend I mean... (yesterday)17:49
madhuri_notmyname:  hii, can you please review this https://review.openstack.org/#/c/78530/4 :)17:50
notmynamepeluse: awesome. great to hear17:50
peluseyup, only found one small bug and have since corrected it (in the pending ssync patch) - everything else worked like a champ17:53
notmynamevery cool. thanks for looking into that17:53
creihtotoolee: oh I had started reviewing that, guess I didn't complete the review sorry17:53
creihtwill look at it later17:53
madhuri_notmyname: ^^^18:02
*** judd7 has quit IRC18:04
*** madhuri_ has quit IRC18:04
pelusetorgomatic:  how's the reconciliator-machine coming?  Anything I can help with (review or test)?18:05
*** mmcardle has quit IRC18:06
torgomaticpeluse: nothing really comes to mind... I've mostly got the dequeue-and-move guy, now I'm working on the two enqueue-when-things-happen guys18:07
torgomaticactually...18:07
torgomaticthere might be one piece18:07
torgomaticwith the pending changes on the ec branch, the container knows its policy, and X-SPI comes along on object updates18:08
torgomaticI have a plan to make the container server 409 if the X-SPI in the object update doesn't match what's in container_stat18:08
torgomaticthat's a separable piece from the rest of this18:09
*** mkerrin has joined #openstack-swift18:09
torgomaticpeluse: ^^18:09
zaitcevWhat's that X-SPI mumbo jumbo?18:09
torgomaticX-Storage-Policy-Index18:10
*** mkerrin has quit IRC18:10
*** booi has quit IRC18:12
*** booi has joined #openstack-swift18:13
pelusetorgomatic:  cool, I can do that.  which in your series of patches are still valid and which should I start with?18:13
torgomaticpeluse: the first three are still good, I think. I abandoned the one I didn't want any more18:14
*** piyush has joined #openstack-swift18:14
torgomaticI should probably go rebase all that stuff real quick, though18:14
torgomaticlet me finish putting a card in Trello, then I'll make sure all those patch sets are rebased... sec18:14
*** piyush1 has joined #openstack-swift18:15
pelusecool, let me know when they're rebased as I need to update my acct rollup patch also which is based on one of yours (I forget which) as well.18:16
torgomaticpeluse: ok, trello card is in, rebasing now18:18
*** piyush has quit IRC18:18
torgomaticpeluse: alright, looks like it was up-to-date already; no rebase needed18:19
*** mjseger has joined #openstack-swift18:23
mjsegertime for a silly question...  I just discovered a whole bunch of switches in the swift client I never knew were there because I never see them with help.  how come?18:25
notmynamemjseger: probably inattention. what switches?18:27
pelusetorgomatic:  excellent, need to eat some lunch, finish some presentation material (argh) and then I'll get started.  I'll add a card for it now as well18:28
torgomaticpeluse: cool, thanks18:28
mjsegerseems like all the optional ones, even the --lh I added to the swift list command.  I also just discovered --object-threads for delete?18:28
mjsegerI also see -a/-all for delete but those aren't in help either, even though they do hava a help option, so I'm wondering if there's some 'special' syntax to enable seeing them?18:29
*** mkerrin has joined #openstack-swift18:31
mjsegerbasically I was trying to figure out what swift delete was faster than my getput delete and saw code dealing with threading which I never realized existed until I saw the --container-threads switch and the corresponding help message18:31
notmynamemjseger: wait, so they are in the help messages or aren't in the help messages?18:31
mjsegerthey're in the code but when I say 'swift help' I never see them18:32
mjsegerdo you?18:32
claygsome flags only make sense for some commands18:32
*** dmsimard has quit IRC18:32
claygso there's 'swift upload -h' which has different text than 'swift download -h'18:32
notmynameeg `swift upload --help`18:32
peluseBTW, tsg (Tushar) found a bug in the delete -all with swift client, a missing join() I think - not sure if he pushed a patch for it or not but I'll ping him.  It won't work w/o it though (well, maybe for a small cluster without a lot of data)18:33
*** dmsimard has joined #openstack-swift18:33
claygtorgomatic: did you already get the tombstone-timestamp in the 404 response and gatekeeper x-override-spi changes in?18:34
*** fbo is now known as fbo_away18:34
torgomaticclayg: no, haven't done those yet18:34
mjsegertry this: swift help | grep obj.  but the parser.add.option says "help='Number of threads to use for '                       'deleting containers')18:34
mjsegeroops, I meant to grep for container18:35
claygtorgomatic: so that's other stuff peluse could help with - or that's what you're doing now?18:35
torgomaticclayg: right now I'm working on the add-to-reconciler-queue stuff during container replication, so yes, that's other stuff18:36
claygtorgomatic: killer18:37
*** gyee has joined #openstack-swift18:37
peluseclayg:  any chance you can take a gander at the ssync patch for polciies?  Its had lots of eyes and I think is ready18:37
*** SkyRocknRoll__ has quit IRC18:37
*** dmsimard has quit IRC18:38
mjsegernotmyname: I just looked at the man page, something I rarely do and the --container-threads option isn't listed there either18:38
pelusessync policy patch"  https://review.openstack.org/#/c/65347/18:38
*** Trixboxer has quit IRC18:40
zaitcevFunny how the long list of reviewers on SP-to-ssync does not include gholt18:41
mjsegernotmyname: I finally figured it out!  you only get to see the switches on syntax errors.  in other words if I say swift delete -x I see the switches18:43
*** judd7 has joined #openstack-swift18:43
*** tdasilva has joined #openstack-swift18:56
*** _bluev has quit IRC19:07
*** mmcardle has joined #openstack-swift19:09
*** mmcardle has quit IRC19:20
*** tongli has quit IRC19:20
claygmjseger: what if you say "swift delete --help"19:21
claygpeluse: let me see what I can do19:22
mjsegernotmyname: clagyg: sorry, I just saw your note about getting help on a sub-command.  I thought that might work but never hit the right syntax.  there is no help on subcommand help, at least not that I noticed.19:22
mjsegerso in other words, here I've been playing with the swift command for a couple years and never knew you could do threaded deletes!19:22
mjsegerI ended up writing my own and after a bunch of time comparisons it looks like mine is more then twice as fast, so I guess there's room for improvement if anyone care19:23
peluseclayg: gracias... and I'm sure notmyname will thank you as well :)19:24
mjsegernotmyname: clayg: digging yet deeper, it looks like help is ONLY displayed on errors or no container name.  I think the reason delete --help works, is because --help is an unknown switch.19:29
*** tanee is now known as tanee-away19:33
mjsegerclayg: notmyname: sorry to be a pain on this when there are clearly more important things to worry about, but --help after a command does not give help, it produces the error message19:35
mjsegerand the reason I know this is because 'swift delete' gives help for delete, but 'swift delete --help' give help for swift, in other words syntax error usage19:36
claygmjseger: "swift delete --help" and "swift delete --this-is-not-an-option" give different results (and exit codes) for me19:38
mjsegerclayg: oops, you're right BUT 'swift delete' gives help on delete and 'swift delete --help' gives  help on swift, not delete, even though it's included19:40
mjsegerthis is no code in the swift client that deal with --help for a command that I can see.  all I see is this for printing the additional switches:19:41
mjseger    if (not args and not options.yes_all) or (args and options.yes_all):19:41
mjsegersince there are no args, the help text gets printed.19:42
claygyeah idk19:42
mjsegeror if you say 'swift delete foo --all', assuming there is no container called foo, you also get the message19:43
mjsegerI'll add it to my list of things to do post-summit19:43
mjsegerI know people are busy and don't need extra stuff to review19:43
*** bada_ has quit IRC19:45
*** bada has joined #openstack-swift19:46
*** lsokolova has joined #openstack-swift19:49
claygis len(POLICIES) == 1 the right question?  can a new swift cluster post storage policies come up having a single storage policy who's index != 0?20:02
torgomaticclayg: nope, if you don't have a index=0 one will be provided20:03
pelusegood Q though!20:03
peluseclayg:  I can make that small change you suggested as well, no problema20:04
peluseclayg:  but I'll wait til you're done looking in case you spot other stuff like that20:07
claygpeluse: yeah20:07
claygpeluse: you should plan on getting zaitcev's thing for the trailing space too.20:07
zaitcevI thought that check was for compatibility in case there weren't any policies at all.20:08
zaitcevTherefore the one provided20:09
claygtorgomatic: peluse: k, so i found the code that ensures sp0 - but so i'm a little confused20:09
peluseOK20:09
peluseabout which part?20:09
claygtorgomatic: peluse: I can "turn off" policy 1 (remove it from swift.conf and elete the datadirs and remove the rings - it's gone), but there's no way to turn off policy 0?20:09
pelusezaitcev:  which check do you mean?20:11
peluseclayg:  correct20:11
pelusepolicy 0 is meant to be the legacy policy - what to use in the absence of any policy (maps to object-ring.gz)20:12
peluseor .ring or whatever it is :)20:12
*** dmsimard has joined #openstack-swift20:12
*** dmsimard1 has joined #openstack-swift20:16
pelusezaitcev:  if you mean len(POLICIES)==1 its meant to assure that during a rolling upgrade that only the old version of msg format is sent to avoid an error on the rcvr side.  You can't create a new policy until the upgrade is complete so this checks to see if a new policy has been crated to know itf its safe to use the new msg format20:17
*** dmsimard has quit IRC20:17
zaitcevpeluse: I thought it was pretty obvious.20:18
pelusezaitcev:  and if its not clear from earlier Q&A, there's always a policy 0 no matter what so if there's only one policy it means the user hasn't created any20:18
pelusecool20:18
*** lsokolova has quit IRC20:20
pelusealso, clayg, torgomatic and zaitcev, I did lots of rolling upgrade tests over the weekend and went through these scenarios on a real cluster (well a cluster of VMs but not an AIO) and its still, of course, possible to get errors if you have a half and half cluster and start creating policies and using them...20:20
*** lpabon has quit IRC20:22
claygpeluse: so I think it should all be done in intialize_request?20:36
claygsame place we set self.device self.partition, just set self.storage_policy based off the header in the REPLICATION request20:36
claygwell.. and send the header in the REPLICATION request20:36
claygbut that's fine because Sender has the job20:37
claygsend the spi along with every object is a best wasteful20:38
*** pberis has quit IRC20:39
*** pberis has joined #openstack-swift20:40
*** pberis has quit IRC20:40
*** pberis has joined #openstack-swift20:40
peluseclayg: yeah I guess that would work as there's just one policy per REPLICATION reqeust20:48
peluseOK, I've got no problem w/that change.  Clarrification though, I think you meant "a byte" wasteful... :)20:50
clayga byte per BILLIONS of objects, yeah i've got it coded up, it does turn out to be a bit simpler IMHO20:52
claygi'll finish up the change and we'll see if we can get someone else to chime in20:52
peluseclayg:  OK, great thanks so are you going to push it then?20:53
openstackgerritAlex Pecoraro proposed a change to openstack/swift: Allow hostname for nodes in Ring  https://review.openstack.org/7454220:58
*** judd7_ has joined #openstack-swift20:59
claygpeluse: yup21:00
*** judd7 has quit IRC21:00
peluseclayg: you rock man, thanks!21:04
*** pberis has quit IRC21:05
*** pberis has joined #openstack-swift21:05
*** tdasilva has left #openstack-swift21:20
claygpeluse: i'm in yield_hashes test update hell!21:54
*** _bluev has joined #openstack-swift21:54
*** piyush1 has left #openstack-swift21:57
*** judd7 has joined #openstack-swift22:09
*** judd7_ has quit IRC22:09
openstackgerritClay Gerrard proposed a change to openstack/swift: Add Storage Policy Support to ssync  https://review.openstack.org/7947722:25
claygpeluse: hrmm... looking at the line count I think your tests were better22:27
claygmy tests may not even pass because I didn't mock policies in tests :\22:27
claygpeluse: anyway feel free to fix or push over or just comment on the review - i'll keep working on it later22:28
*** ecl has joined #openstack-swift22:32
*** ecl has quit IRC22:32
*** tdasilva has joined #openstack-swift22:32
*** _bluev has quit IRC22:35
peluseclayg:  will do, thanks man!22:38
*** Midnightmyth has quit IRC22:39
*** judd7 has quit IRC22:39
*** judd7 has joined #openstack-swift22:40
*** miurahr has joined #openstack-swift22:43
*** nacim has quit IRC22:48
*** _bluev has joined #openstack-swift22:57
*** _bluev has quit IRC22:58
*** tdasilva has left #openstack-swift23:11
*** occup4nt has joined #openstack-swift23:15
*** occup4nt is now known as occupant23:16
*** judd7_ has joined #openstack-swift23:20
*** judd7 has quit IRC23:20
*** mkollaro has quit IRC23:25
*** dmsimard1 has quit IRC23:48
*** bada_ has joined #openstack-swift23:53
*** bada has quit IRC23:56

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