-
Notifications
You must be signed in to change notification settings - Fork 12
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
Figure out future of /retest
app
#1377
Comments
cc @mmorel-35 cc @zirain |
EG(envoy gateway) use retest app, and EG use github action only. |
i spent some time looking at this at the end of last week essentially Envoys needs are different - it looks for failed im thinking the best way forward is to shift the envoy code to the ill update as i have some time |
Thanks for your updates, one it's ready, please ping me to give a try. |
We made a load of changes to the retest app recently to work with Envoy's new CI framework
This may have broken some users as the logic is now more tied to Envoy's checks and essentially the
github/checks
action which has related logicIt was my intention to fold the action into that one
I reckon we can support a generic
/retest
one way/other so i guess its a case of figuring out how/where to maintain the codeThe text was updated successfully, but these errors were encountered: