Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cephadm-adopt: Fixes binding network for alertmanager #7576

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

asm0deuz
Copy link
Collaborator

Alertmanager was bind to default * network instead of grafana_server_addr as it was before. Now on if grafana_server_addr is defined, it will be bind to that network.

Alertmanager was bind to default * network instead of grafana_server_addr
as it was before. Now on if grafana_server_addr is defined, it will be
bind to that network.

Signed-off-by: Teoman ONAY <tonay@ibm.com>
@asm0deuz
Copy link
Collaborator Author

jenkins test centos-container-cephadm_adopt

@asm0deuz
Copy link
Collaborator Author

jenkins test centos-non_container-update

@asm0deuz
Copy link
Collaborator Author

jenkins test centos-non_container-all_daemons

Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jul 29, 2024
@guits guits removed the stale label Jul 30, 2024
@asm0deuz asm0deuz merged commit 0bf3398 into ceph:main Aug 5, 2024
28 checks passed
@asm0deuz asm0deuz deleted the BZ2269009_alertmanager branch August 5, 2024 09:29
asm0deuz added a commit to asm0deuz/ceph-ansible that referenced this pull request Oct 2, 2024
Regression from ceph#7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
asm0deuz added a commit to asm0deuz/ceph-ansible that referenced this pull request Oct 11, 2024
Regression from ceph#7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
asm0deuz added a commit to asm0deuz/ceph-ansible that referenced this pull request Oct 12, 2024
Regression from ceph#7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
asm0deuz added a commit that referenced this pull request Oct 14, 2024
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
mergify bot pushed a commit that referenced this pull request Oct 14, 2024
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 31be495)

# Conflicts:
#	infrastructure-playbooks/cephadm-adopt.yml
mergify bot pushed a commit that referenced this pull request Oct 14, 2024
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 31be495)
asm0deuz added a commit that referenced this pull request Oct 14, 2024
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 31be495)
asm0deuz added a commit that referenced this pull request Oct 14, 2024
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 31be495)
asm0deuz added a commit that referenced this pull request Oct 16, 2024
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 31be495)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants