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

add hardware id on ipdiscover #270

Merged

Conversation

thfiliol1
Copy link

Important notes

OCSInventory-server : Communication server that manage the inventory

General informations :

Operating system : CentOS 7

Server informations :

PHP Version : 7.2.8
Web Server : Apache/2.4.6 (CentOS) OpenSSL/1.0.2k-fips mod_wsgi/3.4 Python/2.7.5 PHP/7.2.8 mod_perl/2.0.11 Perl/v5.16.3
Database Server : MariaDB Server version 10.4.12-MariaDB-log
Version OCSReports: 2.8

Status :

READY

Description :

  • Improved ipdiscover, discovery of assets linked to the machine that discovered #them

assets in ipdiscover linked to the machine that discovers them
@gillesdubois
Copy link
Member

Hi @thfiliol1

THanks for your contribution.

However, the hardware ID is not present at all in this table (netmaps) by default.
Your PR seem ok for me but we need to create the related PR on ocs reports to add / alter the database model.

It might take some time for us to perform these tests / changes but don't worry it will be merged asap :)

Regards,
Gilles.

@Neustradamus
Copy link

Any news about this PR?

@charleneauger charleneauger merged commit 2549ad7 into OCSInventory-NG:master Oct 29, 2021
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.

4 participants