Skip to content

Bugfix DB Cache E2E Tests not actually running with --db-push-drafts-to-git=true flag - #1143

Draft
Catalin-Stratulat-Ericsson wants to merge 3 commits into
kptdev:mainfrom
Nordix:bugfix-e2e-test-runs-with-pushDraftsToGit
Draft

Bugfix DB Cache E2E Tests not actually running with --db-push-drafts-to-git=true flag#1143
Catalin-Stratulat-Ericsson wants to merge 3 commits into
kptdev:mainfrom
Nordix:bugfix-e2e-test-runs-with-pushDraftsToGit

Conversation

@Catalin-Stratulat-Ericsson

@Catalin-Stratulat-Ericsson Catalin-Stratulat-Ericsson commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Bugfix DB Cache E2E Tests not actually running with --db-push-drafts-to-git=true flag


NOTE PLEASE BE AWARE THE 2 PRS BELOW MUST BE MERGED IN BEFORE THIS ONE. E2E TESTS WILL FAIL UNTIL THOSE FIXES ARE IN


Description

  • What changed: make target variable exporting + enabled skipped crd test
  • Why it’s needed: right now tests are using make targets which are intended to have --db-push-drafts-to-git=true flag but is in fact false
  • How it works: exporting make target variable so that its actually used by E2E Tests + enabling previously skipped crd tests

Related Issue(s)


Type of Change

  • Bug fix
  • New feature
  • Enhancement
  • Refactor
  • Documentation
  • Tests
  • Other: ________

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes
  • Tests added/updated
  • Documentation added/updated
  • All tests and gating checks pass

Testing Instructions (Optional)


Additional Notes (Optional)

  • Known issues:
  • Further improvements:
  • Review notes:

AI Disclosure

  • I have used AI in the creation of this PR.

If so, please describe how:

  • Kiro just for a bit of understanding on the test cases

Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
@netlify

netlify Bot commented Jul 29, 2026

Copy link
Copy Markdown

Deploy Preview for kpt-porch ready!

Name Link
🔨 Latest commit b6fb227
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a6b0d232008310008e3ec06
😎 Deploy Preview https://deploy-preview-1143--kpt-porch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@efiacor
efiacor marked this pull request as ready for review July 30, 2026 08:35
@efiacor
efiacor requested review from a team July 30, 2026 08:35
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 30, 2026
@sonarqubecloud

Copy link
Copy Markdown

@rendre-greyling

Copy link
Copy Markdown
Contributor

Just to also add a note here: DB push to git feature has a lot of problems and obscure edge cases which fail. Currently working on a full refactor of it which will solve all of these. This should only be merged once that is in to avoid unnecessarily chasing more known bugs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold #ededed size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DB Cache E2E Tests not actually running with --db-push-drafts-to-git=true flag

2 participants