-
Notifications
You must be signed in to change notification settings - Fork 169
React to changes kvm.sh for nightly an other versions #36
Conversation
vlesierse
commented
Feb 22, 2015
- Use KVM_USER_HOME instead of KRE_USER_HOME
- Change folder from packages to runtimes
Looks like beta 4 breaks the test application..... Good news is that the image builds again. |
@vlesierse if you wanna debug, you can see the failed container's logs here https://circle-artifacts.com/gh/aspnet/aspnet-docker/5/artifacts/0/home/ubuntu/aspnet-docker/container-logs/345773ce8cdb.log |
Thnx. I need to adjust the ci script to checkout the dev branch of the home repo before building the image. This branch has the proper samples to work with the latest dev version. Edit: |
@@ -7,9 +7,11 @@ set -e # Exit immediately upon failure | |||
|
|||
for tag in `.ci/find-tags.sh $1`; do | |||
TAG=${IMAGE}:${tag} | |||
BRANCH=master | |||
if [[ ${tag} == nightly ]]; then BRANCH=dev; fi |
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.
if [[ "$tag" == "nightly" ]]
would be better here.
I agree with the code commends and will change them. Tonight I've tried to get the samples running within the Docker container, but I keep running into this error. Really annoying. |
@vlesierse this looks like something asp.net folks might know. try opening an issue or just head to jabbr.net aspnetvnext channel to ask what's going on. |
Thnx, I've added an issue |
It appeared that the update to beta 3 broke the other versions as well. I had to make the same adjustments to get this to pass the tests. Main reason the change to KVM_USER_HOME and runtimes instead of packages. There are a lot of commits now. I will squash them when it's ok and ready to merge. |
@@ -23,4 +23,6 @@ RUN LIBUV_VERSION=1.0.0-rc2 \ | |||
&& rm -rf /usr/local/src/libuv-$LIBUV_VERSION \ | |||
&& ldconfig | |||
|
|||
ENV PATH $PATH:$KRE_USER_HOME/packages/default/bin | |||
COPY NuGet.Config /root/.config/NuGet/ |
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.
could this be $HOME instead of /root/? We stopped using /root/ a while ago.
@vlesierse it seems like CI is green even on nightly now! that's wonderful. wanna squash your commits into one and git push -f to your branch to update the pr now? |
- Use KVM_USER_HOME instead of KRE_USER_HOME - Change folder from packaged to runtimes
Done |
@vlesierse thanks. do beta1/beta2 images also need to be updated with normally, since we set the env variable at the beginning, $ export A=B
$ echo "env | grep ^A=" | sh
A=B |
No, only for the nightly and maybe for beta 3. The older beta releases downloaded the wrong kvm.sh from the release branch and broke the build. That one is also fixed. I will do the change tomorrow so that it can be merged. |
The actual reason of the I've tried putting the environment variables in front of the This line is actually taken from the ASP.NET 5 Home page. I expect another naming change to |
@vlesierse oh sorry that is my bad. I read KRE_USER_HOME and KVM_USER_HOME as the same things. From beta3 and onwards we must start that environment variable and call it match KRE_USER_HOME. Yes there'll be probably some huge naming changes that's going to break us, I'm excited about that, too. 😂 LGTM. Thanks! |
Fix kvm.sh for nightly and other versions
@ahmetalpbalkan I see the CI build on the master branch is failing, but it looks like something is going wrong internally in ASP.NET/.NET. |
Yeah that's certainly interesting since the same build was passing on PR and I have rebuilt failing build several times now. I can see it's mono internal error but wasn't happening on PR branch. Clearly interesting. |
Here's the failure while starting the app, I'm starting to rebuild your PR CI as well (https://circleci.com/gh/aspnet/aspnet-docker/23) to see if it's a merge-related issue or a dependency changed meanwhile:
|
Seems like the package has been changed over night when the build suddenly fails. The HelloWeb app does succeed. |
@vlesierse any ideas how we can locate this? |
The changesets aren't that spectaculair. |
@vlesierse let's take |
Yes, I totally agree on that. Nightlies intend to break..... |