feat: bump otel-collector to 0.151.0

[sht-bot]

Automated formula bump for otel-collector.

• Previous version: 0.147.0
• New version: 0.151.0

[coderabbitai]

Walkthrough

The OpenTelemetry Collector Contrib Homebrew formula is updated from version 0.147.0 to 0.151.0, with corresponding changes to the download URL and SHA256 hash to match the new upstream release.

Changes

Cohort / File(s)	Summary
Version Update Formula/otel-collector.rb	Updated formula version from 0.147.0 to 0.151.0, including corresponding URL and SHA256 hash updates to reference the new upstream release.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• feat(otel-collector): add otel collector formula #1 — Originally introduced the OtelCollector formula; this PR updates the same formula to a newer version.
• chore(main): release 0.1.0 #3 — Previously modified the otel-collector formula; this PR continues version maintenance on the same formula.

Suggested reviewers

• yordis

Poem

🐰 Hops and brews, a tale to tell,
Version bumps, we all do well,
From 0.147, up we climb,
To 0.151, a newer time!
The collector dances, fresh and free,
Updated with integrity! 🥕

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: bumping the otel-collector formula version from 0.147.0 to 0.151.0.
Description check	✅ Passed	The description is directly related to the changeset, clearly stating the automated formula bump and version change details.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bump-otel-collector-0.151.0

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (2)

Formula/otel-collector.rb (2)

13-19: Service config/log paths look consistent, but consider ensuring log dir exists.

You set log_path var/"log/otelcol-contrib.log" and error_log_path var/"log/otelcol-contrib.err.log". The post_install currently only creates etc/"otelcol-contrib". If var/"log" isn’t created elsewhere by Homebrew services, first-run logging could fail.

Consider adding var/"log" creation in post_install (or confirming it’s handled automatically in your environment).

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Formula/otel-collector.rb` around lines 13 - 19, The service config sets
log_path and error_log_path to var/"log/otelcol-contrib.log" and
var/"log/otelcol-contrib.err.log" but post_install only creates
etc/"otelcol-contrib"; update the post_install method to ensure the var/"log"
directory exists (e.g., create or mkpath var/"log") before the service starts so
first-run logging won't fail; locate the post_install method and add the
directory creation adjacent to the existing etc/"otelcol-contrib" creation,
referencing the same var/"log" target used by log_path/error_log_path.

---

21-23: Minor: ensure etc/otelcol-contrib creation aligns with the caveats/config expectation.

post_install creates (etc/"otelcol-contrib"), and caveats + service both reference #{etc}/otelcol-contrib/config.yaml. Since no default config.yaml is created here, this is fine if the user supplies it, but it’s worth double-checking that your tap/formula conventions always provide a config (otherwise users get a “missing config” on start).

Not a blocker based on this PR alone.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Formula/otel-collector.rb` around lines 21 - 23, post_install currently only
ensures the directory (etc/"otelcol-contrib") exists but caveats and service
reference #{etc}/otelcol-contrib/config.yaml; update post_install (or the
formula's caveats/service text) to be consistent by either creating a default
config.yaml in post_install (e.g., install a minimal config template into
etc/"otelcol-contrib"/config.yaml) or modify the caveats/service descriptions to
explicitly state that the user must supply config.yaml before starting; locate
the logic in the post_install method and the caveats/service strings to make
this single consistent change.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@Formula/otel-collector.rb`:
- Around line 13-19: The service config sets log_path and error_log_path to
var/"log/otelcol-contrib.log" and var/"log/otelcol-contrib.err.log" but
post_install only creates etc/"otelcol-contrib"; update the post_install method
to ensure the var/"log" directory exists (e.g., create or mkpath var/"log")
before the service starts so first-run logging won't fail; locate the
post_install method and add the directory creation adjacent to the existing
etc/"otelcol-contrib" creation, referencing the same var/"log" target used by
log_path/error_log_path.
- Around line 21-23: post_install currently only ensures the directory
(etc/"otelcol-contrib") exists but caveats and service reference
#{etc}/otelcol-contrib/config.yaml; update post_install (or the formula's
caveats/service text) to be consistent by either creating a default config.yaml
in post_install (e.g., install a minimal config template into
etc/"otelcol-contrib"/config.yaml) or modify the caveats/service descriptions to
explicitly state that the user must supply config.yaml before starting; locate
the logic in the post_install method and the caveats/service strings to make
this single consistent change.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a4f6d3fb-eb79-40a7-8ccf-8423b217561d

📥 Commits

Reviewing files that changed from the base of the PR and between d51e5e8 and 4502a4d.

📒 Files selected for processing (1)

• Formula/otel-collector.rb