Skip to content

Commit 87247a8

Browse files
Revert "Add synchronized timestamped logging for Pro CI integration tests"
This reverts commit 8257b20.
1 parent 5c87ee9 commit 87247a8

File tree

2 files changed

+2
-269
lines changed

2 files changed

+2
-269
lines changed

.github/workflows/pro-integration-tests.yml

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -230,33 +230,16 @@ jobs:
230230
- name: Generate file-system based entrypoints
231231
run: cd spec/dummy && bundle exec rake react_on_rails:generate_packs
232232

233-
# Install moreutils for the 'ts' command to add timestamps to logs
234-
# This enables synchronized side-by-side log viewing with bin/view-synchronized-logs
235-
- name: Install timestamp utility for log synchronization
236-
run: sudo apt-get install -y moreutils
237-
238-
- name: Run Pro Node renderer in background with timestamped logging
233+
- name: Run Pro Node renderer in background
239234
run: |
240235
cd spec/dummy
241-
mkdir -p tmp/ci-logs
242-
yarn run node-renderer 2>&1 | ts '[%Y-%m-%d %H:%M:%.S]' | tee tmp/ci-logs/node-renderer.log &
236+
yarn run node-renderer &
243237
244238
- name: Run Rails server in background
245239
run: |
246240
cd spec/dummy
247241
RAILS_ENV="test" rails server &
248242
249-
- name: Tail Rails test log with timestamps in background
250-
run: |
251-
cd spec/dummy
252-
# Wait for log file to be created
253-
timeout=10
254-
while [ ! -f log/test.log ] && [ $timeout -gt 0 ]; do
255-
sleep 0.5
256-
timeout=$((timeout - 1))
257-
done
258-
tail -f log/test.log 2>&1 | ts '[%Y-%m-%d %H:%M:%.S]' > tmp/ci-logs/rails-server.log &
259-
260243
- name: Wait for Rails server to start
261244
run: |
262245
timeout=60
@@ -316,13 +299,6 @@ jobs:
316299
name: pro-rspec-yarn-error-log
317300
path: react_on_rails_pro/spec/dummy/yarn-error.log
318301

319-
- name: Store synchronized server logs
320-
uses: actions/upload-artifact@v4
321-
if: always()
322-
with:
323-
name: pro-rspec-synchronized-logs
324-
path: react_on_rails_pro/spec/dummy/tmp/ci-logs/
325-
326302
# Playwright E2E tests with Redis service
327303
dummy-app-node-renderer-e2e-tests:
328304
needs:

bin/view-synchronized-logs

Lines changed: 0 additions & 243 deletions
This file was deleted.

0 commit comments

Comments
 (0)