-
Notifications
You must be signed in to change notification settings - Fork 430
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
Update install test.sh with public documentation install commands #1496
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good to me. I believe you have tested on louhi pipeline
Yes, I've tested on louhi pipeline. |
gpgkey=https://packages.cloud.google.com/yum/doc/yum-key.gpg | ||
https://packages.cloud.google.com/yum/doc/rpm-package-key.gpg | ||
https://packages.cloud.google.com/yum/doc/rpm-package-key.gpg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: remove extra space.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copy pasted the install command from https://cloud.google.com/storage/docs/gcsfuse-install because of which this extra space crept in. 😅 Going ahead with the merge.
…c documentation (#1496)
…c documentation (#1496)
…c documentation (#1496)
…c documentation (#1496)
…c documentation (#1496)
Description
Update install_test.sh with the install commands mentioned here.
Link to the issue in case of a bug fix.
NA
Testing details