puppet unit tests fail due to missing oslo dependency

Bug #1569534 reported by Alex Schultz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Alex Schultz

Bug Description

Since the switch of openstack modules to use the new oslo dependency, we need to update our fixtures to include the oslo module for testing.

Steps to reproduce:
Run unit tests on modules that consume upstream modules without explicitly having the oslo module as a dependency.

Actual result:
2016-04-12 18:52:45.749 | Puppet::Parser::AST::Resource failed with error ArgumentError: Invalid resource type oslo::log at /home/jenkins/workspace/gate-fuel-library-puppet-unit-3.4-centos-7/deployment/puppet/cluster/spec/fixtures/modules/heat/manifests/logging.pp:142 on node centos-7-rax-iad-90082

Expected results:
Passing tests.

Workaround:
Add oslo module to the .fixtures.yml for the modules.

Version: 10

Affected modules:
ceph
cluster
openstack
osnailyfacter
vmware

Tags: area-library
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/304825
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=8d44fde1faca805d625d4dd53716c4bcf880da1d
Submitter: Jenkins
Branch: master

commit 8d44fde1faca805d625d4dd53716c4bcf880da1d
Author: Alex Schultz <email address hidden>
Date: Tue Apr 12 13:11:00 2016 -0600

    Add oslo for puppet testing dependencies

    This change adds oslo to the fixtures for the ceph, cluster, openstack,
    osnailyfacter and vmware modules. The oslo module is being used by the
    upstream puppet openstack modules to centralize the oslo configuration
    items so we need to pull it in for our rspec tests.

    Change-Id: Ia12fc7555d142626606bade86a6e3387eedf9c6e
    Closes-Bug: #1569534

Changed in fuel:
status: In Progress → Fix Committed
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.