fix: default admin user to 'lab', case-insensitive OS detection for iSCSI
Some checks failed
CI/CD / typecheck (push) Failing after 10s
CI/CD / test (push) Failing after 10s
CI/CD / lint (push) Failing after 22s
CI/CD / build (push) Has been skipped
CI/CD / publish-rpm (push) Has been skipped
CI/CD / publish-deb (push) Has been skipped

- Firstboot script defaults admin user to 'lab' instead of bastion's
  config.adminUser (which was 'michal' from host system)
- iSCSI OS detection uses case-insensitive match for 'fedora'

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Michal
2026-04-01 15:13:53 +01:00
parent 6f13e284fd
commit 87e09af941
2 changed files with 3 additions and 2 deletions

View File

@@ -154,7 +154,7 @@ fi
const hostname = request.query.hostname ?? "unknown"; const hostname = request.query.hostname ?? "unknown";
const role = (request.query.role ?? "infra") as Role; const role = (request.query.role ?? "infra") as Role;
const mac = request.query.mac ?? "unknown"; const mac = request.query.mac ?? "unknown";
const user = request.query.user ?? config.adminUser; const user = request.query.user ?? "lab";
const script = renderFirstbootScript({ const script = renderFirstbootScript({
hostname, hostname,

View File

@@ -13,7 +13,8 @@ export const enableIscsi: Operation = async (ctx): Promise<OperationResult> => {
// Install the package (detect distro) // Install the package (detect distro)
const osRelease = await ctx.ssh.exec("cat /etc/os-release", sshOpts(ctx)); const osRelease = await ctx.ssh.exec("cat /etc/os-release", sshOpts(ctx));
const isFedora = osRelease.stdout.includes("fedora") || osRelease.stdout.includes("rhel") || osRelease.stdout.includes("centos"); const osLower = osRelease.stdout.toLowerCase();
const isFedora = osLower.includes("fedora") || osLower.includes("rhel") || osLower.includes("centos");
const pkg = isFedora ? "iscsi-initiator-utils" : "open-iscsi"; const pkg = isFedora ? "iscsi-initiator-utils" : "open-iscsi";
const installCmd = isFedora ? `dnf install -y ${pkg}` : `apt-get install -y ${pkg}`; const installCmd = isFedora ? `dnf install -y ${pkg}` : `apt-get install -y ${pkg}`;