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

[ELABORATION] Add RPDE tracing to test suite to help debug most common error in RPDE implementation #607

Open
nickevansuk opened this issue Oct 11, 2023 · 2 comments
Assignees

Comments

@nickevansuk
Copy link
Collaborator

Dumping feed pages to a file with a request ID for when a timeout occurs, and link to that file from the test suite results. Could dump RPDE pages out into files, with timestamp and current URL, and then reference the files in the Broker error. Also consolidate output into one place.

Adding tracing may also identify nuanced logic errors within the test suite itself (#545), though this is hard to confirm without doing this.

@lukehesluke
Copy link
Contributor

lukehesluke commented Feb 20, 2024

This is presently blocked by #627 as the solution to this issue will need to change test output somewhat

@lukehesluke lukehesluke changed the title Add RPDE tracing to test suite to help debug most common error in RPDE implementation [ELABORATION] Add RPDE tracing to test suite to help debug most common error in RPDE implementation Mar 21, 2024
@lukehesluke lukehesluke moved this from 🏗 In progress to ✅ Done in OpenActive Infrastructure Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants