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

Feature/component monitoring #118

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

patrickskowronekdkfz
Copy link
Contributor

This PR adds the bridgehead monitoring component.
It allows Monitoring tools like icinga2 to check running component.

Copy link
Member

@TKussel TKussel left a comment

Choose a reason for hiding this comment

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

In the BBMRI case, the monitoring app is not added to the Beam-Proxy apps.
The Monitoring components for both projects are missing a container_name directive

Sidenote: Please ensure, that your application can deal with a delayed-started beam Proxy. depends_on is not sufficient, as the container might be running but the Beam Proxy not yet fully operational.

@TKussel TKussel changed the title Feature/component moniotring Feature/component monitoring Sep 5, 2023
@Threated
Copy link
Member

Threated commented Sep 7, 2023

I made sure that this component is very silent on startup when it is not able to reach the proxy yet. No beam-connect type of endless retrying spamming the logs.

Copy link
Member

@lablans lablans left a comment

Choose a reason for hiding this comment

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

Also, before we merge, we need to communicate these changes in the individual projects

bbmri/modules/eric-compose.yml Outdated Show resolved Hide resolved
bbmri/modules/gbn-compose.yml Outdated Show resolved Hide resolved
ccp/docker-compose.yml Outdated Show resolved Hide resolved
ccp/docker-compose.yml Outdated Show resolved Hide resolved
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.

4 participants