Remove more makefile, add missing line in versions (#221)

Fix two non-breaking problems discovered in the updated release process
as of v0.24.0.

Makefile

@@ -98,8 +98,6 @@ endif
 	git commit -m "multimode changes $(RELEASE_CANDIDATE)"
 	# ensure a clean branch (that was a test--gotidy should be idempotent and should not change the working dir again)
 	git diff -s --exit-code || (echo "local repository not clean"; exit 1)
-	git add .
-	git commit -m "remove replace statements $(RELEASE_CANDIDATE)" || (echo "no multimod changes to commit")
 
 # Install OTC's builder at the version WHICH MUST MATCH collector/otelarrowcol-build.yaml
 BUILDER = builder

pkg/otel/common/arrow/allocator.go

@@ -54,6 +54,10 @@ var limitRegexp = regexp.MustCompile(`memory limit exceeded: requested (\d+) out
 // the code below.  The formatting uses a "%v" which means we lose the
 // error wrapping facility that would let us easily extract the
 // object.  Therefore, we use a regexp to unpack memory limit errors.
+//
+// TODO: https://github.com/apache/arrow/pull/41989 has fixed the
+// upstream Arrow library, when Arrow v17 is released we can simplify
+// this function to unwrap an error vs parse the message.
 func NewLimitErrorFromError(err error) (error, bool) {
 	matches := limitRegexp.FindStringSubmatch(err.Error())
 	if len(matches) != 4 {

versions.yaml

@@ -16,3 +16,4 @@ module-sets:
       - github.com/open-telemetry/otel-arrow/collector/processor/concurrentbatchprocessor
       - github.com/open-telemetry/otel-arrow/collector/processor/obfuscationprocessor
       - github.com/open-telemetry/otel-arrow/collector/receiver/filereceiver
+      - github.com/open-telemetry/otel-arrow/collector/test