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

Bundle testing #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Bundle testing #1

wants to merge 5 commits into from

Conversation

mmalina
Copy link
Owner

@mmalina mmalina commented May 9, 2023

No description provided.

@mmalina mmalina force-pushed the bundle-testing branch 5 times, most recently from 290588f to 92a7491 Compare May 10, 2023 13:49
@mmalina mmalina force-pushed the bundle-testing branch 10 times, most recently from 8c8f9ac to 4127808 Compare May 17, 2023 11:53
This adds a new PR workflow that installs a kind cluster, tekton,
tkn and then runs tests (defined as pipelines) for tasks.

Documentation is added to CONTRIBUTING.md.

Two test pipelines for apply-mapping are included.
One is a basic use case when there is no extra mapping file.
The other is more elaborate with several components and
a mapping file.

Signed-off-by: Martin Malina <[email protected]>
Sometimes the test script failed if the pipeline
was not available immediately after creating it.

Signed-off-by: Martin Malina <[email protected]>
@mmalina mmalina force-pushed the bundle-testing branch 3 times, most recently from 6b0effb to 3c1c994 Compare May 19, 2023 08:27
The mechanism for waiting for tekton pods didn't work properly.
Now we first wait for the pods to appear and then we wait
for them to be ready.

Signed-off-by: Martin Malina <[email protected]>
mmalina added 2 commits May 19, 2023 10:42
The default timeout is 4 hours. Let's kill
the step it's not done within 10 minutes.

This is really just a failsafe for the while
loop - if for whatever reason the tekton
pods never appear.

Signed-off-by: Martin Malina <[email protected]>
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