Skip to content

Conversation

Ogenbertrand
Copy link
Collaborator

Closes: #103

Copy link
Collaborator

@forkimenjeckayang forkimenjeckayang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First , please bear in mind the helm chart which is already existing is fucntional. For local testing , I suggest you create a new directory and have files very similar to what we already have becuase

  • You wlll not necessarily need to enable ssl or tls while testting locally.
  • Redis url will need to change : "redis://:$(REDIS_PASSWORD)@{{ .Release.Name }}-redis-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
  • Also , I do not know how you were able to test given that , bitnami is migrating their free Docker images away from the original docker.io/bitnami organization on Docker Hub, hte images have been moved to a legacy repository at docker.io/bitnamilegacy. I was not even able to pull because of that.

But finally tested with my setup on minikube
image

You can refer to some changes done here. Riuhgky done to test but you can make somehting better from it for local tetsing configurations: https://github.com/adorsys/status-list-server/tree/test-deployemnt

Copy link
Collaborator

@forkimenjeckayang forkimenjeckayang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments to address @Ogenbertrand
When done, I will test the script

@Ogenbertrand Ogenbertrand force-pushed the feat-test-deployement-k8s branch from 7d6817d to 59ab67c Compare October 6, 2025 14:33
Copy link
Collaborator

@mbunwe-victor mbunwe-victor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @Ogenbertrand, l have added some comments. Please could you take a look?
Also, I tested the script and it worked.
image

@Ogenbertrand
Copy link
Collaborator Author

@forkimenjeckayang and @mbunwe-victor the PR is ready now.

Copy link
Collaborator

@forkimenjeckayang forkimenjeckayang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few coments to check @Ogenbertrand
Your script works.
Just adjust the few comments , it will help avoid this warning.

[INFO] Installing Helm chart...
I1007 09:46:22.397561 2948695 warnings.go:110] "Warning: spec.template.spec.containers[0].env[16].name: duplicate name \"POSTGRES_PASSWORD\""
NAME: status-list-test

Copy link
Collaborator

@mbunwe-victor mbunwe-victor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @Ogenbertrand, for addressing my comments—just this last comment.

Copy link
Collaborator

@forkimenjeckayang forkimenjeckayang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressing this should be all from me

Copy link
Collaborator

@mbunwe-victor mbunwe-victor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please @Ogenbertrand, help check these last comments. So, as mentioned in the doc file, I achieved this with the suggestions I added.

status-list-server git:(feat-test-deployement-k8s) ✗ ./k8s-deployment-test.sh status
[INFO] Deployment Status:
Namespace: statuslist
Cluster: status-list-test

[INFO] Pods:
NAME                                        READY   STATUS    RESTARTS   AGE
status-list-server-local-6df565b46c-4gq6d   1/1     Running   0          2m29s
status-list-test-postgresql-0               1/1     Running   0          2m29s
status-list-test-redis-master-0             1/1     Running   0          2m29s
[INFO] Services:
NAME                              TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
status-list-server-local          ClusterIP   10.96.44.195    <none>        8081/TCP   2m29s
status-list-test-postgresql       ClusterIP   10.96.130.143   <none>        5432/TCP   2m29s
status-list-test-postgresql-hl    ClusterIP   None            <none>        5432/TCP   2m29s
status-list-test-redis-headless   ClusterIP   None            <none>        6379/TCP   2m29s
status-list-test-redis-master     ClusterIP   10.96.29.40     <none>        6379/TCP   2m29s
status-list-test-redis-replicas   ClusterIP   10.96.215.156   <none>        6379/TCP   2m29s
[INFO] Deployments:
NAME                       READY   UP-TO-DATE   AVAILABLE   AGE
status-list-server-local   1/1     1            1           2m29s

Now this is handled:

### Service Discovery
All services now use consistent naming:
- **Deployment**: `status-list-server-local`
- **Service**: `status-list-server-local`

Copy link
Collaborator

@mbunwe-victor mbunwe-victor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Ogenbertrand Ogenbertrand merged commit 601b73a into main Oct 8, 2025
6 checks passed
@Ogenbertrand Ogenbertrand deleted the feat-test-deployement-k8s branch October 8, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Write a Script to test the deployment of the statuslist server into a kubernetes cluster

3 participants