)]}'
{"id":"openstack%2Fwatcher~339285","triplet_id":"openstack%2Fwatcher~master~Id65225cdbacb77f53ba7330186f820a721337356","project":"openstack/watcher","branch":"master","topic":"bug/1598946","hashtags":[],"change_id":"Id65225cdbacb77f53ba7330186f820a721337356","subject":"Handle race condition of sync from multiple decision engines","status":"ABANDONED","created":"2016-07-08 00:08:04.000000000","updated":"2016-08-16 14:48:28.000000000","total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2b8d4dafc3ed843ffda866ae93833bcb453a0d09","_number":339285,"virtual_id_number":339285,"owner":{"_account_id":11235,"name":"Edwin Zhai","email":"edwin.zhai@intel.com","username":"gzhai"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},{"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},{"value":-1,"date":"2016-07-08 00:46:32.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"all":[{"value":-1,"date":"2016-07-20 15:42:00.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},{"value":-1,"date":"2016-07-13 10:40:01.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":[{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},{"date":"2016-07-13 14:22:06.000000000","_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},{"date":"2016-07-13 12:37:43.000000000","_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},{"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-07-08 00:46:32.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-07-13 10:40:01.000000000","updated_by":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"reviewer":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"state":"REVIEWER"},{"updated":"2016-07-13 12:37:43.000000000","updated_by":{"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},"reviewer":{"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},"state":"REVIEWER"},{"updated":"2016-07-13 14:22:06.000000000","updated_by":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"reviewer":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"state":"REVIEWER"},{"updated":"2016-07-20 15:42:00.000000000","updated_by":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"reviewer":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"state":"REVIEWER"}],"messages":[{"id":"f7b95f1194c4ad65da84987e689c1802c13545db","author":{"_account_id":11235,"name":"Edwin Zhai","email":"edwin.zhai@intel.com","username":"gzhai"},"date":"2016-07-08 00:08:04.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"efb2a62da8df4b5c10091589a1887c04b56bcc96","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-08 00:46:32.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-watcher-docs http://docs-draft.openstack.org/85/339285/1/check/gate-watcher-docs/2adce3f//doc/build/html/ : SUCCESS in 4m 10s\n- gate-watcher-pep8 http://logs.openstack.org/85/339285/1/check/gate-watcher-pep8/024bb8c/ : FAILURE in 3m 20s\n- gate-watcher-python27 http://logs.openstack.org/85/339285/1/check/gate-watcher-python27/248115a/ : FAILURE in 4m 14s\n- gate-watcher-python34 http://logs.openstack.org/85/339285/1/check/gate-watcher-python34/1b6a8f6/ : FAILURE in 3m 28s\n- gate-watcher-python35-nv http://logs.openstack.org/85/339285/1/check/gate-watcher-python35-nv/e64dd53/ : FAILURE in 5m 47s (non-voting)\n- watcher-coverage http://logs.openstack.org/85/339285/1/check/watcher-coverage/e83f547/ : FAILURE in 4m 05s (non-voting)\n- gate-watcher-dsvm-multinode-nv http://logs.openstack.org/85/339285/1/check/gate-watcher-dsvm-multinode-nv/d21384c/ : FAILURE in 36m 07s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"f8454bfdcd03ba95cd5c047f360e8965e096a997","author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"date":"2016-07-13 10:40:01.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)\n\nStrategyAlreadyExists needs to be caught here:\nhttps://review.openstack.org/gitweb?p\u003dopenstack/watcher.git;a\u003dblob;f\u003dwatcher/objects/strategy.py;h\u003d60b35701c17647bc98e7be7f891beed0431c2ec6;hb\u003ddb1eb089d9a0340d10c2076d9c7d093afd1a91a9#l159","accounts_in_message":[],"_revision_number":1},{"id":"bc15097ae77ecd310733a1a59e2bf0615bf7b34d","author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"date":"2016-07-13 10:44:57.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"852f468bdb328c10ea2924a2518bc1f023e7ae99","author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"date":"2016-07-13 12:37:09.000000000","message":"Patch Set 1: Code-Review-2\n\nThis is not a bug. We should discuss that. IMHO, we don\u0027t need be HA ready for now. Moreover, you are missing many things.","accounts_in_message":[],"_revision_number":1},{"id":"4c2b9a3491102377b8e9029004d23b9d65c4e9c0","author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"date":"2016-07-20 15:42:00.000000000","message":"Patch Set 1: Code-Review-1","accounts_in_message":[],"_revision_number":1},{"id":"6d2c44db647cfb5676e080915eb617a1af2980e6","author":{"_account_id":11235,"name":"Edwin Zhai","email":"edwin.zhai@intel.com","username":"gzhai"},"date":"2016-08-10 14:42:55.000000000","message":"Abandoned\n\nas discussed, it\u0027s not a bug","accounts_in_message":[],"_revision_number":1},{"id":"9323c4af935eeab8efc046a92e73a103b36f06d2","author":{"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},"date":"2016-08-16 14:47:13.000000000","message":"Patch Set 1:\n\nshould be abandoned after mid-cycle discussions\nhttps://etherpad.openstack.org/p/newton-watcher-midcycle","accounts_in_message":[],"_revision_number":1},{"id":"58538b8562a01938e0a70936fb6fda64ae532ae9","author":{"_account_id":11750,"name":"Antoine Cabot","email":"antoine.cabot@b-com.com","username":"antoinecabot"},"date":"2016-08-16 14:48:28.000000000","message":"Patch Set 1:\n\nsorry for my last comment, it has already been abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"db1eb089d9a0340d10c2076d9c7d093afd1a91a9","revisions":{"db1eb089d9a0340d10c2076d9c7d093afd1a91a9":{"kind":"REWORK","_number":1,"created":"2016-07-08 00:08:04.000000000","uploader":{"_account_id":11235,"name":"Edwin Zhai","email":"edwin.zhai@intel.com","username":"gzhai"},"ref":"refs/changes/85/339285/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/85/339285/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/85/339285/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/85/339285/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/85/339285/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/85/339285/1"}}},"commit":{"parents":[{"commit":"64903ce56c700d23a28d9056e6525ad754ad1027","subject":"Merge \"Enable strategy parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/64903ce56c700d23a28d9056e6525ad754ad1027"}]}],"author":{"name":"Edwin Zhai","email":"edwin.zhai@intel.com","date":"2016-07-07 07:26:04.000000000","tz":0},"committer":{"name":"Edwin Zhai","email":"edwin.zhai@intel.com","date":"2016-07-08 00:06:59.000000000","tz":0},"subject":"Handle race condition of sync from multiple decision engines","message":"Handle race condition of sync from multiple decision engines\n\nTo avoid race condition of goal/strategy creation when sync from\nmultiple decision engines, takes the following changes:\n\n* makes name column in goal/strategy table as unique, so that creating\n  a new goal/strategy with same name as existed one would cause\n  AlreadyExists exception.\n* refactors goal/strategy creation code to be aware of another engine\n  possibly doing the same thing.\n* comparison of goal with efficacy specification is changed to check\n  source code of schema instead of schema object, or else a goal\n  created by one engine will be treated stale and deleted immediately.\n\nChange-Id: Id65225cdbacb77f53ba7330186f820a721337356\nCloses-Bug: #1598946\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/db1eb089d9a0340d10c2076d9c7d093afd1a91a9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/db1eb089d9a0340d10c2076d9c7d093afd1a91a9"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"64903ce56c700d23a28d9056e6525ad754ad1027","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}