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

re-add logs in claim file #1725

Conversation

edcdavid
Copy link
Member

@edcdavid edcdavid commented Dec 13, 2023

Depends-On: redhat-best-practices-for-k8s/certsuite-claim#110
This PR proposed a new claim format to support reporting the following separately and re-add logs in claim file (and parser window):

  • the skip reason
  • checkDetails (failure reason)
  • the check logs

updated parser is at: redhat-best-practices-for-k8s/parser#28

TODO: versioning parser and linking

pkg/checksdb/check.go Outdated Show resolved Hide resolved
pkg/checksdb/check.go Outdated Show resolved Hide resolved
pkg/checksdb/check.go Outdated Show resolved Hide resolved
@edcdavid edcdavid force-pushed the re-add-log-in-claim branch from 2111aca to 86f3ba3 Compare December 14, 2023 14:39
@edcdavid edcdavid force-pushed the re-add-log-in-claim branch from 3c55ba6 to af7b248 Compare December 14, 2023 17:07
@edcdavid edcdavid force-pushed the re-add-log-in-claim branch from 82b6a8b to 1bdbf4c Compare December 14, 2023 18:27
@edcdavid edcdavid merged commit 7e79d17 into redhat-best-practices-for-k8s:ginkgo_removal Dec 14, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants