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

BZ 1442942: Fix paths for file includes #3936

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

mtnbikenc
Copy link
Member

@mtnbikenc mtnbikenc commented Apr 17, 2017

ansible-playbook --syntax-check revealed path issues which are fixed.

Fixes 1442942

@mtnbikenc mtnbikenc self-assigned this Apr 17, 2017
@mtnbikenc mtnbikenc requested a review from kwoodson April 17, 2017 13:45
@sdodson
Copy link
Member

sdodson commented Apr 17, 2017

Can we make this a CI job?

@mtnbikenc
Copy link
Member Author

@sdodson Already have something in the works for automating checks for all entry point playbooks. (which is how I found this one)

@mtnbikenc
Copy link
Member Author

aos-ci-test

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_NOT_containerized, aos-ci-jenkins/OS_3.6_NOT_containerized_e2e_tests" for 0204597 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for 0204597 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_containerized, aos-ci-jenkins/OS_3.6_containerized_e2e_tests" for 0204597 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for 0204597 (logs)

@mtnbikenc
Copy link
Member Author

[merge]

@openshift-bot
Copy link

[test]ing while waiting on the merge queue

@openshift-bot
Copy link

Evaluated for openshift ansible test up to 0204597

@mtnbikenc
Copy link
Member Author

Flake openshift/origin#13067 oc new-app [It] should succeed with a --name of 58 characters
Flake openshift/origin#13705 Dumping container logs

@kwoodson
Copy link
Contributor

flake openshift/origin#13516

@sdodson
Copy link
Member

sdodson commented Apr 18, 2017

flake openshift/origin#13386

@sdodson
Copy link
Member

sdodson commented Apr 18, 2017

[merge]

@mtnbikenc
Copy link
Member Author

@mtnbikenc mtnbikenc changed the title Fix paths for file includes BZ 1442942: Fix paths for file includes Apr 18, 2017
@openshift-bot
Copy link

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/48/) (Base Commit: 00095b2)

@mtnbikenc
Copy link
Member Author

Failed on openshift/origin#13806 Not a git repository
Failed on openshift/origin#13271 Driver devicemapper failed

@sdodson
Copy link
Member

sdodson commented Apr 19, 2017

[merge]

@mtnbikenc
Copy link
Member Author

Flake openshift/origin#13705 Dumping container logs
Flake openshift/origin#13271 Driver devicemapper failed
re[merge]

@openshift-bot
Copy link

Evaluated for openshift ansible merge up to 0204597

@openshift-bot
Copy link

continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/251/) (Base Commit: 14dbd7c)

@mtnbikenc
Copy link
Member Author

mtnbikenc commented Apr 19, 2017

known flake? chmod: cannot access ‘/etc/origin/’: No such file or directory

@mtnbikenc
Copy link
Member Author

Merging based on temporary merge queue rules.

@mtnbikenc mtnbikenc merged commit 9ace041 into openshift:master Apr 19, 2017
@mtnbikenc mtnbikenc deleted the fix-ca-paths branch April 19, 2017 19:20
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.

4 participants