Skip to content

Commit

Permalink
Remove IMAGE_OS setting
Browse files Browse the repository at this point in the history
  • Loading branch information
nealrichardson committed Feb 19, 2025
1 parent acdbf28 commit 8696714
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 9 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/deploy_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
fetch-depth: 0
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- uses: actions/setup-python@v4
with:
python-version: 3.8.x
Expand Down Expand Up @@ -44,7 +44,7 @@ jobs:
fetch-depth: 0
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- name: Delete unused stuff to save space
run: sudo rm -rf /usr/share/dotnet /usr/local/lib/android

Expand All @@ -58,18 +58,18 @@ jobs:
examples
sparse-checkout-cone-mode: false
token: ${{ secrets.CONNECT_PAT }}

- name: Build docker container-image
run: |
cd test/connect-rsconnect-python/test/rsconnect-python/
docker compose --profile rsconnect build
- name: Restore dist
uses: actions/download-artifact@v4
with:
name: distributions
path: dist/

- name: Run rsconnect-python Tests
env:
CONNECT_LICENSE: "${{ secrets.RSC_LICENSE }}"
Expand All @@ -81,7 +81,6 @@ jobs:
QUARTO_VERSION: "1.4.546"
# This allows us to start Connect separately in our own docker container
CONNECT_SERVER: "http://localhost:3939"
IMAGE_OS: "rstudio/connect:ubuntu22"
remote: "yes"
run: |
cd integration-testing
Expand All @@ -90,7 +89,7 @@ jobs:
docker compose up -d connect-cli
docker compose up -d client-cli
docker compose run --rm client-cli just ../test/connect-rsconnect-python/test/rsconnect-python/_start-dev
# Videos are captured whether the suite fails or passes
- name: Save videos
uses: actions/upload-artifact@v4
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,6 @@ jobs:

# This allows us to start Connect separately in our own docker container
CONNECT_SERVER: "http://localhost:3939"
IMAGE_OS: "rstudio/connect:ubuntu22"
remote: "yes"
run: |
cd integration-testing
Expand Down
1 change: 0 additions & 1 deletion integration-testing/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ services:
QUARTO_VERSION: ${QUARTO_VERSION}
PY_VERSION: ${PY_VERSION}
API_KEY: ${ADMIN_API_KEY}
IMAGE_OS: ${IMAGE_OS}

# customized connect built with updated quarto version
# used for nightly deploy_tests.yml that include quarto projects
Expand Down

0 comments on commit 8696714

Please sign in to comment.