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

Added NetworkInterfaceRef to Machine status #1126

Merged
merged 3 commits into from
Oct 2, 2024

Conversation

Rohit-0505
Copy link
Contributor

Proposed Changes

  • Add NetworkInterfaceRefof type corev1.LocalObjectReference to Machine.Status.NetworkInterfaceStatus struct for identifying the claimed NetworkInterface
  • Update logic to set claimed NetworkInterfaceRef in machinepoollet
  • Add test cases

Fixes #1117

@Rohit-0505 Rohit-0505 requested a review from a team as a code owner September 23, 2024 10:50
@github-actions github-actions bot added size/L enhancement New feature or request labels Sep 23, 2024
@Rohit-0505 Rohit-0505 self-assigned this Sep 23, 2024
@Rohit-0505 Rohit-0505 force-pushed the enh/machine_status_nic_claim_ref branch 3 times, most recently from 95ad626 to d22b6d1 Compare September 24, 2024 04:07
@Rohit-0505 Rohit-0505 force-pushed the enh/machine_status_nic_claim_ref branch from d22b6d1 to 1985c05 Compare September 24, 2024 04:24
Copy link
Member

@lukasfrank lukasfrank left a comment

Choose a reason for hiding this comment

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

LGTM

@lukasfrank lukasfrank changed the title Enhance Machine status to track a reference to its claimed NetworkInterface resource Added NetworkInterfaceRef to Machine status Sep 30, 2024
@balpert89
Copy link
Contributor

LGTM

@lukasfrank lukasfrank merged commit 07f85be into main Oct 2, 2024
10 checks passed
@lukasfrank lukasfrank deleted the enh/machine_status_nic_claim_ref branch October 2, 2024 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The Machine status should track a reference for its claimed NetworkInterface resource
3 participants