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

tests: remove --console usage #883

Closed
wants to merge 1 commit into from
Closed

tests: remove --console usage #883

wants to merge 1 commit into from

Conversation

cyphar
Copy link
Member

@cyphar cyphar commented Jun 6, 2016

The previous usage of --console was very misleading, and is not how the
flag should be used by users. In addition, due to other bugs and design
issues, --console does not work in user namespaces. By removing
--console usage in the tests, we can now (in principle) run all of the
tests in user namespaces.

Signed-off-by: Aleksa Sarai asarai@suse.de

@cyphar
Copy link
Member Author

cyphar commented Jun 7, 2016

The failures are because it looks like terminal: false and -d don't play properly together. The process isn't actually detached AFAICS. So we hit the timeout for test runners.

/cc @crosbymichael

@cyphar cyphar mentioned this pull request Jun 7, 2016
46 tasks
@crosbymichael
Copy link
Member

looking

The previous usage of --console was very misleading, and is not how the
flag should be used by users. In addition, due to other bugs and design
issues, --console does not work in user namespaces. By removing
--console usage in the tests, we can now (in principle) run all of the
tests in user namespaces.

Signed-off-by: Aleksa Sarai <asarai@suse.de>
@cyphar cyphar added this to the 1.0.0 milestone Jun 23, 2016
@cyphar
Copy link
Member Author

cyphar commented Sep 12, 2016

Superceded by #1018.

@cyphar cyphar closed this Sep 12, 2016
@cyphar cyphar deleted the tests-remove-console-usage branch September 12, 2016 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants