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

CI flakes under "Ensure cleanup crons are working" #1171

Closed
chadwhitacre opened this issue Nov 24, 2021 · 11 comments
Closed

CI flakes under "Ensure cleanup crons are working" #1171

chadwhitacre opened this issue Nov 24, 2021 · 11 comments

Comments

@chadwhitacre
Copy link
Member

chadwhitacre commented Nov 24, 2021

Seeing integration tests fail quite frequently, rerun sometimes works. Noticing a few failure modes, here is one (cf. #1170):

https://github.com/getsentry/self-hosted/runs/4317568879?check_suite_focus=true

Ensure cleanup crons are working ...
  + grep -q -E -e '\-cleanup\s+running\s+' -e '\-cleanup[_-].+\s+Up\s+'
  + docker compose --ansi never --env-file /home/runner/work/self-hosted/self-hosted/.env ps
  ++ teardown ERR 121
  ++ '[' 0 -eq 1 ']'
  ++ DID_TEAR_DOWN=1
  ++ '[' ERR '!=' EXIT ']'
  ++ echo 'An error occurred, caught SIGERR on line 121'
  An error occurred, caught SIGERR on line 121
  ++ echo 'Tearing down ...'
  Tearing down ...
  ++ rm /tmp/tmp.9Qws9UthXM
  ++ echo Done.
  Done.
  + teardown EXIT 1
  + '[' 1 -eq 1 ']'
  + return 0
  Error: Process completed with exit code 255.
  ##[debug]Finishing: Integration Test

Not consistent between 1.28 and 2.0.1.

Line:

$dc ps | grep -q -E -e '\-cleanup\s+running\s+' -e '\-cleanup[_-].+\s+Up\s+'

@chadwhitacre
Copy link
Member Author

@github-actions
Copy link

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@aminvakil
Copy link
Collaborator

I was trying to understand what has happened in times prior to regression starting to happen yesterday and could it be because of #1119?

I couldn't understand why this should fail sometimes, maybe others have some idea?

@chadwhitacre chadwhitacre changed the title De-flake the integration tests CI flakes under "Ensure cleanup crons are working" Feb 3, 2022
@chadwhitacre
Copy link
Member Author

Another on #1289:

https://github.com/getsentry/self-hosted/runs/5058157966?check_suite_focus=true#step:4:3617

My hunch is that cleanup runs but the test hits either before or after.

@chadwhitacre
Copy link
Member Author

@chadwhitacre
Copy link
Member Author

@aminvakil
Copy link
Collaborator

aminvakil commented Feb 12, 2022

@chadwhitacre
Copy link
Member Author

@aminvakil
Copy link
Collaborator

@chadwhitacre
Copy link
Member Author

chadwhitacre commented Mar 15, 2022

@aminvakil
Copy link
Collaborator

@github-actions github-actions bot locked and limited conversation to collaborators Apr 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Archived in project
Development

No branches or pull requests

3 participants