Skip to content

Add DockerFile for ChromeDevTools #39624

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 3 commits into from
Sep 5, 2020

Conversation

TimvdLippe
Copy link
Contributor

@TimvdLippe TimvdLippe commented Jul 16, 2020

Note that I was not able to verify it fully works, as it throws an
authentication error on typescript/typescript on the Docker Hub.

This is part of #39568

CC @weswigham

Note that I was not able to verify it fully works, as it throws an
authentication error on typescript/typescript on the Docker Hub.

This is part of microsoft#39568

CC @weswigham
RUN mkdir /typescript
RUN tar -xzvf /typescript.tgz -C /typescript
RUN ln -s /typescript/package ./node_modules/typescript
RUN autoninja -C out/Default front_end
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For some reason if this last step is a CMD then it doesn't do anything at all. I am not sure why, but the RUN works as expected.

Copy link
Member

Choose a reason for hiding this comment

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

A docker file is split into two phases, build and run. All the RUNs build the FS in the Dockerfile. The CMD is what is executed (by default) when you actually execute the built docker image. We use that here - only the CMD output is baseline'd.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah I see, thanks for the explanation.

WORKDIR /
RUN mkdir devtools
WORKDIR devtools
RUN fetch devtools-frontend
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Jul 16, 2020
@weswigham
Copy link
Member

weswigham commented Jul 16, 2020

typescript/typescript is built locally from the Dockerfile in the repo root before running the docker tests. gulp runtests --ru=docker --t=devtools should work.

@TimvdLippe
Copy link
Contributor Author

I was able to make it run with:

npm run gulp -- runtests --ru=docker --t=chrome-devtools-frontend-next

I am not sure how best to make the error log as useful for you. The CMD now makes sure that into only shows any failures, if there are any. This should remove all noise from Ninja and produce a stable output.

@weswigham
Copy link
Member

@typescript-bot user test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jul 17, 2020

Heya @weswigham, I've started to run the parallelized community code test suite on this PR at 6517bc9. You can monitor the build here.

@TimvdLippe
Copy link
Contributor Author

@weswigham It's been a while since we worked on this PR. Could you let me know what the latest status update for it is?

@weswigham weswigham merged commit 8d7afc2 into microsoft:master Sep 5, 2020
@TimvdLippe TimvdLippe deleted the devtools-docker-file branch September 5, 2020 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants