-
Notifications
You must be signed in to change notification settings - Fork 0
Adding kindling issue request integration test #256
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds integration testing for the Kindling issue reporting system. It introduces a comprehensive test suite that validates different Kindling configurations (fronted and proxyless) for reporting issues with large attachments.
Key changes:
- New integration test file with tests for fronted and proxyless Kindling configurations
- Updated Kindling dependency to latest version
- Increased default HTTP timeout from 60 seconds to 10 minutes to accommodate large file uploads
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| kindling_test.go | Adds integration tests for Kindling issue reporting with fronted and proxyless setups |
| go.mod | Updates kindling dependency to version 20251215193336 |
| common/constants.go | Increases DefaultHTTPTimeout from 60 seconds to 10 minutes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ng-integration-test
… kindling currently amp is able to fetch the configuration but dnstt timeout
fix: writing random log size on test log
3ea9777 to
80fa000
Compare
No description provided.