Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

smoke_test: Fix Failure on Windows GitHub Workflow #571

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
+30 −18
Open

smoke_test: Fix Failure on Windows GitHub Workflow #571

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
40f8602
smoke_test: fix failure on Windows GitHub workflow
rlxdev Jan 16, 2025
5b078d0
smoke_test: use latest OPA version by default
rlxdev Jan 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 12 additions & 7 deletions .github/actions/setup-dependencies-macos/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,29 +5,34 @@ inputs:
default: "macos"
opa-version:
required: true
default: "v0.60.0"
python-version:
required: true

runs:
using: "composite"
steps:
- name: Setup virtualenv
- name: Virtual Environment Activation Definition - macOS or linux
shell: bash
run: |
python -m venv .venv
source .venv/bin/activate
run: >
echo "SCUBAGOGGLES_ACTIVATE_VENV=. .venv/bin/activate"
>> "$GITHUB_ENV"

- name: Install dependencies
- name: Setup virtual environment and install dependencies
shell: bash
run: |
python -m venv .venv
${{ env.SCUBAGOGGLES_ACTIVATE_VENV }}
python -m pip install .
pip install -r requirements.txt
pip install pytest
pip install selenium

- name: Download OPA executable
shell: bash
env:
opa_version: ${{ inputs.opa-version == 'latest' && ' '
|| format('-v {0}', inputs.opa-version) }}
run: |
${{ env.SCUBAGOGGLES_ACTIVATE_VENV }}
scubagoggles setup -m -nc -nd -d ~/scubagoggles -r ~/scubagoggles -c credentials.json
scubagoggles getopa -v ${{ inputs.opa-version }}
scubagoggles -l d getopa ${{ env.opa_version }}
19 changes: 12 additions & 7 deletions .github/actions/setup-dependencies-windows/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,29 +5,34 @@ inputs:
default: "windows"
opa-version:
required: true
default: "v0.60.0"
python-version:
required: true

runs:
using: "composite"
steps:
- name: Setup virtualenv
- name: Virtual Environment Activation Definition - Windows
shell: powershell
run: |
python -m venv .venv
.venv\Scripts\activate.ps1
run: >
echo "SCUBAGOGGLES_ACTIVATE_VENV=.venv\Scripts\activate.ps1"
>> "$Env:GITHUB_ENV"

- name: Install dependencies
- name: Setup virtual environment and install dependencies
shell: powershell
run: |
python -m venv .venv
${{ env.SCUBAGOGGLES_ACTIVATE_VENV }}
python -m pip install .
pip install -r requirements.txt
pip install pytest
pip install selenium

- name: Download OPA executable
shell: powershell
env:
opa_version: ${{ inputs.opa-version == 'latest' && ' '
|| format('-v {0}', inputs.opa-version) }}
run: |
${{ env.SCUBAGOGGLES_ACTIVATE_VENV }}
scubagoggles setup -m -nc -nd -d ~/scubagoggles -r ~/scubagoggles -c credentials.json
scubagoggles getopa -v ${{ inputs.opa-version }}
scubagoggles -l d getopa ${{ env.opa_version }}
10 changes: 6 additions & 4 deletions .github/workflows/run_smoke_test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ on:
description: "Choose OPA version"
required: true
type: string
default: "v0.60.0"
default: latest

jobs:
configuration:
Expand All @@ -53,9 +53,9 @@ jobs:

# Default values for other events
else
operatingsystem_val="['macos-latest']"
operatingsystem_val="['windows-latest', 'macos-latest']"
pythonversion_val="['3.10']"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than in DownloadAndInstall.md where we indicate "v3.9 or higher", I don't see any references to a definitive minimum supported python version in the code. Recommend we move this out of the workflow in favor of a more maintainable location that doesn't require a code change when we decide to bump to v3.11, etc. Setting a GitHub secret as "MIN_PYTHON_VERSION"="3.10" may be a short-term alternative.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Support for Python 3.9 is dropping as of October this year, and we have another year for 3.10. I didn’t change the Python version used for this workflow (3.10). I think it’s a reasonable version to test with, considering 3.9 is going away. We can always look at other ways to have the workflow get the Python version to use. This PR is specific to the smoke test not working on Windows, and the issue is resolved with this PR.

opaversion_val="0.60.0"
opaversion_val="latest"
fi
echo "operating-system=$operatingsystem_val" >> "$GITHUB_OUTPUT"
echo "python-version=$pythonversion_val" >> "$GITHUB_OUTPUT"
Expand Down Expand Up @@ -106,4 +106,6 @@ jobs:
json: ${{ secrets.GWS_GITHUB_AUTOMATION_CREDS }}

- name: Run ScubaGoggles and check for correct output
run: pytest ./scubagoggles/Testing/Functional/SmokeTests/ -vvv --subjectemail="${{ secrets.GWS_SUBJECT_EMAIL }}" --customerdomain="${{ secrets.GWS_DOMAIN }}"
run: |
${{ env.SCUBAGOGGLES_ACTIVATE_VENV }}
pytest ./scubagoggles/Testing/Functional/SmokeTests/ -vvv --subjectemail="${{ secrets.GWS_SUBJECT_EMAIL }}" --customerdomain="${{ secrets.GWS_DOMAIN }}"
Loading