Skip to content

docs(1.9.0): remove environment_check.sh#1069

Merged
derekbit merged 1 commit intolonghorn:masterfrom
derekbit:issue-9239
Mar 11, 2025
Merged

docs(1.9.0): remove environment_check.sh#1069
derekbit merged 1 commit intolonghorn:masterfrom
derekbit:issue-9239

Conversation

@derekbit
Copy link
Copy Markdown
Member

Which issue(s) this PR fixes:

Issue longhorn/longhorn#9239

What this PR does / why we need it:

environment_check.sh is removed since Longhorn v1.9.0 and is replaced by longhornctl.

Special notes for your reviewer:

Additional documentation or context

@derekbit derekbit requested a review from a team as a code owner March 11, 2025 00:25
@derekbit derekbit self-assigned this Mar 11, 2025
@derekbit derekbit requested review from c3y1huang and removed request for a team March 11, 2025 00:25
@derekbit derekbit requested review from PhanLe1010, innobead, mantissahz and shuo-wu and removed request for jhkrug and jillian-maroket March 11, 2025 00:25
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 11, 2025

Deploy Preview for longhornio ready!

Name Link
🔨 Latest commit 9476327
🔍 Latest deploy log https://app.netlify.com/sites/longhornio/deploys/67cfcbb12f47d6000864c5d6
😎 Deploy Preview https://deploy-preview-1069--longhornio.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Copy Markdown
Contributor

@jillian-maroket jillian-maroket left a comment

Choose a reason for hiding this comment

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

Review done

Comment thread content/docs/1.9.0/important-notes/_index.md Outdated
innobead
innobead previously approved these changes Mar 11, 2025
Copy link
Copy Markdown
Member

@innobead innobead left a comment

Choose a reason for hiding this comment

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

LGTM

environment_check.sh is removed since Longhorn v1.9.0 and is
replaced by longhornctl.

Ref: https://longhorn.io/docs/1.9.0/advanced-resources/longhornctl/

Longhorn 9239

Signed-off-by: Derek Su <derek.su@suse.com>
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