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

Transition certification suite to ginkgoless. #1622

Conversation

greyerof
Copy link
Contributor

@greyerof greyerof commented Nov 15, 2023

Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

PR #1620 needs to be merged first, as this code uses the helper function
"testhelper.GetNoContainersUnderTestSkipFn()" created by that PR.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.

@greyerof greyerof added non backward compatible PR makes any NBC change in some TC, including removal or test ID changes. do not merge ginkgoless labels Nov 15, 2023
Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.
@greyerof greyerof force-pushed the ginkgo_removal_certification_ts branch from 7106d02 to 0692c81 Compare November 16, 2023 08:54
generated_policy.json Outdated Show resolved Hide resolved
@greyerof greyerof merged commit 5526612 into redhat-best-practices-for-k8s:ginkgo_removal Nov 17, 2023
7 checks passed
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 17, 2023
…for-k8s#1622)

Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 17, 2023
…for-k8s#1622)

Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.
greyerof added a commit to greyerof/certsuite that referenced this pull request Nov 21, 2023
The content of PR redhat-best-practices-for-k8s#1622 was accidentally removed from ginkgo_removal
branch by commit 1c11b94.
@greyerof greyerof mentioned this pull request Nov 21, 2023
greyerof added a commit that referenced this pull request Nov 21, 2023
The content of PR #1622 was accidentally removed from ginkgo_removal
branch by commit 1c11b94.
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 21, 2023
…for-k8s#1622)

Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 21, 2023
…for-k8s#1648)

The content of PR redhat-best-practices-for-k8s#1622 was accidentally removed from ginkgo_removal
branch by commit 1c11b94.
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 21, 2023
…for-k8s#1622)

Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 21, 2023
…for-k8s#1648)

The content of PR redhat-best-practices-for-k8s#1622 was accidentally removed from ginkgo_removal
branch by commit 1c11b94.
sebrandon1 pushed a commit to sebrandon1/certsuite that referenced this pull request Nov 21, 2023
…for-k8s#1622)

Non-backward compatibility: the tc/check
"affiliated-certification-helm-version" will be skipped if no helm
chart releases were discovered. Also, in case we discovered some chart
releases and helm is v3, every chart release will be added as a compliant
report object.

Also, added the expansion of the "args..." in the ClaimFilePrintf call
to Logf.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ginkgoless non backward compatible PR makes any NBC change in some TC, including removal or test ID changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants