Gateway ping process should be moved from netconfig to connectivity checker

Bug #1624288 reported by Stanislaw Bogatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Invalid
High
Stanislaw Bogatkin
Mitaka
Fix Released
High
Stanislaw Bogatkin
Newton
Invalid
High
Stanislaw Bogatkin

Bug Description

Gateway ping function doesn't work in case of net-verification from bootstrap node. So for those cases it should be moved from net-checker to connectivity tests.

Tested on current 9.1 GA

Tags: area-library
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/371395

Changed in fuel:
status: New → In Progress
tags: added: area-library
Revision history for this message
Andrey Maximov (maximov) wrote :

moved to 9.2 since it doesn't affect users directly.

Changed in fuel:
milestone: 10.0 → 11.0
Changed in fuel:
status: Invalid → Confirmed
Changed in fuel:
status: Confirmed → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/371395
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=8cd083474fbfdd41dad14503ed952b101d2dcc94
Submitter: Jenkins
Branch: stable/mitaka

commit 8cd083474fbfdd41dad14503ed952b101d2dcc94
Author: Stanislaw Bogatkin <email address hidden>
Date: Wed Aug 31 17:24:58 2016 +0300

    Move gateway ping to connectivity tests

    Gateway ping is actually function of connectivity checker, not netconfig
    one, so it should be moved from there. Also it breaks the case when
    netconfig tasks ran on bootstrap node for network verification graph
    when there is no default gateway yet.

    Change-Id: I8c5fbd1b78eeb0abbae57c66a5b0c1d17b81fad2
    (cherry picked from commit 660d98018455aa53f56d1946a39be8fa1364f8a2)
    Closes-Bug: #1624288

tags: added: on-verification
Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Verification on 9.2 is postponed: waiting for fix https://review.openstack.org/#/c/410085/ is merged.

Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Verified on 9.2 snapshot #747.

Actual results:
Net-verification graph is executed successfully:

[root@nailgun ~]# ll /etc/fuel/graphs/
total 12
drwxr-xr-x. 2 root root 4096 Jan 11 12:51 deletion
drwxr-xr-x. 2 root root 4096 Jan 11 12:51 net-verification
drwxr-xr-x. 2 root root 4096 Jan 11 12:51 provision
[root@nailgun ~]# fuel2 graph upload -e 1 -t my_type -d /etc/fuel/graphs/net-verification/
Deployment graph was successfully uploaded.
[root@nailgun ~]# fuel2 graph execute -e 1 -t my_type
Deployment task with id 2 for the environment 1 has been started.

Deploy is finished successfully:
[root@nailgun ~]# fuel2 task list
+----+--------+------------+------------+---------+--------+---------+----------+
| id | status | name | graph_type | cluster | result | dry_run | progress |
+----+--------+------------+------------+---------+--------+---------+----------+
| 1 | ready | dump | None | None | {} | False | 100 |
| 2 | ready | deploy | None | 1 | {} | False | 100 |
| 3 | ready | deployment | my_type | 1 | {} | False | 100 |
+----+--------+------------+------------+---------+--------+---------+----------+

tags: removed: on-verification
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.