Tuesday, 2016-08-30

*** ogelbukh has joined #openstack-swift00:04
*** dmorita has quit IRC00:05
*** dmorita has joined #openstack-swift00:05
ogelbukhcheers00:07
*** onovy has quit IRC00:11
*** onovy has joined #openstack-swift00:13
*** Suyi_ has quit IRC00:16
*** patchbot has joined #openstack-swift00:16
*** nadeem has quit IRC00:18
*** glange has quit IRC00:29
*** glange has joined #openstack-swift00:30
*** ChanServ sets mode: +v glange00:30
openstackgerritMerged openstack/swift: Use more specific asserts in test/unit/cli tests  https://review.openstack.org/34276900:44
*** dipatov has quit IRC00:53
*** m_kazuhiro has joined #openstack-swift01:00
*** adu has quit IRC01:06
*** klrmn has quit IRC01:12
claygtimburke: do you know how to make lp bug #1583305 associated with a backport?01:14
openstackLaunchpad bug 1583305 in OpenStack Object Storage (swift) "auditor status files cause replicator to report errors" [Undecided,Fix released] https://launchpad.net/bugs/1583305 - Assigned to Charles Hsu (charles0126)01:14
claygtimburke: I'm pretty sure there is a thing in launchpad where you can be like "also effects some old version of swift"01:15
timburkeclayg: i kinda remember seeing that sometimes, too... not sure how to do it though01:15
timburkemy approach tends to be "submit the patch, worry about bug reports later"01:16
claygi'm not sure I know how to submit backport patches *either*01:16
claygmaybe it's written down somewhere...01:16
notmynamebackports are simply patches submitted to a stable/<whatever> branch01:18
notmynameassuming the patch cleanly applies from master, then a cherry-pick is all that's needed01:18
timburkeclayg: `git checkout gerrit/stable/mitaka && git review -X Ib15a0987288d9ee32432c1998aefe638ca3b223b && git review`01:19
*** patchbot has quit IRC01:19
*** patchbot has joined #openstack-swift01:20
timburkeclayg: so now we've got https://review.openstack.org/36251401:20
timburkegimme https://review.openstack.org/#/c/362514/ patchbot!01:20
patchbotpatch 362514 - swift (stable/mitaka) - Ignore auditor status files to prevent replicator ...01:20
timburkeyay!01:20
claygomg timburke you're like a *wizard*01:21
timburkeclayg: i *think* we maybe want the "Target to series" link? i'm not sure, though; if i do that, do i need to specify both mitaka *and* trunk? or something?01:28
mattoliverauclayg: http://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes01:30
*** trananhkma has joined #openstack-swift01:31
claygmattoliverau: how did you *find that*01:41
* clayg is surrounded by wizards01:41
mattoliverauclayg: I've had to use it before, so book marked it ;) This way you don't have to spend time finding it like I did :P01:41
* mattoliverau also has some old stable swift SAIOs he has in the cloud so requirements are all clean.. again cause I had to do it before :P (I should probably make a mitaka one now that it's stable).01:43
mattoliverauclayg: also I think I'm this close (| |) to having version 1 of our readonly large container sharding thing happening. (famous last words). Well almost ready to start trying it out and debugging that is, just need to finalise some container-server extras and get the container-replicator multi-db aware.. ok so maybe I'm more like this close (|    |).. but soon I'll hopefully be testing the SwiftStack very large container again.01:48
*** tqtran has quit IRC02:15
*** nadeem has joined #openstack-swift02:39
*** thurloat has quit IRC02:47
*** swat30 has quit IRC02:47
openstackgerritzheng yin proposed openstack/python-swiftclient: Add swift tempurl and swift auth command docstring  https://review.openstack.org/36021202:47
openstackgerritzheng yin proposed openstack/python-swiftclient: Add swift tempurl and swift auth command docstring  https://review.openstack.org/36021202:57
openstackgerritzheng yin proposed openstack/python-swiftclient: Add swift tempurl and swift auth command docstring  https://review.openstack.org/36021202:58
*** daemontool has quit IRC03:07
*** _JZ_ has quit IRC03:09
*** daemontool has joined #openstack-swift03:11
*** tqtran has joined #openstack-swift03:13
*** manous has quit IRC03:14
*** tqtran has quit IRC03:18
notmynameta da!03:22
notmynamep 1234 patch 1234 https://review.openstack.org/1234 https://review.openstack.org/#/c/1234/03:22
patchbothttps://review.openstack.org/#/c/1234/ - tempest - Removing link doctoring in test_images (MERGED)03:22
patchbothttps://review.openstack.org/#/c/1234/ - tempest - Removing link doctoring in test_images (MERGED)03:22
patchbotpatch 1234 - tempest - Removing link doctoring in test_images (MERGED)03:22
patchbotpatch 1234 - tempest - Removing link doctoring in test_images (MERGED)03:22
*** abhitechie has joined #openstack-swift03:22
notmynamejlvillal: different URL form works now03:23
mattoliveraunotmyname: nice03:26
*** mingyu has joined #openstack-swift03:29
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873603:33
*** abhitechie has quit IRC03:42
*** _JZ_ has joined #openstack-swift03:45
*** links has joined #openstack-swift03:51
*** chsc has joined #openstack-swift03:53
*** chsc has joined #openstack-swift03:53
*** mingyu has quit IRC03:58
*** abhitechie has joined #openstack-swift03:59
*** hosanai has joined #openstack-swift04:06
*** ChanServ sets mode: +v hosanai04:06
*** chsc has quit IRC04:09
*** ppai has joined #openstack-swift04:42
*** psachin has joined #openstack-swift04:44
*** m_kazuhiro has quit IRC04:51
*** m_kazuhiro has joined #openstack-swift04:52
*** nadeem has quit IRC04:57
*** mingyu has joined #openstack-swift04:58
*** nadeem has joined #openstack-swift05:00
*** mingyu has quit IRC05:03
*** lifeless_ is now known as lifeless05:15
*** bkopilov has joined #openstack-swift05:27
*** _JZ_ has quit IRC05:28
*** mingyu has joined #openstack-swift06:00
*** ChubYann has quit IRC06:00
*** mingyu has quit IRC06:04
*** rcernin has joined #openstack-swift06:19
*** acorwin has quit IRC06:19
*** acorwin has joined #openstack-swift06:23
*** pcaruana has joined #openstack-swift06:26
openstackgerritgengchc2 proposed openstack/liberasurecode: Fix a typo in the erasurecode file  https://review.openstack.org/36263806:32
openstackgerritgecong proposed openstack/swift: Remove white space between print and ()  https://review.openstack.org/34253206:39
*** Tiriel has joined #openstack-swift06:42
TirielHello! Is it possible to serve swift's account, object and container services over SSL?06:46
TirielI'm trying to run them with apache and mod_wsgi, I can configure SSL on apache, but I figure I should be configuring what certificates to use elsewhere too06:47
TirielI'm trying to run them with apache and mod_wsgi, I can configure SSL on apache, but I figure I should be configuring what certificates to use elsewhere too06:47
openstackgerritChristian Schwede proposed openstack/swift: Deprecate swift-temp-url  https://review.openstack.org/34817306:49
*** nadeem has quit IRC06:53
*** mingyu has joined #openstack-swift06:53
openstackgerritgecong proposed openstack/swift: Remove white space between print and ()  https://review.openstack.org/36265006:54
*** m_kazuhiro has quit IRC06:58
*** nadeem has joined #openstack-swift07:04
*** hseipp has joined #openstack-swift07:04
*** tesseract- has joined #openstack-swift07:07
*** sheel has joined #openstack-swift07:08
*** arnoldchen has joined #openstack-swift07:09
*** arnoldchen has quit IRC07:10
*** arnoldchen has joined #openstack-swift07:10
*** arnoldchen has quit IRC07:10
*** geaaru has joined #openstack-swift07:12
*** tqtran has joined #openstack-swift07:16
*** tqtran has quit IRC07:20
*** Trixboxer has joined #openstack-swift07:21
*** kota_ has quit IRC07:42
*** takashi has joined #openstack-swift07:48
*** tanee_away is now known as tanee07:52
*** m_kazuhiro has joined #openstack-swift08:00
*** dmorita has quit IRC08:00
*** kota_ has joined #openstack-swift08:02
*** ChanServ sets mode: +v kota_08:02
*** dmorita has joined #openstack-swift08:03
Tirielhow do I troubleshoot swift's 503 Service Unavailable when ran via mod_wsgi? I don't see anything on the logs08:05
*** dmorita has quit IRC08:08
openstackgerritDavanum Srinivas (dims) proposed openstack/swift: [WIP] Testing latest u-c  https://review.openstack.org/31844108:10
*** SkyRocknRoll has joined #openstack-swift08:22
lifelessTiriel: which logs are you looking at? apache error.log is the default log for mod_wsgi IIRC08:23
*** m_kazuhiro has quit IRC08:31
Tiriellifeless: all of them, error.log proxy-* syslog, I even ran the app under uwsgi, nothing08:32
TirielIn fact, I am running it via servlet now and it still shows the same behaviour08:33
*** asettle has joined #openstack-swift08:46
*** daemontool has quit IRC08:46
*** daemontool has joined #openstack-swift08:49
openstackgerritMerged openstack/swift: Remove white space between print and ()  https://review.openstack.org/36265008:50
*** nadeem has quit IRC08:51
*** nadeem has joined #openstack-swift08:59
*** links has quit IRC09:00
*** dmorita has joined #openstack-swift09:05
*** nadeem has quit IRC09:06
*** dmorita has quit IRC09:10
*** links has joined #openstack-swift09:15
openstackgerritChristian Schwede proposed openstack/swift: Fix swiftdir option and usage of storage policy aliases  https://review.openstack.org/34469309:23
lifelessTiriel: are you sure the request is reaching your process09:31
*** geaaru has quit IRC09:36
*** kei_yama has quit IRC09:39
*** geaaru has joined #openstack-swift09:50
claygi wouldn't imagine there's any way a proxy service's use of httplib would know how to connect to a ssl wrapped storage service endpoint - not quite sure how it would blow up tho09:58
claygi doubt anyone ever sets ssl=True -> https://github.com/openstack/swift/blob/master/swift/common/bufferedhttp.py#L18709:59
claygi can't think of any way you could represent you want/expect ssl in the ring10:00
claygalso - why are you trying to do this?  ;)10:00
*** dmorita has joined #openstack-swift10:06
*** daemontool has quit IRC10:09
*** daemontool has joined #openstack-swift10:09
openstackgerritDavanum Srinivas (dims) proposed openstack/swift: [WIP] Testing latest u-c  https://review.openstack.org/31844110:10
*** dmorita has quit IRC10:11
*** mahatic has quit IRC10:16
openstackgerritMerged openstack/swift: add test for expirer processes == process  https://review.openstack.org/36072010:16
Tiriellifeless: yes, I do see the proxy server getting the request, but that's about it10:18
Tiriel"GET /v1/AUTH_945603bd32ee473e912fa7056903de6e?format=json HTTP/1.1" 503 445 "-" "osc-lib keystoneauth1/2.12.1 python-requests/2.11.1 CPython/2.7.9"10:26
*** m_kazuhiro has joined #openstack-swift10:27
*** dfflanders has quit IRC10:30
*** kota_ has quit IRC10:45
*** abhinavtechie has joined #openstack-swift10:48
*** abhitechie has quit IRC10:50
openstackgerritMerged openstack/swift: Authorise versioned write PUTs before copy  https://review.openstack.org/35762810:52
*** kota_ has joined #openstack-swift10:55
*** ChanServ sets mode: +v kota_10:55
*** mahatic has joined #openstack-swift11:11
*** tqtran has joined #openstack-swift11:17
*** tqtran has quit IRC11:22
*** dmorita has joined #openstack-swift11:22
*** dmorita has quit IRC11:27
openstackgerritzheng yin proposed openstack/swift: Add test to validate the error strings for process and processes in test_expirer  https://review.openstack.org/36091411:29
*** jordanP has joined #openstack-swift11:29
openstackgerritzheng yin proposed openstack/swift: Add test to validate the error strings for process and processes in test_expirer  https://review.openstack.org/36091411:32
*** asettle has quit IRC11:35
*** asettle has joined #openstack-swift11:36
*** manous has joined #openstack-swift11:49
*** asettle has quit IRC11:51
*** clayg has quit IRC11:53
*** m_kazuhiro has quit IRC11:58
*** mingyu has quit IRC12:00
*** trananhkma has quit IRC12:05
*** ttrumm has joined #openstack-swift12:11
*** abhinavtechie has quit IRC12:31
*** dmorita has joined #openstack-swift12:34
*** adu has joined #openstack-swift12:38
*** dmorita has quit IRC12:39
*** _JZ_ has joined #openstack-swift12:47
*** cdelatte has joined #openstack-swift12:49
*** ppai has quit IRC12:58
*** manous_ has joined #openstack-swift13:01
*** Tiriel has quit IRC13:02
*** ttrumm has quit IRC13:04
*** ttrumm has joined #openstack-swift13:04
*** manous has quit IRC13:05
*** asettle has joined #openstack-swift13:07
*** hosanai has quit IRC13:08
*** ttrumm has quit IRC13:09
*** takashi has quit IRC13:09
*** david-lyle has quit IRC13:10
*** klamath has joined #openstack-swift13:10
*** hosanai has joined #openstack-swift13:15
*** ChanServ sets mode: +v hosanai13:15
*** sheel has quit IRC13:16
*** vint_bra has joined #openstack-swift13:17
*** david-lyle has joined #openstack-swift13:18
*** adu has quit IRC13:21
*** SkyRocknRoll has quit IRC13:24
*** daemontool has quit IRC13:25
*** daemontool has joined #openstack-swift13:27
*** StraubTW has joined #openstack-swift13:28
*** hosanai has quit IRC13:36
*** Vinsh has quit IRC13:43
*** baojg has joined #openstack-swift13:46
*** dmorita has joined #openstack-swift13:46
*** daemontool has quit IRC13:49
*** daemontool has joined #openstack-swift13:50
*** dmorita has quit IRC13:51
*** npf has joined #openstack-swift13:56
*** dmorita has joined #openstack-swift14:07
*** dmorita has quit IRC14:11
*** sbelous__ has joined #openstack-swift14:15
*** baojg has quit IRC14:16
*** baojg has joined #openstack-swift14:18
*** _JZ_ has quit IRC14:20
*** Vinsh has joined #openstack-swift14:25
*** daemontool has quit IRC14:28
*** baojg has quit IRC14:28
*** daemontool has joined #openstack-swift14:28
*** daemontool has quit IRC14:42
*** david-lyle has quit IRC14:43
*** diogogmt has joined #openstack-swift14:45
*** david-lyle has joined #openstack-swift14:48
*** psachin has quit IRC14:50
*** asettle has quit IRC14:56
*** asettle has joined #openstack-swift14:57
*** chsc has joined #openstack-swift15:00
*** _JZ_ has joined #openstack-swift15:01
*** mingyu has joined #openstack-swift15:01
*** david-lyle has quit IRC15:02
*** diogogmt has quit IRC15:06
*** mingyu has quit IRC15:06
*** sbelous__ has quit IRC15:07
*** diogogmt has joined #openstack-swift15:08
*** npf has quit IRC15:14
*** tqtran has joined #openstack-swift15:19
*** david-lyle has joined #openstack-swift15:22
*** tqtran has quit IRC15:23
*** mingyu has joined #openstack-swift15:26
*** links has quit IRC15:28
*** Jeffrey4l_ has quit IRC15:32
notmynamegood morning15:32
jrichlinotmyname: good morning! timburke: were you still wanting to make a bug for patch 337960, or maybe discuss it here or at the next meeting?15:33
patchbothttps://review.openstack.org/#/c/337960/ - swift - Send correct SLO ETag for container updates15:33
joeljwrightnotmyname: good morning15:34
*** david-lyle has quit IRC15:40
*** hseipp has quit IRC15:45
*** dmorita has joined #openstack-swift15:46
*** david-lyle has joined #openstack-swift15:50
*** chsc has quit IRC15:54
timburkegood morning15:56
timburkejrichli: that's a good idea that i still haven't gotten around to :-)15:56
timburkei'll make some time for it today15:56
jrichligreat.  I am interested in discussing it as well.  Let me know if I can help.15:58
joeljwrighttimburke: for the client release I'm reviewing the headers -> str patch and the accept gzip responses patch15:58
joeljwrighttimburke: those look like the only ones that are important/close enough to land15:59
joeljwrighttimburke: thoughts?15:59
timburkejoeljwright: i was debating about trying for patch 298968 and its follow-up, patch 300697, but maybe that's overly ambitious16:01
patchbothttps://review.openstack.org/#/c/298968/ - python-swiftclient - Adding keystoneauth sessions support16:01
patchbothttps://review.openstack.org/#/c/300697/ - python-swiftclient - Add v1password keystoneauth plugin16:01
joeljwrighttimburke: yeah, I looked at those...16:01
joeljwrighttimburke: I kind of felt I wanted more time with them though16:02
timburkei'll check if patch 360212 looks good; if so, may as well get that in, too16:02
patchbothttps://review.openstack.org/#/c/360212/ - python-swiftclient - Add swift tempurl and swift auth command docstring16:02
timburkefair enough16:02
joeljwrighttimburke: yeah, 360212 is worth it :)16:02
timburkeand patch 361465 is another little doc-cleanup that's probably worth it16:03
patchbothttps://review.openstack.org/#/c/361465/ - python-swiftclient - Change Examples from headings to topics16:03
notmynametimburke: joeljwright: ok. then land them asap ;-)16:03
joeljwrightright, they're on my list16:04
joeljwrightwill try to get them all reviewed tonight16:04
timburkenotmyname: given that more than half of those are *my* patches, there's only so much i can do ;-)16:07
joeljwrighttimburke: the gzip one at least already has a +216:08
*** dmorita has quit IRC16:11
*** dmorita has joined #openstack-swift16:11
jlvillalnotmyname: Not sure why we got this message in #openstack-ironic.  <patchbot> You've given me 5 invalid commands within the last 60 seconds; I'm now ignoring you for 10 minutes.16:12
openstackgerritClay Gerrard proposed openstack/swift: Add test to validate the error strings in test_expirer  https://review.openstack.org/36091416:16
EmilienMcschwede: commented on your puppet swith patch16:19
cschwedeEmilienM: oh my, that was quick! thx, will do :)16:20
*** david-lyle has quit IRC16:21
*** david-lyle has joined #openstack-swift16:23
notmynamejlvillal: hmmm16:23
notmynamejlvillal: that's a new one16:23
*** jordanP has quit IRC16:23
jlvillalnotmyname: It didn't say who gave invalid commands. Or if it was the whole channel.16:24
notmynamejlvillal: so it seems like I may need to diagnose some bugs and/or config issues16:24
jlvillalnotmyname: Thanks.  FYI: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-08-30.log   search for 'ignoring'16:25
notmynamejlvillal: ok, how about I /part it from there for now while I figure stuff out16:26
jlvillalnotmyname: Sure. It seems to be working again, for what this worth.16:26
jlvillals/this/that is/16:26
*** SkyRocknRoll has joined #openstack-swift16:26
notmynamejlvillal: yeah, I just don't like unexpected stuff like that happening in a channel where I'm not active16:27
jlvillalnotmyname: Makes sense. Thanks again.16:27
pdardeaunotmyname: is patchbot your creation?16:29
notmynamepdardeau: yes16:29
pdardeaunotmyname: very cool!16:30
*** biyiklioglu has joined #openstack-swift16:30
timburkepdardeau: source is at https://github.com/notmyname/Patches/16:31
*** biyiklioglu has quit IRC16:31
notmynameyeah, timburke has helped out on it too16:31
*** david-lyle_ has joined #openstack-swift16:36
*** tqtran has joined #openstack-swift16:36
*** ppai has joined #openstack-swift16:36
*** ChubYann has joined #openstack-swift16:38
*** david-lyle has quit IRC16:39
timburkelet me test something with patchbot...16:39
timburkehuh. nope. weird16:40
*** clayg has joined #openstack-swift16:46
*** ChanServ sets mode: +v clayg16:46
claygscre -x16:46
claygwhoa16:46
claygsomebody needs to merge master into hummingbird?16:46
claygI don't think it has a keymaster?16:46
*** mingyu has quit IRC16:51
*** diogogmt has quit IRC16:52
*** nadeem has joined #openstack-swift16:58
*** mingyu has joined #openstack-swift17:02
*** diogogmt has joined #openstack-swift17:07
*** rcernin has quit IRC17:12
*** arch-nemesis has joined #openstack-swift17:13
*** tesseract- has quit IRC17:14
*** Suyi_ has joined #openstack-swift17:14
*** nadeem has quit IRC17:14
*** nadeem has joined #openstack-swift17:15
*** asettle has quit IRC17:16
*** asettle has joined #openstack-swift17:17
*** SkyRocknRoll has quit IRC17:18
*** klrmn has joined #openstack-swift17:20
*** tesseract- has joined #openstack-swift17:24
*** npf has joined #openstack-swift17:24
*** pcaruana has quit IRC17:25
*** tesseract- has quit IRC17:28
*** npf has quit IRC17:30
*** asettle has quit IRC17:37
*** dmorita has quit IRC17:40
*** dmorita has joined #openstack-swift17:43
*** SkyRocknRoll has joined #openstack-swift17:43
*** klamath has quit IRC17:44
*** chsc has joined #openstack-swift17:47
*** SkyRocknRoll has quit IRC17:57
*** chianingwang has joined #openstack-swift17:59
*** asettle has joined #openstack-swift17:59
*** dmorita has quit IRC17:59
timburkejrichli: created https://bugs.launchpad.net/swift/+bug/161857317:59
openstackLaunchpad bug 1618573 in OpenStack Object Storage (swift) "SLO etag doesn't match between container listings and object GET/HEAD" [Undecided,New] - Assigned to Tim Burke (1-tim-z)17:59
jrichlitimburke: nice, ill check it out18:00
*** dmorita has joined #openstack-swift18:05
*** tesseract- has joined #openstack-swift18:05
*** tesseract- has quit IRC18:05
*** tesseract- has joined #openstack-swift18:06
*** zaitcev has joined #openstack-swift18:06
*** ChanServ sets mode: +v zaitcev18:06
openstackgerritTim Burke proposed openstack/swift: Send correct SLO ETag for container updates  https://review.openstack.org/33796018:07
*** tesseract- has quit IRC18:08
*** tesseract- has joined #openstack-swift18:10
*** geaaru has quit IRC18:17
zaitcevGuys, I want to run .functests against a test cluster and I found that there's no way to pass my CA certificate to them. Before I go add a cert= option, is there a free-of-charge CA that's recognized by semi-recent systems, specifically in Requests?18:18
zaitcevOr, better yet, can I stick my CA cert somewhere in /etc so that Requests trusts it by default?18:19
timburkezaitcev: that latter one sounds promising. not sure where to put it off hand, though18:20
notmynamezaitcev: letsencrypt is the current hotness for free certs and it's widely recognized18:21
zaitcevOh well, it's open source... Can't be that difficult.18:21
zaitcevAh18:21
notmynameI used it for a website. not sure how well it works for swift18:22
*** klamath has joined #openstack-swift18:24
*** McMurlock1 has joined #openstack-swift18:24
*** tqtran has quit IRC18:24
clarkbzaitcev: if the functests use python requests there is an env var you can set to override the location of trusted certs for requests18:25
*** tqtran has joined #openstack-swift18:25
clarkbzaitcev: I would have to lookup the requests docs to figure out what it is exactly though18:25
zaitcevclarkb: Absolutely, I'll RTFM myself, but thatnks for the tip18:26
*** asettle has quit IRC18:27
*** tqtran has quit IRC18:28
*** tqtran has joined #openstack-swift18:28
zaitcev"By default, Requests bundles a set of root CAs that it trusts, sourced from the Mozilla trust store." -- I knew it would not be like cp /q/zaitcev/arc/CA/certs/cacert.pem /etc/pki/CA/certs18:29
clarkbzaitcev: the distro packages for requests almost always patch that out to use the system certs but yes its annoying18:29
*** tesseract- has quit IRC18:30
*** tongli has joined #openstack-swift18:30
*** tesseract- has joined #openstack-swift18:30
*** tesseract- has quit IRC18:30
*** tesseract- has joined #openstack-swift18:31
openstackgerritClay Gerrard proposed openstack/swift: Merge remote-tracking branch 'origin/master' into feature/hummingbird  https://review.openstack.org/36311118:34
clayg^ no idea if i did that right?18:35
notmynameclayg: yeah, at first glance it looks right18:35
notmynameclayg: did you have a clean merge locally?18:35
claygnotmyname: it did!18:36
claygnotmyname: and all tox tests passed?18:36
notmynamegood. and then that's what I'd expect to see in gerrit for a clean merge18:36
claygnotmyname: phew18:36
notmynameso either you or someone else +2/+A it and you're good18:36
claygnadeem: hook me up with a quick double check on patch 363111?18:37
patchbothttps://review.openstack.org/#/c/363111/ - swift (feature/hummingbird) - Merge remote-tracking branch 'origin/master' into ...18:37
*** tesseract- has quit IRC18:37
claygnadeem: it merges a few new features from upstream - it should all be backwards compat but it'd be a super bummer if it broke your workflow18:37
nadeemlet me have a quick look18:38
claygnadeem: AWESOAWEOME!!18:40
*** manous_ has quit IRC18:40
*** ppai has quit IRC18:41
nadeemclayg: LGTM18:44
*** dmorita has quit IRC18:45
claygf'ing A!18:45
*** tqtran has quit IRC18:46
*** tqtran has joined #openstack-swift18:46
claygOMFG!18:47
claygi am so sorry for the emails!?18:47
*** dmorita has joined #openstack-swift18:47
*** delattec has joined #openstack-swift18:48
*** Suyi_1 has joined #openstack-swift18:50
*** Raymii has quit IRC18:50
*** a1|away has quit IRC18:50
*** AbyssOne has quit IRC18:50
*** cdelatte has quit IRC18:51
*** etienneme has quit IRC18:51
*** Suyi_ has quit IRC18:52
*** glange has quit IRC18:52
*** chmouel has quit IRC18:52
*** cebreidian has quit IRC18:52
*** glange has joined #openstack-swift18:52
*** ChanServ sets mode: +v glange18:52
*** manous_ has joined #openstack-swift18:53
*** nadeem_ has joined #openstack-swift18:55
*** nadeem has quit IRC18:55
*** Raymii has joined #openstack-swift18:57
*** chmouel has joined #openstack-swift18:57
*** cebreidian has joined #openstack-swift18:58
*** dmorita has quit IRC18:58
*** AbyssOne has joined #openstack-swift18:59
zaitcevI implemented /info for Hummingbird because I thought that .functests would not run without it, and now I found that they work just fine, maybe skipping a few more tests than usual.19:01
tdasilvatimburke: trying to address your comment here: https://review.openstack.org/#/c/361439/2/swift/common/middleware/versioned_writes.py@819 are you suggestion raising an error like we do in keymaster check of encryption_root_secret?19:03
patchbotpatch 361439 - swift - Add option to set default version mode19:03
tdasilvaor just logging a warning?19:03
*** manous_ has quit IRC19:04
*** david-lyle_ is now known as david-lyle19:05
*** etienneme has joined #openstack-swift19:05
*** a1|away has joined #openstack-swift19:05
timburketdasilva: i was thinking error, since (1) it's a misconfiguration and (2) whatever they thought we were going to do, it's almost certainly *not* what's going to happen19:08
tdasilvayeah, i was debating on whether to just raise an error or to log and use the default 'stack'. I think raise error makes sense, especially since it follows what keymaster is already doing19:10
tdasilvaclayg: lol, just noticed all the emails now hehehe19:11
zaitcevme three19:13
claygi tried to ask in #openstack-dev if there's anything that can be done - but I think the whole thing is netsplit19:13
claygtdasilva: timburke: you guys are debating making the default value for x-versions-mode operator configurable yeah?19:15
tdasilvaclayg: correct19:15
timburkefwiw, i just have a rule that deletes everything with subject that includes "Fix proposed to swift (feature/" and containing the text "You received this bug notification because you are a member of Swift Core security contacts"19:15
claygtimburke: WELL DONE SIR19:16
tdasilvaclayg: got any opinions/suggestions?19:16
claygtdasilva: yeah I think it's shit for clients that they can't could on a swift endpoint doing the same thing when the create a container with x-versions-location unless they first check /info?19:17
clayg*can't count on19:17
*** manous_ has joined #openstack-swift19:17
claygdon't get wrong - i *hate* fucking stack mode x-verions-location and would love to deprecate it and all that jazz - but we shouldn't make it un-reliaable just because we hate it?19:18
claygin fact we probably just have to live with it19:18
timburkeclayg: on the other hand, you *can* rely on it if you always set x-versions-mode19:19
tdasilvaclayg: not sure I understand your point. it doesn't change anything for clients if they just want things to remain as they are today (assuming operators didn't change the default mode to history in the config file)19:19
timburkeworst case, old clusters will ignore it19:19
claygtimburke: and no client does that ever19:19
claygtimburke: but there's always hope for the future!19:20
timburkeclayg: and does that doom https://review.openstack.org/#/c/342857 from the start?19:21
patchbotpatch 342857 - swift - Add defaulter middleware19:21
claygtdasilva: "assuming operators didn't change the default" - I think the idea is that it would be great to give swift clusters going forward (either new ones, or existing ones) a way to move toward the far superior history based versions mode - but I'm giving them a config option that makes it so clients don't know what the x-versions-location header w/o x-versions-mode (new) actually does/means19:22
*** manous_ has quit IRC19:23
claygtimburke: well, as i'm sure you know we have some use-cases what an operator needs some ability to influence default client behavior - so it's hard to say that the defaulter is "doomed" - but it's cirtainly complicated19:24
tdasilvaclayg: I think I understand your point. basically you prefer that clients always send the header x-versions-mode if they want something different from the behavior that it is today.19:24
tdasilvaclayg: I think we just have to weigh that against the pro of not having to send yet another header to just set object versioning. I'm mainly thinking of green field installations that would just prefer to always default to history mode19:26
tdasilvaone one hand it is nice to always have to send x-versions-mode, OTOH it is nice to not have to send yet another header ;)19:27
claygtdasilva: yeah I think that's the head of the nail - if I had a time machine (swift 2.10 is not released?) i would probably *prefer* that x-versions-location become some sort of weird backwards compat thing and going forward we document "versioning" as "tell us the container you want to use for history mode based versioning; oh btw there's an older API that no one cares about"19:27
tdasilvaclayg: I kinda proposed that back in bristol19:28
claygtdasilva: yeah I think i was not in that conversation - i'm satisfied to have sour grapes here - but I'm not sure I'm convinced I'm wrong?19:29
claygI don't even like the x-versions-location header!  I can't ever remember if it's x-version-location or x-versions-location19:29
claygx-history-container would be a nice purly syntatic ehancement IMHO19:30
tdasilvaclayg: maybe there will be a yet new object versioning middleware using symlinks :P19:30
claygtdasilva: so true!?19:31
*** dmorita has joined #openstack-swift19:32
*** dmorita has quit IRC19:33
claygtdasilva: but I think the semantics of history mode versions are going to be the same, I think the symlinks may become *mostly* an implementation detail - but idk the way POST is going to work I think it would cirtainly be discoverable to the point that we'll probably have all three versions forever (yay!)19:33
tdasilvaheheh19:34
claygtdasilva: maybe in that respect it's good that we reserve x-history-container for the future history v2!?19:34
claygoh, or we could just co x-versions-location x-versions-mode: history x-versions-version: 2?19:35
*** dmorita has joined #openstack-swift19:35
*** dmorita has quit IRC19:35
tdasilvayeah, I agree with on those points. I envision for versioning to catch the 307 from symlink post and forward the POST to the correct place??? but that's as much thought i've given it. haven't though about the failure scnearios and how eventual consisntency might complicate things19:36
*** dmorita has joined #openstack-swift19:37
timburkehaving versioned_writes accept `x-history-container: ...` and split it out to `x-container-sysmeta-versions-mode: history` and `x-container-sysmeta-versions-location: ...` might not be horrible...19:37
timburketdasilva: clayg: the symlinks thing will be kinda messy no matter what. if we're going to work properly for containers that had data uploaded before versioning was enabled, we're still going to need to issue the GET on every PUT; we'd just need to watch for it being a symlink to an object in the archive container19:39
*** rcernin has joined #openstack-swift19:39
claygtimburke: ugh, that sound *so* hard - so glad you're thinking about it :'(19:41
tdasilvalol19:42
jrichliclayg: +119:42
clayg^ I'm going to pretend jrichli is agreeing that it's hard to remember x-versions-location vs. x-version-location19:43
jrichlithat is hard too!  but so is thinking about our symlink challenges19:43
*** dmorita has quit IRC19:44
tdasilvabtw all are welcome to check out patch 232162 :D19:45
patchbothttps://review.openstack.org/#/c/232162/ - swift - Symlink implementation.19:45
* tdasilva thinks the middleware implementation will ended up being easier than figuring out how clients will use it :P19:46
*** dmorita has joined #openstack-swift19:46
jrichlitdasilva: true.19:47
claygtimburke: you gotta link me that bug19:51
timburkeclayg: https://bugs.python.org/issue2223319:51
*** dmorita has quit IRC19:54
*** tqtran has quit IRC19:54
*** takashi has joined #openstack-swift19:55
*** dmorita has joined #openstack-swift19:55
*** tqtran has joined #openstack-swift19:56
*** dmorita has quit IRC20:07
*** tqtran has quit IRC20:08
*** tqtran has joined #openstack-swift20:09
*** tqtran has quit IRC20:12
*** tqtran has joined #openstack-swift20:21
*** gyee has joined #openstack-swift20:25
openstackgerritMerged openstack/swift: Merge remote-tracking branch 'origin/master' into feature/hummingbird  https://review.openstack.org/36311120:29
*** rcernin has quit IRC20:33
*** rcernin has joined #openstack-swift20:33
claygtimburke: that bug is epic20:37
*** klrmn has quit IRC20:39
*** dmorita has joined #openstack-swift20:48
*** asettle has joined #openstack-swift20:49
*** dmorita has quit IRC20:49
*** dmorita has joined #openstack-swift20:49
openstackgerritTimur Alperovich proposed openstack/swift: Close the iterators in string_along.  https://review.openstack.org/36319920:53
openstackgerritJoel Wright proposed openstack/python-swiftclient: Add swift tempurl and swift auth command docstring  https://review.openstack.org/36021220:58
*** jamielennox is now known as jamielennox|away21:06
*** joeljwright1 has joined #openstack-swift21:14
*** tqtran has quit IRC21:15
*** tqtran has joined #openstack-swift21:15
openstackgerritTimur Alperovich proposed openstack/swift: Close the iterators in string_along.  https://review.openstack.org/36319921:28
*** mingyu has quit IRC21:29
mattoliverauMorning21:48
notmynamehello, mattoliverau21:49
*** mingyu has joined #openstack-swift21:49
*** chrichip has joined #openstack-swift21:49
*** asettle has quit IRC21:51
*** asettle has joined #openstack-swift21:51
*** nadeem_ has quit IRC21:53
*** asettle has quit IRC21:56
*** McMurlock1 has quit IRC21:58
*** manous_ has joined #openstack-swift22:02
*** chrichip has quit IRC22:06
*** manous_ has quit IRC22:07
*** manous_ has joined #openstack-swift22:10
*** mingyu has quit IRC22:11
*** manous_ has quit IRC22:15
*** manous_ has joined #openstack-swift22:17
*** diogogmt has quit IRC22:20
*** manous_ has quit IRC22:22
*** manous_ has joined #openstack-swift22:27
*** joeljwright1 has quit IRC22:29
*** manous_ has quit IRC22:31
*** diogogmt has joined #openstack-swift22:32
*** manous_ has joined #openstack-swift22:33
*** klamath has quit IRC22:34
csmart\o22:37
mattoliveraucsmart: o/22:38
openstackgerritTim Burke proposed openstack/python-swiftclient: Accept gzip-encoded API responses  https://review.openstack.org/18495622:38
*** hosanai has joined #openstack-swift22:41
*** ChanServ sets mode: +v hosanai22:41
*** chsc has quit IRC22:42
pdardeaumattoliverau: o/22:45
mattoliveraupdardeau: hey man22:46
*** StraubTW has quit IRC22:57
*** takashi has quit IRC22:58
*** mingyu has joined #openstack-swift23:01
*** takashi has joined #openstack-swift23:07
*** mingyu has quit IRC23:08
*** tqtran has quit IRC23:09
*** tqtran has joined #openstack-swift23:14
*** Vinsh has quit IRC23:17
*** mingyu has joined #openstack-swift23:19
openstackgerritMerged openstack/python-swiftclient: Add swift tempurl and swift auth command docstring  https://review.openstack.org/36021223:21
*** takashi has quit IRC23:23
*** kei_yama has joined #openstack-swift23:27
*** rcernin has quit IRC23:35
*** mingyu has quit IRC23:39
*** mingyu has joined #openstack-swift23:51
*** Vinsh has joined #openstack-swift23:57

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