Skip to content

Do not run djstripe checks on builders #12159

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

Merged
merged 2 commits into from
May 8, 2025
Merged

Conversation

humitos
Copy link
Member

@humitos humitos commented May 6, 2025

We found this while performing the deploy.
Builders do not have access to the database, and djstripe runs some checks
that access the database to know if there are API keys defined there.

References:

@humitos humitos requested a review from a team as a code owner May 6, 2025 17:31
@humitos humitos requested a review from stsewd May 6, 2025 17:31
@humitos humitos marked this pull request as draft May 6, 2025 17:48
We found this while performing the deploy.
Builders do not have access to the database, and `djstripe` runs some checks
that access the database to know if there are API keys defined there.
@humitos humitos force-pushed the humitos/djstripe-builders branch from 67033fa to abd1ca7 Compare May 7, 2025 08:30
@humitos humitos marked this pull request as ready for review May 7, 2025 08:30
@humitos humitos changed the title Do not install djstripe on builders Do not run djstripe checks on builders May 7, 2025
@humitos humitos added the PR: hotfix Pull request applied as hotfix to release label May 7, 2025
@humitos
Copy link
Member Author

humitos commented May 7, 2025

I'm hotfixing this now so I can finish the deploy we started yesterday. I'd appreciate a review here and any suggestion you may have since this is a hack for now to allow us to move forward with the original plan.

@humitos humitos merged commit d856614 into main May 8, 2025
2 of 4 checks passed
@humitos humitos deleted the humitos/djstripe-builders branch May 8, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: hotfix Pull request applied as hotfix to release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants