Bug #1229324 “extraneous vim editor configuration comments” : Bugs : Ceilometer

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-09-23: Fix proposed to ceilometer (master)

#1

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

Changed in ceilometer:
assignee:	nobody → Terri Yu (terriyu)
status:	New → In Progress

Terri Yu (terriyu) on 2013-09-23

summary:	- extraneous text editor configuration in audit.py + extraneous vim editor configuration comments
description:	updated

Julien Danjou (jdanjou) on 2013-09-24

Changed in ceilometer:
importance:	Undecided → Low

Revision history for this message

Mark McLoughlin (markmc) wrote on 2013-09-25:

#2

I think it would be good to send a brief RFC email to <email address hidden> - someone might come up with a plausible reason to keep these.

Also, I think it's best to defer this churn until Icehouse.

Changed in oslo:
status:	New → Confirmed
importance:	Undecided → Low

Mike Perez (thingee) on 2013-09-30

Changed in cinder:
importance:	Undecided → Low
status:	New → Confirmed
assignee:	nobody → Mike Perez (thingee)

Revision history for this message

John Griffith (john-griffith) wrote on 2013-10-02:

#3

Personally I'm not sure why this is of concern, and definitely not something worth dealing with until the next release IMO.

Changed in cinder:
importance:	Low → Undecided
importance:	Undecided → Low

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-10-07: Fix merged to ceilometer (master)

#5

Reviewed: https://review.openstack.org/47894
Committed: http://github.com/openstack/ceilometer/commit/3142a317532beb28534ea28731f6d0c2999b1b07
Submitter: Jenkins
Branch: master

commit 3142a317532beb28534ea28731f6d0c2999b1b07
Author: Terri Yu <email address hidden>
Date: Fri Sep 27 01:17:55 2013 +0000

Remove extraneous vim configuration comments

Fixes bug # 1229324

Remove line containing

comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

at the top of source code files, except for files in
ceilometer/openstack/common and bin/ceilometer-rpc-zmq-receiver

Change-Id: Ic8643afbc3510c36c41871b8a840a9b9e21eaba7

Changed in ceilometer:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-10-15: Fix merged to swift (master)

#6

Reviewed: https://review.openstack.org/51875
Committed: http://github.com/openstack/swift/commit/a0017e053734ea5e72bfcc3fdc630c8a7a25431a
Submitter: Jenkins
Branch: master

commit a0017e053734ea5e72bfcc3fdc630c8a7a25431a
Author: Greg Lange <email address hidden>
Date: Tue Oct 15 16:23:00 2013 +0000

Removes vim comment

Change-Id: Ibc930638ee2d839e8499a13018c205b615f8b854
Partial-Bug: 1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-10-27: Fix proposed to swift (feature/ec)

#7

Fix proposed to branch: feature/ec
Review: https://review.openstack.org/54029

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-10-29: Fix merged to swift (feature/ec)

#8

Download full text (24.9 KiB)

Reviewed: https://review.openstack.org/54029
Committed: http://github.com/openstack/swift/commit/94d3671b0bbf87fdbff845643963f3f9a97c58b5
Submitter: Jenkins
Branch: feature/ec

commit abcecd26a7b5871f75f0fbddf0d08bbac95bb089
Author: Kun Huang <email address hidden>
Date: Wed Oct 23 21:19:01 2013 +0800

utf8 encode tempurl key

    In tempurl middleware, hmac uses the value of account metadata to
    generate HMAC-SHA1 signature and hmac must accept a str-type string, not
    a unicode string. The meta dict returned from get_info stroges special
    chars as unicode however. So just encode it for tempurl using.

Closes-Bug: #1242644
Change-Id: I4be62eea014a573efc4748470de57dccf00e431d

commit cd2e7df0b69bbd269cd3c4170e0fee8186a07c95
Author: Pete Zaitcev <email address hidden>
Date: Tue Oct 22 17:18:04 2013 -0600

Add an __str__ method to brokers

    A few uses of broker.db_file are in printouts where we do need
    them, so the administrator may know what's up. Seems like an easy
    way to get rid of those is to make brokers identify themselves
    with common __str__. Alternative back-end implementations may
    supply something other than a filename here, for example a cluster
    name and a volume name.

Note that I'm not sure if correct coercion would occur when
brokers are bounced through dictionaries, hence explicit str().

Change-Id: I329788ebd1fbe39ffadcf9f9d5194a74a88dde58

commit 9807a358c6d1314d25e3a41da75be5851fa0ac27
Author: Donagh McCabe <email address hidden>
Date: Fri Aug 23 15:03:08 2013 +0100

Add WWW-Authenticate to 401 responses

    Per http://www.ietf.org/rfc/rfc2616.txt, when a 401 error is returned, the
    Www-Authenticate response header MUST also be returned. The format is
    described in http://www.ietf.org/rfc/rfc2617.txt.

    Swift supports and/or implements a number of authentication schemes
    including tempauth, Keystone, tempurl, formpost and container sync. In
    this fix, we use a catch-all, "Swift". The realm is the account (where
    known) or "unknown" (bad path or where the 401 is returned from code
    that does not have the request). Examples:

Www-Authenticate: Swift realm="AUTH_1234567889"
Www-Authenticate: Swift realm="unknown"

Fixes bug #1215491

Change-Id: I03362789318dfa156d3733ef9348795062a9cfc4

commit ed5101b2002b877518466ae5f9a6d652581238f2
Author: Yuan Zhou <email address hidden>
Date: Sat Oct 19 11:40:35 2013 +0800

Adding more unit tests for audit_location_generator

Change-Id: I40410fbbb79cea8647074f703e4675364c69d930

commit 5202b0e58613738cc81ec63e7c6da14ce5429526
Author: Peter Portante <email address hidden>
Date: Thu Sep 12 19:51:18 2013 -0400

DiskFile API, with reference implementation

    Refactor on-disk knowledge out of the object server by pushing the
    async update pickle creation to the new DiskFileManager class (name is
    not the best, so suggestions welcome), along with the REPLICATOR
    method logic. We also move the mount checking and thread pool storage
    to the new ondisk.Devices object, which then also becomes th...

Reviewed:  https://review.openstack.org/54029
Committed: http://github.com/openstack/swift/commit/94d3671b0bbf87fdbff845643963f3f9a97c58b5
Submitter: Jenkins
Branch:    feature/ec

commit abcecd26a7b5871f75f0fbddf0d08bbac95bb089
Author: Kun Huang <academicgareth@gmail.com>
Date:   Wed Oct 23 21:19:01 2013 +0800

utf8 encode tempurl key
    
    In tempurl middleware, hmac uses the value of account metadata to
    generate HMAC-SHA1 signature and hmac must accept a str-type string, not
    a unicode string. The meta dict returned from get_info stroges special
    chars as unicode however. So just encode it for tempurl using.
    
    Closes-Bug: #1242644
    Change-Id: I4be62eea014a573efc4748470de57dccf00e431d

commit cd2e7df0b69bbd269cd3c4170e0fee8186a07c95
Author: Pete Zaitcev <zaitcev@kotori.zaitcev.us>
Date:   Tue Oct 22 17:18:04 2013 -0600

Add an __str__ method to brokers
    
    A few uses of broker.db_file are in printouts where we do need
    them, so the administrator may know what's up. Seems like an easy
    way to get rid of those is to make brokers identify themselves
    with common __str__. Alternative back-end implementations may
    supply something other than a filename here, for example a cluster
    name and a volume name.
    
    Note that I'm not sure if correct coercion would occur when
    brokers are bounced through dictionaries, hence explicit str().
    
    Change-Id: I329788ebd1fbe39ffadcf9f9d5194a74a88dde58

commit 9807a358c6d1314d25e3a41da75be5851fa0ac27
Author: Donagh McCabe <donagh.mccabe@hp.com>
Date:   Fri Aug 23 15:03:08 2013 +0100

Add WWW-Authenticate to 401 responses
    
    Per http://www.ietf.org/rfc/rfc2616.txt, when a 401 error is returned, the
    Www-Authenticate response header MUST also be returned. The format is
    described in http://www.ietf.org/rfc/rfc2617.txt.
    
    Swift supports and/or implements a number of authentication schemes
    including tempauth, Keystone, tempurl, formpost and container sync. In
    this fix, we use a catch-all, "Swift". The realm is the account (where
    known) or "unknown" (bad path or where the 401 is returned from code
    that does not have the request). Examples:
    
         Www-Authenticate: Swift realm="AUTH_1234567889"
         Www-Authenticate: Swift realm="unknown"
    
    Fixes bug #1215491
    
    Change-Id: I03362789318dfa156d3733ef9348795062a9cfc4

