~xavpaice/charm-graylog:bug/1925272

Branch merges

Propose for merging

Rejected for merging into charm-graylog:master

Celia Wang: Approve on 2021-07-28

🤖 prod-jenkaas-bootstack (community): Approve (continuous-integration) on 2021-06-30

Drew Freiberger (community): Needs Fixing on 2021-06-29

Tom Haddon: Abstain on 2021-06-22

BootStack Reviewers: Pending (mr tracking; do not claim) requested 2022-06-11

BootStack Reviewers: Pending requested 2022-06-11

Diff: 477 lines (+190/-32)

11 files modified

src/config.yaml (+10/-2)
src/reactive/graylog.py (+28/-9)
src/tests/functional/requirements.txt (+1/-0)
src/tests/functional/tests/bundles/bionic-graylog2-tls.yaml (+53/-0)
src/tests/functional/tests/bundles/bionic-graylog3-tls.yaml (+4/-4)
src/tests/functional/tests/bundles/bionic-graylog3.yaml (+4/-4)
src/tests/functional/tests/bundles/bionic-ha.yaml (+4/-4)
src/tests/functional/tests/bundles/focal-graylog2-tls.yaml (+54/-0)
src/tests/functional/tests/tests.yaml (+2/-0)
src/tests/unit/test_graylog.py (+19/-8)
src/tox.ini (+11/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

f7b31f9... by Xav Paice on 2021-04-21

Enable config for Graylog2 TLS

Enables TLS relation for Graylog 2, which was allowed but not tested or
working previously.

Also forces elasticsearch to use a smaller heap so it can be tested on
smaller server instances.

dff5a06... by Xav Paice on 2021-06-23

Update unit name for nrpe functest

The functional test for NRPE checks is changed in this commit to use
'graylog/0' rather than 'nrpe/0' which can be a different unit number in
different models.

Reviewed-on: https://code.launchpad.net/~xavpaice/charm-graylog/+git/graylog-charm/+merge/401670
Reviewed-by: James Troup <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

aec0988... by Xav Paice on 2021-04-23

Update unit name for nrpe functest

The functional test for NRPE checks is changed in this commit to use
'graylog/0' rather than 'nrpe/0' which can be a different unit number in
different models.

ed75b69... by Xav Paice on 2021-06-18

Linting and add exclusions for Black/flake8 conflicts

9de3a43... by Heather Lemon on 2021-04-06

email config options lp#1794505

85cfd97... by Xav Paice on 2021-06-18

Update tests for new ubuntu charm

3b45c83... by Xav Paice on 2021-06-18

Reduce elastic heap size in tests

To allow functional tests to run in smaller VMs, reduce the es_heap_size
to 1G for the test bundles.

0644cef... by Joe Guo on 2021-05-05

src/config.yaml: adjust jvm_heap_size description to be more helpful

Signed-off-by: Joe Guo <email address hidden>

c26232b... by Joe Guo on 2021-05-04

src/tests/unit/test_graylog.py: add tests for validate_jvm_heap_size

Signed-off-by: Joe Guo <email address hidden>

cfe68aa... by Joe Guo on 2021-05-04

src/reactive/graylog.py: validate jvm heap size

if jvm_heap_size is configured to be greater than machine total mem,
graylog java process will keep getting killed and cause REST API to
fail without obvious error message.
Juju status will show "Waiting for: REST API" forever, and functional
tests will timeout/fail.

This patch will check jvm_heap_size to ensure it's valid and not too large.

LP: #1899158

Signed-off-by: Joe Guo <email address hidden>