Mirantis release instead of community Fuel uploaded to fuel-infra.org

Bug #1553556 reported by Mike Scherbakov
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Pawel Brzozowski

Bug Description

I downloaded Fuel community 8.0 release from fuel-infra.org:
http://seed.fuel-infra.org/fuelweb-community-release/fuel-community-8.0.iso.torrent
However, when I installed it, I see Mirantis logo in UI.

/api/version returns Mirantis feature group.

I expected to see Fuel community build, not Mirantis build.

md5 of ISO: e4585b609d02589a75fb5c089e0d4996

Tags: area-build
Mike Scherbakov (mihgen)
description: updated
Mike Scherbakov (mihgen)
Changed in fuel:
assignee: Dmitry Borodaenko (dborodaenko) → Fuel build team (fuel-build)
Revision history for this message
Sergey Kulanov (skulanov) wrote :

@Mike, not sure if build team can help here, since we are exporting correct FEATURE_GROUP in community jobs [1] and correct splash screens are used in ISO.

The issue probably was introduced in fuelmenu project in this patch [2]

[1]. https://github.com/fuel-infra/jenkins-jobs/blob/master/servers/fuel-ci/8.0/builders/community.all.sh#L5
[2]. https://review.openstack.org/#/c/262228/

Changed in fuel:
status: New → Confirmed
Revision history for this message
Vladimir Kozhukalov (kozhukalov) wrote :

It does not matter whether you export or not this FEATURE_GROUP variable. Build system is only responsible for splash screen and ISO menu title. This setting comes from fuel-menu. In master branch it is empty by default [1] while in 8.0 branch it is set to Mirantis [2] because in 8.0 we didn't have working downstream. BTW, why is it critical?

[1] https://github.com/openstack/fuel-menu/blob/master/fuelmenu/settings.yaml#L68
[2] https://github.com/openstack/fuel-menu/blob/stable/8.0/fuelmenu/settings.yaml#L69

Revision history for this message
Mike Scherbakov (mihgen) wrote :

It is critical, because I expect to download community version of Fuel. I'm getting Mirantis proprietary one instead, which asks me to send statistics to mirantis, etc.

Revision history for this message
Vladimir Kozhukalov (kozhukalov) wrote :

Accrding to https://wiki.openstack.org/wiki/Fuel/How_to_contribute

* Critical = Requires massive effort to work around, including un/under-documented commands and edits to config files
* High = Requires modification of config files, interfaces that users aren't expected to use (ie the API when it's _intended_ to work in the CLI / UI (exclusive of interfaces that are intended to only be available via API) or requires custom node yaml (again except when it should exclusively be available)
* Medium: Straight forward commands in the CLI
* Low and Wishlist = as above

The behavior (stats related) could easily be changed by removing 'mirantis' feature group from the list of feature groups using fuelmenu command. So, this bug should be Medium, not Critical.

Revision history for this message
Vladimir Kozhukalov (kozhukalov) wrote :

Am I right, that splash screen and isolinux title do not contain Mirantis proprietary values?

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

I agree that this bug is incorrectly prioritized. Absense of Mirantis proprietary functionality in the default settings for the community ISO is a feature, and that feature is currently broken. I don't think that changing that setting via fuel-menu during Fuel node setup is a valid workaround, so the right priority for this bug is High. Still, we need to fix it quickly, at least manually trigger a rebuild of the ISO with a commit for fuel-menu that reverts https://review.openstack.org/262228.

In 9.0, we should build two RPMs of fuel-menu: one with mirantis feature group in the default settings.yaml and one without. The first one should go into a separate yum repo that would only be included in product ISO and not community ISO, and would override the community RPM of fuel-menu.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-menu (stable/8.0)

Related fix proposed to branch: stable/8.0
Review: https://review.openstack.org/289536

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to fuel-infra/jenkins-jobs (master)

Related fix proposed to branch: master
Change author: Aleksandra Fedorova <email address hidden>
Review: https://review.fuel-infra.org/17820

Changed in fuel:
status: Confirmed → In Progress
Roman Vyalov (r0mikiam)
Changed in fuel:
assignee: Fuel build team (fuel-build) → Aleksandra Fedorova (afedorova)
Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

ISO fuel-community-8.0-1485-2016-03-07_20-34-31.iso passed BVT in http://jenkins-product.srt.mirantis.net:8080/job/8.0.custom.ubuntu.bvt_2/632/console

Devops team please reupload the fuel-community-8.0.iso artifact using this one:
https://ci.fuel-infra.org/job/8.0.publish_fuel_community_iso/156/

Current artifact needs to be cleaned from from trackers and seed-* servers.

Changed in fuel:
importance: Critical → High
assignee: Aleksandra Fedorova (afedorova) → Fuel DevOps (fuel-devops)
status: In Progress → New
Andrey Nikitin (heos)
Changed in fuel:
status: New → Triaged
Changed in fuel:
assignee: Fuel DevOps (fuel-devops) → Pawel Brzozowski (pbrzozowski)
Changed in fuel:
status: Triaged → Fix Released
status: Fix Released → In Progress
Revision history for this message
Pawel Brzozowski (pbrzozowski) wrote :

Published: fuel-community-8.0-1485-2016-03-07_20-34-31.iso 8f05bc8eb81a26c9aa72d14a0c5bd72a

Changed in fuel:
status: In Progress → Fix Released
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/17820
Submitter: Aleksandra Fedorova <email address hidden>
Branch: master

Commit: 963dec7e18a7de1534e99d49c822f6c637dab710
Author: Aleksandra Fedorova <email address hidden>
Date: Mon Mar 7 20:40:37 2016

Fix 8.0 branding in community builds

Cherry-pick patch which changes default for FEATURE_GROUPS variable.

Change-Id: I222ce927746754daae32c848a58768d8430a315a
Related-Bug: #1553556

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-menu (stable/8.0)

Change abandoned by Andreas Jaeger (<email address hidden>) on branch: stable/8.0
Review: https://review.opendev.org/289536
Reason: This repo is retired now, no further work will get merged.

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.