Documentation points to folder that doesn't exist in the product

Description

asks to cope from ${OPENNMS_HOME}/etc/examples/opennms.properties.d/hosted_tsdb.timeseries.properties. The opennms.properties.d folder doesn’t exist under examples

 

Target tix to latest foundation branch.

Acceptance / Success Criteria

None

Activity

Mark Mahacek March 6, 2023 at 8:32 PM

I haven’t had a chance to re-review yet. Will take a look tomorrow.

Ivan Trechekas March 3, 2023 at 7:52 PM
Edited

if you don’t have a complaint about skipped properties I’m ready to merge it.

As Freddy mentioned other properties are for off-heap feature, not sure they are necessary since for testing cloud plugin with TSaaS I didn’t use them at all. My properties that I set described in other section (Enable Time Series Integration Layer). We could add some references for other sections kinda:

You are able to add configuration for the Integration level configuration ( ) and off-heap configuration ( ) there.

Bonnie Robinson March 3, 2023 at 1:47 PM

.Thank you!

Ivan Trechekas March 2, 2023 at 11:20 PM

I spoke with involved people and prepared the PR


so, the main idea here: leaving examples out of the container. to keep the container clean and simple. So I replaced the line with cp command by mandatory specific property line (and skipped other optional lines, since we could specify more properties from , but it will overload the doc).

Also I updated a bit other sentences.

Ivan Trechekas March 2, 2023 at 3:38 PM
Edited

we can ask Marteza to copy that file with the kar file, then it will exists and won’t need any changes in Docs.

we can past the content in the doc instead of the file reference (not sure that we will change it somewhen), and add a notice that if you install plugin as rpm/dep package you can take it from this place ... so, both ways should work.

Fixed

Details

Assignee

Reporter

Story Points

Original estimate

Time tracking

No time logged4h remaining

Sprint

Priority

PagerDuty

Created February 7, 2023 at 6:11 PM
Updated March 6, 2023 at 8:32 PM
Resolved March 6, 2023 at 8:31 PM