)]}'
{"id":"openstack%2Fcloudkitty~683408","triplet_id":"openstack%2Fcloudkitty~master~I6483dfb3e14302e1a514e0ad6adb4f20079fce99","project":"openstack/cloudkitty","branch":"master","hashtags":[],"change_id":"I6483dfb3e14302e1a514e0ad6adb4f20079fce99","subject":"Remove \"V2 Storage is in beta\" log message","status":"ABANDONED","created":"2019-09-20 14:34:53.000000000","updated":"2019-10-03 15:48:50.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d43abc6ce9e547d9014bdc9bb758f868f9b60a53","_number":683408,"virtual_id_number":683408,"owner":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"date":"2019-10-03 15:48:26.000000000","_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},{"value":1,"date":"2019-09-20 15:31:12.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":{"disliked":{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},{"value":-1,"date":"2019-10-03 15:16:12.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},{"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},{"_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},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-09-20 15:31:12.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":"2019-10-03 15:16:12.000000000","updated_by":{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},"reviewer":{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},"state":"REVIEWER"}],"messages":[{"id":"fa197dccf8ca9b3efb829edf2f0002ec06b042a4","author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"date":"2019-09-20 14:34:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"257a2d8d8015a4e58cb02f304933fee2f31522c3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-20 15:31:12.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/246028f95cd1435b9fd0f206e0b38cf1 : SUCCESS in 6m 03s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/59de73830db140eb9c30d0030ebeea32 : SUCCESS in 5m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b48d73a3583345be8e846967a1d00803 : SUCCESS in 3m 30s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/66c5cdaf31e54726b4eee4e291ceeb1d : SUCCESS in 4m 42s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/b6192dee90b04aaa976db3504cb6f956 : SUCCESS in 6m 51s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/90d7515921274dcea5903f4ffff03576 : SUCCESS in 8m 21s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/35d1ff77482847e2933baf081be3725d : SUCCESS in 7m 13s\n- cloudkitty-tempest-full-v2-storage-influxdb https://zuul.opendev.org/t/openstack/build/2cd629a1ec874c56960db183f8c56969 : SUCCESS in 31m 47s\n- cloudkitty-tempest-full-v2-storage-influxdb-py3 https://zuul.opendev.org/t/openstack/build/55503d7327294df49019eaa379107349 : SUCCESS in 39m 34s\n- cloudkitty-tempest-full-v1-storage-sqlalchemy https://zuul.opendev.org/t/openstack/build/59461cbf82b244d89680321ce68cfc27 : SUCCESS in 44m 37s\n- cloudkitty-tempest-full-v1-storage-sqlalchemy-py3 https://zuul.opendev.org/t/openstack/build/622bb684bba14488a3cc796cec90643a : SUCCESS in 42m 19s\n- cloudkitty-tempest-full-ipv6-only https://zuul.opendev.org/t/openstack/build/4d7a568aad584193b77f61d08aa2519e : SUCCESS in 35m 04s\n- cloudkitty-tox-bandit https://zuul.opendev.org/t/openstack/build/5863b2029a4b460f973fceaaeb35e54b : SUCCESS in 3m 24s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"df1d1854660da685ba4620cd37615aeda2bde429","author":{"_account_id":23060,"name":"Luka Peschke","email":"mail@lukapeschke.com","username":"lukapeschke"},"date":"2019-10-03 15:16:12.000000000","message":"Patch Set 1: Code-Review-1\n\nHi Rafael. Thanks for the suggestion, but we\u0027d like to keep the log message. \n\nHowever, this https://review.opendev.org/#/c/686393/ should solve the issue for you: The message will appear only on the first request.","accounts_in_message":[],"_revision_number":1},{"id":"013db76dbb477f984bd408179d90461230c69076","author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"date":"2019-10-03 15:48:26.000000000","message":"Patch Set 1:\n\n\u003e Hi Rafael. Thanks for the suggestion, but we\u0027d like to keep the log\n \u003e message.\n \u003e \n \u003e However, this https://review.opendev.org/#/c/686393/ should solve\n \u003e the issue for you: The message will appear only on the first\n \u003e request.\n\nYes, this PR also solves the problem we had. \nI still disagree with the usefulness of the message tough.","accounts_in_message":[],"_revision_number":1},{"id":"d43abc6ce9e547d9014bdc9bb758f868f9b60a53","author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"date":"2019-10-03 15:48:50.000000000","message":"Abandoned\n\nAbandon this PR in favor of: https://review.opendev.org/#/c/686393/","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"bd7a7dfcfe8cd9bf5841224df501ec8af5eea815","revisions":{"bd7a7dfcfe8cd9bf5841224df501ec8af5eea815":{"kind":"REWORK","_number":1,"created":"2019-09-20 14:34:53.000000000","uploader":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"ref":"refs/changes/08/683408/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cloudkitty","ref":"refs/changes/08/683408/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cloudkitty refs/changes/08/683408/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cloudkitty refs/changes/08/683408/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cloudkitty refs/changes/08/683408/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cloudkitty refs/changes/08/683408/1"}}},"commit":{"parents":[{"commit":"236487e3c3f2576ff4d938b898c62237d61ed9ef","subject":"Merge \"Add support for PDF doc generation\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cloudkitty/commit/236487e3c3f2576ff4d938b898c62237d61ed9ef"}]}],"author":{"name":"Rafael Weingärtner","email":"rafael@apache.org","date":"2019-09-20 14:28:15.000000000","tz":-180},"committer":{"name":"Rafael Weingärtner","email":"rafael@apache.org","date":"2019-09-20 14:28:15.000000000","tz":-180},"subject":"Remove \"V2 Storage is in beta\" log message","message":"Remove \"V2 Storage is in beta\" log message\n\nWe are proposing to remove the log entry message \"V2 Storage is in beta\"\nfrom cloudkitty.storage.get_storage. This type of information should not\nbe documented in the log, but rather in the documentation, release notes,\nand/or README file.\n\nWe started having trouble with it (the message) because we call \"/\"\n(CloudKitty\u0027s root context) to validate if the application is up and running.\n For every call, we would get one entry of that message in our log files.\nThis was generating thousands of useless information in our log files every hour\n\nChange-Id: I6483dfb3e14302e1a514e0ad6adb4f20079fce99\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cloudkitty/commit/bd7a7dfcfe8cd9bf5841224df501ec8af5eea815"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cloudkitty/commit/bd7a7dfcfe8cd9bf5841224df501ec8af5eea815"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
