From d69abb622df02875a91861b24b97e282de7135f9 Mon Sep 17 00:00:00 2001 From: Frances Thai <31225471+notfelineit@users.noreply.github.com> Date: Tue, 2 Apr 2024 09:41:59 -0700 Subject: [PATCH 1/2] [VTAdmin] Remove vtctld web link, improve local example (#15607) Signed-off-by: notfelineit --- examples/common/scripts/vtadmin-up.sh | 7 ++++++- web/vtadmin/build.sh | 9 +++++++++ web/vtadmin/src/components/routes/Vtctlds.tsx | 11 +---------- 3 files changed, 16 insertions(+), 11 deletions(-) diff --git a/examples/common/scripts/vtadmin-up.sh b/examples/common/scripts/vtadmin-up.sh index faa2e6a177f..292a71b99c5 100755 --- 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 index 54d1a5b1926..a85e6a6fa44 100755 --- 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 index 26efd8498b4..33b66b11306 100644 --- 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 ( -
- {row.fqdn ? ( - - {row.hostname} - - ) : ( - row.hostname - )} -
+
{row.hostname}
{row.cluster} From dccef777d7212dfb1d6a9494de96bfad59b29c12 Mon Sep 17 00:00:00 2001 From: notfelineit Date: Thu, 2 May 2024 09:49:48 -0700 Subject: [PATCH 2/2] Trigger rebuild Signed-off-by: notfelineit