Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Update the Longhorn document in Using the Environment Check Script #4450

Closed
roger-ryao opened this issue Aug 19, 2022 · 4 comments
Closed
Assignees
Labels
kind/bug kind/doc Doc request severity/4 Function working but has a minor issue (a minor incident with low impact)
Milestone

Comments

@roger-ryao
Copy link

Describe the bug

After execuetd using-the-environment-check-script , I found out the console output different from the document.

Quick Installation

Example result on the document

daemonset.apps/longhorn-environment-check created
waiting for pods to become ready (0/3)
all pods ready (3/3)
  MountPropagation is enabled!
cleaning up...
daemonset.apps "longhorn-environment-check" deleted
clean up complete

Since v1.3.1, console output result:

[INFO]  Required dependencies are installed.
[INFO]  Waiting for longhorn-environment-check pods to become ready (0/3)...
[INFO]  All longhorn-environment-check pods are ready (3/3).
[INFO]  Required packages are installed.
[INFO]  Cleaning up longhorn-environment-check pods...
[INFO]  Cleanup completed.

To Reproduce

To run script:
curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/v1.4.0/scripts/environment_check.sh | bash

Expected behavior

We need to update the document since 1.3.1+.

Log or Support bundle

N/A

Environment

  • Longhorn version:1.3.1
  • Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Kubectl
  • Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:n/a
    • Number of management node in the cluster:1
    • Number of worker node in the cluster:3
  • Node config
    • OS type and version: ubuntu 20.04
    • CPU per node: 4 cores
    • Memory per node:8GB
    • Disk type(e.g. SSD/NVMe):n/a
    • Network bandwidth between the nodes:n/a
  • Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): KVM
  • Number of Longhorn volumes in the cluster:

Additional context

N/A

@innobead innobead added the kind/doc Doc request label Aug 19, 2022
@innobead innobead added this to the v1.4.0 milestone Aug 19, 2022
@roger-ryao roger-ryao added the severity/4 Function working but has a minor issue (a minor incident with low impact) label Aug 24, 2022
@roger-ryao roger-ryao self-assigned this Oct 28, 2022
@roger-ryao
Copy link
Author

roger-ryao commented Oct 28, 2022

  • 1.2.5
  • 1.2.6-rc1
  • 1.3.1
  • 1.3.2
  • master

@roger-ryao
Copy link
Author

Hi @weizhe0422 :
After I verified it, we lost commit on the 1.3.2 & 1.3.3-dev.
Screenshot_20221028_133053
Screenshot_20221028_133111

@weizhe0422
Copy link
Contributor

Hi @Lichiehyao, the 1.3.2 and 1.3.3 has updated. Please help to verify it. Thanks a lot.

  • v1.3.2
    image
  • v1.3.3
    image

@roger-ryao
Copy link
Author

  • 1.2.5
  • 1.2.6
  • 1.3.1
  • 1.3.2
  • 1.3.3
  • 1.4.0
    Result Passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug kind/doc Doc request severity/4 Function working but has a minor issue (a minor incident with low impact)
Projects
None yet
Development

No branches or pull requests

3 participants