Skip to content

Restore $jira::java_opts for compatibility and remove $jira::jvm_*_additional from the API, as setting JDK type to custom allows full control anyway.#351

Merged
oranenj merged 1 commit intovoxpupuli:masterfrom
cybercom-finland:java_opts
Apr 13, 2021
Merged

Conversation

@oranenj
Copy link
Copy Markdown
Contributor

@oranenj oranenj commented Apr 13, 2021

The _additional API is not useful at all; if people want to fully override their configuration, they can set their JDK type to custom

Test fixes missing for now, but I've tested this against an actual Jira instance

Comment thread manifests/init.pp
Comment thread manifests/init.pp Outdated
If people want to fully override things, they will simply
set JDK type to custom
@oranenj
Copy link
Copy Markdown
Contributor Author

oranenj commented Apr 13, 2021

squashed and reworded commit a bit

@bastelfreak bastelfreak added the bug Something isn't working label Apr 13, 2021
@oranenj oranenj changed the title Restore java_opts and get rid of the _additional configuration variables Restore $jira::java_opts for compatibility and remove $jira::jvm_*_additional from the API, as setting JDK type to custom allows full control anyway. Apr 13, 2021
@oranenj oranenj merged commit bca4365 into voxpupuli:master Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants