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

[RFC] Add database and searchengine templates for services.*.extends support #143

Draft
wants to merge 3 commits into
base: 0.4.x
Choose a base branch
from

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.

1 participant