Skip to content
This repository was archived by the owner on May 31, 2019. It is now read-only.

Add beta6 Dockerfile #77

Merged
merged 1 commit into from
Jul 28, 2015
Merged

Add beta6 Dockerfile #77

merged 1 commit into from
Jul 28, 2015

Conversation

yanchenw
Copy link
Contributor

No description provided.

@ahmetb
Copy link
Contributor

ahmetb commented Jul 28, 2015

Hey @yanchenw looks like beta6 is not tagged on Home repo https://github.com/aspnet/Home/tags and thus cannot locate the samples/ directory. Hence the CI is failing.

@Eilon
Copy link
Contributor

Eilon commented Jul 28, 2015

@pranavkm is in charge of tagging releases.

@kichalla
Copy link
Member

Also we need to update the samples and create a directory for beta6 : https://github.com/aspnet/Home/tree/dev/samples

@ahmetb
Copy link
Contributor

ahmetb commented Jul 28, 2015

@kichalla @pranakvm from a CI tests standpoint, we first checkout to the tag (i.e. beta6) on the aspnet/Home repo and then look for the path 1.0.0-beta6. For instance, for beta5, we pick the sample apps from:

https://github.com/aspnet/Home/tree/v1.0.0-beta5/samples/1.0.0-beta5

Therefore, while tagging beta6 over there, please make sure samples/1.0.0-beta6 directory is included in the tag. Last time we had to change the tag after the release to fix this.

@kichalla
Copy link
Member

@ahmetalpbalkan @yanchenw The samples and tag have been created, but not sure how do you kick off the CI run again to verify if all is ok

@ahmetb
Copy link
Contributor

ahmetb commented Jul 28, 2015

I just kicked off another run.

@kichalla
Copy link
Member

I suspect this would run fail as beta6 sample has a Dockerfile already in it (https://github.com/aspnet/Home/blob/dev/samples/1.0.0-beta6/HelloMvc/Dockerfile)...

I think we need to even have docker files for beta4 and beta5 in the samples and then can remove the following code?: https://github.com/aspnet/aspnet-docker/blob/master/.ci/build-app-image.sh#L21-L25

@ahmetb
Copy link
Contributor

ahmetb commented Jul 28, 2015

@kichalla it would overwrite the existing Dockerfile. :-)

Even if you add dockerfiles to the old versions directories, as long as you don't re-tag versions beta4/5 to include this change, it won't work.

We can modify the script to not to overwrite if the file already exists, however we're setting extra stuff like DNU_TRACE in our drop-in Dockerfile, so I think we're good until this breaks.

@kichalla
Copy link
Member

Ah ok, thanks and the build also passed 😄

:shipit:

ahmetb added a commit that referenced this pull request Jul 28, 2015
@ahmetb ahmetb merged commit dce8a6a into aspnet:master Jul 28, 2015
@ahmetb
Copy link
Contributor

ahmetb commented Jul 28, 2015

Docker Hub builds kicked off. We're live if they pass. https://registry.hub.docker.com/u/microsoft/aspnet/builds_history/82542/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants