Bug #1537042 “versioned_writes middleware is mis-placed in proxy...” : Bugs : OpenStack Object Storage (swift)

Revision history for this message

Alistair Coles (alistair-coles) wrote on 2016-01-22:

#1

This is different behaviour than reported here https://bugs.launchpad.net/swift/+bug/1365862. I cannot reproduce bug 1365862.

Revision history for this message

Alistair Coles (alistair-coles) wrote on 2016-01-22:

#2

I have a func test patch that also reveals this bug: https://gist.github.com/alistairncoles/32a3f6674abfa1cbd6e5

Revision history for this message

Alistair Coles (alistair-coles) wrote on 2016-01-25:

#3

patch proposed here https://review.openstack.org/#/c/271883/1

Changed in swift:
assignee:	nobody → Janie Richling (jrichli)

OpenStack Infra (hudson-openstack) on 2016-01-25

Changed in swift:
status:	New → In Progress

Revision history for this message

Alistair Coles (alistair-coles) wrote on 2016-01-27:

#4

This bug only manifests when the versioned_writes middleware is inserted in the pipeline ahead (to left of) SLO. That can happen if the middleware is not explicitly configured but is auto-inserted incorrectly to left of SLO.

This bug does not manifest when versioned_writes is placed according to the proxy-server.conf sample i.e. after SLO. In that case this other bug manifests: https://bugs.launchpad.net/swift/+bug/1365862.

Alistair Coles (alistair-coles) on 2016-01-27

summary:	- SLO manifests become the large object when versioned + versioned_writes middleware is mis-placed in proxy pipeline
description:	updated
Changed in swift:
assignee:	Janie Richling (jrichli) → Alistair Coles (alistair-coles)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-01-27: Fix proposed to swift (master)

#5

Fix proposed to branch: master
Review: https://review.openstack.org/273073

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-01-30: Change abandoned on swift (master)

#6

Change abandoned by Janie Richling (<email address hidden>) on branch: master
Review: https://review.openstack.org/271883
Reason: In light of what the root problem was, abandoning in favor of https://review.openstack.org/#/c/272724/3

Revision history for this message

Janie Richling (jrichli) wrote on 2016-02-01:

#7

To be clear, both https://review.openstack.org/271883 and https://review.openstack.org/#/c/272724 fix the original title for this "SLO manifests become the large object when versioned". But neither of those changes address the pipeline ordering issue.

Revision history for this message

Alistair Coles (alistair-coles) wrote on 2016-02-02:

#8

For the record, the cause of the original description of this bug is:

- versioned_writes gets misplaced in pipeline to left of SLO

- during PUT handling, versioned writes creates a COPY req *with new environ* for the copy of the version to the versions container

- SLO inserts its copy_hook into the new req environ :(

- when obj controller calls back to the copy_hook, SLO performs segment assembly

- so the manifest version becomes the whole large object

When versioned_writes is correctly placed to the right of SLO in the pipeline:

- during handling a PUT request, SLO inserts its copy hook in the PUT req environ

- versioned writes creates a COPY req for the version copy *with a new req environ* and the slo copy_hook is NOT copied to the copy req environ

- during obj controller copy handling, no copy hook is found so manifest object gets copied without any intervention form SLO :)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-02-09: Fix merged to swift (master)

#9

Reviewed: https://review.openstack.org/273073
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=30d74af6534b754e6ad9bfdcbff4ec494277ca83
Submitter: Jenkins
Branch: master

commit 30d74af6534b754e6ad9bfdcbff4ec494277ca83
Author: Alistair Coles <email address hidden>
Date: Wed Jan 27 13:50:57 2016 +0000

Insert versioned_writes in correct pipeline position

    If not explicitly configured the versioned_writes middleware
    should be auto-inserted in the pipeline after slo and dlo, which
    is where the versioned_writes filter section's comments say it
    should be in proxy-server.conf-sample. At the moment it can end up
    being placed ahead of slo and dlo if they have been explicitly
    configured, which results in the linked bug manifesting.

Closes-Bug: #1537042
Change-Id: I6ac95a331f4ef0d4887311940acc6f8bc00fb4eb

Changed in swift:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-02-10: Fix proposed to swift (feature/crypto)

#10

Fix proposed to branch: feature/crypto
Review: https://review.openstack.org/278283

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-02-10: Fix merged to swift (feature/crypto)

#11

Download full text (17.2 KiB)

Reviewed: https://review.openstack.org/278283
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=adbda24e41dbd8b96d80a54be89513d17da3114e
Submitter: Jenkins
Branch: feature/crypto

commit 87fc21c7cfcce2a3e23a84fddcfd1309cd884716
Author: Alistair Coles <email address hidden>
Date: Mon Feb 8 15:31:42 2016 +0000

Speed up functional testing

    test/functional/tests.py:TestObjectVersioningUTF8 does not clean
    up the versions files it creates because the class's multiple
    inheritance is such that it does not call the tearDown method in
    TestObjectVersioning.

    As a result, any attempt to clean up account containers wastes
    time retrying container delete requests. This occurs either in
    the setUp for TestSloEnv, if the TestSlo class is included in a
    test run, or in the tests.py package tearDown method.

On the author's dev machine this patch reduces the execution
time of functional tests in tests.py by approx 30% or 1 minute.

Change-Id: I8194672bf2ca82435df5868720b6a55a79b94413

commit b173995666b026def3f6558d9a8c972b75449323
Author: Kota Tsuyuzaki <email address hidden>
Date: Thu Feb 4 05:18:42 2016 -0800

Fix missing Accept-Ranges

Since commit 4f2ed8bcd0468f3b69d5fded274d8d6b02ac3d10, the response
header for GET EC object doesn't include the Accept-Ranges header.

This patch fixes it and also adds a few unittests to prevent regression.

Closes-Bug: #1542168

Change-Id: Ibafe56ac87b14bc0028953e620a653cd68dd3f84

commit ae632abbd802228ef98a23c98ee49988e7e5b942
Author: Ondřej Nový <email address hidden>
Date: Tue Feb 2 21:58:52 2016 +0100

Fixed manpages errors.

account-server.conf.5
105: warning: numeric expression expected (got `)')

container-server.conf.5
111: warning: numeric expression expected (got `)')

