Skip to content

Remove helix script install (partial set of Dockerfiles) #1336

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.

Sign up for GitHub

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

Closed
wants to merge 11 commits into from
Closed

Remove helix script install (partial set of Dockerfiles) #1336

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
65a76f2
Remove helix script install
mthalman Jan 30, 2025
075da16
Add back virtual environments
mthalman Feb 3, 2025
297153d
update python version in Windows dockerfiles
mthalman Feb 3, 2025
ee84975
update Windows Dockerfiles to install latest Python version
mthalman Feb 3, 2025
9e2980f
Update python comment for AlmaLinux
mthalman Feb 4, 2025
f48f7ff
Updates to fix issues with Windows Dockerfiles
mthalman Feb 6, 2025
78d5b6a
Create virtual env as helixbot user
mthalman Feb 11, 2025
b8b75aa
updates to Windows dockerfiles
mthalman Feb 11, 2025
01fd064
Fix setting of env vars
mthalman Feb 11, 2025
52f4b8e
Remove virtualenv installation
mthalman Feb 12, 2025
a36b6ec
Forgot to remove this
mthalman Feb 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions src/almalinux/8/helix/amd64/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,6 @@ RUN dnf upgrade --refresh -y \
file \
&& dnf clean all

RUN python3 -m pip download --no-deps helix-scripts --index-url https://dnceng.pkgs.visualstudio.com/public/_packaging/helix-client-prod/pypi/simple \
&& python3 -m pip install ./helix_scripts-*-py3-none-any.whl

# Aspnetcore test runs expect python to be available without version suffix
RUN alternatives --set python /usr/bin/python3 \
&& ln -sf /usr/bin/pip3 /usr/bin/pip
Expand All @@ -34,7 +31,3 @@ ENV LANG=en-US.UTF-8
RUN adduser --uid 1000 --shell /bin/bash --gid adm helixbot \
&& chmod 755 /root \
&& echo "helixbot ALL=(ALL) NOPASSWD: ALL" > /etc/sudoers

USER helixbot

RUN python3 -m venv /home/helixbot/.vsts-env
26 changes: 0 additions & 26 deletions src/fedora/41/helix/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,23 +1,3 @@
FROM library/fedora:41 AS venv

RUN dnf upgrade --refresh -y \
&& dnf install --setopt tsflags=nodocs -y \
dnf-plugins-core \
&& dnf install --setopt=install_weak_deps=False --setopt tsflags=nodocs -y \
openssl \
python3-devel \
python3-pip \
gcc \
libatomic \
redhat-rpm-config \
&& dnf clean all

RUN python3 -m venv /venv \
&& source /venv/bin/activate \
pip download --no-deps helix-scripts --index-url https://dnceng.pkgs.visualstudio.com/public/_packaging/helix-client-prod/pypi/simple \
&& pip download --no-deps helix-scripts --index-url https://dnceng.pkgs.visualstudio.com/public/_packaging/helix-client-prod/pypi/simple \
&& pip install ./helix_scripts-*-py3-none-any.whl

FROM library/fedora:41

# Install Dependencies
Expand Down Expand Up @@ -64,9 +44,3 @@ RUN /usr/sbin/adduser --uid 1000 --shell /bin/bash --group adm helixbot \
&& chmod 755 /root \
&& echo "helixbot ALL=(ALL) NOPASSWD: ALL" > /etc/sudoers \
&& chmod +s /usr/bin/ping

# Install Helix Dependencies
ENV VIRTUAL_ENV=/home/helixbot/.vsts-env
RUN python3 -m venv $VIRTUAL_ENV
ENV PATH="$VIRTUAL_ENV/bin:$PATH"
COPY --from=venv --chown=helixbot /venv $VIRTUAL_ENV
2 changes: 0 additions & 2 deletions src/nanoserver/1809/helix/amd64/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@ ENV PATH="$PATH;C:\Program Files\PowerShell\;C:\Python;C:\python\scripts"
RUN md c:\\helixtmp && pushd c:\\helixtmp &&`
python -m pip install --upgrade pip==20.2 && `
python -m pip install virtualenv==16.6.0 && `
pip download --no-deps helix-scripts --index-url https://dnceng.pkgs.visualstudio.com/public/_packaging/helix-client-prod/pypi/simple && `
for %f in (.\helix_scripts-*-py3-none-any.whl) do (pip install %f) && `
popd && rd /s /q c:\\helixtmp && `
pwsh -Command `
New-Item -Path 'HKLM:\SOFTWARE\Microsoft\Windows\Windows Error Reporting\LocalDumps\' -Name 'dotnet.exe' -Force -ErrorAction SilentlyContinue ; `
Expand Down
2 changes: 0 additions & 2 deletions src/windowsservercore/ltsc2019/helix/amd64/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ RUN curl -SL --output %TEMP%\python.zip https://www.nuget.org/api/v2/package/pyt
RUN md c:\\helixtmp && pushd c:\\helixtmp &&`
C:\Python\python.exe -m pip install --upgrade pip==20.2 --no-warn-script-location && `
C:\Python\python.exe -m pip install virtualenv==16.6.0 --no-warn-script-location && `
C:\Python\python.exe -m pip download --no-deps helix-scripts --index-url https://dnceng.pkgs.visualstudio.com/public/_packaging/helix-client-prod/pypi/simple && `
for %f in (.\helix_scripts-*-py3-none-any.whl) do (C:\Python\python.exe -m pip install %f --no-warn-script-location) && `
popd && rd /s /q c:\\helixtmp && `
powershell -Command `
New-Item -Path 'HKLM:\SOFTWARE\Microsoft\Windows\Windows Error Reporting\LocalDumps\' -Name 'dotnet.exe' -Force -ErrorAction SilentlyContinue ; `
Expand Down