Skip to content

Add HomeAZ to NodeInfoStatus #3738

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

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

timraymond
Copy link
Member

In service of adding HomeAZ awareness to apiserver-vnet-integration, this adds HomeAZ to the NodeInfoStatus portion of the NodeInfo CRD. Similar to other elements of NodeInfoStatus, HomeAZ is an optional property of a node that can be communicated. If present, it strictly follows the AZXX format, where XX are two 0-9 integers.

@Copilot Copilot AI review requested due to automatic review settings June 16, 2025 19:28
@timraymond timraymond requested a review from a team as a code owner June 16, 2025 19:28
@timraymond timraymond requested review from byte-msft and removed request for Copilot June 16, 2025 19:28
Copilot

This comment was marked as resolved.

@Azure Azure deleted a comment from Copilot AI Jun 16, 2025
@timraymond timraymond requested a review from thatmattlong June 23, 2025 14:16
@rbtr
Copy link
Contributor

rbtr commented Jun 23, 2025

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

In service of adding HomeAZ awareness to apiserver-vnet-integration,
this adds HomeAZ to the NodeInfoStatus portion of the NodeInfo CRD.
Similar to other elements of NodeInfoStatus, HomeAZ is an optional
property of a node that can be communicated. If present, it strictly
follows the AZXX format, where XX are two 0-9 integers.
@timraymond timraymond force-pushed the traymond/add-homeaz-to-nodeinfostatus branch from 9c1acf9 to 141028b Compare June 24, 2025 15:27
@timraymond timraymond enabled auto-merge June 24, 2025 15:32
@paulyufan2
Copy link
Contributor

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@timraymond timraymond added this pull request to the merge queue Jun 24, 2025
Merged via the queue into master with commit 8fd1503 Jun 24, 2025
15 of 16 checks passed
@timraymond timraymond deleted the traymond/add-homeaz-to-nodeinfostatus branch June 24, 2025 20:46
thatmattlong added a commit that referenced this pull request Jun 25, 2025
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.

4 participants