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

fix(controller): Support default database port. Fixes #4756 #4757

Merged
merged 1 commit into from Dec 17, 2020

Conversation

alexec
Copy link
Contributor

@alexec alexec commented Dec 16, 2020

Signed-off-by: Alex Collins alex_collins@intuit.com

Checklist:

Signed-off-by: Alex Collins <alex_collins@intuit.com>
@alexec alexec linked an issue Dec 16, 2020 that may be closed by this pull request
@alexec alexec added type/bug type/regression Regression from previous behavior (a specific type of bug) labels Dec 16, 2020
@alexec alexec marked this pull request as ready for review December 16, 2020 17:31
@simster7 simster7 self-assigned this Dec 16, 2020
@alexec alexec merged commit fbb4e8d into argoproj:master Dec 17, 2020
@alexec alexec deleted the fix-4756 branch December 17, 2020 18:21
@simster7 simster7 mentioned this pull request Jan 4, 2021
21 tasks
simster7 pushed a commit that referenced this pull request Jan 4, 2021
Signed-off-by: Alex Collins <alex_collins@intuit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug type/regression Regression from previous behavior (a specific type of bug)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When not specifying port for Postgres DB port is defaulting to 0
2 participants