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

Fix templating of static service files #4178

Merged
merged 1 commit into from May 16, 2017

Conversation

mtnbikenc
Copy link
Member

Static files should use copy: to retrieve from the files\ directory. Maybe?

ansible/ansible#18489
#4131

@mtnbikenc mtnbikenc self-assigned this May 12, 2017
@mtnbikenc mtnbikenc requested review from ashcrow and sdodson May 12, 2017 18:51
Copy link
Member

@ashcrow ashcrow left a comment

Choose a reason for hiding this comment

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

My internal understanding of ansible isn't strong enough to note if that would make a difference but I don't think it would hurt.

@mtnbikenc
Copy link
Member Author

Appears there is a bug in Ansible. Looks in <role_name>/tasks/files instead of <role_name>/files.

Debug:

 27140 1494616290.06971: search_path:
	/home/rteague/dev/clusters/aws-c1/openshift-ansible/roles/openshift_master/tasks/files/atomic-openshift-master.service
	/home/rteague/dev/clusters/aws-c1/openshift-ansible/roles/openshift_master/tasks/atomic-openshift-master.service
	/home/rteague/dev/clusters/aws-c1/openshift-ansible/playbooks/byo/openshift-cluster/upgrades/v3_6/../../../../common/openshift-cluster/upgrades/v3_6/../files/atomic-openshift-master.service
	/home/rteague/dev/clusters/aws-c1/openshift-ansible/playbooks/byo/openshift-cluster/upgrades/v3_6/../../../../common/openshift-cluster/upgrades/v3_6/../atomic-openshift-master.service
> $ ansible --versionansible 2.3.0.0
  config file = /home/rteague/dev/clusters/aws-c1/ansible.cfg
  configured module search path = [u'/usr/lib/python2.7/site-packages/ara/plugins/modules']
  python version = 2.7.13 (default, Jan 12 2017, 17:59:37) [GCC 6.3.1 20161221 (Red Hat 6.3.1-1)]

@mtnbikenc
Copy link
Member Author

mtnbikenc commented May 12, 2017

Added a temporary static link for tasks/files -> ../files until I can figure out if and where this is fixed in Ansible. Confirmed the problem also exists in Ansible 2.2.2.0.

@sdodson
Copy link
Member

sdodson commented May 12, 2017

@mtnbikenc Unfortunately some of these aren't static. All of the services that depend on docker have jinja templates to evaluate.

@sdodson
Copy link
Member

sdodson commented May 12, 2017

The node services.

@mtnbikenc
Copy link
Member Author

@sdodson Yeah, thanks for pointing that out. I'll fix the node service files. I'm still running into this problem and there has been a lot of churn on search path resolution, even recently.

ansible/ansible@ec05cd6

@mtnbikenc
Copy link
Member Author

aos-ci-test

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.6_containerized for c37fbea (logs)

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.5_containerized for c37fbea (logs)

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.6_NOT_containerized for c37fbea (logs)

@ashcrow
Copy link
Member

ashcrow commented May 15, 2017

This looks unrelated to me. Looking for a flake.

2017-05-15 16:26:30,316 - taskrunner - ERROR - Caught exception while running 'GetNodes'
Traceback (most recent call last):
  File "/home/jenkins/shiningpanda/jobs/26eaa329/virtualenvs/d41d8cd9/lib/python2.7/site-packages/taskrunner/main.py", line 144, in _run_tasks
    task.run(context)
  File "/home/jenkins/workspace/openshift-ansible-test-matrix/CONTAINERIZED/_NOT_containerized/OSE_VER/3.6/PYTHON/System-CPython-2.7/TOPOLOGY/openshift-cluster/TargetBranch/master/nodes/openshift-ansible-slave/ci-factory/tasks/get_nodes_task.py", line 280, in run
    desc = self.create_nodes(names)
  File "/home/jenkins/workspace/openshift-ansible-test-matrix/CONTAINERIZED/_NOT_containerized/OSE_VER/3.6/PYTHON/System-CPython-2.7/TOPOLOGY/openshift-cluster/TargetBranch/master/nodes/openshift-ansible-slave/ci-factory/tasks/get_nodes_task.py", line 337, in create_nodes
    vms = self._boot_nodes(names)
  File "/home/jenkins/workspace/openshift-ansible-test-matrix/CONTAINERIZED/_NOT_containerized/OSE_VER/3.6/PYTHON/System-CPython-2.7/TOPOLOGY/openshift-cluster/TargetBranch/master/nodes/openshift-ansible-slave/ci-factory/tasks/get_nodes_task.py", line 415, in _boot_nodes
    user_data=user_data)
  File "/home/jenkins/workspace/openshift-ansible-test-matrix/CONTAINERIZED/_NOT_containerized/OSE_VER/3.6/PYTHON/System-CPython-2.7/TOPOLOGY/openshift-cluster/TargetBranch/master/nodes/openshift-ansible-slave/ci-factory/tasks/nodefactory.py", line 104, in boot_node
    raise Exception('An unexpected exception has '
Exception: An unexpected exception has been encountered

@openshift-bot
Copy link

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

@mtnbikenc
Copy link
Member Author

aos-ci-test

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.5_NOT_containerized for c37fbea (logs)

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.6_containerized for c37fbea (logs)

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.5_containerized for c37fbea (logs)

@openshift-bot
Copy link

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

@sdodson
Copy link
Member

sdodson commented May 16, 2017

aos-ci-test

@openshift-bot
Copy link

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

@openshift-bot
Copy link

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

@openshift-bot
Copy link

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

@openshift-bot
Copy link

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

@mtnbikenc
Copy link
Member Author

[merge]

@openshift-bot
Copy link

Evaluated for openshift ansible merge up to c37fbea

@openshift-bot
Copy link

openshift-bot commented May 16, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/384/) (Base Commit: 15fd420)

@openshift-bot openshift-bot merged commit ef68639 into openshift:master May 16, 2017
@mtnbikenc mtnbikenc deleted the fix-templating branch May 16, 2017 18:10
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.

None yet

4 participants