Project

General

Profile

Actions

Bug #23653

closed

parsing facts break interfaces names

Added by Maksim Malchuk about 6 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
High
Target version:
-
Fixed in Releases:
Found in Releases:

Description

Import interfaces broken while parsing facts:

2018-05-20T10:46:17 88de1713 [app] [D] We have following interfaces 'br-private, enp2s0f0, enp2s0f1, bond1.1005, br-ipo, br-ipo-nic, enp130s0f0, enp130s0f1, enp129s0f0, enp129s0f1, bond0, bond1' based on facts
2018-05-20T10:46:17 88de1713 [app] [D] Saving br_private NIC for host cloud-lab15.example.org
2018-05-20T10:46:18 88de1713 [app] [W] Saving br_private NIC for host cloud-lab15.example.org failed, skipping because:
2018-05-20T10:46:18 88de1713 [app] [W] Identifier has already been taken
2018-05-20T10:46:18 88de1713 [app] [D] Saving enp2s0f0 NIC for host cloud-lab15.example.org
2018-05-20T10:46:18 88de1713 [app] [D] Saving enp2s0f1 NIC for host cloud-lab15.example.org
2018-05-20T10:46:18 88de1713 [app] [D] Saving bond1.1005 NIC for host cloud-lab15.example.org
2018-05-20T10:46:18 88de1713 [app] [D] Saving br_ipo NIC for host cloud-lab15.example.org
2018-05-20T10:46:18 88de1713 [app] [D] Saving br_ipo_nic NIC for host cloud-lab15.example.org
2018-05-20T10:46:18 88de1713 [app] [D] Saving enp130s0f0 NIC for host cloud-lab15.example.org
2018-05-20T10:46:19 88de1713 [app] [D] Saving enp130s0f1 NIC for host cloud-lab15.example.org
2018-05-20T10:46:19 88de1713 [app] [D] Saving enp129s0f0 NIC for host cloud-lab15.example.org
2018-05-20T10:46:19 88de1713 [app] [D] Saving enp129s0f1 NIC for host cloud-lab15.example.org
2018-05-20T10:46:19 88de1713 [app] [D] Saving bond0 NIC for host cloud-lab15.example.org
2018-05-20T10:46:19 88de1713 [app] [D] Saving bond1 NIC for host cloud-lab15.example.org

Some interfaces are broken (renamed) and some absent:

  1. hammer host interface list --host-id 1
    ---|------------|--------------------------------|-------------------|-------------|------------------------
    ID | IDENTIFIER | TYPE | MAC ADDRESS | IP ADDRESS | DNS NAME
    ---|------------|--------------------------------|-------------------|-------------|------------------------
    8 | bond1 | bond | 2c:9d:1e:b1:f5:f6 | |
    7 | bond0 | bond | 2c:9d:1e:b1:f5:f5 | |
    1 | enp129s0f1 | interface (primary, provision) | 2c:9d:1e:b1:f5:f6 | | cloud-lab15.example.org
    6 | enp129s0f0 | interface | 2c:9d:1e:b1:f5:f5 | |
    5 | br_ipo_nic | interface | 52:54:00:cd:a9:9c | |
    4 | br_ipo | interface | 52:54:00:cd:a9:9c | 192.168.1.1 |
    3 | enp2s0f1 | interface | 2c:9d:1e:6c:d3:c6 | |
    2 | enp2s0f0 | interface | 2c:9d:1e:6c:d3:c5 | |
    ---|------------|--------------------------------|-------------------|-------------|------------------------

We need the same behavior used in the: https://github.com/theforeman/foreman/blob/develop/app/services/puppet_fact_parser.rb#L132-L139


Related issues 2 (1 open1 closed)

Related to Foreman - Bug #23654: bridge interface facts don't imported in case of primary interfaceClosedMaksim MalchukActions
Related to Foreman - Bug #23655: vlan interface facts don't imported in case of ansible pluginReady For TestingMaksim MalchukActions
Actions #1

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_ansible/pull/172 added
Actions #2

Updated by Maksim Malchuk about 6 years ago

  • Related to Bug #23654: bridge interface facts don't imported in case of primary interface added
Actions #3

Updated by Maksim Malchuk about 6 years ago

  • Related to Bug #23655: vlan interface facts don't imported in case of ansible plugin added
Actions #4

Updated by The Foreman Bot about 6 years ago

  • Pull request https://github.com/theforeman/foreman_ansible/pull/173 added
Actions #5

Updated by The Foreman Bot almost 5 years ago

  • Fixed in Releases foreman-ansible 3.0.8 added
Actions #6

Updated by Maksim Malchuk almost 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF