Move pympler back into the all extras

David Robertson · David Robertson · commit 108ee83566d8 · 2022-05-06T13:43:43.000+01:00
Undoes a change I made in #12381. I can't fully remember my reasoning, but this changed the contents of the debian packages in a backwards incompatible way. We're not aware of anyone who's been bitten by this, but we still want to fix it. To the reviewer: please be convinced that the debian packages will still contain pympler after this change.
diff --git a/changelog.d/12652.misc b/changelog.d/12652.misc
@@ -0,0 +1 @@
+Move `pympler` back in to the `all` extras.
diff --git a/debian/build_virtualenv b/debian/build_virtualenv
@@ -41,7 +41,6 @@ poetry export \
     --extras all \
     --extras test \
     --extras systemd \
-    --extras cache_memory \
     -o exported_requirements.txt
 deactivate
 rm -rf "$TEMP_VENV"
diff --git a/poetry.lock b/poetry.lock
diff --git a/pyproject.toml b/pyproject.toml
@@ -231,10 +231,11 @@ all = [
     "jaeger-client", "opentracing",
     # jwt
     "pyjwt",
-    #redis
-    "txredisapi", "hiredis"
+    # redis
+    "txredisapi", "hiredis",
+    # cache_memory
+    "pympler",
     # omitted:
-    #   - cache_memory: this is an experimental option
     #   - test: it's useful to have this separate from dev deps in the olddeps job
     #   - systemd: this is a system-based requirement
 ]

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Move `pympler` back in to the `all` extras.