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

[CNS] refactor: remove redundant check for TLSSettings.TLSPort #2780

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

jackieluc
Copy link
Member

Reason for Change:

  • small refactor to remove redundant check for TLSSettings.TLSPort in CNS

Issue Fixed:

Requirements:

Notes:

Sorry, something went wrong.

@jackieluc jackieluc added the cns Related to CNS. label Jun 11, 2024
@jackieluc jackieluc requested a review from a team as a code owner June 11, 2024 16:47
@jackieluc jackieluc requested a review from paulyufan2 June 11, 2024 16:47
@jackieluc
Copy link
Member Author

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulyufan2 paulyufan2 added this pull request to the merge queue Jun 13, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 14, 2024
@jackieluc jackieluc added this pull request to the merge queue Jun 14, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 14, 2024
@jackieluc jackieluc added this pull request to the merge queue Jun 14, 2024
Merged via the queue into Azure:master with commit 33de98f Jun 14, 2024
11 checks passed
@jackieluc jackieluc deleted the remove-redundant-code branch June 14, 2024 20:12
paulyufan2 pushed a commit that referenced this pull request Jun 18, 2024
refactor: remove redundant check for TLSSettings.TLSPort
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cns Related to CNS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants