PATH missing in keystone token_flush cronjob

Bug #1410437 reported by Bjoern
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-ansible
Invalid
Low
Matt Thompson
Icehouse
Fix Released
High
Kevin Carter
Juno
Fix Released
Low
Matt Thompson
Trunk
Invalid
Low
Matt Thompson

Bug Description

keystone manage was installed in a non system path and therefore can not be found when running it without a prefix.
Easy fix is to add the PATH variable in the cronjob

PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin:/sbin

tags: added: low-hanging-fruit
Changed in openstack-ansible:
importance: Undecided → Low
Matt Thompson (mattt416)
tags: added: juno-backport-potential
removed: low-hanging-fruit
Changed in openstack-ansible:
status: New → In Progress
assignee: nobody → Matt Thompson (mattt416)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (master)

Reviewed: https://review.openstack.org/147877
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=4f77d789e877e887f2c3f2d86cf9a5527bb421fd
Submitter: Jenkins
Branch: master

commit 4f77d789e877e887f2c3f2d86cf9a5527bb421fd
Author: Matt Thompson <email address hidden>
Date: Fri Jan 16 15:14:51 2015 +0000

    Include paths in cron jobs

    As reported, keystone-manage is failing to run via cron due to default
    PATH not including /usr/local/bin. This commit includes path while
    also updating the cron jobs in roles/glance_cache_crons/tasks/main.yml
    which are also affected.

    On a side note, we may want to create these crons under /etc/cron.d
    so we can have system cron jobs all under one place, but that can be
    done in a later commit if others agree.

    Change-Id: I9237c2a191c5145e946288e2f1466493844f1ad5
    Closes-Bug: #1410437

Changed in openstack-ansible:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (juno)

Fix proposed to branch: juno
Review: https://review.openstack.org/152450

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (juno)

Reviewed: https://review.openstack.org/152450
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=35e352ead77ee6d6f8d66ae8496f540fb1a21e49
Submitter: Jenkins
Branch: juno

commit 35e352ead77ee6d6f8d66ae8496f540fb1a21e49
Author: Matt Thompson <email address hidden>
Date: Fri Jan 16 15:14:51 2015 +0000

    Include paths in cron jobs

    As reported, keystone-manage is failing to run via cron due to default
    PATH not including /usr/local/bin. This commit includes path while
    also updating the cron jobs in roles/glance_cache_crons/tasks/main.yml
    which are also affected.

    On a side note, we may want to create these crons under /etc/cron.d
    so we can have system cron jobs all under one place, but that can be
    done in a later commit if others agree.

    Change-Id: I9237c2a191c5145e946288e2f1466493844f1ad5
    Closes-Bug: #1410437
    (cherry picked from commit 4f77d789e877e887f2c3f2d86cf9a5527bb421fd)

tags: added: icehouse-backport-potential
removed: juno-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (icehouse)

Fix proposed to branch: icehouse
Review: https://review.openstack.org/158559

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (icehouse)

Reviewed: https://review.openstack.org/158559
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=952b3d569286cd4a904e7e0cdea2596b1c698f4e
Submitter: Jenkins
Branch: icehouse

commit 952b3d569286cd4a904e7e0cdea2596b1c698f4e
Author: Matt Thompson <email address hidden>
Date: Fri Jan 16 15:14:51 2015 +0000

    Include paths in cron jobs

    As reported, keystone-manage is failing to run via cron due to default
    PATH not including /usr/local/bin. This commit includes path while
    also updating the cron jobs in roles/glance_cache_crons/tasks/main.yml
    which are also affected.

    On a side note, we may want to create these crons under /etc/cron.d
    so we can have system cron jobs all under one place, but that can be
    done in a later commit if others agree.

    Conflicts:
            rpc_deployment/roles/keystone_setup/tasks/keystone_token_cron.yml

    Change-Id: I9237c2a191c5145e946288e2f1466493844f1ad5
    Closes-Bug: #1410437
    (cherry picked from commit 4f77d789e877e887f2c3f2d86cf9a5527bb421fd)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.