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

[backport 1.x] Added option for httpHostPort and transportHostPort #400

Closed

Conversation

sbnmanthey
Copy link

Description

This change adds options for httpHostPort and transportHostPort in 1.x (Backport)

Issues Resolved

Resolves #335 in 1.x

Check List

  • Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

  • Helm chart version bumped
  • Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@TheAlgo
Copy link
Member

TheAlgo commented Mar 18, 2023

@sbnmanthey Can you resolve the conflicts here and also address the lint failure? Most likely you need to address the chart version here?

Signed-off-by: Sebastian Manthey <sbnmanthey@gmail.com>
Signed-off-by: Sebastian Manthey <sbnmanthey@gmail.com>
@sbnmanthey
Copy link
Author

@TheAlgo @DandyDeveloper I've bumped the chartVersion as requested, can you approve my PR?

Copy link
Member

@TheAlgo TheAlgo left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @sbnmanthey

@TheAlgo
Copy link
Member

TheAlgo commented Mar 28, 2023

@sbnmanthey Please resolve conflicts. Thank you!

@sbnmanthey
Copy link
Author

Backport was already applied in #411. I close this PR unmerged.

@sbnmanthey sbnmanthey closed this Mar 29, 2023
@sbnmanthey sbnmanthey deleted the add-hostPort-1.x branch March 29, 2023 14:08
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