Skip to content

feat: Add Metadata to OSResource #178

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
May 28, 2025
Merged

feat: Add Metadata to OSResource #178

merged 5 commits into from
May 28, 2025

Conversation

daniele-moro
Copy link
Contributor

Description

This pull request introduces a new metadata field to the OperatingSystemResource. The metadata field is designed to store opaque JSON metadata associated with an OS resource, with validation rules for length and pattern.

ITEP-67894

Any Newly Introduced Dependencies

How Has This Been Tested?

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

@sys-orch sys-orch added the apiv2 label May 26, 2025
SushilLakra
SushilLakra approved these changes May 28, 2025
View reviewed changes
Copy link
Contributor

@SushilLakra SushilLakra left a comment

Choose a reason for hiding this comment

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

LGTM

@daniele-moro daniele-moro enabled auto-merge (squash) May 28, 2025 15:44
@daniele-moro daniele-moro merged commit 7e77d5e into main May 28, 2025
20 checks passed
@daniele-moro daniele-moro deleted the osprof_meta branch May 28, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@raphaelvrosa raphaelvrosa raphaelvrosa approved these changes

@SushilLakra SushilLakra SushilLakra approved these changes

@Andrea-Campanella Andrea-Campanella Awaiting requested review from Andrea-Campanella Andrea-Campanella is a code owner

@cjnolan cjnolan Awaiting requested review from cjnolan cjnolan is a code owner

@damiankopyto damiankopyto Awaiting requested review from damiankopyto damiankopyto is a code owner

@ikyrycho ikyrycho Awaiting requested review from ikyrycho ikyrycho is a code owner

@jkossak jkossak Awaiting requested review from jkossak jkossak is a code owner

@ktaube26 ktaube26 Awaiting requested review from ktaube26 ktaube26 is a code owner

@niket-intc niket-intc Awaiting requested review from niket-intc niket-intc is a code owner

@osinstom osinstom Awaiting requested review from osinstom osinstom is a code owner

@Paiadhithi Paiadhithi Awaiting requested review from Paiadhithi Paiadhithi is a code owner

@PalashGoelIntel PalashGoelIntel Awaiting requested review from PalashGoelIntel PalashGoelIntel is a code owner

@pierventre pierventre Awaiting requested review from pierventre pierventre is a code owner

@rranjan3 rranjan3 Awaiting requested review from rranjan3 rranjan3 is a code owner

@soniabha-intc soniabha-intc Awaiting requested review from soniabha-intc soniabha-intc is a code owner

@sunil-parida sunil-parida Awaiting requested review from sunil-parida sunil-parida is a code owner

@tmatenko tmatenko Awaiting requested review from tmatenko tmatenko is a code owner

@zdw zdw Awaiting requested review from zdw zdw is a code owner

@krishnajs krishnajs Awaiting requested review from krishnajs krishnajs is a code owner

Assignees
No one assigned
Labels
apiv2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@daniele-moro @raphaelvrosa @SushilLakra @sys-orch