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
tools.kata-webhook: Fix lib path #9023
Conversation
When moving the webhook we skipped the common.bash as (close-enough) version is already in `/tests` but we forgot to update the source path, fixing it here. Fixes: kata-containers#8653 Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ldoktor !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @ldoktor!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks Lukas!
/test |
Hello folks, can I ask what is blocking this PR? I can see 3 approvals and the failing tests do not look related to me (actually I see them as waste of resources given this code does not touches anything related, do you think it'd make sense to investigate some time and skip them in such cases? I'd be willing to work on that) |
Sorry for the delay to merge this, @ldoktor . I had to run some flaky jobs many times until all required were passing. |
When moving the webhook we skipped the common.bash as (close-enough) version is already in
/tests
but we forgot to update the source path, fixing it here.