-
Notifications
You must be signed in to change notification settings - Fork 136
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
feat: [TKC-2623] testkube kind Docker image #5867
Conversation
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
wow! should this be in its own repo? could it be possible to pass env/org/agent ids to connect to a control plane? |
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
build/kind/cloud/entrypoint.sh
Outdated
fi | ||
|
||
# Check if cloud url is provided | ||
if [ -z "$CLOUD_URL" ]; then |
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.
Isn't that rather ROOT_DOMAIN
?
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 just followed exsitng agent helm installation command --set testkube-api.cloud.url=agent.testkube.io:443
, will ROOT_DOMAI will be clear for user?
build/kind/cloud/entrypoint.sh
Outdated
log "You can now use Testkube in your Kind Kubernetes cluster." | ||
|
||
# Step 10: Bring docker service back to foreground | ||
fg %1 |
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.
Shouldn't it maybe instead stream Agent + Control Plane logs?
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.
it's just for Agent for now. actually, last part is for demoing purpose, it's not needed for agent work. so user can see that it works. he can always run the shell and check the agent logs
Co-authored-by: Dawid Rusnak <dawid@drcode.pl>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Signed-off-by: Vladislav Sukhin <vladislav@kubeshop.io>
Pull request description
Docker image for running Testkube Agent in Docker (OSS and Cloud)
Checklist (choose whats happened)
Breaking changes
Changes
Fixes