Ory hydra dsn configuration through existing secret causes env var to not be defined

[zyixc]

Preflight checklist

• I could not find a solution in the existing issues, docs, nor discussions.
• I agree to follow this project's Code of Conduct.
• I have read and am following this repository's Contribution Guidelines.
• I have joined the Ory Community Slack.
• I am signed up to the Ory Security Patch Newsletter.

Ory Network Project

No response

Describe the bug

DSN environment not defined for the deployments, jobs and cronjobs.

Reproducing the bug

Predefining the hydra secret with the dsn value and not providing hydra.config.dsn omits the following environment variable.

This change was introduced here: #509. I do not think this was intended, if hydra.config.dsn or demo are both not provided it can just be included. As this is a list of environment variables, later definitions can override the value.

Relevant log output

level=fatal msg=dsn must be set audience=application service_name=Ory Hydra service_version=v2.1.2

Relevant configuration

No response

Version

0.37.0

On which operating system are you observing this issue?

None

In which environment are you deploying?

None

Additional Context

For now this is needed

    secret:
      enabled: false
      nameOverride: hydra
    deployment:
      extraEnv: 
         - name: DSN
            valueFrom:
              secretKeyRef:
                name:  hydra
                key: dsn

[Demonsthere]

Hello there, can you please verify if this case still doesn't work as expected on the latest release/master? We did merge some changes to DSN handling, which should unify the setup