)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28022,"name":"Bharat Kunwar","email":"brtknr@bath.edu","username":"brtknr"},"change_message_id":"c58a0978785a6278b4e9c62861252a8a8aa146ff","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"k8s: Use the same kubectl version as API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the heat-agent we use kubeclt to install"},{"line_number":10,"context_line":"several deployments, it is better if we use"},{"line_number":11,"context_line":"matching versions of kubectl and apiserver"},{"line_number":12,"context_line":"to minimize errors. Additionally, the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f493fa4_32c5915c","line":9,"range":{"start_line":9,"start_character":25,"end_line":9,"end_character":32},"updated":"2020-04-23 13:47:37.000000000","message":"kubectl","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"}],"magnum/drivers/common/templates/kubernetes/fragments/configure-kubernetes-master.sh":[{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"06187e6052fe9e52b3898f3a375f0f9e400b9dd8","unresolved":false,"context_lines":[{"line_number":87,"context_line":"EnvironmentFile\u003d/etc/kubernetes/apiserver"},{"line_number":88,"context_line":"ExecStartPre\u003d/bin/mkdir -p /etc/kubernetes/"},{"line_number":89,"context_line":"ExecStartPre\u003d-/usr/bin/podman rm kube-apiserver"},{"line_number":90,"context_line":"ExecStartPre\u003d-/bin/bash -c \u0027/usr/bin/podman run --privileged --user root --net host --entrypoint /bin/bash --rm --volume /usr/local/bin:/host/usr/local/bin \\${CONTAINER_INFRA_PREFIX:-k8s.gcr.io/}hyperkube:\\${KUBE_TAG} -c \"cp /usr/local/bin/kubectl /host/usr/local/bin/kubectl\"\u0027"},{"line_number":91,"context_line":"ExecStart\u003d/bin/bash -c \u0027/usr/bin/podman run --name kube-apiserver \\\\"},{"line_number":92,"context_line":"    --net host \\\\"},{"line_number":93,"context_line":"    --entrypoint /hyperkube \\\\"}],"source_content_type":"text/x-sh","patch_set":4,"id":"1f493fa4_fae3aed7","side":"PARENT","line":90,"updated":"2020-04-27 22:40:15.000000000","message":"In this way, it seems we\u0027re also using the same version of the kube-apiserver. So could you please help me understand why do we need a separate script to install the kubectl?","commit_id":"5b10eb7001623bc5cf2e82820d9e5dd364c6f064"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"ffd2ea2a11fcaba6c1fabec691163e0ad2e382c8","unresolved":false,"context_lines":[{"line_number":87,"context_line":"EnvironmentFile\u003d/etc/kubernetes/apiserver"},{"line_number":88,"context_line":"ExecStartPre\u003d/bin/mkdir -p /etc/kubernetes/"},{"line_number":89,"context_line":"ExecStartPre\u003d-/usr/bin/podman rm kube-apiserver"},{"line_number":90,"context_line":"ExecStartPre\u003d-/bin/bash -c \u0027/usr/bin/podman run --privileged --user root --net host --entrypoint /bin/bash --rm --volume /usr/local/bin:/host/usr/local/bin \\${CONTAINER_INFRA_PREFIX:-k8s.gcr.io/}hyperkube:\\${KUBE_TAG} -c \"cp /usr/local/bin/kubectl /host/usr/local/bin/kubectl\"\u0027"},{"line_number":91,"context_line":"ExecStart\u003d/bin/bash -c \u0027/usr/bin/podman run --name kube-apiserver \\\\"},{"line_number":92,"context_line":"    --net host \\\\"},{"line_number":93,"context_line":"    --entrypoint /hyperkube \\\\"}],"source_content_type":"text/x-sh","patch_set":4,"id":"1f493fa4_f8a57148","side":"PARENT","line":90,"in_reply_to":"1f493fa4_f84291aa","updated":"2020-04-28 08:43:42.000000000","message":"relaying Bharat\u0027s comment to have the discussion in one place.\n\n \u003e We should consider doing a similar thing after cluster upgrades to\n \u003e ensure that the kubernetes client binary is replaced with the new\n \u003e version too but if kubectl is only required for heat-container-agent\n \u003e bootstrapping, ignore my comment.\n\nkubectl is required (check [0]) and would break because it references /usr/local/bin explicitly. We either must change the absolute bath to just kubectl to be taken from $PATH or continue doing it with the current way and change only the agent as mentioned before. We can use the install kubectl script in the upgrade software deployment too.\n\nAn argument that intalling kubectl together with the update of kubelet container could be made, but has little ground since upgrades are only possible between releases that are not too far away one from another.\n\nhttps://github.com/openstack/magnum/blob/master/magnum/drivers/common/templates/kubernetes/fragments/upgrade-kubernetes.sh","commit_id":"5b10eb7001623bc5cf2e82820d9e5dd364c6f064"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"4a4ebe739c25b9fb0c7cd6af402054552fd5cf3b","unresolved":false,"context_lines":[{"line_number":87,"context_line":"EnvironmentFile\u003d/etc/kubernetes/apiserver"},{"line_number":88,"context_line":"ExecStartPre\u003d/bin/mkdir -p /etc/kubernetes/"},{"line_number":89,"context_line":"ExecStartPre\u003d-/usr/bin/podman rm kube-apiserver"},{"line_number":90,"context_line":"ExecStartPre\u003d-/bin/bash -c \u0027/usr/bin/podman run --privileged --user root --net host --entrypoint /bin/bash --rm --volume /usr/local/bin:/host/usr/local/bin \\${CONTAINER_INFRA_PREFIX:-k8s.gcr.io/}hyperkube:\\${KUBE_TAG} -c \"cp /usr/local/bin/kubectl /host/usr/local/bin/kubectl\"\u0027"},{"line_number":91,"context_line":"ExecStart\u003d/bin/bash -c \u0027/usr/bin/podman run --name kube-apiserver \\\\"},{"line_number":92,"context_line":"    --net host \\\\"},{"line_number":93,"context_line":"    --entrypoint /hyperkube \\\\"}],"source_content_type":"text/x-sh","patch_set":4,"id":"1f493fa4_f84291aa","side":"PARENT","line":90,"in_reply_to":"1f493fa4_fae3aed7","updated":"2020-04-28 08:27:49.000000000","message":"For info:\nThis systemd unit is enabled here [0][1][2], which is ok in ordering.\n\n1. This was in-place here to have the same functionality as the system containers which was coying to /usr/local/bin. /usr/local/bin is not mounted in the heat-agent and can\u0027t be mounted in the agents /usr/local/bin or /usr/bin because other files exist there. It should be in a non-standard path as proposed in this patch. It is probably ok to mount the host\u0027s /usr/local/bin to a non-standard path in the agent and add it to the PATH.\n\nTo make point 1., using this way to get kubectl would require either to change the heat-agent systemd unit to mount the host\u0027s /usr/local/bin, or change all references of kubectl to execute over ssh in the host\u0027s shell which would also need a valid kubeconfig on the host (my other patch fixed this).\n\n2. This way of installing kubectl is more like a side effect of starting the agent. Which required that kubectl will always exists in the kube-apiserver image. We could use the kubelet systemd unit, yet a side effect executed on *every* restart of kubelet/kube-apiserver. With what is proposed in this patch we explicitly check if kubectl is set properly. (For info, the current state is use kubectl embedded in the agent which is 1.15.2 and hard to track kubectl version in the agent)\n\n3. We have the same thing in two places. This patch fixes the duplication as well as the idea of changing how the heat-agent starts.\n\nBottom line, we need a change anyway to have a matching kubectl with the API used from the heat agent. \n\n\n[0] https://github.com/openstack/magnum/blob/master/magnum/drivers/k8s_fedora_coreos_v1/templates/kubemaster.yaml#L852\n[1] https://github.com/openstack/magnum/blob/master/magnum/drivers/common/templates/kubernetes/fragments/enable-services-master.sh#L25","commit_id":"5b10eb7001623bc5cf2e82820d9e5dd364c6f064"}],"magnum/drivers/common/templates/kubernetes/fragments/install-clients.sh":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_1204d549","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_128975c9","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_5293fdfa","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_52fe5d37","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_929585e7","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_92be2563","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_d29f0d01","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_d2b8ad77","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_d2ea6d73","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E002 Tab indents","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_328e31d0","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_32f9114d","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_7298b91a","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_72c1d9e4","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_b2a2414a","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_b2bb6170","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_b2ed218d","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_f28329a8","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_f2fe8936","line":23,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ]"},{"line_number":28,"context_line":"    then"},{"line_number":29,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_12b21595","line":26,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E043 Arithmetic compound has inconsistent return semantics","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8af562f035ad29b5257a71b979444bf9843fb4d3","unresolved":false,"context_lines":[{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ]"},{"line_number":28,"context_line":"    then"},{"line_number":29,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""},{"line_number":30,"context_line":"        exit 1"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1f493fa4_f2acc936","line":27,"updated":"2020-04-23 14:02:43.000000000","message":"pep8: E011 Then keyword is not on same line as if or elif keyword","commit_id":"0ca9e0fa248ac4b0c9f74ec11635331d52f81473"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_1a07b584","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_1ade9510","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_5a013d6b","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_5ab3fdea","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_5ae81db3","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_9acc656b","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_9ad105ff","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_dac6ed48","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_dadb8d18","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E002 Tab indents","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_3a04317b","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_3ab6f1da","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_3aeb91b0","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_7ac7d946","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_7accf960","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_bac16131","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_bad681f0","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_fa092995","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":20,"context_line":"\t--user root \\"},{"line_number":21,"context_line":"\t--volume /srv/magnum/bin:/host/srv/magnum/bin \\"},{"line_number":22,"context_line":"\t${hyperkube_image} \\"},{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_fae009cf","line":23,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E003 Indent not multiple of 4","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\t-c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ]"},{"line_number":28,"context_line":"    then"},{"line_number":29,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_9afaa572","line":26,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E043 Arithmetic compound has inconsistent return semantics","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"374d4620b903c89d592efc0b3f0047e1a86e0e88","unresolved":false,"context_lines":[{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    ((i\u003d${i}+1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ]"},{"line_number":28,"context_line":"    then"},{"line_number":29,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""},{"line_number":30,"context_line":"        exit 1"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1f493fa4_7af51983","line":27,"updated":"2020-04-24 11:02:50.000000000","message":"pep8: E011 Then keyword is not on same line as if or elif keyword","commit_id":"6a9760bd90406b0afd2f7eef32d60cc116312aa7"},{"author":{"_account_id":28022,"name":"Bharat Kunwar","email":"brtknr@bath.edu","username":"brtknr"},"change_message_id":"8f9f0b7e6054311cf90662a80470bb3794d6610a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    ${hyperkube_image} \\"},{"line_number":23,"context_line":"    -c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    i\u003d$((${i} + 1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ] ; then"},{"line_number":28,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"1f493fa4_7d9eb354","line":25,"range":{"start_line":25,"start_character":4,"end_line":25,"end_character":83},"updated":"2020-04-24 12:01:33.000000000","message":"nit: makes sense to put this after the fi before sleep.","commit_id":"44e7a61311973e33a0803c23a25886a7e7282648"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"a8364c5607716a50bd6312194c8ebd09799d0eff","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    ${hyperkube_image} \\"},{"line_number":23,"context_line":"    -c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    i\u003d$((${i} + 1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ] ; then"},{"line_number":28,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"1f493fa4_dd50a760","line":25,"range":{"start_line":25,"start_character":4,"end_line":25,"end_character":83},"in_reply_to":"1f493fa4_7d9eb354","updated":"2020-04-24 12:22:21.000000000","message":"it makes sense to have it as it is for zero based numbering.\n\nSince I\u0027m not doing 1 based this is ok. I could move the increment lower and do -gt 59 -ge 60. But it wouldn\u0027t make any difference.\n\nwhat you said makes sense for starting from 1.","commit_id":"44e7a61311973e33a0803c23a25886a7e7282648"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"c7a8b386b25a58e873bd9b3f1c8ea624178f3b9e","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    ${hyperkube_image} \\"},{"line_number":23,"context_line":"    -c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    i\u003d$((${i} + 1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ] ; then"},{"line_number":28,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"1f493fa4_66b3c4eb","line":25,"range":{"start_line":25,"start_character":4,"end_line":25,"end_character":83},"in_reply_to":"1f493fa4_809358a3","updated":"2020-04-24 14:06:47.000000000","message":"True.\n\nSo we must print the attempt index in the ERROR meesage too. Otherwise, it won\u0027t print in which attempt we are.\n\nAlso we don\u0027t show anything in the case of success we need that too.\n\nI will also print WARNING instread.","commit_id":"44e7a61311973e33a0803c23a25886a7e7282648"},{"author":{"_account_id":28022,"name":"Bharat Kunwar","email":"brtknr@bath.edu","username":"brtknr"},"change_message_id":"16e06640d231c05cd9fbb43cc0af68410559a196","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    ${hyperkube_image} \\"},{"line_number":23,"context_line":"    -c \u0027cp /usr/local/bin/kubectl /host/srv/magnum/bin/kubectl\u0027\""},{"line_number":24,"context_line":"do"},{"line_number":25,"context_line":"    echo \"INFO Attempt ${i}: Trying to install kubectl from hyperkube. Sleeping 5s\""},{"line_number":26,"context_line":"    i\u003d$((${i} + 1))"},{"line_number":27,"context_line":"    if [ ${i} -gt 60 ] ; then"},{"line_number":28,"context_line":"        echo \"ERROR Unable to install kubectl. Abort.\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"1f493fa4_809358a3","line":25,"range":{"start_line":25,"start_character":4,"end_line":25,"end_character":83},"in_reply_to":"1f493fa4_dd50a760","updated":"2020-04-24 12:39:56.000000000","message":"I was only talking about the case where Sleeping 5s doesn\u0027t happen for the last loop. \u003d)","commit_id":"44e7a61311973e33a0803c23a25886a7e7282648"}]}