Skip to content

Commit

Permalink
Remove macOs presto and trino server tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jarodmeng committed Jan 3, 2025
1 parent 7382edf commit 4a09eea
Showing 1 changed file with 9 additions and 22 deletions.
31 changes: 9 additions & 22 deletions .github/workflows/check-standard.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,35 +54,15 @@ jobs:
sudo apt-get update
sudo apt-get install -y docker-compose
# Install Docker and Docker Compose on macOS
- name: Install Docker on macOS
if: matrix.config.os == 'macOS-latest'
run: |
brew install --cask docker
brew install docker-compose
# Start Docker Daemon on macOS
- name: Start Docker Daemon on macOS
if: matrix.config.os == 'macOS-latest'
run: |
open /Applications/Docker.app
# Wait for Docker daemon to be ready
while ! docker system info > /dev/null 2>&1; do
echo "Waiting for Docker daemon to start..."
sleep 5
done
echo "Docker daemon is running."
# Start Docker Compose services
- name: Start services with Docker Compose
if: matrix.config.os == 'ubuntu-latest' || matrix.config.os == 'macOS-latest'
if: matrix.config.os == 'ubuntu-latest'
run: |
docker-compose up -d
# Ensure Presto and Trino servers are running
- name: Wait for Presto and Trino to be ready
if: matrix.config.os == 'ubuntu-latest' || matrix.config.os == 'macOS-latest'
if: matrix.config.os == 'ubuntu-latest'
run: |
for i in {1..30}; do
if curl -s http://localhost:8080/v1/info && curl -s http://localhost:8090/v1/info; then
Expand All @@ -100,3 +80,10 @@ jobs:
- uses: r-lib/actions/check-r-package@v2
with:
upload-snapshots: true

# Stop and remove Docker containers
- name: Stop and remove Docker containers
if: matrix.config.os == 'ubuntu-latest'
run: |
docker stop $(docker ps -q) || true
docker rm $(docker ps -aq) || true

0 comments on commit 4a09eea

Please sign in to comment.