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
Add ability for worker to bind to local docker socket for testing exports #112
Conversation
Thanks for the pull request! Here is what will happen next:
Thank you for contributing! |
034c594
to
e01ad2c
Compare
e01ad2c
to
4be999c
Compare
4be999c
to
40fe5bd
Compare
@@ -145,7 +145,12 @@ impl<'a> DockerExporter<'a> { | |||
); | |||
cmd.env(DOCKER_HOST_ENVVAR, sock); // Use the job-specific `dockerd` | |||
cmd.stdout(Stdio::piped()); | |||
cmd.stderr(Stdio::piped()); | |||
match env::var_os(DEV_MODE) { | |||
Some(_) => debug!("Not exporting in dev mode"), |
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.
Sneaky :) Maybe put in a comment that this part is a workaround for now until we solve the streaming log problem correctly?
@@ -104,7 +104,12 @@ impl<'a> Studio<'a> { | |||
} | |||
} | |||
cmd.stdout(Stdio::piped()); | |||
cmd.stderr(Stdio::piped()); | |||
match env::var_os(DEV_MODE) { | |||
Some(_) => debug!("Studio not displaying stderr"), |
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.
Same as above
40fe5bd
to
6ce40d8
Compare
@@ -97,43 +97,46 @@ impl<'a> Studio<'a> { | |||
cmd.env("PATH", env::var("PATH").unwrap_or(String::from(""))); // Sets `$PATH` | |||
cmd.env(NONINTERACTIVE_ENVVAR, "true"); // Disables progress bars | |||
cmd.env("TERM", "xterm-256color"); // Emits ANSI color codes | |||
// propagate debugging environment variables into Airlock and Studio | |||
// propagate debugging environment variables into Airlock and Studio |
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.
Is the spacing intentional?
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.
rustfmt did that
…orts Signed-off-by: Travis Elliott Davis <edavis@chef.io>
6ce40d8
to
177202e
Compare
Closes: #117
Signed-off-by: Travis Elliott Davis edavis@chef.io