Skip to content

Conversation

@Rohit-0505
Copy link
Contributor

@Rohit-0505 Rohit-0505 commented Aug 1, 2025

Proposed Changes

  • Remove Labels, NetworkLabels from machine api.proto and reuse Attributes field to transport DownwardAPILabels for Network and NIC resources, This approach ensures generic behavior untouched at all layers
  • Add downward-api labels support for Network resource
  • Update the manager.yaml with downward-api-label args
  • Validate the changes with tests
  • Optimize the code with reusable functions in broker and poollet
    Fix: Add downward-api label support to other resources similar to machine #1316

@Rohit-0505 Rohit-0505 requested a review from a team as a code owner August 1, 2025 09:50
@github-actions github-actions bot added enhancement New feature or request size/L labels Aug 1, 2025
@Rohit-0505 Rohit-0505 self-assigned this Aug 1, 2025
@Rohit-0505 Rohit-0505 added this to IaaS Aug 1, 2025
@hardikdr hardikdr added the area/iaas Issues related to IronCore IaaS development. label Aug 2, 2025
@hardikdr hardikdr added this to Roadmap Aug 2, 2025
@github-actions github-actions bot added size/XL and removed size/L labels Aug 5, 2025
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch from 030febb to cc55eb8 Compare August 5, 2025 14:47
@Rohit-0505 Rohit-0505 marked this pull request as draft August 6, 2025 07:52
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch from cc55eb8 to 5a33e0c Compare August 11, 2025 10:00
@github-actions github-actions bot added size/L and removed size/XL labels Aug 11, 2025
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch 2 times, most recently from 59108d9 to 4177ab0 Compare August 11, 2025 10:49
@github-actions github-actions bot added size/XL and removed size/L labels Aug 11, 2025
@Rohit-0505 Rohit-0505 marked this pull request as ready for review August 11, 2025 10:58
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch 3 times, most recently from b81269c to 6f5d001 Compare August 18, 2025 05:27
@Rohit-0505 Rohit-0505 marked this pull request as draft August 20, 2025 09:22
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch 2 times, most recently from 21c9f33 to bdc42e7 Compare August 21, 2025 10:09
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch 2 times, most recently from ca6b233 to 23e6e12 Compare August 22, 2025 04:27
@Rohit-0505 Rohit-0505 marked this pull request as ready for review August 22, 2025 04:33
@Rohit-0505 Rohit-0505 marked this pull request as draft August 25, 2025 11:26
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/enh/network-downward-api-labels-support branch from dd4bc33 to cfdfc16 Compare August 26, 2025 11:08
@Rohit-0505 Rohit-0505 marked this pull request as ready for review August 26, 2025 11:21
@lukasfrank lukasfrank merged commit 0d738b9 into ironcore-dev:main Aug 28, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this to Done in Roadmap Aug 28, 2025
@github-project-automation github-project-automation bot moved this to ✅ Done in IaaS Aug 28, 2025
@Rohit-0505 Rohit-0505 changed the title Add downwardAPILabel support for Network resource Add downwardAPILabel support for Network and updated for NetworkInterface resource Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/iaas Issues related to IronCore IaaS development. enhancement New feature or request size/XL

Projects

Status: ✅ Done
Status: Done

Development

Successfully merging this pull request may close these issues.

Add downward-api label support to other resources similar to machine

5 participants