Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

[Testing] Tests might saturate MyBinder requests #751

Closed
korgan00 opened this issue Feb 21, 2022 · 9 comments
Closed

[Testing] Tests might saturate MyBinder requests #751

korgan00 opened this issue Feb 21, 2022 · 9 comments
Labels
ARCHIVED bug Bugs. Not new features. Not improvements. Bugs. testing Issues related to testing.

Comments

@korgan00
Copy link
Contributor

korgan00 commented Feb 21, 2022

I am not sure if this is a real problem, but I suspect it is. We have to take a look at it.

MyBinder is the service we are using to execute the python CodeCells remotely. It has a limited number of requests per hour that we can do.
When a page loads, it makes a request to connect with MyBinder and be ready to execute any code.
During the tests, there will be many requests visiting every page. Cypress clears the cookies very often so that it won't take advantage of the cached connection.

A possible solution is to put a cookie or localStorage key to prevent requests during testing.

@korgan00 korgan00 added the testing Issues related to testing. label Feb 21, 2022
@eddybrando eddybrando added the bug Bugs. Not new features. Not improvements. Bugs. label Feb 22, 2022
@eddybrando eddybrando added the needs information Issues with need for clarification. label May 20, 2022
@eddybrando
Copy link
Contributor

@Tansito, @vabarbosa, do you consider this to be a bug?

@Tansito
Copy link
Member

Tansito commented May 20, 2022

Mmm... Maybe yes, but definitely this is something that we should increase the priority and take a look whether it's a bug or not I think.

@vabarbosa
Copy link
Contributor

this could be a high priority issue since it is can possibly impact users but i am havent heard of any issues, yet.
after PR #954 the number of mybinder requests should be down significantly since we are no longer loading every single page.

@eddybrando eddybrando added size: M and removed needs information Issues with need for clarification. labels Jun 29, 2022
@eddybrando eddybrando changed the title Tests might saturate MyBinder requests [Testing] Tests might saturate MyBinder requests Jun 29, 2022
@rgaiacs
Copy link

rgaiacs commented Jun 20, 2023

Hi,

https://mybinder.org/ is receiving a huge traffic related to https://github.com/Qiskit/platypus-binder. At the moment, https://github.com/Qiskit/platypus-binder is driving more than half of the traffic in https://mybinder.org/

Screenshot 2023-06-20 at 15-53-58 Pod Activity - Dashboards - Grafana

https://mybinder.org/ is running with reduced capacity since May 2023 (details at https://blog.jupyter.org/mybinder-org-reducing-capacity-c93ccfc6413f) and providing CPU time for Qiskit Textbook (

binderUrl: 'https://mybinder.org',
) means declining CPU time for other users.

Will be possible to use Pyodide to run the code exercises for the textbook?

@vabarbosa
Copy link
Contributor

Hi,

https://mybinder.org/ is receiving a huge traffic related to https://github.com/Qiskit/platypus-binder. At the moment, https://github.com/Qiskit/platypus-binder is driving more than half of the traffic in https://mybinder.org/

Screenshot 2023-06-20 at 15-53-58 Pod Activity - Dashboards - Grafana

https://mybinder.org/ is running with reduced capacity since May 2023 (details at https://blog.jupyter.org/mybinder-org-reducing-capacity-c93ccfc6413f) and providing CPU time for Qiskit Textbook (

binderUrl: 'https://mybinder.org',

) means declining CPU time for other users.

Will be possible to use Pyodide to run the code exercises for the textbook?

thank you @rgaiacs we are investigating

@vabarbosa
Copy link
Contributor

@rgaiacs we've updated our application to (hopefully) significantly minimize the load on mybinder.
this is just the first step. additional updates should be forthcoming as we continue to investigate solutions.
our sincerest apologies for any inconveniences or undo strain we may have been putting on mybinder and the mybinder team. thank you!

@rgaiacs
Copy link

rgaiacs commented Jun 21, 2023

@vabarbosa Thanks for the pull request. I will follow up tomorrow.

@rgaiacs
Copy link

rgaiacs commented Jun 22, 2023

@vabarbosa Thanks again for the pull request! The traffic on mybinder.org from https://github.com/Qiskit/platypus-binder dropped to 1/4 after #2132 was merged. ❤️

Screenshot 2023-06-22 at 08-26-53 Pod Activity - Dashboards - Grafana

@vabarbosa
Copy link
Contributor

This repository is no longer maintained. It has been superseded by IBM Quantum Learning.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ARCHIVED bug Bugs. Not new features. Not improvements. Bugs. testing Issues related to testing.
Projects
No open projects
Status: Bugs
Development

No branches or pull requests

5 participants