-
Notifications
You must be signed in to change notification settings - Fork 576
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
NMS-15776: Parallelize DNS lookups during audit phase #6276
Conversation
...ond/src/main/java/org/opennms/netmgt/provision/service/operations/SaveOrUpdateOperation.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor text updates. Thank you!
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
docs/modules/operation/pages/deep-dive/provisioning/scalability.adoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Bonrob2 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Text LGTM! Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the smoke test failures for this look legitimate, marking this as "request changes" just so it shows as a thing that needs doing before this can be merged
The base branch was changed.
This reverts commit 44c22be.
e5b51f1
to
7d5d17d
Compare
Converting to Draft, since the smoke-tests are still failing. |
7d5d17d
to
aec3bf4
Compare
4488ac6
to
d59d13f
Compare
d59d13f
to
6617ae0
Compare
Finally ready for review. |
These changes parallelize the DNS lookups for interface addresses in the audit phase. The process waits for the searches to complete in the final call to the completeModelImport() method. This means that timeouts do not add up and delay the entire syncronization process.