Skip to content

chore: update to 3.9.3 and add small fixes for license selection#27485

Merged
cannonpalms merged 1 commit into
mainfrom
pbarnett/install-script-fixes
May 30, 2026
Merged

chore: update to 3.9.3 and add small fixes for license selection#27485
cannonpalms merged 1 commit into
mainfrom
pbarnett/install-script-fixes

Conversation

@peterbarnett03
Copy link
Copy Markdown
Contributor

Updates to 3.9.3 and fixes issues on license selection (plus browser opening)

Copy link
Copy Markdown
Contributor

@praveen-influx praveen-influx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a comment for clarification (or improvement), given it's for 3.9.3 it's probably ok to ship it.

Comment thread install_influxdb.sh
Comment on lines +282 to +283
# Open URL in the user's default browser. Runs synchronously and surfaces
# errors so a failed launch is visible.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to do some "timeout" or something - if I ssh on to my box and run this command it'd possibly block

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'll also probably fail from codex/claude/etc if it requires interactivity.

Copy link
Copy Markdown
Contributor

@cannonpalms cannonpalms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Version bump to 3.9.3 is correct (both INFLUXDB_OSS_VERSION and INFLUXDB_ENT_VERSION, plus the help-text default). The accompanying cleanups look good: consolidating the three duplicated license prompts into prompt_license_type() (re-prompting on invalid input and aborting non-interactively rather than silently defaulting to trial is the safer behavior), the synchronous browser-open with error surfacing, the Explorer 80 → 8080 port fix for influxdb3-ui v1.7.0, and the STARTUP_CHOICE normalization. LGTM.

@cannonpalms cannonpalms merged commit 70335b1 into main May 30, 2026
12 of 13 checks passed
@cannonpalms cannonpalms deleted the pbarnett/install-script-fixes branch May 30, 2026 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants