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

feat: add apiserver FQDN to CNS log metadata #3382

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rbtr
Copy link
Contributor

@rbtr rbtr commented Jan 31, 2025

Reason for Change:

Adds AKS apiserver FQDN to CNS log metadata so that logs can be aggregated by Cluster.

Issue Fixed:

Requirements:

Notes:

@rbtr rbtr requested a review from a team as a code owner January 31, 2025 23:37
@rbtr rbtr self-assigned this Jan 31, 2025
@rbtr rbtr added enhancement cns Related to CNS. fix Fixes something. release/latest Change affects latest release train labels Jan 31, 2025
@rbtr rbtr requested a review from paulyufan2 January 31, 2025 23:38
@rbtr
Copy link
Contributor Author

rbtr commented Jan 31, 2025

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rbtr rbtr force-pushed the feat/log-apiserver branch from 6605979 to 6eafd37 Compare February 3, 2025 16:10
timraymond
timraymond previously approved these changes Feb 3, 2025
thatmattlong
thatmattlong previously approved these changes Feb 3, 2025
@rbtr rbtr enabled auto-merge February 3, 2025 18:45
@rbtr
Copy link
Contributor Author

rbtr commented Feb 3, 2025

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rbtr rbtr added this pull request to the merge queue Feb 3, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Feb 4, 2025
@rbtr rbtr dismissed stale reviews from thatmattlong and timraymond via aa4b97e February 4, 2025 15:37
@rbtr rbtr force-pushed the feat/log-apiserver branch from 6eafd37 to aa4b97e Compare February 4, 2025 15:37
@rbtr rbtr enabled auto-merge February 4, 2025 15:38
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the feat/log-apiserver branch from aa4b97e to df54b4c Compare February 5, 2025 22:58
@rbtr
Copy link
Contributor Author

rbtr commented Feb 5, 2025

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cns Related to CNS. enhancement fix Fixes something. release/latest Change affects latest release train
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants