Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate from make to just #6803

Merged
merged 2 commits into from
Nov 18, 2024
Merged

Migrate from make to just #6803

merged 2 commits into from
Nov 18, 2024

Conversation

relud
Copy link
Member

@relud relud commented Nov 13, 2024

also switch from SOCORRO_UID and SOCORRO_GID to USE_UID and USE_GID
also switch from my.env to .env
also switch to using docker-compose.override.yml for volume mounts, so they can be disabled in CI.

@relud relud requested a review from a team as a code owner November 13, 2024 23:08
@relud relud force-pushed the relud-use-just branch 13 times, most recently from 43fe7f3 to 661c814 Compare November 14, 2024 15:30
Copy link
Contributor

@willkg willkg left a comment

Choose a reason for hiding this comment

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

Looks good!

# if you clone the repository with a different directory name, then
# you end up with a different project name and then everything is hosed.
# Setting it here fixes that.
COMPOSE_PROJECT_NAME=socorro
Copy link
Contributor

Choose a reason for hiding this comment

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

For my notes, we added this because Alexis needed it because he ran socorro with a differently named directory name.

bug 1466974

Because this PR adds a name: socorro, we don't need this anymore.

# ---------------------------------------------

# Only use 2 threads for the processor
producer_consumer.number_of_threads=2
Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably the last remnants of configman.

justfile Outdated Show resolved Hide resolved
@relud relud enabled auto-merge November 18, 2024 19:15
@relud relud added this pull request to the merge queue Nov 18, 2024
Merged via the queue into main with commit b453d43 Nov 18, 2024
1 check passed
@relud relud deleted the relud-use-just branch November 18, 2024 19:32
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