-
Notifications
You must be signed in to change notification settings - Fork 9
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
Adding CI tests #59
Adding CI tests #59
Conversation
559551a
to
d6b9918
Compare
@jtaleric @shashank-boyapally @paigerube14 PTAL 👋🏼 . All the payloads are working. We can add/update more scenarios in the follow up PRs if needed. |
Signed-off-by: Vishnu Challa <vchalla@vchalla-thinkpadp1gen2.rmtusnc.csb>
While cutting the release for this change, we need to make sure that appropriate ENVs are populated in prow that fit in with the updated configs. |
/lgtm |
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.
Looks good!
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!
Signed-off-by: Vishnu Challa <vchalla@vchalla-thinkpadp1gen2.rmtusnc.csb> Co-authored-by: Vishnu Challa <vchalla@vchalla-thinkpadp1gen2.rmtusnc.csb>
Type of change
Description
Adding CI tests for orion. They will be triggered against each PR and will help us catch issues early.
Related Tickets & Documents
Checklist before requesting a review
Testing
Tested in local. Attempting to test it from the PR as well.