[component-registry] incompatible logic isn’t same as for fuel-core components

Bug #1526353 reported by Ruslan Khozinov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Andriy Popovych

Bug Description

Fuel-build: 286

Example of component configuration:

- name: 'hypervisor:xen'
  label: 'xen'
  description: 'xen hyperviror'
  compatible:
    - name: 'hypervisor:libvirt:kvm'
  incompatible:
    - name: 'hypervisor:vmware'
    - name: ‘hypervisor:libvirt:qemu'

When I’m checking a xen hypervisor checkbox, others checkboxes are not blocked.

But when I’m checking kvm checkbox, then qemu checkbox is blocked (and vice versa).

So, I think the logic of blocking of checkboxes should be the same for core components and for plugin components.

I mean, when we check a checkbox incompatible checkboxes should be blocked.

Revision history for this message
Ruslan Khozinov (rkhozinov) wrote :
summary: - [PCE][component-registry] incompatible logic isn’t same as for fuel-core
+ [component-registry] incompatible logic isn’t same as for fuel-core
components
description: updated
description: updated
Revision history for this message
Ruslan Khozinov (rkhozinov) wrote :
Revision history for this message
Ruslan Khozinov (rkhozinov) wrote :
Changed in fuel:
assignee: Partner Centric Engineering (fuel-partner-engineering) → Andriy Popovych (popovych-andrey)
Ilya Kutukov (ikutukov)
Changed in fuel:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Andriy Popovych (popovych-andrey) wrote :

From your description: 'hypervisor:libvirt:emu'. Is it mistake here in bug description or it's the same in your components metadata file? Valid option is 'hypervisor:libvirt:qemu'

Revision history for this message
Ruslan Khozinov (rkhozinov) wrote :

This typo is only in the description.

description: updated
Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
assignee: Andriy Popovych (popovych-andrey) → Bulat Gaifullin (bgaifullin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/258651
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=1e4952ba9fff64a97b9ffd0f6e230ffbf11904c8
Submitter: Jenkins
Branch: master

commit 1e4952ba9fff64a97b9ffd0f6e230ffbf11904c8
Author: Andriy Popovych <email address hidden>
Date: Wed Dec 16 22:51:18 2015 +0200

    Create duplex relations for component

    Incompatible/compatible relations are duplex. If some component incompatible/
    compatible with other component it means the last also should have such
    relations.

    Closes-Bug: #1526353
    Change-Id: I0e946329a34480798d7871611363f45cb406904d

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/8.0)

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/274575

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

Reviewed: https://review.openstack.org/274575
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=2493d2d2bf3a1a112dbe59882a8ac8740d173594
Submitter: Jenkins
Branch: stable/8.0

commit 2493d2d2bf3a1a112dbe59882a8ac8740d173594
Author: Andriy Popovych <email address hidden>
Date: Wed Dec 16 22:51:18 2015 +0200

    Create duplex relations for component

    Incompatible/compatible relations are duplex. If some component incompatible/
    compatible with other component it means the last also should have such
    relations.

    Closes-Bug: #1526353
    Change-Id: I0e946329a34480798d7871611363f45cb406904d

Changed in fuel:
assignee: Bulat Gaifullin (bgaifullin) → Andriy Popovych (popovych-andrey)
tags: added: area-pce-vcenter
Revision history for this message
Olesia Tsvigun (otsvigun) wrote :

Verified at Fuel ISO#517

[root@nailgun nsxv-2.0]# cat /etc/fuel/8.0/version.yaml
VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "517"
  build_id: "517"
  fuel-nailgun_sha: "ebede8c0efab72b6e80735eb3a8161f953d84b83"
  python-fuelclient_sha: "4f234669cfe88a9406f4e438b1e1f74f1ef484a5"
  fuel-agent_sha: "658be72c4b42d3e1436b86ac4567ab914bfb451b"
  fuel-nailgun-agent_sha: "b2bb466fd5bd92da614cdbd819d6999c510ebfb1"
  astute_sha: "b81577a5b7857c4be8748492bae1dec2fa89b446"
  fuel-library_sha: "601a0378f3e7f42ee2bc8f3d5f54fc681b6c5102"
  fuel-ostf_sha: "5fe41945c2a49f26c849df1fd46329f6db1ab6b0"
  fuel-mirror_sha: "6f8e1a005446eb49981fe93bd9a67ca944c0a763"
  fuelmenu_sha: "12227354aec1d38f7f51042df64cca59fa7a95f1"
  shotgun_sha: "63645dea384a37dde5c01d4f8905566978e5d906"
  network-checker_sha: "a43cf96cd9532f10794dce736350bf5bed350e9d"
  fuel-upgrade_sha: "616a7490ec7199f69759e97e42f9b97dfc87e85b"
  fuelmain_sha: "a365f05b903368225da3fea9aa42afc1d50dc9b4"

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.