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

ci(kitchen+travis): adjust matrix to add 3000.1 & remove 2017.7 #51

Merged

Conversation

myii
Copy link
Member

@myii myii commented Apr 9, 2020

@myii myii requested a review from dafyddj April 9, 2020 17:36
@dafyddj
Copy link
Contributor

dafyddj commented Apr 10, 2020

My local testing has exposed some issues on a few platforms, but I don't believe they have been introduced specifically by this PR, so on that basis I approve, and intend to fix the issues below separately.

>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 3 actions failed.
>>>>>>     Converge failed on instance <prod-server-ubuntu-1604-2019-2-py3>.  Please see .kitchen/logs/prod-server-ubuntu-1604-2019-2-py3.log for more details
>>>>>>     Converge failed on instance <prod-server-ubuntu-1604-2018-3-py2>.  Please see .kitchen/logs/prod-server-ubuntu-1604-2018-3-py2.log for more details
>>>>>>     Verify failed on instance <prod-server-centos-6-2018-3-py2>.  Please see .kitchen/logs/prod-server-centos-6-2018-3-py2.log for more details
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

@myii myii merged commit 6da8319 into saltstack-formulas:master Apr 10, 2020
@myii
Copy link
Member Author

myii commented Apr 10, 2020

@dafyddj Thanks for the PR, even if you "used" my account to send it through!

@myii myii deleted the ci/add-3000.1-and-remove-2017.7 branch April 10, 2020 15:58
@saltstack-formulas-travis

🎉 This PR is included in version 1.2.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants