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

connectivity: add property flag for junit #1656

Merged
merged 1 commit into from
May 25, 2023

Conversation

brlbil
Copy link
Contributor

@brlbil brlbil commented May 24, 2023

These changes add a new key=value flag --junit-property
Provided key value pairs will be added to the generated junit file.

Properties will be required to distinguish junit test names in cilium connectivity test reporting.
The new flag is added to the kind test.

@brlbil brlbil temporarily deployed to ci May 24, 2023 12:56 — with GitHub Actions Inactive
@brlbil brlbil marked this pull request as ready for review May 24, 2023 13:52
@brlbil brlbil requested review from a team as code owners May 24, 2023 13:52
@tklauser tklauser changed the title connectivity: add property flag fot junit connectivity: add property flag for junit May 24, 2023
@tklauser
Copy link
Member

@brlbil this PR picked up a conflict. Could you please rebase?

@tklauser tklauser added the needs-rebase This PR needs to be rebased because it has merge conflicts. label May 25, 2023
These changes add a new key=value flag `--junit-property`
Provided key value pairs will be added to the generated junit file.
Properties will be required to distinguish junit test names in
`cilium` connectivity test reporting.

Signed-off-by: Birol Bilgin <birol@cilium.io>
@brlbil brlbil force-pushed the pr/brlbil/add-junit-property-flag branch from bf895e4 to fb704a7 Compare May 25, 2023 09:58
@brlbil brlbil temporarily deployed to ci May 25, 2023 09:58 — with GitHub Actions Inactive
@tklauser tklauser removed the needs-rebase This PR needs to be rebased because it has merge conflicts. label May 25, 2023
@tklauser tklauser merged commit b91e265 into cilium:main May 25, 2023
17 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants