URL: https://github.com/freeipa/freeipa/pull/4503
Author: rcritten
 Title: #4503: [Backport][ipa-4-8] Azure: Allow distros to install Python they 
want
Action: opened

PR body:
"""
This PR was opened automatically because PR #4484 was pushed to master and 
backport to ipa-4-8 is required.
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/4503/head:pr4503
git checkout pr4503
From 11e915f1497a30236f84388a5dafe3c30ff7ff30 Mon Sep 17 00:00:00 2001
From: Stanislav Levin <s...@altlinux.org>
Date: Thu, 2 Apr 2020 15:10:20 +0300
Subject: [PATCH] Azure: Allow distros to install Python they want

The platforms may have different Pythons.
But due to [0] the Python installed via the 'UsePythonVersion@0'
task should be compatible with the container's 'libpythonxx.so'.
'AZURE_PYTHON_VERSION' platform variable is introduced to cover
this. So, if your distro has Python3.8, set the mentioned variable
to '3.8', later, this version will be installed by the
'UsePythonVersion@0' Azure task for 'WebUI_Unit_Tests' and 'Tox'
jobs.

To allow tox to run any Python3 environment the 'py3' one is used.
'py3' is the well-known Tox's environment, which utilizes 'python3'
executable.

[0]: https://github.com/microsoft/azure-pipelines-tasks/issues/11070

Fixes: https://pagure.io/freeipa/issue/8254
Signed-off-by: Stanislav Levin <s...@altlinux.org>
---
 ipatests/azure/azure-pipelines.yml            | 6 +++---
 ipatests/azure/templates/variables-fedora.yml | 3 +++
 tox.ini                                       | 2 +-
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/ipatests/azure/azure-pipelines.yml b/ipatests/azure/azure-pipelines.yml
index e35a822bea..b4d5cb2fcd 100644
--- a/ipatests/azure/azure-pipelines.yml
+++ b/ipatests/azure/azure-pipelines.yml
@@ -87,7 +87,7 @@ jobs:
     - template: templates/${{ variables.PREPARE_BUILD_TEMPLATE }}
     - task: UsePythonVersion@0
       inputs:
-        versionSpec: 3.7
+        versionSpec: ${{ variables.AZURE_PYTHON_VERSION }}
         architecture: x64
     - template: templates/${{ variables.PREPARE_TOX_TEMPLATE }}
     - script: |
@@ -96,7 +96,7 @@ jobs:
         export LANG=en_US.utf8
         export LC_CTYPE=en_US.utf8
         locale
-        $(TOX_COMMAND) -e py37,pypi,pylint3
+        $(TOX_COMMAND) -e py3,pypi,pylint3
       displayName: Tox
     - task: PublishTestResults@2
       inputs:
@@ -114,7 +114,7 @@ jobs:
     - template: templates/${{ variables.PREPARE_BUILD_TEMPLATE }}
     - task: UsePythonVersion@0
       inputs:
-        versionSpec: 3.7
+        versionSpec: ${{ variables.AZURE_PYTHON_VERSION }}
         architecture: x64
     - template: templates/${{ variables.PREPARE_WEBUI_TEMPLATE }}
     - script: |
diff --git a/ipatests/azure/templates/variables-fedora.yml b/ipatests/azure/templates/variables-fedora.yml
index d6c95090d3..bec631b873 100644
--- a/ipatests/azure/templates/variables-fedora.yml
+++ b/ipatests/azure/templates/variables-fedora.yml
@@ -15,3 +15,6 @@ variables:
   PREPARE_WEBUI_TEMPLATE: ${{ format('prepare-webui-{0}.yml', variables.IPA_PLATFORM) }}
 
   TOX_COMMAND: tox
+
+  # Python version for UsePythonVersion@0 task
+  AZURE_PYTHON_VERSION: '3.7'
diff --git a/tox.ini b/tox.ini
index 19abffbc80..62b576ea52 100644
--- a/tox.ini
+++ b/tox.ini
@@ -1,6 +1,6 @@
 [tox]
 minversion=2.3.1
-envlist=py36,py37,pylint3,pypi
+envlist=py36,py37,py38,pylint3,pypi
 skip_missing_interpreters=true
 skipsdist=true
 
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/freeipa-devel@lists.fedorahosted.org

Reply via email to