[CI] CI for fuel-specs doesn't work for 9.0

Bug #1569929 reported by Sergey Kulanov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Critical
Ihor Kalnytskyi

Bug Description

CI for fuel-specs doesn't work, it seems because of hardcoded value which is the only 8.0 [1]

When adding 9.0 version, ci failed with error:

======================================================================
FAIL: tests.test_titles.TestTitles.test_template
tags: worker-1
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/base.py", line 82, in test_template
    self.check_lines_wrapping(filename, data)
  File "tests/base.py", line 55, in check_lines_wrapping
    (tpl, i+1))
  File "projects/fuel-specs/.tox/py27/local/lib/python2.7/site-packages/unittest2/case.py", line 702, in assertTrue
    raise self.failureException(msg)
AssertionError: False is not true : specs/9.0/fuel-nonroot-openstack-nodes.rst:48: Line limited to a maximum of 79 characters.

[1]. https://github.com/openstack/fuel-specs/blob/master/tests/test_titles.py#L52

probably we need to fix it by adding 9.0 version as well

Changed in fuel:
assignee: Fuel CI (fuel-ci) → Fuel Python Team (fuel-python)
tags: added: area-python
removed: area-ci
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Igor Kalnitsky (ikalnitsky)
status: New → Confirmed
Dmitry Pyzhov (dpyzhov)
tags: added: non-release
tags: removed: need-info
Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
milestone: 9.0 → 10.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-specs (master)

Reviewed: https://review.openstack.org/307180
Committed: https://git.openstack.org/cgit/openstack/fuel-specs/commit/?id=6ec0b000dd70080b434a3434a284d233123372b4
Submitter: Jenkins
Branch: master

commit 6ec0b000dd70080b434a3434a284d233123372b4
Author: Igor Kalnitsky <email address hidden>
Date: Thu Apr 14 13:09:18 2016 +0300

    Use docutils AST for making smart checks

    Trying to enable testing specs for 9.0, we've encountered different
    unavoidable problems with existing checks. For example, it was
    impossible to have code blocks with line length more than 80 characters.

    This commit uses docutils and make those changes using AST, so we can
    avoid some checks for code-blocks or long links.

    Closes-Bug: #1569929
    Change-Id: Ia501754922b3272acd1a865513d5dffa17981331

Changed in fuel:
status: In Progress → Fix Committed
no longer affects: fuel/mitaka
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.