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

Django check in CI is flakey because of plausible check #920

Closed
obulat opened this issue Mar 15, 2023 · 2 comments
Closed

Django check in CI is flakey because of plausible check #920

obulat opened this issue Mar 15, 2023 · 2 comments
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: mgmt Related to repo management and automations

Comments

@obulat
Copy link
Contributor

obulat commented Mar 15, 2023

Description

Check this CI run: https://github.com/WordPress/openverse/actions/runs/4428714611/jobs/7768294796

Run Django check fails even though there are no changes to Django. (Actually, Django check shouldn't even run there?)

ERROR: for plausible  Cannot start service plausible: driver failed programming external connectivity on endpoint openverse_plausible_1 (1449e7e74b18855f73947ed4cd73f9e1c49ea8d90d610a119ef7683b389ab552): Error starting userland proxy: listen tcp4 0.0.0.0:50288: bind: address already in use

Priority set to medium because it usually passes when you re-run.

@obulat obulat added 🟨 priority: medium Not blocking but should be addressed soon 🛠 goal: fix Bug fix 🤖 aspect: dx Concerns developers' experience with the codebase 🧱 stack: mgmt Related to repo management and automations labels Mar 15, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Openverse Backlog Mar 15, 2023
@dhruvkb
Copy link
Member

dhruvkb commented Mar 15, 2023

This is a port binding issue we frequently see for API-level services running in the 502xx range. It has an existing issue #200 so it can possibly be closed.

This particular occurrence with Plausible will be reduced by #914 as only API-specific services will be created in the future.

Also the Django check will no longer run when #895 is merged.

@obulat
Copy link
Contributor Author

obulat commented Mar 15, 2023

Thank you for the explanation, @dhruvkb, I'll close this issue then.

@obulat obulat closed this as not planned Won't fix, can't repro, duplicate, stale Mar 15, 2023
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Openverse Backlog Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: mgmt Related to repo management and automations
Projects
Archived in project
Development

No branches or pull requests

2 participants