-
Notifications
You must be signed in to change notification settings - Fork 345
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
tests(non-k8s): Add HSP test suite for non-k8s mode #1857
base: main
Are you sure you want to change the base?
Conversation
dabdfe1
to
077f0fb
Compare
Can we include #1879 here? |
740b76c
to
1b8ab71
Compare
@navin772 why does the systemd test fail here? It cannot find the |
Some of the tests are failing, so the |
@navin772 what is failing? how can I help? |
@daemon1024 4 out of the new 8 non-k8s HSP tests are failing which were passing earlier, was there any changes to non-k8s recently? |
Signed-off-by: Navin Chandra <navinchandra772@gmail.com>
Signed-off-by: Navin Chandra <navinchandra772@gmail.com>
Signed-off-by: Navin Chandra <navinchandra772@gmail.com> Co-authored-by: Barun Acharya <barun1024@gmail.com>
Signed-off-by: Navin Chandra <navinchandra772@gmail.com>
524000e
to
3622756
Compare
Purpose of PR?:
Partially fixes #1625
Adds the HSP test suite for non-k8s mode
Does this PR introduce a breaking change?
No
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:
<type>(<scope>): <subject>