Skip to content

Disable synthetic test for deployment update - #1187

Merged
borisurbanik merged 1 commit into
3scale:masterfrom
borisurbanik:bu-disable-log-counter
Aug 11, 2026
Merged

Disable synthetic test for deployment update#1187
borisurbanik merged 1 commit into
3scale:masterfrom
borisurbanik:bu-disable-log-counter

Conversation

@borisurbanik

Copy link
Copy Markdown
Contributor

Reverting part of the integration test due to the PR #1181 is failing integration test consistently now.

The normal reconcile does not generate updates,
this is checked.

There was additional code to trigger a synthetic
reconcile of one deployment to make sure the
logging testing code is working.

This has been disabled because it is currently
flaky and preventing merges - it will be fixed
in follow on PR.

@borisurbanik
borisurbanik requested a review from a team as a code ownerAugust 11, 2026 05:20
@briangallagher

briangallagher commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

StatusScan Engine Critical High Medium LowTotal (0)
Open Source Security0000 0 issues
Licenses0000 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

The normal reconcile does not generate updates,
this is checked.
There was additional code to trigger a synthetic
reconcile of one deployment to make sure the
logging testing code is working.
This has been disabled because it is currently
flaky and preventing merges - it will be fixed
in follow on PR.
@borisurbanik
borisurbanikforce-pushed the bu-disable-log-counter branch from ca87d1d to 986a3baCompareAugust 11, 2026 05:22
@tkan145

Copy link
Copy Markdown
Contributor

/ltgm
/approve

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.55%. Comparing base (c59a4c8) to head (986a3ba).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #1187 +/- ##
==========================================
+ Coverage 44.03% 44.55% +0.51% 
==========================================
Files 204 205 +1 Lines 20960 21171 +211 ==========================================
+ Hits 9230 9433 +203 - Misses 10933 10938 +5 - Partials 797 800 +3 
FlagCoverage Δ
unit44.55% <ø> (+0.51%)⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

ComponentsCoverage Δ
apis/apps/v1alpha1 (u)63.56% <ø> (ø)
apis/capabilities/v1alpha1 (u)3.50% <ø> (ø)
apis/capabilities/v1beta1 (u)20.21% <ø> (ø)
controllers (i)12.08% <50.00%> (ø)
pkg (u)64.23% <92.43%> (+0.53%)⬆️
see 1 file with indirect coverage changes
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@borisurbanik
borisurbanik merged commit 529684b into 3scale:masterAug 11, 2026
22 checks passed
@borisurbanik
borisurbanik deleted the bu-disable-log-counter branch August 11, 2026 08:03
Sign up for freeto 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

@borisurbanik@briangallagher@tkan145@codecov-commenter