Skip to content

Conversation

@andytson-inviqa
Copy link
Contributor

@andytson-inviqa andytson-inviqa commented Dec 15, 2023

attributes:
  services:
    console:
      extends:
        - _database-access
        - _searchengine-access

What do people think of this idea?

I don't want to pollute the variables in console in docker harness, and reuse would be needed in PHP harnesses for php-base, so the ignored _service-name syntax (which is also used for _php-base) could be used here.

@andytson-inviqa
Copy link
Contributor Author

andytson-inviqa commented Dec 15, 2023

ah, well extends works in the helm chart but isn't used in the twig templating of docker-compose.yml yet

@andytson-inviqa
Copy link
Contributor Author

andytson-inviqa commented Dec 15, 2023

although a case against merge of config, in heinsight I'd have made php-base it's own k8s secret and maybe configmap as is confusing it not matching it's sealedsecret encrypted secret name and copied into multiple secrets

@andytson-inviqa andytson-inviqa marked this pull request as ready for review December 15, 2023 14:34
@andytson-inviqa andytson-inviqa marked this pull request as draft December 15, 2023 14:34
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