commit ed5101b2002b877518466ae5f9a6d652581238f2
Author: Yuan Zhou <yuan.zhou@intel.com>
Date:   Sat Oct 19 11:40:35 2013 +0800

Adding more unit tests for audit_location_generator
    
    Change-Id: I40410fbbb79cea8647074f703e4675364c69d930

commit 5202b0e58613738cc81ec63e7c6da14ce5429526
Author: Peter Portante <peter.portante@redhat.com>
Date:   Thu Sep 12 19:51:18 2013 -0400

DiskFile API, with reference implementation
    
    Refactor on-disk knowledge out of the object server by pushing the
    async update pickle creation to the new DiskFileManager class (name is
    not the best, so suggestions welcome), along with the REPLICATOR
    method logic. We also move the mount checking and thread pool storage
    to the new ondisk.Devices object, which then also becomes the new home
    of the audit_location_generator method.
    
    For the object server, a new setup() method is now called at the end
    of the controller's construction, and the _diskfile() method has been
    renamed to get_diskfile(), to allow implementation specific behavior.
    
    We then hide the need for the REST API layer to know how and where
    quarantining needs to be performed. There are now two places it is
    checked internally, on open() where we verify the content-length,
    name, and x-timestamp metadata, and in the reader on close where the
    etag metadata is checked if the entire file was read.
    
    We add a reader class to allow implementations to isolate the WSGI
    handling code for that specific environment (it is used no-where else
    in the REST APIs). This simplifies the caller's code to just use a
    "with" statement once open to avoid multiple points where close needs
    to be called.
    
    For a full historical comparison, including the usage patterns see:
    https://gist.github.com/portante/5488238
    
    (as of master, 2b639f5, Merge
     "Fix 500 from account-quota     This Commit
     middleware")
    --------------------------------+------------------------------------
                                     DiskFileManager(conf)
    
                                       Methods:
                                         .pickle_async_update()
                                         .get_diskfile()
                                         .get_hashes()
    
                                       Attributes:
                                         .devices
                                         .logger
                                         .disk_chunk_size
                                         .keep_cache_size
                                         .bytes_per_sync
    
    DiskFile(a,c,o,keep_data_fp=)    DiskFile(a,c,o)
    
      Methods:                         Methods:
       *.__iter__()
        .close(verify_file=)
        .is_deleted()
        .is_expired()
        .quarantine()
        .get_data_file_size()
                                         .open()
                                         .read_metadata()
        .create()                        .create()
                                         .write_metadata()
        .delete()                        .delete()
    
      Attributes:                      Attributes:
        .quarantined_dir
        .keep_cache
        .metadata
                                    *DiskFileReader()
    
                                       Methods:
                                         .__iter__()
                                         .close()
    
                                       Attributes:
                                        +.was_quarantined
    
    DiskWriter()                     DiskFileWriter()
    
      Methods:                         Methods:
        .write()                         .write()
        .put()                           .put()
    
    * Note that the DiskFile class   * Note that the DiskReader() object
      implements all the methods       returned by the
      necessary for a WSGI app         DiskFileOpened.reader() method
      iterator                         implements all the methods
                                       necessary for a WSGI app iterator
    
                                     + Note that if the auditor is
                                       refactored to not use the DiskFile
                                       class, see
                                       https://review.openstack.org/44787
                                       then we don't need the
                                       was_quarantined attribute
    
    A reference "in-memory" object server implementation of a backend
    DiskFile class in swift/obj/mem_server.py and
    swift/obj/mem_diskfile.py.
    
    One can also reference
    https://github.com/portante/gluster-swift/commits/diskfile for the
    proposed integration with the gluster-swift code based on these
    changes.
    
    Change-Id: I44e153fdb405a5743e9c05349008f94136764916
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit a4e25d6da8d007051b3b0575de47338e210d4315
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Mon Oct 14 13:12:22 2013 -0700

Proxy: use just one greenthread per client
    
    Instead of using two greenthreads and an eventlet.queue.Queue of size
    1 to stream response bodies from backends to clients, just do it with
    a single greenthread that reads and writes in a loop. This should
    lower the amount of CPU used by the proxy in its response streaming.
    
    Client fairness used to be provided implicitly; since the queue only
    held 1 item, the read-from-backend greenthread would block after each
    chunk until the write-to-client greenthread got a chance to run. Now
    fairness is provided by an explicit eventlet.sleep() call, just as it
    is in the object server.
    
    Change-Id: Iae27109f5a3d109ad21ec9a972e39f22150f6dbb

commit a0017e053734ea5e72bfcc3fdc630c8a7a25431a
Author: Greg Lange <greglange@gmail.com>
Date:   Tue Oct 15 16:23:00 2013 +0000

Removes vim comment
    
    Change-Id: Ibc930638ee2d839e8499a13018c205b615f8b854
    Partial-Bug: 1229324

commit d4c658750562f2baa978cc28d341680c6b470070
Author: Greg Lange <greglange@gmail.com>
Date:   Tue Oct 15 15:38:02 2013 +0000

return value on memcache client decr call
    
    Closes-Bug: 1238204
    Change-Id: I7d2ece118159637e5c5bf4bf5dd4ce4d2f6e4523

commit 9c226e3a1530a21d754ac7f1702c35f4a8e5927c
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Thu Oct 10 13:33:05 2013 -0700

Small optimization to container_quotas
    
    When handling an object PUT with X-Copy-From, we only need to fetch
    the source object's size if the container has a byte quota, but we
    were always fetching it. Now we only fetch the size when we need it.
    
    Change-Id: I0c2c6820cdf0022ef57df5fe7dcb2dd466665a4f

commit b61add12e4910208ec115966c259553c94671202
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Wed Oct 9 12:31:26 2013 -0700

Small efficiency improvement in account quotas
    
    If you were performing an object PUT with X-Copy-From and you had
    account_quotas in your pipeline but your account did not have a quota
    set, then we will now avoid an unnecessary call to get_object_info().
    
    Change-Id: I8592d4ad65e45ecf9e0557bc10ce2c2db19732e8

commit 2419df2730c5879092f0058e63a7dfbae6519fa0
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Wed Oct 9 12:25:42 2013 -0700

Fix 500 from account-quota middleware
    
    If a user had set X-Account-Meta-Quota-Bytes to something non-integer
    prior to the installation of the account-quota middleware, then the
    quota check would choke on it. Now a non-integer value is treated as
    "no quota".
    
    Change-Id: I5c38911be1f66fa293aea9c78590d4ce7d184113

commit 9b521fe800ed3984169d37e9772674867745fede
Author: John Dickinson <me@not.mn>
Date:   Tue Oct 8 07:35:43 2013 -0700

CHANGELOG and AUTHORS updates for 1.10.0 release
    
    Change-Id: Iaf8d289c5ddb6ac778243a5e1fc27600bade6feb

commit 221aa9edd33fab41c696431261cb3414f3c02355
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Tue Oct 8 13:34:50 2013 -0700

allow container create even if over account quota
    
    Change-Id: I07278bc3314c26426ff0557fe2f5c69c02d6550c

commit 841b8082cee923eb34f2b1bc89f0c5f4e91ce9b6
Author: Aaron Rosen <arosen@nicira.com>
Date:   Mon Oct 7 22:24:10 2013 -0700

Correct URL in readme
    
    doc.openstack.org -> docs.openstack.org
    
    Change-Id: Iacadb8f70cb94adc002c562a1068ff38be19a9fc

commit 0b370269111957fec7521d284fcbd742ff8b8c13
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Mon Oct 7 21:28:48 2013 -0700

Add a Timeout when getting Memcache connections
    
    The old Timeout behavior when pulling connections of the MemcacheConnPool left
    ambiguity around what timed out and could put more placeholders in the queue
    than the configured max_connections.
    
    To avoid waiting indefinitely on slow severs we raise a custom Timeout when we
    fail get a connection from the pool.  We still error limit the slow server,
    and move onto the next, but we still don't allow more than max_connections.
    
    Change-Id: I9e2409896423d52da69e35c038e5f457c71f705d

commit 6607beab0dc8043251b490471761fa2dd85f2816
Author: Peter Portante <peter.portante@redhat.com>
Date:   Fri Oct 4 08:04:42 2013 -0400

Don't apply timeout to Pool.get operation (leaks)
    
    The connection timeout to a memcache server is performed by using the
    "with Timeout()" construct over the sock.connect() call in the
    .create() method. In addition, the same construct was being applied to
    the Pool.get() call in ._get_conns().
    
    If the maximum number of connections was already created, and the
    Pool.get() called took longer than the connect timeout, then the error
    handling path would add a place holder to the connection
    pool. Eventlet's Pool class allows for additional items to be added to
    the pool, over and above the max_size setting. This additional place
    holder will eventually be pulled and a new connection created to take
    its place.
    
    The fix is to remove the timeout construct in the _get_conns() method.
    
    In addition, we also apply the unit test patch mentioned in the review
    comments for Patch Set 6 of https://review.openstack.org/45134,
    located at http://paste.openstack.org/show/47288/.
    
    Fixes bug 1235027
    
    Change-Id: I786cabefe3e8ddf7d92feb7ebc7cfb613d60a1da
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit 9411a24ba7b5a1380ec8a2aa13d4aa92988d2ff9
Author: Peter Portante <peter.a.portante@gmail.com>
Date:   Mon Oct 7 12:10:31 2013 +0000

Revert "Refactor common/utils methods to common/ondisk"
    
    This reverts commit 7760f41c3ce436cb23b4b8425db3749a3da33d32
    
    Change-Id: I95e57a2563784a8cd5e995cc826afeac0eadbe62
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit 61e135e211822840731d3153e9b249b6b81a1c21
Author: Peter Portante <peter.portante@redhat.com>
Date:   Sat Oct 5 11:27:35 2013 -0400

Fix bad hash_path reference
    
    Mea culpa: these two scripts were missed in commit:
    
      https://review.openstack.org/46956
    
    Fixes bug 1235441
    
    Change-Id: I4303bc808448a79bddbb991526b0cca26150b392
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit c7e65db27da59ddf221d1720362434581ef30311
Author: Pete Zaitcev <zaitcev@kotori.zaitcev.us>
Date:   Fri Oct 4 23:18:07 2013 -0600

Make test_translations test our tree
    
    In order to run the correct classes, Python test framework adjusts
    sys.path. However, these changes are not propagated to subprocesses.
    Therefore, the test actually tries to test installed Swift, not
    the one in which it is running.
    
    The usual suggestion is to run "python setup.py develop" before
    testing, but it's annoying and error-prone. If you forget it,
    you may test the code in /usr very easily, and never know.
    
    Let's just pass the correct path to subprocess. Much safer.
    
    Change-Id: Ic71314e8462cf6e0579d704ffe9fbbfac7e6ba24

commit 3d8f0f18059482412e9b60909de69df90416dfdd
Author: Peter Portante <peter.portante@redhat.com>
Date:   Wed Oct 2 23:58:13 2013 -0400

Fedora 19: need to use /etc/rc.d/rc.local
    
    Change-Id: I80e9a4c40ff99ec09a8eeef935447c6393ea78ec
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit db4547d01dab84142da40fdeb1ca5fbb34ef56ba
Author: Peter Portante <peter.portante@redhat.com>
Date:   Wed Oct 2 23:52:30 2013 -0400

Remove sphinx build warnings
    
    Change-Id: Ic34bbd9cc65d96ea9b8434be7b54e5bcfae28b63
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit def37fb56aea7b9fe4254621e10667712052d3ac
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Wed Sep 25 10:41:41 2013 -0700

Stop reading from object server when client disconnects.
    
    If a client were in the middle of an object GET request and then
    disconnected, the proxy would wait a while (default 60s) and then time
    out the connection. As part of the teardown for this, the proxy would
    attempt to close the connection to the object server, then drain any
    associated buffers. However, this didn't work particularly well,
    resulting in the proxy reading the entire remainder of the object for
    no gain.
    
    Now, the proxy closes the connection hard, by calling .close() on the
    underlying socket._socket object. This is different from calling
    .close() on a socket._socketobject object, which is what you get back
    from socket.socket() and similar methods. Calling .close() on a
    socket._socketobject simply decrements a reference counter on the
    socket._socket, which has been observed in the past to result in
    socket leaks when something holds onto a reference. However, calling
    .close() on a socket._socket actually closes the socket regardless of
    who else has a reference to it.
    
    I had to delete a test assertion that said the object server never got
    SIGPIPE after a GET w/X-Newest. Well, you get a SIGPIPE when you write
    to a closed socket, and now the proxy is actually closing the sockets
    early, so now you *do* get a SIGPIPE.
    
    closes-bug: 1174660
    
    Note that this will cause a regression on bug 1037337; unfortunately,
    the cure is worse than the disease, so out it goes.
    
    Change-Id: I9c7a2e7fdb8b4232e53ea96f86b50e8d34c27221

commit 6b67952442429cff306ceefc83dd206a4a5c16ed
Author: Dirk Mueller <dirk@dmllr.de>
Date:   Wed Oct 2 21:27:40 2013 +0200

assertEquals is deprecated, use assertEqual
    
    The very same functionality for fewer keystrokes.
    
    Change-Id: I8c9513412f398132db5636fdfb7bf8301e8443cf

commit 02e247c1b3d9637c6d6ecfd47ce82d250c576dbb
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Wed Oct 2 11:39:35 2013 -0700

Add "note" box callouts to SAIO for user changes.
    
    The SAIO is purpously cut into two parts, so that you don't have to switch
    back and forth between root and your unprivledged user.  Add some "note" box
    callouts to highlight this changeover.
    
    Change-Id: I8b1a8f0539eac60d4121bdd4dab01df75ecca207

commit 062a67b07d9a16fd4970bb90798ea67d1f99b5cd
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Wed Oct 2 09:57:34 2013 -0700

Allow slightly older dnspython (>= 1.9.4).
    
    This has a couple benefits.
    
    First, it means Ubuntu Precise users can just install python-dnspython
    from packages instead of having to pull one in from source. This
    should also fix an install error with new SAIO boxes where running
    "python setup.py develop" fails unless a satisfactory dnspython is
    already installed.
    
    Second, it matches the dnspython dependency in the global
    requirements. This means that the gates are already running with this
    dnspython dependency, and it means our requirements.txt is one step
    closer to being a subset of the global requirements.txt, which is
    important to some people.
    
    Change-Id: I5d58f488e1e4c4139c9fb20d89f386cab1537e98

commit ae8470131ead095e3bf1c290bac866a5e6e29e79
Author: Chuck Thier <cthier@gmail.com>
Date:   Wed Sep 4 22:20:44 2013 +0000

Pool memcache connections
    
    This creates a pool to each memcache server so that connections will not
    grow without bound.  This also adds a proxy config
    "max_memcache_connections" which can control how many connections are
    available in the pool.
    
    A side effect of the change is that we had to change the memcache calls
    that used noreply, and instead wait for the result of the request.
    Leaving with noreply could cause a race condition (specifically in
    account auto create), due to one request calling `memcache.del(key)` and
    then `memcache.get(key)` with a different pooled connection.  If the
    delete didn't complete fast enough, the get would return the old value
    before it was deleted, and thus believe that the account was not
    autocreated.
    
    ClaysMindExploded
    DocImpact
    Change-Id: I350720b7bba29e1453894d3d4105ac1ea232595b

commit e8a07c4ca784f958e33044c267212925d6452979
Author: Peter Portante <peter.portante@redhat.com>
Date:   Thu Sep 26 15:47:36 2013 -0400

Fedora 19 updates
    
    Change-Id: I95138852e45aa7632218a7107e0e7ba1f6ef373c
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit f9c4eb806f14ce095554d70261504781dbb7aa3e
Author: Brian D. Burns <iosctr@gmail.com>
Date:   Sun Sep 29 15:22:05 2013 -0400

Simplify object path when reporting SLO put errors
    
    When reporting errors for SLO PUT requests, use the object path from the
    manifest without the '/vrs/account' prefix. This is a continuation of
    the same changes made for SLO/bulk delete requests in 6768d5b.
    
    Change-Id: I40c90cccc1b7b5303d9f2b084dccb3be4f4448d8

commit 8ab06eb8cc08a6f5905efde5b07cee3e71805a77
Author: anc <alistair.coles@hp.com>
Date:   Fri Sep 27 10:10:55 2013 +0100

Set path_info in sub_slo manifest GET requests
    
    The new_req Request object created for fetching
    nested sub_slo manifests does not have its
    path_info set to the sub_slo path before being
    passed to GETorHEAD_base.
    
    The sub manifest GET works fine because
    GETorHEAD_base uses its path argument
    (not req.path) to set up its connection,
    BUT if an exception is thrown,
    GETorHEAD_base uses req.path to construct
    the log message - so the exception log would
    have the wrong path.
    
    Fixes bug 1231872
    
    Change-Id: I0f0f969958f814d8a7053440d9de4e50796169e4

commit 180cd18da64b59c39ec1696e4316b18711d1fb82
Author: Brian Curtin <brian.curtin@rackspace.com>
Date:   Tue Sep 24 15:07:59 2013 -0500

Move location of HTTPException in swift_testing
    
    While doing some work around swiftclient Python 3 support, httplib
    imports were changed to use six.moves in order to access httplib in Py2
    and http.client in Py3. Tests related to those changes are failing
    because these functional tests are depending on HTTPException being
    a name in swiftclient.client. These tests should acces that name from
    its canonical location in httplib (or, eventually, a Py3 friendly
    location like six.moves).
    
    Change-Id: I6335d465574045daedab47e5fe23c415d171a83a

commit fa308d60bded8b730dda451321bb7baf5ded2aa8
Author: anc <alistair.coles@hp.com>
Date:   Fri Sep 20 15:08:12 2013 +0100

Fix utf-8 handling in object versions.
    
    Fixes object versioning when object name and/or version
    container name contain multibyte utf-8 characters.
    
    When object names containing non-ASCII characters
    are PUT multiple times into a container with an
    x-versions-location set, subsequent DELETE of the
    object results in a 500 response status.
    
    When the versions container name contains
    non-ASCII characters the first delete of an object
    succeeds but fails to restore previous version of
    object, so second delete incorrectly returns 404.
    
    Fixes bug 1229142
    
    Change-Id: I425440f76b8328f8e119d390bfa4c7022181e89e

commit 4e8b2ffc2e2b97f8bd223c90848ecaaf777bebc9
Author: Peter Portante <peter.portante@redhat.com>
Date:   Mon Sep 23 22:16:59 2013 -0400

Use created container in unit test
    
    Change-Id: I2573be1ac14f65b8008611edf940363b31c8d86e
    Signed-off-by: Peter Portante <peter.portante@redhat.com>

commit d5fcc9aaf3b6707b1ddbc577e9f8c93fa90d6d89
Author: Samuel Merritt <sam@swiftstack.com>
Date:   Mon Sep 23 10:05:34 2013 -0700

Two small account-quota fixes
    
    First: even if a user has exceeded their account quota, they should be
    able to make object POST requests. Updating an object's metadata isn't
    going to make them any more over quota, so should be allowed.
    
    Second: don't bother with the reseller_admin check for container or
    object requests. If I send the header X-Account-Meta-Quota-Bytes: 100
    on e.g. an object PUT request, the proxy will (rightly) ignore it. Now
    account-quotas does too.
    
    Change-Id: I970a76349659acdd8229a324bd33bfe7fe7261a4

commit 56440eb95da79506cc27d92e07f0f5969cc683ce
Author: Christian Schwede <info@cschwede.de>
Date:   Wed Sep 18 19:49:32 2013 +0200

Handle X-Copy-From header in container_quota middleware
    
    Content length of the copied object is checked before
    allowing the copy request according to the container quota.
    
    Closes-Bug: #1201875
    Change-Id: If44b916791e94ac6c66eee04a5727186ce0e56ae

commit f72704fc82803254a6f335fa2a7d2459f86f877d
Author: ZhiQiang Fan <aji.zqfan@gmail.com>
Date:   Fri Sep 20 01:00:54 2013 +0800

Change OpenStack LLC to Foundation
    
    Change-Id: I7c3df47c31759dbeb3105f8883e2688ada848d58
    Closes-bug: #1214176

commit b0aeed1ec7fa2fd315f6293b0fd3d8118e20c7f9
Author: Clay Gerrard <clay.gerrard@gmail.com>
Date:   Mon Sep 16 17:06:09 2013 -0700

Fix default replication options for ring-builder add
    
    Change-Id: I957deeb0e711bfe7cd9d852726c77179a4613ee0

commit 42f4b150e3fcd8c49ecac2e42839449d00190053
Author: Florian Hines <syn@ronin.io>
Date:   Thu Sep 5 18:12:15 2013 -0500

Faster swift-dispersion-populate
    
    - Makes swift-dispersion-populate a bit faster when using a larger
      dispersion_coverage with a larger part_power.
    - Adds option to only run population for container OR objects
    - Adds option to let you resume population at given point (useful if you
      need to resume population after a previous run error'd out or the
      like) by specifying which suffix to start at.
    
    The original populate just randomly used uuid4().hex as a suffix on the
    container/object names until all the partition's required where covered.
    This isn't a big deal if you're only doing 1% coverage on a ring with a
    small part power but takes ages if you're doing 100% on a larger ring.
    
    Change-Id: I52f890a774412c1d6179f12db9081aedc58b6bc2

Victoria Martinez de la Cruz (vkmc) on 2013-11-08

Changed in python-ceilometerclient:
assignee:	nobody → Victoria Martínez de la Cruz (vkmc)

Victoria Martinez de la Cruz (vkmc) on 2013-11-08

Changed in python-ceilometerclient:
assignee:	Victoria Martínez de la Cruz (vkmc) → nobody

Sphoorti (sphoorti-joglekar) on 2013-11-08

Changed in python-ceilometerclient:
assignee:	nobody → Sphoorti (sphoorti-joglekar)

Victoria Martinez de la Cruz (vkmc) on 2013-11-08

Changed in python-ceilometerclient:
status:	New → Confirmed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-11-08: Fix proposed to python-ceilometerclient (master)

#9

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

Changed in python-ceilometerclient:
status:	Confirmed → In Progress

Sphoorti (sphoorti-joglekar) on 2013-11-09

Changed in oslo:
assignee:	nobody → Sphoorti (sphoorti-joglekar)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-11-11: Fix proposed to oslo-incubator (master)

#11

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

Changed in oslo:
status:	Confirmed → In Progress

Thierry Carrez (ttx) on 2013-12-04

Changed in ceilometer:
milestone:	none → icehouse-1
status:	Fix Committed → Fix Released

ZhiQiang Fan (aji-zqfan) on 2013-12-28

Changed in python-heatclient:
assignee:	nobody → ZhiQiang Fan (aji-zqfan)
Changed in heat:
assignee:	nobody → ZhiQiang Fan (aji-zqfan)

ZhiQiang Fan (aji-zqfan) on 2013-12-28

Changed in heat:
assignee:	ZhiQiang Fan (aji-zqfan) → nobody
Changed in python-neutronclient:
assignee:	nobody → ZhiQiang Fan (aji-zqfan)
Changed in heat:
assignee:	nobody → ZhiQiang Fan (aji-zqfan)
Changed in neutron:
assignee:	nobody → ZhiQiang Fan (aji-zqfan)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-12-28: Fix proposed to python-heatclient (master)

#12

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

Changed in python-heatclient:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-12-31: Fix merged to cinder (master)

#13

Reviewed: https://review.openstack.org/63990
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=ff3f2e5735f0d486b72b6881b05ace94d6d34d03
Submitter: Jenkins
Branch: master

commit ff3f2e5735f0d486b72b6881b05ace94d6d34d03
Author: DennyZhang <email address hidden>
Date: Tue Dec 24 22:37:04 2013 -0600

Remove vim header

No need to set tabstop tons of times, this can be set in your vimrc
file instead.

More disucssion:
http://openstack.10931.n7.nabble.com/Remove-vim-modelines-td21780.html

Partial-Bug: #1229324
Change-Id: Ib3a31c8819ee1950d72f7da5627593623d351560

Clint Byrum (clint-fewbar) on 2013-12-31

Changed in heat:
status:	New → Triaged
importance:	Undecided → Low

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-02: Fix proposed to heat (master)

#14

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

Changed in heat:
assignee:	ZhiQiang Fan (aji-zqfan) → DennyZhang (denny-6)
status:	Triaged → In Progress

Revision history for this message

huangtianhua (huangtianhua) wrote on 2014-01-06:

#15

Missing the tempest?

DennyZhang (denny-6) on 2014-01-06

Changed in cinder:
status:	Confirmed → Fix Committed

Revision history for this message

Doug Hellmann (doug-hellmann) wrote on 2014-01-07:

#16

This is fixed in oslo by https://review.openstack.org/#/c/51295/

Changed in oslo:
status:	In Progress → Fix Committed
assignee:	Sphoorti (sphoorti-joglekar) → Joe Gordon (jogo)
milestone:	none → icehouse-2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-09: Fix merged to python-heatclient (master)

#17

Reviewed: https://review.openstack.org/64326
Committed: https://git.openstack.org/cgit/openstack/python-heatclient/commit/?id=d80462893351d36b4495756773d4f9ad2682f4cc
Submitter: Jenkins
Branch: master

commit d80462893351d36b4495756773d4f9ad2682f4cc
Author: ZhiQiang Fan <email address hidden>
Date: Sat Dec 28 15:51:05 2013 +0800

Remove vim header

    Vim settings can be done in your vimrc file, there is no need to add
    them in each source files. Also if set incorrectly, gate (pep8 check)
    will catch your mistakes.

NOTE: openstack.common files are left untouched because they should be
fixed in oslo.

Change-Id: I96fe6479e64a13766e7a974727405423a66cd496
Closes-Bug: #1229324

Changed in python-heatclient:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-14: Fix proposed to tempest (master)

#18

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

Changed in tempest:
assignee:	nobody → Masayuki Igawa (igawa)
status:	New → In Progress

Noboru Arai (arai-h) on 2014-01-14

Changed in glance:
assignee:	nobody → Noboru Arai (arai-h)
status:	New → In Progress
Changed in python-glanceclient:
assignee:	nobody → Noboru Arai (arai-h)
status:	New → In Progress

Revision history for this message

Noboru Arai (arai-h) wrote on 2014-01-14:

#19

The patch of glance/python-glnaceclient is as follows:

* glance(master)
-https://review.openstack.org/#/c/66504/

* python-glanceclient(master)
-https://review.openstack.org/#/c/66511/

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-15: Fix merged to glance (master)

#20

Reviewed: https://review.openstack.org/66504
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=8de10b0f34df60494ceaee29daf79427e7a38d82
Submitter: Jenkins
Branch: master

commit 8de10b0f34df60494ceaee29daf79427e7a38d82
Author: Noboru arai <email address hidden>
Date: Tue Jan 14 15:56:33 2014 +0000

Remove vim header

No need to set tabstop tons of times, this can be set in your vimrc
file instead.

More disucssion:
http://openstack.10931.n7.nabble.com/Remove-vim-modelines-td21780.html

Change-Id: I3b830a5bb831a63c188109e6fad66ba48884fff3
Partial-Bug: #1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-15: Fix merged to python-glanceclient (master)

#21

Reviewed: https://review.openstack.org/66511
Committed: https://git.openstack.org/cgit/openstack/python-glanceclient/commit/?id=869ea2e61ced4d7ae1e4aa902782090996e2736d
Submitter: Jenkins
Branch: master

commit 869ea2e61ced4d7ae1e4aa902782090996e2736d
Author: Noboru arai <email address hidden>
Date: Tue Jan 14 16:39:41 2014 +0000

Remove vim header

No need to set tabstop tons of times, this can be set in your vimrc
file instead.

More disucssion:
http://openstack.10931.n7.nabble.com/Remove-vim-modelines-td21780.html

Change-Id: I2b37758f9dbb2cad6457a879d4ed7ff0aa69ef8e
Partial-Bug: #1229324

Feilong Wang (flwang) on 2014-01-20

Changed in python-glanceclient:
importance:	Undecided → Low

Noboru Arai (arai-h) on 2014-01-22

Changed in glance:
status:	In Progress → Fix Committed
Changed in python-glanceclient:
status:	In Progress → Fix Committed

OpenStack Infra (hudson-openstack) on 2014-01-22

Changed in tempest:
status:	In Progress → Fix Released

Thierry Carrez (ttx) on 2014-01-22

Changed in glance:
milestone:	none → icehouse-2
status:	Fix Committed → Fix Released

Thierry Carrez (ttx) on 2014-01-22

Changed in cinder:
milestone:	none → icehouse-2
status:	Fix Committed → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-23: Fix proposed to tempest (master)

#22

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

Liusheng (liusheng) on 2014-01-23

Changed in nova:
assignee:	nobody → Liusheng (liusheng)

Thierry Carrez (ttx) on 2014-01-23

Changed in oslo:
status:	Fix Committed → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-25: Fix merged to tempest (master)

#23

Reviewed: https://review.openstack.org/68552
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=aa175f619425fc638fe489f7b8e0cd1f621ad79f
Submitter: Jenkins
Branch: master

commit aa175f619425fc638fe489f7b8e0cd1f621ad79f
Author: Masayuki Igawa <email address hidden>
Date: Thu Jan 23 12:53:06 2014 +0900

Remove vim headers 2nd round

    We don't need to have the vim headers in each source files. So we have
    already removed all of it at
     Change-Id: I43703e2289212389c7841f44691ae7849ed1f505
    However, some files still have it now because of the patch time lag.
    Maybe, we should have a hacking rule for this. But it's out of scope
    here.

Change-Id: I8d09cb41ad633719857973a3ea228cd8dd3a9270
Closes-Bug: #1229324

ZhiQiang Fan (aji-zqfan) on 2014-01-26

Changed in python-neutronclient:
assignee:	ZhiQiang Fan (aji-zqfan) → nobody
Changed in neutron:
assignee:	ZhiQiang Fan (aji-zqfan) → nobody

Matt Riedemann (mriedem) on 2014-01-26

Changed in nova:
importance:	Undecided → Low
status:	New → Confirmed

Revision history for this message

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

#24

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

Changed in nova:
status:	Confirmed → In Progress

Revision history for this message

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

#25

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

Sphoorti (sphoorti-joglekar) on 2014-01-27

Changed in swift:
assignee:	nobody → Sphoorti (sphoorti-joglekar)

Sphoorti (sphoorti-joglekar) on 2014-01-27

Changed in swift:
assignee:	Sphoorti (sphoorti-joglekar) → nobody

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-30: Fix merged to tempest (master)

#26

Reviewed: https://review.openstack.org/69318
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=2e9f23e3cb9f0fb430b558bd845e73f285ebca57
Submitter: Jenkins
Branch: master

commit 2e9f23e3cb9f0fb430b558bd845e73f285ebca57
Author: Joe H. Rahme <email address hidden>
Date: Mon Jan 27 11:29:13 2014 +0100

Removes vim headers 3rd round

This is a follow up on:
Change-Id: I8d09cb41ad633719857973a3ea228cd8dd3a9270

The vim headers in all the files are being removed. We've missed these
files because of delays in the patches.

Change-Id: Ie8097db6c60481749c88991864187ade64f39ecb
Closes-Bug: #1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-30: Fix proposed to tempest (master)

#27

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

Liusheng (liusheng) on 2014-02-07

Changed in neutron:
assignee:	nobody → Liusheng (liusheng)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-07: Fix proposed to neutron (master)

#28

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

Changed in neutron:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-07:

#29

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-07:

#30

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

Liusheng (liusheng) on 2014-02-07

Changed in python-neutronclient:
assignee:	nobody → Liusheng (liusheng)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-07: Fix proposed to python-neutronclient (master)

#31

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

Changed in python-neutronclient:
status:	New → In Progress

ChangBo Guo(gcb) (glongwave) on 2014-02-08

Changed in keystone:
assignee:	nobody → Eric Guo (glongwave)
Changed in python-keystoneclient:
assignee:	nobody → Eric Guo (glongwave)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-08: Fix proposed to python-keystoneclient (master)

#32

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

Changed in python-keystoneclient:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-08: Fix proposed to keystone (master)

#33

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

Changed in keystone:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-10: Fix proposed to python-neutronclient (master)

#34

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

Jay Bryant (jsbryant) on 2014-02-10

Changed in python-cinderclient:
assignee:	nobody → Jay Bryant (jsbryant)
importance:	Undecided → Low

Dolph Mathews (dolph) on 2014-02-11

Changed in keystone:
importance:	Undecided → Wishlist
Changed in python-keystoneclient:
importance:	Undecided → Wishlist

OpenStack Infra (hudson-openstack) on 2014-02-12

Changed in python-cinderclient:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix merged to keystone (master)

#35

Reviewed: https://review.openstack.org/72089
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=263b27f2cd9c93adf37528e8822bac00c18eb549
Submitter: Jenkins
Branch: master

commit 263b27f2cd9c93adf37528e8822bac00c18eb549
Author: Eric Guo <email address hidden>
Date: Sat Feb 8 23:15:34 2014 +0800

Remove vim header

We don't need vim modelines in each source file, it can be set in
user's vimrc.

Change-Id: Ie51ad62946afdf39eadcd59edaf8134ec10265c6
Closes-Bug: #1229324

Changed in keystone:
status:	In Progress → Fix Committed
Changed in python-keystoneclient:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix merged to python-keystoneclient (master)

#36

Reviewed: https://review.openstack.org/72083
Committed: https://git.openstack.org/cgit/openstack/python-keystoneclient/commit/?id=d6d0d66374ac8cc70835d4e809c302bfbbeb798b
Submitter: Jenkins
Branch: master

commit d6d0d66374ac8cc70835d4e809c302bfbbeb798b
Author: Eric Guo <email address hidden>
Date: Sat Feb 8 22:22:08 2014 +0800

Remove vim header

We don't need vim modelines in each source file, it can be set in
user's vimrc.

Change-Id: Ic7a61430a0a320ce6b0c4518d9f5d988e35f8aae
Closes-Bug: #1229324

Yongli He (yongli-he) on 2014-02-13

Changed in trove:
assignee:	nobody → Yongli He (yongli-he)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to trove (master)

#37

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

Yongli He (yongli-he) on 2014-02-13

Changed in tuskar:
assignee:	nobody → Yongli He (yongli-he)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to tuskar (master)

#38

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

Yongli He (yongli-he) on 2014-02-13

Changed in horizon:
assignee:	nobody → Yongli He (yongli-he)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to horizon (master)

#39

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13:

#40

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

Yongli He (yongli-he) on 2014-02-13

Changed in savanna:
assignee:	nobody → Yongli He (yongli-he)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to savanna (master)

#41

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to storyboard (master)

#42

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

Yongli He (yongli-he) on 2014-02-13

Changed in storyboard:
assignee:	nobody → Yongli He (yongli-he)

Yongli He (yongli-he) on 2014-02-13

Changed in ironic:
assignee:	nobody → Yongli He (yongli-he)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to ironic (master)

#43

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to python-ironicclient (master)

#44

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

Xurong Yang (idopra) on 2014-02-13

Changed in swift:
assignee:	nobody → Xurong Yang (idopra)
Changed in python-ironicclient:
assignee:	nobody → Xurong Yang (idopra)
Changed in swift:
status:	New → In Progress

Yongli He (yongli-he) on 2014-02-13

Changed in python-ironicclient:
assignee:	Xurong Yang (idopra) → Yongli He (yongli-he)

Xurong Yang (idopra) on 2014-02-13

Changed in python-ironicclient:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to python-troveclient (master)

#45

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

Xurong Yang (idopra) on 2014-02-13

Changed in python-troveclient:
assignee:	nobody → Xurong Yang (idopra)
status:	New → In Progress
Changed in python-ironicclient:
assignee:	Yongli He (yongli-he) → Xurong Yang (idopra)

Yongli He (yongli-he) on 2014-02-13

Changed in tuskar:
status:	New → In Progress

Yongli He (yongli-he) on 2014-02-13

Changed in python-troveclient:
assignee:	Xurong Yang (idopra) → Yongli He (yongli-he)

Xurong Yang (idopra) on 2014-02-13

Changed in python-swiftclient:
assignee:	nobody → Xurong Yang (idopra)
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to python-swiftclient (master)

#46

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix merged to savanna (master)

#47

Reviewed: https://review.openstack.org/73154
Committed: https://git.openstack.org/cgit/openstack/savanna/commit/?id=4926ded13f82357d1fee8fce46801be4f9a9d2ae
Submitter: Jenkins
Branch: master

commit 4926ded13f82357d1fee8fce46801be4f9a9d2ae
Author: He Yongli <email address hidden>
Date: Thu Feb 13 13:54:57 2014 +0800

Remove extraneous vim configuration comments

Remove vim setting

Remove line containing
comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

at the top of source code files, except for files in
openstack/common.

Change-Id: I9fbab33dd368fa65988f590544a626a2ac1b2304
Close-bug: #1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix proposed to python-ironicclient (master)

#48

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix merged to storyboard (master)

#49

Reviewed: https://review.openstack.org/73156
Committed: https://git.openstack.org/cgit/openstack-infra/storyboard/commit/?id=aec7ddfea31e81dc64638c6a297df33a1dc2e08b
Submitter: Jenkins
Branch: master

commit aec7ddfea31e81dc64638c6a297df33a1dc2e08b
Author: He Yongli <email address hidden>
Date: Thu Feb 13 13:58:42 2014 +0800

Remove extraneous vim configuration comments

Remove vim setting:

Remove line containing
comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

at the top of source code files.

Change-Id: Ibb15b799433000bb20d744dba7fa0c0ab664bbbb
Close-bug: #1229324

Xurong Yang (idopra) on 2014-02-13

Changed in taskflow:
assignee:	nobody → Xurong Yang (idopra)
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix merged to tuskar (master)

#50

Reviewed: https://review.openstack.org/73148
Committed: https://git.openstack.org/cgit/openstack/tuskar/commit/?id=ec057bb073e51b21efef2246ee3428bc80e08a18
Submitter: Jenkins
Branch: master

commit ec057bb073e51b21efef2246ee3428bc80e08a18
Author: He Yongli <email address hidden>
Date: Thu Feb 13 13:07:24 2014 +0800

Remove extraneous vim configuration comments

Remove vim setting

Remove line containing
comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

at the top of source code files, except for files in
openstack/common.

Change-Id: I7d43f92e0110b597452183ae22acaf1114efd3f9
Close-bug: #1229324

Revision history for this message

Dolph Mathews (dolph) wrote on 2014-02-13:

#51

Added hacking, as I'm already seeing regressions for this that should be automatically gated.

Dolph Mathews (dolph) on 2014-02-13

Changed in python-keystoneclient:
milestone:	none → 0.6.0

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-13: Fix merged to python-ironicclient (master)

#52

Reviewed: https://review.openstack.org/73166
Committed: https://git.openstack.org/cgit/openstack/python-ironicclient/commit/?id=3622dfa2fb4c66fd8cbb76bc8a730195f4c32305
Submitter: Jenkins
Branch: master

commit 3622dfa2fb4c66fd8cbb76bc8a730195f4c32305
Author: He Yongli <email address hidden>
Date: Thu Feb 13 14:42:16 2014 +0800

Remove vim header

No need to set tabstop tons of times, this can be set in your vimrc
file instead.

More disucssion:
http://openstack.10931.n7.nabble.com/Remove-vim-modelines-td21780.html

Change-Id: Iff0d2e4725355800f923a04a1bfcac3003a6abd2
Close-bug: #1229324

Xurong Yang (idopra) on 2014-02-14

Changed in hacking:
assignee:	nobody → Xurong Yang (idopra)
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-14: Fix proposed to taskflow (master)

#53

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-14:

#54

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

Jiří Stránský (jistr) on 2014-02-17

Changed in tuskar:
importance:	Undecided → Low

Steve Baker (steve-stevebaker) on 2014-02-17

Changed in python-heatclient:
importance:	Undecided → Medium
milestone:	none → v0.2.7

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-18: Fix proposed to tempest (master)

#55

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-19:

#56

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-20: Fix merged to tempest (master)

#57

Reviewed: https://review.openstack.org/74287
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=7709df8dbf753bdea0a166a4f04d248b9fdab081
Submitter: Jenkins
Branch: master

commit 7709df8dbf753bdea0a166a4f04d248b9fdab081
Author: Masayuki Igawa <email address hidden>
Date: Tue Feb 18 17:16:27 2014 +0900

Removes vim headers 5th round

This is a follow up on:
Change-Id: If56b28d0efb6364fe2105f7d62d6a2b05b906070

    The vim headers in all the files are being removed. We've missed these
    files because of delays in the patches.
    I hope this is the last one for Tempest.

Change-Id: I55dcbc7de4106bc5de852c77d85cf97ac5dab18a
Closes-Bug: #1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-22:

#58

Reviewed: https://review.openstack.org/74596
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=fcacf96204590e57266e81ed13c8a60a60f1b73e
Submitter: Jenkins
Branch: master

commit fcacf96204590e57266e81ed13c8a60a60f1b73e
Author: Masayuki Igawa <email address hidden>
Date: Wed Feb 19 14:00:01 2014 +0900

Introduce T106 rule for vi modelines

    We don't need to have the vi modelines in each source file anymore.
    We've already fixed them several times.
     https://review.openstack.org/#/c/66507/
     https://review.openstack.org/#/c/68552/
     https://review.openstack.org/#/c/69318/
     https://review.openstack.org/#/c/70133/
    However, newly some files still have it in its header. So we should
    check it automatically with our HACKING rule.
    This commit introduces T106 rule for that.

Note: This code is copied from Nova's hacking rule.
Change-Id: I347307a5145b2760c69085b6ca850d6a9137ffc6

Change-Id: I5c94ef041a39c2377ba6321ace8934f324287bcf
Closes-Bug: #1229324

Yongli He (yongli-he) on 2014-02-24

Changed in ironic:
status:	New → In Progress
Changed in horizon:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-24: Fix proposed to oslo.vmware (master)

#59

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

Mike May (themistymay) on 2014-02-24

Changed in savanna:
assignee:	Yongli He (yongli-he) → Mike May (themistymay)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-25: Fix merged to oslo.vmware (master)

#60

Reviewed: https://review.openstack.org/75809
Committed: https://git.openstack.org/cgit/openstack/oslo.vmware/commit/?id=1ad429d885ba80f0da295137089a57879eb342ff
Submitter: Jenkins
Branch: master

commit 1ad429d885ba80f0da295137089a57879eb342ff
Author: ChangBo Guo(gcb) <email address hidden>
Date: Mon Feb 24 04:17:29 2014 -0500

Remove vim header

We don't need vim modelines in each source file, it can be set in
user's vimrc.

Change-Id: Ib52a71276d514ca01dd1e21d754c6c150c94a2a5
Closes-Bug: #1229324

Matthias Runge (mrunge) on 2014-02-25

Changed in horizon:
importance:	Undecided → Wishlist
milestone:	none → icehouse-3

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-02-28: Fix merged to ironic (master)

#61

Reviewed: https://review.openstack.org/73160
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=110074a5946cfde4b5419a62c74fe27aabe75117
Submitter: Jenkins
Branch: master

commit 110074a5946cfde4b5419a62c74fe27aabe75117
Author: He Yongli <email address hidden>
Date: Thu Feb 13 14:13:18 2014 +0800

Remove extraneous vim configuration comments for ironic

Remove vim setting:

comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

at the top of source code files, except for files in
openstack/common.

Change-Id: Ie80f76f08e27b7f5f747227c4cca8d79ad98bec6
Partial-Bug: #1229324

Auston McReynolds (amcrn) on 2014-03-03

Changed in trove:
milestone:	none → icehouse-3
status:	New → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-03-04: Fix proposed to keystone (master)

#62

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

Andrew Lazarev (alazarev) on 2014-03-04

Changed in savanna:
importance:	Undecided → Low
milestone:	none → icehouse-3
status:	New → Fix Committed

Thierry Carrez (ttx) on 2014-03-05

Changed in savanna:
milestone:	icehouse-3 → none
Changed in trove:
milestone:	icehouse-3 → none

Thierry Carrez (ttx) on 2014-03-05

Changed in horizon:
milestone:	icehouse-3 → none

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-03-06: Fix proposed to neutron (master)

#63

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-03-08: Fix merged to keystone (master)

#64

Reviewed: https://review.openstack.org/77906
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=11da8deb8f40b54781eead78d31dcbbad4f2707e
Submitter: Jenkins
Branch: master

commit 11da8deb8f40b54781eead78d31dcbbad4f2707e
Author: Marc Koderer <email address hidden>
Date: Tue Mar 4 16:09:23 2014 +0100

Remove vim headers

Second round to remove all vim headers. We don't need to have the
vim headers in each source files.

More discussions:
http://lists.openstack.org/pipermail/openstack-dev/2013-October/017353.html

Change-Id: I43703e2289212389c7841f44691ae7849ed1f505
Closes-Bug: #1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-03-13: Fix proposed to neutron (master)

#65

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-03-14: Fix merged to python-cinderclient (master)

#66

Reviewed: https://review.openstack.org/72498
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=803f0f69a616700329b702b01404d41827c60ce8
Submitter: Jenkins
Branch: master

commit 803f0f69a616700329b702b01404d41827c60ce8
Author: Jay S. Bryant <email address hidden>
Date: Wed Feb 12 10:18:09 2014 -0600

Remove vim header

    Making this change for python-cinderclient to be consistent
    with Cinder. The same change was made to Cinder via
    commit ff3f2e5735f0d486b72b6881b05ace94d6d34d03 .

Partial-bug 1229324
Change-Id: I69fb960f499f0c9a9c9671d46d10170477cf6695

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-04-02: Fix merged to neutron (master)

#67

Reviewed: https://review.openstack.org/84186
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=5a701d730ae2db05c549721fccf3409d4390ab61
Submitter: Jenkins
Branch: master

commit 5a701d730ae2db05c549721fccf3409d4390ab61
Author: Mohammad Banikazemi <email address hidden>
Date: Mon Mar 31 13:01:47 2014 -0400

Removing vim header lines

    Simply removes the vim header line and the empty line
    following it (when present) for files in the api, db, and
    extenssions base directories. In one single _init_ file in
    this patchset, all comment lines are removed.

Change-Id: I81b7168dc2ad74b8c30a87e50a531e157851499e
Partial-Bug: 1229324

Revision history for this message

Thierry Carrez (ttx) wrote on 2014-04-04:

#68

This bug has now too many tasks, so we can't update those anymore (boo LP). The only way to make them disappear is to delete the tasks entirely.

Keystone, Trove and Sahara were fixed and released, i'm removing them so that they don't appear FixCommitted anymore.

no longer affects:	sahara
no longer affects:	keystone
no longer affects:	trove

Steve Baker (steve-stevebaker) on 2014-04-07

no longer affects:

python-heatclient

Revision history for this message

Joe Gordon (jogo) wrote on 2014-04-09:

#69

I am not too keep on putting this enforcement in hacking, because once all the copies of are gone from a project, I don't think new ones will re-appear (ignoring any works in progress that copied from a file with the header). that being said, if someone wants to add it, I am fine with that.

This can also be gated on using local hacking rules.

Changed in hacking:
importance:	Undecided → Wishlist

Thierry Carrez (ttx) on 2014-04-14

no longer affects:

ironic

Thierry Carrez (ttx) on 2014-04-17

Changed in oslo:
milestone:	icehouse-2 → 2014.1

Thierry Carrez (ttx) on 2014-04-17

Changed in ceilometer:
milestone:	icehouse-1 → 2014.1

Thierry Carrez (ttx) on 2014-04-17

Changed in cinder:
milestone:	icehouse-2 → 2014.1

Thierry Carrez (ttx) on 2014-04-17

Changed in glance:
milestone:	icehouse-2 → 2014.1

gordon chung (chungg) on 2014-04-28

Changed in python-ceilometerclient:
assignee:	Sphoorti (sphoorti-joglekar) → gordon chung (chungg)
importance:	Undecided → Low
status:	In Progress → Triaged

Revision history for this message

Openstack Gerrit (openstack-gerrit) wrote on 2014-05-02: Fix merged to python-ceilometerclient (master)

#70

Reviewed: https://review.openstack.org/90811
Committed: https://git.openstack.org/cgit/openstack/python-ceilometerclient/commit/?id=334739663cb2f724b19585dd0010df5ef2620f0b
Submitter: Jenkins
Branch: master

commit 334739663cb2f724b19585dd0010df5ef2620f0b
Author: Gordon Chung <email address hidden>
Date: Mon Apr 28 16:21:40 2014 -0400

extraneous vim editor configuration comments

Many of the source code files have a beginning line:
vim: tabstop=4 shiftwidth=4 softtabstop=4

This should be deleted.

Change-Id: I0b7bd4e727c0597e7c7fb61d42684e908e001134
Closes-Bug: #1229324

Changed in python-ceilometerclient:
status:	Triaged → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-05-13: Fix merged to horizon (master)

#71

Reviewed: https://review.openstack.org/73152
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=e790ac070e9b8b735c54c613d26e1ee92b7af0b0
Submitter: Jenkins
Branch: master

commit e790ac070e9b8b735c54c613d26e1ee92b7af0b0
Author: He Yongli <email address hidden>
Date: Thu Feb 13 13:43:57 2014 +0800

Remove extraneous vim configuration comments

Remove vim setting:
comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

at the top of source code files, except for files in
openstack/common.

Change-Id: I9a5c6b17c6ef7ecec601f4503dfc7b31fc72e90a
Close-bug: #1229324

Nikhil Manchanda (slicknik) on 2014-05-23

Changed in python-troveclient:
importance:	Undecided → Wishlist

Joshua Harlow (harlowja) on 2014-05-24

Changed in taskflow:
status:	In Progress → Fix Committed

Kyle Mestery (mestery) on 2014-06-18

Changed in neutron:
importance:	Undecided → Wishlist
milestone:	none → juno-2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-19: Fix proposed to neutron (master)

#72

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

Liusheng (liusheng) on 2014-06-23

Changed in neutron:
status:	In Progress → Fix Committed
Changed in nova:
status:	In Progress → Fix Committed

Russell Bryant (russellb) on 2014-07-23

Changed in nova:
milestone:	none → juno-2

Kyle Mestery (mestery) on 2014-07-23

Changed in neutron:
milestone:	juno-2 → juno-3

Thierry Carrez (ttx) on 2014-07-29

Changed in neutron:
milestone:	juno-3 → none
status:	Fix Committed → Fix Released
milestone:	none → juno-2
Changed in nova:
milestone:	juno-2 → none
status:	Fix Committed → Fix Released
milestone:	none → juno-2

Dmitry Tantsur (divius) on 2014-09-22

no longer affects:

python-ironicclient

Dougal Matthews (d0ugal) on 2014-09-25

Changed in tuskar:
status:	In Progress → Fix Committed

Revision history for this message

Morgan Fainberg (mdrnstm) wrote on 2014-09-27:

#73

I can't mark this as fixed released for keystoneclient. It was released in 0.6.0 (a long while ago) I'm going to remove keystoneclient from this bug.

no longer affects:

python-keystoneclient

Marc Koderer (m-koderer) on 2014-10-06

Changed in manila:
assignee:	nobody → Marc Koderer (m-koderer)

OpenStack Infra (hudson-openstack) on 2014-10-06

Changed in manila:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-08: Fix merged to manila (master)

#74

Reviewed: https://review.openstack.org/126258
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=ecf603bc0504f8fc090519e91dbc0d972e602b39
Submitter: Jenkins
Branch: master

commit ecf603bc0504f8fc090519e91dbc0d972e602b39
Author: Marc Koderer <email address hidden>
Date: Mon Oct 6 14:56:03 2014 +0200

Remove vim headers

As discussed in [1] remove all vim headers from manila files.

[1]: http://lists.openstack.org/pipermail/openstack-dev/2013-October/017353.html

Change-Id: I341486c66ffe4dad7db8608fdc66868ceb7ad95a
Closes-Bug: #1229324

Changed in manila:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-08: Fix proposed to manila (master)

#75

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-08: Fix proposed to python-neutronclient (master)

#76

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

Changed in python-neutronclient:
assignee:	Liusheng (liusheng) → Christian Berendt (berendt)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-09: Fix merged to manila (master)

#77

Reviewed: https://review.openstack.org/126996
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=025f68504a8ac7445f5626ecad18acd0edfcf3ee
Submitter: Jenkins
Branch: master

commit 025f68504a8ac7445f5626ecad18acd0edfcf3ee
Author: Christian Berendt <email address hidden>
Date: Wed Oct 8 22:40:24 2014 +0200

Remove extraneous vim editor configuration comments

Change-Id: I3cecf4e2a0f00e14e144124512ce8f7b8928d7f8
Partial-Bug: #1229324

Thierry Carrez (ttx) on 2014-10-16

Changed in nova:
milestone:	juno-2 → 2014.2

Thierry Carrez (ttx) on 2014-10-16

Changed in neutron:
milestone:	juno-2 → 2014.2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-16: Fix proposed to heat (master)

#78

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

Changed in heat:
assignee:	DennyZhang (denny-6) → Oleksii Chuprykov (ochuprykov)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-16: Fix merged to heat (master)

#79

Reviewed: https://review.openstack.org/128943
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=29dfc78d046811dc21c934e3866a293eafc28665
Submitter: Jenkins
Branch: master

commit 29dfc78d046811dc21c934e3866a293eafc28665
Author: Oleksii Chuprykov <email address hidden>
Date: Thu Oct 16 17:51:57 2014 +0300

Remove vim headers

No need to set tabstop tons of times, this can be set in your vimrc
file instead.

Closes-Bug: #1229324

Change-Id: Ib859c0af8a29437540a0586dc0af77bb544ece23

Changed in heat:
status:	In Progress → Fix Committed

Jay Dobies (jdob) on 2014-10-16

Changed in tuskar:
status:	Fix Committed → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-10-24: Fix merged to python-neutronclient (master)

#80

Reviewed: https://review.openstack.org/127009
Committed: https://git.openstack.org/cgit/openstack/python-neutronclient/commit/?id=27f02acc71e48cbeb3855952ab9fef613916c757
Submitter: Jenkins
Branch: master

commit 27f02acc71e48cbeb3855952ab9fef613916c757
Author: Christian Berendt <email address hidden>
Date: Wed Oct 8 22:52:02 2014 +0200

Remove extraneous vim editor configuration comments

Change-Id: I792f24c28063bb57dc2fd295903a7d73c43578d9
Partial-Bug: #1229324

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-11-10: Change abandoned on neutron (master)

#81

Change abandoned by pritesh (<email address hidden>) on branch: master
Review: https://review.openstack.org/99508

Joe Gordon (jogo) on 2014-11-13

no longer affects:

hacking

Thierry Carrez (ttx) on 2014-12-18

Changed in heat:
milestone:	none → kilo-1
status:	Fix Committed → Fix Released

Thierry Carrez (ttx) on 2014-12-18

Changed in manila:
milestone:	none → kilo-1
status:	Fix Committed → Fix Released

Joshua Harlow (harlowja) on 2014-12-19

Changed in taskflow:
status:	Fix Committed → Fix Released

Thierry Carrez (ttx) on 2015-04-30

Changed in heat:
milestone:	kilo-1 → 2015.1.0

Thierry Carrez (ttx) on 2015-04-30

Changed in manila:
milestone:	kilo-1 → 2015.1.0

Doug Hellmann (doug-hellmann) on 2015-06-24

Changed in python-ceilometerclient:
status:	Fix Committed → Fix Released

Doug Hellmann (doug-hellmann) on 2015-08-31

Changed in python-glanceclient:
milestone:	none → 1.0.0
status:	Fix Committed → Fix Released

Sean McGinnis (sean-mcginnis) on 2015-11-23

Changed in python-cinderclient:
status:	In Progress → Fix Released

Akihiro Motoki (amotoki) on 2015-12-03

Changed in python-neutronclient:
status:	In Progress → Fix Released

Tatiana Ovchinnikova (tmazur) on 2016-07-12

Changed in manila-ui:
assignee:	nobody → Tatiana Ovchinnikova (tmazur)
Changed in horizon:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-07-12: Fix proposed to manila-ui (master)

#82

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

Changed in manila-ui:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-07-12: Fix merged to manila-ui (master)

#83

Reviewed: https://review.openstack.org/340815
Committed: https://git.openstack.org/cgit/openstack/manila-ui/commit/?id=e3fec776b8900cc200e488ea11472433708668f7
Submitter: Jenkins
Branch: master

commit e3fec776b8900cc200e488ea11472433708668f7
Author: Tatiana Ovchinnikova <email address hidden>
Date: Tue Jul 12 12:21:38 2016 +0300

Remove vim headers

We don't need vim settings in each source file:

comment - # vim: tabstop=4 shiftwidth=4 softtabstop=4

It can be set in user's vimrc.

Change-Id: I205123ae076cf5801f24992cb4897ba2a37bd4b5
Partial-Bug: #1229324

Tatiana Ovchinnikova (tmazur) on 2016-07-13

Changed in manila-ui:
status:	In Progress → Fix Committed

Revision history for this message

Zhao Chao (zhaochao1984) wrote on 2018-02-11:

#84

https://review.openstack.org/#/c/73169/

Changed in python-troveclient:
status:	In Progress → Fix Committed
status:	Fix Committed → Fix Released

Goutham Pacha Ravi (gouthamr) on 2018-12-04

Changed in manila-ui:
importance:	Undecided → Low
status:	Fix Committed → Fix Released

Revision history for this message

Christian Schwede (cschwede) wrote on 2025-04-08:

#86

Has been fixed in commit #a0017e053 and is also checked using tox nowadays. Closing this bug therefore.

Changed in swift:
status:	In Progress → Fix Committed
status:	Fix Committed → Fix Released
Changed in python-swiftclient:
status:	In Progress → Fix Committed
status:	Fix Committed → Fix Released

Ceilometer

extraneous vim editor configuration comments

Bug Description

Duplicates of this bug

Other bug subscribers

Remote bug watches

	Status	Importance	Assigned to	Milestone
Ceilometer	Fix Released	Low	Terri Yu	Ceilometer 2014.1 "icehouse"
Cinder	Fix Released	Low	Mike Perez	Cinder 2014.1 "icehouse"
Glance	Fix Released	Undecided	Noboru Arai	Glance 2014.1 "icehouse"
Glance Client	Fix Released	Low	Noboru Arai	Glance Client 1.0.0 "1.0.0"
OpenStack Compute (nova)	Fix Released	Low	Liusheng	OpenStack Compute (nova) 2014.2 "juno"
OpenStack Dashboard (Horizon)	Fix Released	Wishlist	Yongli He
OpenStack Heat	Fix Released	Low	Oleksii Chuprykov	OpenStack Heat 2015.1.0 "kilo"
OpenStack Object Storage (swift)	Fix Released	Undecided	Xurong Yang
OpenStack Shared File Systems Service (Manila)	Fix Released	Undecided	Marc Koderer	OpenStack Shared File Systems Service (Manila) 2015.1.0 "kilo"
manila-ui	Fix Released	Low	Tatiana Ovchinnikova
neutron	Fix Released	Wishlist	Liusheng	neutron 2014.2 "juno"
oslo-incubator	Fix Released	Low	Joe Gordon	oslo-incubator 2014.1 "icehouse"
python-ceilometerclient	Fix Released	Low	gordon chung	python-ceilometerclient 1.3.0 "1.3.0"
python-cinderclient	Fix Released	Low	Jay Bryant
python-neutronclient	Fix Released	Undecided	Christian Berendt
python-swiftclient	Fix Released	Undecided	Xurong Yang
python-troveclient	Fix Released	Wishlist	Yongli He
storyboard	New	Undecided	Yongli He
taskflow	Fix Released	Undecided	Xurong Yang	taskflow 0.6.0
tempest	Fix Released	Undecided	Masayuki Igawa
tuskar	Fix Released	Low	Yongli He