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

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

Conversation

yurnov
Copy link
Contributor

@yurnov yurnov commented Jun 17, 2024

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:

@yurnov yurnov changed the title move a most of CI from ansible-network to ansible-content-actions for… [WIP] move a most of CI from ansible-network to ansible-content-actions for… Jun 17, 2024
@yurnov yurnov force-pushed the move-ci-to-ansible-content-actions branch from cc92abd to e51172e Compare June 17, 2024 13:43
Copy link

@gravesm
Copy link
Member

gravesm commented Jun 17, 2024

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
Copy link
Contributor Author

yurnov commented Jun 17, 2024

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

@yurnov yurnov closed this Jun 17, 2024
@yurnov yurnov deleted the move-ci-to-ansible-content-actions branch June 25, 2024 13:37
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