Skip to content

Conversation

@secDre4mer
Copy link
Contributor

No description provided.

@secDre4mer secDre4mer requested review from Copilot and gremat January 29, 2026 12:25
@secDre4mer secDre4mer self-assigned this Jan 29, 2026
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 adds additional host information fields to the AIX platform information structure, enhancing the ability to capture system configuration details specific to AIX systems.

Changes:

  • Added OS build version tracking
  • Added LPAR (Logical Partition) detection and configuration details
  • Added memory and vCPU configuration bounds for LPAR systems

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@secDre4mer secDre4mer force-pushed the chore/aix-lpar-config branch from 3fee809 to b5adb8d Compare January 30, 2026 09:27
@secDre4mer secDre4mer requested a review from Copilot January 30, 2026 09:28
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 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

// Whether this system is running in an LPAR partition
LparPartition bool `json:"lpar_partition" textlog:"lpar_partition"`

// LPAR configuration regarding memory / CPUs
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should note here what to expect from those values if the system is not virtualized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants