Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: openstack/devstack
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: beenum22/devstack
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 20 commits
  • 17 files changed
  • 12 contributors

Commits on Feb 8, 2017

  1. Update .gitreview for stable/ocata

    Change-Id: I7dbf2c62bdbb8d372aea5d2bfd0d9b5f55e8cbf6
    dhellmann committed Feb 8, 2017
    Configuration menu
    Copy the full SHA
    572815e View commit details
    Browse the repository at this point in the history
  2. Set max microversions for ocata

    This commit sets the max microversions on the ocata branch so that
    tempest doesn't try to run tests that require a newer api version.
    
    Change-Id: Id033dde91b6a7aa09aca2332fe815fd72be90438
    mtreinish committed Feb 8, 2017
    Configuration menu
    Copy the full SHA
    58de94f View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2017

  1. only apply ebtables race fix on trusty

    Change-Id: Ifc83e7301d9d921ce9ceed349f116584ce03842b
    sdague committed Feb 9, 2017
    Configuration menu
    Copy the full SHA
    cead7a1 View commit details
    Browse the repository at this point in the history

Commits on Feb 13, 2017

  1. Configuration menu
    Copy the full SHA
    291d2d4 View commit details
    Browse the repository at this point in the history

Commits on Feb 14, 2017

  1. Update branch references for stable/ocata

    Change-Id: Id7400d5222a7608b992d6e0f629204265fe458c6
    mtreinish committed Feb 14, 2017
    Configuration menu
    Copy the full SHA
    c1921f7 View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2017

  1. TLS proxy: disable HTTP KeepAlive

    There's a race condition when a client makes a request "at the same
    time" the HTTP connection is being closed by Apache because the
    `KeepAliveTimeout` is expired.
    
    This is explained in detail and can be reproduce using
    https://github.com/mikem23/keepalive-race or
    https://github.com/JordanP/openstack-snippets/blob/master/keepalive-race/keep-alive-race.py
    
    Just disable KeepAlive to fix the
    ('Connection aborted.', BadStatusLine("''",)) error we are seeing.
    
    Change-Id: I46e9f70ee740ec7996c98d386d5289c1491e9436
    (cherry picked from commit 4370925)
    Jordan Pittier committed Feb 15, 2017
    Configuration menu
    Copy the full SHA
    00edb2f View commit details
    Browse the repository at this point in the history

