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

Reset git http.sslverify setting after setup is done #157

Merged
merged 1 commit into from
May 10, 2019

Conversation

dirgim
Copy link
Contributor

@dirgim dirgim commented May 9, 2019

Remember the git http.sslverify config setting and revert it to the original value after setup is done.

@dirgim dirgim requested review from arilivigni and skatlapa May 9, 2019 12:33
@dirgim
Copy link
Contributor Author

dirgim commented May 9, 2019

[test]

Copy link
Member

@arilivigni arilivigni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm and works as expected. @skatlapa unless you have proof otherwise this should be merged

playbooks/setup.yml Show resolved Hide resolved
@dirgim
Copy link
Contributor Author

dirgim commented May 9, 2019

There's an issue with the CI unrelated to the change, the cluster becomes unresponsive before all tests get a chance to run, I'll play around with it some more to see what's the problem.

@dirgim
Copy link
Contributor Author

dirgim commented May 10, 2019

[test]

@dirgim dirgim merged commit 064aa34 into CentOS-PaaS-SIG:master May 10, 2019
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.

3 participants