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

Remove airlock usage in builder-worker #976

Merged
merged 1 commit into from Mar 13, 2019

Conversation

Projects
None yet
4 participants
@chefsalim
Copy link
Member

chefsalim commented Mar 13, 2019

Now that we have consolidated all the build workers to Docker Studio, we can remove the integration code for Airlock. This change removes the configuration and associated Airlock execution code from the worker. The terraform scripts are updated to no longer create a special user and group for studio builds as well.

Signed-off-by: Salim Alam salam@chef.io

tenor-203575642

Remove airlock usage in builder-worker
Signed-off-by: Salim Alam <salam@chef.io>
@thesentinels

This comment has been minimized.

Copy link
Contributor

thesentinels commented Mar 13, 2019

Thanks for the pull request! Here is what will happen next:

  1. Your PR will be reviewed by the maintainers
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@chefsalim chefsalim requested review from eeyun and raskchanky as code owners Mar 13, 2019

@chefsalim chefsalim requested a review from mwrock Mar 13, 2019

@elliott-davis
Copy link
Contributor

elliott-davis left a comment

@raskchanky

This comment has been minimized.

Copy link
Member

raskchanky commented Mar 13, 2019

@chefsalim chefsalim merged commit 1d77bc5 into master Mar 13, 2019

2 checks passed

DCO This commit has a DCO Signed-off-by line
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@chefsalim chefsalim deleted the SA/expunge-worker-airlock branch Mar 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.