Skip to content

Commit

Permalink
Ansible role properities should be prefixed with role name
Browse files Browse the repository at this point in the history
Ansible linter give you a warning about properties not prefixed with role name. This PR should address this issue.
  • Loading branch information
wsmirnow committed Jan 31, 2025
1 parent 8efd24a commit 5f8f5b7
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ This role supports the following,
- `opencast_opensearch_heap_size`
- Memory configuration (default: `1g`)
- Might make sense to set this to `2g` for larger installations.
- `opensearch_api_host`
- `opencast_opensearch_api_host`
- Defaults to `127.0.0.1`.
- `opensearch_api_port`
- `opencast_opensearch_api_port`
- Defaults to `9200`.

## Dependencies
Expand Down
4 changes: 2 additions & 2 deletions defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
opencast_opensearch_heap_size: 1g

# Bind OpenSearch to this specific interface.
opensearch_api_host: 127.0.0.1
opencast_opensearch_api_host: 127.0.0.1

# Bind OpenSearch to this specific port.
opensearch_api_port: 9200
opencast_opensearch_api_port: 9200
4 changes: 2 additions & 2 deletions templates/opensearch.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
cluster.name: "opensearch-cluster"
node.name: "{{ inventory_hostname }}"
network.host: "{{ opensearch_api_host }}"
http.port: "{{ opensearch_api_port }}"
network.host: "{{ opencast_opensearch_api_host }}"
http.port: "{{ opencast_opensearch_api_port }}"
discovery.type: single-node
bootstrap.memory_lock: true
plugins.security.disabled: true

0 comments on commit 5f8f5b7

Please sign in to comment.