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

ref(builder): remove fetcher port #96

Merged
merged 1 commit into from
Feb 12, 2016
Merged

Conversation

aledbf
Copy link
Contributor

@aledbf aledbf commented Feb 11, 2016

No description provided.

@@ -21,8 +21,6 @@ spec:
ports:
- containerPort: 2223
name: ssh
- containerPort: 3000
name: fetcher
env:
- name: "EXTERNAL_PORT"
value: "2223"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aledbf let's leave this as-is for now, since it uses the quay.io/deis/builder:2.0.0-alpha image, which still has the fetcher in it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aledbf aledbf closed this Feb 11, 2016
@arschles
Copy link
Member

@aledbf sorry, I didn't mean that you should close it. I believe that the removal in the deis-dev chart is still worthwhile, after deis/builder#171 is merged!

@arschles arschles added the LGTM1 label Feb 12, 2016
@arschles arschles added this to the v2.0-beta1 milestone Feb 12, 2016
@smothiki smothiki added the LGTM2 label Feb 12, 2016
aledbf added a commit that referenced this pull request Feb 12, 2016
ref(builder): remove fetcher port
@aledbf aledbf merged commit b894261 into deis:master Feb 12, 2016
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.

4 participants