*** chlong has joined #openstack-swift | 00:03 | |
*** nadeem has joined #openstack-swift | 00:04 | |
*** lyrrad has joined #openstack-swift | 00:04 | |
*** alejandrito has quit IRC | 00:08 | |
ho | pdardeau: morning! | 00:08 |
---|---|---|
then3rd | Woot, evyrything is running Liberty. | 00:10 |
*** zaitcev has quit IRC | 00:11 | |
notmyname | then3rd: nice! | 00:11 |
notmyname | then3rd: no client impact? | 00:11 |
then3rd | I had one blip where both backends were down, but haproxy let me know right away. It showed (if i recall) a layer7 timeout, even though the swift-proxy was running. | 00:13 |
then3rd | I'm sure it would've been fine if there were 3+ | 00:14 |
*** lpabon has quit IRC | 00:14 | |
then3rd | Hmm. It seems there may be a problem. Something is running slow.. the healthcheck takes a while to load, so haproxy is dropping it. | 00:33 |
*** rohit_ has quit IRC | 00:33 | |
notmyname | what if you curl directly to a proxy ip/port and ask for the heathcheck? | 00:34 |
*** rohit_ has joined #openstack-swift | 00:34 | |
notmyname | eg `time curl -i http[s]://proxyIP:port/healthcheck` | 00:34 |
notmyname | err...almost definitely would be http there | 00:34 |
*** daddyjoseph97 has quit IRC | 00:35 | |
*** SkyRocknRoll has joined #openstack-swift | 00:35 | |
then3rd | Well, I didn't try curl while it was misbehaving, but I hit one of the proxies directly from my browser. | 00:36 |
then3rd | Lets see if it happens again. | 00:36 |
kota_ | good morning | 00:43 |
kota_ | gmmaha: i thing we don't have to remove that. just "sudo pip install -U pip" via OS maintained python-pip. That way worked well in my clean Ubuntu box in AWS. | 00:45 |
kota_ | s/thing/think/ | 00:45 |
then3rd | Okay, it was dropped again. Curl just stalled the first few times, eventually it returned a 200 after 2+ seconds | 00:48 |
then3rd | Wow, | 00:51 |
*** blmartin has quit IRC | 00:58 | |
*** nakagawamsa has joined #openstack-swift | 01:01 | |
*** lyrrad has quit IRC | 01:01 | |
*** daemontool has quit IRC | 01:08 | |
*** darrenc_afk is now known as darrenc | 01:13 | |
then3rd | That's it. I should've checked my logs a long time ago. | 01:14 |
then3rd | rsyslog is messed up and not ending up in logstash for one of the proxies, but the one that is reporting... | 01:15 |
*** dmorita has quit IRC | 01:18 | |
*** dmorita has joined #openstack-swift | 01:20 | |
then3rd | STDERR: ERROR:oslo.messaging._drivers.impl_rabbit:AMQP server on 127.0.0.1:5672 is unreachable: [Errno 111] ECONNREFUSED.... over and over and over. It's because I kept celiometer enabled, but didn't setup the rabbitmq options yet. Just dropped it from the pipeline. | 01:29 |
*** dmorita has quit IRC | 01:32 | |
*** dmorita has joined #openstack-swift | 01:38 | |
ho | mattoliverau: thanks for review. as for the monkey patch, base instance is created in every method so i think we don't need to clean up. is it better to use mock in this case? | 01:39 |
mattoliverau | ho: that's true (re: cleaning up). So I guess we don't have to worry about it getting dirty. I figure we are importing patch, so we should be consistant and use it.. but maybe it's more of a NIT. | 01:48 |
ho | mattoliverau: thanks. I will follow our history :-) | 01:51 |
*** nadeem has quit IRC | 01:55 | |
*** haomaiwa_ has joined #openstack-swift | 01:56 | |
openstackgerrit | Charles Hsu proposed openstack/python-swiftclient: Add another content-type for directory. https://review.openstack.org/258927 | 01:59 |
*** haomaiwa_ has quit IRC | 02:01 | |
*** janonymous_ has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-swift | 02:03 | |
openstackgerrit | Hisashi Osanai proposed openstack/swift: Fix duplication for headers in Access-Control-Expose-Headers https://review.openstack.org/258392 | 02:06 |
*** Jeffrey4l has joined #openstack-swift | 02:13 | |
kota_ | recent gerrit UI change still makes me to confuse the patch chain dependencies (or conflict with) :/ | 02:17 |
*** badari_ has quit IRC | 02:21 | |
mattoliverau | kota_: yeah I find that too | 02:22 |
kota_ | mattoliverau: i commented 2 duplicated patches (just i was thinking they are depending) :/ | 02:23 |
kota_ | and then, i noticed the duplication by notmyname comment | 02:24 |
kota_ | my bad | 02:24 |
openstackgerrit | Merged openstack/swift: Fix func test --until-failure and --no-discover options https://review.openstack.org/258578 | 02:28 |
*** kragniz has quit IRC | 02:41 | |
*** kragniz has joined #openstack-swift | 02:46 | |
*** haomaiwang has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-swift | 03:01 | |
*** lcurtis has quit IRC | 03:01 | |
*** JohnnyCI has joined #openstack-swift | 03:02 | |
*** JohnnyCI has left #openstack-swift | 03:02 | |
*** sanchitmalhotra has joined #openstack-swift | 03:02 | |
openstackgerrit | Merged openstack/swift: Pass HTTP_REFERER down to subrequests https://review.openstack.org/258280 | 03:04 |
*** rminmin has joined #openstack-swift | 03:10 | |
*** Jeffrey4l has quit IRC | 03:14 | |
*** links has joined #openstack-swift | 03:14 | |
*** nadeem has joined #openstack-swift | 03:16 | |
*** venkat has joined #openstack-swift | 03:19 | |
*** garthb has joined #openstack-swift | 03:23 | |
openstackgerrit | Hisashi Osanai proposed openstack/swift: Add X-Delete-After and X-Delete-At functional test https://review.openstack.org/255626 | 03:26 |
*** Jeffrey4l has joined #openstack-swift | 03:27 | |
*** Jeffrey4l has quit IRC | 03:33 | |
*** gyee has quit IRC | 03:40 | |
*** Jeffrey4l has joined #openstack-swift | 03:40 | |
*** janonymous has quit IRC | 03:43 | |
*** SkyRocknRoll has quit IRC | 03:46 | |
*** yatin has joined #openstack-swift | 03:49 | |
*** Jeffrey4l__ has joined #openstack-swift | 03:57 | |
*** Jeffrey4l has quit IRC | 03:58 | |
*** Jeffrey4l__ is now known as Jeffrey4l | 03:58 | |
*** SkyRocknRoll has joined #openstack-swift | 04:00 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-swift | 04:01 | |
*** suresh has joined #openstack-swift | 04:03 | |
suresh | Is there any way to add meta quota bytes to an swift account with out adding reselleradmin role. Please someone help. | 04:06 |
*** haomaiwang has quit IRC | 04:15 | |
*** haomaiwang has joined #openstack-swift | 04:16 | |
openstackgerrit | Bing Hu proposed openstack/python-swiftclient: ecnode auth token in swiftclient log message Closes-bug:#1516692 https://review.openstack.org/259273 | 04:17 |
*** lpabon has joined #openstack-swift | 04:21 | |
*** proteusguy_ has quit IRC | 04:22 | |
*** cebruns has quit IRC | 04:27 | |
*** albertom has quit IRC | 04:28 | |
*** cebruns has joined #openstack-swift | 04:28 | |
*** albertom has joined #openstack-swift | 04:29 | |
wbhuber | notmyname: kota_: left a comment for u 2 for patch 248210 | 04:32 |
patchbot | wbhuber: https://review.openstack.org/#/c/248210/ - Update Erasure Coding Overview doc to remove Beta ... | 04:32 |
*** proteusguy_ has joined #openstack-swift | 04:34 | |
*** janonymous has joined #openstack-swift | 04:36 | |
*** takashi has joined #openstack-swift | 04:46 | |
portante | then3rd: are you still have problems with rsyslog? | 04:47 |
suresh | Is there any way to add meta quota bytes to an swift account with out adding reselleradmin role. Please someone help. | 04:47 |
notmyname | suresh: no | 04:48 |
*** badari has joined #openstack-swift | 04:49 | |
*** david-lyle has quit IRC | 04:50 | |
suresh | notmyname: tnqq | 04:52 |
*** esker has quit IRC | 04:52 | |
suresh | notmyname: if i added reseller admin role to an account it is accepting to upload more than what i set the size in quota-bytes | 04:54 |
notmyname | suresh: the quotas are soft quotas, not hard limits. http://docs.openstack.org/developer/swift/api/container_quotas.html | 04:55 |
openstackgerrit | Bing Hu proposed openstack/swift: catch lock fail exception in container updater Closes-bug: #1400939 https://review.openstack.org/259280 | 04:56 |
openstack | bug 1400939 in OpenStack Object Storage (swift) "LockTimeout exception not caught in container updater process_container" [Undecided,New] https://launchpad.net/bugs/1400939 | 04:56 |
notmyname | wow. finding different patch sets (and which one you are currently looking at) is a *lot* harder in the new gerrit UI | 04:57 |
notmyname | (there's a tiny "pull down" menu in the top right) | 04:58 |
*** klrmn has quit IRC | 04:59 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-swift | 05:01 | |
suresh | notmyname: I understanded this but what i am asking is if i assigned reselleradmin role to user it is accepting to upload more. But when i am removing the role it is not allowing to upload more than the quota-bytes | 05:03 |
notmyname | Zyric: still here? | 05:09 |
Zyric | notmyname: Hello, yes I'm here | 05:10 |
notmyname | Zyric: I'm (finally) looking at patch 241978 | 05:10 |
patchbot | notmyname: https://review.openstack.org/#/c/241978/ - Add support for storage policies to have more than... | 05:10 |
notmyname | Zyric: the logic in change_primary_name() worries. me. can you walk me through it? | 05:11 |
notmyname | Zyric: what happens if the policy only has one name | 05:11 |
notmyname | ? | 05:11 |
Zyric | notmyname: If a policy only has one name, and you enter a name that doesn't exist, it adds it as the new primary name. | 05:13 |
Zyric | If it's a valid name | 05:13 |
suresh | If i assigned reselleradmin role to user it is accepting to upload more. But when i am removing the role it is not allowing to upload more than the quota-bytes | 05:14 |
suresh | please anyone hlep | 05:14 |
notmyname | Zyric: but it calls remove_name() on the old one first. doesn't that error if it's the only name? you haven't added the new name to the list yet | 05:14 |
Zyric | notmyname: Ah, you're right, it would prompt an error in that case. | 05:16 |
mattoliverau | suresh: assigning a reseller role to a user gives them permissions to set ACLs on the account etc. If they have the role or not it doesn't change how they are affected by any account quotas. If there is an account quota then there is a soft limit applied. (thats my understanding anyway) | 05:18 |
mattoliverau | notmyname: you can use 'n' and 'p' to move through the patchsets as well, not that it really helps cause you need to look at the top to see what patchset your up to. | 05:19 |
mattoliverau | suresh: tho I could be wrong, but would need to go look at the code | 05:20 |
notmyname | I'm looking at the code now | 05:20 |
Zyric | notmyname: Is there anything else you can see that needs fixing? I'm working on an account version of patch 212824 at the moment and not sure how to go about switching back to work on the multiple names patch without breaking things. | 05:22 |
patchbot | Zyric: https://review.openstack.org/#/c/212824/ - Let operators add watchers to object audit | 05:22 |
suresh | mattoliverau: thanq | 05:23 |
*** lpabon has quit IRC | 05:24 | |
notmyname | Zyric: that's all I saw. I was looking at a diff from patch set 7 (instead of the whole thing). I liked patch set 7, so I was taking the shortcut to first look at just those changes | 05:25 |
notmyname | Zyric: so switching between concurrent things you're working on... | 05:25 |
notmyname | Zyric: judicious use of git ;-) | 05:25 |
notmyname | Zyric: make sure you're using different branches (ie not just patching master) for each thing you're doing. and for little things, `git stash` is your friend | 05:26 |
notmyname | others may have different suggestions for managing workflow | 05:26 |
mahatic | nope, git stash and git apply are my friends too | 05:26 |
mattoliverau | git branch, git stash, and sometimes pushing up to github clone so I can access what I'm working on on swift clusters in the cloud | 05:28 |
Zyric | Thanks everyone, I'll try some git tricks then. Will update the multiple names patch change_primary_name() stuff after I come back from dinner :) | 05:29 |
notmyname | mahatic: `git stash pop` is a good alternative to `git stash apply` to keep the stash cleaner | 05:30 |
mattoliverau | You can always git commit and then flatten with git rebase -i, or continually commit --amend. But that is only required if working on something more complex then a smaller change. | 05:31 |
mahatic | notmyname: ah I see. True! thanks | 05:31 |
notmyname | also, I think "alias.amend=commit --amend" is a really really useful git alias to have | 05:32 |
notmyname | so I can `git amend -a -m foobar` and it works. with `git commit --amend` it's too easy to forget one "-" and to `git commit -amend` and have everything added to the current commit with the commit message of "end" | 05:33 |
*** ppai has joined #openstack-swift | 05:37 | |
notmyname | suresh: I tried to duplicate what you are describing. I suspect you're getting the larger amount allowed because of the account info that is cached | 05:37 |
notmyname | suresh: if there is account info recently cached in the system, swift won't go all the way to disk to refresh it every time it's requested | 05:38 |
notmyname | and that's where quota info is kept | 05:38 |
notmyname | suresh: hmm...maybe not | 05:39 |
notmyname | suresh: oh, yeah. now i see it in the code. the reseller admin bypasses quotas | 05:40 |
notmyname | suresh: which makes sense, since the reseller can set the quota to anything, there's no point in blocking requests based on a quota | 05:41 |
notmyname | Zyric: cool that you're looking at the account auditor hooks. torgomatic has been out sick this week, but definitely find him online. he'll offer good input on it | 05:43 |
*** badari has quit IRC | 05:46 | |
mattoliverau | cool, well there you go then, resellers are magical :) | 05:51 |
*** SkyRocknRoll has quit IRC | 05:53 | |
*** haomaiwang has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-swift | 06:01 | |
*** takashi has quit IRC | 06:04 | |
*** serverascode has quit IRC | 06:05 | |
*** nottrobin has quit IRC | 06:06 | |
*** nottrobin has joined #openstack-swift | 06:11 | |
*** serverascode has joined #openstack-swift | 06:14 | |
*** david-lyle has joined #openstack-swift | 06:22 | |
*** trifon has joined #openstack-swift | 06:22 | |
*** trifon has quit IRC | 06:32 | |
*** haomaiw__ has joined #openstack-swift | 06:32 | |
*** haomaiwang has quit IRC | 06:34 | |
*** trifon has joined #openstack-swift | 06:36 | |
*** takashi has joined #openstack-swift | 06:38 | |
*** dmorita has quit IRC | 06:45 | |
*** dmorita has joined #openstack-swift | 06:45 | |
*** SkyRocknRoll has joined #openstack-swift | 06:46 | |
*** nadeem has quit IRC | 06:48 | |
*** dmorita has quit IRC | 06:50 | |
*** zhill has quit IRC | 07:00 | |
*** haomaiw__ has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-swift | 07:01 | |
*** SkyRocknRoll has quit IRC | 07:02 | |
venkat | Hi all, | 07:03 |
venkat | how to run a single testcase-file in swift? | 07:03 |
*** dmorita has joined #openstack-swift | 07:05 | |
*** esker has joined #openstack-swift | 07:07 | |
*** zhill has joined #openstack-swift | 07:09 | |
*** dmorita has quit IRC | 07:10 | |
*** esker has quit IRC | 07:12 | |
*** mac_ified has quit IRC | 07:14 | |
*** SkyRocknRoll has joined #openstack-swift | 07:14 | |
*** suresh has quit IRC | 07:15 | |
mattoliverau | venkat: nosetests <path to file> | 07:22 |
*** ppai has quit IRC | 07:23 | |
*** nfdeswqa has joined #openstack-swift | 07:28 | |
venkat | ok, thanks | 07:29 |
venkat | is it possible to run these tests from eclipse directly? | 07:29 |
ntt | Hi, someone can give me some advises for buying storage servers for swift? (vendor, model, ecc....) | 07:35 |
*** chlong has quit IRC | 07:37 | |
*** ppai has joined #openstack-swift | 07:37 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift: Sleep enough for trampoling https://review.openstack.org/255096 | 07:37 |
*** rcernin has joined #openstack-swift | 07:41 | |
*** garthb has quit IRC | 07:46 | |
*** ppai has quit IRC | 07:52 | |
*** rminmin has quit IRC | 07:55 | |
*** arnox has joined #openstack-swift | 07:58 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-swift | 08:01 | |
*** ppai has joined #openstack-swift | 08:05 | |
*** rledisez has joined #openstack-swift | 08:07 | |
*** nfdeswqa has quit IRC | 08:11 | |
yatin | I'm getting Insufficient storage error when creating object | 08:11 |
yatin | I checked all configuration params for object-server conf | 08:12 |
yatin | is there minimum size of disk, as i'm using saio with 4 disks of 2GB each | 08:13 |
yatin | ? | 08:13 |
yatin | able to create container under test account but not able to put object | 08:14 |
yatin | curl -i -XPUT -H "X-Auth-Token: AUTH_tk62a1cc8c872243e2b568cab13bd8a75c" http://swift:8080/v1/AUTH_test/c/foo --data-binary 1234 | 08:14 |
*** d0ugal has joined #openstack-swift | 08:14 | |
yatin | i'm using tempAuth | 08:15 |
kota_ | yatin: Are all disks mounted? | 08:15 |
yatin | kota_: yes,,, df -h shows mounted | 08:15 |
kota_ | oh, well. if all disks are used for account/container/object, should be same behavior. | 08:16 |
yatin | kota_: account-server/1.conf , devices = /srv/node/sdb | 08:16 |
yatin | kota_: the same is path in fstab, so disk is mounted | 08:17 |
kota_ | how many replicas do you set? | 08:17 |
yatin | kota_: ring i'm building as swift-ring-builder object.builder add r1z1-127.0.0.1:6010/sdb1 1 | 08:17 |
yatin | kota_: 3 | 08:17 |
takashi | yatin_: Did you check the permission of /srv/node/sdb? | 08:17 |
yatin | kota_: swift-ring-builder object.builder create 5 3 1 | 08:18 |
yatin | kota_: all change to current user and its 755 | 08:18 |
kota_ | sounds good | 08:19 |
yatin | kota_: all disk(vdi files) are of 2G, usage is 2%, doesn't look of storage | 08:19 |
yatin | kota_: doens't look out of storage | 08:20 |
yatin | kota_: this is my df -h line: /dev/sdb1 2.0G 33M 2.0G 2% /srv/node/sdb | 08:20 |
kota_ | exactly | 08:20 |
yatin | kota_: /etc/fstab, /dev/sdb1/srv/node/sdb xfs noatime,nodiratime,nobarrier,logbufs=80 2 | 08:20 |
openstackgerrit | Mahati Chamarthy proposed openstack/swift: Modify unit tests to include real crypto https://review.openstack.org/211451 | 08:21 |
kota_ | and "swift-ring-builder <builder file>" shows all disks you intend? | 08:21 |
yatin | kota_: under /srv/node/sdb/, created 2 directories, sdb1 & sdb5 | 08:21 |
yatin | kota_: using sdb1 into swift-ring-builder | 08:22 |
kota_ | just sdb1 only? | 08:22 |
yatin | swift-ring-builder object.builder create 5 3 1 | 08:22 |
yatin | swift-ring-builder object.builder add r1z1-127.0.0.1:6010/sdb1 1 | 08:22 |
yatin | kota_: yes, and other policy uses sdb5 | 08:22 |
kota_ | curious | 08:23 |
yatin | swift-ring-builder object-1.builder create 5 2 1 | 08:23 |
yatin | swift-ring-builder object-1.builder add r1z1-127.0.0.1:6010/sdb5 1 | 08:23 |
yatin | kota_: oh! policy-0 used 4 disk | 08:23 |
kota_ | you should add more disks to ring | 08:23 |
yatin | kota_: yes, default policy has 4 disk, as /sdb1 sdc2 sdd3 sde4 | 08:23 |
*** geaaru has joined #openstack-swift | 08:24 | |
kota_ | that's good, each replica should be assigned to unique devices | 08:24 |
*** ppai has quit IRC | 08:24 | |
yatin | kota_: policy-0 ring | 08:24 |
yatin | wift-ring-builder object.builder create 5 3 1 | 08:24 |
yatin | swift-ring-builder object.builder add r1z1-127.0.0.1:6010/sdb1 1 | 08:24 |
yatin | swift-ring-builder object.builder add r1z2-127.0.0.1:6020/sdc2 1 | 08:24 |
yatin | swift-ring-builder object.builder add r1z3-127.0.0.1:6030/sdd3 1 | 08:24 |
yatin | swift-ring-builder object.builder add r1z4-127.0.0.1:6040/sde4 1 | 08:24 |
*** jordanP has joined #openstack-swift | 08:24 | |
kota_ | and all (sdb1, sdc2, sdd3, sde4) are mounted, right? | 08:24 |
yatin | kota_: yes, all mounted | 08:24 |
yatin | /dev/sdb1 2.0G 33M 2.0G 2% /srv/node/sdb | 08:25 |
yatin | /dev/sdc1 2.0G 33M 2.0G 2% /srv/node/sdc | 08:25 |
yatin | /dev/sdd1 2.0G 33M 2.0G 2% /srv/node/sdd | 08:25 |
yatin | /dev/sde1 2.0G 33M 2.0G 2% /srv/node/sde | 08:25 |
kota_ | wait... | 08:25 |
kota_ | ring shows sdb1 but you looks to mount it to sdb ? | 08:26 |
kota_ | looks like inconsistency between ring and actual mount point. | 08:26 |
yatin | kota_: yes...this could be possible, let me know, what are all data you need from conf | 08:27 |
yatin | kota_: i'm creating 2 directories into each disk, and giving that directories to ring | 08:27 |
kota_ | i think you should set devices from mount point path | 08:29 |
kota_ | so | 08:29 |
kota_ | like swift-ring-builder object.builder add r1z1-127.0.0.1:6010/sdb 1 | 08:29 |
kota_ | and set "devices=/srv/node" in conf | 08:30 |
yatin | kota_: I have four services for account, container, object and hence i have put account-sever/1.conf as /srv/node/sdb | 08:31 |
yatin | kota_: /etc/swift/account-server/1.conf has similar 2.conf 3.conf 4.conf | 08:31 |
kota_ | if so, the mount points should be under /srv/node/sdb/ | 08:32 |
kota_ | like /srv/node/sdb/sdb1 ... | 08:32 |
yatin | kota_: want to use each service towards each disk, so, 1.conf says devices = /srv/node/sdb | 08:32 |
yatin | kota_: 2.conf devices = /srv/node/sdc, 3.conf devices = /srv/node/sdd 4.conf devices = /srv/node/sde | 08:32 |
yatin | kota_: not right? | 08:33 |
kota_ | not | 08:33 |
kota_ | i think your ring setting has enough capability for that | 08:34 |
yatin | kota_: if i change to : swift-ring-builder object.builder add r1z1-127.0.0.1:6010/sdb | 08:34 |
kota_ | each port has 1 device | 08:34 |
yatin | kota_: that will give me 4 disk, I created directories to make it as 8 disk | 08:35 |
kota_ | that means each service has 1 device | 08:35 |
kota_ | ah, you mean you want to use 1 disk as 2 disks, right? | 08:35 |
yatin | kota_: yes | 08:35 |
yatin | kota_: by split into 2 dirs | 08:36 |
*** silor has joined #openstack-swift | 08:36 | |
kota_ | i don't recommend such a use case (at least in production) but... | 08:36 |
*** venkat has quit IRC | 08:36 | |
kota_ | if you want, you set disk as loop back | 08:37 |
kota_ | or | 08:37 |
yatin | kota_: this is not prod, virtualbox want to extend saio example from 1 disk to 4 disks | 08:37 |
kota_ | false to mount check. | 08:37 |
kota_ | and | 08:37 |
yatin | kota_: mount_check is false | 08:37 |
*** ppai has joined #openstack-swift | 08:37 | |
kota_ | set your all devices (strictly directories you want) to ring. | 08:38 |
kota_ | i.e. you have to set 8 directories into your ring. | 08:38 |
kota_ | if split 4 devices to 8 devices. | 08:39 |
d0ugal | I'm getting this error, which I thought was authentication related, but I am having no problems doing other operations. Is it possible there are restructions on doing a PUT? or what else can I check? http://paste.openstack.org/show/482259/ | 08:39 |
yatin | kota_: this mount_check should be false all object too? | 08:41 |
kota_ | d0ugal: it doesn't look like authentication error. | 08:41 |
yatin | kota_: i have it set true, could be problem? | 08:41 |
* yatin checking object put | 08:42 | |
yatin | kota_: worked...conf problem, mount_check false needed....:) | 08:43 |
d0ugal | kota_: ah, I was mostly guessing based on something a colleague said. Do you know where I should try looking to debug it? | 08:43 |
*** silor has quit IRC | 08:43 | |
kota_ | d0ugal: could you give me the response body? that would be helpful to know what was going on. | 08:44 |
kota_ | yatin: i think so | 08:44 |
kota_ | yatin: your conf setting suggests each disk should be mounted under /srv/node/sdb but you mounted under /srv/node/, right? | 08:45 |
* d0ugal hunts for it | 08:45 | |
yatin | kota_: no, i mounted under same path | 08:45 |
yatin | kota_: please check df output above | 08:45 |
yatin | kota_: node/ I just put node dir for convention | 08:46 |
kota_ | yatin: no no no | 08:47 |
kota_ | yatin: if you set "devices=/srv/node" swift assumes subdirs are mount points (i.e. /srv/node/sdb should be a mount point) | 08:48 |
*** venkat has joined #openstack-swift | 08:48 | |
kota_ | yatin: but you set "devices=/srv/node/sdb". That means some mount points should be under /srv/node/sdb (i.e. likely /srv/node/sdb/sdb) | 08:49 |
yatin | kota_: and same sdb given into ring-builder? | 08:49 |
d0ugal | kota_: The full response body is just: <html><h1>Precondition Failed</h1><p>A precondition for this request was not met.</p></html> | 08:51 |
* d0ugal hunts for logs | 08:51 | |
kota_ | yatin: sorry, i didn't get your mean | 08:52 |
kota_ | yatin: but if ring has /sdb and you made a dir /srv/node/sdb/sdb and you set devices=/srv/node/sdb | 08:53 |
kota_ | i think that work well. | 08:53 |
* yatin checking multi-node swift doc..brb | 08:53 | |
kota_ | d0ugal: curious | 08:54 |
*** zhill has quit IRC | 08:54 | |
yatin | kota_: still not clear. devices=/srv/node/sdb having 2 dirs sdb1 and sdb5 and ring has sdb1 | 08:56 |
kota_ | yatin: you should add also sdb5 into your ring. | 08:56 |
yatin | kota_: added for policy-1 | 08:56 |
kota_ | policy-1? | 08:57 |
yatin | kota_: for object-1.builder | 08:57 |
yatin | kota_: ring | 08:57 |
*** acoles_ is now known as acoles | 08:57 | |
yatin | kota_: 4 disk, have two dir each, 1 set of four dirs from 4 disk to default storage policy | 08:58 |
yatin | kota_: and other set of 4 dirs from 4 different disk to another storage policy | 08:59 |
yatin | kota_: want to test storage policy separation with different replications values... | 09:00 |
kota_ | ic | 09:00 |
*** haomaiwang has quit IRC | 09:01 | |
kota_ | all disks has subdirs? | 09:01 |
yatin | kota_: correct | 09:01 |
mahatic | acoles: hi, could you take a look at this https://review.openstack.org/211451 sometime? it'll help move things forward. | 09:01 |
*** haomaiwang has joined #openstack-swift | 09:01 | |
yatin | kota_: thanks a ton for all the way your help...really appreciated | 09:01 |
*** Jeffrey4l has quit IRC | 09:01 | |
yatin | kota_: :) | 09:02 |
kota_ | yatin: no worries, sorry i couldn't complete | 09:02 |
yatin | kota_: next thing i want to try out if ACLs | 09:03 |
yatin | kota_: -if | 09:03 |
yatin | kota_: do you have a min? i got some quick questions | 09:04 |
*** Jeffrey4l has joined #openstack-swift | 09:04 | |
yatin | kota_: about tempAuth and account and container ACLs | 09:04 |
kota_ | d0ugal: that looks a default PreconditionFaild message.... | 09:05 |
kota_ | yatin: yup | 09:05 |
d0ugal | kota_: Yeah, I think so | 09:06 |
d0ugal | kota_: I get it with the CLI when I try a second upload. http://paste.openstack.org/show/482261/ | 09:06 |
d0ugal | so, it seems I can't update an object? | 09:06 |
yatin | kota_: for pipeline tempAuth, swift defines account, user, password and role | 09:07 |
yatin | kota_: in swift .admin .reseller_admin is called role or group? | 09:07 |
kota_ | i think "role" fits your mean. | 09:08 |
yatin | kota_: so, if any user has no role assign, then to allow him read/write access swift provides ACLs. is that right? | 09:09 |
acoles | good morning | 09:09 |
mahatic | good morning | 09:09 |
kota_ | acoles: good morning | 09:09 |
kota_ | yatin: yes | 09:10 |
kota_ | but | 09:10 |
yatin | acoles: good morning! its afternoon here ;) | 09:10 |
*** rohit_ has quit IRC | 09:10 | |
yatin | mahatic: good morning | 09:10 |
acoles | notmyname: (from scrollback) i have a bash alias amend='git commit -a --amend' :) | 09:11 |
kota_ | oh, yeah | 09:11 |
mahatic | yatin: it's afternoon here too, I was wishing acoles ;) | 09:11 |
kota_ | revert my "but" | 09:11 |
yatin | mahatic: good afternoon :) | 09:11 |
kota_ | yatin, d0ugal: sorry I'm running out of time. | 09:11 |
kota_ | shoud leave my office. | 09:12 |
acoles | ok so we need an international convention on timezone neutral greetings | 09:12 |
acoles | yo! :) | 09:12 |
d0ugal | kota_: np, thanks! | 09:12 |
yatin | kota_: thanks for your time | 09:13 |
mahatic | acoles: :D will remember to use that ;) | 09:13 |
acoles | mahatic: ack re your patch, will try | 09:13 |
mahatic | acoles: thanks | 09:13 |
yatin | acoles: kudos! | 09:13 |
* acoles has a mismatch between todo list and available time :/ | 09:14 | |
kota_ | acoles: I noticed you wrote down your suggestion to change object listing, oh no. just I was making acks in inline. | 09:14 |
kota_ | to my container-listing patch. | 09:14 |
kota_ | acoles: anyways, i commit my commens, they are temtative tho. sorry. | 09:15 |
acoles | kota_: which patch? what did i do wrong? | 09:15 |
* yatin going on swift acl tour to grab concepts and few questions | 09:15 | |
kota_ | acoles: patch 198634 | 09:15 |
patchbot | kota_: https://review.openstack.org/#/c/198634/ - Support last modified on listing containers | 09:15 |
kota_ | acoles: not wrong. they are great comments but i didn't have enough time to consider them. | 09:16 |
kota_ | so sorry but much appreciated. | 09:16 |
kota_ | will see them again | 09:17 |
acoles | kota_: oic. yeah, i just wasn't sure. actually, I tried to find a nicer way to implement the "ceil" in your func test, then gave up (what you have is fine!) and then noticed that the container HEAD is already rounded. | 09:17 |
acoles | kota_: _sleep_enough - nice! | 09:24 |
*** jistr has joined #openstack-swift | 09:25 | |
*** km has quit IRC | 09:28 | |
openstackgerrit | Hisashi Osanai proposed openstack/swift: WIP: Fix signal handling for daemons with InternalClient https://review.openstack.org/259347 | 09:29 |
*** openstackgerrit has quit IRC | 09:32 | |
*** openstackgerrit has joined #openstack-swift | 09:33 | |
ho | ^^ couldnt' reproduce the problem. i might misunderstand something... | 09:44 |
*** dmorita has joined #openstack-swift | 09:48 | |
*** dmorita has quit IRC | 09:53 | |
*** links has quit IRC | 09:54 | |
openstackgerrit | Alistair Coles proposed openstack/swift: Sleep enough for trampoline https://review.openstack.org/255096 | 09:56 |
acoles | kota_: ^^ just fixed typo in commit message then +2 | 09:58 |
*** haomaiwang has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-swift | 10:01 | |
*** esker has joined #openstack-swift | 10:01 | |
ho | acoles: you may fix the commit message from new gerrit (edit buttom then select "Commit Message") | 10:02 |
acoles | ho: yes, thats how i did it. but it does cause a new patchset version to be created. which makes sense i guess. | 10:03 |
acoles | ho: in fact, you could edit commit message in the old gerrit, now you can also edit patch files, which is scary imho | 10:04 |
ho | acoles: i see. it's useful :-) | 10:04 |
acoles | ho: it concerns me that it encourages people to make changes to code on gerrit and therefore not run unit/pep8 checks locally before submitting the change. | 10:05 |
*** esker has quit IRC | 10:06 | |
*** links has joined #openstack-swift | 10:06 | |
*** nakagawamsa has quit IRC | 10:09 | |
ho | acoles: yeah, good ui has bad side effect... | 10:10 |
*** mahatic has quit IRC | 10:12 | |
ho | i will leave now. have a nice day & weekend! | 10:15 |
*** ho has quit IRC | 10:16 | |
acoles | ho: thanks, and you | 10:20 |
*** trifon has quit IRC | 10:35 | |
*** esker has joined #openstack-swift | 10:37 | |
*** SkyRocknRoll has quit IRC | 10:39 | |
*** esker has quit IRC | 10:42 | |
d0ugal | kota_: FYI, if you are curious, the error I had seemed to be related to versioned containers (and me probably using them wrongly) | 10:45 |
*** suresh has joined #openstack-swift | 10:46 | |
suresh | mattoliverau: Thnqq for your answer | 10:49 |
*** ekarlso has quit IRC | 10:51 | |
*** ekarlso has joined #openstack-swift | 10:51 | |
*** dmorita has joined #openstack-swift | 10:54 | |
*** dmorita has quit IRC | 10:58 | |
*** jordanP has quit IRC | 10:59 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-swift | 11:01 | |
*** haomaiwang has quit IRC | 11:14 | |
openstackgerrit | Alistair Coles proposed openstack/swift: Fix rst errors so that html docs are complete https://review.openstack.org/259388 | 11:26 |
acoles | notmyname: ^^ more disappearing doc | 11:32 |
*** yatin has quit IRC | 11:35 | |
*** sanchitmalhotra has quit IRC | 11:36 | |
*** suresh has quit IRC | 11:40 | |
*** esker has joined #openstack-swift | 11:49 | |
*** links has quit IRC | 11:50 | |
*** silor has joined #openstack-swift | 11:50 | |
*** esker has quit IRC | 11:54 | |
*** links has joined #openstack-swift | 12:06 | |
*** Jeffrey4l has quit IRC | 12:06 | |
*** links has quit IRC | 12:16 | |
openstackgerrit | Bing Hu proposed openstack/python-swiftclient: Support uploading to an object in swift from stdin Closes-bug: #1521342 https://review.openstack.org/259410 | 12:22 |
openstack | bug 1521342 in python-swiftclient "support uploading from a stdin stream" [Wishlist,In progress] https://launchpad.net/bugs/1521342 - Assigned to Bing Hu (hubingsh) | 12:22 |
*** esker has joined #openstack-swift | 12:25 | |
*** esker has quit IRC | 12:30 | |
*** venkat has quit IRC | 12:33 | |
openstackgerrit | Bing Hu proposed openstack/python-swiftclient: Support uploading to an object in swift from stdin Closes-bug: #1521342 https://review.openstack.org/259410 | 12:38 |
openstack | bug 1521342 in python-swiftclient "support uploading from a stdin stream" [Wishlist,In progress] https://launchpad.net/bugs/1521342 - Assigned to Bing Hu (hubingsh) | 12:38 |
*** SkyRocknRoll has joined #openstack-swift | 12:38 | |
*** venkat has joined #openstack-swift | 12:44 | |
*** lyrrad has joined #openstack-swift | 12:45 | |
*** ____bb has joined #openstack-swift | 12:49 | |
*** ____bb has quit IRC | 12:52 | |
*** fesp has joined #openstack-swift | 12:59 | |
*** odsail has joined #openstack-swift | 13:00 | |
*** odsail has quit IRC | 13:00 | |
*** esker has joined #openstack-swift | 13:01 | |
*** fesp has quit IRC | 13:03 | |
*** esker has quit IRC | 13:06 | |
*** jordanP has joined #openstack-swift | 13:15 | |
*** links has joined #openstack-swift | 13:25 | |
*** venkat has quit IRC | 13:25 | |
*** thumpba has joined #openstack-swift | 13:32 | |
*** breitz has quit IRC | 13:33 | |
*** breitz has joined #openstack-swift | 13:34 | |
*** esker has joined #openstack-swift | 13:37 | |
*** esker has quit IRC | 13:41 | |
*** haomaiwa_ has joined #openstack-swift | 13:46 | |
*** links has quit IRC | 13:54 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/swift: Updated from global requirements https://review.openstack.org/88736 | 13:54 |
*** ppai has quit IRC | 13:55 | |
*** haomaiwa_ has quit IRC | 14:01 | |
*** haomaiwang has joined #openstack-swift | 14:01 | |
*** dustins|out is now known as dustins | 14:05 | |
*** robefran has joined #openstack-swift | 14:07 | |
*** Jeffrey4l has joined #openstack-swift | 14:11 | |
*** esker has joined #openstack-swift | 14:13 | |
*** esker has quit IRC | 14:18 | |
*** yatin has joined #openstack-swift | 14:25 | |
*** badari has joined #openstack-swift | 14:26 | |
*** dslevin has quit IRC | 14:36 | |
*** chsc has joined #openstack-swift | 14:43 | |
*** trifon has joined #openstack-swift | 14:43 | |
*** blmartin has joined #openstack-swift | 14:48 | |
*** esker has joined #openstack-swift | 14:49 | |
*** yatin has quit IRC | 14:52 | |
*** esker has quit IRC | 14:53 | |
*** yatin has joined #openstack-swift | 14:55 | |
*** dmorita has joined #openstack-swift | 14:56 | |
*** chsc has quit IRC | 14:57 | |
*** haomaiwang has quit IRC | 15:01 | |
*** dmorita has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-swift | 15:01 | |
*** chsc has joined #openstack-swift | 15:02 | |
*** yatin has quit IRC | 15:04 | |
*** lpabon has joined #openstack-swift | 15:05 | |
*** david-lyle has quit IRC | 15:08 | |
*** rledisez1 has joined #openstack-swift | 15:21 | |
*** esker has joined #openstack-swift | 15:22 | |
*** esker has quit IRC | 15:24 | |
*** esker has joined #openstack-swift | 15:24 | |
*** tsg has joined #openstack-swift | 15:34 | |
*** chsc has quit IRC | 15:35 | |
*** [1]eranrom has joined #openstack-swift | 15:36 | |
*** eranrom has quit IRC | 15:39 | |
*** [1]eranrom is now known as eranrom | 15:39 | |
*** janonymous_ has joined #openstack-swift | 15:44 | |
acoles | peluse: might you get chance to look at patch 231121 one more time before holidays? | 15:45 |
patchbot | acoles: https://review.openstack.org/#/c/231121/ - Make ECDiskFile report all fragments found on disk | 15:45 |
openstackgerrit | Bill Huber proposed openstack/swift: Update Erasure Coding Overview doc to remove Beta version https://review.openstack.org/248210 | 15:46 |
*** SkyRocknRoll has quit IRC | 15:48 | |
*** minwoob has joined #openstack-swift | 15:49 | |
peluse | acoles: yup, will do | 15:50 |
acoles | peluse: thanks! | 15:50 |
gmmaha | kota_: thanks for the comment. Let me run the setup on a new box and confirm we dont need it. | 15:52 |
*** klrmn has joined #openstack-swift | 15:54 | |
janonymous_ | Please review these patches: https://review.openstack.org/#/c/208222/ , https://review.openstack.org/#/c/227855/ ,https://review.openstack.org/#/c/254276/ | 15:54 |
*** chinaer has joined #openstack-swift | 15:59 | |
*** lcurtis has joined #openstack-swift | 15:59 | |
*** haomaiwang has quit IRC | 16:01 | |
*** haomaiwang has joined #openstack-swift | 16:01 | |
*** blmartin has quit IRC | 16:06 | |
peluse | WTF, when did gerrit change? I can't even find the button to copy a line to pull down a specific patch?? | 16:07 |
*** blmartin has joined #openstack-swift | 16:07 | |
onovy | wednesday? | 16:09 |
janonymous_ | hahaa, on the right top corner under Download i think. | 16:09 |
onovy | yep, Download | 16:09 |
onovy | top right | 16:09 |
peluse | heh, thanks guys :) I see it now | 16:09 |
onovy | but it's terrible. for example reply button is on top of page? so you want to reply do last comment ... scroll to top to hit Reply -> PERFECT! | 16:10 |
acoles | onovy: peluse: somebody needs to start a "give-me-back-the-gerrit-i-loved" petition :) | 16:12 |
onovy | :)) | 16:12 |
peluse | acoles: totally agree! I don't think this new i/f. it reminds me of windows 10 or something | 16:12 |
acoles | lol | 16:12 |
*** esker has quit IRC | 16:12 | |
peluse | yeah, the ultimate insult! | 16:13 |
acoles | peluse: it reminds me of the kind of UI that a software engineer would design | 16:13 |
peluse | LOL | 16:14 |
janonymous_ | Also there is no way to see dependent patches, i donno | 16:14 |
*** rcernin has quit IRC | 16:15 | |
peluse | yeah., what was wrong with the old UI anyway? maaaaaan.... | 16:15 |
acoles | peluse: anyway, if its *my* patch you are looking at, hit keyboard shortcut 'a', +2, workflow +1 :P :P | 16:15 |
acoles | janonymous_: top right, related patches == dependent | 16:15 |
peluse | acoles: I shall follow those directions precisely :) | 16:15 |
tdasilva | janonymous_: just a question on patch 208222, is there a way to catch 'warn' being used by developers, maybe during pep8 test or something? | 16:16 |
patchbot | tdasilva: https://review.openstack.org/#/c/208222/ - Python 3 deprecated the logger.warn method in favo... | 16:16 |
acoles | peluse: that was totally a joke!!! | 16:16 |
peluse | heh, no doubt acoles :) | 16:16 |
acoles | c'mon, i'm not that brazen | 16:16 |
janonymous_ | acoles: Ohh.. :) | 16:16 |
janonymous_ | tdasilva: Updating the fake logger does this i guess. | 16:17 |
tdasilva | janonymous: but only for tests, no? | 16:18 |
janonymous_ | No, updating fake logger tells when warn is used in actual code also if the func is covered. | 16:19 |
onovy | why is this not done in pep8/hacking like assertEqual(s)? | 16:20 |
*** diazjf has joined #openstack-swift | 16:20 | |
janonymous_ | onovy: there is no best way to do so, but it was probably one of the better options. | 16:22 |
janonymous_ | you could try using warn in code and run unittests if you want | 16:22 |
janonymous_ | it would blow up a warning to user saying to use warning | 16:23 |
onovy | best way is to put it in hacking imho :) | 16:25 |
*** arnox has left #openstack-swift | 16:25 | |
*** minwoob has quit IRC | 16:26 | |
janonymous_ | onovy: this patch is already pending from 5 months now i guess :( | 16:27 |
onovy | ah... | 16:27 |
*** zhill has joined #openstack-swift | 16:28 | |
onovy | janonymous_, link? can't found it | 16:29 |
janonymous_ | onovy: just above, tdasilva shared it. | 16:30 |
janonymous_ | patch 208222 | 16:30 |
patchbot | janonymous_: https://review.openstack.org/#/c/208222/ - Python 3 deprecated the logger.warn method in favo... | 16:30 |
janonymous_ | tdasilva: can i help ? | 16:30 |
onovy | ehm. you don't understand me | 16:30 |
onovy | http://github.com/openstack-dev/hacking // this hacking | 16:30 |
onovy | https://github.com/openstack-dev/hacking/blob/master/hacking/checks/python23.py#L106 // here it checks assertEquals | 16:31 |
tdasilva | janonymous: basically I had the same question as onovy, why not add to hacking? | 16:31 |
*** chinaer has quit IRC | 16:31 | |
tdasilva | can it be done? | 16:31 |
onovy | and my question is: why not add this same check for logger.warn | 16:31 |
onovy | i think it can be done (simple) | 16:31 |
janonymous_ | okay, no problem then | 16:31 |
onovy | hacking is for this checks :) | 16:32 |
janonymous_ | but i have to search a bit on how to add it | 16:33 |
onovy | np. send me a review for hacking i will take a look | 16:34 |
onovy | (when you will have it of course :) | 16:35 |
acoles | notmyname: reviews with a +2 and a -1 do not appear anywhere on the review dashboard, I think? e.g. patch 259165 ;) | 16:38 |
patchbot | acoles: https://review.openstack.org/#/c/259165/ - remove beta label from EC | 16:38 |
janonymous_ | So, i would push up a change in hacking repo , but does that makes this patch dependent on that ? | 16:39 |
peluse | WTF I commented -1 on patch 259165 and its not there anymore | 16:41 |
patchbot | peluse: https://review.openstack.org/#/c/259165/ - remove beta label from EC | 16:41 |
onovy | janonymous_, hacking can be patched and released very fast | 16:41 |
peluse | oh wait, I commented on patch 248210 | 16:41 |
patchbot | peluse: https://review.openstack.org/#/c/248210/ - Update Erasure Coding Overview doc to remove Beta ... | 16:41 |
*** petertr7_away is now known as petertr7 | 16:42 | |
janonymous_ | tdasilva: please let me know what do u think.. | 16:43 |
tdasilva | janonymous: i like the idea of adding to hacking | 16:45 |
janonymous_ | sure then, i would update hacking repo :) | 16:45 |
*** jistr has quit IRC | 16:49 | |
*** petertr7 is now known as petertr7_away | 16:52 | |
*** dmorita has joined #openstack-swift | 16:57 | |
notmyname | good morning | 16:59 |
notmyname | acoles: yeah, I need to revisit the dashboards after the gerrit upgrade | 17:00 |
janonymous_ | notmyname: o/ | 17:00 |
*** haomaiwang has quit IRC | 17:01 | |
*** nadeem has joined #openstack-swift | 17:01 | |
*** dmorita has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-swift | 17:01 | |
acoles | notmyname: my dashboard broke :/ i was missing some label: prefixes before Workflow and Verified. | 17:01 |
*** jordanP has quit IRC | 17:03 | |
*** klrmn has quit IRC | 17:03 | |
*** zhill has quit IRC | 17:05 | |
*** gyee has joined #openstack-swift | 17:06 | |
*** yatin has joined #openstack-swift | 17:09 | |
*** robefran has quit IRC | 17:15 | |
*** nadeem has quit IRC | 17:16 | |
*** nadeem has joined #openstack-swift | 17:17 | |
openstackgerrit | Bill Huber proposed openstack/swift: Update Erasure Coding Overview doc to remove Beta version https://review.openstack.org/248210 | 17:18 |
notmyname | acoles: thanks for the doc updates. approved | 17:20 |
acoles | notmyname: is weird, build_sphinx spews out errors but then reports success, so gate passes | 17:21 |
notmyname | wbhuber: re patch 248210 | 17:24 |
patchbot | notmyname: https://review.openstack.org/#/c/248210/ - Update Erasure Coding Overview doc to remove Beta ... | 17:24 |
notmyname | wbhuber: I agree with what acoles said on patch 259165 | 17:24 |
patchbot | notmyname: https://review.openstack.org/#/c/259165/ - remove beta label from EC | 17:24 |
*** rohit_ has joined #openstack-swift | 17:24 | |
notmyname | wbhuber: don't try to invent something between beta and prod, and don't make forward-looking statements. just leave it as "you can use it" | 17:25 |
notmyname | wbhuber: I definitely think yours is better than mine because (1) yours came first and (2) yours has the change to the ssync docs | 17:25 |
notmyname | wbhuber: what would you think about dropping more of the words at the top of the EC doc? | 17:26 |
openstackgerrit | Merged openstack/swift: Not used parameter https://review.openstack.org/254626 | 17:26 |
wbhuber | notmyname: you got it. | 17:27 |
wbhuber | notmyname: will remove the entire section at the top of the EC doc now and resubmit a patch in a few. | 17:27 |
notmyname | wbhuber: hey, that's, just, like, my opioning, man | 17:28 |
wbhuber | notmyname: i think that makes more sense. your, peluse's and alistar's opinions are more than just mine. gotta go with the majority, right? | 17:29 |
notmyname | wbhuber: I abandoned my patch | 17:30 |
wbhuber | notmyname: nifty. | 17:30 |
notmyname | wbhuber: but another patch set on yours, and I'll +2 | 17:30 |
openstackgerrit | Bill Huber proposed openstack/swift: Update Erasure Coding Overview doc to remove Beta version https://review.openstack.org/248210 | 17:31 |
notmyname | wbhuber: boom. there it is :-) | 17:32 |
notmyname | acoles: could you take 60 seconds and look at that one too, please? | 17:33 |
*** zhill has joined #openstack-swift | 17:34 | |
acoles | looking | 17:35 |
acoles | wait, this is whuber's? | 17:35 |
wbhuber | yes, wbhuber. | 17:36 |
notmyname | yeah | 17:36 |
wbhuber | ps://review.openstack.org/#/c/248210/ - | 17:36 |
clayg | notmyname: I'm sure acoles thinks his fixes for EC should go in before we take off the beta label | 17:36 |
*** trifon has quit IRC | 17:37 | |
clayg | acoles: well not sure, you care about the label - but sure you want those patches reviewed ;) | 17:37 |
*** zhill has quit IRC | 17:38 | |
acoles | wbhuber: got it, thanks | 17:40 |
acoles | clayg: sure I do :) | 17:40 |
acoles | clayg: but i thought about it and i don't think there is anything *broken* that they fix, just performance improvements | 17:41 |
acoles | clayg: (although its been a while since i worked on them) | 17:41 |
acoles | clayg: actually, i stumbled across a nice corner case while sorting out the mismatched suffix hashing that you spotted with the fast-post patch | 17:42 |
openstackgerrit | Bill Huber proposed openstack/swift: Update Erasure Coding Overview doc to remove Beta version https://review.openstack.org/248210 | 17:43 |
acoles | i fixed the hashing to deal with the differing meta filename combinations, added some assertions to probe tests, and... boom... | 17:43 |
wbhuber | acoles: i like your object-server.conf sample recommendation. made a single change to the newest patch. | 17:43 |
wbhuber | notmyname: that means you're needed to check it out once again. | 17:44 |
notmyname | done | 17:44 |
acoles | clayg: ssync will not reconcile a node with (t2.meta, t0.ts) and another with just (t0.ts) ... and I *think* that could happen with failing nodes, handoffs taking PUTs etc | 17:45 |
*** yatin has quit IRC | 17:46 | |
*** diogogmt has joined #openstack-swift | 17:47 | |
*** yatin has joined #openstack-swift | 17:48 | |
peterlisak | hello, is someone available to review two small patches: https://review.openstack.org/#/c/253037/ and https://review.openstack.org/#/c/253038/ | 17:49 |
*** trifon has joined #openstack-swift | 17:53 | |
*** david-lyle has joined #openstack-swift | 17:54 | |
*** zhill has joined #openstack-swift | 17:54 | |
*** rledisez has quit IRC | 17:55 | |
*** robefran has joined #openstack-swift | 17:56 | |
*** david-lyle has quit IRC | 17:59 | |
*** david-lyle has joined #openstack-swift | 18:00 | |
*** haomaiwang has quit IRC | 18:01 | |
*** haomaiwang has joined #openstack-swift | 18:01 | |
*** diogogmt has quit IRC | 18:02 | |
*** diogogmt has joined #openstack-swift | 18:03 | |
*** rcernin has joined #openstack-swift | 18:05 | |
openstackgerrit | Merged openstack/swift: Fix rst errors so that html docs are complete https://review.openstack.org/259388 | 18:10 |
*** klrmn has joined #openstack-swift | 18:11 | |
*** yatin has quit IRC | 18:19 | |
*** dmorita has joined #openstack-swift | 18:24 | |
*** dmorita has quit IRC | 18:25 | |
*** diogogmt has quit IRC | 18:25 | |
*** dmorita_ has joined #openstack-swift | 18:25 | |
*** zaitcev has joined #openstack-swift | 18:26 | |
*** ChanServ sets mode: +v zaitcev | 18:26 | |
notmyname | clayg: peluse: I'd love your opinion (in gerrit, as a review comment) on patch 248210 :-) | 18:29 |
patchbot | notmyname: https://review.openstack.org/#/c/248210/ - Update Erasure Coding Overview doc to remove Beta ... | 18:29 |
peluse | notmyname: OK, on a call but will take a look when I'm done | 18:29 |
*** HenryG is now known as HenryG_afk | 18:29 | |
*** rcernin has quit IRC | 18:36 | |
*** janonymous_ has quit IRC | 18:37 | |
*** geaaru has quit IRC | 18:38 | |
*** diogogmt has joined #openstack-swift | 18:41 | |
*** aix has quit IRC | 18:56 | |
*** haomaiwang has quit IRC | 19:01 | |
*** haomaiwang has joined #openstack-swift | 19:01 | |
acoles | good night | 19:03 |
*** diazjf has quit IRC | 19:04 | |
*** acoles is now known as acoles_ | 19:06 | |
*** nadeem has quit IRC | 19:07 | |
flaper87 | notmyname: (et al) https://review.openstack.org/#/c/259168/ | 19:16 |
*** openstackgerrit has quit IRC | 19:17 | |
*** openstackgerrit has joined #openstack-swift | 19:18 | |
*** nadeem has joined #openstack-swift | 19:24 | |
*** zaitcev has quit IRC | 19:30 | |
*** nadeem has quit IRC | 19:35 | |
*** zaitcev has joined #openstack-swift | 19:37 | |
*** ChanServ sets mode: +v zaitcev | 19:37 | |
*** aix has joined #openstack-swift | 19:38 | |
*** HenryG_afk is now known as HenryG | 19:42 | |
*** diazjf has joined #openstack-swift | 19:49 | |
*** zaitcev has quit IRC | 19:55 | |
*** haomaiwang has quit IRC | 20:01 | |
*** haomaiwang has joined #openstack-swift | 20:01 | |
*** esker has joined #openstack-swift | 20:01 | |
*** esker has joined #openstack-swift | 20:02 | |
*** petertr7_away is now known as petertr7 | 20:02 | |
*** esker has quit IRC | 20:02 | |
*** esker has joined #openstack-swift | 20:03 | |
*** zaitcev has joined #openstack-swift | 20:09 | |
*** ChanServ sets mode: +v zaitcev | 20:09 | |
Zyric | Morning | 20:13 |
*** torgomatic has joined #openstack-swift | 20:15 | |
*** ChanServ sets mode: +v torgomatic | 20:15 | |
notmyname | Zyric: hello | 20:15 |
Zyric | notmyname: I had another look at patch 241978 and changing the primary name of a policy with only one name to something new doesn't appear to remove the old name or cause errors | 20:17 |
patchbot | Zyric: https://review.openstack.org/#/c/241978/ - Add support for storage policies to have more than... | 20:17 |
Zyric | Because names are only removed if the new name isn't equal to policy.name (which is the same as the first element) and also is in the .alias_list | 20:19 |
notmyname | Zyric: I'll have to look again (and this time run some tests) | 20:19 |
*** jamielennox is now known as jamielennox|away | 20:21 | |
*** lyrrad has quit IRC | 20:21 | |
*** esker has quit IRC | 20:22 | |
*** zaitcev has quit IRC | 20:25 | |
*** robefran has quit IRC | 20:25 | |
*** zaitcev has joined #openstack-swift | 20:36 | |
*** ChanServ sets mode: +v zaitcev | 20:36 | |
notmyname | we made an infographic https://swiftstack.com/blog/2015/12/18/five-years-working-on-swift/ | 20:36 |
*** zhill has quit IRC | 20:40 | |
onovy | notmyname, nice! | 20:44 |
onovy | what metric did you choose for stackalytics graph? loc, commits, ...? | 20:44 |
notmyname | onovy: IIRC it's http://stackalytics.com/?release=all&module=swift-group&metric=person-day | 20:45 |
onovy | ah, effort. thanks | 20:45 |
notmyname | the person-day effort is the only one there that might be good to actually track. I don't want to encourage anyone to get commits or comments on lines of code just to go up in that ranking! | 20:46 |
onovy | of course | 20:46 |
*** dmorita_ has quit IRC | 20:47 | |
pdardeau | notmyname: very nice! | 20:47 |
onovy | seznam.cz 14th :), but in Mitaka only: 4th place | 20:47 |
*** petertr7 is now known as petertr7_away | 20:48 | |
notmyname | onovy: :-) | 20:48 |
pdardeau | notmyname: either contributors or contributers is spelled incorrectly. not sure which. i used to be ok at speling ;-) | 20:50 |
notmyname | lol | 20:50 |
clayg | not sure how to submit patches to blog images :\ | 20:51 |
notmyname | I do. new versions should be up momentarily ;-) | 20:52 |
notmyname | process involves me walking over to the design person in marketing. very manual. no scripting or automation at all | 20:52 |
notmyname | pdardeau: http://i.imgur.com/Z65ypzU.jpg | 20:54 |
*** dmorita has joined #openstack-swift | 20:55 | |
pdardeau | notmyname: lol. sweet! | 20:55 |
notmyname | pdardeau: it's fixed now. hit reload | 20:55 |
*** petertr7_away is now known as petertr7 | 20:56 | |
*** zhill has joined #openstack-swift | 20:57 | |
pdardeau | notmyname: i saw it! very nice. swank! ;) | 20:57 |
*** robefran has joined #openstack-swift | 20:58 | |
*** haomaiwang has quit IRC | 21:01 | |
*** silor has quit IRC | 21:01 | |
*** gyee has quit IRC | 21:01 | |
*** 17WABC36L has joined #openstack-swift | 21:01 | |
onovy | https://review.openstack.org/#/c/257066/4 do we need reply from some else? swauth is ok. | 21:02 |
notmyname | onovy: thanks | 21:04 |
onovy | :) | 21:04 |
blmartin | Oh, onovy, I answered your question about the force option on the review page | 21:04 |
onovy | blmartin, yep. you are right, my arguments are invalid :) | 21:05 |
blmartin | well not invalid so much as a different take on the matter. I think it is valid and something that may or may not be wanted. I'm not really sure | 21:06 |
onovy | notmyname, exactly i didn't fix it in swauth. gholt already did it few months/years ago :) | 21:06 |
onovy | blmartin, your solution is compatible with original approach. so go this way and maybe change it later in another commit? | 21:07 |
blmartin | That sounds fair to me | 21:07 |
*** blmartin has left #openstack-swift | 21:18 | |
*** dmorita has quit IRC | 21:24 | |
*** changbl has quit IRC | 21:26 | |
*** nadeem has joined #openstack-swift | 21:30 | |
*** nadeem has quit IRC | 21:31 | |
*** nadeem has joined #openstack-swift | 21:31 | |
*** gyee has joined #openstack-swift | 21:32 | |
*** diogogmt has quit IRC | 21:32 | |
*** diogogmt has joined #openstack-swift | 21:33 | |
*** changbl has joined #openstack-swift | 21:39 | |
*** petertr7 is now known as petertr7_away | 21:46 | |
*** dmorita has joined #openstack-swift | 21:49 | |
notmyname | onovy: still here? | 21:49 |
onovy | yes | 21:50 |
notmyname | I think I found what I was looking for. patch 229532 | 21:50 |
patchbot | notmyname: https://review.openstack.org/#/c/229532/ - swift-init: SIGKILL daemon after kill_wait period. | 21:51 |
onovy | but 10:50 pm here :) | 21:51 |
notmyname | heh | 21:51 |
onovy | perfect, what you found? | 21:51 |
notmyname | onovy: I was thinking about the change a little more, and I don't want to change the current behavior, I don't think. ie the really-kill thing is good. but not with the same command as the existing one | 21:52 |
*** trifon has quit IRC | 21:52 | |
notmyname | so I was bringing it back up to rememeber which way you had implemented it | 21:52 |
notmyname | and to leave that comment if it was implemented as a kill-after-timeout on the existing command | 21:52 |
onovy | so add option --kill-after-timeout ? | 21:53 |
notmyname | yeah, something like that | 21:53 |
onovy | i'm ok with this option. i will use it inside debian init scripts and everything will be perfect | 21:53 |
notmyname | right. and that way it won't impact existing tools and workflows, and it also avoids questions about graceful restarts | 21:54 |
onovy | perfect, will you write it to gerrit? | 21:55 |
notmyname | onovy: just brainstorming here, what about a `swift-init <server> stop --kill-after-timeout=30` version? then swift-init passes that timeout to a manager, very similarly to how you're doing it now | 21:55 |
notmyname | I don't know if it's good to expose that timeout or not | 21:55 |
onovy | timeout can be changed now | 21:55 |
onovy | --timeout param i think | 21:56 |
notmyname | ah ok | 21:56 |
onovy | two timeouts are useless :) | 21:56 |
notmyname | agreed | 21:56 |
onovy | this option will work with: stop, restart, reload, gracefull-stop, right? | 21:56 |
onovy | and in all commands will kill process __group__ after timeout? | 21:56 |
onovy | gracefull-stop = shutdown | 21:57 |
notmyname | maybe not reload. that's just supposed to reload the configs, right | 21:57 |
onovy | reload = shutdown + start | 21:57 |
notmyname | (It's been a while since I looked at this part. I could be forgetting something) | 21:57 |
onovy | reload = stop (gracefull) + start | 21:57 |
notmyname | ah, right. reload = graceful stop + start, right? | 21:57 |
notmyname | right | 21:57 |
notmyname | and restart = kill + start | 21:57 |
onovy | yes | 21:58 |
onovy | shutdown = stop(gracefull) | 21:58 |
onovy | kill=stop | 21:58 |
notmyname | right | 21:58 |
clayg | notmyname: well sigterm | 21:58 |
notmyname | ok, thanks. I'll leave a comment in gerrit | 21:58 |
clayg | shutdown is HUP | 21:58 |
onovy | stop=sigterm, stop(gracefull)=sighup | 21:58 |
*** 17WABC36L has quit IRC | 22:01 | |
notmyname | comment left | 22:01 |
*** haomaiwang has joined #openstack-swift | 22:01 | |
openstackgerrit | Merged openstack/swift: Update Erasure Coding Overview doc to remove Beta version https://review.openstack.org/248210 | 22:02 |
onovy | notmyname, thank you! | 22:03 |
openstackgerrit | Merged openstack/swift: Deprecated param timeout removed from memcached https://review.openstack.org/257066 | 22:04 |
openstackgerrit | Merged openstack/swift: Fix incorrect kwarg in auth middleware example https://review.openstack.org/258973 | 22:04 |
notmyname | whoa. flurry of merges | 22:05 |
*** gyee has quit IRC | 22:06 | |
*** petertr7_away is now known as petertr7 | 22:10 | |
*** petertr7 is now known as petertr7_away | 22:21 | |
*** badari has quit IRC | 22:28 | |
notmyname | #success removed the beta tag on the swift erasure code docs | 22:33 |
openstackstatus | notmyname: Added success to Success page | 22:33 |
*** nadeem has quit IRC | 22:36 | |
*** rcernin has joined #openstack-swift | 22:52 | |
*** lcurtis has quit IRC | 22:54 | |
openstackgerrit | Ondřej Nový proposed openstack/swift: swift-init: New option kill-after-timeout https://review.openstack.org/229532 | 22:57 |
onovy | and now i'm going to bed :) | 22:58 |
*** lpabon has quit IRC | 22:58 | |
*** haomaiwang has quit IRC | 23:01 | |
*** haomaiwang has joined #openstack-swift | 23:01 | |
*** trifon has joined #openstack-swift | 23:01 | |
*** rcernin has quit IRC | 23:08 | |
*** rohit_ has quit IRC | 23:10 | |
*** ianbrown has quit IRC | 23:12 | |
*** ianbrown has joined #openstack-swift | 23:12 | |
*** diazjf has quit IRC | 23:13 | |
*** SorinSen has quit IRC | 23:17 | |
*** mac_ified has joined #openstack-swift | 23:18 | |
then3rd | Good afternoon, all. | 23:18 |
notmyname | hello | 23:24 |
notmyname | Zyric: yup, I agree and have understand better on patch 241978 | 23:25 |
patchbot | notmyname: https://review.openstack.org/#/c/241978/ - Add support for storage policies to have more than... | 23:25 |
then3rd | Howdy. I have another general question I've been thinking about. | 23:25 |
Zyric | notmyname: Neat, thank you! | 23:25 |
then3rd | If I want to design a lean [hardware] cluster, is it common to combine roles/tiers? Here's sort of what I have in mind. Two types of proxy nodes. First, a combination haproxy/swift-proxy node. Simply add more of these if a region needs to handle more traffic; haproxy distributes load to backend swift proxies via the 10G network. The other type of proxy node would still run haproxy/swift in addition | 23:27 |
then3rd | to keystone, nova, horizon and supporting services. Currently I can't imagine needing more than three or four proxies total. I don't want to "waste" hardware by having standalone controller nodes. 2+ required. Storage nodes would only need to be powerful enough to handle the storagere/replications services. | 23:27 |
notmyname | then3rd: right | 23:28 |
notmyname | then3rd: there's generally 3 different SKUs you can end up with in a swift cluster | 23:28 |
notmyname | I know I've written this down before. let me see if I can find it | 23:28 |
then3rd | Oh, excellent. | 23:28 |
notmyname | then3rd: this should help https://swiftstack.com/docs/admin/hardware.html | 23:29 |
*** nadeem has joined #openstack-swift | 23:30 | |
then3rd | Since I didn't do the initial architechture, I haven't considered hardware until now. | 23:30 |
then3rd | That was a helpful article. I have a feeling some of my hardware isn't enough and some is overkill. | 23:40 |
notmyname | then3rd: swift is pretty flexible. note that the swiftstack docs will push to opinionated recommendations, both to ensure success and to keep our support burden lower | 23:41 |
notmyname | then3rd: you can see that what you have now is working. and that's great! | 23:42 |
*** nadeem has quit IRC | 23:42 | |
then3rd | Is it unusual to run haproxy on the same nodes as swift proxy? The vip would only be active on the primary, and balacnes between localhost and the other swift proxies via the cluster network. | 23:43 |
then3rd | Because that's how it was built. :D | 23:44 |
notmyname | no, that's fine | 23:44 |
then3rd | As long as I don't run out of bandwidth/cpu/memory. XD | 23:44 |
* clayg considers the set of things possible when removing the constraint of running out of bandwidth/cpu/memory | 23:47 | |
*** dmorita has quit IRC | 23:47 | |
notmyname | clayg: "everything", I think | 23:48 |
then3rd | heh. How large are the clusters some of you guys operate? | 23:50 |
notmyname | (technically, it's the customers who are operating them with tools we write) all sizes. everything from a few dozen TB to many PB | 23:52 |
*** dmorita has joined #openstack-swift | 23:55 | |
Zyric | notmyname: I have an account watcher mostly working now, just finishing tests - but there's a lot of duplicate code that probably could be shared with the object watcher. | 23:56 |
Zyric | Should I attempt to unify the two or leave them separate for now, maybe leave that for another patch? | 23:56 |
notmyname | Zyric: I'd suggest leaving it separate for now, but expect that the patch that lands will have them unified | 23:56 |
notmyname | Zyric: ie expect torgomatic's patch to land and you'll be able to use what's in there (or refactor to a common place) | 23:57 |
notmyname | Zyric: point is, don't get distracted by the perfect design instead of making progress towards solving the problem | 23:57 |
Zyric | notmyname: Cool, that's similar to what I was thinking. | 23:58 |
Zyric | At the moment all it does is collect the account paths, is there better/other data that would be good to collect? | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!