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

test: add coverage for service creation after installing a container #45

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

reubenmiller
Copy link
Contributor

Due to a bug in the tedge-container-plugin-ng s6-overlay definition, services representing the containers wasn't being created due to a permissions problem.

The system test now covers the functionality.

Copy link

github-actions bot commented Nov 21, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
13 0 1 13 100 1m16.542067999s

Passed Tests

Name ⏱️ Duration Suite
Grace period to allow container to startup 5.008 s Operations
tedge-container-plugin service is up 0.106 s Operations
Get Logfile Request 2.336 s Operations
Get Configuration File 4.661 s Operations
Execute Shell Command 2.321 s Operations
Install application using docker compose 13.182 s Operations
Get Container Logs 2.318 s Operations
Trigger self update via local command 11.419 s Self-Update
Self update should only update if there is a new image 4.422 s Self-Update
Self update using software update operation 28.000 s Self-Update
Cloud Connection is Online 0.116 s Telemetry
Service status 0.219 s Telemetry
Sends measurements 2.431 s Telemetry

@reubenmiller reubenmiller force-pushed the test-container-monitoring branch from c9db12f to 5c62642 Compare November 21, 2024 23:49
@reubenmiller reubenmiller added this pull request to the merge queue Nov 21, 2024
Merged via the queue into main with commit 4e2a310 Nov 21, 2024
2 checks passed
@reubenmiller reubenmiller deleted the test-container-monitoring branch November 25, 2024 10:36
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.

1 participant