Skip to content

feat: remove migrated tests#1860

Merged
flacatus merged 1 commit into
konflux-ci:mainfrom
Troy876:KFLUXDP-961-remove-tests
Jun 10, 2026
Merged

feat: remove migrated tests#1860
flacatus merged 1 commit into
konflux-ci:mainfrom
Troy876:KFLUXDP-961-remove-tests

Conversation

@Troy876

@Troy876 Troy876 commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Description

Removes the now migrated tests of integration-service, konflux-demo, and release from test/

Issue ticket number and link

https://redhat.atlassian.net/browse/KFLUXDP-961

@openshift-ci openshift-ci Bot requested review from kasemAlem and mmalina April 30, 2026 13:13
@openshift-ci

openshift-ci Bot commented Apr 30, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ascerra for approval. For more information see the Code Review Process.

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

@Troy876 Troy876 force-pushed the KFLUXDP-961-remove-tests branch from 28edfb7 to 7e6cfbd Compare April 30, 2026 14:42
@Troy876 Troy876 force-pushed the KFLUXDP-961-remove-tests branch 2 times, most recently from 4716e74 to 8660ac0 Compare May 12, 2026 13:48
@Troy876

Troy876 commented May 15, 2026

Copy link
Copy Markdown
Contributor Author

/retest

3 similar comments
@Troy876

Troy876 commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@Troy876

Troy876 commented May 20, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@Troy876

Troy876 commented May 21, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@Troy876 Troy876 force-pushed the KFLUXDP-961-remove-tests branch from 8660ac0 to a4636c4 Compare May 21, 2026 10:37
@Troy876

Troy876 commented May 25, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@Troy876 Troy876 force-pushed the KFLUXDP-961-remove-tests branch from a4636c4 to 84ff46c Compare June 2, 2026 11:11
@flacatus

flacatus commented Jun 4, 2026

Copy link
Copy Markdown
Member

lets remove also the build ones

@flacatus

flacatus commented Jun 4, 2026

Copy link
Copy Markdown
Member

cc @tisutisu @dirgim

@Troy876 Troy876 force-pushed the KFLUXDP-961-remove-tests branch 2 times, most recently from 74d4fa1 to 6633bf7 Compare June 4, 2026 10:26
@flacatus

flacatus commented Jun 7, 2026

Copy link
Copy Markdown
Member

@Troy876 we have to add this repository the only purpose is to act as library for Konflux components tests. and say DO NOT add tests in this repo. In an elegant way :D

@Troy876

Troy876 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

@Troy876 we have to add this repository the only purpose is to act as library for Konflux components tests. and say DO NOT add tests in this repo. In an elegant way :D

So are we planning to archive this repo or do we just need to tell people not to add tests? We could do that via the PR template, so when someone creates a PR they're met with a message saying not to add tests here.

@flacatus

flacatus commented Jun 8, 2026

Copy link
Copy Markdown
Member

we are not archiving this repo. Will be maintained by build team. Just dont add tests here that are not related with this repo

@flacatus

flacatus commented Jun 9, 2026

Copy link
Copy Markdown
Member

@Troy876 can we add the info to readme to merge this pr?

@Troy876 Troy876 force-pushed the KFLUXDP-961-remove-tests branch from 6633bf7 to 9e150a7 Compare June 10, 2026 10:44
@Troy876

Troy876 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

@Troy876 can we add the info to readme to merge this pr?

I've added a note to the README.md

Important: This repository serves as a shared testing library and framework for Konflux components — not as a place to host tests. Please do not add tests here. Tests belong in the repository of the component they cover.

@flacatus

Copy link
Copy Markdown
Member

/retest

@flacatus flacatus merged commit 41a9b19 into konflux-ci:main Jun 10, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants