-
Notifications
You must be signed in to change notification settings - Fork 85
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
Expose a NodePort service for the admin server by default #668
Conversation
21eff83
to
042edab
Compare
042edab
to
2233d26
Compare
Addressed the comments @balamurugana |
@@ -129,7 +129,11 @@ function unexport_admin_server() { | |||
} | |||
|
|||
function add_drives() { | |||
echo -e 'ALL\nALL\nYes\n' | ./kubectl-directpv format --api-server localhost:40443 --allowed --force | |||
# Get the nodeport service url |
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.
remove function export_admin_server()
and function unexport_admin_server()
functests/common.sh
Outdated
echo -e 'ALL\nALL\nYes\n' | ./kubectl-directpv format --api-server localhost:40443 --allowed --force | ||
# Get the nodeport service url | ||
url=$(minikube service --namespace=directpv-min-io admin-service --url) | ||
adminServer=${url#"http://"} |
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.
rename as admin_server
functests/common.sh
Outdated
url=$(minikube service --namespace=directpv-min-io admin-service --url) | ||
adminServer=${url#"http://"} | ||
|
||
echo -e 'ALL\nALL\nYes\n' | ./kubectl-directpv format --admin-server "$adminServer" --allowed --force |
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.
use ${admin_server}
2233d26
to
90231d6
Compare
No description provided.