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

Adding --webhook-secure-port=0 to helmchart since it is needed workaround for issue #200 #204

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Adding --webhook-secure-port=0 to helmchart since it is needed workaround for issue #200 #204

merged 1 commit into from
Apr 19, 2024

Conversation

akaokunc
Copy link
Contributor

@akaokunc akaokunc commented Apr 18, 2024 •
edited

…ound for issue #200 which some users are experiencing

General:

  • Have you removed all sensitive information, including but not limited to access keys and passwords?
  • Have you checked to ensure there aren't other open or closed Pull Requests for the same bug/feature/question?

Pull Request Guidelines:

  1. Does your submission pass tests?
  2. Have you added tests?
  3. Are you addressing a single feature in this PR?
  4. Are your commits atomic, addressing one change per commit?
  5. Are you following the conventions of the language?
  6. Have you saved your large formatting changes for a different PR, so we can focus on your work?
  7. Have you explained your rationale for why this feature is needed?
  8. Have you linked your PR to an open issue

@akaokunc akaokunc linked an issue Apr 18, 2024 that may be closed by this pull request
tcp 0.0.0.0:10260: bind: address already in use #200
Closed
@akaokunc akaokunc marked this pull request as ready for review April 18, 2024 06:20
@akaokunc akaokunc requested a review from rahulait April 18, 2024 06:20
rahulait
rahulait approved these changes Apr 18, 2024
View reviewed changes
Copy link
Contributor

@rahulait rahulait left a comment

Choose a reason for hiding this comment

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

LGTM.

@rahulait rahulait merged commit 1e9005a into linode:main Apr 19, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahulait rahulait rahulait approved these changes

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

Successfully merging this pull request may close these issues.

tcp 0.0.0.0:10260: bind: address already in use
2 participants
@akaokunc @rahulait