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 (backport #7576) #7591

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Aug 5, 2024

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.


This is an automatic backport of pull request #7576 done by Mergify.

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 <[email protected]>
(cherry picked from commit 0bf3398)
@asm0deuz
Copy link
Collaborator

asm0deuz commented Aug 5, 2024

jenkins test centos-non_container-update

2 similar comments
@asm0deuz
Copy link
Collaborator

asm0deuz commented Aug 6, 2024

jenkins test centos-non_container-update

@asm0deuz
Copy link
Collaborator

asm0deuz commented Aug 6, 2024

jenkins test centos-non_container-update

@guits
Copy link
Collaborator

guits commented Aug 7, 2024

jenkins test centos-container-cephadm_adopt

@guits
Copy link
Collaborator

guits commented Aug 7, 2024

jenkins test centos-non_container-update

@guits
Copy link
Collaborator

guits commented Aug 7, 2024

the failure in non_container-update isn't related to this patch

@guits guits merged commit 6d3558f into stable-8.0 Aug 7, 2024
27 of 28 checks passed
@guits guits deleted the mergify/bp/stable-8.0/pr-7576 branch August 7, 2024 11:33
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