Merge v1.0.0-rc6xx into v1.x.x (#1042)

This pulls one bug fix from rc6xx that weren't ported to v1.x.x:

* **Fix gap in ring buffer when updating a missing value**

We merge the branches anyway to make it perfectly clear and obvious that
now all changes in 6xx were incorporated to v1.x.x.

Author: llucax

RELEASE_NOTES.md

@@ -59,3 +59,4 @@
 - Fix the handling of canceled tasks in the data sourcing and resampling actor.
 - Fix a bug in PV power distribution by excluding inverters that haven't sent any data since startup.
 - Prevent stacking of power requests to avoid delays in processing when the power request frequency exceeds the processing time.
+- Fixes a bug in the ring buffer in case the updated value is missing and creates a gap in time.

src/frequenz/sdk/timeseries/_ringbuffer/buffer.py

@@ -491,8 +491,11 @@ def _update_gaps(
         # New missing entry that is not already in a gap?
         if record_as_missing:
             if not found_in_gaps:
+                # If there are no gaps and the new value is not subsequent to the
+                # newest value, we need to start the new gap after the newest value
+                start_gap = min(newest + self._sampling_period, timestamp)
                 self._gaps.append(
-                    Gap(start=timestamp, end=timestamp + self._sampling_period)
+                    Gap(start=start_gap, end=timestamp + self._sampling_period)
                 )
         elif len(self._gaps) > 0:
             if found_in_gaps:

tests/timeseries/test_ringbuffer.py

@@ -275,12 +275,12 @@ def test_gaps() -> None:  # pylint: disable=too-many-statements
     assert buffer.count_covered() == 5
     assert len(buffer.gaps) == 0
 
-    # whole range gap suffers from sdk#646
+    # whole range gap
     buffer.update(Sample(dt(99), None))
-    assert buffer.oldest_timestamp == dt(95)  # bug: should be None
-    assert buffer.newest_timestamp == dt(99)  # bug: should be None
-    assert buffer.count_valid() == 4  # bug: should be 0 (whole range gap)
-    assert buffer.count_covered() == 5  # bug: should be 0
+    assert buffer.oldest_timestamp is None
+    assert buffer.newest_timestamp is None
+    assert buffer.count_valid() == 0
+    assert buffer.count_covered() == 0
     assert len(buffer.gaps) == 1