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

WIP: SSL mode support for postgres in Catalog API #316

Closed
wants to merge 2 commits into from

Conversation

pavelgopanenko
Copy link
Collaborator

Since version 3.598.6

ssl:
enabled: false
mode: verify-full
clientCert: ./path/to/client-cert.pem
Copy link
Contributor

Choose a reason for hiding this comment

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

Мы указываем путь к серту, а то как этот серт появляется по этому пути - не описано.
Нужен какой-то секрет еще, который будет маунтится по этому пути, скорее всего.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Предполагается, что клиент пересоберёт образы куда зашьет файлы.

Copy link
Contributor

Choose a reason for hiding this comment

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

Звучит не совсем так, как мы ожидаем. Чтобы воспользоваться функионалом - мы должны предоставить возможность сделать это без вмешательства в наши образы

Давайте сделаем по примеру #304
Там есть описание секртеов, и инит контейнеров, которые будут из секретов подкладывать как и куда нужно серты в том виде, в котором они ожидаются

@endryhold
Copy link
Contributor

В названии ПРа фигурирует тлько catalog, а по факту еще и ключи

Добавьте версии соответсвующих приложений, для которых это будет работать в Chart.yaml в appVersion

@pavelgopanenko
Copy link
Collaborator Author

PR закрыт, мы не готовы этим заниматься в порядке своей инициативы

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.

3 participants