wangdefa/tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Compare commits

base: wangdefa:71d487c59afcc1c6658e0ccb33fbdbd75bd89cda
Branches Tags
wangdefa:main
..
compare: wangdefa:1be07db2767fec79a185433867ddf68456e1025b
Branches Tags
wangdefa:main

4 Commits
71d487c59a ... 1be07db276

Author SHA1 Message Date
Wang Defa
1be07db276 fix: 避免 remote_loader.sh 中重复定义 readonly 变量 
## 问题描述

当脚本使用远程加载时，出现警告：
```
/tmp/tmp.xxx: line 10: REMOTE_BASE_URL: readonly variable
```

## 根本原因

`REMOTE_BASE_URL` 被重复定义为 readonly 变量：

1. 调用脚本中定义（如 oci/create_instance.sh:16）：
   ```bash
   readonly REMOTE_BASE_URL="..."
   ```

2. remote_loader.sh:10 中再次定义：
   ```bash
   readonly REMOTE_BASE_URL="..."
   ```

Bash 不允许重新赋值 readonly 变量，即使值相同也会产生警告。

## 修复方案

在 remote_loader.sh 中添加检查，只有在变量未定义时才设置：

```bash
# 旧代码
readonly REMOTE_BASE_URL="..."

# 新代码
if [[ -z "${REMOTE_BASE_URL:-}" ]]; then
    readonly REMOTE_BASE_URL="..."
fi
```

## 影响

- ✅ 消除警告信息
- ✅ 保持向后兼容
- ✅ 允许调用者自定义 REMOTE_BASE_URL
- ✅ 如果未定义则使用默认值
 2025-12-26 15:23:48 +08:00
Wang Defa
89f24a7fef fix: 修复所有脚本的 process substitution 兼容性问题 
## 问题描述

在使用 `set -u` 严格模式时，`source <(curl ...)` 或 `source <(wget ...)`
的 process substitution 方式会在脚本退出时产生错误：

```
/dev/fd/63: line 1: fifo: unbound variable
```

## 根本原因

Process substitution 创建的临时文件描述符（如 /dev/fd/63）在退出时
与 Bash 的 `set -u` 严格模式存在兼容性问题，导致错误消息。

## 修复方案

将 process substitution 替换为临时文件方案：

**旧方案（有问题）：**
```bash
source <(curl -fsSL "$url")
```

**新方案（兼容性好）：**
```bash
temp_loader=$(mktemp)
curl -fsSL "$url" -o "$temp_loader"
source "$temp_loader"
rm -f "$temp_loader"
```

## 修改的文件

批量修复了所有 7 个脚本的远程加载逻辑：

- oci/create_instance.sh
- linux/create_raid0_array.sh
- linux/install_oh_my_zsh.sh
- linux/repartition_disks.sh
- gcp/create_ai_projects.sh
- gcp/delete_all_projects.sh
- common/demo_usage.sh

## 优势

- ✅ 避免 process substitution 的兼容性问题
- ✅ 与 `set -u` 严格模式完全兼容
- ✅ 显式的临时文件管理，更易理解
- ✅ 确保所有分支都正确清理临时文件
- ✅ 保持 curl/wget 双重支持不变
 2025-12-26 15:20:51 +08:00
Wang Defa
7d04ad7532 fix: 彻底修复 FIFO 清理问题（避免 trap 冲突） 
## 问题根源

存在**两个 EXIT trap 冲突**：
1. `oci/create_instance.sh:364` - 清理 FIFO 的 trap
2. `common/remote_loader.sh:23` - 清理临时目录的 trap

在 Bash 中，每次设置新的 trap 会**覆盖**之前的 trap。
当 remote_loader.sh 加载时，它的 `trap cleanup_remote_libs EXIT`
会覆盖 main 函数中设置的 FIFO 清理 trap。

## 错误表现

```
/dev/fd/63: line 1: fifo: unbound variable
```

- FIFO 文件不会被清理（因为 trap 被覆盖）
- Process substitution 退出时产生 trap 交互错误

## 修复方案

**移除 trap，改用显式清理：**

```bash
# 旧方案（有问题）
trap "rm -f \"$fifo\"" EXIT
configure_network "$ocid" "$fifo" &
read subnet_id < "$fifo"

# 新方案（可靠）
configure_network "$ocid" "$fifo" &
local bg_pid=$!           # 保存后台进程 PID
read subnet_id < "$fifo"  # 读取结果
wait "$bg_pid"            # 等待后台进程完成
rm -f "$fifo"             # 手动清理 FIFO
```

## 优势

- ✅ 避免 trap 冲突
- ✅ 显式的清理流程，更易理解
- ✅ 确保后台任务完成后再清理
- ✅ 与 remote_loader.sh 的 trap 兼容
 2025-12-26 15:14:07 +08:00
Wang Defa
b0f1d5d600 fix: 修复 OCI 脚本中的 trap 变量作用域问题 
## 问题描述

脚本执行完成后出现错误：
```
/dev/fd/63: line 1: fifo: unbound variable
```

## 根本原因

在 oci/create_instance.sh:364 行使用了：
```bash
trap 'rm -f "$fifo"' EXIT
```

单引号导致 `$fifo` 在 EXIT 时才展开，但此时 `fifo` 是 local 变量，
已超出作用域。由于 `set -u` 严格模式，访问未定义变量会报错。

## 修复方案

将单引号改为双引号：
```bash
trap "rm -f \"$fifo\"" EXIT
```

这样 `$fifo` 在 trap 设置时就会展开为实际的文件路径（如 /tmp/tmp.xxx），
trap 命令会记录具体路径而非变量名，避免作用域问题。

## 影响

- ✅ 修复 EXIT trap 的 unbound variable 错误
- ✅ 保持功能不变（临时 FIFO 文件仍会正确清理）
- ✅ 与 set -euo pipefail 严格模式兼容
 2025-12-26 15:09:49 +08:00
8 changed files with 134 additions and 46 deletions
Expand all files Collapse all files

15
common/demo_usage.sh
View File

@@ -48,20 +48,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2

6
common/remote_loader.sh
View File

@@ -6,8 +6,10 @@
# 版本: 1.0.0
# ============================================================================
# 远程仓库 URL 配置
readonly REMOTE_BASE_URL="${REMOTE_LIB_URL:-https://gitea.bcde.io/wangdefa/tools/raw/branch/main}"
# 远程仓库 URL 配置（如果调用者未定义，则使用默认值）
if [[ -z "${REMOTE_BASE_URL:-}" ]]; then
readonly REMOTE_BASE_URL="${REMOTE_LIB_URL:-https://gitea.bcde.io/wangdefa/tools/raw/branch/main}"
fi
# 临时目录用于存储下载的文件
readonly REMOTE_TMP_DIR="$(mktemp -d)"

15
gcp/create_ai_projects.sh
View File

@@ -48,20 +48,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2

15
gcp/delete_all_projects.sh
View File

@@ -49,20 +49,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2

15
linux/create_raid0_array.sh
View File

@@ -49,20 +49,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2

15
linux/install_oh_my_zsh.sh
View File

@@ -48,20 +48,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2

15
linux/repartition_disks.sh
View File

@@ -49,20 +49,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2

28
oci/create_instance.sh
View File

@@ -48,20 +48,31 @@ load_common_libs() {
# 使用远程库
if [[ "$use_remote" == "true" ]]; then
# 下载到临时文件（避免 process substitution 与 set -u 的交互问题）
local temp_loader
temp_loader=$(mktemp)
if command -v curl &>/dev/null; then
echo "[INFO] 使用 curl 下载远程库..." >&2
if curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" -o "$temp_loader" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(curl -fsSL "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
elif command -v wget &>/dev/null; then
echo "[INFO] 使用 wget 下载远程库..." >&2
if wget -qO "$temp_loader" "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null; then
# shellcheck disable=SC1090
if source <(wget -qO- "${REMOTE_BASE_URL}/common/remote_loader.sh" 2>/dev/null); then
if source "$temp_loader"; then
rm -f "$temp_loader"
return 0
fi
fi
fi
rm -f "$temp_loader"
echo "[ERROR] 无法加载公共库" >&2
echo "[ERROR] - 本地库不存在" >&2
echo "[ERROR] - 远程下载失败（需要 curl 或 wget）" >&2
@@ -77,7 +88,7 @@ load_common_libs
readonly DEFAULT_NUMBER=1
readonly DEFAULT_SHAPE="VM.Standard.A1.Flex"
readonly DEFAULT_SHAPE_CONFIG="1+6"
readonly DEFAULT_IMAGE_NAME="Canonical-Ubuntu-20.04-aarch64-2025.05.20-0"
readonly DEFAULT_IMAGE_NAME="Canonical-Ubuntu-20.04-aarch64-2025.07.23-0"
readonly DEFAULT_BOOT_VOLUME_SIZE=0
readonly DEFAULT_VPU=120
readonly DEFAULT_DOMAIN=0
@@ -361,9 +372,18 @@ main() {
local fifo
fifo=$(mktemp -u)
mkfifo "$fifo"
trap 'rm -f "$fifo"' EXIT
# 启动后台任务
configure_network "$ocid" "$fifo" &
local bg_pid=$!
# 读取结果
read subnet_id < "$fifo"
# 等待后台任务完成并清理
wait "$bg_pid"
rm -f "$fifo"
log_success "网络配置完成 (子网ID: $subnet_id)"
else
log_info "使用指定的子网ID: $subnet_id"