Commits on Feb 16, 2017

  1. Switch devstack to use qcow cirros img

    This commit switches devstack to use the published qcow2 cirros image
    instead of the AMI version. Using AMI was mostly a historical artifact
    dating pretty far back, but in the real world no one really uses AMI
    images with openstack clouds. This change reflects that and also
    enables tempest ro remove its deprecated config options for using AMI
    as a fallback on misconfiguration (which was just there to support
    devstack's defaults)
    
    Change-Id: Id65ebae73b28da7185cb349b714b659af51ef77f
    (cherry picked from commit 6fc332d)
    mtreinish authored and sdague committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    f61ca5f View commit details
    Browse the repository at this point in the history

Commits on Feb 24, 2017

  1. Enable placement section on controller in multinode setup

    Currently placement api section will be configured on controller
    only if service n-cpu is running. It breaks multi node setup.
    Closes-Bug: #1667219
    
    Change-Id: I8b0f60f253859f704bb9831d7dac8f55df353ac7
    (cherry picked from commit f58b373)
    prashantshetty committed Feb 24, 2017
    Configuration menu
    Copy the full SHA
    770f043 View commit details
    Browse the repository at this point in the history

Commits on Feb 28, 2017

  1. Fix api extension settings in tempest for Ocata

    On stable branches, we are supposed to hard code the extension list of
    what API extensions are available when we branch the project so that
    test cases against new features will be skipped. However, we didn't do
    this on Ocata release.
    
    Change-Id: I1a4d1ea8c8d34b8cd1020e76597db6c67f85f6ec
    dlux committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    d26bcdf View commit details
    Browse the repository at this point in the history

Commits on Mar 1, 2017

  1. Ensure we install setuptools from requirements

    Use pip_install_gr so we get the version pinned by requirements.  The
    depends-on is an example of where we're trying to pin to workaround
    issues.
    
    Depends-On: I9c57c08a150571c5bb62235d502839394d53a4c1
    Change-Id: I780cca681b12a3e9d228dbf2fd9fa6e8ab1a82e1
    (cherry picked from commit 1e7f738)
    ianw authored and Denis Buliga committed Mar 1, 2017
    Configuration menu
    Copy the full SHA
    9379c95 View commit details
    Browse the repository at this point in the history

Commits on Mar 3, 2017

  1. Skips enabling kernel bridge firewall in container

    Calling enable_kernel_bridge_firewall inside a
    container, devstack will crash because it tries to
    load a kernel module by calling 'sudo modprobe' on
    net.bridge.
    
    Change-Id: Id4718c065d5a8c507d49f38e19c2796a64221aa4
    Closes-Bug: #1662194
    (cherry picked from commit 0bf75a4)
    Denis Buliga committed Mar 3, 2017
    Configuration menu
    Copy the full SHA
    28b034f View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2017

  1. Configuration menu
    Copy the full SHA
    ac05478 View commit details
    Browse the repository at this point in the history
  2. Revert "Switch devstack to use qcow cirros img"

    This was not supposed to be merged yet, it breaks grenade.
    There needs to be a fix applied to master and then included in this backport.
    
    This reverts commit f61ca5f.
    
    Change-Id: I6321039117dcff80bb87e43f0e49082071018936
    mtreinish committed Mar 5, 2017
    Configuration menu
    Copy the full SHA
    14b3b96 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    d319ee9 View commit details
    Browse the repository at this point in the history

Commits on Mar 6, 2017

  1. Merge "Skips enabling kernel bridge firewall in container" into stabl…

    …e/ocata
    Jenkins authored and openstack-gerrit committed Mar 6, 2017
    Configuration menu
    Copy the full SHA
    79c19b6 View commit details
    Browse the repository at this point in the history

Commits on Mar 8, 2017

  1. Make declared variables global

    When variables use the 'declare' directive, it is by default a local
    variable. While other variables have global scope.
    
    For example:
       declare -A AN_ARRAY   # local in scope
       foo=1                 # global in scope
    
    This causes errors to occur as some of the variables will be local only
    and others will be global.
    
    Update the code, as appropriate, so that variables using the 'declare'
    directive also include the '-g' flag to have them also be global. Not
    every instance of a declared variable has been updated.
    
    Closes-Bug: #1669509
    Co-Authored-By: John L. Villalovos <[email protected]>
    Change-Id: I2180b68fe861ad19c6d4ec0df0f9f8a528347862
    (cherry picked from commit afef8bf)
    sdague and John L. Villalovos committed Mar 8, 2017
    Configuration menu
    Copy the full SHA
    49f5b9f View commit details
    Browse the repository at this point in the history

Commits on Mar 13, 2017

  1. Use qemu-kvm-ev package on centos

    For the latest qemu-kvm, you have to use the qemu-kvm-ev package,
    which is based off the qemu-kvm-rhev package, which is explained in
    [1] but you probably can't read it.  The gist is, that qemu-kvm-rhev
    is a later build of kvm that is incompatible with the base version
    provided.  qemu-kvm-rhev is only provided with the RHV (ovirt) and
    RHOS (openstack) products.  CentOS rebuilds this package as
    qemu-kvm-ev as part of it's virtualisation SIG.
    
    I9a972e3fde2e4e552f6fc98350820c07873c3de3 has bumped up the minimum
    qemu version to 2.1.0.  It seems there is a an issue (bug #1668164)
    where having the qemu-system package installed gets picked up if
    installed, and reports the incorrect version to nova, causing failure.
    
    This removes the installs from files/rpms/nova as it is all being done
    in function-libvirt.  We only install the qemu-kvm-ev package on
    centos and remove the old work-around.
    
    [1] https://access.redhat.com/solutions/629513
    [2] https://wiki.centos.org/SpecialInterestGroup/Virtualization
    
    Change-Id: Ide91b261f35fb19d8bd7155ca016fa3b76a45ea1
    (cherry picked from commit 52bb641)
    ianw authored and arnagaram committed Mar 13, 2017
    Configuration menu
    Copy the full SHA
    eec5c1a View commit details
    Browse the repository at this point in the history

Commits on Mar 15, 2017

  1. Fix install_libvirt for other RHEL-based distros

    Since https://review.openstack.org/#/c/438325 landed
    it only works for Centos 7, but not for other
    RHEL-based distributions: Virtuozzo and, probably, RHEV.
    
    Both of above have own version for qemu-kvm package: qemu-kvm-vz and qemu-kvm-rhev,
    accordingly. These packages provide "qemu-kvm", like qemu-kvm-ev,
    and, when you call "yum install qemu-kvm", they replace the default OS package.
    
    Change-Id: I46da627c0da8925064862fdc283db81591979285
    (cherry picked from 008aa3e)
    Evgeny Antyshev authored and ianw committed Mar 15, 2017
    Configuration menu
    Copy the full SHA
    53bc172 View commit details
    Browse the repository at this point in the history

Commits on Mar 17, 2017

  1. Use string comparison to compare UUID values

    Was using the '-ne' integer comparison operator to compare UUID
    values. This caused error messages like:
    
    /opt/stack/new/devstack/lib/tempest: line 226: [[: dfae26ac-1780-4677-902d: value too great for base (error token is "902d")
    
    Change it to use '!=' string comparison operator
    
    Change-Id: Ib7c9197dd0fe58addf33b4f82beea6de64f6b10b
    (cherry picked from commit 3345a6d)
    John L. Villalovos committed Mar 17, 2017
    Configuration menu
    Copy the full SHA
    2c45d94 View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2017

  1. Replaced cirros_arch from x86_64 to arm

    beenum22 authored and Your Name committed Mar 26, 2017
    Configuration menu
    Copy the full SHA
    7844112 View commit details
    Browse the repository at this point in the history
Loading