Skip to content

Adding GitHub actions ci.yml for PRs #167

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

Merged
merged 2 commits into from
Apr 23, 2025
Merged

Conversation

capolrik
Copy link
Collaborator

@capolrik capolrik commented Apr 1, 2025

Type of change

  • New feature

Description

Adding CI GH actions.

Checklist before requesting a review

  • I have performed a self-review of my code.

Testing

Each time a PR is open, GH actions will deploy a kind cluster, pulling the code in the PR and launch a simple test to see if the default behavior is working as attended.

@capolrik capolrik force-pushed the gh-ci branch 5 times, most recently from 7645528 to 20b2802 Compare April 1, 2025 09:56
@capolrik capolrik marked this pull request as ready for review April 1, 2025 12:08
@capolrik capolrik changed the title WIP adding GitHub actions ci.yml for PRs Adding GitHub actions ci.yml for PRs Apr 1, 2025
@capolrik
Copy link
Collaborator Author

capolrik commented Apr 1, 2025

Waiting the PR #159 to be merged before adding UDN tests

@capolrik capolrik force-pushed the gh-ci branch 8 times, most recently from 00316f6 to cd16227 Compare April 14, 2025 12:38
@josecastillolema
Copy link

This looks good to me! I will let the maintainers (@rsevilla87 @jtaleric) make the final call
The only thing I am not sure is if leaving all 3 tests together (default, l2-udn, l3-udn) or better to split this into two different tests, one for default and other for udn.

@rsevilla87 rsevilla87 self-requested a review April 22, 2025 10:34
Signed-off-by: Charles CAPORALI <[email protected]>
Copy link
Member

@rsevilla87 rsevilla87 left a comment

Choose a reason for hiding this comment

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

lgtm

@capolrik capolrik merged commit 2a358cf into cloud-bulldozer:main Apr 23, 2025
6 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.

4 participants