Merge ~seyeongkim/charm-graylog:increase_timeout_for_request into charm-graylog:master

Proposed by Seyeong Kim
Status: Merged
Approved by: Erhan Sunar
Approved revision: 43e4cb8232fdea674cfaa38ecfa324f78a295da3
Merged at revision: c9843b7aee427aacd795f1be13317b338a0ea4a9
Proposed branch: ~seyeongkim/charm-graylog:increase_timeout_for_request
Merge into: charm-graylog:master
Diff against target: 64 lines (+4/-17)
2 files modified
src/lib/charms/layer/graylog/api.py (+1/-1)
src/reactive/graylog.py (+3/-16)
Reviewer Review Type Date Requested Status
Ramesh Sattaru (community) Approve
Erhan Sunar (community) Approve
🤖 prod-jenkaas-bootstack (community) continuous-integration Approve
Review via email: mp+436739@code.launchpad.net

Commit message

increasing timeout for request

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Erhan Sunar (esunar) wrote :

There is a linting error(it blocks running of functional tests) can you fix?

black --check --exclude '/(\.eggs|\.git|\.tox|\.venv|\.build|dist|charmhelpers|mod)/' .

review: Needs Fixing
Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :
review: Approve (continuous-integration)
Revision history for this message
Erhan Sunar (esunar) :
review: Approve
Revision history for this message
Ramesh Sattaru (rameshcan) :
review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision c9843b7aee427aacd795f1be13317b338a0ea4a9

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/src/lib/charms/layer/graylog/api.py b/src/lib/charms/layer/graylog/api.py
index 7445fa6..8640650 100644
--- a/src/lib/charms/layer/graylog/api.py
+++ b/src/lib/charms/layer/graylog/api.py
@@ -59,7 +59,7 @@ class GraylogApi:
59 self.token = None59 self.token = None
60 self.input_types = None60 self.input_types = None
61 self.req_retries = 461 self.req_retries = 4
62 self.req_timeout = 362 self.req_timeout = 10
63 self.verify = verify63 self.verify = verify
6464
65 @retry(65 @retry(
diff --git a/src/reactive/graylog.py b/src/reactive/graylog.py
index 640e262..5e81f22 100644
--- a/src/reactive/graylog.py
+++ b/src/reactive/graylog.py
@@ -598,11 +598,8 @@ def graylog_remove_filebeat_input():
598 inputs = g.log_input_get()598 inputs = g.log_input_get()
599599
600 if inputs is not None:600 if inputs is not None:
601
602 for inp in inputs:601 for inp in inputs:
603
604 if inp["title"].lower() == "juju_beats_input":602 if inp["title"].lower() == "juju_beats_input":
605
606 g.log_input_remove(inp["id"])603 g.log_input_remove(inp["id"])
607 hookenv.log(604 hookenv.log(
608 "Removed beats input: {} ({})".format(inp["title"], inp["id"])605 "Removed beats input: {} ({})".format(inp["title"], inp["id"])
@@ -933,10 +930,7 @@ def setup_gelf_tcp_input(gelf):
933 "title": "juju_gelf_tcp_input",930 "title": "juju_gelf_tcp_input",
934 "type": type,931 "type": type,
935 "global": "true",932 "global": "true",
936 "configuration": {933 "configuration": {"bind_address": "0.0.0.0", "port": gelf_port},
937 "bind_address": "0.0.0.0",
938 "port": gelf_port,
939 },
940 }934 }
941935
942 proto = "TCP"936 proto = "TCP"
@@ -1107,11 +1101,7 @@ def validate_jvm_heap_size(str_size, total_mem=0):
1107 raise ValueError("jvm heap size must be a multiple of 1024")1101 raise ValueError("jvm heap size must be a multiple of 1024")
1108 else:1102 else:
1109 # only allow these units, and scale is 10241103 # only allow these units, and scale is 1024
1110 units = {1104 units = {"k": 1024, "m": 1024 * 1024, "g": 1024 * 1024 * 1024}
1111 "k": 1024,
1112 "m": 1024 * 1024,
1113 "g": 1024 * 1024 * 1024,
1114 }
1115 for unit, scale in units.items():1105 for unit, scale in units.items():
1116 if str_size.endswith(unit):1106 if str_size.endswith(unit):
1117 str_size = str_size[:-1] # rm unit1107 str_size = str_size[:-1] # rm unit
@@ -1517,10 +1507,7 @@ def _maybe_install_ca_certificates_hook():
15171507
1518def _maybe_configure_graylog_jvm_keystore():1508def _maybe_configure_graylog_jvm_keystore():
1519 templating.render(1509 templating.render(
1520 "default-graylog-server",1510 "default-graylog-server", SERVER_DEFAULT_CONF_FILE, context={}, perms=0o644
1521 SERVER_DEFAULT_CONF_FILE,
1522 context={},
1523 perms=0o644,
1524 )1511 )
15251512
15261513

Subscribers

People subscribed via source and target branches

to all changes: