search

drogue-iot / drogue-cloud

Cloud Native IoT
https://drogue.io
Apache License 2.0
114 stars 30 forks source link

Add a device state service #274

Closed ctron closed 2 years ago

ctron commented 2 years ago

This PR adds a device state service. The intended use case is to provide some internal functionality for tracking a device state, like MQTT's LWT, "device is connected", or to which pod a device is currently connected.

Part of this also is to handle the case that the original pod is no longer capable of destroying the device state, as it was terminated.

ctron commented 2 years ago

Big PR, but LGTM after skimming through! Should it be started by the drogue-cloud-server as well, or is this only required if using the persisted variant of the service?

You are right, I didn't yet check if this works with the server. Will do!

ctron commented 2 years ago

/test

drogue-bot commented 2 years ago

Aye, aye, captain! … Triggered System Test worklow!

drogue-bot commented 2 years ago
Total Passed Failed Ignored Filtered Duration
❌ 147 75 72 0 0 30m 51s

Git: https://github.com/drogue-iot/drogue-cloud @ refs/heads/feature/device_state_1

Commit: c195b71513c7ee658e405652db2acc41f271e745
Author: Jens Reimann <jreimann@redhat.com>
Date: Fri, 06 May 2022 12:38:48 -0200

    test: fix tests after API changes

Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/2282373794

Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/05/06/test-run-2282373794.html

ctron commented 2 years ago

/test

drogue-bot commented 2 years ago

Aye, aye, captain! … Triggered System Test worklow!

drogue-bot commented 2 years ago
Total Passed Failed Ignored Filtered Duration
❌ 147 105 42 0 0 31m 27s

Git: https://github.com/drogue-iot/drogue-cloud @ refs/heads/feature/device_state_1

Commit: c195b71513c7ee658e405652db2acc41f271e745
Author: Jens Reimann <jreimann@redhat.com>
Date: Fri, 06 May 2022 12:38:48 -0200

    test: fix tests after API changes

Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/2293216217

Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/05/09/test-run-2293216217.html

ctron commented 2 years ago

/test

drogue-bot commented 2 years ago

Aye, aye, captain! … Triggered System Test worklow!

drogue-bot commented 2 years ago
Total Passed Failed Ignored Filtered Duration
❌ 147 102 45 0 0 32m 16s

Git: https://github.com/drogue-iot/drogue-cloud @ refs/heads/feature/device_state_1

Commit: c195b71513c7ee658e405652db2acc41f271e745
Author: Jens Reimann <jreimann@redhat.com>
Date: Fri, 06 May 2022 12:38:48 -0200

    test: fix tests after API changes

Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/2293854671

Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/05/09/test-run-2293854671.html

ctron commented 2 years ago

/test

drogue-bot commented 2 years ago

Aye, aye, captain! … Triggered System Test worklow!

drogue-bot commented 2 years ago
Total Passed Failed Ignored Filtered Duration
✅ 147 147 0 0 0 31m 55s

Git: https://github.com/drogue-iot/drogue-cloud @ refs/heads/feature/device_state_1

Commit: f11d31b9d47776935226f82956462a28b9377c79
Author: Jens Reimann <jreimann@redhat.com>
Date: Tue, 10 May 2022 07:14:50 -0200

    chore: update charts

Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/2299655154

Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/05/10/test-run-2299655154.html