object-expirer.conf.5
79: warning: numeric expression expected (got `)')

object-server.conf.5
114: warning: numeric expression expected (got `)')

    proxy-server.conf.5
    121: warning: numeric expression expected (got `)')
    331: warning: numeric expression expected (got `[')
    1005: warning: macro `*' not defined

Change-Id: I203dcfde83035e3b1dcb91109b72b5d08bb7840e

commit e47aaaacf15e8f0c7226400541a97238447a7f56
Author: Tim Burke <email address hidden>
Date: Wed Feb 3 12:52:29 2016 -0800

Stop nesting functions unnecessarily

Change-Id: Iff120d0bac8a075c37bbddcd2bb0fe85145f1749

commit 26327e1e8b1d37faa764ec586f5bee0e1560eea2
Author: Darrell Bishop <email address hidden>
Date: Thu Jan 21 11:18:18 2016 -0800

Allow IPv6 addresses/hostnames in StatsD target

    The log_statsd_host value can now be an IPv6 address or a hostname
    which only resolves to an IPv6 address. In both cases, the new
    behavior is to use an AF_INET6 socket on which .sendto() is called
    with the originally-configured hostname (or IP). This means the
    Swift process is not caching a DNS resolution for the lifetime of
    the process (a good thing).

If a hostname resolves to both an IPv6 or IPv4 address, an AF_IN...

Reviewed:  https://review.openstack.org/278283
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=adbda24e41dbd8b96d80a54be89513d17da3114e
Submitter: Jenkins
Branch:    feature/crypto

commit 87fc21c7cfcce2a3e23a84fddcfd1309cd884716
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Mon Feb 8 15:31:42 2016 +0000

Speed up functional testing
    
    test/functional/tests.py:TestObjectVersioningUTF8 does not clean
    up the versions files it creates because the class's multiple
    inheritance is such that it does not call the tearDown method in
    TestObjectVersioning.
    
    As a result, any attempt to clean up account containers wastes
    time retrying container delete requests. This occurs either in
    the setUp for TestSloEnv, if the TestSlo class is included in a
    test run, or in the tests.py package tearDown method.
    
    On the author's dev machine this patch reduces the execution
    time of functional tests in tests.py by approx 30% or 1 minute.
    
    Change-Id: I8194672bf2ca82435df5868720b6a55a79b94413

commit b173995666b026def3f6558d9a8c972b75449323
Author: Kota Tsuyuzaki <tsuyuzaki.kota@lab.ntt.co.jp>
Date:   Thu Feb 4 05:18:42 2016 -0800

Fix missing Accept-Ranges
    
    Since commit 4f2ed8bcd0468f3b69d5fded274d8d6b02ac3d10, the response
    header for GET EC object doesn't include the Accept-Ranges header.
    
    This patch fixes it and also adds a few unittests to prevent regression.
    
    Closes-Bug: #1542168
    
    Change-Id: Ibafe56ac87b14bc0028953e620a653cd68dd3f84

commit ae632abbd802228ef98a23c98ee49988e7e5b942
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Tue Feb 2 21:58:52 2016 +0100

Fixed manpages errors.
    
    account-server.conf.5
    105: warning: numeric expression expected (got `)')
    
    container-server.conf.5
    111: warning: numeric expression expected (got `)')
    
    object-expirer.conf.5
    79: warning: numeric expression expected (got `)')
    
    object-server.conf.5
    114: warning: numeric expression expected (got `)')
    
    proxy-server.conf.5
    121: warning: numeric expression expected (got `)')
    331: warning: numeric expression expected (got `[')
    1005: warning: macro `*' not defined
    
    Change-Id: I203dcfde83035e3b1dcb91109b72b5d08bb7840e

commit e47aaaacf15e8f0c7226400541a97238447a7f56
Author: Tim Burke <tim.burke@gmail.com>
Date:   Wed Feb 3 12:52:29 2016 -0800

Stop nesting functions unnecessarily
    
    Change-Id: Iff120d0bac8a075c37bbddcd2bb0fe85145f1749

commit 26327e1e8b1d37faa764ec586f5bee0e1560eea2
Author: Darrell Bishop <darrell@swiftstack.com>
Date:   Thu Jan 21 11:18:18 2016 -0800

Allow IPv6 addresses/hostnames in StatsD target
    
    The log_statsd_host value can now be an IPv6 address or a hostname
    which only resolves to an IPv6 address.  In both cases, the new
    behavior is to use an AF_INET6 socket on which .sendto() is called
    with the originally-configured hostname (or IP).  This means the
    Swift process is not caching a DNS resolution for the lifetime of
    the process (a good thing).
    
    If a hostname resolves to both an IPv6 or IPv4 address, an AF_INET
    socket is used (i.e. only the IPv4 address will receive the UDP
    packet).
    
    The old behavior is preserved: any invalid IP address literals and
    failures in DNS resolution or actual StatsD packet sending do not
    halt the process or bubble up; they are caught, logged, and
    otherwise ignored.
    
    Change-Id: Ibddddcf140e2e69b08edf3feed3e9a5fa17307cf

commit 2f8e6040470948cf17c5fc5fddd0991a415d2a4c
Author: Eran Rom <eranr@il.ibm.com>
Date:   Wed Feb 3 08:59:53 2016 +0200

Fixing typo in unit test
    
    The typo does not influence the correctness of the test
    yet its ugly. bhhha.
    
    Change-Id: I62e511bae23523892b87309dd3c50aea61742f2d

commit f27ad34e1d4e5dc306eed3ff620a294a4c301f31
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Tue Feb 2 11:44:39 2016 -0800

Document use-case for slow option
    
    Change-Id: Iec4087a896a2277179e3720d802cca101fa7ad54

commit 16976a0f14ee2b5f7abc76086f9abfb9f337a655
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Sun Jan 31 10:30:06 2016 +0100

Changed EC backend from jerasure to liberasurecode in examples and docs
    
    liberasurecode_rs_vand is build-in liberasurecode, so you don't need
    another depedency libjerasure2.
    
    liberasurecode_rs_vand is supported by pyeclib from 1.0.8
    version, so bumping version up.
    
    Closes-Bug: #1534325
    Change-Id: If2d96875694df8fd48c5278395859aaa165cb566

commit 30d74af6534b754e6ad9bfdcbff4ec494277ca83
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Wed Jan 27 13:50:57 2016 +0000

Insert versioned_writes in correct pipeline position
    
    If not explicitly configured the versioned_writes middleware
    should be auto-inserted in the pipeline after slo and dlo, which
    is where the versioned_writes filter section's comments say it
    should be in proxy-server.conf-sample. At the moment it can end up
    being placed ahead of slo and dlo if they have been explicitly
    configured, which results in the linked bug manifesting.
    
    Closes-Bug: #1537042
    Change-Id: I6ac95a331f4ef0d4887311940acc6f8bc00fb4eb

commit ccdf4a9f309af5fe1401a6e4b776b08c3f6a655c
Author: Christian Schwede <cschwede@redhat.com>
Date:   Tue Feb 2 09:38:55 2016 +0100

Document slow option in etc/object-server.conf
    
    Change-Id: Ic9940b0b830a468887878f7b0d7ca42c2cbbebd5

commit b19dc1ddecdc68760ca2f5a2b7b4bbdce2199b35
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Mon Feb 1 15:30:59 2016 -0800

Always fix devices with multiple part-replica assignments
    
    I've found that given a sufficiently bad replica2part2dev table we can
    accidently not entirely fix palcement when more than two replicas of a
    part are assigned to the duplicate devices.
    
    It shows up most on > 3 replica rings when you have two *different*
    devices both holding two replicas.  But you can see it on a three
    replica ring when all three replicas are assigned to the same device.
    
    Change-Id: Ieb213c1a259815a2ed657291242919cda568c7b5

commit c722fa1ab318b233fc11f8e1bd8758dbe33e87bb
Author: dangming <dangming@unitedstack.com>
Date:   Mon Feb 1 16:13:54 2016 +0800

Fix spelling error
    
    Change-Id: I4a1400c4d383acd59727ada3661a8dffc67825fb

commit 4ba290bac19b1b6e8f709848fd35cda33b8bd1c4
Author: Qiaowei Ren <qiaowei.ren@intel.com>
Date:   Mon Feb 1 16:44:52 2016 +0800

Replace exit() by sys.exit()
    
    sys.exit() is considered good to use here.
    
    Change-Id: I365b0dbb50c8f1953b41665ba49145ac6db1b24f

commit 9aff76bd18e609eb657db14f2d60e605ee28d472
Author: Qiaowei Ren <qiaowei.ren@intel.com>
Date:   Mon Feb 1 17:15:32 2016 +0800

fix typo
    
    informations -> information
    
    Change-Id: I6bcb9bb9fd24b873b49de4a686b223550dd703ac

commit a8e0dc550bf35010595ce791433cd68c4b8d5287
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Sun Jan 31 06:15:17 2016 +0000

Imported Translations from Zanata
    
    For more information about this automatic import see:
    https://wiki.openstack.org/wiki/Translations/Infrastructure
    
    Change-Id: If1cd20e187720f5ee02c6a1574f612c8a8e26e30

commit 02c06585e6e0403d1a983fccaba5093e3d3fa7c5
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Fri Jan 29 22:46:50 2016 +0100

Renamed variable for better code readability
    
    Change-Id: I22d8db0dd9edc39672fc9997895a24f669975e15

commit 595ec464e3fdc5a766eee9baa06edd074576c73e
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Fri Jan 29 15:28:56 2016 +0000

Trivial fix for sphinx warnings
    
    Including a * in the param name causes a sphinx warning,
    messes up the html formatting, and strictly the * isn't part of
    the param name.
    
    Change-Id: I6be6a7aa2a371eaae9f8e501435838c1e15f4a1c

commit cc90834a8ed7bd84440bf132a94978a47ee2f3da
Author: Mingyu Li <li.mingyu@99cloud.net>
Date:   Fri Jan 29 03:48:47 2016 +0900

A minor modification of the Deployment Guide
    
    In the section of General Service Tuning, there is a sentence
    "Our Storage servers all run together on the same servers".
    The word "servers" is used twice with different meanings, which
    would be a little confusing. This patch makes it more understandable.
    
    Change-Id: I6d806456aeb7d0ecf297c04dacbd271b94a0425c
    Closes-Bug: #1526697

commit a1776b9c1f23fbd604114b377434e4269d2f1a2e
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Thu Jan 28 16:47:37 2016 +0000

Let equal Timestamps not be unequal
    
    Make the result of Timestamp(x) != Timestamp(x) be False.
    
    In python 2.7 this requires the __ne__ method to be defined [1].
    "The truth of x==y does not imply that x!=y is false." The
    functools.total_ordering decorator does not autocreate a __ne__
    method.
    
    In python 3 the __ne__ method is not required [2]. "By default,
    __ne__() delegates to __eq__() and inverts the result".
    
    This patch puts back the __ne__ method removed in [3]. Whilst no tests
    fail on master with python2.7, they do on this patch [4] and it seems
    dangerous to have this absurd behaviour lurking.
    
    [1] https://docs.python.org/2/reference/datamodel.html#object.__ne__
    [2] https://docs.python.org/3.4/reference/datamodel.html#object.__ne__
    [3] Change-Id: Id26777ac2c780316ff10ef7d954c48cc1fd480b5
    [4] Change-Id: Ia597cd460bb5fd40aa92e886e3e18a7542603d01
    
    Change-Id: I01fbfa310df3c74390f8e8c2e9ffff81bbf05e47

commit c3886eea15f11104764fb201e85202dd871c3027
Author: Thomas Goirand <thomas@goirand.fr>
Date:   Thu Jan 28 09:15:28 2016 +0000

Fix a few English mistakes in man
    
    These errors are producing lintian warnings, so fixing them
    helps having less errors when checking for Debian packages.
    
    Change-Id: Iff99a8d5f2276515f42d758d110a43cae757db28

commit 400cc707273bab0b70a3cdf5dec1c55c2edc4e22
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Wed Jan 27 15:35:40 2016 +0000

Remove a print statement from tests
    
    Change-Id: I213b7e4bc4f9e6de76ebab572c377af61072d9c5

commit e45d30d6b7b596b80c63e75018ad07582d705ee1
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Wed Jan 27 10:18:37 2016 +0000

Remove unnecessary code from test_account.py
    
    Change-Id: Ia197ba08a606421244d5b94a900bdc39bc5da647

commit 9ef15453fadb847db0f0fb92e935785b544da3fc
Author: Kazuhiro MIYAHARA <miyahara.kazuhiro@lab.ntt.co.jp>
Date:   Thu Jan 21 14:18:31 2016 +0900

Fix handling of "Permission Denied" error from NamedTemporaryFile function
    
    If "Permission Denied" has happen in NamedTemporaryFile function in
    dump_recon_cache method, swift will log a message of reference to a variable
    without assignment and not log a message of "Permission Denied".
    This patch fixes the handling and add an unit test.
    
    Co-Authored-By: Kota Tsuyuzaki <tsuyuzaki.kota@lab.ntt.co.jp>
    Change-Id: Iafdd94905e9e9c81f5966a923324b50c18fcf592

commit 78cfb137d5d3d6f4770923a1fb086adaa9c0676f
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Wed Nov 11 19:05:09 2015 -0800

Ring Overview updates
    
    Change-Id: Ic5a8ec5be9fec7fa812fdb7b91890d5fc2fbf967

commit 24227e8bb0d4d369eeab61ab5c20730b4f84b36a
Author: Richard Hawkins <richard.hawkins@rackspace.com>
Date:   Wed Jan 13 10:16:29 2016 -0600

_get_info_cache should encode unicode metadata
    
    Closes-Bug: 1533768
    Change-Id: I67bbd0e8802d2a34cb3a0b0fafe26bbc62221032

commit fc9a3852e44b2bf1258fcb24a6c12ecfb2795f44
Author: Tim Burke <tim.burke@gmail.com>
Date:   Mon Jan 25 09:25:20 2016 -0800

Don't .lower() a literal
    
    It feels silly, and we don't do it for any of the other headers in
    headers_to_container_info.
    
    While we're at it, clean up a stray '
    
    Change-Id: I0745038cc3832a77d064e515c37cacbdcb97c4d9
    Related-Change: Iea3d06de80210e9e504e296d4572583d7ffabeac

commit 5c6576d1eb3c4c042bb46dd48f23d2fd474eba07
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Mon Jan 25 18:20:45 2016 +0100

Fixed docs: Title underline too short
    
    Change-Id: If96b0e799a4671e7b7f3f5bea2628db75afa9578

commit 9c403d5e1bb1f404445f1731e3ff1b5c034336a6
Author: Victor Stinner <vstinner@redhat.com>
Date:   Mon Oct 19 16:31:12 2015 +0200

Port swift.common.utils.Timestamp to Python 3
    
    * Fix Timestamp.isoformat(): on Python 3, round manually to use the
      same rounding method than Python 2.
    * Use @functoos.total_ordering on Timestamp instead of the __cmp__()
      special method. On Python 3, the __cmp__() method is no more used,
      and other special methods must be defined (__eq__, __lt__, etc.).
      Only define __eq__() and __lt__() methods but use
      @functools.total_ordering to generate other methods.
    * Fix also a Python 3 issues in swift.common.swob._datetime_property:
      replace (float, int, long) with (float,) + six.integer_types. The
      long type was renamed to int in Python 3.
    
    Change-Id: Id26777ac2c780316ff10ef7d954c48cc1fd480b5

commit c790db89369221473bcbfc23f301424ab8d30d60
Author: Andrew Welleck <awellec@us.ibm.com>
Date:   Mon Jan 25 10:03:18 2016 -0600

Additional step for Debugging Issues in SAIO
    
    Adding an additional debugging step for the SAIO "Debugging Issues"
    section. Adding step to check if memcache is running and to start
    it if its not.
    
    Change-Id: I0bfa563f595dbefcf907494451fad45e6310db88

commit d47155af262d52c284f262bf6c303d0587000bba
Author: Victor Stinner <vstinner@redhat.com>
Date:   Mon Oct 19 17:22:24 2015 +0200

Add __next__() methods to utils iterators for py3
    
    On Python 3, next(obj) calls obj.__next__(), not obj.next(). Add an
    alias from __next__() to next() to be compatible with Python 2 and
    Python 3.
    
    Change-Id: Ida104d3bd7cdba557e523f18df43d56847060054

commit 0f8319dceb2d21a9d1ec74f78203b033f25c0969
Author: Tim Burke <tim.burke@gmail.com>
Date:   Fri Jan 22 13:31:34 2016 -0800

Use calendar.timegm not time.mktime
    
    time.mktime produces a POSIX timestamp that takes into account the local
    timezone, but we're parsing a Last-Modified header with a known (GMT)
    timezone. As a result, if one proxy-server is configured with a timezone
    other than GMT/UTC, we may have previous versions in the wrong order.
    
    Change-Id: I320e2368b243f4245725e73bfabc7ad19bc5bacb

commit 72d972166ff66d3c5313356c1e1e3984fee9f012
Author: paul luse <paul.e.luse@intel.com>
Date:   Thu Jan 21 13:30:15 2016 -0700

Update AUTHORS
    
    +Luse core emeritus
    
    Change-Id: Ie175f410ddc24a2d5e64bc191364418a3c450f3f

commit cec2aa0b4dbf0e1bf2de62945f5abdf494925a8e
Author: Victor Stinner <vstinner@redhat.com>
Date:   Fri Oct 9 17:56:39 2015 +0200

py3: Use the six module in the xprofile middleware
    
    * Replace bytes_() with a isinstance(content, six.text_type)
      and content.encode('utf-8')
    * Get parse_qs() from six.moves.urllib.parse
    
    Change-Id: I4a506975814bff098ade25c950c154e50dc9626e

commit 6c32da14f48b435aa4e5efea48904bd288a532a2
Author: Victor Stinner <vstinner@redhat.com>
Date:   Mon Oct 19 16:38:24 2015 +0200

Port FileLikeIter to Python 3
    
    Port FileLikeIter and _MultipartMimeFileLikeObject and
    swift.common.utils to Python 3:
    
    * Add a __next__() alias to the next() method. On Python 3, the
      next() method is no more used, __next__() is required.
    * Use literal byte strings: FileLikeIter _MultipartMimeFileLikeObject
      are written to handle binary files.
    * test_close(): replace .FileLikeIter('abcdef') with
      FileLikeIter([b'a', b'b', b'c']). On Python 3, list(b'abc') returns
      [97, 98, 99], whereas ['a', 'b', 'c'] is returned on Python 2.
    * Update unit FileLikeIter tests to use byte strings.
    
    Change-Id: Ibacddb70b22f624ecd83e374749578feddf8bca8

commit caa127d59a63a25cc6d08e985e536f90300a6156
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Thu Nov 12 11:34:44 2015 -0800

Monkeypatch everything the same
    
    There's three spots in the code where eventlet monkeypatches
    things. In one place, it was monkeypatching socket and thread; in the
    other two, only socket. This commit monkeypatches socket and thread
    everywhere.
    
    Hopefully, this lets us avoid code that, for example, works in the
    object server but fails in the object updater.
    
    Change-Id: Ia30d069c5ac89f4bfbe52233d03d96738144b614

commit f56f29ef7a692b6c3204e25d41eace3d68dceece
Author: Peter Lisák <peter.lisak@firma.seznam.cz>
Date:   Thu Dec 3 16:48:18 2015 +0100

Add info about state of ring file to default command.
    
    Try to find ring file, load and compare it with builder file, then show result state.
    Examples:
    Ring file object.ring.gz not found, probably it hasn't been written yet
    Ring file object.ring.gz is up-to-date
    Ring file object.ring.gz is obsolete
    Ring file object.ring.gz is invalid: ValueError('string length not a multiple of item size',)
    
    Change-Id: I4d769aa5fe1c2b1167ec088aa372874f7d13ae48

commit 165fa1fd40b5e4b13cf5188712593e2275bef821
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Fri Jan 15 11:44:21 2016 -0800

add test for zero weight region get_more_nodes
    
    Change-Id: If537981e8deadd9c3528dcb30a15011c7781e334

tags:

added: in-feature-crypto

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-03-08: Fix proposed to swift (feature/hummingbird)

#12

Fix proposed to branch: feature/hummingbird
Review: https://review.openstack.org/290148

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-03-08: Fix merged to swift (feature/hummingbird)

#13

Download full text (71.7 KiB)

Reviewed: https://review.openstack.org/290148
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=0f7f1de233919a0b046349a3e31ae7fc8675a1c5
Submitter: Jenkins
Branch: feature/hummingbird

commit d6b4587a554b51ba733b151e0d924735b63d07e0
Author: Olga Saprycheva <email address hidden>
Date: Tue Mar 8 10:57:56 2016 -0600

Removed redundant file for flake8 check

Change-Id: I4322978aa20ee731391f7709bbd79dee140fc703

commit 643dbce134140530eef2ae62c42fef1107f905ed
Author: OpenStack Proposal Bot <email address hidden>
Date: Tue Mar 8 06:35:49 2016 +0000

Imported Translations from Zanata

For more information about this automatic import see:
https://wiki.openstack.org/wiki/Translations/Infrastructure

Change-Id: I96b8ff1287bf219c5f8d56a3a4868c1063a953f9

commit 83713d37f0331c5ce9d377f4b4e8724551ae30ca
Author: Daisuke Morita <email address hidden>
Date: Mon Mar 7 18:30:47 2016 -0800

Missing comments for storage policy parameter

There are missing comments about storege_policy_index so appropriate
comments are added.

Change-Id: I3de3f0e6864e65918ca1a13cce70f19c23d295f5

commit 2cff2dec3d1c4588f5103e39679c43b3dded6dcb
Author: Olga Saprycheva <email address hidden>
Date: Fri Mar 4 15:19:39 2016 -0600

Fixed pep8 and flake8 errors in doc/source/conf.py and updated flake8 commands in tox.ini to test it.

Change-Id: I2add370e4cfb55d1388e3a8b41f688a7f3f2c621

commit 043fbca6d08648baa314ea2236f1ccdca8785f16
Author: Christian Schwede <email address hidden>
Date: Fri Mar 4 09:33:17 2016 +0000

Remove Erasure Coding beta status from docs

    This removes notes stating support for Erasure coding as beta. Questions
    regarding the stability of EC are coming up regularly, and are often referring
    to the docs that state EC as still in beta.

Besides this, a note marking statsd support as beta has been removed as well.

Change-Id: If4fb6a5c4cb741d42953db3cee8cb17a1d774e15

commit 09c73b86e9255f28fbd4cf571a52c17d549a8f9a
Author: Pete Zaitcev <email address hidden>
Date: Thu Mar 3 10:24:28 2016 -0700

Fix a crash in exception printout

Says the number of arguments does not match the number of '%'.

Change-Id: I8b5e395a07328fb9d4ac7a19f8ed2ae1637bee3b

commit fad5fabe0a22e8a86635a66523dd3d3d3b1fa705
Author: Tim Burke <email address hidden>
Date: Thu Mar 3 15:07:08 2016 +0000

During functional tests, 404 response to a DELETE is successful

Previously, we would only consider 204 responses successful, which would
cause some spurious gate failures, such as

http://logs.openstack.org/66/287666/3/check/gate-swift-dsvm-functional/c6d2673/console.html#_2016-03-03_13_41_07_846

Change-Id: Ic8c300647924352a297a2781b50064f7657038b4

commit e91de49d6864b3794f8dc5acd9c1bf0c2f7409d1
Author: Alistair Coles <email address hidden>
Date: Mon Aug 10 10:30:10 2015 -0500

Update container on fast-POST

    This patch makes a number of changes to enable content-type
    metadata to be updated when using the fast-POST mode of
    operation, as proposed in the associated spec ...

Reviewed:  https://review.openstack.org/290148
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=0f7f1de233919a0b046349a3e31ae7fc8675a1c5
Submitter: Jenkins
Branch:    feature/hummingbird

commit d6b4587a554b51ba733b151e0d924735b63d07e0
Author: Olga Saprycheva <osapryc@us.ibm.com>
Date:   Tue Mar 8 10:57:56 2016 -0600

Removed redundant file for flake8 check
    
    Change-Id: I4322978aa20ee731391f7709bbd79dee140fc703

commit 643dbce134140530eef2ae62c42fef1107f905ed
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Tue Mar 8 06:35:49 2016 +0000

Imported Translations from Zanata
    
    For more information about this automatic import see:
    https://wiki.openstack.org/wiki/Translations/Infrastructure
    
    Change-Id: I96b8ff1287bf219c5f8d56a3a4868c1063a953f9

commit 83713d37f0331c5ce9d377f4b4e8724551ae30ca
Author: Daisuke Morita <morita.daisuke@ntti3.com>
Date:   Mon Mar 7 18:30:47 2016 -0800

Missing comments for storage policy parameter
    
    There are missing comments about storege_policy_index so appropriate
    comments are added.
    
    Change-Id: I3de3f0e6864e65918ca1a13cce70f19c23d295f5

commit 2cff2dec3d1c4588f5103e39679c43b3dded6dcb
Author: Olga Saprycheva <osapryc@us.ibm.com>
Date:   Fri Mar 4 15:19:39 2016 -0600

Fixed pep8 and flake8 errors in doc/source/conf.py and updated flake8 commands in tox.ini to test it.
    
    Change-Id: I2add370e4cfb55d1388e3a8b41f688a7f3f2c621

commit 043fbca6d08648baa314ea2236f1ccdca8785f16
Author: Christian Schwede <cschwede@redhat.com>
Date:   Fri Mar 4 09:33:17 2016 +0000

Remove Erasure Coding beta status from docs
    
    This removes notes stating support for Erasure coding as beta. Questions
    regarding the stability of EC are coming up regularly, and are often referring
    to the docs that state EC as still in beta.
    
    Besides this, a note marking statsd support as beta has been removed as well.
    
    Change-Id: If4fb6a5c4cb741d42953db3cee8cb17a1d774e15

commit 09c73b86e9255f28fbd4cf571a52c17d549a8f9a
Author: Pete Zaitcev <zaitcev@kotori.zaitcev.us>
Date:   Thu Mar 3 10:24:28 2016 -0700

Fix a crash in exception printout
    
    Says the number of arguments does not match the number of '%'.
    
    Change-Id: I8b5e395a07328fb9d4ac7a19f8ed2ae1637bee3b

commit fad5fabe0a22e8a86635a66523dd3d3d3b1fa705
Author: Tim Burke <tim.burke@gmail.com>
Date:   Thu Mar 3 15:07:08 2016 +0000

During functional tests, 404 response to a DELETE is successful
    
    Previously, we would only consider 204 responses successful, which would
    cause some spurious gate failures, such as
    
    http://logs.openstack.org/66/287666/3/check/gate-swift-dsvm-functional/c6d2673/console.html#_2016-03-03_13_41_07_846
    
    Change-Id: Ic8c300647924352a297a2781b50064f7657038b4

commit e91de49d6864b3794f8dc5acd9c1bf0c2f7409d1
Author: Alistair Coles <alistair.coles@hp.com>
Date:   Mon Aug 10 10:30:10 2015 -0500

Update container on fast-POST
    
    This patch makes a number of changes to enable content-type
    metadata to be updated when using the fast-POST mode of
    operation, as proposed in the associated spec [1].
    
    * the object server and diskfile are modified to allow
      content-type to be updated by a POST and the updated value
      to be stored in .meta files.
    
    * the object server accepts PUTs and DELETEs with older
      timestamps than existing .meta files. This is to be
      consistent with replication that will leave a later .meta
      file in place when replicating a .data file.
    
    * the diskfile interface is modified to provide accessor
      methods for the content-type and its timestamp.
    
    * the naming of .meta files is modified to encode two
      timestamps when the .meta file contains a content-type value
      that was set prior to the latest metadata update; this
      enables consistency to be achieved when rsync is used for
      replication.
    
    * ssync is modified to sync meta files when content-type
      differs between local and remote copies of objects.
    
    * the object server issues container updates when handling
      POST requests, notifying the container server of the current
      immutable metadata (etag, size, hash, swift_bytes),
      content-type with their respective timestamps, and the
      mutable metadata timestamp.
    
    * the container server maintains the most recently reported
      values for immutable metadata, content-type and mutable
      metadata, each with their respective timestamps, in a single
      db row.
    
    * new probe tests verify that replication achieves eventual
      consistency of containers and objects after discrete updates
      to content-type and mutable metadata, and that container-sync
      sync's objects after fast-post updates.
    
    [1] spec change-id: I60688efc3df692d3a39557114dca8c5490f7837e
    
    Change-Id: Ia597cd460bb5fd40aa92e886e3e18a7542603d01

commit 3c61ab4678a7aa9ff256ace4bc97ab449607fd49
Author: asettle <alexandra.settle@rackspace.com>
Date:   Wed Feb 10 17:58:05 2016 +1000

Operational procedures guide
    
    This is the operational procedures guide that HPE used
    to operate and monitor their public Swift systems.
    It has been made publicly available.
    
    Change-Id: Iefb484893056d28beb69265d99ba30c3c84add2b

commit 8e4347afd59aa30708ef03a78202652aaadda2e9
Author: Takashi Kajinami <kajinamit@nttdata.co.jp>
Date:   Tue Nov 17 16:15:59 2015 +0900

Fix proxy-server's support for chunked transferring in GET object
    
    Proxy-server now requires Content-Length in the response header
    when getting object and does not support chunked transferring with
    "Transfer-Encoding: chunked"
    
    This doesn't matter in normal swift, but prohibits us from putting
    any middelwares to execute something like streaming processing of
    objects, which can't calculate the length of their response body
    before they start to send their response.
    
    Change-Id: I60fc6c86338d734e39b7e5f1e48a2647995045ef

commit 9dda1c9526c5b3e0ac905985921605d833bef3cc
Author: Takashi Kajinami <kajinamit@nttdata.co.jp>
Date:   Fri Dec 25 10:30:01 2015 +0900

Update parameters about authtoken middleware in proxy-server.conf.sample
    
    This patch replaces some configration parameters about authtoken
    middleware in proxy-server.conf, which will be deprecated.
    
    The following parameters are to be deprecated.
     auth_admin_prefix, auth_host, auth_port, auth_protocol,
     identity_uri, admin_token, admin_user, admin_password,
     admin_tenant_name
    
    Change-Id: I373734933189c87c4094203b0752dd3762689034

commit 087fa4fa01fc36343f2ce0974d3b36ec0b98afc8
Author: Andreas Jaeger <aj@suse.com>
Date:   Mon Feb 29 20:00:38 2016 +0100

Remove unused pngmath Sphinx extension
    
    There's no RST file that uses ".. math" and thus
    the pngmath Sphinx extension is not used and can
    get removed.
    
    Change-Id: I3ea4b529025c8ac3c4092f3720124647861e6668

commit 9c5b01e7c7699fbfd221e98ccde1afa9361c8119
Author: Hisashi Osanai <osanai.hisashi@jp.fujitsu.com>
Date:   Wed Dec 9 08:08:22 2015 +0000

Add container quota functional test
    
    This patch provides one container quota functional test.
    
    Change-Id: I16354c15fc434d70f9bb6b9b1e18b09357195c83

commit efdc6ee8dbc434e9189a6b85b924c322c3ff6ae3
Author: Hisashi Osanai <osanai.hisashi@jp.fujitsu.com>
Date:   Thu Dec 10 01:15:30 2015 +0000

Add X-Delete-After and X-Delete-At functional test
    
    This patch provides functional tests for X-Delete-After and X-Delete-At.
    
    Change-Id: I55b0d0584428413aa7a31f66defec55f9b1bbcd9

commit bd93d44bb46350f716f1dfb66fb43c2ab4ad8c50
Author: Takashi Kajinami <kajinamit@nttdata.co.jp>
Date:   Sun Feb 28 13:41:19 2016 +0900

Make sure all temp files get deleted in test_ringbuilder
    
    This patch makes test_ringbuilder create a temporal directory,
    run ring builder commands under it, and delete it after testing
    for each test cases, to fix temp file leaking.
    
    Change-Id: I6f59fe095ea6485af0e60b5a8e8fc3892e0a0f90

commit 10b5765c60790f8a3ae2a94d6a51736ec2c99ce0
Author: benjkeller <bjkeller@us.ibm.com>
Date:   Thu Feb 25 18:54:00 2016 -0600

Rename test_object_delete_at_aysnc
    
    This renames test_object_delete_at_aysnc to
    test_object_delete_at_async to match with the function
    async_update which it tests.
    
    Change-Id: I726afcbf08a3449c7af2834e573b97be378a86cd
    Closes-Bug: 1550067

commit 9e53bb47efa28983109da8ceca5d316925bc6804
Author: John Dickinson <me@not.mn>
Date:   Wed Feb 24 09:36:40 2016 -0800

updated some weird whitespace character to a normal one
    
    Change-Id: I187fb8fc8d10f950bbebc586999eced0bc117432

commit 54e53ae87b6e87229dc77cb7c814ab9733ea7013
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Fri Feb 19 09:30:10 2016 +0000

Silence SkipTest noise in in-process test teardown
    
    setup_package and teardown_package get called twice when
    running functional tests using nose. With in-process mode
    the first call to teardown_package clears global config
    which causes the test swiftclient to raise a SkipTest error
    during the second call to teardown_package.
    
    To reproduce:
    SWIFT_TEST_IN_PROCESS=1 nosetests ./test/functional/test_account.py
    
    This patch simply tests for config existence before attempting
    to create a test swiftclient in the teardown_package function.
    
    Also fix a related issue whereby in_process flag would be reset to
    False during second call to setup_package, thus causing some of the
    in process cleanup in teardown_package to never be executed.
    
    Change-Id: I074dcd3d39aa46b262632024b047556ca471e8b8

commit 076134784b4eb1c2e55b28218dc0d64a57795d63
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Tue Feb 23 17:28:08 2016 -0800

Add warn method back to LogAdapter
    
    The change in 684c4c04 to improve py3 support breaks older middleware
    that consumes the SwiftLogger.
    
    Good for Swift code to not to use warn because it's deprecated.
    
    Bad for LogAdapter to not faithfully provide the current python logging
    module API (deprecated or otherwise).
    
    Change-Id: Ib0baf66b7f33f91562867076ff0526a287d8e5a6

commit eb7ca115e6501edd9225dce8784b8b3d3e094bb1
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Tue Feb 23 14:00:34 2016 -0800

Fix StatsD tests to not use real DNS
    
    In common/test_utils.py, TestStatsdLogging had the majority of its
    test cases calling the real socket.getaddrinfo(), which uses real
    DNS. This is very slightly slower than using a mock getaddrinfo() when
    the machine running the tests has functioning DNS, but on a machine
    with no network connection at all, the tests are excruciatingly slow
    due to timeouts.
    
    This commit mocks things out as appropriate. There's still one user of
    the real getaddrinfo(), but it's for ::1, so that's just local
    resolution based on /etc/hosts.
    
    Timing numbers for "./.unittests test.unit.common.test_utils:TestStatsdLogging":
    
     * network, without this patch: 1.8s
     * no network, without this patch: 221.2s (ouch)
     * network, with this patch: 1.1s
     * no network, with this patch: 1.1s
    
    Change-Id: I1a2d6f24fc9bb928894fb1fd8383516250e29e0c

commit b97adf70ade2982fea5354c55f1eea9f98442fa7
Author: Thiago da Silva <thiago@redhat.com>
Date:   Tue Feb 23 16:23:51 2016 -0500

fix formatting of bulk operations doc
    
    Change-Id: I247d2867c55d6bbf2288dfb74271f57c6f88b243

commit cfd50212358372694ba0713ff8c3bc313c6f1a92
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Thu Feb 18 21:06:40 2016 -0800

dry out container unittests
    
    Change Ifcc4392ffa4df485ce3c62a35d0fcc8c93de74aa introduced
    TestContainerController4Replicas as a subclass of
    TestContainerController. And there is duplicated code
    block to override setUp method.
    
    This patch removed the duplicated code block.
    
    Change-Id: Iad0b64cba52554b89dbd959475a2ce7c0d9a20eb

commit 406e8996fb288e9851e02c3f3faa536a9919b28f
Author: Takashi Kajinami <kajinamit@nttdata.co.jp>
Date:   Mon Feb 22 16:10:09 2016 +0900

Use 201 insted of 200 in PUT test case
    
    There are some unit test cases which assumes 200 as the response
    of PUT request, but 200 is never returned in PUT.
    This patch replaces 200 in PUT by 201, and adapt unit test cases
    to real case.
    
    Change-Id: I35fc7a4fe7fcc5558a9e6c6371105793c2710f3e

commit 5b18455cef57d052df3cda3e9179c923d3aadcad
Author: Hisashi Osanai <osanai.hisashi@jp.fujitsu.com>
Date:   Fri Jan 22 07:06:45 2016 +0000

Add unit tests for checking responses in ContainerController
    
    This patch provides unit tests to check responses for various situation
    in three/four replicas env.
    
    Change-Id: Ifcc4392ffa4df485ce3c62a35d0fcc8c93de74aa

commit 002513d21786df0d6c71e9772ec4688e2c14d8de
Author: Tim Burke <tim.burke@gmail.com>
Date:   Thu Jan 7 11:56:33 2016 -0800

Make write-only container access consistent
    
    Previously, if a user could write to (but not read from) a container,
    the behavior for object POST would vary depending on whether
    object_post_as_copy was enabled (403 response) or disabled (202
    response).
    
    Now, POSTs will consistently be allowed, regardless of whether fast-POST
    is enabled.
    
    Change-Id: I1d6dcbc4f5034a322a1073850fc3b059ebb1c0fa

commit 9a6c6b55713f896609b7127560ae99fabbf759e7
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Thu Feb 18 17:38:42 2016 +0100

Fixed log format in object auditor
    
    Change-Id: I218e08446c5ffda2db7c774ec8bb9d28ce9f63e7

commit 2d55960a221c9934680053873bf1355c4690bb19
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Thu Jan 14 18:31:21 2016 +0000

Fix inconsistent suffix hashes after ssync of tombstone
    
    Consider two replicas of the same object whose ondisk files
    have diverged due to failures:
    
      A has t2.ts
      B has t1.data, t4.meta
    
    (The DELETE at t2 did not make it to B. The POST at t4 was
    rejected by A.)
    
    After ssync replication the two ondisk file sets will not be
    consistent:
    
      A has t2.ts  (ssync cannot POST t4.meta to this node)
      B has t2.ts, t4.meta (ssync should not delete t4.meta,
                            there may be a t3.data somewhere)
    
    Consequenty the two nodes will report different hashes for the
    object's suffix, and replication will repeat, always with the
    inconsistent outcome. This scenario is reproduced by the probe
    test added in this patch.
    
    (Note that rsync replication does result in (t2.ts, t4.meta)
    on both nodes.)
    
    The solution is to change the way that suffix hashes are
    calculated. Currently the names of *all* files found in each
    object dir are added to the hash.  With this patch the
    timestamps of only those files that could be used to
    construct a valid diskfile are added to the hash. File
    extensions are appended to the timestamp so that in most
    'normal' situations the result of the hashing is the same
    as before this patch. That avoids a storm of hash mismatches
    when this patch is deployed in an existing cluster.
    
    In the problem case described above, t4.meta is no longer
    added to the hash, since it is not useful for constructing
    a diskfile. (Note that t4.meta is not deleted because it
    may become useful should a t3.data be replicated in future).
    
    Closes-Bug: 1534276
    Change-Id: I99e88b8d5f5d9bc22b42112a99634ba942415e05

commit ddeb0cde9fc828fa79dce90df4975f66737ba054
Author: Kota Tsuyuzaki <tsuyuzaki.kota@lab.ntt.co.jp>
Date:   Wed Feb 17 21:17:34 2016 -0800

Fix object-server to handle newer ts file
    
    Swift object-server will return 409 Conflict when the incomming
    request X-Timestamp is older than the local disk file to prevent
    wasted data transfer and disk space consumption. However, IFF,
    the local disk file is a tombstone, current object-server will
    write the data in the device and cleanup it according to timestamp
    comparison result in hash_cleanup_list_dir, and then return 201
    Created. That's wasted and far from the semantics in the normal case.
    
    This patch fixes object-server to catch the DiskFileDeleted exception
    at PUT request, and then, set the timestamp from tombstone as
    original_timestamp to be able to compare with request timestamp.
    
    Co-Authored-By: Kazuhiro Miyahara <miyahara.kazuhiro@lab.ntt.co.jp>
    Change-Id: I078c9cb90707a3c320708e76ea42cbfa73e1ea4b
    Closes-Bug: #1546865

commit 878688cae00576ec0bdb0a66eca7ada95e661c0a
Author: Béla Vancsics <vancsics@inf.u-szeged.hu>
Date:   Mon Dec 7 17:02:40 2015 +0100

Reduce code duplication
    
    Reduced source code by extracting duplicated code
    (swift/common/direct_client.py)
    http://openqa.sed.hu/dashboard/index/5?did=1
    
    Change-Id: I47fa6ae12c646e8e42fb0a46a017971f05d883fc
    Closes-Bug: 1536067

commit c3201f256cf415988a3cb1c42e9c6b7835195fa3
Author: Kazuhiro MIYAHARA <miyahara.kazuhiro@lab.ntt.co.jp>
Date:   Fri Jan 8 18:03:33 2016 +0900

Remove execute permissions from doc files and swift.conf-sample
    
    Some doc files and swift.conf-sample were added execute permissions in past changes.
    This patch remove execute permissions from them.
    
    Change-Id: Id8844989a8321578e9207566ebd6660f5b9523f0

commit e46d6b17251ab538595b1deb5581360a286b8fd1
Author: Christian Schwede <cschwede@redhat.com>
Date:   Wed Feb 17 20:47:26 2016 +0100

Add note on using printable chars for swift_hash_path_suffix/prefix
    
    Using unprintable characters for swift_hash_path_prefix/suffix might lead to
    hard problems when parsing of these values changes, for example due to newer
    Python versions or changes in the parsers itself. Let's avoid this and add a
    note that deployers should use printable strings for these values.
    
    Change-Id: I976982b753b6af831ab91d7190f50f8f15bf73bf

commit b8fa48080812ab0f6bac7ae19c0290da08f6a6e4
Author: Donagh McCabe <donagh.mccabe@hp.com>
Date:   Thu Feb 4 16:19:13 2016 +0000

Added links to API reference and usage guides
    
    Added a link to the API reference (where headers and query
    strings are documented; click the "detail" button to see).
    Also added a reference to Swift section of the OpenStack end
    user guide. This contains some additional details about the API.
    
    No attempt was made to reconcile duplicate information. Instead
    this patch links documents that might otherwise be overlooked.
    However, I fixed text, originally in a table, that had become
    garbled in a prior patch.
    
    Change-Id: I0910cbeb0c8bffc00e510f35585603e7b7a67790

commit aa7204d106ae33eba9219514f31f008510c9db53
Author: Kazuhiro MIYAHARA <miyahara.kazuhiro@lab.ntt.co.jp>
Date:   Fri Feb 12 20:48:34 2016 +0900

Remove '#! /usr/bin/env python' from unexecutable files
    
    'cli/recon.py' and 'cli/ring_builder_analyzer.py' have '#! /usr/bin/env
    python' in spite of they don't have execute permissions.  This patch
    removes '#! /usr/bin/env python' from them.
    
    Change-Id: I1917ccc84b1673af3d862be1796f54595f94c5ca

commit db87a5a9889d436f2e2aacdc8923d55ca0d3d1ca
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Tue Feb 16 10:52:01 2016 -0800

Mock sleep in direct_client retry tests
    
    ... and fixup some assertions, and other mocks
    
    Change-Id: Iadf839d1d3d735109fed24049fc3d829776b7b65

commit 8eb30afd7c39f6e6ba031f72c6a61f45b28d37f6
Author: Tim Burke <tim.burke@gmail.com>
Date:   Tue Feb 16 10:25:39 2016 -0800

Static methods should be @staticmethods
    
    Change-Id: Ifee5d68e00bbb3571aaac885cdd7490c79732985

commit 4a44e27e00795c5f12176ef346cd2249fca54fd8
Author: Chaozhe.Chen <chaozhe.chen@easystack.cn>
Date:   Tue Feb 16 23:44:27 2016 +0800

Replace assertEqual(None, *) with assertIsNone in tests
    
    As swift no longer supports Python 2.6, replace assertEqual(None, *)
    with assertIsNone in tests to have more clear messages in case of
    failure.
    
    Change-Id: I94af3e8156ef40465d4f7a2cb79fb99fc7bbda56
    Closes-Bug: #1280522

commit 42f4b3fc1e69a046c1e70e73afe847e4423fa2da
Author: Christian Schwede <cschwede@redhat.com>
Date:   Tue Feb 16 10:08:55 2016 +0000

Add SwiftHLM to associated projects
    
    Change-Id: I5ea3de34e61f22abec803b35fec8adde18a793e9

commit 025ebf2901d5a972796b73538bb306847d7d06b6
Author: Ankur Gupta <ankur.gupta@intel.com>
Date:   Wed Feb 10 14:36:16 2016 -0600

Docstring omission in class BaseDiskFileManager.
    
    Added documentation for missing Docstring variables.
    
    Change-Id: I29a53b8141c5607815f234a6123e2289200bca34

commit 4906b4c431edc436f165b163a228b0a221950c79
Author: Janonymous <janonymous.codevulture@gmail.com>
Date:   Tue Jan 12 12:50:43 2016 +0530

Fix missing txn_id logs in GreenAsyncPile's spawned functions
    
    This commit ensures that the logger thread_locals
    value is passed to and set in _get_conn_response methods
    executed in a green thread.
    
    Added partial bug tag because in bug description a more
    relevant fix is suggested which would fix the bug completely
    but for now this makes sense to add this commit for logging.
    
    Co-Authored-By: Clay Gerrard <clay.gerrard@gmail.com>
    Change-Id: I13bbf174fdca89318d69bb0674ed23dc9ec25b9a
    Partial-Bug: #1409302

commit 511a1b8698889c430bf673ed916ae41a96b99c5c
Author: Kazuhiro MIYAHARA <miyahara.kazuhiro@lab.ntt.co.jp>
Date:   Fri Feb 12 19:01:59 2016 +0900

Remove execute permission from swift/cli/ringbuilder.py
    
    In Change-Id: Ia63f59a8faca1fad990784f27532ca07a2125454,
    'bin/swift-ring-builder' is renamed to 'swift/cli/ringbuilder.py'.
    However, execute permission and '#! /usr/bin/env python' are left on
    the file.
    
    This patch removes execute permission and '#! /usr/bin/env python'
    from the file.
    
    Change-Id: I66370f825ce9ed9a40ec21f2745aa6aff22c8709

commit 973d9358d2e6e029409d1902831b4680e02952c9
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Thu Feb 11 10:20:10 2016 +0100

Bump PyECLib requirement to >= 1.2.0
    
    This was discussed at last Swift meeting [1] and conclusion was
    to require newest PyECLib version.
    
    [1] http://eavesdrop.openstack.org/meetings/swift/2016/swift.2016-02-10-21.00.log.html#l-268
    
    Depends-On: I0fd58f26bc0a3e7715356ff74344981f22786152
    Change-Id: I54e89d99b48e00e614d2d457814c00354763c83b

commit d3dd137cad896c7f741592e88f076acab2636a3c
Author: Tomas Matlocha <tomas.matlocha@firma.seznam.cz>
Date:   Tue Jan 5 12:32:50 2016 +0100

Additional info log message for drive-audit
    
    An additional info log message was added for case of
    running drive-audit without failed device unmounting.
    
    Change-Id: I11abee40a712b6c6de65e63626b6f7f0a9c9f4c7

commit c4158bd674f34c8a0648c98974ba474fcd1c2b7a
Author: Sarafraj Singh <Sarafraj.Singh@intel.com>
Date:   Thu Feb 11 20:52:47 2016 +0000

Fixed params on autocreate_account docstring
    
    ... in proxy.controllers.base
    
    Change-Id: Ic5c27cbf0f19b65fcaa3e6587583f03e56d646a0

commit f56d18e143d0a38584e4c383ea9afe46e67a0af6
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Thu Feb 11 10:59:43 2016 -0800

Fix typos in comments
    
    Change-Id: I4f98d447bd2ddd8eeb2f4da66d069bd7d6f00dc6

commit cc8dc88b55b44ca2a1fd65c049f3cfec22eb5525
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Thu Feb 4 16:02:42 2016 +0100

Script for checking sanity of manpages
    
    Change-Id: Ia27b0ecf6a3267eb51f8a6562721dca72f128915

commit bcada66b9065c94cd39498a5330c727cc575a6fe
Author: root <lr699s@att.com>
Date:   Wed Feb 10 09:21:53 2016 -0600

Removed unused parameter in server.py
    
    The variable max_large_object_get_time is no longer used and was
    removed to reflect the change.
    
    Change-Id: I43051181dcb38245de6d13fab63876e83f46fc39
    Closes-Bug: #1538834

commit b5311f63dbf5787bda3495a58c8ef7a6a557daa4
Author: gh159m <gh159m@att.com>
Date:   Wed Feb 10 10:36:59 2016 -0600

Removed default value for log_statsd_host
    
    Multiple files and documents showed that log_statsd_host had
    a default value, usually localhost.  This was incorrect, instead
    setting a value for log_statsd_host enables statsd logging.
    
    Removed any reference of log_statsd_host having a default value.
    
    Also changed descriptions to show setting a value enables logging.
    
    Change-Id: I3ca5c0e8b8e4981de3aa6db0c476072b5a59723d
    Closes-Bug: #1542227

commit b98c1c60f9c581c76677c19baae60ae720e3c7f3
Author: Christian Berendt <berendt@b1-systems.de>
Date:   Tue Feb 9 07:32:48 2016 +0100

Remove executable flag from sample config file
    
    swift.conf-sample should not be executbale
    
    Change-Id: I960058e2af10b43efc6324a7534d9a23c810e39a

commit 81a4355c2d0907035805c300a9fa5aaa6d0e70c4
Author: Christopher Bartz <bartz@dkrz.de>
Date:   Fri Feb 5 09:51:11 2016 +0100

Adds storage policy option to recon
    
    With this patch, recon gets policy-aware in regard to the hosts to retrieve.
    If no policy is passed and the server_type is object,
    all hosts of all policies are retrieved.
    Previously, recon did only retrieve the hosts of the storage-policy 0.
    
    Change-Id: If5735cd6721eac504aed8aaf3884cb91b6a0fcac
    Closes-Bug: 1541491

commit 87fc21c7cfcce2a3e23a84fddcfd1309cd884716
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Mon Feb 8 15:31:42 2016 +0000

Speed up functional testing
    
    test/functional/tests.py:TestObjectVersioningUTF8 does not clean
    up the versions files it creates because the class's multiple
    inheritance is such that it does not call the tearDown method in
    TestObjectVersioning.
    
    As a result, any attempt to clean up account containers wastes
    time retrying container delete requests. This occurs either in
    the setUp for TestSloEnv, if the TestSlo class is included in a
    test run, or in the tests.py package tearDown method.
    
    On the author's dev machine this patch reduces the execution
    time of functional tests in tests.py by approx 30% or 1 minute.
    
    Change-Id: I8194672bf2ca82435df5868720b6a55a79b94413

commit 3b94bd45406f9d457d37caa10c856a19c4cec22a
Author: Mingyu Li <li.mingyu@99cloud.net>
Date:   Sun Feb 7 21:41:08 2016 +0800

Keep the Usage of exit()/sys.exit() Consistent
    
    It's better to keep the usage of exit()/sys.exit() consistent
    in one file. Furthermore, sys.exit() is considered good to be
    used in production code, while exit is for interactive shell.
    
    Change-Id: Ia3092853a648922588e2bc11db37d6decdec1b48

commit f05da4c8fa37a285bb2ca9b45a355dbe57f7add4
Author: janonymous <janonymous.codevulture@gmail.com>
Date:   Sat Feb 6 16:59:57 2016 +0530

Changing the base version to python2.7 for pep8 checks
    
    tox was taking python3 as it's base version which lead to
    tox pep8 check failing on local env:
    http://paste.openstack.org/show/486183/
    
    Change-Id: Ia822ec5b5e12a1fc30b676f1e6d95e287b9b3368

commit 13edc9a865cd203ddf28a3f782426a1b3b94f7a1
Author: Sarafraj Singh <Sarafraj.Singh@intel.com>
Date:   Fri Feb 5 17:28:26 2016 +0000

Improved test coverage of form_signature
    
    Added four new test cases to execute path of invalid arguments. This
    improved test coverage of file specified from 80% to 91%.
    
    Change-Id: I63c2e7bab3f01121301d78b687687208a58401c0

commit b173995666b026def3f6558d9a8c972b75449323
Author: Kota Tsuyuzaki <tsuyuzaki.kota@lab.ntt.co.jp>
Date:   Thu Feb 4 05:18:42 2016 -0800

Fix missing Accept-Ranges
    
    Since commit 4f2ed8bcd0468f3b69d5fded274d8d6b02ac3d10, the response
    header for GET EC object doesn't include the Accept-Ranges header.
    
    This patch fixes it and also adds a few unittests to prevent regression.
    
    Closes-Bug: #1542168
    
    Change-Id: Ibafe56ac87b14bc0028953e620a653cd68dd3f84

commit d78e861d850db96989f4fb77ad31e3170034f15d
Author: Richard Hawkins <richard.hawkins@rackspace.com>
Date:   Wed Jan 13 13:15:07 2016 -0600

Add option for label to static web listings
    
    By default listings are rendered with a label of:
    "Listing of /v1/account/container/path"
    
    This change adds a new custom metadata key/value pair
    "X-Container-Meta-Web-Listings-Label: <label>" that
    when set, will cause the following:
    "Listing of <label>/path" to be rendered instead.
    
    Change-Id: I47c5d21834f47e566b380920196926882ff83abf

commit ae632abbd802228ef98a23c98ee49988e7e5b942
Author: Ondřej Nový <ondrej.novy@firma.seznam.cz>
Date:   Tue Feb 2 21:58:52 2016 +0100

Fixed manpages errors.
    
    account-server.conf.5
    105: warning: numeric expression expected (got `)')
    
    container-server.conf.5
    111: warning: numeric expression expected (got `)')
    
    object-expirer.conf.5
    79: warning: numeric expression expected (got `)')
    
    object-server.conf.5
    114: warning: numeric expression expected (got `)')
    
    proxy-server.conf.5
    121: warning: numeric expression expected (got `)')
    331: warning: numeric expression expected (got `[')
    1005: warning: macro `*' not defined
    
    Change-Id: I203dcfde83035e3b1dcb91109b72b5d08bb7840e

commit f2fca9aafa17e45b668c6f76b256927681cc463a
Author: Alistair Coles <alistair.coles@hpe.com>
Date:   Tue Feb 2 11:43:44 2016 +0000

Fix time-shifting of objects PUT with container-sync
    
    When container-sync PUTs an object to a destination container
    it uses the timestamp from the container row rather than the
    actual timestamp of the object being copied. The actual timestamp
    of the object can be newer, so the sync'd object may end up with
    the right content but at the wrong, older, timestamp.
    
    This patch changes the timestamp sent with the sync'd object
    to be that of the actual source object being sent.
    
    Drive-by fix to make code more readable by removing a variable
    rename mid-function, fix a typo and remove a redundant function
    call.
    
    Change-Id: I800e6de4cdeea289864414980a96f5929281da04
    Closes-Bug: #1540884

commit 050e3b35dd88f7b31b6810d9ac88548ef371e3ce
Author: venkatamahesh <venkatamaheshkotha@gmail.com>
Date:   Thu Feb 4 16:47:15 2016 +0530

Use uppercase 'S' in word "OpenStack"
    
    Change-Id: I4692aa58b5173b1b662d5eee19fac770ae5aaed0