8000 Remove 'debug' option from CI matrix by artemgavrilov · Pull Request #328 · percona/postgres · GitHub
[go: up one dir, main page]

Skip to content

8000 Remove 'debug' option from CI matrix #328

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

Open
wants to merge 1 commit into
base: TDE_REL_17_STABLE
Choose a base branch
from

Conversation

artemgavrilov
Copy link
Collaborator
@artemgavrilov artemgavrilov commented May 13, 2025

There is no need to run both 'debug' and 'debugoptimized' variants in CI. 'debugoptimized' gives us everything that needed for CI run.

This PR requires repo settings update, as some debug jobs are marked as required

There is no need to run both 'debug' and 'debugoptimized' variants in
CI. 'debugoptimized' gives us everything that needed for CI run.
@artemgavrilov artemgavrilov marked this pull request as ready for review May 13, 2025 10:29
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.28%. Comparing base (e7e63e3) to head (d9e7b8f).
Report is 6 commits behind head on TDE_REL_17_STABLE.

❌ Your project status has failed because the head coverage (80.28%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@                Coverage Diff                 @@
##           TDE_REL_17_STABLE     #328   +/-   ##
==================================================
  Coverage              80.28%   80.28%           
==================================================
  Files                     22       22           
  Lines                   2577     2577           
  Branches                 395      395           
==================================================
  Hits                    2069     2069           
  Misses                   428      428           
  Partials                  80       80           
Components Coverage Δ
access 82.77% <ø> (ø)
catalog 87.47% <ø> (ø)
common 91.80% <ø> (ø)
encryption 75.70% <ø> (ø)
keyring 71.83% <ø> (ø)
src 66.74% <ø> (ø)
smgr 96.96% <ø> (ø)
transam ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator
@AndersAstrand AndersAstrand left a comment

Choose a reason for hiding this comment

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

I think this is the right thing to do (fewer tests to wait for), but ofc we need to wait for @dutow to actually make the change.

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.

4 participants
0