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

Cleanup some tests and documentation #615

Merged
merged 6 commits into from
Nov 4, 2020
Merged

Cleanup some tests and documentation #615

merged 6 commits into from
Nov 4, 2020

Conversation

kylemanna
Copy link
Owner

Localize a lot of the testing by dropping things like --privileged and --net=host to make testing safer and approach of least privilege.

  • Drop --net=host everywhere - this terrorized my dev machine.
  • Drop --privileged this isn't really needed other then messing with sysctl values. Warnings are printed now.
  • Update docs so I never read of these things again.
  • Attempt to tear-down tests slightly better. It's still a mess though.
  • Silence iptables checks.
  • Propagate DEBUG environment variable deeper to help debugging.

* Clean-up processes properly.
* Instead encourage the user to change how they invoke docker.
* It's fine for these to fail, it's expected.
* Fix all the test to not require this flag.
* Fix all the docs to not mention this flag.
* This is really hard to work on other things while running this test
  when the host networking stack is being manipulated, primarily the
  default route.
* Propagate DEBUG flag deeper where possible.
@kylemanna kylemanna self-assigned this Nov 2, 2020
@kylemanna kylemanna changed the title Cleanup some tests and documents Cleanup some tests and documentation Nov 2, 2020
@kylemanna kylemanna merged commit e47f86b into master Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant