From 29be7882b3b71f3be0974467662d7e2ef7aa3b38 Mon Sep 17 00:00:00 2001 From: yatinappsmith <84702014+yatinappsmith@users.noreply.github.com> Date: Fri, 1 Nov 2024 12:00:59 +0530 Subject: [PATCH] ci: Removed port 22 from ted in ci-test (#37180) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Description Removed port 22 from ted in ci-test Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Sanity" ### :mag: Cypress test results > [!IMPORTANT] > 🟣 🟣 🟣 Your tests are running. > Tests running at: > Commit: 8edce8c7fd10a59321ae73a3d09eb6d401a29c0a > Workflow: `PR Automation test suite` > Tags: `@tag.Sanity` > Spec: `` >
Fri, 01 Nov 2024 06:19:19 UTC ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No ## Summary by CodeRabbit - **Chores** - Simplified CI workflow by removing unnecessary SSH port exposure. - Enhanced caching mechanism for test run results. - Improved artifact management with better organization of logs and reports. - Streamlined handling of environment variables for Cypress tests. --- .github/workflows/ci-test-custom-script.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci-test-custom-script.yml b/.github/workflows/ci-test-custom-script.yml index 575110b1b8..530a0677b0 100644 --- a/.github/workflows/ci-test-custom-script.yml +++ b/.github/workflows/ci-test-custom-script.yml @@ -150,7 +150,7 @@ jobs: sudo /etc/init.d/ssh stop ; mkdir -p ~/git-server/keys ted_tag="${{inputs.ted_tag}}" - docker run --name test-event-driver -d -p 22:22 -p 5001:5001 -p 3306:3306 \ + docker run --name test-event-driver -d -p 5001:5001 -p 3306:3306 \ -p 5433:5432 -p 28017:27017 -p 25:25 -p 4200:4200 -p 5000:5000 -p 3001:3000 -p 6001:6001 -p 8001:8000 --privileged --pid=host --ipc=host --volume /:/host -v ~/git-server/keys:/git-server/keys \ "appsmith/test-event-driver:${ted_tag:-latest}" docker run --name cloud-services -d -p 8000:80 -p 8090:8090 \