Skip to content

Commit

Permalink
Fix Python Installation version in Github CI. Address failing system …
Browse files Browse the repository at this point in the history
…tests environment issues (#4759)

* Refs #20965: Install a fix python version in ubuntu and windows cis

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20965: Decide Python environment depending on the version

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20965: Prevent Qt installation from setting up python in Ubuntu CI

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20965: Add some comments

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

---------

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
(cherry picked from commit 3b2e781)
  • Loading branch information
Mario-DL authored and mergify[bot] committed May 9, 2024
1 parent 565890a commit 2bcec0c
Show file tree
Hide file tree
Showing 5 changed files with 214 additions and 163 deletions.
8 changes: 8 additions & 0 deletions .github/workflows/reusable-ubuntu-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ jobs:
path: src/fastrtps
ref: ${{ inputs.fastdds-branch }}

- name: Install Fix Python version
uses: eProsima/eProsima-CI/external/setup-python@v0
with:
python-version: '3.11'

- name: Get minimum supported version of CMake
uses: eProsima/eProsima-CI/external/get-cmake@v0
with:
Expand Down Expand Up @@ -114,12 +119,15 @@ jobs:
ref: ${{ steps.get_shapes_demo_branch.outputs.deduced_branch }}

# Required for Shapes Demo
# Do not setup python as it will internally modify the pythonLocation env variable
# and we want to use a fix version
- name: Install Qt
uses: jurplel/install-qt-action@v2.13.0
with:
version: '5.15.2'
dir: '${{ github.workspace }}/qt_installation/'
modules: 'qtcharts'
setup-python: 'false'

- name: Colcon build
continue-on-error: false
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/reusable-windows-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ jobs:
submodules: true
ref: ${{ inputs.fastdds_branch }}

- name: Install Fix Python version
uses: eProsima/eProsima-CI/external/setup-python@v0
with:
python-version: '3.11'

- name: Get minimum supported version of CMake
uses: eProsima/eProsima-CI/external/get-cmake@v0
with:
Expand Down
Loading

0 comments on commit 2bcec0c

Please sign in to comment.