Skip to content

fix(4380): idempotent openshift results postgres append - #4381

Open
jkhelil wants to merge 1 commit into
tektoncd:mainfrom
jkhelil:fix/4380-results-postgres-append
Open

fix(4380): idempotent openshift results postgres append#4381
jkhelil wants to merge 1 commit into
tektoncd:mainfrom
jkhelil:fix/4380-results-postgres-append

Conversation

@jkhelil

@jkhelil jkhelil commented Nov 3, 2026

Copy link
Copy Markdown
Member

Fixes #4380

@jkhelil
fix(4380): idempotent openshift results postgres append
2c5b3b1 
Signed-off-by: Jawed khelil <jkhelil@redhat.com>
Assisted-by: Composer (via Cursor)
@tekton-robot tekton-robot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Nov 3, 2026
@tekton-robot

tekton-robot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from jkhelil after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Nov 3, 2026
@codecov

codecov Bot commented Nov 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 30.82%. Comparing base (4bf6f29) to head (2c5b3b1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #4381   +/-   ##
=======================================
  Coverage   30.82%   30.82%           
=======================================
  Files         478      478           
  Lines       23418    23418           
=======================================
  Hits         7218     7218           
  Misses      16200    16200
Flag Coverage Δ
unit-tests 30.82% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@jkhelil

jkhelil commented Nov 3, 2026

Copy link
Copy Markdown
Member Author

/release-note-none

@tekton-robot tekton-robot added release-note-none Denotes a PR that doesnt merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Nov 3, 2026
@pratap0007

pratap0007 commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

/retest

@pratap0007

pratap0007 commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anithapriyanatarajan anithapriyanatarajan Awaiting requested review from anithapriyanatarajan
@pratap0007 pratap0007 Awaiting requested review from pratap0007

Assignees

@pratap0007 pratap0007

Labels

lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesnt merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

OpenShift Results fails to reconcile: postgres manifest duplicated without YAML separators

3 participants

@jkhelil @tekton-robot @pratap0007