Skip to content
Snippets Groups Projects
  1. Apr 20, 2019
    • chenxing's avatar
      Increase retries numbers of grafana check task · 20ef211f
      chenxing authored
      20 seconds may be too short to wait for grafana ready, Let's keep the
      check task 60 seconds.
      
      backport: rocky
      
      Change-Id: Ib219ad215d1ef2147ba3591f8c398feb4f3c8888
      Closes-Bug: #1821285
      20ef211f
  2. Apr 17, 2019
    • Doug Szumski's avatar
      Set external web URL for Prometheus services · 9d495504
      Doug Szumski authored
      This change ensures that URLs returned from these services reference
      the HAProxy endpoint, rather than the host on which the service is
      running.
      
      Closes-Bug: #1825150
      Change-Id: I7f966ff749ea37620f1bde7019a598cb9505fa45
      9d495504
  3. Apr 10, 2019
  4. Apr 09, 2019
  5. Apr 08, 2019
    • Mark Goddard's avatar
      Use ironic inspector 'dnsmasq' PXE filter by default · 86e83fae
      Mark Goddard authored
      With Docker CE, the daemon sets the default policy of the iptables
      FORWARD chain to DROP. This causes problems for provisioning bare metal
      servers when ironic inspector is used with the 'iptables' PXE filter.
      It's not entirely clear why these two things interact in this way,
      but switching to the 'dnsmasq' filter works around the issue, and is
      probably a good move anyway because it is more efficient.
      
      We have added a migration task here to flush and remove the ironic-inspector
      iptables chain since inspector does not do this itself currently.
      
      Change-Id: Iceed5a096819203eb2b92466d39575d3adf8e218
      Closes-Bug: #1823044
      86e83fae
    • Mark Goddard's avatar
      Make monasca notification templates optional · a0e21411
      Mark Goddard authored
      backport: stein
      
      If I deploy monasca by setting enable_monasca to true, the monasca_notification
      restarts with the following error:
      
      ERROR:__main__:MissingRequiredSource: /var/lib/kolla/config_files/notification_templates/* file is not found
      
      These templates are optional, so we need to mark this directory as optional in
      config.json.
      
      Change-Id: Ia2dd835daa7ab1153617cc92f17c2d8d498c73e0
      Closes-Bug: #1823726
      a0e21411
    • Mark Goddard's avatar
      Remove shutdown of MariaDB · d93c604d
      Mark Goddard authored
      Since we are now in the Train cycle, we can be sure that any running
      MariaDB containers can be safely stopped, and we do not need to perform
      an explicit shutdown prior to restarting them.
      
      Change-Id: I5450690f1cbe0c995e8e4b01a76e90dac2574d61
      Related-Bug: #1820325
      d93c604d
    • Mark Goddard's avatar
      Set previous release to Stein · 3e77798f
      Mark Goddard authored
      Now that the stable/stein branch has been cut, we can set the previous
      release to Stein. This is done in kolla-ansible for rolling upgrades,
      and in CI configuration for upgrade tests.
      
      Change-Id: I87269738db9521fc22a6ce3aee67d9ab00d47e2a
      3e77798f
  6. Apr 03, 2019
  7. Apr 02, 2019
  8. Apr 01, 2019
    • 白永君's avatar
      Change action to kolla_action · 4bdb0f35
      白永君 authored
      the action show be kolla_action in kolla-ansible/cyborg
      Closes-bug: #1822555
      Change-Id: I3dd909ae785dd43d5faabbc3fdd5f8bb8aca370b
      4bdb0f35
  9. Mar 29, 2019
  10. Mar 28, 2019
    • Michal Nasiadka's avatar
      Add support for ovsdb conversion · 2a6070b9
      Michal Nasiadka authored
      After upgrade we should check if OVSDB doesn't need conversion to new
      version - this patch adds that to ovsdb start script.
      
      Change-Id: Ifa8766d050b506708142a1970121ce5944c6bae1
      Closes-Bug: #1792496
      2a6070b9
  11. Mar 27, 2019
    • jamesbagwell's avatar
      Removing '/certificates' entry in generate.yml as this causes an · c0a3970e
      jamesbagwell authored
      incorrect path when generating certificates.
      
      The 'setting permissions on key' task fails because the task looks for
      the haproxy.key in an invalid path. The certificates_dir is defined as
      '{{ node_config }}/certificates' in the main.yml . The 'Setting
      permissions on Key' task has a path of '{{ certificates_dir
      }}/certificates/private/haproxy.key which is incorrect. Removing the
      'certificates' in the path corrects this problem and allows the user to
      successfully create certificates using 'kolla-ansible certificates'.
      
      Change-Id: I37b10b994b05d955b6f67c908df1472231a91160
      Closes-Bug: 1821805
      c0a3970e
    • Serhat Demircan's avatar
      Retry perform a synced flush task while upgrading elasticsearch · adb02958
      Serhat Demircan authored
      The synced flush fails due to concurrent indexing operations.
      The HTTP status code in that case will be 409 CONFLICT. We can
      retry this task until returns success.
      
      Change-Id: I57f9a009b12715eed8dfcf829a71f418d2ce437b
      adb02958
  12. Mar 26, 2019
  13. Mar 25, 2019
    • Mark Goddard's avatar
      Remove recurse: yes for owner/perms on /etc/kolla · 6b0be5c5
      Mark Goddard authored
      When kolla-ansible bootstrap-servers is run, it executes one of the
      following two tasks:
      
      - name: Ensure node_config_directory directory exists for user kolla
        file:
          path: "{{ node_config_directory }}"
          state: directory
          recurse: true
          owner: "{{ kolla_user }}"
          group: "{{ kolla_group }}"
          mode: "0755"
        become: True
        when: create_kolla_user | bool
      
      - name: Ensure node_config_directory directory exists
        file:
          path: "{{ node_config_directory }}"
          state: directory
          recurse: true
          mode: "0755"
        become: True
        when: not create_kolla_user | bool
      
      On the first run, normally node_config_directory (/etc/kolla/) doesn't
      exist, so it is created with kolla:kolla ownership and 0755 permissions.
      
      If we then run 'kolla-ansible deploy', config files are created for
      containers in this directory, e.g. /etc/kolla/nova-compute/. Permissions
      for those files should be set according to 'config_owner_user' and
      'config_owner_group'.
      
      If at some point we again run kolla-ansible bootstrap-servers, it will
      recursively set the ownership and permissions of all files in /etc/kolla
      to kolla:kolla / 0755.
      
      The solution is to change bootstrap-servers to not set the owner and
      permissions recursively. It's also arguable that /etc/kolla should be
      owned by 'config_owner_user' and 'config_owner_group', rather than
      kolla:kolla, although that's a separate issue.
      
      Change-Id: I24668914a9cedc94d5a6cb835648740ce9ce6e39
      Closes-Bug: #1821599
      6b0be5c5
  14. Mar 23, 2019
    • Mark Goddard's avatar
      Fix MariaDB 10.3 upgrade · b25c0ee4
      Mark Goddard authored
      Upgrading MariaDB from Rocky to Stein currently fails, with the new
      container left continually restarting. The problem is that the Rocky
      container does not shutdown cleanly, leaving behind state that the new
      container cannot recover. The container does not shutdown cleanly
      because we run dumb-init with a --single-child argument, causing it to
      forward signals to only the process executed by dumb-init. In our case
      this is mysqld_safe, which ignores various signals, including SIGTERM.
      After a (default 10 second) timeout, Docker then kills the container.
      
      A Kolla change [1] removes the --single-child argument from dumb-init
      for the MariaDB container, however we still need to support upgrading
      from Rocky images that don't have this change. To do that, we add new
      handlers to execute 'mysqladmin shutdown' to cleanly shutdown the
      service.
      
      A second issue with the current upgrade approach is that we don't
      execute mysql_upgrade after starting the new service. This can leave the
      database state using the format of the previous release. This patch also
      adds handlers to execute mysql_upgrade.
      
      [1] https://review.openstack.org/644244
      
      Depends-On: https://review.openstack.org/644244
      Depends-On: https://review.openstack.org/645990
      Change-Id: I08a655a359ff9cfa79043f2166dca59199c7d67f
      Closes-Bug: #1820325
      b25c0ee4
  15. Mar 22, 2019
    • Mark Goddard's avatar
      Fix booting instances after nova-compute upgrade · 192dcd1e
      Mark Goddard authored
      After upgrading from Rocky to Stein, nova-compute services fail to start
      new instances with the following error message:
      
      Failed to allocate the network(s), not rescheduling.
      
      Looking in the nova-compute logs, we also see this:
      
      Neutron Reported failure on event
      network-vif-plugged-60c05a0d-8758-44c9-81e4-754551567be5 for instance
      32c493c4-d88c-4f14-98db-c7af64bf3324: NovaException: In shutdown, no new
      events can be scheduled
      
      During the upgrade process, we send nova containers a SIGHUP to cause
      them to reload their object version state. Speaking to the nova team in
      IRC, there is a known issue with this, caused by oslo.service performing
      a full shutdown in response to a SIGHUP, which breaks nova-compute.
      There is a patch [1] in review to address this.
      
      The workaround employed here is to restart the nova compute service.
      
      [1] https://review.openstack.org/#/c/641907
      
      Change-Id: Ia4fcc558a3f62ced2d629d7a22d0bc1eb6b879f1
      Closes-Bug: #1821362
      192dcd1e
    • Mark Goddard's avatar
      Update openstack_previous_release_name to rocky · 98df4dd8
      Mark Goddard authored
      This is used for version pinning during rolling upgrades.
      
      Change-Id: I6e878a8f7c9e0747d8d60cb4527c5f8f039ec15a
      98df4dd8
    • Scott Solkhon's avatar
      Add mising handlers for external Ceph. · c70d8066
      Scott Solkhon authored
      When Nova, Glance, or Cinder are deployed alongside an external Ceph deployment
      handlers will fail to trigger if keyring files are updated, which results in the
      containers not being restarted.
      
      This change adds the missing 'when' conditions for nova-libvirt, nova-compute,
      cinder-volume, cinder-backup, and glance-api containers.
      
      Change-Id: I8e183aac9a72e7a7210f7edc7cdcbaedd4fbcaa9
      c70d8066
  16. Mar 21, 2019
    • Mark Goddard's avatar
      Fix neutron rolling upgrade · 55633ebf
      Mark Goddard authored
      Services were being passed as a JSON list, then iterated over in the
      neutron-server container's extend_start.sh script like this:
      
      ['neutron-server'
      'neutron-fwaas'
      'neutron-vpnaas']
      
      I'm not actually sure why we have to specify services explicitly, it
      seems liable to break if we have other plugins that need migrating.
      
      Change-Id: Ic8ce595793cbe0772e44c041246d5af3a9471d44
      55633ebf
    • Michal Nasiadka's avatar
      Fix placement-api WSGI error · 779a8013
      Michal Nasiadka authored
      RDO is packaging placement-api with bundled httpd config
      and it conflicts with kolla-ansible generated one.
      
      Change-Id: I018a4ed1b2282e8a789b63e3893e61db2fde8cf2
      779a8013
  17. Mar 20, 2019
    • Mark Goddard's avatar
      Fix swift reconfigure · 5629b5aa
      Mark Goddard authored
      Reconfiguring Swift currently fails to restart containers if
      configuration changes. This is because kolla_set_configs is executed in
      the containers as the default swift user, which does not have permission
      to access all necessary files.
      
      This change uses the root user to execute the command instead, which
      allows it to exit with the correct status of 1 if the config files
      differ.
      
      Change-Id: I2a2363c71430a7173bb5253662412ae5dba09654
      5629b5aa
    • Mark Goddard's avatar
      Don't pull images during upgrade · 58d6dc3b
      Mark Goddard authored
      When adding the rolling upgrade support, some upgrade procedures were
      modified to pull images explicitly. This is done inconsistently between
      services, and is a change in behaviour from Rocky and earlier releases.
      
      This change removes all image pulling from upgrade tasks.
      
      Change-Id: Id0fed17714235e1daed60b83b1f30620f097eb97
      58d6dc3b
  18. Mar 18, 2019
  19. Mar 15, 2019
  20. Mar 14, 2019
    • Victor Coutellier's avatar
      Fix action_plugins python3 compatibility · 85f7da97
      Victor Coutellier authored
      Fix filemode in the merge_configs and merge_yaml action plugin to
      be compatible with python3
      
      Change-Id: Ief64c5bdcd717141281e23c255a49ec02a96aef2
      Closes-Bug: #1820134
      85f7da97
    • Scott Solkhon's avatar
      Support separate Swift storage networks · a781c643
      Scott Solkhon authored
      Adds support to seperate Swift access and replication traffic from other storage traffic.
      
      In a deployment where both Ceph and Swift have been deployed,
      this changes adds functionalality to support optional seperation
      of storage network traffic. This adds two new network interfaces
      'swift_storage_interface' and 'swift_replication_interface' which maintain
      backwards compatibility.
      
      The Swift access network interface is configured via 'swift_storage_interface',
      which defaults to 'storage_interface'. The Swift replication network
      interface is configured via 'swift_replication_interface', which
      defaults to 'swift_storage_interface'.
      
      If a separate replication network is used, Kolla Ansible now deploys separate
      replication servers for the accounts, containers and objects, that listen on
      this network. In this case, these services handle only replication traffic, and
      the original account-, container- and object- servers only handle storage
      user requests.
      
      Change-Id: Ib39e081574e030126f2d08f51de89641ddb0d42e
      a781c643
    • caoyuan's avatar
      Add ceilometer_ipmi container into ceilometer role · 16900c2e
      caoyuan authored
      refer to [0]
      
      [0]: https://docs.openstack.org/ceilometer/latest/install/install-compute-rdo.html
      
      
      Co-Authored-By: default avatarzhulingjie <easyzlj@gmail.com>
      
      Change-Id: I4cda336dedb3d807b80d13bcc219268a8d667b4d
      16900c2e
  21. Mar 13, 2019
  22. Mar 11, 2019
    • Pierre Riteau's avatar
      Fix failure to mount cephfs · 42dd7784
      Pierre Riteau authored
      Commit 2f6b1c68 changed the way the
      cephfs source path was generated and dropped the source path component,
      keeping only the list of IPs and ports. This results in failures to
      mount cephfs with the following message:
      
          source mount path was not specified
          failed to resolve source
      
      Change-Id: I94d18ec064971870264ae8d0b279564f2172e548
      Closes-Bug: #1819502
      42dd7784
    • Erol Guzoglu's avatar
      Support the prometheus elasticsearch exporter · 14ab9a7c
      Erol Guzoglu authored
      This patch implements the support for the elasticsearch-exporter in
      kolla-ansible
      
      The configuration and prechecks are reused from the other exporters
      
      Depends-On: Id138f12e10102a6dd2cd8d84f2cc47aa29af3972
      Change-Id: Iae0eac0179089f159804490bf71f1cf2c38dde54
      14ab9a7c
Loading