Skip to content

Commit

Permalink
Refs #19048: Use eprosima-ci for python packages
Browse files Browse the repository at this point in the history
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
  • Loading branch information
EduPonz committed Nov 30, 2023
1 parent 6225186 commit 0c26132
Show file tree
Hide file tree
Showing 5 changed files with 48 additions and 47 deletions.
19 changes: 0 additions & 19 deletions .github/actions/install-python-packages/action.yml

This file was deleted.

13 changes: 9 additions & 4 deletions .github/workflows/documentation-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,15 @@ on:

push:
branches:
- 'master'
- '2.6.x'
paths-ignore:
- '**.md'
- '**.txt'
- '!**/CMakeLists.txt'

pull_request:
branches:
- 'master'
- '2.6.x'
paths-ignore:
- '**.md'
- '**.txt'
Expand Down Expand Up @@ -93,9 +93,14 @@ jobs:
path: src/fastdds-docs
ref: ${{ env.ACTION_BRANCH_NAME }}

- name: Install colcon
uses: eProsima/eProsima-CI/ubuntu/install_colcon@v0

- name: Install required python packages
run: |
pip3 install -r src/fastdds-docs/docs/requirements.txt
uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
with:
upgrade: false
requirements_file_name: src/fastdds-docs/docs/requirements.txt

- name: Build documentation
run: |
Expand Down
12 changes: 9 additions & 3 deletions .github/workflows/reusable-windows-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -163,10 +163,16 @@ jobs:
exit 1
}
- name: Install colcon and other python packages
run: |
pip3 install -U colcon-common-extensions vcstool colcon-mixin xmlschema
- name: Install colcon
uses: eProsima/eProsima-CI/windows/install_colcon@v0

- name: Install Python dependencies
uses: eProsima/eProsima-CI/windows/install_python_packages@v0
with:
packages: vcstool xmlschema

- name: Patch colcon for issue with VS2022
run: |
# patch colcon issue with visual studio 2022
$patch = gci -Include build.py -Recurse `
-Path ((gcm colcon).source | Split-Path | Join-Path -ChildPath "..\Lib\site-packages\colcon_cmake\*")
Expand Down
34 changes: 20 additions & 14 deletions .github/workflows/sanitizer-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,21 @@ jobs:
FASTDDS_BRANCH: ${{ github.head_ref || github.event.inputs.fastdds_branch || 'master' }}

steps:
- uses: eProsima/eProsima-CI/ubuntu/install_apt_packages@v0
- name: Install wget
uses: eProsima/eProsima-CI/ubuntu/install_apt_packages@v0
with:
packages: wget

- uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
- name: Install colcon
uses: eProsima/eProsima-CI/ubuntu/install_colcon@v0

- name: Install Python dependencies
uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
with:
packages: vcstool
packages: vcstool setuptools gcovr tomark xmltodict jsondiff pandas

- uses: eProsima/eProsima-CI/ubuntu/get_file_from_repo@v0
- name: Get fastrtps.repos file
uses: eProsima/eProsima-CI/ubuntu/get_file_from_repo@v0
with:
source_repository: eProsima/Fast-DDS
source_repository_branch: ${{ env.FASTDDS_BRANCH }}
Expand All @@ -60,9 +66,6 @@ jobs:
- name: Install apt packages
uses: ./src/fastrtps/.github/actions/install-apt-packages

- name: Install Python packages
uses: ./src/fastrtps/.github/actions/install-python-packages

- name: Install GTest
uses: eProsima/eProsima-CI/ubuntu/install_gtest@v0
with:
Expand Down Expand Up @@ -113,15 +116,21 @@ jobs:
DEFAULT_DISCOVERY_SERVER_BRANCH: ${{ github.event.inputs.discovery_server_branch || 'v1.2.1' }}

steps:
- uses: eProsima/eProsima-CI/ubuntu/install_apt_packages@v0
- name: Install wget
uses: eProsima/eProsima-CI/ubuntu/install_apt_packages@v0
with:
packages: wget

- uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
- name: Install colcon
uses: eProsima/eProsima-CI/ubuntu/install_colcon@v0

- name: Install Python dependencies
uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
with:
packages: vcstool
packages: vcstool setuptools gcovr tomark xmltodict jsondiff pandas

- uses: eProsima/eProsima-CI/ubuntu/get_file_from_repo@v0
- name: Get fastrtps.repos file
uses: eProsima/eProsima-CI/ubuntu/get_file_from_repo@v0
with:
source_repository: eProsima/Fast-DDS
source_repository_branch: ${{ env.FASTDDS_BRANCH }}
Expand Down Expand Up @@ -149,9 +158,6 @@ jobs:
- name: Install apt packages
uses: ./src/fastrtps/.github/actions/install-apt-packages

- name: Install Python packages
uses: ./src/fastrtps/.github/actions/install-python-packages

- name: Install GTest
uses: eProsima/eProsima-CI/ubuntu/install_gtest@v0
with:
Expand Down
17 changes: 10 additions & 7 deletions .github/workflows/thread-sanitizer.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,15 +54,21 @@ jobs:
CXX: g++-12

steps:
- uses: eProsima/eProsima-CI/ubuntu/install_apt_packages@v0
- name: Install wget
uses: eProsima/eProsima-CI/ubuntu/install_apt_packages@v0
with:
packages: wget

- uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
- name: Install colcon
uses: eProsima/eProsima-CI/ubuntu/install_colcon@v0

- name: Install Python dependencies
uses: eProsima/eProsima-CI/ubuntu/install_python_packages@v0
with:
packages: vcstool
packages: vcstool setuptools gcovr tomark xmltodict jsondiff pandas

- uses: eProsima/eProsima-CI/ubuntu/get_file_from_repo@v0
- name: Get fastrtps.repos file
uses: eProsima/eProsima-CI/ubuntu/get_file_from_repo@v0
with:
source_repository: eProsima/Fast-DDS
source_repository_branch: ${{ env.FASTDDS_BRANCH }}
Expand All @@ -83,9 +89,6 @@ jobs:
- name: Install apt packages
uses: ./src/fastrtps/.github/actions/install-apt-packages

- name: Install Python packages
uses: ./src/fastrtps/.github/actions/install-python-packages

- name: Install GTest
uses: eProsima/eProsima-CI/ubuntu/install_gtest@v0
with:
Expand Down

0 comments on commit 0c26132

Please sign in to comment.