Skip to content

feat: Add description field to the OSResource in Inventory, REST API and OSProfile#3

Merged
daniele-moro merged 9 commits intomainfrom
add-description-os-resource-api
May 21, 2025
Merged

feat: Add description field to the OSResource in Inventory, REST API and OSProfile#3
daniele-moro merged 9 commits intomainfrom
add-description-os-resource-api

Conversation

@daniele-moro
Copy link
Copy Markdown
Contributor

@daniele-moro daniele-moro commented Apr 1, 2025

Description

This pull request introduces a new description field to the OperatingSystemResource schema and updates related files to support this addition. Additionally, it includes dependency updates in the go.mod files.

Schema and Feature Enhancements:

  • Added a description field to the OperatingSystemResource schema in multiple OpenAPI and protobuf files. This field supports a maximum length of 1000 characters and includes a regex pattern for validation.

Fixes ITEP-21413

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

Comment thread api/api/openapi/edge-infrastructure-manager-openapi-all.yaml
Comment thread api/internal/worker/handlers/invhandlers/os.go
Comment thread api/internal/worker/handlers/invhandlers/os.go
Comment thread inventory/api/os/v1/os.proto
@github-actions
Copy link
Copy Markdown

This pull request is stale because it has been open 30 days with no activity. Make a comment or update the PR to avoid closing PR after 15 days.

@daniele-moro daniele-moro changed the title Add description field to the OSResource in Inventory and REST API feat: Add description field to the OSResource in Inventory, REST API and OSProfile May 20, 2025
Copy link
Copy Markdown

@raphaelvrosa raphaelvrosa left a comment

Choose a reason for hiding this comment

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

Set 0.0.4-dev in apiv2 version.

@daniele-moro daniele-moro enabled auto-merge (squash) May 21, 2025 08:09
@daniele-moro daniele-moro merged commit 386228c into main May 21, 2025
45 checks passed
@daniele-moro daniele-moro deleted the add-description-os-resource-api branch May 21, 2025 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants