gate-check logging improvements

Bug #1425482 reported by Jesse Pretorius
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Medium
Tom Cameron
Icehouse
Fix Released
Medium
Jesse Pretorius
Juno
Fix Released
Medium
Ian Cordasco
Trunk
Fix Released
Medium
Tom Cameron

Bug Description

Currently we only make use of the console output for all gate check debugging. The console log is very verbose and often doesn't have the real information we need in order to identify the problem properly.

I suggest that we do the following:
 - collect other logs which can add insight and store them as artifacts
 - split outputs into multiple logs so that the console outputs are closer to a summary... we should be able to use it almost like a dashboard to determine where else to look for details of what went wrong
 - split the host information into separate logs, and if possible collect similar information from inside the containers
 - run ansible in standard mode (not very verbose mode), but have it output debug logs to a file (not sure whether this is possible somehow)

Whatever is done needs to work for both openstack-infra and external CI's.

Some references:
 - https://github.com/openstack-infra/project-config/blob/610bd506c457ab858971061c1718eae7f13ff8d6/jenkins/jobs/keystoneclient-functional.yaml#L32-L35
 - https://github.com/openstack-infra/project-config/blob/610bd506c457ab858971061c1718eae7f13ff8d6/jenkins/jobs/macros.yaml#L522-L542

