)]}'
{"id":"openstack%2Fopenstack-ansible-os_placement~756578","triplet_id":"openstack%2Fopenstack-ansible-os_placement~master~I239d7e843b86f92d7caf04a28f6f7bc43f086984","project":"openstack/openstack-ansible-os_placement","branch":"master","topic":"install_git","hashtags":[],"change_id":"I239d7e843b86f92d7caf04a28f6f7bc43f086984","subject":"Ensure git package is installed","status":"ABANDONED","created":"2020-10-07 19:39:24.000000000","updated":"2020-10-09 00:58:12.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9916850c15d34535084e523c3c0ac6b9e8529ca0","_number":756578,"virtual_id_number":756578,"owner":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2020-10-07 21:53:01.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"date":"2020-10-09 00:58:08.000000000","_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"disliked":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2020-10-07 20:45:39.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},{"value":2,"date":"2020-10-07 20:41:39.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-10-07 20:41:39.000000000","updated_by":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"reviewer":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"state":"REVIEWER"},{"updated":"2020-10-07 20:45:39.000000000","updated_by":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"reviewer":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"state":"REVIEWER"},{"updated":"2020-10-07 21:53:01.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"6fc47ca7ef6c0cf17ba9ab5eb1c89f5bd362193e","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2020-10-07 19:39:24.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"590e80b3a1e826171f99ceacd00166e45de23fa9","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2020-10-07 19:40:03.000000000","message":"Patch Set 1:\n\nCurrently getting this error during deployments\n\nERROR: Error [Errno 2] No such file or directory: \u0027git\u0027: \u0027git\u0027 while executing command git clone -q https://opendev.org/openstack/placement /tmp/pip-install-bcu6rh41/openstack-placement\\nERROR: Cannot find command \u0027git\u0027 - do you have \u0027git\u0027 installed and in your PATH?\\nWARNING: You are using pip version 19.3; however, version 20.2.3 is available.\\nYou should consider upgrading via the \u0027pip install --upgrade pip\u0027 command.\\n\"}","accounts_in_message":[],"_revision_number":1},{"id":"a5498d0ba744a3fb91d1cc9684a5111459ea4bb9","author":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"date":"2020-10-07 20:41:39.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"376b37bd688d0160b411eaa033343d3ba55d0673","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2020-10-07 20:45:39.000000000","message":"Patch Set 1: Code-Review-1\n\nCould use a bit more detail because i\u0027m not sure this is happening in CI? The git clone should be done in the repo_server container/host and that should already have git installed from here https://github.com/openstack/openstack-ansible-repo_server/blob/master/vars/debian.yml#L23","accounts_in_message":[],"_revision_number":1},{"id":"1b4d0acb930bec8e29597e6303c7c2d5ffce622a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-07 21:53:01.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-ansible-linters https://zuul.opendev.org/t/openstack/build/23459beba54b415f80e26435113f0fed : SUCCESS in 8m 16s\n- openstack-ansible-deploy-aio_metal-debian-buster https://zuul.opendev.org/t/openstack/build/921cc9e9cfc445e0aaa0becd3623d064 : SUCCESS in 1h 03m 50s\n- openstack-ansible-deploy-aio_metal-centos-8 https://zuul.opendev.org/t/openstack/build/7631de87813b4d679d8b221241d03f5c : SUCCESS in 1h 22m 07s\n- openstack-ansible-deploy-aio_metal-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/4b354b13c1c844f7b63a8cc8fdc77bc9 : SUCCESS in 1h 15m 06s\n- openstack-ansible-deploy-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/f5ec270f41c74628837d40d4219c87fd : SUCCESS in 1h 05m 30s\n- openstack-ansible-upgrade-aio_metal-centos-8 https://zuul.opendev.org/t/openstack/build/b4952ed5b18448419b9150710b8c2489 : SUCCESS in 2h 01m 03s (non-voting)\n- openstack-ansible-upgrade-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/69a2f1018348465c9e9180741ff76b60 : SUCCESS in 1h 38m 53s\n- openstack-ansible-deploy-aio_distro_metal-centos-8 https://zuul.opendev.org/t/openstack/build/ab921a74e4d94ae38eb1b0324499c91f : SUCCESS in 1h 10m 32s\n- openstack-ansible-deploy-aio_distro_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/442b50b79c67487f80211483c5de140f : SUCCESS in 1h 05m 37s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c9ea8093bc184eddb30f2c6047fcafdd : SUCCESS in 5m 27s","accounts_in_message":[],"_revision_number":1},{"id":"73d5b93faf7671a13fad40cdbf48ee6045ba0b56","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2020-10-07 22:52:32.000000000","message":"Patch Set 1:\n\nOh, ok that must be the difference then.\nIn my deployments, I\u0027m not using a dedicated repo-server, the venvs are being built on each individual host.\n\nhttps://github.com/openstack/ansible-role-python_venv_build/blob/master/vars/main.yml#L49","accounts_in_message":[],"_revision_number":1},{"id":"07099d9b79d6db3e5f61a81fdf4e4334407b6a79","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2020-10-08 07:29:32.000000000","message":"Patch Set 1:\n\nAh that makes sense - you could see if adding \u0027git\u0027 here sorts it out for all hosts https://github.com/openstack/ansible-role-python_venv_build/blob/master/vars/main.yml#L16","accounts_in_message":[],"_revision_number":1},{"id":"cd8bdfcb64352dbbfbd2a351f12c46bf1a1f3551","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2020-10-09 00:58:08.000000000","message":"Patch Set 1:\n\nTested that today. It works and seems like a much better place to fix this issue.\nCreated https://review.opendev.org/#/c/756939/ instead.","accounts_in_message":[],"_revision_number":1},{"id":"7096c33f7fce74568a74a71e2f97961177508997","author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"date":"2020-10-09 00:58:12.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"0a24a912be273bc17e5eb4d4f57ef23bd0b23890","revisions":{"0a24a912be273bc17e5eb4d4f57ef23bd0b23890":{"kind":"REWORK","_number":1,"created":"2020-10-07 19:39:24.000000000","uploader":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"ref":"refs/changes/78/756578/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-os_placement","ref":"refs/changes/78/756578/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_placement refs/changes/78/756578/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_placement refs/changes/78/756578/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_placement refs/changes/78/756578/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-os_placement refs/changes/78/756578/1"}}},"commit":{"parents":[{"commit":"3dde9765af0afb66aa32718962fd6419acfd06e6","subject":"Merge \"Remove support for lxc2 config keys\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_placement/commit/3dde9765af0afb66aa32718962fd6419acfd06e6"}]}],"author":{"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","date":"2020-10-07 19:34:50.000000000","tz":-420},"committer":{"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","date":"2020-10-07 19:39:23.000000000","tz":-420},"subject":"Ensure git package is installed","message":"Ensure git package is installed\n\ngit is required to clone the placement repo during source based\ninstallations.\n\nChange-Id: I239d7e843b86f92d7caf04a28f6f7bc43f086984\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_placement/commit/0a24a912be273bc17e5eb4d4f57ef23bd0b23890"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_placement/commit/0a24a912be273bc17e5eb4d4f57ef23bd0b23890"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
