Skip to content

fix: address conflicts with ENV/script/process names#79

Draft
tigris wants to merge 1 commit into
mainfrom
fix-debezium-server-conflicts
Draft

fix: address conflicts with ENV/script/process names#79
tigris wants to merge 1 commit into
mainfrom
fix-debezium-server-conflicts

Conversation

@tigris
Copy link
Copy Markdown

@tigris tigris commented Oct 31, 2025

devbox plugin docs suggest to use ENV vars whereever possible for config, as this will allow users to override things where needed

I think we should reduce use of init_hook where possible since it runs every time you enter the directory (assuming direnv allow is on)

SELF NOTES for picking back up:

  • use path:/Users/path/to/plugin to include your own copy
  • I'm not entirely sure as yet how to make devbox pull in your changes after you make updates to the plugin - but renaming the directory and updating devbox.json certainly works

Issues:

  • this plugin creates a user, an outbox table and heartbeats in your database by default, but many systems might already have DB migrations for this, considering those need to be created in prod and we usually manage an RDS schema via migrations
  • this is especially troublesome in the case of influx because the devbox run populate script uses .sql files that look to be a snapshot of the database post migration (as is the rails standard i believe) - this means on a fresh clone doing devbox services up the plugin would try and create tables and users before other things have had a chance to setup the database
    • perhaps we need a config option (env var?) that tells the plugin not to bother creating psql resources (we already did it, it's part of our database migration workflow)

devbox plugin docs suggest to use ENV vars whereever possible for
config, as this will allow users to override things where needed

I think we should reduce use of init_hook where possible since it runs
every time you enter the directory (assuming direnv allow is on)
@tigris tigris force-pushed the fix-debezium-server-conflicts branch from eb4f103 to d01eb5f Compare November 3, 2025 23:38
Copy link
Copy Markdown

@abrokenjester abrokenjester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants