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

fix(self-update): remove updater container after collecting logs #53

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

reubenmiller
Copy link
Contributor

@reubenmiller reubenmiller commented Dec 4, 2024

Remove the self-update container after the self update has completed (and the logs have been collected from it).

Copy link

github-actions bot commented Dec 4, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
19 0 1 19 100 8m24.469108999s

Passed Tests

Name ⏱️ Duration Suite
Ignore Containers Marked With A Specific Label 27.853 s Container
Grace period to allow container to startup 5.052 s Operations
Service is up 0.170 s Operations
Get Logfile Request 3.163 s Operations
Get Configuration File 4.811 s Operations
Execute Shell Command 2.433 s Operations
Install application using docker compose 15.535 s Operations
Get Container Logs 2.410 s Operations
Get Container Logs without explicit container name 2.422 s Operations
Get Container Logs For Non-existent container 2.414 s Operations
Update From Legacy Versions 139.292 s Self-Update-Compat
Trigger self update via local command 27.861 s Self-Update
Self update should only update if there is a new image 29.470 s Self-Update
Self update using software update operation 105.766 s Self-Update
Rollback when trying to install a non-tedge based image 23.921 s Self-Update
Self update using software update operation using Container type 108.764 s Self-Update
Cloud Connection is Online 0.184 s Telemetry
Service status 0.317 s Telemetry
Sends measurements 2.581 s Telemetry

@reubenmiller reubenmiller added this pull request to the merge queue Dec 4, 2024
Merged via the queue into main with commit cd428a4 Dec 4, 2024
8 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.

1 participant