Skip to content

Conversation

harshadixit12
Copy link
Contributor

@harshadixit12 harshadixit12 commented Apr 22, 2025

We are retaining this check only for the certificate entity - since ID is a required property for this.
This is done to mitigate #1611 to an extent.

Reverts #1572
Depends on Kong/go-database-reconciler#254

@harshadixit12 harshadixit12 force-pushed the fix/check-certificate-existence-on-select-tag-mismatch branch from 4a97257 to 5755262 Compare April 22, 2025 09:30
@codecov-commenter
Copy link

codecov-commenter commented Apr 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.82%. Comparing base (4af5997) to head (ff4e048).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1608   +/-   ##
=======================================
  Coverage   28.82%   28.82%           
=======================================
  Files          67       67           
  Lines        6785     6785           
=======================================
  Hits         1956     1956           
  Misses       4687     4687           
  Partials      142      142           

☔ 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.

@harshadixit12 harshadixit12 force-pushed the fix/check-certificate-existence-on-select-tag-mismatch branch from 5755262 to 12bfb27 Compare April 23, 2025 07:34
@harshadixit12 harshadixit12 merged commit ef4c758 into main Apr 23, 2025
24 checks passed
@harshadixit12 harshadixit12 deleted the fix/check-certificate-existence-on-select-tag-mismatch branch April 23, 2025 10:02
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