Skip to content

Commit e577ff3

Browse files
authored
Comment out selenium/standalone-chrome container images. (#55812)
1 parent 12e4c64 commit e577ff3

File tree

2 files changed

+24
-2
lines changed

2 files changed

+24
-2
lines changed

src/Components/benchmarkapps/Wasm.Performance/dockerfile

+12-1
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,18 @@ RUN .dotnet/dotnet publish -c Release --no-restore -o /app ./src/Components/benc
2929
RUN chmod +x /app/Wasm.Performance.Driver
3030

3131
WORKDIR /app
32-
FROM selenium/standalone-chrome:124.0 as final
32+
# NOTE: This has been commented out because it is causing our builds to get a build warning
33+
# because we are pulling this container image from docker.io. We should consider whether
34+
# we need this code in our repo at all, and if not remove it.
35+
# If we do need it then we need to get the container image imported into mcr.microsoft.com
36+
#
37+
# I have opened up a PR to do this, however it is not certain we'll be allowed to do this
38+
# and there is further legal/compliance work that needs to be done. In the meantime commenting
39+
# this out should get our builds to green again whilst this issue is resolved.
40+
#
41+
# PR: https://github.com/microsoft/mcr/pull/3232
42+
#
43+
# FROM selenium/standalone-chrome:124.0 as final
3344
COPY --from=build ./app ./
3445
COPY ./exec.sh ./
3546

src/Components/benchmarkapps/Wasm.Performance/local.dockerfile

+12-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,15 @@
1-
FROM selenium/standalone-chrome:latest as final
1+
# NOTE: This has been commented out because it is causing our builds to get a build warning
2+
# because we are pulling this container image from docker.io. We should consider whether
3+
# we need this code in our repo at all, and if not remove it.
4+
# If we do need it then we need to get the container image imported into mcr.microsoft.com
5+
#
6+
# I have opened up a PR to do this, however it is not certain we'll be allowed to do this
7+
# and there is further legal/compliance work that needs to be done. In the meantime commenting
8+
# this out should get our builds to green again whilst this issue is resolved.
9+
#
10+
# PR: https://github.com/microsoft/mcr/pull/3232
11+
#
12+
# FROM selenium/standalone-chrome:latest as final
213

314
ENV StressRunDuration=0
415

0 commit comments

Comments
 (0)