[fuel-ci] error in test_cannot_change_repos_on_operational_cluster

Bug #1441509 reported by Aleksandra Fedorova
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Ihor Kalnytskyi
6.0.x
Invalid
Undecided
Ihor Kalnytskyi

Bug Description

fuel-web master is broken due to the following error:

self = <nailgun.test.integration.test_attributes.TestAlwaysEditable testMethod=test_cannot_change_repos_on_operational_cluster>

    def test_cannot_change_repos_on_operational_cluster(self):
        self.cluster.status = consts.CLUSTER_STATUSES.operational
        self.db.flush()

        data = {'editable': {}}
        data['editable'].update(self._reposetup)
        data['editable'].update({'access': {}}) # always_editable is False

        self._put(data, expect_code=403)

        attrs = self.cluster.attributes.editable
        self.assertEqual(attrs['repo_setup']['repos']['value'], [{
            'type': 'rpm',
            'name': 'mos',
            'uri': 'http://127.0.0.1:8080/2014.2-6.1/centos/x86_64',
> 'priority': 20,
        }])
E AssertionError: Lists differ: [{u'priority': None, u'type': ... != [{'priority': 20, 'type': 'rpm...
E
E First differing element 0:
E {u'priority': None, u'type': u'rpm', u'name': u'mos', u'uri': u'http://127.0.0.1:8080/2014.2-6.1/centos/x86_64'}
E {'priority': 20, 'type': 'rpm', 'name': 'mos', 'uri': 'http://127.0.0.1:8080/2014.2-6.1/centos/x86_64'}
E
E - [{u'name': u'mos',
E ? - -
E
E + [{'name': 'mos',
E - u'priority': None,
E ? - ^^^^
E
E + 'priority': 20,
E ? ^^
E
E - u'type': u'rpm',
E ? - -
E
E + 'type': 'rpm',
E - u'uri': u'http://127.0.0.1:8080/2014.2-6.1/centos/x86_64'}]
E ? - -
E
E + 'uri': 'http://127.0.0.1:8080/2014.2-6.1/centos/x86_64'}]

nailgun/test/integration/test_attributes.py:429: AssertionError

Issue is Critical as it blocks CI

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Igor Kalnitsky (ikalnitsky)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/171528
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=a73ec6b86ce4f8f396ee21490034bdb851d81c2e
Submitter: Jenkins
Branch: master

commit a73ec6b86ce4f8f396ee21490034bdb851d81c2e
Author: Igor Kalnitsky <email address hidden>
Date: Wed Apr 8 12:00:29 2015 +0300

    Fix test_cannot_change_repos_on_operational_env

    Since the patch Ief7cb7440db02ecbc0dd42a2cb05b8f66b83e7a0 the default
    priority value for CentOS's MOS repo was changes to 'None', and the
    other patch I0c58a03b2cd83e1fb196f5a55002d5a869277c6f didn't know
    nothing about it. Parallel merging of them led to broken test.

    This patch fix test to fit new criteria.

    Change-Id: I0e2d2d280f85e19bc73dfc08cab407733e7f3dc2
    Closes-Bug: #1441509

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