From 553a2e91036e0c0d67fa0c3e76836e5178d2ec58 Mon Sep 17 00:00:00 2001
From: Davit <davedabyeet@gmail.com>
Date: Sun, 30 Jun 2024 21:03:05 +0400
Subject: [PATCH 1/7] use dynamic length for deployments

---
 src/components/DeploymentsByFilter/index.js | 102 ++++++++++----------
 1 file changed, 50 insertions(+), 52 deletions(-)

diff --git a/src/components/DeploymentsByFilter/index.js b/src/components/DeploymentsByFilter/index.js
index f9849b9f..93be94a9 100644
--- a/src/components/DeploymentsByFilter/index.js
+++ b/src/components/DeploymentsByFilter/index.js
@@ -118,10 +118,12 @@ const DeploymentsByFilter = ({ deployments }) => {
     });
   };
 
+  const sortedFilteredItems = sortedItems.filter(deployment => filterResults(deployment));
+
   return (
     <Deployments>
       <div className="filters">
-        <label>{sortedItems.length == 1 ? `1 deployment` : `${sortedItems.length} deployments`}</label>
+        <label>{sortedItems.length == 1 ? `1 deployment` : `${sortedFilteredItems.length} deployments`}</label>
         <label></label>
         <input
           type="text"
@@ -162,58 +164,54 @@ const DeploymentsByFilter = ({ deployments }) => {
           <label>Duration</label>
           <label></label>
         </DeploymentsHeader>
-        {!sortedItems.filter(deployment => filterResults(deployment)).length && (
-          <div className="data-none">No deployments</div>
-        )}
-        {sortedItems
-          .filter(deployment => filterResults(deployment))
-          .map(deployment => {
-            return (
-              <div className="data-row row-heading" key={deployment.id}>
-                <div className="project">
-                  <ProjectLink projectSlug={deployment.environment.project.name}>
-                    {formatString(deployment.environment.project.name, 'project')}
-                  </ProjectLink>
-                </div>
-                <div className="environment">
-                  <DeploymentsLink
-                    environmentSlug={deployment.environment.openshiftProjectName}
-                    projectSlug={deployment.environment.project.name}
-                  >
-                    {formatString(deployment.environment.name, 'environment')}
-                  </DeploymentsLink>
-                </div>
-                <div className="cluster">{formatString(deployment.environment.openshift.name, 'cluster')}</div>
-                <div className="name">
-                  <DeploymentLink
-                    deploymentSlug={deployment.name}
-                    environmentSlug={deployment.environment.openshiftProjectName}
-                    projectSlug={deployment.environment.project.name}
-                    key={deployment.id}
-                  >
-                    {deployment.name}
-                  </DeploymentLink>
-                </div>
-                <div className="priority">{deployment.priority}</div>
-                <div className="started">
-                  {moment.utc(deployment.created).local().format('DD MMM YYYY, HH:mm:ss (Z)')}
-                </div>
-                <div className={`status buildstep ${deployment.status}`}>
-                  {deployment.status.charAt(0).toUpperCase() + deployment.status.slice(1)}
-
-                  {!['complete', 'cancelled', 'failed'].includes(deployment.status) && deployment.buildStep && (
-                    <HoverTag text={deployment.buildStep} maxWidth="160px" tooltipPosition="bottom" />
-                  )}
-                </div>
-                <div className="duration">{getDeploymentDuration(deployment)}</div>
-                <div>
-                  {['new', 'pending', 'queued', 'running'].includes(deployment.status) && (
-                    <CancelDeployment deployment={deployment} afterText="Cancelled" beforeText="Cancel" />
-                  )}
-                </div>
+        {!sortedFilteredItems.length && <div className="data-none">No deployments</div>}
+        {sortedFilteredItems.map(deployment => {
+          return (
+            <div className="data-row row-heading" key={deployment.id}>
+              <div className="project">
+                <ProjectLink projectSlug={deployment.environment.project.name}>
+                  {formatString(deployment.environment.project.name, 'project')}
+                </ProjectLink>
+              </div>
+              <div className="environment">
+                <DeploymentsLink
+                  environmentSlug={deployment.environment.openshiftProjectName}
+                  projectSlug={deployment.environment.project.name}
+                >
+                  {formatString(deployment.environment.name, 'environment')}
+                </DeploymentsLink>
+              </div>
+              <div className="cluster">{formatString(deployment.environment.openshift.name, 'cluster')}</div>
+              <div className="name">
+                <DeploymentLink
+                  deploymentSlug={deployment.name}
+                  environmentSlug={deployment.environment.openshiftProjectName}
+                  projectSlug={deployment.environment.project.name}
+                  key={deployment.id}
+                >
+                  {deployment.name}
+                </DeploymentLink>
+              </div>
+              <div className="priority">{deployment.priority}</div>
+              <div className="started">
+                {moment.utc(deployment.created).local().format('DD MMM YYYY, HH:mm:ss (Z)')}
+              </div>
+              <div className={`status buildstep ${deployment.status}`}>
+                {deployment.status.charAt(0).toUpperCase() + deployment.status.slice(1)}
+
+                {!['complete', 'cancelled', 'failed'].includes(deployment.status) && deployment.buildStep && (
+                  <HoverTag text={deployment.buildStep} maxWidth="160px" tooltipPosition="bottom" />
+                )}
+              </div>
+              <div className="duration">{getDeploymentDuration(deployment)}</div>
+              <div>
+                {['new', 'pending', 'queued', 'running'].includes(deployment.status) && (
+                  <CancelDeployment deployment={deployment} afterText="Cancelled" beforeText="Cancel" />
+                )}
               </div>
-            );
-          })}
+            </div>
+          );
+        })}
       </DeploymentsDataTable>
     </Deployments>
   );

From 1d6e6044508bddf6d0daaf3d31e779b59cfe0645 Mon Sep 17 00:00:00 2001
From: Davit <davedabyeet@gmail.com>
Date: Sun, 30 Jun 2024 22:29:28 +0400
Subject: [PATCH 2/7] show which user is being removed

---
 src/components/Organizations/SharedStyles.tsx | 3 +++
 src/components/Organizations/Users/index.js   | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/components/Organizations/SharedStyles.tsx b/src/components/Organizations/SharedStyles.tsx
index ba652c07..4fe024a6 100644
--- a/src/components/Organizations/SharedStyles.tsx
+++ b/src/components/Organizations/SharedStyles.tsx
@@ -450,6 +450,9 @@ export const RemoveModalParagraph = styled.p`
   line-height: 24px;
   span {
     font-weight: bold;
+    &.highlight {
+      color: #4b84ff;
+    }
   }
 `;
 
diff --git a/src/components/Organizations/Users/index.js b/src/components/Organizations/Users/index.js
index ef62e6c0..fb7882bd 100644
--- a/src/components/Organizations/Users/index.js
+++ b/src/components/Organizations/Users/index.js
@@ -132,7 +132,7 @@ const Users = ({ users = [], organization, organizationId, organizationName, ref
             </Tooltip>
             {!user.email.startsWith('default-user') ? (
               <>
-                <Tooltip overlayClassName="orgTooltip" placement="bottom" title="Delete user">
+                <Tooltip overlayClassName="orgTooltip" placement="bottom" title="Remove user">
                   <DeleteOutlined
                     className="delete"
                     onClick={() => {
@@ -144,7 +144,8 @@ const Users = ({ users = [], organization, organizationId, organizationName, ref
                 <Modal isOpen={deleteUserModalOpen && selectedUser === user?.id} onRequestClose={closeUserModal}>
                   <RemoveModalHeader>Remove user?</RemoveModalHeader>
                   <RemoveModalParagraph>
-                    This action will remove this user from all groups, you might not be able to reverse this.
+                    This action will remove user <span className="highlight">{user.email}</span> from all groups, you
+                    might not be able to reverse this.
                   </RemoveModalParagraph>
 
                   <Footer>

From eb95f7be584393fffdb9d775dbc766bd7d342c74 Mon Sep 17 00:00:00 2001
From: Davit <davedabyeet@gmail.com>
Date: Mon, 1 Jul 2024 01:55:20 +0400
Subject: [PATCH 3/7] change wording

---
 src/components/Organizations/Users/index.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/Organizations/Users/index.js b/src/components/Organizations/Users/index.js
index fb7882bd..e3579f8c 100644
--- a/src/components/Organizations/Users/index.js
+++ b/src/components/Organizations/Users/index.js
@@ -144,8 +144,8 @@ const Users = ({ users = [], organization, organizationId, organizationName, ref
                 <Modal isOpen={deleteUserModalOpen && selectedUser === user?.id} onRequestClose={closeUserModal}>
                   <RemoveModalHeader>Remove user?</RemoveModalHeader>
                   <RemoveModalParagraph>
-                    This action will remove user <span className="highlight">{user.email}</span> from all groups, you
-                    might not be able to reverse this.
+                    This action will remove user <span className="highlight">{user.email}</span> from all groups in this
+                    organization, you might not be able to reverse this.
                   </RemoveModalParagraph>
 
                   <Footer>

From dce718b174081d0ebbf9beb0b8bee3ed0c82292d Mon Sep 17 00:00:00 2001
From: Toby Bellwood <toby.bellwood@amazee.io>
Date: Wed, 3 Jul 2024 16:57:37 +1000
Subject: [PATCH 4/7] chore: set network-timeout on yarn run build

---
 Dockerfile | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index e5dd783e..bdae267f 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,5 +1,5 @@
 # Node builder image
-FROM uselagoon/node-20-builder:latest as builder
+FROM uselagoon/node-20-builder:latest AS builder
 
 COPY . /app/
 
@@ -25,7 +25,7 @@ ARG GRAPHQL_API
 ENV GRAPHQL_API=$GRAPHQL_API
 
 # Build app
-RUN yarn run build
+RUN yarn --network-timeout 300000 run build
 
 EXPOSE 3000
-CMD ["yarn", "start"]
\ No newline at end of file
+CMD ["yarn", "start"]

From c0b73468f568b34853490591257407b2672f868f Mon Sep 17 00:00:00 2001
From: Toby Bellwood <toby.bellwood@amazee.io>
Date: Wed, 3 Jul 2024 17:26:17 +1000
Subject: [PATCH 5/7] chore: set network-timeout on yarn install

---
 Dockerfile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index bdae267f..2805fc09 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -3,7 +3,7 @@ FROM uselagoon/node-20-builder:latest AS builder
 
 COPY . /app/
 
-RUN yarn install
+RUN yarn install --network-timeout 300000
 
 
 # Node service image
@@ -25,7 +25,7 @@ ARG GRAPHQL_API
 ENV GRAPHQL_API=$GRAPHQL_API
 
 # Build app
-RUN yarn --network-timeout 300000 run build
+RUN yarn run build
 
 EXPOSE 3000
 CMD ["yarn", "start"]

From f3a42244220dba8b59102dcfa5b432f383941aad Mon Sep 17 00:00:00 2001
From: shreddedbacon <b@benjackson.email>
Date: Wed, 3 Jul 2024 19:33:24 +1000
Subject: [PATCH 6/7] chore: add sidecar

---
 test/docker-compose.yaml            | 14 +++++++++++++-
 test/keycloak/configure-keycloak.sh |  2 +-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/test/docker-compose.yaml b/test/docker-compose.yaml
index c49e7404..83cbf2bc 100644
--- a/test/docker-compose.yaml
+++ b/test/docker-compose.yaml
@@ -28,6 +28,12 @@ services:
         condition: service_completed_successfully # don't start the lagoon migrations until the db migrations is completed
       keycloak:
         condition: service_started
+  api-sidecar-handler:
+    # this is neded for the internal dns references
+    container_name: apisidecarhandler
+    image: testlagoon/api-sidecar-handler:main
+    ports:
+      - '3333:3333'
   api:
     image: testlagoon/api:main
     ports:
@@ -44,8 +50,14 @@ services:
       - S3_BAAS_ACCESS_KEY_ID=minio
       - S3_BAAS_SECRET_ACCESS_KEY=minio123
       - CONSOLE_LOGGING_LEVEL=trace
+      - SIDECAR_HANDLER_HOST=apisidecarhandler
     depends_on:
-      - api-lagoon-migrations
+      api-lagoon-migrations:
+        condition: service_started
+      keycloak:
+        condition: service_started
+      api-sidecar-handler:
+        condition: service_started
   api-redis:
     image: testlagoon/api-redis:main
   keycloak:
diff --git a/test/keycloak/configure-keycloak.sh b/test/keycloak/configure-keycloak.sh
index 0aa788c0..aee9bf06 100755
--- a/test/keycloak/configure-keycloak.sh
+++ b/test/keycloak/configure-keycloak.sh
@@ -10,7 +10,7 @@ function is_keycloak_running {
 function configure_user_passwords {
 
   LAGOON_DEMO_USERS=("guest@example.com" "reporter@example.com" "developer@example.com" "maintainer@example.com" "owner@example.com")
-  LAGOON_DEMO_ORG_USERS=("orguser@example.com" "orgviewer@example.com" "orgowner@example.com" "platformowner@example.com")
+  LAGOON_DEMO_ORG_USERS=("orguser@example.com" "orgviewer@example.com" "orgadmin@example.com" "orgowner@example.com" "platformowner@example.com")
 
   for i in ${LAGOON_DEMO_USERS[@]}
   do

From 34d1e3a89aa81b68d46aa395d820d3818ff6ab70 Mon Sep 17 00:00:00 2001
From: shreddedbacon <b@benjackson.email>
Date: Wed, 3 Jul 2024 19:58:21 +1000
Subject: [PATCH 7/7] chore: increase command timeout

---
 cypress/cypress.config.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cypress/cypress.config.ts b/cypress/cypress.config.ts
index 9954393f..27492a32 100644
--- a/cypress/cypress.config.ts
+++ b/cypress/cypress.config.ts
@@ -2,6 +2,7 @@ import { defineConfig } from 'cypress';
 
 export default defineConfig({
   requestTimeout: 15000,
+  defaultCommandTimeout: 8000,
   e2e: {
     env: {
       api: 'http://0.0.0.0:33000/graphql',