*** vinsh has quit IRC | 01:00 | |
*** ChubYann has quit IRC | 01:07 | |
*** ChubYann has joined #openstack-swift | 01:08 | |
*** vinsh has joined #openstack-swift | 01:09 | |
*** MooingLemur has joined #openstack-swift | 01:10 | |
*** vinsh has quit IRC | 01:13 | |
kota_ | good morning | 02:09 |
---|---|---|
mattoliverau | kota_ morning | 02:16 |
kota_ | mattoliverau: hello | 02:24 |
*** winggundamth has joined #openstack-swift | 02:40 | |
*** Jeffrey4l has quit IRC | 02:51 | |
*** sudorandom has quit IRC | 02:56 | |
*** ntata has quit IRC | 02:56 | |
*** sudorandom has joined #openstack-swift | 02:56 | |
*** vinsh has joined #openstack-swift | 03:09 | |
*** hugokuo has quit IRC | 03:12 | |
*** vinsh has quit IRC | 03:14 | |
*** hugokuo has joined #openstack-swift | 03:17 | |
*** sanchitmalhotra has joined #openstack-swift | 03:23 | |
*** sanchitmalhotra has quit IRC | 03:28 | |
*** tqtran has joined #openstack-swift | 03:35 | |
*** tqtran has quit IRC | 03:39 | |
*** links has joined #openstack-swift | 03:46 | |
charz_ | mattoliverau: kota_ morning | 03:56 |
mattoliverau | charz_: morning | 03:56 |
*** links has quit IRC | 04:17 | |
*** Jeffrey4l has joined #openstack-swift | 04:42 | |
*** Jeffrey4l has quit IRC | 04:43 | |
*** Jeffrey4l has joined #openstack-swift | 04:44 | |
*** MooingLemur has quit IRC | 05:06 | |
*** MooingLemur has joined #openstack-swift | 05:12 | |
*** MooingLemur has quit IRC | 05:14 | |
*** MooingLemur has joined #openstack-swift | 05:16 | |
*** vinsh has joined #openstack-swift | 05:16 | |
*** ppai has joined #openstack-swift | 05:35 | |
*** MooingLemur has quit IRC | 05:39 | |
*** MooingLemur has joined #openstack-swift | 05:40 | |
*** vinsh has quit IRC | 05:53 | |
*** takashi_ has joined #openstack-swift | 06:03 | |
lifeless | notmyname: to get -x behaviour you need the backend runner to support that today; we could in principle use that to cancel backends once we see a failure - file a wishlist ticket | 06:07 |
lifeless | notmyname: SkipTest is totally backend specific - the subunit protocol defines skips semantically, so as long as the backend emits valid subunit there you're golden | 06:07 |
*** ChubYann has quit IRC | 06:57 | |
*** sams-gleb has joined #openstack-swift | 07:11 | |
*** pcaruana has joined #openstack-swift | 07:17 | |
*** tqtran has joined #openstack-swift | 07:35 | |
*** tqtran has quit IRC | 07:35 | |
*** silor has joined #openstack-swift | 08:02 | |
*** oshritf has joined #openstack-swift | 08:06 | |
*** hseipp has joined #openstack-swift | 08:06 | |
*** oshritf has quit IRC | 08:06 | |
*** silor1 has joined #openstack-swift | 08:07 | |
*** silor has quit IRC | 08:08 | |
*** silor1 is now known as silor | 08:08 | |
*** sams-gleb has quit IRC | 08:09 | |
*** sams-gleb has joined #openstack-swift | 08:09 | |
*** oshritf has joined #openstack-swift | 08:10 | |
*** sams-gle_ has joined #openstack-swift | 08:12 | |
*** sams-gleb has quit IRC | 08:12 | |
*** oshritf has quit IRC | 08:14 | |
*** rledisez has joined #openstack-swift | 08:24 | |
*** amoralej|off is now known as amoralej | 08:27 | |
*** sams-gle_ has quit IRC | 08:31 | |
*** sams-gleb has joined #openstack-swift | 08:32 | |
*** sams-gleb has quit IRC | 08:36 | |
*** ahale has quit IRC | 08:44 | |
*** ppai_ has joined #openstack-swift | 08:45 | |
*** ppai has quit IRC | 08:45 | |
*** cbartz has joined #openstack-swift | 08:46 | |
*** geaaru has joined #openstack-swift | 08:47 | |
*** oshritf has joined #openstack-swift | 08:47 | |
*** joeljwright has joined #openstack-swift | 08:48 | |
*** ChanServ sets mode: +v joeljwright | 08:48 | |
*** mahatic_ has joined #openstack-swift | 08:50 | |
*** mahatic has quit IRC | 08:51 | |
*** oshritf has quit IRC | 08:51 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/swift-bench: Replace assertEquals with assertEqual https://review.openstack.org/400126 | 08:59 |
*** sanchitmalhotra has joined #openstack-swift | 09:02 | |
*** sanchitmalhotra has quit IRC | 09:03 | |
*** sanchitmalhotra has joined #openstack-swift | 09:03 | |
*** ahale has joined #openstack-swift | 09:08 | |
*** ahale has quit IRC | 09:13 | |
onovy | cschwede: hi, are you there? | 09:18 |
*** asettle has joined #openstack-swift | 09:35 | |
*** abalfour has quit IRC | 09:47 | |
*** abalfour has joined #openstack-swift | 09:48 | |
cschwede | onovy: Hi! Yes, I’m here for a bit | 10:00 |
*** ahale has joined #openstack-swift | 10:02 | |
*** sams-gleb has joined #openstack-swift | 10:02 | |
openstackgerrit | Mathias Bjoerkqvist proposed openstack/swift: WIP: Storing encryption root secret in Barbican https://review.openstack.org/364878 | 10:15 |
openstackgerrit | Mathias Bjoerkqvist proposed openstack/swift: WIP: Storing encryption root secret in Barbican https://review.openstack.org/364878 | 10:21 |
*** gabor_antal has quit IRC | 10:25 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift: Fix stats calculation in object-reconstructor https://review.openstack.org/283946 | 10:54 |
*** geaaru has quit IRC | 10:56 | |
kota_ | acoles: thanks for doing great work at patch 397259 | 11:01 |
patchbot | https://review.openstack.org/#/c/397259/ - swift - Use per policy diskfile manager in object replicator | 11:01 |
kota_ | acoles: i confirmed it works well so that I squash the patch into patch 283946 | 11:02 |
patchbot | https://review.openstack.org/#/c/283946/ - swift - Fix stats calculation in object-reconstructor | 11:02 |
*** jordanP has joined #openstack-swift | 11:03 | |
*** tesseract has joined #openstack-swift | 11:10 | |
*** tesseract is now known as Guest93740 | 11:11 | |
*** geaaru has joined #openstack-swift | 11:12 | |
onovy | cschwede: ad your review. what about add "connection timeout" for replicator daemon to config? | 11:13 |
onovy | i think it's much better to have solo connection and read timeout | 11:14 |
onovy | because if host is down, connection timeout mark them as bad really quickly | 11:14 |
onovy | https://review.openstack.org/#/c/391617/ | 11:14 |
patchbot | patch 391617 - swift - object-replicator cleanup | 11:14 |
cschwede | onovy: if you do that you still need a big warning note for deployers that the default behavior changes | 11:16 |
cschwede | onovy: because the behavior is different, no matter what the timeout is | 11:16 |
cschwede | onovy: even if both timeouts are by default 60 seconds, which is probably not wanted as well | 11:16 |
cschwede | onovy: it’s a bit tricky imo. either we keep the current behavior, or we change it but require config updates from every deployer | 11:18 |
*** geaaru has quit IRC | 11:19 | |
*** mvk has quit IRC | 11:29 | |
*** acoles_ is now known as acoles | 11:31 | |
*** geaaru has joined #openstack-swift | 11:39 | |
*** Jeffrey4l has quit IRC | 11:45 | |
*** takashi_ has quit IRC | 12:11 | |
*** vint_bra has joined #openstack-swift | 12:12 | |
acoles | gate failures? if your patch is failing gate func tests then you most likely will need to rebase since this landed https://review.openstack.org/399888 | 12:13 |
patchbot | patch 399888 - swift - Removed "in-process-" from func env tox name (MERGED) | 12:13 |
-openstackstatus- NOTICE: We are currently having capacity issues with our ubuntu-xenial nodes. We have addressed the issue but will be another few hours before new images have been uploaded to all cloud providers. | 12:20 | |
*** kei_yama has quit IRC | 12:21 | |
openstackgerrit | Alistair Coles proposed openstack/swift: breaking down tests.py into smaller pieces https://review.openstack.org/399275 | 12:24 |
acoles | tdasilva: ^^ rebase only | 12:24 |
*** catintheroof has joined #openstack-swift | 12:31 | |
*** mvk has joined #openstack-swift | 12:39 | |
*** Jeffrey4l has joined #openstack-swift | 12:41 | |
*** Jeffrey4l has quit IRC | 12:42 | |
*** Jeffrey4l has joined #openstack-swift | 12:45 | |
*** Jeffrey4l has quit IRC | 12:47 | |
*** Jeffrey4l has joined #openstack-swift | 12:49 | |
*** sanchitmalhotra has quit IRC | 12:50 | |
*** Jeffrey4l has quit IRC | 12:52 | |
*** Jeffrey4l has joined #openstack-swift | 12:52 | |
openstackgerrit | Mahati Chamarthy proposed openstack/swift: Move documented reclaim_age option to correct location https://review.openstack.org/374419 | 12:58 |
openstackgerrit | Mahati Chamarthy proposed openstack/swift: Reduce backend requests for SLO If-Match / HEAD requests https://review.openstack.org/347538 | 12:59 |
openstackgerrit | Mahati Chamarthy proposed openstack/swift: Confirm receipt of SLO PUT with etag https://review.openstack.org/390901 | 13:05 |
mahatic_ | sorry for the noise, rebasing | 13:08 |
mahatic_ | timburke: did one of yours - 347538 since 390901 is dependent on it | 13:10 |
*** SkyRocknRoll has joined #openstack-swift | 13:12 | |
*** amoralej is now known as amoralej|lunch | 13:24 | |
*** Jeffrey4l has quit IRC | 13:26 | |
*** mvk has quit IRC | 13:29 | |
*** ppai_ has quit IRC | 13:29 | |
*** klamath has joined #openstack-swift | 14:07 | |
onovy | cschwede: but if it is good change, why don't do it? | 14:09 |
onovy | yep, it's not backward compatible, but i should get better timeout when node is done | 14:10 |
onovy | *it | 14:10 |
*** lxsli_away is now known as lxsli | 14:20 | |
*** _JZ_ has joined #openstack-swift | 14:27 | |
tdasilva | acoles: thanks! | 14:31 |
acoles | tdasilva: np, good morning | 14:32 |
tdasilva | acoles: good afternoon to you sir :) | 14:33 |
tdasilva | acoles: just noticed you and jrichli +A patch 399275, thanks! I saw your comments there only after I had sent patchset 2, so I didn't ignore your comments | 14:34 |
patchbot | https://review.openstack.org/#/c/399275/ - swift - breaking down tests.py into smaller pieces | 14:34 |
acoles | tdasilva: I think my comments reduced to "IDK, anything <5k lines works" ;) | 14:34 |
tdasilva | acoles: agree with keeping TestFile, TestContainers and TestAccounts in tests.py (at least for now) and also agree with not doing any refactoring right now, but i think it would be best to break up the features (middleware) in different files | 14:35 |
tdasilva | just cause we keep adding new tests (e.g., symlinks will add about 1K lines of tests) so keeping all feature tests in one file seems like we would just end up in the same place we are today | 14:36 |
*** amoralej|lunch is now known as amoralej | 14:37 | |
acoles | tdasilva: yes, agree. I'm sure there will be times when its hard to know which feature a test should be added under (e.g symlinks+versions), but maybe we just go with arbitrary precedence - e.g. symlinks+versions tests (if there are any) would go in test_versioned_writes, other symlink tests in test_symlinks ?? | 14:39 |
acoles | tdasilva: jrichli made a point about duplicated test filenames, might be worth considering | 14:40 |
tdasilva | acoles: yeah, honestly I can't come up with a great solution for the multiple feature test, so I agree it will end up being pretty arbitrary where those tests go. Hopefully there won't be too many of those that create a real problem. but I think what you suggested is better than having a test_symlink_versions.py file | 14:43 |
*** mvk has joined #openstack-swift | 14:45 | |
tdasilva | acoles, jrichli: in terms of that duplicated names I wasn't too concerned tbh. We will now have test/unit/common/middleware/test_versioned_writes.py and test/functional/test_versioned_writes.py, but IMO that's ok, i think it's less confusing than naming it something else, plust we already have obj/server.py and container/server.py and so on | 14:45 |
openstackgerrit | Mathias Bjoerkqvist proposed openstack/swift: WIP: Storing encryption root secret in Barbican https://review.openstack.org/364878 | 14:46 |
*** ntata has joined #openstack-swift | 14:50 | |
*** ntata has quit IRC | 14:53 | |
openstackgerrit | Alistair Coles proposed openstack/swift: Update README.md to clarify policies allowed https://review.openstack.org/400286 | 14:57 |
openstackgerrit | Alistair Coles proposed openstack/swift: go: Update README.md to clarify policies allowed https://review.openstack.org/400286 | 14:59 |
*** gabor_antal has joined #openstack-swift | 15:12 | |
*** oshritf has joined #openstack-swift | 15:12 | |
*** sams-gle_ has joined #openstack-swift | 15:12 | |
*** sams-gleb has quit IRC | 15:15 | |
*** sams-gle_ has quit IRC | 15:16 | |
*** oshritf has quit IRC | 15:23 | |
*** oshritf has joined #openstack-swift | 15:24 | |
*** sams-gleb has joined #openstack-swift | 15:27 | |
cschwede | onovy: i’m not against it per se, but in the case with two configurable timeouts the patch will need some notes as well, and we need to think about some sane defaults. a 5 second default timeout with the patch applied is to short imo, when it was 60 seconds before | 15:29 |
onovy | cschwede: we have 500ms timeout from proxy->store | 15:30 |
onovy | is 10x really too short? | 15:30 |
cschwede | onovy: if you have global clusters then it is likely that the proxy never talks to a remote storage server directly, and is using a slow wan link - 5 second might be enough, but it was 60 seconds before and we should take that into account | 15:31 |
onovy | ah, right. global clusters | 15:31 |
*** vinsh has joined #openstack-swift | 15:33 | |
cschwede | onovy: the challenge here is that the replication itself might have finished successfully, but then we can’t finalize it because this small requests hits a timeout - and boom you need to start over later again | 15:34 |
onovy | yep, that's right. my motivation is to timeout as fast as possible if node is not available | 15:34 |
onovy | so maybe we should have bigger timeout AFTER rsync, and smaller before | 15:35 |
cschwede | onovy: sounds like a good idea! | 15:35 |
onovy | i think i will split it into two patches | 15:36 |
onovy | first: refactor it for direct_client | 15:36 |
onovy | second: change it to two timeouts | 15:36 |
*** siva_krish has joined #openstack-swift | 15:39 | |
*** siva_krish has left #openstack-swift | 15:41 | |
*** oshritf has quit IRC | 15:43 | |
*** rledisez has quit IRC | 15:45 | |
*** oshritf has joined #openstack-swift | 15:46 | |
*** oshritf has quit IRC | 15:53 | |
*** pcaruana has quit IRC | 16:06 | |
*** Guest93740 has quit IRC | 16:11 | |
openstackgerrit | Ondřej Nový proposed openstack/swift: object-replicator cleanup https://review.openstack.org/391617 | 16:13 |
onovy | cschwede: ^ thanks | 16:16 |
*** oshritf has joined #openstack-swift | 16:16 | |
*** douglascorrea has joined #openstack-swift | 16:24 | |
*** tuan_luong has joined #openstack-swift | 16:28 | |
*** oshritf has quit IRC | 16:29 | |
*** oshritf has joined #openstack-swift | 16:33 | |
notmyname | good morning | 16:34 |
*** Tuan_ has joined #openstack-swift | 16:36 | |
*** tuan_luong has quit IRC | 16:39 | |
*** Tuan_ has quit IRC | 16:47 | |
notmyname | FYI https://review.openstack.org/#/c/394261/ is a doc spec that seems to imply the swift team is signing up for writing an upgrade guide in the ocata cycle | 16:47 |
patchbot | patch 394261 - docs-specs - [ops-guide] Add project upgrade notes | 16:47 |
*** asettle has quit IRC | 17:00 | |
*** mmotiani1 has joined #openstack-swift | 17:14 | |
*** hseipp has quit IRC | 17:28 | |
*** nikivi has joined #openstack-swift | 17:39 | |
*** nikivi has quit IRC | 17:52 | |
*** jordanP has quit IRC | 18:01 | |
*** dmorita has joined #openstack-swift | 18:01 | |
*** dmorita has quit IRC | 18:03 | |
*** dmorita_ has joined #openstack-swift | 18:03 | |
*** oshritf has quit IRC | 18:03 | |
*** vinsh has quit IRC | 18:13 | |
onovy | notmyname: hi | 18:15 |
onovy | notmyname: sry for this: https://review.openstack.org/#/c/399892/ if you don't want it | 18:15 |
patchbot | patch 399892 - openstack-infra/project-config - Swift: Make Python 3.5 check voting (MERGED) | 18:15 |
onovy | but i wrote: please wait for John's comment, before +A this :) thanks | 18:16 |
onovy | they merged :] | 18:16 |
patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 18:16 |
notmyname | onovy: meh. I do appreciate your comment on it :-) | 18:16 |
*** joeljwright has quit IRC | 18:16 | |
onovy | so you are fine with py35 gating (voting)? | 18:16 |
notmyname | probably | 18:16 |
notmyname | I mean, I am now! ;-) | 18:17 |
onovy | cool, thanks :) | 18:17 |
*** ChubYann has joined #openstack-swift | 18:20 | |
notmyname | acoles: tdasilva: mattoliverau: kota_: jrichli: cschwede: I want to make sure you saw the change that onovy made this weekend (that I pushed through) to the tox.ini. had to fix the gate, but it might affect your local test environment https://review.openstack.org/#/c/399888/ | 18:21 |
patchbot | patch 399888 - swift - Removed "in-process-" from func env tox name (MERGED) | 18:21 |
notmyname | onovy: thanks again for being on top of that | 18:21 |
*** torgomatic has joined #openstack-swift | 18:22 | |
*** ChanServ sets mode: +v torgomatic | 18:22 | |
*** tqtran has joined #openstack-swift | 18:23 | |
acoles | notmyname: saw it, gate was failing patches that weren't rebased on it | 18:24 |
tdasilva | notmyname: ack | 18:25 |
timburke | good morning | 18:25 |
*** oshritf has joined #openstack-swift | 18:27 | |
openstackgerrit | Alistair Coles proposed openstack/swift: Fix stats calculation in object-reconstructor https://review.openstack.org/283946 | 18:33 |
*** oshritf has quit IRC | 18:33 | |
openstackgerrit | Merged openstack/swift-bench: Replace assertEquals with assertEqual https://review.openstack.org/400126 | 18:33 |
acoles | kota_: ^^ improved the unit test that got impacted by replicator change | 18:33 |
onovy | notmyname: np! | 18:40 |
acoles | onovy: notmyname out of curiosity,what was the problem that required the tox env name change? | 18:40 |
notmyname | acoles: linux has a 128 char limit on the shebang line | 18:41 |
notmyname | and infra builds the tox env to include the job name and the env name | 18:41 |
notmyname | so with a long env name (which also is generally similar to the job name), you go over rthe limit | 18:41 |
notmyname | func-in-process-encryption, IIRC, was the longest | 18:42 |
notmyname | also because I added -xfs-tmpdir to all of the job names. that was the change since last week | 18:42 |
acoles | notmyname: OIC ^^ so it was your fault ;) | 18:43 |
notmyname | yep. all my fault | 18:43 |
acoles | hummingbird will need a merge from master to get the new tox envs | 18:43 |
*** amoralej is now known as amoralej|off | 18:44 | |
*** vinsh has joined #openstack-swift | 18:48 | |
notmyname | ah, good point | 18:50 |
*** acoles is now known as acoles_ | 18:50 | |
*** jamielennox is now known as jamielennox|away | 18:52 | |
torgomatic | Weinberg's Second Law: If builders built buildings the way programmers wrote programs, then the first woodpecker that came along would destroy civilization. | 18:53 |
*** jamielennox|away is now known as jamielennox | 19:00 | |
*** cbartz has quit IRC | 19:02 | |
*** chlong has joined #openstack-swift | 19:03 | |
onovy | a! john's fail, /me fixed! syc! | 19:06 |
notmyname | :-) | 19:08 |
*** geaaru has quit IRC | 19:08 | |
notmyname | hmm... https://review.openstack.org/400378 didn't get announced | 19:10 |
patchbot | patch 400378 - swift (stable/newton) - 2.10.1 authors/changelog updates | 19:10 |
notmyname | ah, probably hasn't been configured for that branch | 19:10 |
notmyname | I'll hold off on the mitaka one until the 2 in queue get landed | 19:11 |
*** stream10 has joined #openstack-swift | 19:14 | |
*** diogogmt has joined #openstack-swift | 19:26 | |
openstackgerrit | Mathias Bjoerkqvist proposed openstack/swift: WIP: Storing encryption root secret in Barbican https://review.openstack.org/364878 | 19:35 |
*** silor has quit IRC | 19:41 | |
*** vinsh has quit IRC | 19:45 | |
*** vinsh has joined #openstack-swift | 19:47 | |
*** douglascorrea has quit IRC | 20:07 | |
*** douglascorrea has joined #openstack-swift | 20:08 | |
*** vinsh has quit IRC | 20:10 | |
*** douglascorrea has quit IRC | 20:12 | |
*** asettle has joined #openstack-swift | 20:26 | |
*** zaitcev has joined #openstack-swift | 20:38 | |
*** ChanServ sets mode: +v zaitcev | 20:38 | |
*** ndk_ has quit IRC | 20:49 | |
*** ndk_ has joined #openstack-swift | 20:51 | |
mattoliverau | Morning | 20:55 |
*** asettle has quit IRC | 21:08 | |
*** asettle has joined #openstack-swift | 21:08 | |
*** vinsh has joined #openstack-swift | 21:12 | |
*** asettle has quit IRC | 21:13 | |
openstackgerrit | Ondřej Nový proposed openstack/swift: Added comment for "user" option in drive-audit config https://review.openstack.org/400406 | 21:14 |
onovy | timburke: :) | 21:15 |
timburke | onovy: i didn't mean you had to do it *right now* :P | 21:15 |
onovy | :) | 21:15 |
*** portante has quit IRC | 21:19 | |
*** portante has joined #openstack-swift | 21:20 | |
*** stream10 has quit IRC | 21:36 | |
*** sams-gleb has quit IRC | 21:52 | |
*** sams-gleb has joined #openstack-swift | 21:52 | |
*** sams-gleb has quit IRC | 21:57 | |
*** mmotiani1 has quit IRC | 21:59 | |
*** vint_bra has quit IRC | 22:14 | |
openstackgerrit | Tim Burke proposed openstack/swift: Reduce backend requests for SLO If-Match / HEAD requests https://review.openstack.org/347538 | 22:41 |
*** chlong has quit IRC | 22:52 | |
notmyname | https://review.openstack.org/400435 for 2.7.1 | 22:53 |
patchbot | patch 400435 - swift (stable/mitaka) - 2.7.1 changelog updates | 22:53 |
notmyname | kota_: hisashi: hope everyone is ok | 22:57 |
kota_ | notmyname: thx, I'm safe and around Tokyo it seems not so impacted | 23:15 |
*** SkyRocknRoll has quit IRC | 23:21 | |
*** kei_yama has joined #openstack-swift | 23:23 | |
*** _JZ_ has quit IRC | 23:25 | |
*** douglascorrea has joined #openstack-swift | 23:28 | |
*** tqtran has quit IRC | 23:42 | |
*** tqtran has joined #openstack-swift | 23:42 | |
*** openstack has joined #openstack-swift | 23:45 | |
*** douglascorrea has quit IRC | 23:48 | |
*** douglascorrea has joined #openstack-swift | 23:49 | |
*** douglascorrea has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!