-
Notifications
You must be signed in to change notification settings - Fork 169
Conversation
Hi @friism, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution! TTYL, MSOTBOT; |
Thanks for PR! Build is failing at the moment, can you please take a look? Also there shouldn't be two READMEs, please remove the one in beta2. We just need to keep one readme next to a Dockerfile so that Hub can generate a description from readme file. In this case we decided to keep it next to latest. it'll be fine if you just remove the old one in beta2. Can you also squash the commits into one please? |
@ahmetalpbalkan for some reason, the two sample apps are locked to beta1. I've sent an update to update them to beta3: dotnet/aspnetcore#348 |
@msotclas I've gotten the CLA signed, thanks! |
@ahmetalpbalkan commits squashed, beta2 README removed. |
LGTM. I'll merge once samples are fixed. I guess there's no need to hurry at this point. |
I had to change the last line of the docker file to:
|
@friism sounds about right. I remember you using /runtimes/.. at some point? |
Lemme take a look |
Oh sorry that was the nightly PR #36. |
@ahmetalpbalkan beta3 runtimes are now in ~/.k/runtimes/.. |
@akoeplinger thanks. We need to find out if it respects the install path we provide. |
Fix pushed, thanks @JontyMC |
@friism the samples are updated however it's now failing with a different error. (output) can you please check?
|
@friism it looks like Home/samples are updated with beta3 stuff and now 1.0.0-beta1 imaage cannot run these? https://circle-artifacts.com/gh/aspnet/aspnet-docker/36/artifacts/0/home/ubuntu/aspnet-docker/container-logs/ee6864cee948.log can you please check if this is the case, if so we can probably discontinue the old versions. |
@ahmetalpbalkan taking a look now |
@ahmetalpbalkan done, crossing my fingers for build :-) |
looking |
Weird, works fine on my local machine with master checkout of the home samples. Checking the circle crap. |
@ahmetalpbalkan I don't get it, the CircleCI errors are not like what you post. Where are you getting that? I'm looking at https://circleci.com/gh/aspnet/aspnet-docker/39 |
@ahmetalpbalkan the log that you're posting, which of the sample project is that? |
@ahmetalpbalkan is there any public chat/IRC place where you consistently hang out? Like vnext Jabbr? https://jabbr.net/#/rooms/AspNetvNext |
alright, trying to build the images the way that you do: https://github.com/aspnet/aspnet-docker/blob/master/.ci/build-app-image.sh#L14 |
@friism I grabbed the container id from CircleCI output logs, clicked on Artifacts on CircleCI and you can see that all containers have log files over there. We save container logs in each build so that we can troubleshoot the problem like this. you can find me on irc.freenode.net 7/24 (ahmetalpbalkan). |
I've looked into this problem and it appears that the version tags doesn't match with the versions in the project.json of samples. They are still on beta1 in the beta3 tag. My suggestion is to create a sample application within this repository. This repository will be less dependent on what is happening in the Home repository. |
Ya, it looks like they didn't move the beta3 tag: https://github.com/aspnet/Home/releases |
@friism @vlesierse is the project.json the only problem? If so, just like we inject Dockerfiles, we can "sed -i" the versions as well. I'm not really a big fan of maintaining samples. :-( |
@ahmetalpbalkan for this particular case it is. But there is not guarantee for breaking code changes in the future. I've tested to change the project.json and it is sufficient. |
@vlesierse feel free to fork or create another samples repo if you like. we can refer to your repo in the CI tests. looks like there's no other way we can handle this very well until we have a stable release. |
I have forked the |
@vlesierse feel free to send the PR. 😄 it's better when proposal comes from you. |
It feels weird to be relying on samples hosted in @vlesierse's account, but up to you guys. @ahmetalpbalkan what about factoring the tests into a separate repo that has the (non-broken) samples? |
I agree that this isn't sustainable solution for the future. I will at least create an issue to the |
Do we just need to retag the home branch with beta3 working samples? |
@davidfowl looks like it. |
If the v1.0.0-beta3 tag points to commit |
@feiling Tests are passing now. Merging and updating Docker Hub. |
💃 |
Great |
No description provided.