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

MESOS/CI: call docker-run interactively and with terminal #17842

Merged

Conversation

sttts
Copy link
Contributor

@sttts sttts commented Nov 26, 2015

  • passing -i to enable signal handling in a tty.
  • passing -t to enable passing of the TERM env var.

Alternative to #17748

- passing `-i` to enable signal handling in a tty.
- passing `-t` to enable passing of the TERM env var.
@k8s-github-robot
Copy link

Labelling this PR as size/XS

@k8s-github-robot k8s-github-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 26, 2015
@k8s-bot
Copy link

k8s-bot commented Nov 26, 2015

GCE e2e test build/test passed for commit e795aff.

@karlkfi karlkfi added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 30, 2015
@karlkfi
Copy link
Contributor

karlkfi commented Nov 30, 2015

This is a better solution, thanks.

@k8s-github-robot
Copy link

Continuous integration appears to have missed, closing and re-opening to trigger it

@k8s-github-robot
Copy link

@k8s-bot test this

Tests are more than 48 hours old. Re-running tests.

@k8s-bot
Copy link

k8s-bot commented Nov 30, 2015

GCE e2e test build/test passed for commit e795aff.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

k8s-github-robot pushed a commit that referenced this pull request Nov 30, 2015
@k8s-github-robot k8s-github-robot merged commit 0d7d879 into kubernetes:master Nov 30, 2015
@karlkfi karlkfi deleted the sttts-docker-it branch November 30, 2015 21:14
s-urbaniak pushed a commit to mesosphere-backup/kubernetes that referenced this pull request Dec 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform/mesos area/test area/test-infra lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants