-
Notifications
You must be signed in to change notification settings - Fork 8
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
Test network-engine against devel & stable, py2 & py3 #15
Test network-engine against devel & stable, py2 & py3 #15
Conversation
Although the roles are only "supported" on released versions of Ansible we should ensure there are no regressions in the devel branch. Some of the parsers require features that were first added in Ansible 2.5, so no point testing on older versions stable: py2 & py3 deve: py2
Build succeeded.
|
recheck |
Build succeeded.
|
@@ -2,7 +2,9 @@ | |||
check: | |||
jobs: | |||
- tox-linters | |||
- ansible-role-tests | |||
- ansible-role-tests-stable-py2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The job definitions for ansible/ansible should be in ansible/ansible.
The reason I put them on ansible-zuul-jobs was to a hack to work out a network integration zuul role.
What you need to do is :
- Install SF GH app on Ansible/ansible
- Create .zuul.yml with job definitions on ansible/ansible
- Put a depends-on on the ansible/ansible PR containing the .zuul.yml that depends on the roles on this ansible-zuul-jobs PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is for testing network-engine
, so I think this is the right place.
#13 will need moving to ansible/ansible
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah sorry, got confused between the two links. This is good then
recheck |
Build succeeded.
|
@@ -2,7 +2,9 @@ | |||
check: | |||
jobs: | |||
- tox-linters | |||
- ansible-role-tests | |||
- ansible-role-tests-stable-py2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah sorry, got confused between the two links. This is good then
recheck |
Build succeeded.
|
This role adds pre-commit integration tests for
network-engine
for both Python 2 and Python 3 using the latest stable released version of Ansible.In addition we use Ansible devel with Python 2 to ensure there aren't any upstream changes that might cause us issues.
The
network-engine
role only supports Ansible 2.5.0 and newer, so no point testing on older Ansible releases.Although the roles are only "supported" on released versions of Ansible
we should ensure there are no regressions in the devel branch of Ansible.
stable: py2 & py3
deve: py2