Skip to content

feat: queue pending service deletion api calls if the immediate removal fails#183

Merged
reubenmiller merged 2 commits intomainfrom
feat-delete-services-after-network-outage
Mar 25, 2026
Merged

feat: queue pending service deletion api calls if the immediate removal fails#183
reubenmiller merged 2 commits intomainfrom
feat-delete-services-after-network-outage

Conversation

@reubenmiller
Copy link
Collaborator

@reubenmiller reubenmiller commented Mar 23, 2026

Improve the reliability of the service deletions from the cloud in the event that the local Cumulocity proxy (hosted by the tedge-mapper-c8y service) is not available at the time of deletion.

The tedge-container-plugin service will now:

  • queue failed service requests and try to delete them on the next status check
  • trigger a resync when the tedge-mapper-service is discoverred to be healthy again

Resolves #182

@github-actions
Copy link

github-actions bot commented Mar 23, 2026

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
39 0 0 39 100 9m27.575931s

Passed Tests

Name ⏱️ Duration Suite
Get Container Logs 5.529 s Container-Logs
Get Container Logs with only last N lines 0.630 s Container-Logs
Get Container Logs By Operation 9.685 s Container-Logs
Remove Container 8.602 s Container-Remove
Remove Container Non Existent Container Should Not Through An Error 0.126 s Container-Remove
Restart Container 0.604 s Container-Restart
Restart Unknown Container Fails 0.140 s Container-Restart
Update to tedge-container-plugin-ng 22.463 s Installation
Check for Update 6.933 s Operations-Clone
Clone Existing Container 27.743 s Operations-Clone
Clone Existing Container by Timeout Whilst Waiting For Exit 18.348 s Operations-Clone
Clone Existing Container but Waiting For Exit 25.544 s Operations-Clone
Ignore Containers With Given Label 21.314 s Operations-Clone
Install/uninstall container-image 16.495 s Operations-Container-Image
Install/uninstall not existent container image 6.777 s Operations-Container-Image
Install/uninstall container package from private repository - credentials file 12.621 s Operations-Private-Registries
Install/uninstall container package from private repository - credentials script 14.979 s Operations-Private-Registries
Install/uninstall container package from private repository - credentials script with cache 20.195 s Operations-Private-Registries
Install/uninstall container package from private repository - engine credentials 17.828 s Operations-Private-Registries
Install/uninstall container package from private repository - docker from docker 51.739 s Operations-Private-Registries
Get Configuration 5.069 s Operations
Install/uninstall container-group package 19.101 s Operations
Install/uninstall container-group package with non-existent image 11.080 s Operations
Install invalid container-group 3.154 s Operations
Install container-group with multiple files - app1 15.961 s Operations
Install container-group with multiple files - app2 7.646 s Operations
Install/uninstall container package 17.313 s Operations
Install/uninstall container package from file 11.523 s Operations
Manual container creation/deletion 17.428 s Operations
Manual container creation/deletion with error on run 7.385 s Operations
Manual container created and then killed 10.131 s Operations
Remove Orphaned Cloud Services 19.551 s Operations
Remove Orphaned Cloud Services eventually if Cumulocity Proxy is Unavailable at deletion time 39.070 s Operations
Install container group that uses host volume mount 7.744 s Operations
Self Update Is Present Using Self Type 0.189 s Self
Self Update Is Present Using Container Type 0.190 s Self
Self Update Is Not Present 0.191 s Self
Service status 17.070 s Telemetry-Main
Sends measurements 69.488 s Telemetry-Main

@reubenmiller reubenmiller force-pushed the feat-delete-services-after-network-outage branch from a921865 to 6668823 Compare March 23, 2026 20:45
@reubenmiller reubenmiller temporarily deployed to Test Pull Request March 23, 2026 20:45 — with GitHub Actions Inactive
Using a unique MQTT client id is required in run-once modes as these instances should not affect the existing functionality of the monitoring service
@reubenmiller reubenmiller force-pushed the feat-delete-services-after-network-outage branch from 6668823 to adba611 Compare March 25, 2026 09:19
@reubenmiller reubenmiller deployed to Test Pull Request March 25, 2026 09:19 — with GitHub Actions Active
@reubenmiller reubenmiller added this pull request to the merge queue Mar 25, 2026
Merged via the queue into main with commit 00f1086 Mar 25, 2026
7 checks passed
@reubenmiller reubenmiller deleted the feat-delete-services-after-network-outage branch March 25, 2026 12:40
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.

Queue service sync requests when the Cumulocity local proxy service is not available or the device is offline

1 participant