Skip to content

Conversation

@BrewTestBot
Copy link
Contributor

Created by brew bump


Created with brew bump-formula-pr.

  • resource blocks have been checked for updates.

@github-actions github-actions bot added bump-formula-pr PR was created using `brew bump-formula-pr` alias labels May 16, 2025
@daeho-ro daeho-ro force-pushed the bump-influxdb-3.0.3 branch from 30ddcfe to 0fbd273 Compare May 17, 2025 05:53
@github-actions github-actions bot added the automerge-skip `brew pr-automerge` will skip this pull request label May 17, 2025
@daeho-ro
Copy link
Member

daeho-ro commented May 17, 2025

Considerations.

  1. It seems that the version 2 is still supported 1 more year, so I think we have to split version 2 and 3.
    https://www.influxdata.com/legal/support-policy/
  2. I have tried to fix version 3 build but runtime error happens.
dyld[75832]: Library not loaded: @rpath/Python3.framework/Versions/3.9/Python3
  Referenced from: <FDD8939C-7326-30FC-A666-C93E57777B3C> /opt/homebrew/Cellar/influxdb/3.0.3/bin/influxdb3
  Reason: no LC_RPATH's found
  1. There isn't a way to serve some configuration file, user is limited to customize their server option.

@IngmarStein
Copy link
Contributor

@daeho-ro thanks for looking into this!

I think 2) can be solved by adding depends_on "python@3.13".
Per https://docs.influxdata.com/influxdb3/core/reference/config-options/#data-dir, INFLUXDB3_DATA_DIR should be INFLUXDB3_DB_DIR (the naming is inconsistent). INFLUXDB3_DATA_DIR results in Serve command failed: Cannot parse object store config: Specified File for the object store, required configuration missing for data-dir.

https://github.com/influxdata/influxdb/blob/main/CONTRIBUTING.md#build-profiles suggests adding --profile release to the cargo install command.

As for configuration: there are a number of formulae like this (for example prometheus or https://github.com/grafana/homebrew-grafana/blob/master/alloy.rb) which wrap the binary with a script that reads additional command line flags from a file and/or sources a file with environment variables.

@IngmarStein
Copy link
Contributor

InfluxDB 2 and 3 definitely should be split:

@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

@github-actions github-actions bot added the stale No recent activity label May 20, 2025
@github-actions github-actions bot closed this May 22, 2025
@github-actions github-actions bot deleted the bump-influxdb-3.0.3 branch May 22, 2025 00:15
@chenrui333
Copy link
Member

#231738

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alias automerge-skip `brew pr-automerge` will skip this pull request bump-formula-pr PR was created using `brew bump-formula-pr` stale No recent activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants