Skip to content

Conversation

@mershad-manesh
Copy link
Contributor

All Contributors

Contribution Checklist

  • Please make an issue in the OpenNMS issue tracker if there isn't one already.
    Once there is an issue, please:
    1. update the title of this PR to be in the format: ${JIRA-ISSUE-NUMBER}: subject of pull request
    2. update the Jira link at the bottom of this comment to refer to the real issue number
    3. prefix your commit messages with the issue number, if possible
    4. once you've created this PR, please link to it in a comment in the Jira issue
      Don't worry if this sounds like a lot, we can help you get things set up properly.
  • If this code is likely to affect the UI, did you name your branch with -smoke in it to trigger smoke tests?
  • If this is a new or updated feature, is there documentation for the new behavior?
  • If this is new code, are there unit and/or integration tests?
  • If this PR targets a foundation-* branch, does it try to avoid changing files in $OPENNMS_HOME/etc/?

What's Next?

A PR should be assigned at least 2 reviewers. If you know that someone would be a good person to review your code, feel free to add them.

If you need help making additions or changes to the documentation related to your changes, please let us know.

In any case, if anything is unclear or you want help getting your PR ready for merge, please don't hesitate to say something in the comments here,
or in the #opennms-development chat channel.

Once reviewer(s) accept the PR and the branch passes continuous integration, the PR is eligible for merge.

At that time, if you have commit access (are an OpenNMS Group employee or a member of the OGP) you are welcome to merge the PR when you're ready.
Otherwise, a reviewer can merge it for you.

Thanks for taking time to contribute!

External References

@dino2gnt
Copy link
Contributor

dino2gnt commented Oct 16, 2025

On RHELatives:

[root@34horizon etc]# ls -l /opt/opennms/share
lrwxrwxrwx. 1 opennms opennms 12 Sep 11 14:30 /opt/opennms/share -> /var/opennms
[root@34horizon etc]# ls -l /var/opennms
total 24
drwxrwxr-x. 28 opennms opennms 12288 Sep 11 14:30 etc-pristine
drwxrwxr-x.  4 opennms opennms    56 Sep 17 15:22 mibs
drwxrwxr-x.  2 opennms opennms    21 Sep 11 14:30 reports
drwxrwxr-x.  5 opennms opennms    63 Sep 11 14:30 rrd
drwxrwxr-x.  3 opennms opennms   109 Sep 11 14:30 web-inf-pristine
drwxrwxr-x.  2 opennms opennms  4096 Sep 11 14:30 xsds

On these debs, no share:

root@34horizon-deb:/opt/opennms# ls -l /opt/opennms
total 36
drwxr-xr-x  2 opennms opennms  4096 Oct 16 15:23 agent
drwxr-xr-x  2 opennms opennms  4096 Oct 16 15:23 bin
drwxr-xr-x  2 opennms opennms  4096 Oct 16 15:24 data
drwxr-xr-x  2 opennms opennms  4096 Oct 16 15:23 deploy
drwxr-xr-x 30 opennms opennms 12288 Oct 16 15:24 etc
drwxr-xr-x  4 opennms opennms  4096 Oct 16 15:23 jetty-webapps
lrwxrwxrwx  1 opennms opennms    24 Oct 16 15:23 lib -> /usr/share/java/opennms/
drwxr-xr-x 28 opennms opennms  4096 Oct 16 15:23 system
root@34horizon-deb:/opt/opennms# ls -l /var/lib/opennms
total 28
drwxr-xr-x 29 opennms opennms 12288 Oct 16 15:23 etc-pristine
drwxr-xr-x  3 opennms opennms  4096 Oct 16 15:23 mibs
drwxr-xr-x  2 opennms opennms  4096 Oct 16 14:25 reports
drwxr-xr-x  3 opennms opennms  4096 Oct 16 15:23 web-inf-pristine
drwxr-xr-x  2 opennms opennms  4096 Oct 16 15:23 xsds

Also no rrd, but that may be because this instance has never started.

Copy link
Contributor

@marshallmassengill marshallmassengill left a comment

Choose a reason for hiding this comment

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

Doc changes seem good.

Copy link
Contributor

@synqotik synqotik left a comment

Choose a reason for hiding this comment

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

Code changes LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants