Skip to content

Conversation

@pranjayyelkotwar
Copy link

@pranjayyelkotwar pranjayyelkotwar commented Jul 15, 2025

Updates compose.gitlab.yml to enable gitlab container registry. The changes to compose.local.secure.yml are invalid and need to be reverted.

@sonarqubecloud
Copy link

Copy link
Contributor

@prasadtalasila prasadtalasila left a comment

Choose a reason for hiding this comment

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

@pranjayyelkotwar, please try the secure production installation and see if it works. Thanks.

- "443:443"
volumes:
- "/var/run/docker.sock:/var/run/docker.sock:ro"
- "/etc/letsencrypt:/etc/letsencrypt:ro"
Copy link
Contributor

Choose a reason for hiding this comment

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

letsencrypt does not work for localhost. That is why we are using mkcert in secure localhost. Please use secure production installation to integrate the container registry.

- "--entrypoints.web.http.redirections.entrypoint.permanent=true"
- "--providers.file.directory=/etc/traefik/dynamic"
- "--providers.file.watch=true"
- "--certificatesresolvers.manual.acme.tlschallenge=true"
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we need these three?

labels:
- "traefik.enable=true"
- "traefik.http.routers.client.rule=PathPrefix(`/`)"
- "traefik.http.routers.client.rule=Host(`release.dtaas-digitaltwin.com`)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use compose.server.secure.yml file for installation.

@prasadtalasila prasadtalasila changed the title modifying the yml files Adds gitlab container registry settings to installation files Sep 5, 2025
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.

2 participants