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
Fix Multi Cluster Deployment in Devspace. #8610
Conversation
added a new tox task to list port mapping in multi cluster deployments
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.
Thank you for accommodating the changes @rasswanth-s. This looks much neater now!
scripts/k8s/launch_enclave.sh
Outdated
bash -c '\ | ||
export CLUSTER_NAME=testenclave1 CLUSTER_HTTP_PORT=9083 DEVSPACE_PROFILE=enclave && \ | ||
tox -e dev.k8s.start -- --volume /sys/kernel/security:/sys/kernel/security --volume /dev/tmprm0:/dev/tmprm0 && \ | ||
tox -e dev.k8s.hotreload' |
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'm thinking since you're creating these scripts - might as well make them tox commands?
tox -e dev.k8s.launch.domain
tox -e dev.k8s.launch.gateway
tox -e dev.k8s.launch.enclave
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.
That's a good idea , adding them now 👍
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.
Resolved in 2c15bd0
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 have added a posargs to tox, so we could easily switch between deploy and hotreload mode.
Domain in Deploy Mode
tox -e dev.k8s.launch.domain
Domain in Hot Reload Mode
tox -e dev.k8s.launch.domain -- hotreload
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.
awesome work
Description
The PR fixes
Affected Dependencies
Devspace deployments.
How has this been tested?
Checklist