Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Integrate Coder Cloud Agent #2086

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
nhooyr merged 29 commits into coder:master from nhooyr:master
Oct 9, 2020
Merged

Integrate Coder Cloud Agent #2086

nhooyr merged 29 commits into coder:master from nhooyr:master
Oct 9, 2020

Conversation

@nhooyr
Copy link
Contributor

@nhooyr nhooyr commented Sep 11, 2020

Will allow users the following flow:

  1. Install code-server.
  2. Run code-server.
  3. Run code-server --expose my-name.
  4. Authenticate with GitHub and then access their code-server instance over HTTPS at https://<github-user>-<instance-name>.cdr.dev.

Still a WIP and things above may change. We also need to open source the Go agent.

Sort of an alternative to Microsoft's Self Hosted Azure Codespaces. See #2077

geiseri, kylecarbs, and mahmoudajawad reacted with thumbs up emoji mahmoudajawad reacted with rocket emoji janus-reith reacted with eyes emoji
Copy link
Contributor

geiseri commented Sep 13, 2020

It would be interesting to see this on gitlab too. Might be a better value, than competing against github.

massimocireddu and danielsteiner reacted with thumbs up emoji

Copy link
Contributor

sr229 commented Sep 15, 2020

This might be prone to name collisions as there might be cases you'll want to run it on a seperate system should they choose not to use a container - we should add at least a randomized identifier which is how Codespaces has done it.

Copy link
Contributor Author

nhooyr commented Oct 6, 2020

This might be prone to name collisions as there might be cases you'll want to run it on a seperate system should they choose not to use a container - we should add at least a randomized identifier which is how Codespaces has done it.

You get to pick the name every time. We'd rather have a short URL. If there is a conflict, we can append some random text to the name ourselves @sreya

Copy link
Collaborator

sreya commented Oct 6, 2020

As @nhooyr said you get to pick the name everytime you register a new server, the names won't conflict across different users because it is a unique key composed of the server name and your github username. For now as a user, you cannot register two servers with the same name.

Copy link
Contributor Author

nhooyr commented Oct 7, 2020

It would be interesting to see this on gitlab too. Might be a better value, than competing against github.

We only use GitHub for auth, we can add multiple options down the line.

@nhooyr nhooyr marked this pull request as ready for review October 7, 2020 18:11
@nhooyr nhooyr requested a review from sreya October 7, 2020 18:11
nhooyr added 20 commits October 7, 2020 15:58
We only want to use an old version for glibc which the centos:7
image takes care of.
The old version of git used in debian:8 was causing problems
with the uid/gid passthrough with no user in passwd.
Copy link
Contributor Author

nhooyr commented Oct 9, 2020

Made it clear --coder-bind is in beta and it's not shown by default in help output unless CS_BETA is non empty in the environment.

@nhooyr nhooyr merged commit daa1c86 into coder:master Oct 9, 2020
@nhooyr nhooyr deleted the master branch October 9, 2020 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sreya sreya sreya left review comments

@code-asher code-asher code-asher approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /