EXTRA_DEB_REPOS is not used with KEEP_BEFORE=no

Bug #1484137 reported by Alexander Tsamutali
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Dennis Dmitriev

Bug Description

Recently we found that when we run deploy test without KEEP_BEFORE=no, EXTRA_DEB_REPOS is not used and thus we don't actually test new package. [1] Please fix it. Workarounds better than setting KEEP_BEFORE=no are welcome too.

[1] http://osci-jenkins.srt.mirantis.net:8080/job/7.0.mos-new.deploy-deb/734/console

Changed in fuel:
assignee: nobody → Fuel QA Team (fuel-qa)
importance: Undecided → High
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: Alexander Tsamutali <email address hidden>
Review: https://review.fuel-infra.org/10376

Changed in fuel:
milestone: none → 7.0
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/10376
Submitter: Alexander Tsamutali <email address hidden>
Branch: master

Commit: 79a1ceaf648fdc4e44e2e5478a5a2918b64ea0fb
Author: Alexander Tsamutali <email address hidden>
Date: Wed Aug 12 15:09:26 2015

[openstack-ci] Set KEEP_BEFORE=no in deploy tests

Change-Id: I46f71078c3e4afe8592c0e58a7c7b48870e2c1e5
Related-Bug: 1484137

Changed in fuel:
importance: High → Critical
Changed in fuel:
status: New → Fix Committed
assignee: Fuel QA Team (fuel-qa) → Alexander Tsamutali (astsmtl)
Changed in fuel:
status: Fix Committed → New
assignee: Alexander Tsamutali (astsmtl) → Fuel QA Team (fuel-qa)
Changed in fuel:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

Fix proposed to branch: master
Review: https://review.openstack.org/212559

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Dennis Dmitriev (ddmitriev)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/212559
Committed: https://git.openstack.org/cgit/stackforge/fuel-qa/commit/?id=ba298889026f1aeeee490411336a2b91a3c9e12e
Submitter: Jenkins
Branch: master

commit ba298889026f1aeeee490411336a2b91a3c9e12e
Author: Dennis Dmitriev <email address hidden>
Date: Thu Aug 13 17:34:00 2015 +0300

    Add REPLACE_DEFAULT_REPOS_ONLY_ONCE variable to manage repos list

    export REPLACE_DEFAULT_REPOS_ONLY_ONCE=true
     - Along with REPLACE_DEFAULT_REPOS=true , enables replacing default
       repositories only when Fuel admin node has been installed

    export REPLACE_DEFAULT_REPOS_ONLY_ONCE=false
     - Along with REPLACE_DEFAULT_REPOS=true , enables replacing default
       repositories only before a new cluster is created by tests.

    export REPLACE_DEFAULT_REPOS=false
     - Disables replacing repositories, Nailgun keeps it's defaults.

    Change-Id: Ief45650580cdd897bb785f286d850d27baf06adc
    Closes-Bug:#1484137

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to infra/ci-test-request (master)

Related fix proposed to branch: master
Change author: Alexander Tsamutali <email address hidden>
Review: https://review.fuel-infra.org/10581

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to infra/ci-test-request (stable/6.1)

Related fix proposed to branch: stable/6.1
Change author: Alexander Tsamutali <email address hidden>
Review: https://review.fuel-infra.org/10582

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to infra/ci-test-request (master)

Reviewed: https://review.fuel-infra.org/10581
Submitter: Alexander Tsamutali <email address hidden>
Branch: master

Commit: 475fac01d5321cade7f53fe74620a015be27a542
Author: Alexander Tsamutali <email address hidden>
Date: Wed Aug 19 15:52:09 2015

export REPLACE_DEFAULT_REPOS_ONLY_ONCE=false

Change-Id: Ia38a18aef3b264a1cc307d2eeed5c9df3b1b814d
Related-Bug: 1484137

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to infra/ci-test-request (stable/6.1)

Reviewed: https://review.fuel-infra.org/10582
Submitter: Alexander Tsamutali <email address hidden>
Branch: stable/6.1

Commit: 18a4b69f5f2e715f7928cbabf82cf546e2f10865
Author: Alexander Tsamutali <email address hidden>
Date: Wed Aug 19 15:53:31 2015

export REPLACE_DEFAULT_REPOS_ONLY_ONCE=false

Change-Id: Ia38a18aef3b264a1cc307d2eeed5c9df3b1b814d
Related-Bug: 1484137

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/10584
Submitter: Artem Silenkov <email address hidden>
Branch: master

Commit: 24587e8256951ee8cfe5c89c6a28e4e58585bd5c
Author: Alexander Tsamutali <email address hidden>
Date: Wed Aug 19 17:01:00 2015

Revert "[openstack-ci] Set KEEP_BEFORE=no in deploy tests"

This reverts commit 79a1ceaf648fdc4e44e2e5478a5a2918b64ea0fb.

Change-Id: I1c33a098f027cea0c9aead7ceda69a4bb6638ef7
Related-Bug: 1484137

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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.