Skip to content

update image tag to v2 in yaml #1034

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

Merged
merged 1 commit into from
Sep 30, 2019
Merged

update image tag to v2 in yaml #1034

merged 1 commit into from
Sep 30, 2019

Conversation

yeya24
Copy link
Contributor

@yeya24 yeya24 commented Sep 26, 2019

Signed-off-by: yeya24 [email protected]

Describe in detail the problem you had and how this PR fixes it

Update image tag to v2

Is there an open issue you can link to?

Copy link
Contributor

@sr229 sr229 left a comment

Choose a reason for hiding this comment

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

I don't get the point of this PR. If this was having problems with deploying you may not have exposed the port properly. We use 8080 by default. This will not work.

@yeya24
Copy link
Contributor Author

yeya24 commented Sep 27, 2019

I don't get the point of this PR. If this was having problems with deploying you may not have exposed the port properly. We use 8080 by default. This will not work.

So can you deploy code-server in your k8s environment with the default yaml? I have tried it and it failed.

@yeya24
Copy link
Contributor Author

yeya24 commented Sep 27, 2019

Error:

➜  examples git:(master) kp service/code-server -n code-server 8080:8080
Forwarding from 127.0.0.1:8080 -> 8080
Forwarding from [::1]:8080 -> 8080
Handling connection for 8080
Handling connection for 8080
E0926 23:17:03.453318    9495 portforward.go:400] an error occurred forwarding 8080 -> 8080: error forwarding port 8080 to pod 5e4dc4bd6c1413afe3fe56f04a70311b0c79da2746bc69c4146a0d61d47e10ab, uid : exit status 1: 2019/09/27 03:17:02 socat[5168] E connect(5, AF=2 127.0.0.1:8080, 16): Connection refused
E0926 23:17:03.453510    9495 portforward.go:400] an error occurred forwarding 8080 -> 8080: error forwarding port 8080 to pod 5e4dc4bd6c1413afe3fe56f04a70311b0c79da2746bc69c4146a0d61d47e10ab, uid : exit status 1: 2019/09/27 03:17:02 socat[5169] E connect(5, AF=2 127.0.0.1:8080, 16): Connection refused
Handling connection for 8080
Handling connection for 8080
E0926 23:17:03.460688    9495 portforward.go:400] an error occurred forwarding 8080 -> 8080: error forwarding port 8080 to pod 5e4dc4bd6c1413afe3fe56f04a70311b0c79da2746bc69c4146a0d61d47e10ab, uid : exit status 1: 2019/09/27 03:17:02 socat[5170] E connect(5, AF=2 127.0.0.1:8080, 16): Connection refused
E0926 23:17:03.461991    9495 portforward.go:400] an error occurred forwarding 8080 -> 8080: error forwarding port 8080 to pod 5e4dc4bd6c1413afe3fe56f04a70311b0c79da2746bc69c4146a0d61d47e10ab, uid : exit status 1: 2019/09/27 03:17:02 socat[5171] E connect(5, AF=2 127.0.0.1:8080, 16): Connection refused
Handling connection for 8080
Handling connection for 8080

Then check about the logs of code-server pod:

➜  examples git:(master) ✗ kg po -n code-server
NAME                           READY   STATUS    RESTARTS   AGE
code-server-79dccb9f86-l96m7   1/1     Running   0          92s
➜  examples git:(master) ✗ klo code-server-79dccb9f86-l96m7 -n code-server
(node:6) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
INFO  code-server v1.1156-vsc1.33.1
INFO  Additional documentation: http://github.com/cdr/code-server
INFO  Initializing {"data-dir":"/home/coder/.local/share/code-server","extensions-dir":"/home/coder/.local/share/code-server/extensions","working-dir":"/home/coder/project","log-dir":"/home/coder/.cache/code-server/logs/20190927031909135"}
INFO  Starting shared process [1/5]...
INFO  Starting webserver... {"host":"0.0.0.0","port":8443}
WARN  No certificate specified. This could be insecure.
WARN  Documentation on securing your setup: https://github.com/cdr/code-server/blob/master/doc/security/ssl.md
INFO
INFO  Password: e712bb1d04007e245a645bdd
INFO
INFO  Started (click the link below to open):
INFO  https://localhost:8443/
INFO
WARN  stderr {"data":"(node:19) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.\n"}
INFO  Connected to shared process

Code-server is running in 8443 instead of 8080

@sr229
Copy link
Contributor

sr229 commented Sep 27, 2019

Unfortunately, it seems that latest is v1, but we written all of the docs for v2. Please edit your local copy to a v2 tag and run it again. v1 is a deprecated version.

@yeya24
Copy link
Contributor Author

yeya24 commented Sep 27, 2019

Unfortunately, it seems that latest is v1, but we written all of the docs for v2. Please edit your local copy to a v2 tag and run it again. v1 is a deprecated version.

OK. So I will update the tag in this PR.

@yeya24 yeya24 changed the title fix ports in yaml update image tag to v2 in yaml Sep 28, 2019
Copy link
Contributor

@sr229 sr229 left a comment

Choose a reason for hiding this comment

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

This is it chief

@code-asher code-asher merged commit d7d3368 into coder:master Sep 30, 2019
@yeya24 yeya24 deleted the fix-port branch September 30, 2019 16:19
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.

3 participants