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
    dhellmann committed Feb 8, 2017
    Configuration menu
    Copy the full SHA
    572815eView 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
    mtreinish committed Feb 8, 2017
    Configuration menu
    Copy the full SHA
    58de94fView 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
    sdague committed Feb 9, 2017
    Configuration menu
    Copy the full SHA
    cead7a1View commit details
    Browse the repository at this point in the history

Commits on Feb 13, 2017

  1. Configuration menu
    Copy the full SHA
    291d2d4View 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
    mtreinish committed Feb 14, 2017
    Configuration menu
    Copy the full SHA
    c1921f7View 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
    00edb2fView 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@sdague
    mtreinish authored and sdague committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    f61ca5fView 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
    prashantshetty committed Feb 24, 2017
    Configuration menu
    Copy the full SHA
    770f043View 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
    dlux committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    d26bcdfView 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
    ianw authored and Denis Buliga committed Mar 1, 2017
    Configuration menu
    Copy the full SHA
    9379c95View 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
    28b034fView commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2017

  1. Configuration menu
    Copy the full SHA
    ac05478View 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
    mtreinish committed Mar 5, 2017
    Configuration menu
    Copy the full SHA
    14b3b96View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    d319ee9View commit details
    Browse the repository at this point in the history

Commits on Mar 6, 2017

  1. Configuration menu
    Copy the full SHA
    79c19b6View 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
    sdague and John L. Villalovos committed Mar 8, 2017
    Configuration menu
    Copy the full SHA
    49f5b9fView 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@arnagaram
    ianw authored and arnagaram committed Mar 13, 2017
    Configuration menu
    Copy the full SHA
    eec5c1aView 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)
    @ianw
    Evgeny Antyshev authored and ianw committed Mar 15, 2017
    Configuration menu
    Copy the full SHA
    53bc172View 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
    2c45d94View 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
    beenum22 authored and Your Name committed Mar 26, 2017
    Configuration menu
    Copy the full SHA
    7844112View commit details
    Browse the repository at this point in the history
Loading