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
Rubocop and other updates/fixes for TK #62
Conversation
This allows for our self signed testing cert to be verified properly by ruby. New code was added to the openstack-identity cookbook which required this to work in TK.
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.
kitchen test identity-centos-73
fails for me giving an error on resource execute[copy self-signed ca-cert]
.
@knightsamar that should be fixed now. Thanks for catching that! |
@@ -2,6 +2,7 @@ | |||
"env_run_lists": {}, | |||
"run_list": [ | |||
"recipe[openstack_test::hosts]", | |||
"recipe[openstack_test::cacert]", |
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.
Where exactly is this cookbook openstack_test ?
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.
Test kitchen works well btw 👍
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.
@knightsamar it's in the cookbook, look at Berksfile to see where exactly
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.
Thanks. This is an interesting setup. Hopefully I can make time to read the code and understand how you are making these things work.
!bump patch * |
Jenkins has merged this PR into |
No description provided.