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.

Sign up for GitHub

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

Jump to bottom

Fix startup time for local development environment #10118

Open
RayBB opened this issue Dec 5, 2024 · 7 comments · May be fixed by #10179
Open

Fix startup time for local development environment #10118

RayBB opened this issue Dec 5, 2024 · 7 comments · May be fixed by #10179
Labels
Affects: Developers Lead: @jimchamp Issues overseen by Jim (Front-end Lead, BookNotes) [managed] Needs: Staff / Internal Reviewed a PR but don't have merge powers? Use this. Priority: 2 Important, as time permits. [managed] Theme: Development Issues related to the developer experience and the dev environment. [managed] Type: Bug Something isn't working. [managed]

Comments

@RayBB
Copy link
Collaborator

RayBB commented Dec 5, 2024

Problem

docker compose up now seems to take several minutes before the homepage is ready.
Per this discussion it is possibly related to security changes blocking calls to some of the OL/IA services and timeouts taking a long time.

It would be really great to fix and improve this to avoid frustration for developers.

Reproducing the bug

  1. Go to ...
  2. Do ...
  • Expected behavior:
  • Actual behavior:

Context

  • Browser (Chrome, Safari, Firefox, etc):
  • OS (Windows, Mac, etc):
  • Logged in (Y/N):
  • Environment (prod, dev, local): prod

Breakdown

Requirements Checklist

  • [ ]

Related files

Stakeholders

Instructions for Contributors

  • Please run these commands to ensure your repository is up to date before creating a new branch to work on this issue and each time after pushing code to Github, because the pre-commit bot may add commits to your PRs upstream.
@RayBB RayBB added Type: Bug Something isn't working. [managed] Needs: Breakdown This big issue needs a checklist or subissues to describe a breakdown of work. [managed] Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels Dec 5, 2024
@Spaarsh
Copy link
Contributor

Spaarsh commented Dec 5, 2024

I wish to triage this issue, but I am unable to access the slack discussion. Could you please add me to the channel?

@RayBB
Copy link
Collaborator Author

RayBB commented Dec 5, 2024

I don't have power to add people to slack but the discussion doesn't say much. You can look into the issue if you wish.

@Deathnote45
Copy link

hi, @RayBB can you explain me this issue briefly

@RayBB
Copy link
Collaborator Author

RayBB commented Dec 5, 2024

The issue is the app starts slowly, likely caused by timeouts on requests to to IA.

@Spaarsh
Copy link
Contributor

Spaarsh commented Dec 6, 2024

@RayBB Okay, no problem! I am interested in working on this issue.

@RayBB
Copy link
Collaborator Author

RayBB commented Dec 6, 2024

@Spaarsh you can look into it and share what you find. However, until staff triages the issue and decides how to proceed I won't assign you. It's likely a bit more complicated issue and I can't provide much guidance on it.

@raissg
Copy link

raissg commented Dec 11, 2024

Hello, we are a group of university students, and as part of our project, we are contributing to open-source repositories. We would really like to work on this issue, and since our project deadline is approaching, we are eager to get started as soon as possible. Could we please be assigned to this issue? We’ll make sure to follow all contributing guidelines and complete the work on time. Thank you for considering us!

@github-actions github-actions bot added the Needs: Response Issues which require feedback from lead label Dec 12, 2024
@mekarpeles mekarpeles added Lead: @jimchamp Issues overseen by Jim (Front-end Lead, BookNotes) [managed] Priority: 2 Important, as time permits. [managed] Theme: Development Issues related to the developer experience and the dev environment. [managed] Affects: Developers and removed Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels Dec 16, 2024
@schu96 schu96 linked a pull request Dec 22, 2024 that will close this issue
Open
@RayBB RayBB added Needs: Staff / Internal Reviewed a PR but don't have merge powers? Use this. and removed Needs: Breakdown This big issue needs a checklist or subissues to describe a breakdown of work. [managed] Needs: Response Issues which require feedback from lead labels Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Affects: Developers Lead: @jimchamp Issues overseen by Jim (Front-end Lead, BookNotes) [managed] Needs: Staff / Internal Reviewed a PR but don't have merge powers? Use this. Priority: 2 Important, as time permits. [managed] Theme: Development Issues related to the developer experience and the dev environment. [managed] Type: Bug Something isn't working. [managed]
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use mock data for homepage charts to reduce startup loading time schu96/openlibrary
5 participants
@RayBB @mekarpeles @Spaarsh @Deathnote45 @raissg