Skip to content

Commit a0a6225

Browse files
authored
Remove python venv from workflow (#242)
* Temporary change actions reference for testing Signed-off-by: Raúl <[email protected]> * Fix project dependencies action Signed-off-by: Raúl <[email protected]> * Setup python venv for failing jobs Signed-off-by: Raúl <[email protected]> * Remove python venv from workflow Signed-off-by: Raul Sanchez-Mateos <[email protected]> * Enable github action in PRs Signed-off-by: Raul Sanchez-Mateos <[email protected]> * Update artifact name Signed-off-by: Raul Sanchez-Mateos <[email protected]> * Update input argument call Signed-off-by: Raul Sanchez-Mateos <[email protected]> * Set action branches back to v0 Signed-off-by: Raul Sanchez-Mateos <[email protected]> * Set action branches references to testing branch Signed-off-by: Raul Sanchez-Mateos <[email protected]> * Set action branches references back to main Signed-off-by: Raul Sanchez-Mateos <[email protected]> --------- Signed-off-by: Raúl <[email protected]> Signed-off-by: Raul Sanchez-Mateos <[email protected]>
1 parent b554f95 commit a0a6225

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

.github/actions/project_dependencies/action.yml

+1-5
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,6 @@ runs:
4040
using: composite
4141
steps:
4242

43-
- name: Setup Python Virtual Environment
44-
uses: eProsima/eProsima-CI/multiplatform/setup_python_venv@v0
45-
if: runner.os == 'Linux'
46-
4743
- name: Install Fast DDS dependencies
4844
uses: eProsima/eProsima-CI/multiplatform/install_fastdds_dependencies@v0
4945
with:
@@ -58,7 +54,7 @@ runs:
5854
- name: Download dependencies artifact
5955
uses: eProsima/eProsima-CI/multiplatform/download_dependency@v0
6056
with:
61-
artifact_name: built_fastdds_${{ inputs.custom_version_build }}_${{ inputs.os }}_${{ inputs.cmake_build_type }}${{ inputs.dependencies_artifact_postfix }}
57+
artifact_name: build_fastdds_${{ inputs.custom_version_build }}_${{ inputs.os }}_${{ inputs.cmake_build_type }}${{ inputs.dependencies_artifact_postfix }}
6258
workflow_source: build_fastdds.yml
6359
workflow_source_repository: eProsima/eProsima-CI
6460
target_workspace: ${{ inputs.target_workspace }}

.github/workflows/reusable-workflow.yml

+6-5
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ jobs:
103103
os: ${{ matrix.os }}
104104
cmake_build_type: ${{ matrix.cmake_build_type }}
105105
custom_version_build: ${{ inputs.custom_version_build }}
106-
dependencies_artifact_postfix: ${{ github.event.inputs.dependencies_artifact_postfix }}
106+
dependencies_artifact_postfix: ${{ inputs.dependencies_artifact_postfix }}
107107
secret_token: ${{ secrets.GITHUB_TOKEN }}
108108

109109
- name: Compile and run tests
@@ -146,7 +146,7 @@ jobs:
146146
os: ubuntu-22.04
147147
cmake_build_type: Release
148148
custom_version_build: ${{ inputs.custom_version_build }}
149-
dependencies_artifact_postfix: ${{ github.event.inputs.dependencies_artifact_postfix }}
149+
dependencies_artifact_postfix: ${{ inputs.dependencies_artifact_postfix }}
150150
secret_token: ${{ secrets.GITHUB_TOKEN }}
151151

152152
- name: Compile and run tests
@@ -193,7 +193,7 @@ jobs:
193193
os: ubuntu-22.04
194194
cmake_build_type: Release
195195
custom_version_build: ${{ inputs.custom_version_build }}
196-
dependencies_artifact_postfix: ${{ github.event.inputs.dependencies_artifact_postfix }}
196+
dependencies_artifact_postfix: ${{ inputs.dependencies_artifact_postfix }}
197197
secret_token: ${{ secrets.GITHUB_TOKEN }}
198198

199199
- name: Compile and run tests
@@ -223,7 +223,7 @@ jobs:
223223
os: ubuntu-22.04
224224
cmake_build_type: Release
225225
custom_version_build: ${{ inputs.custom_version_build }}
226-
dependencies_artifact_postfix: ${{ github.event.inputs.dependencies_artifact_postfix }}
226+
dependencies_artifact_postfix: ${{ inputs.dependencies_artifact_postfix }}
227227
secret_token: ${{ secrets.GITHUB_TOKEN }}
228228

229229
- name: Compile and run tests
@@ -254,7 +254,7 @@ jobs:
254254
os: ubuntu-22.04
255255
cmake_build_type: Release
256256
custom_version_build: ${{ inputs.custom_version_build }}
257-
dependencies_artifact_postfix: ${{ github.event.inputs.dependencies_artifact_postfix }}
257+
dependencies_artifact_postfix: ${{ inputs.dependencies_artifact_postfix }}
258258
secret_token: ${{ secrets.GITHUB_TOKEN }}
259259

260260
- name: Compile and run tests
@@ -296,6 +296,7 @@ jobs:
296296
uses: eProsima/eProsima-CI/ubuntu/sphinx_docs@v0
297297
with:
298298
checkout_path: src/fastdds_statistics_backend
299+
custom_version_build: ${{ inputs.custom_version_build }}
299300
path_to_requirements: src/fastdds_statistics_backend/docs/requirements.txt
300301
docs_subpackage_name: ${{ env.docs_packages_names }}
301302
secret_token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/test.yml

+3
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,10 @@ on:
2020
dependencies_artifact_postfix:
2121
description: 'Postfix name to add to artifact name to download dependencies. This is use to download a specific artifact version from eProsima-CI.'
2222
required: true
23+
type: string
24+
default: '_nightly'
2325

26+
pull_request:
2427
push:
2528
branches:
2629
- main

0 commit comments

Comments
 (0)