gate failures due to tools/config/check_uptodate.sh

Bug #1297999 reported by Ildiko Vancsa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Critical
Ildiko Vancsa
Havana
Fix Released
Critical
Ryan Petrello

Bug Description

Ceilometer tests are failing on the gate due to the following error:

2014-03-26 15:52:53.583 | py27 runtests: commands[1] | /home/jenkins/workspace/gate-ceilometer-python27/tools/config/check_uptodate.sh
2014-03-26 15:52:55.271 | --- /tmp/ceilometer.KH1Xhy/ceilometer.conf.sample 2014-03-26 15:52:55.000000000 +0000
2014-03-26 15:52:55.271 | +++ etc/ceilometer/ceilometer.conf.sample 2014-03-26 15:48:32.000000000 +0000
2014-03-26 15:52:55.271 | @@ -772,6 +772,12 @@
2014-03-26 15:52:55.272 | # with Identity API Server. (integer value)
2014-03-26 15:52:55.272 | #http_request_max_retries=3
2014-03-26 15:52:55.272 |
2014-03-26 15:52:55.272 | +# Allows to pass in the name of a fake http_handler callback
2014-03-26 15:52:55.272 | +# function used instead of httplib.HTTPConnection or
2014-03-26 15:52:55.272 | +# httplib.HTTPSConnection. Useful for unit testing where
2014-03-26 15:52:55.272 | +# network is not available. (string value)
2014-03-26 15:52:55.272 | +#http_handler=<None>
2014-03-26 15:52:55.272 | +
2014-03-26 15:52:55.272 | # Single shared secret with the Keystone configuration used
2014-03-26 15:52:55.272 | # for bootstrapping a Keystone installation, or otherwise
2014-03-26 15:52:55.273 | # bypassing the normal authentication process. (string value)
2014-03-26 15:52:55.273 | @@ -809,24 +815,20 @@
2014-03-26 15:52:55.273 | # value)
2014-03-26 15:52:55.273 | #signing_dir=<None>
2014-03-26 15:52:55.273 |
2014-03-26 15:52:55.273 | -# Optionally specify a list of memcached server(s) to use for
2014-03-26 15:52:55.273 | -# caching. If left undefined, tokens will instead be cached
2014-03-26 15:52:55.273 | -# in-process. (list value)
2014-03-26 15:52:55.273 | +# If defined, the memcache server(s) to use for caching (list
2014-03-26 15:52:55.273 | +# value)
2014-03-26 15:52:55.273 | # Deprecated group/name - [DEFAULT]/memcache_servers
2014-03-26 15:52:55.273 | #memcached_servers=<None>
2014-03-26 15:52:55.274 |
2014-03-26 15:52:55.274 | -# In order to prevent excessive effort spent validating
2014-03-26 15:52:55.274 | -# tokens, the middleware caches previously-seen tokens for a
2014-03-26 15:52:55.274 | -# configurable duration (in seconds). Set to -1 to disable
2014-03-26 15:52:55.274 | -# caching completely. (integer value)
2014-03-26 15:52:55.274 | +# In order to prevent excessive requests and validations, the
2014-03-26 15:52:55.274 | +# middleware uses an in-memory cache for the tokens the
2014-03-26 15:52:55.274 | +# Keystone API returns. This is only valid if memcache_servers
2014-03-26 15:52:55.274 | +# is defined. Set to -1 to disable caching completely.
2014-03-26 15:52:55.274 | +# (integer value)
2014-03-26 15:52:55.274 | #token_cache_time=300
2014-03-26 15:52:55.275 |
2014-03-26 15:52:55.275 | -# Determines the frequency at which the list of revoked tokens
2014-03-26 15:52:55.275 | -# is retrieved from the Identity service (in seconds). A high
2014-03-26 15:52:55.275 | -# number of revocation events combined with a low cache
2014-03-26 15:52:55.275 | -# duration may significantly reduce performance. (integer
2014-03-26 15:52:55.275 | -# value)
2014-03-26 15:52:55.275 | -#revocation_cache_time=300
2014-03-26 15:52:55.275 | +# Value only used for unit testing (integer value)
2014-03-26 15:52:55.275 | +#revocation_cache_time=1
2014-03-26 15:52:55.275 |
2014-03-26 15:52:55.275 | # (optional) if defined, indicate whether token data should be
2014-03-26 15:52:55.275 | # authenticated or authenticated and encrypted. Acceptable
2014-03-26 15:52:55.276 | check_uptodate.sh: ceilometer.conf.sample is not up to date.
2014-03-26 15:52:55.276 | check_uptodate.sh: Please run /home/jenkins/workspace/gate-ceilometer-python27/tools/config/generate_sample.sh.
2014-03-26 15:52:55.276 | ERROR: InvocationError: '/home/jenkins/workspace/gate-ceilometer-python27/tools/config/check_uptodate.sh'

The issue seems to keystoneclient related, e.g.: 'revocation_cahce_time' was increased in the following commit: https://review.openstack.org/#/c/78700/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/83164

Changed in ceilometer:
assignee: nobody → Ildiko Vancsa (ildiko-vancsa)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/83164
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=63ff15ed95f71f5272c64b82ff148d35f75782d4
Submitter: Jenkins
Branch: master

commit 63ff15ed95f71f5272c64b82ff148d35f75782d4
Author: Ildiko Vancsa <email address hidden>
Date: Wed Mar 26 18:33:27 2014 +0100

    Fix ceilometer.conf.sample mismatch

    ceilometer.conf.sample mismatch issue makes the tests failing. This patch
    removes the checkupdate.sh test from tox.ini, like it was removed from nova
    too, until having a proper solution for handling the changes in
    ceilometer.conf.sample, to fix the gate quickly.

    Closes-Bug: #1297999

    Change-Id: I7889580b3df2d30d80703d7ef540194e7f71ec51

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
Ryan Petrello (ryan-petrello) wrote :

Could this be back ported to stable/havana? I'm gating pecan reviews against ceilometer stable/havana and so tests for pecan are still currently failing.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/havana)

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/83414

Revision history for this message
Ildiko Vancsa (ildiko-vancsa) wrote :

Hi Ryan,

Sorry, I haven't checked this yesterday and today I was way overloaded, thanks for backporting!

Best Regards,
Ildiko

Changed in ceilometer:
importance: Undecided → Critical
Changed in ceilometer:
milestone: none → icehouse-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/havana)

Reviewed: https://review.openstack.org/83414
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=7cb5f4cbffe576644fe4c48dee9ca215047f17c2
Submitter: Jenkins
Branch: stable/havana

commit 7cb5f4cbffe576644fe4c48dee9ca215047f17c2
Author: Ildiko Vancsa <email address hidden>
Date: Wed Mar 26 18:33:27 2014 +0100

    Fix ceilometer.conf.sample mismatch

    ceilometer.conf.sample mismatch issue makes the tests failing. This patch
    removes the checkupdate.sh test from tox.ini, like it was removed from nova
    too, until having a proper solution for handling the changes in
    ceilometer.conf.sample, to fix the gate quickly.

    Closes-Bug: #1297999

    (cherry picked from commit 63ff15ed95f71f5272c64b82ff148d35f75782d4)

    Conflicts:

     tox.ini
    Change-Id: Ia638fb19ce25788a418b59509ec3527e643aac4e

Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: icehouse-rc1 → 2014.1
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.