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

have the release workflow run the smoketest. #232

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

jmazzitelli
Copy link
Contributor

If it fails, title the PR with a loud warning and indicate it should not be merged. Otherwise, create the PR as it normally does.

fixes kiali/kiali#6704

with a loud warning and indicate it should not be merged. Otherwise,
create the PR as it normally does.

fixes kiali/kiali#6704
@jmazzitelli jmazzitelli self-assigned this Nov 16, 2023
@jmazzitelli jmazzitelli marked this pull request as draft November 16, 2023 17:29
@jmazzitelli jmazzitelli marked this pull request as ready for review November 16, 2023 17:33
@jmazzitelli
Copy link
Contributor Author

this should work now

@jmazzitelli
Copy link
Contributor Author

I did two tests .. 1. smoke test fails and 2. smoke test succeeded. It behaves as expected both times. This is ready for merge.

@jmazzitelli
Copy link
Contributor Author

This is what the resulting PR will look like if the smoke test fails:
image

Copy link

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wunderbar

@jmazzitelli jmazzitelli merged commit 25346bc into kiali:master Nov 16, 2023
1 check passed
@jmazzitelli jmazzitelli deleted the 6704-fix-smoketest branch November 16, 2023 18:51
Matiasmct pushed a commit to giffgaff/kiali-charts-backup that referenced this pull request Nov 29, 2023
kiali#232)

with a loud warning and indicate it should not be merged. Otherwise,
create the PR as it normally does.

fixes kiali/kiali#6704
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

helm-charts smoke test GH action fails to start
2 participants