)]}'
{"id":"openstack%2Fopenstack-helm~789135","triplet_id":"openstack%2Fopenstack-helm~master~I53238d592e1d0f794b7a6c7a619cce232fe9cf9f","project":"openstack/openstack-helm","branch":"master","topic":"aodh_timezone","hashtags":[],"change_id":"I53238d592e1d0f794b7a6c7a619cce232fe9cf9f","subject":"Provide an option about pod\u0027s time for aodh chart","status":"ABANDONED","created":"2021-05-03 07:09:55.000000000","updated":"2021-12-23 08:26:23.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1f1e0b017a7525f7fe4a235739289da31a39538e","_number":789135,"virtual_id_number":789135,"owner":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-06-11 08:21:46.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"}],"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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"}],"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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-05-03 07:11:21.000000000","updated_by":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-05-03 14:38:38.000000000","updated_by":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"reviewer":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"state":"REVIEWER"},{"updated":"2021-05-05 06:25:36.000000000","updated_by":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"reviewer":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"state":"REVIEWER"},{"updated":"2021-05-09 02:29:30.000000000","updated_by":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"reviewer":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"state":"CC"},{"updated":"2021-05-11 15:16:24.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":"2021-05-19 03:54:58.000000000","updated_by":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"reviewer":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"state":"REVIEWER"},{"updated":"2021-05-25 00:50:50.000000000","updated_by":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"reviewer":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"state":"REVIEWER"}],"messages":[{"id":"f60943f5912c6b5ef204af8108000a028921013c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-05-03 07:09:55.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a407f9986cb2dd6c1844402a0552d48522e4bed6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-03 08:08:49.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/27e9c59acfa6470e9923f06b2d1e7cee : SUCCESS in 5m 26s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7254f7eb7a004793a31672a83f841bde : SUCCESS in 3m 49s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/098bebed96f745dbac841c683df267bd : SUCCESS in 4m 53s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/acb0349359e54ba0aa823a3c804b9986 : SUCCESS in 44m 18s\n- openstack-helm-cinder-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/c4f659b838144c96abee214dbf6a404d : SUCCESS in 47m 57s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/acda5a1d072f40e3b9eaefe9eb7baf55 : SUCCESS in 51m 20s\n- openstack-helm-compute-kit-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/d9ad452fb6c743eeaf6432c737e7cce6 : SUCCESS in 53m 21s","accounts_in_message":[],"_revision_number":1},{"id":"14f203db339e472a1cd5506328a5982e65051bb5","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2021-05-03 14:38:38.000000000","message":"Patch Set 1: Code-Review-1\n\nThe container and host should both be using UTC by default, not sure why setting everything to local time is needed?","accounts_in_message":[],"_revision_number":1},{"id":"12fce3baea6ef7780ce413e2af253eb8f8570f8d","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-05-04 01:45:27.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e The container and host should both be using UTC by default, not sure why setting everything to local time is needed?\n\nI believe that more places will change the time zone according to their geographic location. \nFor example, in China, \nwe generally use CST. I believe that other regions have similar problems, which will cause inconsistencies in the container with the host time, and some problems will occur.Especially the time of the log is wrong, which is not conducive to viewing。\n\n We can change the host directly, but the container is not easy to modify the time zone directly, but if we set it to be the same as the host, the user can customize the time zone at will, whether it is the default utc or other time zones are consistent. I feel it is necessary","accounts_in_message":[],"_revision_number":1},{"id":"06bb31b930a737af6422dfd680dbdfa98a8736b3","author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"date":"2021-05-05 06:25:36.000000000","message":"Patch Set 1: Code-Review-1\n\n\u003e Patch Set 1:\n\u003e \n\u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \n\u003e \u003e The container and host should both be using UTC by default, not sure why setting everything to local time is needed?\n\u003e \n\u003e I believe that more places will change the time zone according to their geographic location. \n\u003e For example, in China, \n\u003e we generally use CST. I believe that other regions have similar problems, which will cause inconsistencies in the container with the host time, and some problems will occur.Especially the time of the log is wrong, which is not conducive to viewing。\n\u003e \n\u003e  We can change the host directly, but the container is not easy to modify the time zone directly, but if we set it to be the same as the host, the user can customize the time zone at will, whether it is the default utc or other time zones are consistent. I feel it is necessary\n\nI am a little concerned over the use of hostPath, if this would require some privilege escalation.  Maybe easier to use the TZ environment variable and make it configurable so you can adjust downstream as needed?","accounts_in_message":[],"_revision_number":1},{"id":"858bc94c42e73744efa336255e5739805c750a23","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-05-05 07:45:36.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e \u003e Patch Set 1:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \u003e \n\u003e \u003e \u003e The container and host should both be using UTC by default, not sure why setting everything to local time is needed?\n\u003e \u003e \n\u003e \u003e I believe that more places will change the time zone according to their geographic location. \n\u003e \u003e For example, in China, \n\u003e \u003e we generally use CST. I believe that other regions have similar problems, which will cause inconsistencies in the container with the host time, and some problems will occur.Especially the time of the log is wrong, which is not conducive to viewing。\n\u003e \u003e \n\u003e \u003e  We can change the host directly, but the container is not easy to modify the time zone directly, but if we set it to be the same as the host, the user can customize the time zone at will, whether it is the default utc or other time zones are consistent. I feel it is necessary\n\u003e \n\u003e I am a little concerned over the use of hostPath, if this would require some privilege escalation.  Maybe easier to use the TZ environment variable and make it configurable so you can adjust downstream as needed?\n\nPodPreset is also a way, but it seems not as good as hostpath, we can discuss it together at the IRC meeting next week.","accounts_in_message":[],"_revision_number":1},{"id":"99b77e8868ef2969552c641f567ab65c00d32407","author":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"date":"2021-05-09 02:29:30.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 1:\n\u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e The container and host should both be using UTC by default, not sure why setting everything to local time is needed?\n\u003e \u003e \u003e \n\u003e \u003e \u003e I believe that more places will change the time zone according to their geographic location. \n\u003e \u003e \u003e For example, in China, \n\u003e \u003e \u003e we generally use CST. I believe that other regions have similar problems, which will cause inconsistencies in the container with the host time, and some problems will occur.Especially the time of the log is wrong, which is not conducive to viewing。\n\u003e \u003e \u003e \n\u003e \u003e \u003e  We can change the host directly, but the container is not easy to modify the time zone directly, but if we set it to be the same as the host, the user can customize the time zone at will, whether it is the default utc or other time zones are consistent. I feel it is necessary\n\u003e \u003e \n\u003e \u003e I am a little concerned over the use of hostPath, if this would require some privilege escalation.  Maybe easier to use the TZ environment variable and make it configurable so you can adjust downstream as needed?\n\u003e \n\u003e PodPreset is also a way, but it seems not as good as hostpath, we can discuss it together at the IRC meeting next week.\n\nPodPreset(alpha feature) has been deprecated ,   see: https://github.com/kubernetes/kubernetes/pull/94090, and TZ env is better,but maybe need a tadata pkg.","accounts_in_message":[],"_revision_number":1},{"id":"79de0a0e76f9147849ea3932196736e688eb51ac","author":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"date":"2021-05-11 00:32:32.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e \u003e Patch Set 1:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Patch Set 1:\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e Patch Set 1: Code-Review-1\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e The container and host should both be using UTC by default, not sure why setting everything to local time is needed?\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e I believe that more places will change the time zone according to their geographic location. \n\u003e \u003e \u003e \u003e For example, in China, \n\u003e \u003e \u003e \u003e we generally use CST. I believe that other regions have similar problems, which will cause inconsistencies in the container with the host time, and some problems will occur.Especially the time of the log is wrong, which is not conducive to viewing。\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e  We can change the host directly, but the container is not easy to modify the time zone directly, but if we set it to be the same as the host, the user can customize the time zone at will, whether it is the default utc or other time zones are consistent. I feel it is necessary\n\u003e \u003e \u003e \n\u003e \u003e \u003e I am a little concerned over the use of hostPath, if this would require some privilege escalation.  Maybe easier to use the TZ environment variable and make it configurable so you can adjust downstream as needed?\n\u003e \u003e \n\u003e \u003e PodPreset is also a way, but it seems not as good as hostpath, we can discuss it together at the IRC meeting next week.\n\u003e \n\u003e PodPreset(alpha feature) has been deprecated ,   see: https://github.com/kubernetes/kubernetes/pull/94090, and TZ env is better,but maybe need a tzdata pkg.","accounts_in_message":[],"_revision_number":1},{"id":"9a0666bf44adb0f88dbe28d81d901a42708f48a7","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2021-05-11 15:16:24.000000000","message":"Patch Set 1: Code-Review-1\n\n++ what\u0027s said. Can we make this optional or toggleable so folks who want it can use it but it is not enabled by default so the behavior doesn\u0027t change on them?","accounts_in_message":[],"_revision_number":1},{"id":"5dc16523ca4c2500394e03acc60b2de908ea07ec","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-05-13 09:19:12.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b7970ef949ecab424ad54ca2604b792f30477beb","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-05-13 09:21:16.000000000","message":"Patch Set 3: Published edit on patch set 2.","accounts_in_message":[],"_revision_number":3},{"id":"c6945d2beed81b609a8d58da48dcea3c827f7c87","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-13 10:19:18.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1d8de639e1184e778dba7ce5aa122100 : SUCCESS in 6m 36s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/13cf2e8bbdec4951877327575607babb : SUCCESS in 5m 41s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/7dfb9cd0c2544df590b2c38a128dd0c8 : FAILURE in 4m 58s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/68b88a4f75e048208f2ce64beca0c7b7 : SUCCESS in 48m 47s\n- openstack-helm-cinder-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/0bef3565b9fe40da87429e438af3516e : SUCCESS in 41m 03s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/c81f8f5b743f4dee901d9c050c500015 : SUCCESS in 49m 43s\n- openstack-helm-compute-kit-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/ee083cc059da4e73925e0a4673901576 : SUCCESS in 50m 39s","accounts_in_message":[],"_revision_number":3},{"id":"3020fcb9eb908df4ee3d92ba5181519924411df8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-05-13 10:45:23.000000000","message":"Patch Set 4: Published edit on patch set 3.","accounts_in_message":[],"_revision_number":4},{"id":"ad8515ea9131b514c78f88d54f6e07e665dd9288","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-13 11:53:33.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/77f0e3512ca64e55afb14b89ae0db43d : SUCCESS in 5m 17s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/f5a008764f1b43eab96dc150977ca5f7 : SUCCESS in 4m 18s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/f46d3739f5c64db0afa0e30854251398 : SUCCESS in 5m 17s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/dee0b5bf96b84c9d96aa363b504f847e : SUCCESS in 42m 39s\n- openstack-helm-cinder-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/2b725bbe625341d69b20f4c3bd79bd45 : SUCCESS in 48m 42s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/c9ff518f5d1f452083b31b3d0018a8cc : SUCCESS in 1h 01m 34s\n- openstack-helm-compute-kit-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/ec1fcad32b9549a2b306bebbac84f702 : SUCCESS in 48m 28s","accounts_in_message":[],"_revision_number":4},{"id":"bbeebf1265d242d4220f6fd50e0440dd423a39b9","author":{"_account_id":33173,"name":"liupeng0518","display_name":"Samuel Liu","email":"liupeng0518@gmail.com","username":"liupeng0518"},"date":"2021-05-19 03:54:58.000000000","message":"Patch Set 4: Code-Review+1\n\n/lgtm","accounts_in_message":[],"_revision_number":4},{"id":"2bcca4aefbed7ec055f2c0b8ae635af6d7d1eb99","author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"date":"2021-05-26 22:08:10.000000000","message":"Patch Set 4: Code-Review+2","accounts_in_message":[],"_revision_number":4},{"id":"483f4cbd3325ca448c0cf11dd546a90c54ada23a","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-06-03 06:22:18.000000000","message":"Patch Set 4: Code-Review-1\n\nuse utc\nif that\u0027s not enough use kubectl logs --timestamps","accounts_in_message":[],"_revision_number":4},{"id":"db971cc7411dc1b343b918a75e428ba9a873c5bc","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-06-03 06:26:09.000000000","message":"Patch Set 4: -Code-Review\n\ni vehemently disagree with this on several fronts\n\nthe best i can do for now is recuse myself","accounts_in_message":[],"_revision_number":4},{"id":"ea6197b8d65bab171455f03101f6dc930a0663bc","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-06-03 06:52:09.000000000","message":"Patch Set 4:\n\n\u003e Patch Set 4: -Code-Review\n\u003e \n\u003e i vehemently disagree with this on several fronts\n\u003e \n\u003e the best i can do for now is recuse myself\n\nThis idea is for people in non UTC time-zone,\n\nAnd I looked at kolla-ansible, and I\u0027ve been doing it all the time\n\nhttps://github.com/openstack/kolla-ansible/blob/master/ansible/roles/aodh/defaults/main.yml#L160\n\ncompared with others, hostpath is my concern\n\nIf most people think it\u0027s unreasonable, I\u0027ll abandon these ps.","accounts_in_message":[],"_revision_number":4},{"id":"137cffbd8551e6d82cad60ed64914b14183e9a9f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-06-11 07:22:35.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"fc87ef637649cae3656f45307c7195ebddadae91","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-11 08:21:46.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/261baa41810c42f7bacb03fdc106534d : SUCCESS in 6m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/cc8755aef4324bb5a89cc04976562c82 : SUCCESS in 4m 56s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/a377d0b3a89b4bd7bd7fed28da57196b : SUCCESS in 5m 57s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/7ce735151f7a48b897582ee7d02e5e71 : SUCCESS in 45m 50s\n- openstack-helm-cinder-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/589e5c3ca8454dde90e989966d4ea8a9 : SUCCESS in 46m 32s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/1c6836612d624593b71499d1ccc49fb2 : SUCCESS in 48m 18s\n- openstack-helm-compute-kit-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/15516b0c2ca342eabc163a2188861544 : SUCCESS in 55m 24s","accounts_in_message":[],"_revision_number":5},{"id":"1f1e0b017a7525f7fe4a235739289da31a39538e","tag":"autogenerated:gerrit:abandon","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-12-23 08:26:23.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"f7e2b079b5860cd286c95b6ed9fd382e98ae7930","revisions":{"e0c96d4af84086dced36d6a1f3c0dc4d7b3b2c4e":{"kind":"REWORK","_number":1,"created":"2021-05-03 07:09:55.000000000","uploader":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"ref":"refs/changes/35/789135/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/35/789135/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/1"}}},"commit":{"parents":[{"commit":"5233582991889bef3d65ecd452519c3ee9195f9e","subject":"Remove support for openstack releases older than T","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/5233582991889bef3d65ecd452519c3ee9195f9e"}]}],"author":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-03 07:04:48.000000000","tz":480},"committer":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-03 07:09:49.000000000","tz":480},"subject":"Fixed pod\u0027s localtime issue about aodh","message":"Fixed pod\u0027s localtime issue about aodh\n\nMake container uses the host\u0027s localtime, ensure that the time in the container is the same as the host time.\n\nChange-Id: I53238d592e1d0f794b7a6c7a619cce232fe9cf9f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/e0c96d4af84086dced36d6a1f3c0dc4d7b3b2c4e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/e0c96d4af84086dced36d6a1f3c0dc4d7b3b2c4e"}]},"branch":"refs/heads/master"},"f1c9ac965452cd900bba236d1931df6c49dee80c":{"kind":"REWORK","_number":2,"created":"2021-05-13 09:19:12.000000000","uploader":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"ref":"refs/changes/35/789135/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/35/789135/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/2"}}},"commit":{"parents":[{"commit":"109dc623419006ba327c1e6337e4de54a66a261c","subject":"Merge \"Remove keystone paste ini file\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/109dc623419006ba327c1e6337e4de54a66a261c"}]}],"author":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-13 09:17:51.000000000","tz":480},"committer":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-13 09:18:59.000000000","tz":480},"subject":"Provide an option about pod\u0027s time for aodh chart","message":"Provide an option about pod\u0027s time for aodh chart\n\nIf someone want to change the time-zone, this option can make the time in the container is the same as the host time.\n\nChange-Id: I53238d592e1d0f794b7a6c7a619cce232fe9cf9f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f1c9ac965452cd900bba236d1931df6c49dee80c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f1c9ac965452cd900bba236d1931df6c49dee80c"}]},"branch":"refs/heads/master"},"6660e9f8ee59d15ff49453847a211048e6780e5a":{"kind":"REWORK","_number":3,"created":"2021-05-13 09:21:16.000000000","uploader":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"ref":"refs/changes/35/789135/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/35/789135/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/3"}}},"commit":{"parents":[{"commit":"109dc623419006ba327c1e6337e4de54a66a261c","subject":"Merge \"Remove keystone paste ini file\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/109dc623419006ba327c1e6337e4de54a66a261c"}]}],"author":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-13 09:17:51.000000000","tz":480},"committer":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-13 09:21:09.000000000","tz":0},"subject":"Provide an option about pod\u0027s time for aodh chart","message":"Provide an option about pod\u0027s time for aodh chart\n\nIf someone want to change the time-zone, this option can make the time in the container is the same as the host time.\n\nChange-Id: I53238d592e1d0f794b7a6c7a619cce232fe9cf9f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/6660e9f8ee59d15ff49453847a211048e6780e5a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/6660e9f8ee59d15ff49453847a211048e6780e5a"}]},"branch":"refs/heads/master"},"713b67ef9ce866fbe704c3262c424507f8abbea3":{"kind":"REWORK","_number":4,"created":"2021-05-13 10:45:23.000000000","uploader":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"ref":"refs/changes/35/789135/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/35/789135/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/4"}}},"commit":{"parents":[{"commit":"109dc623419006ba327c1e6337e4de54a66a261c","subject":"Merge \"Remove keystone paste ini file\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/109dc623419006ba327c1e6337e4de54a66a261c"}]}],"author":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-13 09:17:51.000000000","tz":480},"committer":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-05-13 10:45:12.000000000","tz":0},"subject":"Provide an option about pod\u0027s time for aodh chart","message":"Provide an option about pod\u0027s time for aodh chart\n\nIf someone want to change the time-zone, this option can make the time in the container is the same as the host time.\n\nChange-Id: I53238d592e1d0f794b7a6c7a619cce232fe9cf9f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/713b67ef9ce866fbe704c3262c424507f8abbea3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/713b67ef9ce866fbe704c3262c424507f8abbea3"}]},"branch":"refs/heads/master"},"f7e2b079b5860cd286c95b6ed9fd382e98ae7930":{"kind":"REWORK","_number":5,"created":"2021-06-11 07:22:35.000000000","uploader":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"ref":"refs/changes/35/789135/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/35/789135/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/35/789135/5"}}},"commit":{"parents":[{"commit":"f7ef7a6ef02d5e163373e3327c421641f71279b5","subject":"Merge \"Mount rabbitmq TLS secret\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f7ef7a6ef02d5e163373e3327c421641f71279b5"}]}],"author":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-06-11 07:20:11.000000000","tz":480},"committer":{"name":"jinyuanliu","email":"liujinyuan@inspur.com","date":"2021-06-11 07:21:26.000000000","tz":480},"subject":"Provide an option about pod\u0027s time for aodh chart","message":"Provide an option about pod\u0027s time for aodh chart\n\nIf someone want to change the time-zone, this option can make the time in the container is the same as the host time.\n\nChange-Id: I53238d592e1d0f794b7a6c7a619cce232fe9cf9f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f7e2b079b5860cd286c95b6ed9fd382e98ae7930"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/f7e2b079b5860cd286c95b6ed9fd382e98ae7930"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}