Skip to content

Feature/grafana mongo advanced - #748

Open
crosmuller wants to merge 11 commits into
mainfrom
feature/grafana_mongo_advanced
Open

crosmuller wants to merge 11 commits into
mainfrom
feature/grafana_mongo_advanced

Conversation

@crosmuller

Copy link
Copy Markdown
Contributor
  • custom mongo roles
  • prometheus frontend option in haproxy


{% endif -%}


Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wat doet dit? En weet je zeker dat je gene logd van wilt?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En waarom moet het eigenljk via de loadbalancer?

@@ -1,34 +1,76 @@
- name: Create mongo database users cluster # requires pymongo 4+
- name: Create mongo database users and roles cluster # requires pymongo 4+
when:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moet hier niet

  throttle: 1

bij?

name: "{{ item.name }}"
password: "{{ item.password }}"
roles: "{{ item.role | default('readWrite')}}"
replica_set: "{{ mongo_replica_set_name }}"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is wel veel duplicatie zo. Het enige verschil lijkt die replica_set. Werkt het niet om die leeg of false te maken voor de standalone casus?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants