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.

Sign up for GitHub

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

Jump to bottom

Eventing (NATS) chart values #78

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Eventing (NATS) chart values #78

wants to merge 1 commit into from

Conversation

corcoja
Copy link
Member

@corcoja corcoja commented Oct 12, 2022

Changelog

New

  • More granular configuration of the NATS server with host, posrt, username and password.
  • Possibility to retrieve NATS password from a secret.
  • NATS password stored as an environment variable for each micro-service.

Removed

  • Removed unnecessary properties from configmap. Default values are configurable from service's application.properties file.

@corcoja corcoja changed the title Better Eventing (NATS) configuration Eventing (NATS) chart values Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@florentinvintila florentinvintila Awaiting requested review from florentinvintila

@gchickma gchickma Awaiting requested review from gchickma gchickma is a code owner

@tlawrie tlawrie Awaiting requested review from tlawrie tlawrie is a code owner

@morarucostel morarucostel Awaiting requested review from morarucostel morarucostel is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@corcoja