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

Fix partial network ports inventory #18074

Draft
wants to merge 2 commits into
base: 10.0/bugfixes
Choose a base branch
from

Conversation

trasher
Copy link
Contributor

@trasher trasher commented Oct 16, 2024

Description

Fix partial network ports inventory
closes #18062

@trasher
Copy link
Contributor Author

trasher commented Oct 16, 2024

As draft for now, I still have things to check (network ports...)

@trasher trasher marked this pull request as ready for review October 17, 2024 08:51
@g-bougard
Copy link
Member

Hi @trasher

thank you, I tested your patch, that's better. But I see last one problem: if the ip changes, the updated network card show 2 ips, the new and the previous in place of only the new one:
image
Here ipv4 & ipv6 was changed and old ips are kept for ipv4 & ipv6 where only new ones was expected.

@trasher trasher marked this pull request as draft October 17, 2024 09:49
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.

Partial inventory duplicates VPN networkport
2 participants