wrong default for tenant_network_mask_bits

Bug #1479092 reported by Mark Vanderwiel on 2015-07-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Medium
Mark Vanderwiel

Bug Description

cookbook template has:

[network]

api_version = 2.0
catalog_type = network

tenant_network_cidr = 10.100.0.0/16

tenant_network_mask_bits = 29

tenant_networks_reachable = false

public_network_id =

public_router_id =

quantum_available = false

With this tempest test failure:

FAIL: tempest.tests.common.utils.linux.test_remote_client.TestRemoteClient.test_assign_static_ip
AssertionError: Expected call: exec_command('set -eu -o pipefail; PATH=$PATH:/sbin; sudo ip addr add 10.0.0.2/28 dev eth0')
Actual call: exec_command('set -eu -o pipefail; PATH=$PATH:/sbin; sudo ip addr add 10.0.0.2/29 dev eth0')

the tenant_network_mask_bits = 29, should be 28.

See the default here: https://github.com/openstack/tempest/blob/91a9548c147393fade7930d7e00c2a0a1ec3bac0/tempest/config.py#L473

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

Changed in openstack-chef:
status: New → In Progress

Reviewed: https://review.openstack.org/206683
Committed: https://git.openstack.org/cgit/openstack/cookbook-openstack-integration-test/commit/?id=af0763c67f25af20964bf9bc10ea3717855035ac
Submitter: Jenkins
Branch: master

commit af0763c67f25af20964bf9bc10ea3717855035ac
Author: Mark Vanderwiel <email address hidden>
Date: Tue Jul 28 14:48:45 2015 -0500

    Fix default for tenant_network_mask_bits

    Fix the template default. Maybe another patch can look at why
    we even need these defaults in the template forcing us to stay in
    sync.

    Change-Id: Id704566bc91291252d9dd35d41afed136dabe901
    Closes-Bug: #1479092

Changed in openstack-chef:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers