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

chore(e2e): Get logs from postgres container to debug deadlock errors #3792

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

moduli
Copy link
Collaborator

@moduli moduli commented Oct 3, 2023

This PR updates the GitHub Action workflow that runs the e2e tests to grab logs from the postgres container. Hopefully, this will shed some light on a flaky deadlock error we've been seeing.

Messages:   	Error from controller when performing delete on scope
        	            	
        	            	Error information:
        	            	  Kind:                Internal
        	            	  Message:             scope.(Service).deleteFromRepo: unable to delete scope:
        	            	  iam.(Repository).DeleteScope: failed for o_TG0TF3JWcq:
        	            	  iam.(Repository).delete: db.DoTx: iam.(Repository).delete: db.Delete: unknown,
        	            	  unknown: error #0: dbw.Delete: ERROR: deadlock detected (SQLSTATE 40P01)
        	            	  Status:              500
        	            	  context:             Error from controller when performing delete on scope

@moduli moduli added the pr/no-milestone Ignores the Milestone Check label Oct 3, 2023
@vercel
Copy link

vercel bot commented Oct 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
boundary ⬜️ Ignored (Inspect) Oct 3, 2023 1:43pm

# run to help diagnose a deadlock issue
if: contains(matrix.filter, 'e2e_docker') && steps.run.outcome == 'failure'
run: |
docker logs database
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where does the config for this container live? I just want to double check is logging configuration to see if we should adjust anything there.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok yea that config looks good.

@moduli moduli merged commit a4023b8 into main Oct 3, 2023
59 checks passed
@moduli moduli deleted the moduli-e2e-postgres-logs-ICU-11110 branch October 3, 2023 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-milestone Ignores the Milestone Check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants