From 5b33768e4ccd41e9f76e74d993b53c369177e430 Mon Sep 17 00:00:00 2001
From: Sagar Khalasi <sagar@appsmith.com>
Date: Tue, 27 Aug 2024 12:33:20 +0530
Subject: [PATCH] ci: Updated server logs in CI (#35896)

## Description
Adding server logs


Fixes #`35895`

## Automation

/ok-to-test tags=""

### :mag: Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!CAUTION]
> If you modify the content in this section, you are likely to disrupt
the CI result for your PR.

<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [x] No


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **New Features**
- Enhanced CI workflow with new steps for collecting and uploading
Docker logs to improve debugging capabilities during failures.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
---
 .github/workflows/ci-test-custom-script.yml | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ci-test-custom-script.yml b/.github/workflows/ci-test-custom-script.yml
index 925e4c85b7..ef8aea989e 100644
--- a/.github/workflows/ci-test-custom-script.yml
+++ b/.github/workflows/ci-test-custom-script.yml
@@ -441,7 +441,20 @@ jobs:
           name: server-logs-${{ matrix.job }}
           path: app/server/server-logs.log
           overwrite: true
-
+          
+      - name: Collect docker log as file
+        if: always()
+        run: |
+          docker logs appsmith >& app/server/docker-logs.log
+ 
+      - name: Upload server docker logs bundle on failure 
+        uses: actions/upload-artifact@v4
+        if: always()
+        with:
+            name: docker-logs-${{ matrix.job }}
+            path: app/server/docker-logs.log
+            overwrite: true
+  
       # Set status = success
       - name: Save the status of the run
         run: |