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

cleanup unused peer config #14922

Merged
merged 10 commits into from
Oct 10, 2023
Merged

cleanup unused peer config #14922

merged 10 commits into from
Oct 10, 2023

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Mar 28, 2023

Purpose:

Provide clarity around which services use which ports by removing the unused port config entries.

Current Behavior:

There are unused port config entries.

New Behavior:

The default config file is cleaner and the configure command doesn't update the unused port entries.

Testing Notes:

Draft For:

@altendky altendky added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Cleanup Code cleanup labels Mar 28, 2023
@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Apr 11, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label Apr 11, 2023
@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Apr 12, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link
Contributor

This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed by the relevant parties.

@github-actions github-actions bot added the stale-pr Flagged as stale and in need of manual review label May 28, 2023
@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label Jul 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2023

Conflicts have been resolved. A maintainer will review the pull request shortly.

@altendky altendky marked this pull request as ready for review October 4, 2023 21:19
@altendky altendky requested a review from a team as a code owner October 4, 2023 21:19
@altendky altendky requested a review from emlowe October 5, 2023 12:04
Copy link
Contributor

@emlowe emlowe left a comment

Choose a reason for hiding this comment

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

Really nice cleanup work!

@cmmarslender cmmarslender merged commit 3d728c2 into main Oct 10, 2023
@cmmarslender cmmarslender deleted the cleanup_unused_peer_config branch October 10, 2023 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Cleanup Code cleanup stale-pr Flagged as stale and in need of manual review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants