Skip to content

Updated registry collect - #974

Merged
trasher merged 22 commits into
glpi-project:mainfrom
g-bougard:feature/updated-registry-collect
Aug 12, 2026
Merged

Updated registry collect#974
trasher merged 22 commits into
glpi-project:mainfrom
g-bougard:feature/updated-registry-collect

Conversation

@g-bougard

Copy link
Copy Markdown
Member

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.

Description

Add registry collect modes: Check path existence and Check if key is defined and Add a Recursion depth option to the registry collect to read sub-keys recursively.

@g-bougard
g-bougard force-pushed the feature/updated-registry-collect branch from 8a4cf3d to 81e71ce Compare July 6, 2026 14:16
@g-bougard
g-bougard requested review from stonebuzz and trasher July 6, 2026 14:19
@g-bougard

Copy link
Copy Markdown
Member Author

Agent-side the dev is submitted in glpi-project/glpi-agent#1201

Agent will test glpi-inventory is at least v1.6.9 before enabling the feature. I'll have to change the related test if this PR is not integrated in 1.6.9.

@trasher trasher left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM from a code point of view. Can't test

@stonebuzz stonebuzz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@stonebuzz

Copy link
Copy Markdown
Collaborator

Awaiting customer feedback.

@trasher
trasher merged commit 835031a into glpi-project:main Aug 12, 2026
7 checks passed
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.

4 participants