)]}'
{"kuryr_tempest_plugin/tests/scenario/base.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9c4126b41d8faf43ff03cb7165ccafd4993bd5c4","unresolved":false,"context_lines":[{"line_number":491,"context_line":"                service_name, spec_type\u003dspec_type, namespace\u003dnamespace)"},{"line_number":492,"context_line":"            cls.verify_lbaas_endpoints_configured(service_name, pod_num)"},{"line_number":493,"context_line":"            cls.service_name \u003d service_name"},{"line_number":494,"context_line":"            if not spec_type \u003d\u003d \"ClusterIP\":"},{"line_number":495,"context_line":"                cls.wait_service_status(cls.service_ip,"},{"line_number":496,"context_line":"                                        CONF.kuryr_kubernetes.lb_build_timeout,"},{"line_number":497,"context_line":"                                        protocol, port,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fdfeff1_fe6c1137","line":494,"range":{"start_line":494,"start_character":15,"end_line":494,"end_character":43},"updated":"2019-02-13 07:31:53.000000000","message":"spec_type !\u003d \"ClusterIP\"","commit_id":"94fee881502c6df640367b4bf7c262040a501956"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9c4126b41d8faf43ff03cb7165ccafd4993bd5c4","unresolved":false,"context_lines":[{"line_number":491,"context_line":"                service_name, spec_type\u003dspec_type, namespace\u003dnamespace)"},{"line_number":492,"context_line":"            cls.verify_lbaas_endpoints_configured(service_name, pod_num)"},{"line_number":493,"context_line":"            cls.service_name \u003d service_name"},{"line_number":494,"context_line":"            if not spec_type \u003d\u003d \"ClusterIP\":"},{"line_number":495,"context_line":"                cls.wait_service_status(cls.service_ip,"},{"line_number":496,"context_line":"                                        CONF.kuryr_kubernetes.lb_build_timeout,"},{"line_number":497,"context_line":"                                        protocol, port,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fdfeff1_9e67054e","line":494,"range":{"start_line":494,"start_character":12,"end_line":494,"end_character":14},"updated":"2019-02-13 07:31:53.000000000","message":"we should add something for the else, right?","commit_id":"94fee881502c6df640367b4bf7c262040a501956"},{"author":{"_account_id":21302,"name":"Gena","email":"gcheresh@redhat.com","username":"gcheresh"},"change_message_id":"fbcb51a3a57a6fecd7c0fc8a0a46934f3e2c3639","unresolved":false,"context_lines":[{"line_number":491,"context_line":"                service_name, spec_type\u003dspec_type, namespace\u003dnamespace)"},{"line_number":492,"context_line":"            cls.verify_lbaas_endpoints_configured(service_name, pod_num)"},{"line_number":493,"context_line":"            cls.service_name \u003d service_name"},{"line_number":494,"context_line":"            if not spec_type \u003d\u003d \"ClusterIP\":"},{"line_number":495,"context_line":"                cls.wait_service_status(cls.service_ip,"},{"line_number":496,"context_line":"                                        CONF.kuryr_kubernetes.lb_build_timeout,"},{"line_number":497,"context_line":"                                        protocol, port,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fdfeff1_9a038810","line":494,"range":{"start_line":494,"start_character":12,"end_line":494,"end_character":14},"in_reply_to":"9fdfeff1_9e67054e","updated":"2019-02-13 15:11:23.000000000","message":"Not at the moment, I was thinking to preserve behavior if the type is LB and for ClusterIP type to check in the test itself","commit_id":"94fee881502c6df640367b4bf7c262040a501956"},{"author":{"_account_id":21302,"name":"Gena","email":"gcheresh@redhat.com","username":"gcheresh"},"change_message_id":"fbcb51a3a57a6fecd7c0fc8a0a46934f3e2c3639","unresolved":false,"context_lines":[{"line_number":491,"context_line":"                service_name, spec_type\u003dspec_type, namespace\u003dnamespace)"},{"line_number":492,"context_line":"            cls.verify_lbaas_endpoints_configured(service_name, pod_num)"},{"line_number":493,"context_line":"            cls.service_name \u003d service_name"},{"line_number":494,"context_line":"            if not spec_type \u003d\u003d \"ClusterIP\":"},{"line_number":495,"context_line":"                cls.wait_service_status(cls.service_ip,"},{"line_number":496,"context_line":"                                        CONF.kuryr_kubernetes.lb_build_timeout,"},{"line_number":497,"context_line":"                                        protocol, port,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fdfeff1_5a1a007d","line":494,"range":{"start_line":494,"start_character":15,"end_line":494,"end_character":43},"in_reply_to":"9fdfeff1_fe6c1137","updated":"2019-02-13 15:11:23.000000000","message":"Done","commit_id":"94fee881502c6df640367b4bf7c262040a501956"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"65b533731470c5e93a0e9d81d620ee7eacbb6b07","unresolved":false,"context_lines":[{"line_number":666,"context_line":"    def _run_threaded_and_assert("},{"line_number":667,"context_line":"            self, fn, predicate, repetitions\u003d100, threads\u003d8, fn_timeout\u003d1,"},{"line_number":668,"context_line":"            retry_repetitions\u003d10):"},{"line_number":669,"context_line":"        time.sleep(60)"},{"line_number":670,"context_line":""},{"line_number":671,"context_line":"        tp \u003d pool.ThreadPool(processes\u003dthreads)"},{"line_number":672,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9fdfeff1_cdceceb6","line":669,"range":{"start_line":669,"start_character":7,"end_line":669,"end_character":22},"updated":"2019-03-04 12:03:47.000000000","message":"why sleep here? we should ensure it is ready in a different way. Sleeps on gates (that sometimes are way too slow) may get us some surprises/flakiness","commit_id":"037375db33c03cb2befc9eb9aca2473e8559b094"},{"author":{"_account_id":21302,"name":"Gena","email":"gcheresh@redhat.com","username":"gcheresh"},"change_message_id":"e64b39e8ae5fe8a752376e4f5eba8f4abfe95744","unresolved":false,"context_lines":[{"line_number":666,"context_line":"    def _run_threaded_and_assert("},{"line_number":667,"context_line":"            self, fn, predicate, repetitions\u003d100, threads\u003d8, fn_timeout\u003d1,"},{"line_number":668,"context_line":"            retry_repetitions\u003d10):"},{"line_number":669,"context_line":"        time.sleep(60)"},{"line_number":670,"context_line":""},{"line_number":671,"context_line":"        tp \u003d pool.ThreadPool(processes\u003dthreads)"},{"line_number":672,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9fdfeff1_60d489bc","line":669,"range":{"start_line":669,"start_character":7,"end_line":669,"end_character":22},"in_reply_to":"9fdfeff1_cdceceb6","updated":"2019-03-04 13:19:59.000000000","message":"On my local gate it passed only after adding this sleep, so I wanted to make sure all the tests pass upstream before trying to fix that, but it\u0027s still not the case","commit_id":"037375db33c03cb2befc9eb9aca2473e8559b094"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"b31893cf7bef2c783d19ba68ca4e42d7545fb235","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        try:"},{"line_number":163,"context_line":"            pod \u003d cls.k8s_client.CoreV1Api().read_namespaced_pod(pod_name,"},{"line_number":164,"context_line":"                                                                 namespace)"},{"line_number":165,"context_line":"            return pod.status.host_ip"},{"line_number":166,"context_line":"        except kubernetes.client.rest.ApiException:"},{"line_number":167,"context_line":"            return None"},{"line_number":168,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5fc1f717_1519aa20","line":165,"range":{"start_line":165,"start_character":30,"end_line":165,"end_character":37},"updated":"2019-03-14 08:14:11.000000000","message":"why host_ip?? it should be pod_ip (given the method is call get_pod_ip)","commit_id":"a1b85f62152e00533bb8aa2b8d0aba908edd4e21"},{"author":{"_account_id":21302,"name":"Gena","email":"gcheresh@redhat.com","username":"gcheresh"},"change_message_id":"724d674bd8930cecfa000ae8cec16150de681cc8","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        try:"},{"line_number":163,"context_line":"            pod \u003d cls.k8s_client.CoreV1Api().read_namespaced_pod(pod_name,"},{"line_number":164,"context_line":"                                                                 namespace)"},{"line_number":165,"context_line":"            return pod.status.host_ip"},{"line_number":166,"context_line":"        except kubernetes.client.rest.ApiException:"},{"line_number":167,"context_line":"            return None"},{"line_number":168,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5fc1f717_f5697ef8","line":165,"range":{"start_line":165,"start_character":30,"end_line":165,"end_character":37},"in_reply_to":"5fc1f717_1519aa20","updated":"2019-03-14 08:47:16.000000000","message":"indeed, the copy/paste from function below messed up, thanks","commit_id":"a1b85f62152e00533bb8aa2b8d0aba908edd4e21"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"d111e615432d2899cfdc1bfc5f0a0c39a9630832","unresolved":false,"context_lines":[{"line_number":685,"context_line":"    def _run_threaded_and_assert("},{"line_number":686,"context_line":"            self, fn, predicate, repetitions\u003d100, threads\u003d8, fn_timeout\u003d1,"},{"line_number":687,"context_line":"            retry_repetitions\u003d10):"},{"line_number":688,"context_line":"        time.sleep(300)"},{"line_number":689,"context_line":""},{"line_number":690,"context_line":"        tp \u003d pool.ThreadPool(processes\u003dthreads)"},{"line_number":691,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":8,"id":"ffb9cba7_0fb6948b","line":688,"range":{"start_line":688,"start_character":8,"end_line":688,"end_character":23},"updated":"2019-04-29 12:52:52.000000000","message":"it would be great to avoid time.sleep here","commit_id":"442362f7594b719eb103f7f1784c81de9602628b"},{"author":{"_account_id":21302,"name":"Gena","email":"gcheresh@redhat.com","username":"gcheresh"},"change_message_id":"cede8681f41e55f04aa34b847ee57ff2ced3bdd2","unresolved":false,"context_lines":[{"line_number":685,"context_line":"    def _run_threaded_and_assert("},{"line_number":686,"context_line":"            self, fn, predicate, repetitions\u003d100, threads\u003d8, fn_timeout\u003d1,"},{"line_number":687,"context_line":"            retry_repetitions\u003d10):"},{"line_number":688,"context_line":"        time.sleep(300)"},{"line_number":689,"context_line":""},{"line_number":690,"context_line":"        tp \u003d pool.ThreadPool(processes\u003dthreads)"},{"line_number":691,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":8,"id":"ffb9cba7_d23d83c4","line":688,"range":{"start_line":688,"start_character":8,"end_line":688,"end_character":23},"in_reply_to":"ffb9cba7_0fb6948b","updated":"2019-06-06 05:07:14.000000000","message":"absolutely agree, I put it here as on my environment it solved the problem, but eventually it should be removed of course","commit_id":"442362f7594b719eb103f7f1784c81de9602628b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"023b742d5b4bf8eb14f712019d49c276f063171e","unresolved":false,"context_lines":[{"line_number":273,"context_line":"        # and rest of functions from [1] that takes timeout as parameter are"},{"line_number":274,"context_line":"        # hanging from some reason (on OS select) although the command"},{"line_number":275,"context_line":"        # completed. To resolve that we set the \u0027_request_timeout\u0027 for"},{"line_number":276,"context_line":"        # \u0027connect_get_namespaced_pod_exec\u0027 and f_timeout for the resp"},{"line_number":277,"context_line":"        # of functions."},{"line_number":278,"context_line":"        # [1] https://github.com/kubernetes-client/python-base/blob/master/"},{"line_number":279,"context_line":"        # stream/ws_client.py"}],"source_content_type":"text/x-python","patch_set":16,"id":"9fb8cfa7_0980db45","line":276,"range":{"start_line":276,"start_character":69,"end_line":276,"end_character":70},"updated":"2019-06-25 12:42:43.000000000","message":"?","commit_id":"ca12210bb3c37324389e9222b16729ffe5128cb3"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"b8cc216e6694c9ea9d16b40c580082a8edc4303a","unresolved":false,"context_lines":[{"line_number":273,"context_line":"        # and rest of functions from [1] that takes timeout as parameter are"},{"line_number":274,"context_line":"        # hanging from some reason (on OS select) although the command"},{"line_number":275,"context_line":"        # completed. To resolve that we set the \u0027_request_timeout\u0027 for"},{"line_number":276,"context_line":"        # \u0027connect_get_namespaced_pod_exec\u0027 and f_timeout for the resp"},{"line_number":277,"context_line":"        # of functions."},{"line_number":278,"context_line":"        # [1] https://github.com/kubernetes-client/python-base/blob/master/"},{"line_number":279,"context_line":"        # stream/ws_client.py"}],"source_content_type":"text/x-python","patch_set":16,"id":"9fb8cfa7_bcb45937","line":276,"range":{"start_line":276,"start_character":69,"end_line":276,"end_character":70},"in_reply_to":"9fb8cfa7_0980db45","updated":"2019-06-26 10:20:56.000000000","message":"Done","commit_id":"ca12210bb3c37324389e9222b16729ffe5128cb3"}],"kuryr_tempest_plugin/tests/scenario/test_namespace.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"d111e615432d2899cfdc1bfc5f0a0c39a9630832","unresolved":false,"context_lines":[{"line_number":250,"context_line":"            pod_name, namespace\u003dnamespace_name)"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        # Check connectivity to pod in the namespace from host pod resides on"},{"line_number":253,"context_line":"        self.ping_ip_address(pod_ip)"},{"line_number":254,"context_line":"        # check connectivity from Pod to host pod resides on"},{"line_number":255,"context_line":"        cmd \u003d ["},{"line_number":256,"context_line":"            \"/bin/sh\", \"-c\", \"ping -c 4 {dst_ip}\u003e/dev/null ; echo $?\".format("}],"source_content_type":"text/x-python","patch_set":8,"id":"ffb9cba7_af71880e","line":253,"range":{"start_line":253,"start_character":8,"end_line":253,"end_character":36},"updated":"2019-04-29 12:52:52.000000000","message":"shouldn\u0027t this be executed from the host_ip_of_pod instead of from whatever is running tempest?","commit_id":"442362f7594b719eb103f7f1784c81de9602628b"},{"author":{"_account_id":21302,"name":"Gena","email":"gcheresh@redhat.com","username":"gcheresh"},"change_message_id":"cede8681f41e55f04aa34b847ee57ff2ced3bdd2","unresolved":false,"context_lines":[{"line_number":250,"context_line":"            pod_name, namespace\u003dnamespace_name)"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        # Check connectivity to pod in the namespace from host pod resides on"},{"line_number":253,"context_line":"        self.ping_ip_address(pod_ip)"},{"line_number":254,"context_line":"        # check connectivity from Pod to host pod resides on"},{"line_number":255,"context_line":"        cmd \u003d ["},{"line_number":256,"context_line":"            \"/bin/sh\", \"-c\", \"ping -c 4 {dst_ip}\u003e/dev/null ; echo $?\".format("}],"source_content_type":"text/x-python","patch_set":8,"id":"ffb9cba7_1e9c845b","line":253,"range":{"start_line":253,"start_character":8,"end_line":253,"end_character":36},"in_reply_to":"ffb9cba7_af71880e","updated":"2019-06-06 05:07:14.000000000","message":"You are right, will fix it in the next patch","commit_id":"442362f7594b719eb103f7f1784c81de9602628b"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"54ddb9b95124fa94af84c890be4637df3d000652","unresolved":false,"context_lines":[{"line_number":238,"context_line":"                                         subnet_ns2_name)"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"    @decorators.idempotent_id(\u0027bddd5441-1244-429d-a125-b53ddfb132a9\u0027)"},{"line_number":241,"context_line":"    def test_host_to_namespace_pod_connectivity(self):"},{"line_number":242,"context_line":"        # Create namespace and pod in that namespace"},{"line_number":243,"context_line":"        namespace_name, namespace \u003d self.create_namespace()"},{"line_number":244,"context_line":"        self.addCleanup(self.delete_namespace, namespace_name)"}],"source_content_type":"text/x-python","patch_set":17,"id":"9fb8cfa7_1cd805ed","line":241,"range":{"start_line":241,"start_character":8,"end_line":241,"end_character":47},"updated":"2019-06-26 11:01:09.000000000","message":"wouldn\u0027t it be more appropriate \u0027test_connectivity_between_host_pod_namespace\u0027? since we\u0027re testing connectivity host to/from namespace pod","commit_id":"494f19c526caa04d8d644b2b60b58582c618f3f5"}]}