ansible-collections / kubernetes.core

The collection includes a variety of Ansible content to help automate the management of applications in Kubernetes and OpenShift clusters, as well as the provisioning and maintenance of clusters themselves.
Other
204 stars 130 forks source link

[WIP] move a most of CI from ansible-network to ansible-content-actions for… #752

Closed yurnov closed 2 weeks ago

yurnov commented 2 weeks ago

move a most of CI from ansible-network to ansible-content-actions for #751

SUMMARY

This PR aims to unify the CI process for kubernetes.core with another ansible-collections.

ISSUE TYPE

CI

COMPONENT NAME

.github/workflows/changelog.yaml .github/workflows/integration-tests.yaml .github/workflows/linters.yaml .github/workflows/sanity-tests.yaml .github/workflows/unit-tests.yaml

ADDITIONAL INFORMATION

Refer:

softwarefactory-project-zuul[bot] commented 2 weeks ago

Build succeeded. https://ansible.softwarefactory-project.io/zuul/buildset/bbe628de3bdd492289a4c7e9e69964b7

:heavy_check_mark: ansible-galaxy-importer SUCCESS in 4m 18s :heavy_check_mark: build-ansible-collection SUCCESS in 6m 53s

gravesm commented 2 weeks ago

A few comments before you spend too much time on this. Integration tests will likely never work on ansible-content-actions because there's too much set up specific to this collection. While we did previously consider moving sanity, linter and unit tests to ansible-content-actions, our understanding is that it is not in fact supported. Until we can get clarity around the state and future of that action, we're unlikely to switch CI over.

yurnov commented 2 weeks ago

A few comments before you spend too much time on this. Integration tests will likely never work on ansible-content-actions because there's too much set up specific to this collection. While we did previously consider moving sanity, linter and unit tests to ansible-content-actions, our understanding is that it is not in fact supported. Until we can get clarity around the state and future of that action, we're unlikely to switch CI over.

Thanks, @gravesm for that comment, I will close this PR to not waste time