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

setup_nfs_provisioner not fully automated #50

Open
manojnkumar opened this issue Oct 1, 2021 · 1 comment · May be fixed by #51
Open

setup_nfs_provisioner not fully automated #50

manojnkumar opened this issue Oct 1, 2021 · 1 comment · May be fixed by #51
Assignees

Comments

@manojnkumar
Copy link

Here are a few issues with the this utils/setup_nfs_provisioner.sh script:

a) Does not install unzip with is required.
b) Does not pull a version of yq for ppc64le
c) yq fails to parse deployment.yaml

./setup_nfs_provisioner.sh: line 82: unzip: command not found
./setup_nfs_provisioner.sh: line 84: cd: external-storage-master/nfs-client/: No such file or directory
INFO: generating deployment.yaml for ppc64le...
./setup_nfs_provisioner.sh: line 107: deploy/deployment-ppc64le.yaml: No such file or directory
ERROR: failed to update deployment.yaml -- aborting...

After installing unzip and yq:

[root@rdr-rhop-f5d9a8-mon01-bastion-0 utils]# ./setup_nfs_provisioner.sh  /home/nfsshare  192.168.0.204
./setup_nfs_provisioner.sh: line 29: [: argument expected
INFO: proceeding with making this node as NFS server...
Last metadata expiration check: 1 day, 1:09:33 ago on Thu 30 Sep 2021 01:19:56 PM UTC.
Package nfs-utils-1:2.3.3-41.el8_4.2.ppc64le is already installed.
Dependencies resolved.
Nothing to do.
Complete!
Warning: ALREADY_ENABLED: nfs
success
Warning: ALREADY_ENABLED: rpc-bind
success
Warning: ALREADY_ENABLED: mountd
success
success
INFO: downloading and extracting external-storage zip...
INFO: generating deployment.yaml for ppc64le...
Error: unknown command "write" for "yq"
Run 'yq --help' for usage.
ERROR: failed to update deployment.yaml -- aborting...
@miyamotoh
Copy link
Collaborator

Thanks @manojnkumar for reporting this. The script was first written with yq v3, and it has since released v4 which is incompatible with v3. Time to revisit and make necessary fixes that'd work with yq v4.

@miyamotoh miyamotoh self-assigned this Oct 5, 2021
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 a pull request may close this issue.

2 participants