Skip to content

Conversation

@HJLebbink
Copy link
Member

@HJLebbink HJLebbink commented Oct 21, 2025

As requested by @donatello,

This PR removes the NodeHostname field from the InventoryJobStatus struct and its associated documentation. The change eliminates the ability to track which node is running an inventory job.

Key changes:

Removed NodeHostname field from InventoryJobStatus struct definition
Removed NodeHostname documentation from API documentation table

@HJLebbink HJLebbink requested a review from Copilot October 21, 2025 10:27
@HJLebbink HJLebbink self-assigned this Oct 21, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the NodeHostname field from the InventoryJobStatus struct and its associated documentation. The change eliminates the ability to track which node is running an inventory job.

Key changes:

  • Removed NodeHostname field from InventoryJobStatus struct definition
  • Removed NodeHostname documentation from API documentation table

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
api-inventory-ext.go Removed NodeHostname field from InventoryJobStatus struct
docs/API.md Removed NodeHostname field documentation from API reference table

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@HJLebbink HJLebbink marked this pull request as ready for review October 21, 2025 11:04
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@harshavardhana harshavardhana merged commit 9207380 into minio:master Oct 24, 2025
5 checks passed
@HJLebbink HJLebbink deleted the update-InventoryJobStatus branch October 27, 2025 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants