refac: Replace move_tables with movetables

Signed-off-by: Noble Mittal <noblemittal@outlook.com>
vitessio · Sep 19, 2024 · 7911f66 · 7911f66
1 parent 843bb32
commit 7911f66
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/go/vt/vtadmin/api.go b/go/vt/vtadmin/api.go
@@ -423,7 +423,7 @@ func (api *API) Handler() http.Handler {
 	router.HandleFunc("/workflow/{cluster_id}/{keyspace}/{name}/status", httpAPI.Adapt(vtadminhttp.GetWorkflowStatus)).Name("API.GetWorkflowStatus")
 	router.HandleFunc("/workflow/{cluster_id}/{keyspace}/{name}/start", httpAPI.Adapt(vtadminhttp.StartWorkflow)).Name("API.StartWorkflow")
 	router.HandleFunc("/workflow/{cluster_id}/{keyspace}/{name}/stop", httpAPI.Adapt(vtadminhttp.StopWorkflow)).Name("API.StopWorkflow")
-	router.HandleFunc("/workflow/{cluster_id}/move_tables", httpAPI.Adapt(vtadminhttp.MoveTablesCreate)).Name("API.MoveTablesCreate").Methods("POST")
+	router.HandleFunc("/workflow/{cluster_id}/movetables", httpAPI.Adapt(vtadminhttp.MoveTablesCreate)).Name("API.MoveTablesCreate").Methods("POST")
 
 	experimentalRouter := router.PathPrefix("/experimental").Subrouter()
 	experimentalRouter.HandleFunc("/tablet/{tablet}/debug/vars", httpAPI.Adapt(experimental.TabletDebugVarsPassthrough)).Name("API.TabletDebugVarsPassthrough")

diff --git a/go/vt/vtadmin/http/workflows.go b/go/vt/vtadmin/http/workflows.go
@@ -125,7 +125,7 @@ func StopWorkflow(ctx context.Context, r Request, api *API) *JSONResponse {
 // MoveTablesCreate implements the http wrapper for the VTAdminServer.MoveTablesCreate
 // method.
 //
-// Its route is /workflow/{cluster_id}/move_tables
+// Its route is /workflow/{cluster_id}/movetables
 func MoveTablesCreate(ctx context.Context, r Request, api *API) *JSONResponse {
 	vars := r.Vars()
 	decoder := json.NewDecoder(r.Body)

diff --git a/web/vtadmin/src/api/http.ts b/web/vtadmin/src/api/http.ts
@@ -454,7 +454,7 @@ export interface CreateMoveTablesParams {
 }
 
 export const createMoveTables = async ({ clusterID, request }: CreateMoveTablesParams) => {
-    const { result } = await vtfetch(`/api/workflow/${clusterID}/move_tables`, {
+    const { result } = await vtfetch(`/api/workflow/${clusterID}/movetables`, {
         body: JSON.stringify(request),
         method: 'post',
     });

diff --git a/web/vtadmin/src/components/App.tsx b/web/vtadmin/src/components/App.tsx
@@ -111,7 +111,7 @@ export const App = () => {
                         </Route>
 
                         {!isReadOnlyMode() && (
-                            <Route exact path="/workflows/move_tables/create">
+                            <Route exact path="/workflows/movetables/create">
                                 <CreateMoveTables />
                             </Route>
                         )}

diff --git a/web/vtadmin/src/components/routes/Workflows.tsx b/web/vtadmin/src/components/routes/Workflows.tsx
@@ -216,7 +216,7 @@ export const Workflows = () => {
                                 title="Create Workflow"
                                 className="!text-[32px] w-16 h-16"
                             >
-                                <Link to="/workflows/move_tables/create">
+                                <Link to="/workflows/movetables/create">
                                     <MenuItem>MoveTables</MenuItem>
                                 </Link>
                                 <MenuItem disabled>Reshard</MenuItem>

diff --git a/web/vtadmin/src/components/routes/createWorkflow/CreateMoveTables.tsx b/web/vtadmin/src/components/routes/createWorkflow/CreateMoveTables.tsx
@@ -231,7 +231,7 @@ export const CreateMoveTables = () => {
                             />
                             <Label label="All Tables" />
                         </div>
-                        If enabled, the move will copy all the tables from source keyspace.
+                        If enabled, the workflow will copy all the tables from source keyspace.
                     </div>
 
                     <h3 className="mt-8 mb-2">Advanced</h3>
@@ -291,7 +291,7 @@ export const CreateMoveTables = () => {
                             />
                             <Label label="Auto Start" />
                         </div>
-                        If enabled, the move will be started automatically.
+                        If enabled, the workflow will be started automatically.
                     </div>
 
                     {clustersQuery.isError && (