Skip to content

Conversation

justinc1
Copy link
Collaborator

Running more than one CI test per HyperCore cluster is a bit problematic. PR refactor integ-test job to same pattern as we have for integ-seq-test - include the helper workflow z_ansible-test.yml. This allows running tests on all 4 clusters in parallel, and at most 1 test pre cluster.

Some tests require physical cluster. Those are run in newly added integ-physical job, on https://10.5.11.50.

We test examples with simple ansible-playbook ... example_name.yml. This does not work for some examples, so do not test those to avoid false failures.

@justinc1 justinc1 self-assigned this May 22, 2024
justinc1 added 3 commits May 22, 2024 17:43
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
E.g. those that cannot work on VSNS

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Skip examples that cannot be tested with simple
ansible-playbook ... file.yml

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
@justinc1
Copy link
Collaborator Author

justinc1 commented May 22, 2024

Copy link
Collaborator

@shoriminimoe shoriminimoe left a comment

Choose a reason for hiding this comment

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

This is so much better! Thank you! 👏

justinc1 added 2 commits May 23, 2024 09:08
Actual job names are like
'integ-seq-run (https://10.5.11.201) / ansible-test (utils_login)'
Code was searching for job with name like
'integ-seq-run (https://10.5.11.201), ansible-test (utils_login)'

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
@justinc1 justinc1 merged commit d48b2b1 into main May 23, 2024
@justinc1 justinc1 deleted the ci-refactor branch May 23, 2024 13:18
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.

2 participants