append string to Message text for disabled messages

[vladak]

In order to make operational aspect easier, it would help if the cause of disabled projects could be propagated to the UI when using the opengrok-mirror script. The disabled: true behavior will be preserved however if the value is a string, it will be appended to the message. I.e. when running the tool with this config:

disabled_command:
  command:
    - http://localhost:8080/source/api/v1/messages
    - POST
    - messageLevel: info
      duration: PT5M
      tags: ['%PROJECT%']
      text: disabled project
projects:
  opengrok:
    disabled: true
  opengrok-1.0:
    disabled: true
    disabled-reason: "foo bar"

the opengrok project's disabled message will be:

{"messageLevel": "info", "duration": "PT5M", "tags": ["opengrok"], "text": "disabled project"}

while opengrok-1.0's will be:

{"messageLevel": "info", "duration": "PT5M", "tags": ["opengrok-1.0"], "text": "disabled project: foo bar"}

[ktulinger]

What about another configuration property like disabled-reason or similar instead of introducing this variable duality?

[vladak]

had to bump GitPython version due to gitpython-developers/GitPython#983

[vladak]

What about another configuration property like disabled-reason or similar instead of introducing this variable duality?

yes, that would be cleaner.

[vladak]

Thanks for the review !

[vladak]

will update https://github.com/oracle/opengrok/wiki/Repository-synchronization

[coveralls]

Pull Request Test Coverage Report for Build 5133

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 414 unchanged lines in 10 files lost coverage.
• Overall coverage decreased (-0.1%) to 73.349%

---

Files with Coverage Reduction	New Missed Lines	%
opengrok-indexer/src/main/java/org/opengrok/indexer/analysis/ExpandTabsReader.java	1	86.27%
opengrok-indexer/src/main/java/org/opengrok/indexer/search/QueryBuilder.java	1	78.57%
opengrok-indexer/src/main/java/org/opengrok/indexer/web/Prefix.java	1	97.06%
opengrok-indexer/src/main/java/org/opengrok/indexer/history/BazaarRepository.java	2	69.61%
opengrok-indexer/src/main/java/org/opengrok/indexer/util/LineBreaker.java	6	80.65%
opengrok-indexer/src/main/java/org/opengrok/indexer/util/SourceSplitter.java	10	79.59%
opengrok-indexer/src/main/java/org/opengrok/indexer/search/Results.java	30	0%
opengrok-indexer/src/main/java/org/opengrok/indexer/search/context/Context.java	33	59.41%
opengrok-indexer/src/main/java/org/opengrok/indexer/web/SearchHelper.java	103	25.31%
opengrok-indexer/src/main/java/org/opengrok/indexer/web/PageConfig.java	227	39.22%

Totals
Change from base Build 5125:	-0.1%
Covered Lines:	37793
Relevant Lines:	51525

---

💛 - Coveralls