description: updated
description: updated
Ian Cordasco (icordasc)
Changed in openstack-ansible:
status: New → Triaged
assignee: nobody → Tom Cameron (tom-cameron)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to os-ansible-deployment (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/159931

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to os-ansible-deployment (master)

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

commit 4d675bf7f576d4c63513f4f28ac713ffe21fb8e2
Author: Ian Cordasco <email address hidden>
Date: Mon Mar 2 18:01:22 2015 -0600

    Symlink /openstack/log so gate logs can be published

    Ant needs a path that is local to the directory from which it operates.
    To be able to publish our logs, we need to symlink /openstack/log to
    /home/jenkins/workspace/os-ansible-deployment-dvsm-check-commit/logs.

    This also allows us to remove the log publisher we wrote and just use
    the devstack-logs publisher instead.

    Related-Bug: 1425482
    Depends-On: If94e6b97c159000c83fadc6e4dd72d56ee90d3b9
    Change-Id: Iee055beb853cbb76a073c8060d6920d317f5bd64

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to os-ansible-deployment (icehouse)

Related fix proposed to branch: icehouse
Review: https://review.openstack.org/160999

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to os-ansible-deployment (juno)

Related fix proposed to branch: juno
Review: https://review.openstack.org/161000

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

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

commit ca4d70c2ce3adcba39a8ed9388d4f13082b814a9
Author: Ian Cordasco <email address hidden>
Date: Mon Mar 2 18:01:22 2015 -0600

    Symlink /openstack/log so gate logs can be published

    Ant needs a path that is local to the directory from which it operates.
    To be able to publish our logs, we need to symlink /openstack/log to
    /home/jenkins/workspace/os-ansible-deployment-dvsm-check-commit/logs.

    This also allows us to remove the log publisher we wrote and just use
    the devstack-logs publisher instead.

    Conflicts:
     scripts/gate-check-commit.sh

    Related-Bug: 1425482
    Depends-On: If94e6b97c159000c83fadc6e4dd72d56ee90d3b9
    Change-Id: Iee055beb853cbb76a073c8060d6920d317f5bd64
    (cherry picked from commit 4d675bf7f576d4c63513f4f28ac713ffe21fb8e2)

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

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

commit 64f3fae41e9da3aef0f1a961babc8036c7892519
Author: Ian Cordasco <email address hidden>
Date: Mon Mar 2 18:01:22 2015 -0600

    Symlink /openstack/log so gate logs can be published

    Ant needs a path that is local to the directory from which it operates.
    To be able to publish our logs, we need to symlink /openstack/log to
    /home/jenkins/workspace/os-ansible-deployment-dvsm-check-commit/logs.

    This also allows us to remove the log publisher we wrote and just use
    the devstack-logs publisher instead.

    Conflicts:
     scripts/gate-check-commit.sh

    Related-Bug: 1425482
    Depends-On: If94e6b97c159000c83fadc6e4dd72d56ee90d3b9
    Change-Id: Iee055beb853cbb76a073c8060d6920d317f5bd64
    (cherry picked from commit 4d675bf7f576d4c63513f4f28ac713ffe21fb8e2)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to os-ansible-deployment (master)

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

commit 1f09957c23dd6e11b15648a9eec873b0fa925f6e
Author: Ian Cordasco <email address hidden>
Date: Fri Feb 27 11:11:54 2015 -0600

    Use bind-mounted directory for tempest logs

    In an effort to improve the artifacts we have from the gate, make
    tempest record its logs in the utility container's bind-mounted log
    directory. The gate publisher will then automatically copy the file.

    Related-bug: 1425482
    Change-Id: I5811c2d34772ef4711d9c483038edb5a5c1a6897

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to os-ansible-deployment (juno)

Related fix proposed to branch: juno
Review: https://review.openstack.org/161895

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to os-ansible-deployment (icehouse)

Related fix proposed to branch: icehouse
Review: https://review.openstack.org/161896

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

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

commit 465124d845a9aa317a05383d204df361ac3b9d4b
Author: Ian Cordasco <email address hidden>
Date: Fri Feb 27 11:11:54 2015 -0600

    Use bind-mounted directory for tempest logs

    In an effort to improve the artifacts we have from the gate, make
    tempest record its logs in the utility container's bind-mounted log
    directory. The gate publisher will then automatically copy the file.

    Conflicts:
     rpc_deployment/roles/tempest/templates/tempest.conf.j2

    Related-bug: 1425482
    Change-Id: I5811c2d34772ef4711d9c483038edb5a5c1a6897
    (cherry picked from commit 1f09957c23dd6e11b15648a9eec873b0fa925f6e)

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

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

commit 4f64cb385f0f5809ad4375e80071af32d253b0fe
Author: Ian Cordasco <email address hidden>
Date: Fri Feb 27 11:11:54 2015 -0600

    Use bind-mounted directory for tempest logs

    In an effort to improve the artifacts we have from the gate, make
    tempest record its logs in the utility container's bind-mounted log
    directory. The gate publisher will then automatically copy the file.

    Conflicts:
     rpc_deployment/roles/tempest/templates/tempest.conf.j2

    Related-bug: 1425482
    Change-Id: I5811c2d34772ef4711d9c483038edb5a5c1a6897
    (cherry picked from commit 1f09957c23dd6e11b15648a9eec873b0fa925f6e)

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

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

commit 5d7db80effd00b0d7f9cfd12892aa444abd5a92c
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 5 10:17:58 2015 +0000

    Make and symlink log directory in aio bootstrap

    This patch creates the /openstack/log directory within the AIO bootstrap
    script to prevent 'unstable' results in openstack-infra's jenkins.

    It also moves the symlink to that directory to much earlier in the
    gate-check script for the same purpose.

    Co-Authored-By: Ian Cordasco <email address hidden>
    Closes-Bug: #1428744
    Partial-Bug: #1425482
    Change-Id: Ib74b5fd216d43e74ca264c14818cfef6e2dc0c70

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/162014

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/162015

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

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

commit 36258244ec7c278c075568ae68bc720d574165be
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 5 10:17:58 2015 +0000

    Make and symlink log directory in aio bootstrap

    This patch creates the /openstack/log directory within the AIO bootstrap
    script to prevent 'unstable' results in openstack-infra's jenkins.

    It also moves the symlink to that directory to much earlier in the
    gate-check script for the same purpose.

    Conflicts:
            scripts/bootstrap-aio.sh
            scripts/gate-check-commit.sh

    Co-Authored-By: Ian Cordasco <email address hidden>
    Closes-Bug: #1428744
    Partial-Bug: #1425482
    Change-Id: Ib74b5fd216d43e74ca264c14818cfef6e2dc0c70
    (cherry picked from commit 5d7db80effd00b0d7f9cfd12892aa444abd5a92c)

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

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

commit d52f83abe18e1e6b20f39e77058188b9a4901d33
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 5 10:17:58 2015 +0000

    Make and symlink log directory in aio bootstrap

    This patch creates the /openstack/log directory within the AIO bootstrap
    script to prevent 'unstable' results in openstack-infra's jenkins.

    It also moves the symlink to that directory to much earlier in the
    gate-check script for the same purpose.

    Conflicts:
            scripts/bootstrap-aio.sh
            scripts/gate-check-commit.sh

    Co-Authored-By: Ian Cordasco <email address hidden>
    Closes-Bug: #1428744
    Partial-Bug: #1425482
    Change-Id: Ib74b5fd216d43e74ca264c14818cfef6e2dc0c70
    (cherry picked from commit 5d7db80effd00b0d7f9cfd12892aa444abd5a92c)

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

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

commit 6f38cd47546dd96080f6016230f1c3480f76b470
Author: Kevin Carter <email address hidden>
Date: Thu Mar 5 14:46:06 2015 -0600

    Create ansible log file and directory

    This commit adds the ansible log file to the aio bootstrap script
    and creates a vanilla logging directory that will allow us to collect
    additional information on what is happening within a given ansible run.

    Partial-Bug: #1425482
    Partial-Bug: #1428744
    Change-Id: I2786f73334b99bfa98d344c50337c86f75fae273

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

commit a9244537d62662ab3342d1c1bdb74e3b1f23ad0e
Author: David Wilde <email address hidden>
Date: Fri Mar 6 17:34:35 2015 -0600

    Trim ansible's output to console.log

    Since I2786f73334b99bfa98d344c50337c86f75fae273 ansible's output is
    logged properly to a file and published at the end of a gate check.
    This trims the output from ansible to the console to make the console
    log that is published far shorter and easier to read.

    Co-Authored-By: Ian Cordasco <email address hidden>
    Change-Id: I98fb57a56c1b7ddd6558e265cf6e65b159a05123
    Partial-bug: 1425482

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/162626

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/162627

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

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

commit 817c6066cfa5e1263f7de5ecf3ea96d45849a59c
Author: David Wilde <email address hidden>
Date: Fri Mar 6 17:34:35 2015 -0600

    Trim ansible's output to console.log

    Since I2786f73334b99bfa98d344c50337c86f75fae273 ansible's output is
    logged properly to a file and published at the end of a gate check.
    This trims the output from ansible to the console to make the console
    log that is published far shorter and easier to read.

    Co-Authored-By: Ian Cordasco <email address hidden>
    Change-Id: I98fb57a56c1b7ddd6558e265cf6e65b159a05123
    Partial-bug: 1425482
    (cherry picked from commit a9244537d62662ab3342d1c1bdb74e3b1f23ad0e)

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

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

commit 69ee7f64f452f8266cfa93c2f88155968adc5eaf
Author: David Wilde <email address hidden>
Date: Fri Mar 6 17:34:35 2015 -0600

    Trim ansible's output to console.log

    Since I2786f73334b99bfa98d344c50337c86f75fae273 ansible's output is
    logged properly to a file and published at the end of a gate check.
    This trims the output from ansible to the console to make the console
    log that is published far shorter and easier to read.

    Co-Authored-By: Ian Cordasco <email address hidden>
    Change-Id: I98fb57a56c1b7ddd6558e265cf6e65b159a05123
    Partial-bug: 1425482
    (cherry picked from commit a9244537d62662ab3342d1c1bdb74e3b1f23ad0e)

Revision history for this message
Hugh Saunders (hughsaunders) wrote :
Download full text (5.1 KiB)

I am now seeing AIO consoles in master that have no ansible output:

For example (from http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_064)

========================================================================

[2015-03-10 10:25:16.064](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_064) | if [ "${DEPLOY_TEMPEST}" == "yes" ]; then
[2015-03-10 10:25:16.064](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_064) | # Deploy tempest
[2015-03-10 10:25:16.064](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_064) | install_bits os-tempest-install.yml
[2015-03-10 10:25:16.064](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_064) | fi
[2015-03-10 10:25:16.064](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_064) | ++ '[' yes == yes ']'
[2015-03-10 10:25:16.065](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_065) | ++ install_bits os-tempest-install.yml
[2015-03-10 10:25:16.065](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_065) | ++ successerator openstack-ansible -v --forks 10 os-tempest-install.yml
[2015-03-10 10:25:16.065](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_065) | ++ set +e
[2015-03-10 10:25:16.065](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_065) | date +%s
[2015-03-10 10:25:16.065](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_065) | +++ date +%s
[2015-03-10 10:25:16.066](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_066) | ++ OP_START_TIME=1425983116
[2015-03-10 10:25:16.066](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_066) | ++ RETRY=0
[2015-03-10 10:25:16.066](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_066) | ++ false
[2015-03-10 10:25:16.066](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_066) | ++ '[' 1 -ne 0 -a 0 -lt 1 ']'
[2015-03-10 10:25:16.066](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_066) | ++ RETRY=1
[2015-03-10 10:25:16.066](http://logs.openstack.org/83/162183/1/check/os-ansible-deployment-dsvm-check-commit/d02ab06/console.html#_2015-03-10_10_25_16_06...

Read more...

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/163770

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/163772

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/163809

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/163810

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

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

commit ed090eb1094472dcef755e6ca319310e5a0e7eac
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 12 11:26:07 2015 +0000

    Adjust method for trimming ansible output debug information

    This patch adjusts the method introduced in
    I98fb57a56c1b7ddd6558e265cf6e65b159a05123 to trim the debug information
    from the ansible output.

    This adjustment allows the output to come to the console immediately,
    instead of piping to a file and doing the processing after the ansible
    run is complete.

    Change-Id: I515adcad34ef76b8ea47fe90bc6572fb3aec1d2c
    Partial-bug: #1425482

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

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

commit f24cde7c04a02dae3118c8798a1b6acf4c656e5b
Author: Kevin Carter <email address hidden>
Date: Thu Mar 5 14:46:06 2015 -0600

    Create ansible log file and directory

    This commit adds the ansible log file to the aio bootstrap script
    and creates a vanilla logging directory that will allow us to collect
    additional information on what is happening within a given ansible run.

    Partial-Bug: #1425482
    Partial-Bug: #1428744
    Change-Id: I2786f73334b99bfa98d344c50337c86f75fae273
    (cherry picked from commit 6f38cd47546dd96080f6016230f1c3480f76b470)

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

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

commit af1e740678facc4c7c0f651b45ec1fe970ea6e27
Author: Kevin Carter <email address hidden>
Date: Thu Mar 5 14:46:06 2015 -0600

    Create ansible log file and directory

    This commit adds the ansible log file to the aio bootstrap script
    and creates a vanilla logging directory that will allow us to collect
    additional information on what is happening within a given ansible run.

    Partial-Bug: #1425482
    Partial-Bug: #1428744
    Change-Id: I2786f73334b99bfa98d344c50337c86f75fae273
    (cherry picked from commit 6f38cd47546dd96080f6016230f1c3480f76b470)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

commit 0270e9fb4610f8184912afe6d4c602fcbb1f1b61
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 12 11:26:07 2015 +0000

    Adjust method for trimming ansible output debug information

    This patch adjusts the method introduced in
    I98fb57a56c1b7ddd6558e265cf6e65b159a05123 to trim the debug information
    from the ansible output.

    This adjustment allows the output to come to the console immediately,
    instead of piping to a file and doing the processing after the ansible
    run is complete.

    Change-Id: I515adcad34ef76b8ea47fe90bc6572fb3aec1d2c
    Partial-bug: #1425482
    (cherry picked from commit ed090eb1094472dcef755e6ca319310e5a0e7eac)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on os-ansible-deployment (juno)

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/163809
Reason: As per Kevin's comments, it is better to abandon this approach.

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

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

commit f611600fd41f66b583442c70d259a1556eb0ed69
Author: Jesse Pretorius <email address hidden>
Date: Fri Mar 13 12:51:24 2015 +0000

    Reduce script verbosity

    Currently the verbose shell builtin is used, resulting in the entire
    scripts contents being output in the logs. The instance information is
    also output to stdout. This is unnecessarily verbose and prone to
    failure. To ensure that the instance information is correctly rendered
    failures from within the instance information commands will now always
    return true and be logged within the `/openstack/log/ansible-logging`
    directory. Each time that the function is called a new log will be
    created tagged with the datecode in seconds. This commit also removes
    the verbose builtin from the scripts, removes the unused
    os-ansible-aio-check script.

    Partial-Bug: #1425482
    Change-Id: I59952d077a1a8dda12278e57a165d4bac272ed5f

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

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

commit 67be3d09aa043df4411aced6f55131c65a361f0c
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 12 11:26:07 2015 +0000

    Adjust method for trimming ansible output debug information

    This patch adjusts the method introduced in
    I98fb57a56c1b7ddd6558e265cf6e65b159a05123 to trim the debug information
    from the ansible output.

    This adjustment allows the output to come to the console immediately,
    instead of piping to a file and doing the processing after the ansible
    run is complete. It also ensures that the return code passes through the
    output trimming.

    This commit combines the following cherry-picks:
    - ed090eb1094472dcef755e6ca319310e5a0e7eac
    - b76d43e8c60bcb3060ee4ee99b6fac8acf0650fb

    Partial-bug: #1425482
    Closes-bug: #1432295
    Co-Authored-By: Hugh Saunders <email address hidden>
    Change-Id: I515adcad34ef76b8ea47fe90bc6572fb3aec1d2c

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.