)]}'
{"id":"openstack%2Fopenstack-helm~551039","triplet_id":"openstack%2Fopenstack-helm~master~I2c6bda226db2b45b448ca5cd4d7e507cb06adab5","project":"openstack/openstack-helm","branch":"master","topic":"threading","hashtags":[],"change_id":"I2c6bda226db2b45b448ca5cd4d7e507cb06adab5","subject":"Neutron: run agents as bash child processes","status":"ABANDONED","created":"2018-03-09 00:25:30.000000000","updated":"2018-06-16 19:13:41.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"419480a323010305965e568f74e89b73bc54e773","_number":551039,"virtual_id_number":551039,"owner":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"date":"2018-03-13 05:32:01.000000000","_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},{"date":"2018-06-16 19:13:38.000000000","_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":1,"date":"2018-03-12 16:35:16.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-03-10 22:39:14.000000000","updated_by":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"reviewer":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"state":"REVIEWER"},{"updated":"2018-03-12 16:35:16.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"},{"updated":"2018-03-13 05:32:01.000000000","updated_by":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"reviewer":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"state":"REVIEWER"}],"messages":[{"id":"67bfaf6c3478098f7013f4c1cf36edfb1cbf46af","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-09 00:25:30.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"806b26b536a6f896c30d8824c12ca7ce6a295eca","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-09 02:44:49.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/39/551039/1/check/build-openstack-sphinx-docs/4f207dc/html/ : SUCCESS in 3m 39s\n- openstack-helm-linter http://logs.openstack.org/39/551039/1/check/openstack-helm-linter/cfcff4a/ : SUCCESS in 1m 43s\n- openstack-helm-dev-deploy-ceph-ovs http://logs.openstack.org/39/551039/1/check/openstack-helm-dev-deploy-ceph-ovs/d2db85b/ : SUCCESS in 57m 50s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/39/551039/1/check/openstack-helm-dev-deploy-nfs-ovs/8a00ee9/ : SUCCESS in 59m 07s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/39/551039/1/check/openstack-helm-dev-deploy-ceph-lb/4a3ddc6/ : SUCCESS in 1h 01m 36s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/39/551039/1/check/openstack-helm-dev-deploy-nfs-lb/b47c139/ : SUCCESS in 52m 47s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/39/551039/1/check/openstack-helm-multinode-ubuntu/3c746ae/ : SUCCESS in 1h 54m 29s\n- openstack-helm-multinode-centos http://logs.openstack.org/39/551039/1/check/openstack-helm-multinode-centos/668e7c2/ : FAILURE in 56m 13s (non-voting)\n- openstack-helm-multinode-fedora http://logs.openstack.org/39/551039/1/check/openstack-helm-multinode-fedora/ac2ff59/ : SUCCESS in 1h 49m 00s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/39/551039/1/check/openstack-helm-ironic-ubuntu/34d0933/ : SUCCESS in 1h 13m 10s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/39/551039/1/check/openstack-helm-armada-fullstack-deploy/3aa7bd2/ : SUCCESS in 2h 14m 16s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/39/551039/1/check/openstack-helm-kolla-deploy-ceph-ovs/f9e81dd/ : SUCCESS in 1h 01m 02s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/39/551039/1/check/openstack-helm-ocata-loci-deploy-ceph/a290b2e/ : SUCCESS in 1h 02m 00s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/39/551039/1/check/openstack-helm-ocata-kolla-deploy-ceph/ff5d0aa/ : SUCCESS in 1h 00m 26s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"70107af41fd5fbc38e9d2ca2db1d7f18efb87daa","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-09 06:40:32.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"922465a3f0c1465ff6de4962d44964981a6d48e9","author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"date":"2018-03-09 08:53:28.000000000","message":"Patch Set 2:\n\nare you sure? as far as I know, bash is not considered as a good init process for the possibility that its child processes might be terminated not cleanly. I have to catch up with the recent status of sharing pid namespace in k8s but using \"pause\" as init process looks good to me.","accounts_in_message":[],"_revision_number":2},{"id":"e8e1bf6b94575cf55503d0657c50e5fb9778efc0","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-09 09:02:40.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/39/551039/2/check/build-openstack-sphinx-docs/14e66d0/html/ : SUCCESS in 3m 26s\n- openstack-helm-linter http://logs.openstack.org/39/551039/2/check/openstack-helm-linter/0c1d47b/ : SUCCESS in 1m 08s\n- openstack-helm-dev-deploy-ceph-ovs http://logs.openstack.org/39/551039/2/check/openstack-helm-dev-deploy-ceph-ovs/e563087/ : FAILURE in 46m 37s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/39/551039/2/check/openstack-helm-dev-deploy-nfs-ovs/03abe54/ : SUCCESS in 49m 30s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/39/551039/2/check/openstack-helm-dev-deploy-ceph-lb/0f0a05b/ : SUCCESS in 56m 32s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/39/551039/2/check/openstack-helm-dev-deploy-nfs-lb/e6c9408/ : SUCCESS in 50m 53s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/39/551039/2/check/openstack-helm-multinode-ubuntu/0fbb2ab/ : SUCCESS in 1h 49m 13s\n- openstack-helm-multinode-centos http://logs.openstack.org/39/551039/2/check/openstack-helm-multinode-centos/1038af9/ : FAILURE in 56m 31s (non-voting)\n- openstack-helm-multinode-fedora http://logs.openstack.org/39/551039/2/check/openstack-helm-multinode-fedora/16b465c/ : SUCCESS in 1h 53m 57s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/39/551039/2/check/openstack-helm-ironic-ubuntu/762e207/ : SUCCESS in 1h 13m 06s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/39/551039/2/check/openstack-helm-armada-fullstack-deploy/e45759f/ : SUCCESS in 2h 20m 18s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/39/551039/2/check/openstack-helm-kolla-deploy-ceph-ovs/9108b8e/ : SUCCESS in 59m 01s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/39/551039/2/check/openstack-helm-ocata-loci-deploy-ceph/8cbb25f/ : SUCCESS in 1h 02m 00s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/39/551039/2/check/openstack-helm-ocata-kolla-deploy-ceph/d546d86/ : SUCCESS in 57m 30s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"847683f8ac88e8e7d0ec6d15c0613469bed1e3b1","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-09 16:07:28.000000000","message":"Patch Set 3: Patch Set 2 was rebased","accounts_in_message":[],"_revision_number":3},{"id":"d9d4bbd09ba3aa23040fc8453c5413118ead4857","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-09 18:34:35.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/39/551039/3/check/build-openstack-sphinx-docs/280a460/html/ : SUCCESS in 2m 57s\n- openstack-helm-linter http://logs.openstack.org/39/551039/3/check/openstack-helm-linter/2a70c07/ : SUCCESS in 51s\n- openstack-helm-dev-deploy-ceph-ovs http://logs.openstack.org/39/551039/3/check/openstack-helm-dev-deploy-ceph-ovs/ff1b172/ : SUCCESS in 57m 53s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/39/551039/3/check/openstack-helm-dev-deploy-nfs-ovs/4e9bc62/ : SUCCESS in 50m 45s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/39/551039/3/check/openstack-helm-dev-deploy-ceph-lb/0f1cd13/ : SUCCESS in 57m 27s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/39/551039/3/check/openstack-helm-dev-deploy-nfs-lb/72e06a8/ : SUCCESS in 48m 42s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/39/551039/3/check/openstack-helm-multinode-ubuntu/c06b15f/ : SUCCESS in 1h 46m 30s\n- openstack-helm-multinode-centos http://logs.openstack.org/39/551039/3/check/openstack-helm-multinode-centos/e655e64/ : FAILURE in 54m 40s (non-voting)\n- openstack-helm-multinode-fedora http://logs.openstack.org/39/551039/3/check/openstack-helm-multinode-fedora/91fa271/ : SUCCESS in 1h 56m 05s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/39/551039/3/check/openstack-helm-ironic-ubuntu/3d81337/ : SUCCESS in 1h 06m 27s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/39/551039/3/check/openstack-helm-armada-fullstack-deploy/0c667e2/ : SUCCESS in 2h 19m 51s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/39/551039/3/check/openstack-helm-kolla-deploy-ceph-ovs/604cd0f/ : SUCCESS in 58m 53s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/39/551039/3/check/openstack-helm-ocata-loci-deploy-ceph/feb0817/ : SUCCESS in 1h 00m 41s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/39/551039/3/check/openstack-helm-ocata-kolla-deploy-ceph/c400a7b/ : SUCCESS in 58m 32s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"424a116dfb1e1c977e986f6f4c2e0c455100860b","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-10 11:23:55.000000000","message":"Patch Set 3:\n\n\u003e are you sure? as far as I know, bash is not considered as a good\n \u003e init process for the possibility that its child processes might be\n \u003e terminated not cleanly. I have to catch up with the recent status\n \u003e of sharing pid namespace in k8s but using \"pause\" as init process\n \u003e looks good to me.\n\nIt\u0027s not good, but its much better than what we have today (nothing), if you could address this in a cleaner way it would be really apprecated.","accounts_in_message":[],"_revision_number":3},{"id":"c5fd0ff704c73a80edb90b460351e75ee23fc277","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2018-03-10 12:04:32.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"54667b85b0f5c1cc8a041b651b9fd3c1f863bffd","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-10 13:09:33.000000000","message":"Patch Set 3: Workflow-1\n\nI\u0027m gonna tidy this up a bit, and cleanly shutdown the processes.","accounts_in_message":[],"_revision_number":3},{"id":"0069f4595c0945ab8479b359ce23df3b43dbf091","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-10 13:09:37.000000000","message":"Patch Set 3: Code-Review-1","accounts_in_message":[],"_revision_number":3},{"id":"c4e346bc93eaa32e7aef631cccae165003adfe6c","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2018-03-10 22:39:14.000000000","message":"Patch Set 3: -Code-Review","accounts_in_message":[],"_revision_number":3},{"id":"9c748e0a6256166f894d2c38eaaf3922d54f31f5","author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"date":"2018-03-12 01:36:57.000000000","message":"Patch Set 3:\n\n\u003e \u003e are you sure? as far as I know, bash is not considered as a good\n \u003e \u003e init process for the possibility that its child processes might\n \u003e be\n \u003e \u003e terminated not cleanly. I have to catch up with the recent status\n \u003e \u003e of sharing pid namespace in k8s but using \"pause\" as init process\n \u003e \u003e looks good to me.\n \u003e \n \u003e It\u0027s not good, but its much better than what we have today\n \u003e (nothing), if you could address this in a cleaner way it would be\n \u003e really apprecated.\n\nsure, I\u0027ll look at it.","accounts_in_message":[],"_revision_number":3},{"id":"4f02c628d4c93cf8c3cb974444e231a62d3ec836","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-12 02:49:06.000000000","message":"Patch Set 3:\n\n\u003e \u003e \u003e are you sure? as far as I know, bash is not considered as a\n \u003e good\n \u003e \u003e \u003e init process for the possibility that its child processes might\n \u003e \u003e be\n \u003e \u003e \u003e terminated not cleanly. I have to catch up with the recent\n \u003e status\n \u003e \u003e \u003e of sharing pid namespace in k8s but using \"pause\" as init\n \u003e process\n \u003e \u003e \u003e looks good to me.\n \u003e \u003e\n \u003e \u003e It\u0027s not good, but its much better than what we have today\n \u003e \u003e (nothing), if you could address this in a cleaner way it would be\n \u003e \u003e really apprecated.\n \u003e \n \u003e sure, I\u0027ll look at it.\n\nhey - I looked into this a bit as well - and have found two paths forward:\n\n1) we just run the services in the host pid namespace, and let the hosts init system do its thing.\n\n2) we  use a lifcycle hook to cleanly terminate the process\n\nI could implement either pretty quick in this ps - any preference for which way we go?","accounts_in_message":[],"_revision_number":3},{"id":"8428431b633a217e171241688cd3469c45d2d141","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-12 03:58:11.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"641d8c076d0a1e8148f476968ec5f36dd3ed4a95","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-12 03:59:44.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"d5f3e1854b844646e6830e7ed6eaa1544defaf45","author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"date":"2018-03-12 04:57:10.000000000","message":"Patch Set 5:\n\n\u003e \u003e \u003e \u003e are you sure? as far as I know, bash is not considered as a\n \u003e \u003e good\n \u003e \u003e \u003e \u003e init process for the possibility that its child processes\n \u003e might\n \u003e \u003e \u003e be\n \u003e \u003e \u003e \u003e terminated not cleanly. I have to catch up with the recent\n \u003e \u003e status\n \u003e \u003e \u003e \u003e of sharing pid namespace in k8s but using \"pause\" as init\n \u003e \u003e process\n \u003e \u003e \u003e \u003e looks good to me.\n \u003e \u003e \u003e\n \u003e \u003e \u003e It\u0027s not good, but its much better than what we have today\n \u003e \u003e \u003e (nothing), if you could address this in a cleaner way it would\n \u003e be\n \u003e \u003e \u003e really apprecated.\n \u003e \u003e\n \u003e \u003e sure, I\u0027ll look at it.\n \u003e \n \u003e hey - I looked into this a bit as well - and have found two paths\n \u003e forward:\n \u003e \n \u003e 1) we just run the services in the host pid namespace, and let the\n \u003e hosts init system do its thing.\n \u003e \n \u003e 2) we  use a lifcycle hook to cleanly terminate the process\n \u003e \n \u003e I could implement either pretty quick in this ps - any preference\n \u003e for which way we go?\n\ngreat. I personally prefer using host pid namespace for now - for the first release. We could consider per-pod configuration of PID namespace sharing later as the feature is merged to k8s recently.","accounts_in_message":[],"_revision_number":5},{"id":"4b858c764f84a60307c923828f0440a845eeb445","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-12 07:01:16.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/39/551039/5/check/build-openstack-sphinx-docs/3bb85d7/html/ : SUCCESS in 3m 49s\n- openstack-helm-linter http://logs.openstack.org/39/551039/5/check/openstack-helm-linter/02bfc55/ : SUCCESS in 1m 09s\n- openstack-helm-dev-deploy-ceph-ovs http://logs.openstack.org/39/551039/5/check/openstack-helm-dev-deploy-ceph-ovs/d7ae8cd/ : FAILURE in 1h 10m 27s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/39/551039/5/check/openstack-helm-dev-deploy-nfs-ovs/516bac8/ : FAILURE in 58m 30s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/39/551039/5/check/openstack-helm-dev-deploy-ceph-lb/931acf8/ : POST_FAILURE in 9m 28s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/39/551039/5/check/openstack-helm-dev-deploy-nfs-lb/f6cd296/ : FAILURE in 1h 02m 03s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/39/551039/5/check/openstack-helm-multinode-ubuntu/05d5a31/ : FAILURE in 1h 30m 06s\n- openstack-helm-multinode-centos http://logs.openstack.org/39/551039/5/check/openstack-helm-multinode-centos/c631688/ : FAILURE in 54m 40s (non-voting)\n- openstack-helm-multinode-fedora http://logs.openstack.org/39/551039/5/check/openstack-helm-multinode-fedora/f4e6bf7/ : FAILURE in 1h 38m 54s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/39/551039/5/check/openstack-helm-ironic-ubuntu/bdb638e/ : FAILURE in 1h 13m 25s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/39/551039/5/check/openstack-helm-armada-fullstack-deploy/8831837/ : TIMED_OUT in 2h 56m 35s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/39/551039/5/check/openstack-helm-kolla-deploy-ceph-ovs/50cae87/ : FAILURE in 1h 24m 31s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/39/551039/5/check/openstack-helm-ocata-loci-deploy-ceph/7243005/ : FAILURE in 1h 22m 21s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/39/551039/5/check/openstack-helm-ocata-kolla-deploy-ceph/eed9003/ : FAILURE in 1h 07m 51s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"f55caf268abc5a13cebde75eb2003d6552c67ead","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-03-12 13:29:54.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"c13264e4e871ae542c54f229192dcbb1bc774638","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-03-12 16:35:16.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/39/551039/6/check/build-openstack-sphinx-docs/0c854e1/html/ : SUCCESS in 3m 29s\n- openstack-helm-linter http://logs.openstack.org/39/551039/6/check/openstack-helm-linter/5ccbc25/ : SUCCESS in 1m 27s\n- openstack-helm-dev-deploy-ceph-ovs http://logs.openstack.org/39/551039/6/check/openstack-helm-dev-deploy-ceph-ovs/1714559/ : SUCCESS in 59m 59s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/39/551039/6/check/openstack-helm-dev-deploy-nfs-ovs/9f7d5eb/ : SUCCESS in 51m 10s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/39/551039/6/check/openstack-helm-dev-deploy-ceph-lb/c93a65f/ : SUCCESS in 58m 32s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/39/551039/6/check/openstack-helm-dev-deploy-nfs-lb/4147c3d/ : SUCCESS in 53m 24s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/39/551039/6/check/openstack-helm-multinode-ubuntu/1c4d322/ : SUCCESS in 1h 53m 34s\n- openstack-helm-multinode-centos http://logs.openstack.org/39/551039/6/check/openstack-helm-multinode-centos/83436d2/ : FAILURE in 57m 42s (non-voting)\n- openstack-helm-multinode-fedora http://logs.openstack.org/39/551039/6/check/openstack-helm-multinode-fedora/6df1910/ : SUCCESS in 1h 51m 00s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/39/551039/6/check/openstack-helm-ironic-ubuntu/aeaa70d/ : FAILURE in 1h 25m 49s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/39/551039/6/check/openstack-helm-armada-fullstack-deploy/7752a86/ : TIMED_OUT in 2h 55m 13s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/39/551039/6/check/openstack-helm-kolla-deploy-ceph-ovs/55bd342/ : SUCCESS in 59m 51s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/39/551039/6/check/openstack-helm-ocata-loci-deploy-ceph/c077d7a/ : SUCCESS in 1h 00m 28s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/39/551039/6/check/openstack-helm-ocata-kolla-deploy-ceph/6e76a9b/ : SUCCESS in 1h 01m 35s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"8bddbe6d48264abfa7e71d69d7d0ae9e5530cd95","author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"date":"2018-03-13 05:32:01.000000000","message":"Patch Set 6:\n\nwhy you prefer lifecycle hook? pid namespace sharing is made for handling exactly the same issue we’re having - zombie process. when k8s 1.10 is released, with support of pid namespace sharing, we could do something like below.\n\n{{ if ge .Capabilities.KubeVersion.Minor “10” }}\nshareProcessNamespace: true\n{{ else }}\nhostPID: true\n{{ end }}\n\nanyway, current patch set should work as well. so, it’s up to you.","accounts_in_message":[],"_revision_number":6},{"id":"757a5d31d796ae1092e2ddd73872003c8814d2c2","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-06-16 19:13:38.000000000","message":"Patch Set 6:\n\n\u003e why you prefer lifecycle hook? pid namespace sharing is made for\n \u003e handling exactly the same issue we’re having - zombie process. when\n \u003e k8s 1.10 is released, with support of pid namespace sharing, we\n \u003e could do something like below.\n \u003e \n \u003e {{ if ge .Capabilities.KubeVersion.Minor “10” }}\n \u003e shareProcessNamespace: true\n \u003e {{ else }}\n \u003e hostPID: true\n \u003e {{ end }}\n \u003e \n \u003e anyway, current patch set should work as well. so, it’s up to you.\n\nThis is cleaner, implemented in: https://review.openstack.org/#/c/575924/, thanks Hyunsun.","accounts_in_message":[],"_revision_number":6},{"id":"645c2f8b30d1afc7b1a11aa8ab30c4667184f25c","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2018-06-16 19:13:41.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"61969842c6738514e8bcd9d7ae74ad44a1d23beb","revisions":{"15e77f1832f1c1b55c264d03419a0b815865490b":{"kind":"REWORK","_number":1,"created":"2018-03-09 00:25:30.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/39/551039/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/39/551039/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/1"}}},"commit":{"parents":[{"commit":"7934f58f2ca80e15500abf7c83348d67708d6f90","subject":"WIP: Reduce the number of workers spawned by services","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/7934f58f2ca80e15500abf7c83348d67708d6f90"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"subject":"WIP: Neutron: run agents as bash child processes","message":"WIP: Neutron: run agents as bash child processes\n\nThis PS moves the neutron agents to run as child processes of bash\nto prevent defunct process sprawl.\n\nChange-Id: I2c6bda226db2b45b448ca5cd4d7e507cb06adab5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/15e77f1832f1c1b55c264d03419a0b815865490b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/15e77f1832f1c1b55c264d03419a0b815865490b"}]},"branch":"refs/heads/master"},"58e8ae1f539a3d16e8058f466c8876abbcc6e942":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2018-03-09 06:40:32.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/39/551039/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/39/551039/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/2"}}},"commit":{"parents":[{"commit":"7934f58f2ca80e15500abf7c83348d67708d6f90","subject":"WIP: Reduce the number of workers spawned by services","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/7934f58f2ca80e15500abf7c83348d67708d6f90"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 06:40:29.000000000","tz":0},"subject":"Neutron: run agents as bash child processes","message":"Neutron: run agents as bash child processes\n\nThis PS moves the neutron agents to run as child processes of bash\nto prevent defunct process sprawl.\n\nChange-Id: I2c6bda226db2b45b448ca5cd4d7e507cb06adab5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/58e8ae1f539a3d16e8058f466c8876abbcc6e942"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/58e8ae1f539a3d16e8058f466c8876abbcc6e942"}]},"branch":"refs/heads/master"},"3676f2a2d0d7a21fd3a85b34389f0bc320dce145":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2018-03-09 16:07:28.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/39/551039/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/39/551039/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/3"}}},"commit":{"parents":[{"commit":"02767f6d76ec04a0849032754c6a0c9248ef3cb3","subject":"Reduce the number of workers spawned by services","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/02767f6d76ec04a0849032754c6a0c9248ef3cb3"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 16:07:28.000000000","tz":0},"subject":"Neutron: run agents as bash child processes","message":"Neutron: run agents as bash child processes\n\nThis PS moves the neutron agents to run as child processes of bash\nto prevent defunct process sprawl.\n\nChange-Id: I2c6bda226db2b45b448ca5cd4d7e507cb06adab5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/3676f2a2d0d7a21fd3a85b34389f0bc320dce145"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/3676f2a2d0d7a21fd3a85b34389f0bc320dce145"}]},"branch":"refs/heads/master"},"88076b357373ed6145b2d873e0cb612eb7240a93":{"kind":"REWORK","_number":4,"created":"2018-03-12 03:58:11.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/39/551039/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/39/551039/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/4"}}},"commit":{"parents":[{"commit":"02767f6d76ec04a0849032754c6a0c9248ef3cb3","subject":"Reduce the number of workers spawned by services","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/02767f6d76ec04a0849032754c6a0c9248ef3cb3"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-12 03:57:53.000000000","tz":0},"subject":"Neutron: run agents as bash child processes","message":"Neutron: run agents as bash child processes\n\nThis PS moves the neutron agents to run as child processes of bash\nto prevent defunct process sprawl.\n\nChange-Id: I2c6bda226db2b45b448ca5cd4d7e507cb06adab5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/88076b357373ed6145b2d873e0cb612eb7240a93"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/88076b357373ed6145b2d873e0cb612eb7240a93"}]},"branch":"refs/heads/master"},"74d58da53b4e39cc4bd073015e0a357dba87b85c":{"kind":"REWORK","_number":5,"created":"2018-03-12 03:59:44.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/39/551039/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/39/551039/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/5"}}},"commit":{"parents":[{"commit":"02767f6d76ec04a0849032754c6a0c9248ef3cb3","subject":"Reduce the number of workers spawned by services","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/02767f6d76ec04a0849032754c6a0c9248ef3cb3"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-12 03:59:32.000000000","tz":0},"subject":"Neutron: run agents as bash child processes","message":"Neutron: run agents as bash child processes\n\nThis PS moves the neutron agents to run as child processes of bash\nto prevent defunct process sprawl.\n\nChange-Id: I2c6bda226db2b45b448ca5cd4d7e507cb06adab5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/74d58da53b4e39cc4bd073015e0a357dba87b85c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/74d58da53b4e39cc4bd073015e0a357dba87b85c"}]},"branch":"refs/heads/master"},"61969842c6738514e8bcd9d7ae74ad44a1d23beb":{"kind":"REWORK","_number":6,"created":"2018-03-12 13:29:54.000000000","uploader":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"ref":"refs/changes/39/551039/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/39/551039/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/39/551039/6"}}},"commit":{"parents":[{"commit":"02767f6d76ec04a0849032754c6a0c9248ef3cb3","subject":"Reduce the number of workers spawned by services","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/02767f6d76ec04a0849032754c6a0c9248ef3cb3"}]}],"author":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-09 00:24:10.000000000","tz":0},"committer":{"name":"Pete Birley","email":"pete@port.direct","date":"2018-03-12 13:29:44.000000000","tz":0},"subject":"Neutron: run agents as bash child processes","message":"Neutron: run agents as bash child processes\n\nThis PS moves the neutron agents to run as child processes of bash\nto prevent defunct process sprawl.\n\nChange-Id: I2c6bda226db2b45b448ca5cd4d7e507cb06adab5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/61969842c6738514e8bcd9d7ae74ad44a1d23beb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/61969842c6738514e8bcd9d7ae74ad44a1d23beb"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}