[VTAdmin] Remove vtctld web link, improve local example (#15607)

Signed-off-by: notfelineit <notfelineit@gmail.com>
vitessio · Apr 2, 2024 · a4988f3 · a4988f3
1 parent ec2080c
commit a4988f3
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 11 deletions.
diff --git a/examples/common/scripts/vtadmin-up.sh b/examples/common/scripts/vtadmin-up.sh
@@ -24,6 +24,8 @@ web_dir="${script_dir}/../../../web/vtadmin"
 vtadmin_api_port=14200
 vtadmin_web_port=14201
 
+echo "vtadmin-api http-origin set to \"http://${hostname}:${vtadmin_web_port}\""
+
 vtadmin \
  --addr "${hostname}:${vtadmin_api_port}" \
  --http-origin "http://${hostname}:${vtadmin_web_port}" \
@@ -48,9 +50,12 @@ vtadmin-api is running!
  - PID: ${vtadmin_api_pid}
 "
 
+echo "Building vtadmin-web..."
+source "${web_dir}/build.sh"
+
 # Wait for vtadmin to successfully discover the cluster
 expected_cluster_result="{\"result\":{\"clusters\":[{\"id\":\"${cluster_name}\",\"name\":\"${cluster_name}\"}]},\"ok\":true}"
-for _ in {0..300}; do
+for _ in {0..100}; do
  result=$(curl -s "http://${hostname}:${vtadmin_api_port}/api/clusters")
  if [[ ${result} == "${expected_cluster_result}" ]]; then
  break

diff --git a/web/vtadmin/build.sh b/web/vtadmin/build.sh
@@ -24,6 +24,12 @@ web_dir="${script_dir}"
 
 vtadmin_api_port=14200
 
+if [ -z "${hostname}" ]
+then
+ hostname=$(hostname -f)
+ output "\n\033[1;32mhostname was empty, set it to \"${hostname}\"\033[0m"
+fi
+
 # Download nvm and node
 if [[ -z ${NVM_DIR} ]]; then
  export NVM_DIR="$HOME/.nvm"
@@ -50,6 +56,9 @@ npm --prefix "$web_dir" --silent install
 
 export PATH=$PATH:$web_dir/node_modules/.bin/
 
+vite_vtadmin_api_address="http://${hostname}:${vtadmin_api_port}"
+output "\n\033[1;32mSetting VITE_VTADMIN_API_ADDRESS to \"${vite_vtadmin_api_address}\"\033[0m"
+
 VITE_VTADMIN_API_ADDRESS="http://${hostname}:${vtadmin_api_port}" \
  VITE_ENABLE_EXPERIMENTAL_TABLET_DEBUG_VARS="true" \
  npm run --prefix "$web_dir" build
diff --git a/web/vtadmin/src/components/routes/Vtctlds.tsx b/web/vtadmin/src/components/routes/Vtctlds.tsx
@@ -37,7 +37,6 @@ export const Vtctlds = () => {
  cluster: v.cluster?.name,
  clusterID: v.cluster?.id,
  hostname: v.hostname,
- fqdn: v.FQDN,
  }));
 
  const filtered = filterNouns(filter, mapped);
@@ -50,15 +49,7 @@ export const Vtctlds = () => {
  return (
  <tr key={row.hostname}>
  <DataCell>
- <div className="font-bold">
- {row.fqdn ? (
- <a href={`//${row.fqdn}`} rel="noopener noreferrer" target="_blank">
- {row.hostname}
- </a>
- ) : (
- row.hostname
- )}
- </div>
+ <div className="font-bold">{row.hostname}</div>
  </DataCell>
  <DataCell>
  {row.cluster}