Movatterモバイル変換


[0]ホーム

URL:


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

example: added docker local workspace#1025

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
kylecarbs merged 1 commit intocoder:mainfromntimo:task/docker-workspace-example
Apr 18, 2022
Merged

example: added docker local workspace#1025

kylecarbs merged 1 commit intocoder:mainfromntimo:task/docker-workspace-example
Apr 18, 2022

Conversation

ntimo
Copy link
Contributor

This PR adds a new example template that can be used with thecoder start --dev mode to provision a workspace container on the local machine.

greyscaled, kylecarbs, coadler, and jsjoeio reacted with hooray emoji
greyscaled added a commit that referenced this pull requestApr 15, 2022
Summary:PRs like#1025 feel like they deserve a doc: type, but we didn't have one.Furthermore our definitions for correct and fix were stale.
Copy link
Contributor

@coadlercoadler left a comment

Choose a reason for hiding this comment

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

┌──────────────────────────────────────────────────────────────────┐│ RESOURCE                    STATUS             ACCESS            │├──────────────────────────────────────────────────────────────────┤│ docker_container.workspace  ephemeral                            ││ └─ dev (linux, amd64)       ⦾ connecting [0s]   coder ssh colin  │├──────────────────────────────────────────────────────────────────┤│ docker_volume.coder_volume  ephemeral                            │└──────────────────────────────────────────────────────────────────┘The colin workspace has been created!colin at coder in ~/Projects/coder/coder/examples/docker-local (task/docker-workspace-example)$ coder ssh colincoder@d982a97c7cac:/$

Works great!

kylecarbs reacted with hooray emoji
@coadler
Copy link
Contributor

Could you try rebasing? I think I pushed a fix to allow actions to run for external prs.

ntimo reacted with thumbs up emoji

@ntimo
Copy link
ContributorAuthor

@coadler seems like some of the tests are failing because of missing env variables. I guess since its a fork the CI job does not have access to the same env vars as the ones in the main repo, which of course is good. But then it would maybe be helpful to somehow exclude these jobs from forks if possible?

@kylecarbs
Copy link
Member

@ntimo this is our first external fork, so we'll be removing those steps from requiring the variables! I'll fix that now.

ntimo reacted with thumbs up emoji

greyscaled added a commit that referenced this pull requestApr 15, 2022
Summary:PRs like#1025 feel like they deserve a doc: type, but we didn't have one.Furthermore our definitions for correct and fix were stale.
@coadler
Copy link
Contributor

Should be fixed now in#1048. Apologies for the delay!

ntimo reacted with thumbs up emoji

@ntimontimo changed the titlefeat: added docker local workspace exampleexample:: added docker local workspaceApr 16, 2022
@ntimontimo changed the titleexample:: added docker local workspaceexample: added docker local workspaceApr 16, 2022
@kylecarbskylecarbs merged commit9faa39a intocoder:mainApr 18, 2022
@kylecarbskylecarbs deleted the task/docker-workspace-example branchApril 18, 2022 15:20
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@coadlercoadlercoadler approved these changes

@kylecarbskylecarbskylecarbs 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.

3 participants
@ntimo@coadler@kylecarbs

[8]ページ先頭

©2009-2025 Movatter.jp