Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intergration env support #283

Open
wants to merge 90 commits into
base: main
Choose a base branch
from
Open

Conversation

Christiantyemele
Copy link
Collaborator

@Christiantyemele Christiantyemele commented Dec 5, 2024

This is the architecture followed for the mediator delployment: current implementation can be found on page2 here

…into 245-integration-environment-for-didcomm-mediator-server-deployment-e6
…into 245-integration-environment-for-didcomm-mediator-server-deployment-e6
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove this folder and the containing file. We don't need it.

.dockerignore Show resolved Hide resolved
.gitignore Show resolved Hide resolved
Dockerfile Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
docker-compose.yml Show resolved Hide resolved
Copy link
Collaborator

@stephane-segning stephane-segning left a comment

Choose a reason for hiding this comment

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

It's missing, the storage path in the deployment. There should be a PVC there.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This deployment lacks of a volume for the storage dir.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use https://artifacthub.io/packages/helm/bitnami/mongodb instead of rewriting the mongo deployment files.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this secret meant for?

Copy link
Collaborator

Choose a reason for hiding this comment

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

If it's for mongo, please use variables instead. This shouldn't be in clear text here like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devOps deployment/intergration related enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration Environment for DIDComm Mediator Server Deployment E6
7 participants