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

Clicking on override in the past on Web schedule disables editing buttons until refresh, maybe crashes backend #1221

Closed
thomasmitchell opened this issue Jan 25, 2023 · 4 comments
Assignees
Labels

Comments

@thomasmitchell
Copy link
Contributor

thomasmitchell commented Jan 25, 2023

Steps to reproduce:

  1. Make a schedule
  2. Make a recurring shift
  3. Make an override shift that begins and ends in the past.
  4. Refresh the page
  5. Attempt to click on the override shift to edit it.
  6. Notice that the Add Rotation and Add Override buttons have been disabled as if to bring up the modal, but because no modal ever surfaces, they cannot be reenabled without refreshing the page.

Additional notes:

By doing this repeatedly, it seems that I can crash the engine backend, though it doesn't log why this occurs. It does not necessarily appear that the entire process crashes, but it stops successfully responding to liveness checks in Kubernetes and the Grafana plugin stops working, so it gets restarted.

Notably, the request to POST https://<domain>/api/plugin-proxy/grafana-oncall-app/api/internal/v1/oncall_shifts/preview/?date=2023-01-23 that occurs after clicking on the override shift never gets a response, indicating that the thread responsible for responding to this either crashes or hangs. It's possible that if enough of these stack up, the process will stop responding to requests?

@Ukochka
Copy link
Contributor

Ukochka commented Mar 3, 2023

To be able to fix this properly and easier we need backend changes @vadimkerr could you please help?

@vadimkerr
Copy link
Member

vadimkerr commented Mar 3, 2023

I was able to reproduce this, definitely a backend bug.

@Ukochka Ukochka added the bug Something isn't working label Mar 6, 2023
@vadimkerr
Copy link
Member

vadimkerr commented Mar 7, 2023

#1474 should prohibit creating & updating past overrides.
However, it won't fix the issue with buttons being disabled after the API returns 400 errors, I think this should be handled on the frontend side. @Ukochka could you please take a look at this?

@vadimkerr vadimkerr reopened this Mar 7, 2023
@matiasb
Copy link
Contributor

matiasb commented May 11, 2023

There were UI updates related to validation/when update/create buttons are enabled that prevent this now, closing.

@matiasb matiasb closed this as completed May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants