)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"7f566532af4c3953e39191408bac4db5f9d654b2","unresolved":false,"context_lines":[{"line_number":14,"context_line":"For Ubuntu, all of the python 2/3 modules we need already exist in"},{"line_number":15,"context_line":"Ubuntu repositories."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"OpenSUSE has some package dependencies that aren\u0027t satisfied in the"},{"line_number":18,"context_line":"Pike repo. Those changes will come in another patch."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ic4d1bace3568a2b9ad35442b9ab1e24e0b486f9f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"7f287b81_a26f5f26","line":18,"range":{"start_line":17,"start_character":0,"end_line":18,"end_character":52},"updated":"2017-09-07 21:27:09.000000000","message":"The problem with openSUSE is that the Pike repository provides these python packages but zypper refuses to update the system ones because they are located on a different repository. The solution is to force zypper to come up with a solution. This can be done by passing --force along with the packages that needs updating so zypper forces an update on all of them\n\n\n\ndiff --git a/tasks/pre_install.yml b/tasks/pre_install.yml\nindex a028a67..88e344b 100644\n--- a/tasks/pre_install.yml\n+++ b/tasks/pre_install.yml\n@@ -28,6 +28,9 @@\n   package:\n     name: \"{{ pip_install_distro_packages }}\"\n     state: \"{{ pip_install_package_state }}\"\n+    # NOTE(hwoarang) need to pass --force so zypper can\n+    # fetch the updates with vendor change\n+    force: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027zypper\u0027) | ternary(\u0027yes\u0027, omit) }}\"\n   register: install_packages\n   until: install_packages|success\n   retries: 5\ndiff --git a/vars/suse-42.yml b/vars/suse-42.yml\nindex 033e40f..230ff25 100644\n--- a/vars/suse-42.yml\n+++ b/vars/suse-42.yml\n@@ -33,3 +33,6 @@ pip_install_distro_packages:\n   - python-pyOpenSSL\n   - python-requests\n   - python-urllib3\n+  - python-cryptography\n+  - python-idna\n+  - python-cffi\n\nWith that patch I can confirm that openSUSE is happy with the rest of your changes.","commit_id":"bf99e76524b47b870a5e377f1c9c4f5ece3d7860"}],"defaults/main.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"77a22e02c892070a9159e62d8eb8bd6bce022765","unresolved":false,"context_lines":[{"line_number":77,"context_line":"pip_install_distro_packages: \"{{ pip_install_build_packages | ternary(pip_install_distro_build_packages, []) }}\""},{"line_number":78,"context_line":"pip_install_remove_distro_packages: []"},{"line_number":79,"context_line":"pip_install_distro_build_packages: []"},{"line_number":80,"context_line":"pip_install_distro_packages: []"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"# Some pip installations require the system to have a compiler present"},{"line_number":83,"context_line":"# Toggling this will install a compiler and python-dev libraries to the host"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_fc17e044","line":80,"updated":"2017-09-01 15:10:37.000000000","message":"We should probably put these into each of the distro content, or in vars/main","commit_id":"c22d2ed765302d0078c1c475e97b9c6efd9e5760"}],"tasks/pre_install_yum.yml":[{"author":{"_account_id":23163,"name":"Markos Chandras (hwoarang)","email":"markos@chandras.me","username":"mchandras"},"change_message_id":"c6e9482b93fd45e7c88bc95c490ba8d6d0d4f151","unresolved":false,"context_lines":[{"line_number":107,"context_line":"  tags:"},{"line_number":108,"context_line":"    - add-external-repo"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"- name: Install distro packages"},{"line_number":111,"context_line":"  package:"},{"line_number":112,"context_line":"    name: \"{{ pip_install_distro_packages }}\""},{"line_number":113,"context_line":"    state: \"{{ pip_install_package_state }}\""},{"line_number":114,"context_line":"  tags:"},{"line_number":115,"context_line":"    - add-repo-keys"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f287b81_1773beb3","line":115,"range":{"start_line":110,"start_character":0,"end_line":115,"end_character":19},"updated":"2017-08-31 17:37:47.000000000","message":"btw this fails if the the pip_install_distro_packages is empty (which is for SUSE at the moment) and that\u0027s why the CI job is failing","commit_id":"6f61dd8a6b4d7377967ab199c7220339f52335b2"}],"vars/redhat-7.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"77a22e02c892070a9159e62d8eb8bd6bce022765","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"pip_install_external_repo_key_package: \"https://repos.fedorapeople.org/repos/openstack/openstack-ocata/rdo-release-ocata.rpm\""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"pip_install_distro_packages:"},{"line_number":19,"context_line":"  - pyOpenSSL"},{"line_number":20,"context_line":"  - python-ndg_httpsclient"},{"line_number":21,"context_line":"  - python2-pyasn1"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_9c20dc9d","line":18,"updated":"2017-09-01 15:10:37.000000000","message":"Will we have no issue with undefined pip_install_distro_build_packages and pip_required_pip_packages ?","commit_id":"c22d2ed765302d0078c1c475e97b9c6efd9e5760"},{"author":{"_account_id":13095,"name":"Marc Gariépy","email":"gariepy.marc@gmail.com","username":"mgariepy"},"change_message_id":"1046122bbe5545d57e66147e4851d32b8a8b745f","unresolved":false,"context_lines":[{"line_number":21,"context_line":"  - python2-pyasn1"},{"line_number":22,"context_line":"  - python2-pyasn1-modules"},{"line_number":23,"context_line":"  - python2-requests"},{"line_number":24,"context_line":"  - python2-urllib3"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"pip_epel_prep_distro_packages:"},{"line_number":27,"context_line":"  - epel-release"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7f287b81_bcecb829","line":24,"updated":"2017-09-01 15:16:13.000000000","message":"need to keep in mind that pip pkg might be present on the system. so we maybe need to uninstall them ?","commit_id":"679e22563d565dcbdcefec053d7f87de12b78214"}],"vars/ubuntu-16.04.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"77a22e02c892070a9159e62d8eb8bd6bce022765","unresolved":false,"context_lines":[{"line_number":19,"context_line":"uca_repo: \"deb {{ uca_apt_repo_url }} {{ uca_repo_dist }} main\""},{"line_number":20,"context_line":"pip_install_external_repo_key_package: \"ubuntu-cloud-keyring\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"pip_install_distro_packages:"},{"line_number":23,"context_line":"  - python-ndg-httpsclient"},{"line_number":24,"context_line":"  - python-openssl"},{"line_number":25,"context_line":"  - python3-openssl"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f287b81_1c34ecdd","line":22,"updated":"2017-09-01 15:10:37.000000000","message":"see other remarks","commit_id":"c22d2ed765302d0078c1c475e97b9c6efd9e5760"}]}
