Remove old synch_id concept from autotest
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e4c08273f039d97349999818e25fedbb13cdcb4b commit e4c08273f039d97349999818e25fedbb13cdcb4b Author: Allen Li <ayatane@chromium.org> Date: Thu Feb 09 20:47:41 2017 [autotest] Remove synch_id concept BUG= chromium:687781 TEST=None Change-Id: Ibbef8f7b58d7bb26dcbbe85df308b9e0c4b86bd9 Reviewed-on: https://chromium-review.googlesource.com/435711 Commit-Ready: Allen Li <ayatane@chromium.org> Tested-by: Allen Li <ayatane@chromium.org> Reviewed-by: Allen Li <ayatane@chromium.org> [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/frontend/afe/rdb_model_extensions.py [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/scheduler/scheduler_models.py [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/server/frontend.py [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/scheduler/shard/shard_client_unittest.py [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/frontend/afe/models.py [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/cli/host_unittest.py [modify] https://crrev.com/e4c08273f039d97349999818e25fedbb13cdcb4b/cli/job_unittest.py
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/eb97ee7fffb4a2ef3155b07f2aa71f04d574b642 commit eb97ee7fffb4a2ef3155b07f2aa71f04d574b642 Author: Allen Li <ayatane@chromium.org> Date: Mon Feb 27 18:15:58 2017 [autotest] Remove synch_id from database BUG= chromium:687781 TEST=Run migrate.py sync back and forth Change-Id: Ib066b19d90b18ae9926e8aec2c871b963e6a2465 Reviewed-on: https://chromium-review.googlesource.com/441107 Commit-Ready: Allen Li <ayatane@chromium.org> Tested-by: Allen Li <ayatane@chromium.org> Reviewed-by: Dan Shi <dshi@google.com> [modify] https://crrev.com/eb97ee7fffb4a2ef3155b07f2aa71f04d574b642/frontend/afe/rdb_model_extensions.py [modify] https://crrev.com/eb97ee7fffb4a2ef3155b07f2aa71f04d574b642/frontend/afe/models_test.py [modify] https://crrev.com/eb97ee7fffb4a2ef3155b07f2aa71f04d574b642/scheduler/scheduler_models.py [modify] https://crrev.com/eb97ee7fffb4a2ef3155b07f2aa71f04d574b642/frontend/afe/doctests/001_rpc_test.txt [add] https://crrev.com/eb97ee7fffb4a2ef3155b07f2aa71f04d574b642/frontend/migrations/110_remove_synch_id.py
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c744f5fa4992f996768a4857c28fcc8045ea9df1 commit c744f5fa4992f996768a4857c28fcc8045ea9df1 Author: Allen Li <ayatane@chromium.org> Date: Fri Mar 03 18:49:41 2017 [autotest] Remove synch_id from database BUG= chromium:687781 TEST=Run migrate.py sync back and forth Change-Id: Ieed2052c9aa6958cf3fb6081ee310499c233d112 Reviewed-on: https://chromium-review.googlesource.com/449972 Reviewed-by: Allen Li <ayatane@chromium.org> Commit-Queue: Allen Li <ayatane@chromium.org> Tested-by: Allen Li <ayatane@chromium.org> [modify] https://crrev.com/c744f5fa4992f996768a4857c28fcc8045ea9df1/frontend/afe/rdb_model_extensions.py [modify] https://crrev.com/c744f5fa4992f996768a4857c28fcc8045ea9df1/frontend/afe/models_test.py [modify] https://crrev.com/c744f5fa4992f996768a4857c28fcc8045ea9df1/scheduler/scheduler_models.py [modify] https://crrev.com/c744f5fa4992f996768a4857c28fcc8045ea9df1/frontend/afe/doctests/001_rpc_test.txt [add] https://crrev.com/c744f5fa4992f996768a4857c28fcc8045ea9df1/frontend/migrations/112_remove_synch_id.py
The database migration CL is getting reverted (again). We've decided to do DB cleanups in one fell swoop.
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2d3884ade80ee5a8e2316ddc0996e7a1a12d58da commit 2d3884ade80ee5a8e2316ddc0996e7a1a12d58da Author: Prathmesh Prabhu <pprabhu@chromium.org> Date: Mon Mar 06 23:57:01 2017 Revert "[autotest] Remove synch_id from database" This reverts commit c744f5fa4992f996768a4857c28fcc8045ea9df1. Reason for revert: We don't want lab downtime today. We decided to batch up some outstanding DB changes and do it in one fell swoop. I'm just the executor... Original change's description: > [autotest] Remove synch_id from database > > BUG= chromium:687781 > TEST=Run migrate.py sync back and forth > > Change-Id: Ieed2052c9aa6958cf3fb6081ee310499c233d112 > Reviewed-on: https://chromium-review.googlesource.com/449972 > Reviewed-by: Allen Li <ayatane@chromium.org> > Commit-Queue: Allen Li <ayatane@chromium.org> > Tested-by: Allen Li <ayatane@chromium.org> > TBR=pprabhu@chromium.org,xixuan@chromium.org,ayatane@chromium.org BUG= chromium:687781 BUG= chromium:691084 Change-Id: Ia822a826f8f63dbe32c435adf9e4aea2045a6c3f Reviewed-on: https://chromium-review.googlesource.com/450837 Reviewed-by: Ningning Xia <nxia@chromium.org> Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org> Commit-Queue: Ningning Xia <nxia@chromium.org> Tested-by: Ningning Xia <nxia@chromium.org> [modify] https://crrev.com/2d3884ade80ee5a8e2316ddc0996e7a1a12d58da/frontend/afe/rdb_model_extensions.py [modify] https://crrev.com/2d3884ade80ee5a8e2316ddc0996e7a1a12d58da/frontend/afe/models_test.py [modify] https://crrev.com/2d3884ade80ee5a8e2316ddc0996e7a1a12d58da/scheduler/scheduler_models.py [modify] https://crrev.com/2d3884ade80ee5a8e2316ddc0996e7a1a12d58da/frontend/afe/doctests/001_rpc_test.txt [add] https://crrev.com/2d3884ade80ee5a8e2316ddc0996e7a1a12d58da/frontend/migrations/113_add_back_sync_id_temporarily.py
We actually want to keep this.
Comment 1 by bugdroid1@chromium.org
, Feb 9 2017