Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(HMS-4895): add unit tests to tekton #389

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

avisiedo
Copy link
Contributor

Add the unit tests execution for the tekton pipeline.

https://issues.redhat.com/browse/HMS-4895

@avisiedo avisiedo self-assigned this Oct 28, 2024
@avisiedo avisiedo force-pushed the hms-4895-add-unit-tests branch from c668596 to 29dd65d Compare October 28, 2024 13:19
@avisiedo avisiedo changed the title build(HMS-4895: add unit tests to tekton build(HMS-4895): add unit tests to tekton Oct 29, 2024
@avisiedo avisiedo force-pushed the hms-4895-add-unit-tests branch 7 times, most recently from fde7bf6 to ea14c83 Compare November 18, 2024 13:52
@avisiedo avisiedo force-pushed the hms-4895-add-unit-tests branch from ea14c83 to 446742e Compare December 4, 2024 10:34
Add the unit tests execution for the tekton pipeline.

https://issues.redhat.com/browse/HMS-4895

Signed-off-by: Alejandro Visiedo <[email protected]>
Signed-off-by: Alejandro Visiedo <[email protected]>
Signed-off-by: Alejandro Visiedo <[email protected]>
My thought is that field is not necessary and it is overriding
the expected behavior, that is run the embed script.

Signed-off-by: Alejandro Visiedo <[email protected]>
Signed-off-by: Alejandro Visiedo <[email protected]>
Signed-off-by: Alejandro Visiedo <[email protected]>
There was an error creating the PipelineRun: idmsvc-backend-on-pull-request-

creating pipelinerun idmsvc-backend-on-pull-request- in namespace insights-management-tenant has failed.

Tekton Controller has reported this error: admission webhook "validation.webhook.pipeline.tekton.dev" denied the request: validation failed: invalid value: couldn't add link between run-unit-tests and clone-repository-oci-ta: task run-unit-tests depends on clone-repository-oci-ta but clone-repository-oci-ta wasn't present in Pipeline: spec.pipelineSpec.tasks

Signed-off-by: Alejandro Visiedo <[email protected]>
@avisiedo avisiedo force-pushed the hms-4895-add-unit-tests branch from 446742e to 1dbb51d Compare December 10, 2024 07:20
@avisiedo
Copy link
Contributor Author

/retest

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.

1 participant