Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add lima template for coder #2452

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 6 commits into from
Jun 17, 2022
Merged

add lima template for coder #2452

merged 6 commits into from
Jun 17, 2022

Conversation

johnstcn
Copy link
Member

@johnstcn johnstcn commented Jun 17, 2022

This PR adds a lima template for Coder.

You can now run limactl start --name=coder ./examples/lima/coder.yaml and have a "prod-like" Coder instance up and running within a minute or so.

@johnstcn johnstcn self-assigned this Jun 17, 2022
@johnstcn johnstcn marked this pull request as ready for review June 17, 2022 14:02
@johnstcn johnstcn requested review from a team and bpmct June 17, 2022 14:02
Copy link
Member

@mafredri mafredri 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 so awesome! Makes me want to install lima just to try it out. 😆


# Lima VM Template

This provides a [Lima](https://github.com/lima-vm/lima) template for Coder.
Copy link
Member

Choose a reason for hiding this comment

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

Suggestion: We might not want to call this "template" to avoid confusion with actual coder templates (and examples/templates directory). Maybe "sample" or just remove?

export DEBIAN_FRONTEND=noninteractive
apt-get update -y
apt-get install terraform=1.1.9
apt-mark hold terraform
Copy link
Member

Choose a reason for hiding this comment

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

jiffy

All Done! Your Coder instance is accessible at http://localhost:3000

Username: "[email protected]"
Password: `LIMA_INSTANCE=coder lima cat /home/${USER}.linux/.config/coderv2/password` 🤫
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Password: `LIMA_INSTANCE=coder lima cat /home/${USER}.linux/.config/coderv2/password` 🤫
Password: $(LIMA_INSTANCE=coder lima cat /home/${USER}.linux/.config/coderv2/password) 🤫

Minor suggestion, $() is the preferred form, but I didn't test if it works in this scenario 😅.

Copy link
Member Author

@johnstcn johnstcn Jun 17, 2022

Choose a reason for hiding this comment

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

Nope, it doesn't execute in the output, it's more of an indication for the user to run it 😅
I'll change that wording a bit so it's clearer!

@johnstcn johnstcn enabled auto-merge (squash) June 17, 2022 18:33
@johnstcn johnstcn merged commit 0b6efce into main Jun 17, 2022
@johnstcn johnstcn deleted the cj/lima-template branch June 17, 2022 20:12
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.

2 participants