Remove ansible from the base and tools images

A user can still install ansible, ansible-galaxy as they need using pip3. This will reduce the size of the image. ``` pip3 install ansible ansible-galaxy collection install \ azure.azcollection --force -p ./collections ``` Signed-off-by: Suraj Deshmukh <[email protected]>
Azure · Apr 30, 2024 · 7966b0d · 7966b0d
1 parent c1896ea
commit 7966b0d
Show file tree

Hide file tree

Showing 15 changed files with 5 additions and 157 deletions.
diff --git a/linux/ansible/ansible b/linux/ansible/ansible
diff --git a/linux/ansible/ansible-config b/linux/ansible/ansible-config
diff --git a/linux/ansible/ansible-connection b/linux/ansible/ansible-connection
diff --git a/linux/ansible/ansible-console b/linux/ansible/ansible-console
diff --git a/linux/ansible/ansible-doc b/linux/ansible/ansible-doc
diff --git a/linux/ansible/ansible-galaxy b/linux/ansible/ansible-galaxy
diff --git a/linux/ansible/ansible-inventory b/linux/ansible/ansible-inventory
diff --git a/linux/ansible/ansible-playbook b/linux/ansible/ansible-playbook
diff --git a/linux/ansible/ansible-pull b/linux/ansible/ansible-pull
diff --git a/linux/ansible/ansible-vault b/linux/ansible/ansible-vault
diff --git a/linux/base.Dockerfile b/linux/base.Dockerfile
@@ -31,7 +31,6 @@ RUN bash ./tdnfinstall.sh \
   gpgme \
   gnupg2 \
   autoconf \
-  ansible \
   bash-completion \
   build-essential \
   binutils \
@@ -156,14 +155,6 @@ ENV LANG="en_US.utf8"
 RUN pip3 install --upgrade sfctl \
   && pip3 install --upgrade mssql-scripter
 
-# # BEGIN: Install Ansible in isolated Virtual Environment
-COPY ./linux/ansible/ansible*  /usr/local/bin/
-RUN chmod 755 /usr/local/bin/ansible* \
-  && cd /opt \
-  && virtualenv -p python3 ansible \
-  && /bin/bash -c "source ansible/bin/activate && pip3 list --outdated --format=freeze | cut -d '=' -f1 | xargs -n1 pip3 install -U && pip3 install ansible && pip3 install pywinrm\>\=0\.2\.2 && deactivate" \
-  && ansible-galaxy collection install azure.azcollection --force -p /usr/share/ansible/collections
-
 # Install latest version of Istio
 ENV ISTIO_ROOT /usr/local/istio-latest
 RUN curl -sSL https://git.io/getLatestIstio | sh - \

diff --git a/linux/powershell/PSCloudShellUtility/PSCloudShellUtility.psm1 b/linux/powershell/PSCloudShellUtility/PSCloudShellUtility.psm1
@@ -1803,7 +1803,6 @@ function Get-PackageVersion() {
     $packageVersionDetections = @(
         @{displayname = "Node.JS"; command = "node"; args = "--version"; match = "v(.*)"},
         @{displayname = "Cloud Foundry CLI"; command = "cf"; args = "-v"; match = "cf version (.*)"},
-        @{displayname = "Ansible"; command = "ansible"; args = "--version"; match = "ansible \[core ([\d\.]+)\]"},
         @{displayname = "Istio"; command = "istioctl"; args = "version -s --remote=false"; match = "(.+)"},
         @{displayname = "Go"; command = "go"; args = "version"; match = "go version go(\S+) .*"},
         @{displayname = "Packer"; command = "packer"; args = "version"; match = "Packer v(.+)"},

diff --git a/linux/tools.Dockerfile b/linux/tools.Dockerfile
@@ -22,8 +22,8 @@ RUN tdnf clean all && \
 
 # Install any Azure CLI extensions that should be included by default.
 RUN az extension add --system --name ai-examples -y \
-&& az extension add --system --name ssh -y \
-&& az extension add --system --name ml -y
+    && az extension add --system --name ssh -y \
+    && az extension add --system --name ml -y
 
 # Install kubectl
 RUN az aks install-cli \
@@ -49,14 +49,9 @@ RUN npm install -q -g @pnp/cli-microsoft365
 
 # Install Bicep CLI
 RUN curl -Lo bicep https://github.com/Azure/bicep/releases/latest/download/bicep-linux-x64 \
-  && chmod +x ./bicep \
-  && mv ./bicep /usr/local/bin/bicep \
-  && bicep --help
-
-# Temp: fix ansible modules. Proper fix is to update base layer to use regular python for Ansible.
-RUN mkdir -p /usr/share/ansible/collections/ansible_collections/azure/azcollection/ \
-    && wget -nv -q -O /usr/share/ansible/collections/ansible_collections/azure/azcollection/requirements-azure.txt https://raw.githubusercontent.com/ansible-collections/azure/dev/requirements-azure.txt \
-    && /opt/ansible/bin/python -m pip install -r /usr/share/ansible/collections/ansible_collections/azure/azcollection/requirements-azure.txt
+    && chmod +x ./bicep \
+    && mv ./bicep /usr/local/bin/bicep \
+    && bicep --help
 
 # Copy and run script to Install powershell modules and setup Powershell machine profile
 COPY ./linux/powershell/PSCloudShellUtility/ /usr/local/share/powershell/Modules/PSCloudShellUtility/

diff --git a/tests/PSinLinuxCloudShellImage.Tests.ps1 b/tests/PSinLinuxCloudShellImage.Tests.ps1
@@ -77,12 +77,6 @@ Describe "Various programs installed with expected versions" {
         $paths | Should -Contain "~/.local/bin"
     }
 
-    It "Ansible pwsh has modules" {
-        Test-Path -Path "/usr/share/ansible/collections/ansible_collections/azure/azcollection/" | Should -Be $true
-        $process = Start-Process -FilePath /opt/ansible/bin/python -ArgumentList "-c `"import msrest`"" -Wait -PassThru
-        $process.ExitCode | Should -Be 0
-    }
-
     It "Has various environment vars" {
         $env:AZUREPS_HOST_ENVIRONMENT | Should -Be "cloud-shell/1.0"
     }

diff --git a/tests/command_list b/tests/command_list
@@ -39,17 +39,6 @@ agetty
 alias
 alternatives
 anacron
-ansible
-ansible-config
-ansible-connection
-ansible-console
-ansible-doc
-ansible-galaxy
-ansible-inventory
-ansible-playbook
-ansible-pull
-ansible-test
-ansible-vault
 apparmor_parser
 apparmor_status
 applygnupgdefaults