Skip to content
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.

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

Improve isAgentCanDo() to check module installed flags #627

Merged
merged 9 commits into from
Jan 31, 2025

Conversation

eduardomozart
Copy link
Contributor

By default, the Deploy Package uses isAgentCanDo() function to determine which hosts to install a package, but it was also offering to install packages on ESXi remove inventory hosts which doesn't have the Deploy module installed. This PR ensures that a host which doesn't have a module installed be available to select packages to install.

Checklist before requesting a review

Please delete options that are not relevant.

  • 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.
  • This change requires a documentation update.

Description

  • It fixes # (issue number, if applicable)
  • Here is a brief description of what this PR does

Screenshots (if appropriate):

By default, the Deploy Package uses ``isAgentCanDo()`` function to determine which hosts to install a package, but it was also offering to install packages on ESXi remove inventory hosts which doesn't have the Deploy module installed. This PR ensures that a host which doesn't have a module installed be available to select packages to install.
@eduardomozart eduardomozart changed the title Improve isAgentCanDo to check module installed flags Improve isAgentCanDo() to check module installed flags Jan 31, 2025
eduardomozart added a commit to eduardomozart/glpi-inventory-plugin that referenced this pull request Jan 31, 2025
Copy link
Collaborator

@stonebuzz stonebuzz left a comment

Choose a reason for hiding this comment

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

Seems ok for me

@stonebuzz stonebuzz requested a review from trasher January 31, 2025 09:58
Copy link
Collaborator

@trasher trasher left a comment

Choose a reason for hiding this comment

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

Seems correct from a code point of view, I have no way to test.

@trasher trasher merged commit e1563fe into glpi-project:main Jan 31, 2025
8 checks passed
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