-
Notifications
You must be signed in to change notification settings - Fork 26
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
Update devcontainer for v0.12 #23
Conversation
RUN apt-get update && /bin/bash /tmp/library-scripts/docker-in-docker-debian.sh | ||
ENTRYPOINT ["/usr/local/share/docker-init.sh"] | ||
VOLUME [ "/var/lib/docker" ] | ||
CMD ["sleep", "infinity"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This ins't needed. The use of features
for docker-in-docker supersedes this.
RUN wget -O /home/rad-vscode-bicep.vsix https://get.radapp.dev/tools/vscode/stable/rad-vscode-bicep.vsix | ||
RUN wget -O /home/rad-vscode.vsix https://get.radapp.dev/tools/vscode/stable/rad-vscode.vsix | ||
RUN wget -O /home/rad-vscode-bicep.vsix https://radiuspublic.blob.core.windows.net/tools/vscode-extensibility/edge/rad-vscode-bicep.vsix | ||
RUN wget -O /home/rad-vscode.vsix https://radiuspublic.blob.core.windows.net/tools/vscode/edge/rad-vscode.vsix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't have versioned links for these yet. We should do this when we're ready to cut a build.
tutorial/template.bicep
Outdated
environment: app.properties.environment | ||
application: app.id | ||
secrets: { | ||
connectionString: '' // Put your connection string here! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The starter does work so I ripped it out. I'm planning a follow-up here to use k8s extensiblity.
name: 'webapp' | ||
location: 'global' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Locations will all need to be updated to resourceGroup().location
to pass validation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then we should turn off that validation. That's a really misleading thing for us to put in our code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM in terms of the changes to the devcontainer parts, not sure if the follow up for the k8s extensibility is coming in a next commit here or another PR.
I'll follow up on the changes to the bicep file separately. |
@AaronCrawfis - it looks like the bicep validation is failing due to using the old compiler. Is this something that's already being updated or did I do something wrong? |
@Reshrahim @AaronCrawfis - do either of you know of plans to migrate the bicep file that's failing in this PR's validation? or have ideas on whether we need a different version in the repo? |
This thread seems to be the culprit: k3s-io/k3s#4638 |
@AaronCrawfis @jasonviviano - I've also updated the Bicep for the sample, and mongo works now! This has a dependency on this change: radius-project/radius#3165 |
430d208
to
5ad4167
Compare
5ad4167
to
f4775b4
Compare
For now I've manually built and pushed the base image.