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

add integration test for bluechi-is-online agent --monitor #1025

Merged

Conversation

nsimsolo
Copy link
Contributor

@nsimsolo nsimsolo commented Jan 14, 2025

Adding integration test for bluechi-is-online agent --monitor

--monitor keeps monitoring as long as agent is online and exits if it detects an offline state.

  1. Start node and agent, keep it running and verify --monitor does not return output.
  2. Stop agent and verify --monitor returns 1.

Fixes: #1021

@coveralls
Copy link

coveralls commented Jan 14, 2025

Coverage Status

coverage: 82.251% (+0.02%) from 82.236%
when pulling e54509f on nsimsolo:bluechi-is-online-agent-monitor
into 1cb7527 on eclipse-bluechi:main.

Copy link
Member

@engelmi engelmi left a comment

Choose a reason for hiding this comment

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

Small NIT comment, otherwise looks good to me.

@engelmi
Copy link
Member

engelmi commented Jan 14, 2025

Please remove the merge commit.

@nsimsolo nsimsolo force-pushed the bluechi-is-online-agent-monitor branch 2 times, most recently from 5109d6b to d4391f5 Compare January 14, 2025 15:04
@engelmi
Copy link
Member

engelmi commented Jan 15, 2025

@nsimsolo Please pull the latest changes from main. This should fix the failing pipelines.

@nsimsolo nsimsolo force-pushed the bluechi-is-online-agent-monitor branch from d4391f5 to e54509f Compare January 15, 2025 08:17
Copy link
Member

@engelmi engelmi left a comment

Choose a reason for hiding this comment

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

LGTM

@engelmi engelmi merged commit 5bc1b9a into eclipse-bluechi:main Jan 15, 2025
22 checks passed
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.

Add integration tests for bluechi-is-online agent with --monitor parameter
3 participants