Skip to content

ci: added default values in case repo variables are not present#1938

Merged
Prashansa-K merged 4 commits intomainfrom
ci/default-timeout-val
Mar 20, 2026
Merged

ci: added default values in case repo variables are not present#1938
Prashansa-K merged 4 commits intomainfrom
ci/default-timeout-val

Conversation

@Prashansa-K
Copy link
Contributor

@Prashansa-K Prashansa-K commented Mar 16, 2026

Adding some default values in timeouts extracted from repo variables.
This would unblock open-source contributions via forks where we could at least test against
open source gateway wherever possible.

All kinds of changes may not work as we would require EE and Konnect testing but things
like lib updates, doc updates, etc. can be allowed via forks.

Update: Added a go timeout for tests as Konnect tests were failing after the default 10minute marker.
Updated that to 20m for now.

@codecov-commenter
Copy link

codecov-commenter commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.54%. Comparing base (959c522) to head (9bfde5a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1938      +/-   ##
==========================================
- Coverage   33.58%   33.54%   -0.05%     
==========================================
  Files          76       76              
  Lines        6711     6711              
==========================================
- Hits         2254     2251       -3     
- Misses       4278     4280       +2     
- Partials      179      180       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Prashansa-K Prashansa-K merged commit efa9eef into main Mar 20, 2026
40 checks passed
@Prashansa-K Prashansa-K deleted the ci/default-timeout-val branch March 20, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants