Move upload_core_repos pre-deployment task to fuel-library

Bug #1508486 reported by Vladimir Kuklin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Bartłomiej Piotrowski
8.0.x
Fix Released
High
Bartłomiej Piotrowski

Bug Description

Currently we have a set of pre-deployment tasks run by Nailgun. One of them is upload_core_repos.

We need to put its code and metadata into fuel-library.

description: updated
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Changed in fuel:
importance: High → Medium
status: Confirmed → Triaged
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Bartłomiej Piotrowski (bpiotrowski)
tags: added: tech-debt
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/248057

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to puppet-modules/puppetlabs-apt (master)

Related fix proposed to branch: master
Change author: Sergii Golovatiuk <email address hidden>
Review: https://review.fuel-infra.org/14443

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on puppet-modules/puppetlabs-apt (master)

Change abandoned by Bart?omiej Piotrowski <email address hidden> on branch: master
Review: https://review.fuel-infra.org/14443

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to puppet-modules/puppetlabs-apt (2.2.0-mos)

Related fix proposed to branch: 2.2.0-mos
Change author: Sergii Golovatiuk <email address hidden>
Review: https://review.fuel-infra.org/14449

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to puppet-modules/puppetlabs-apt (2.2.0-mos)

Reviewed: https://review.fuel-infra.org/14449
Submitter: Ivan Berezovskiy <email address hidden>
Branch: 2.2.0-mos

Commit: 19583b8a85744f914a0690060576cd189fd12dbf
Author: Sergii Golovatiuk <email address hidden>
Date: Mon Dec 7 10:06:14 2015

Change behavior with port in URI location

When location contains port in URI, pinning policy are set to default
(500) even if config has higher value. This patch strips port from URI
leaving Host only.

Pull request: https://github.com/puppetlabs/puppetlabs-apt/pull/574
Related-bug: 1508486

Change-Id: I5f3804f1f8929c0741dd639271961d543f8c557f

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/254165

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (master)

Change abandoned by Maciej Kwiek (<email address hidden>) on branch: master
Review: https://review.openstack.org/254165
Reason: We can't delete these tasks due to older version support.

Changed in fuel:
milestone: 8.0 → 9.0
description: updated
summary: - We need to extract all pre-deployment tasks from Nailgun
+ Move upload_core_repos pre-deployment task to fuel-library
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/248057
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=4a6c031e79bbb98b0dc0a51a9963dc170cbdaa34
Submitter: Jenkins
Branch: master

commit 4a6c031e79bbb98b0dc0a51a9963dc170cbdaa34
Author: Bartłomiej Piotrowski <email address hidden>
Date: Fri Nov 20 13:47:08 2015 +0100

    Replace upload_core_repos with Puppet-only task

    Currently the code responsible for configuring repositories on the
    managed nodes is kept directly it Nailgun. This commit introduces a new
    task setup_repositories that replaces upload_core_repos.

    Closes-Bug: 1508486
    Change-Id: I1b83e3bfaebecdb8455d5697e320f24fb4941536

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

Related fix proposed to branch: master
Review: https://review.openstack.org/268976

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

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

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

Reviewed: https://review.openstack.org/269012
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=f0c286624901bb2fc2dbef23fdc236b4a82b231b
Submitter: Jenkins
Branch: stable/8.0

commit f0c286624901bb2fc2dbef23fdc236b4a82b231b
Author: Bartłomiej Piotrowski <email address hidden>
Date: Fri Nov 20 13:47:08 2015 +0100

    Replace upload_core_repos with Puppet-only task

    Currently the code responsible for configuring repositories on the
    managed nodes is kept directly it Nailgun. This commit introduces a new
    task setup_repositories that replaces upload_core_repos.

    Closes-Bug: 1508486
    Closes-Bug: 1533682

    Change-Id: I1b83e3bfaebecdb8455d5697e320f24fb4941536
    (cherry picked from commit 4a6c031e79bbb98b0dc0a51a9963dc170cbdaa34)

Revision history for this message
Alexandr Kostrikov (akostrikov-mirantis) wrote :

There is such line on successful deployments:
root@node-1:~# grep -r "setup_repo" /var/log/puppet.log
2016-02-04 14:37:27 +0000 Scope(Class[main]) (notice): MODULAR: setup_repositories.pp

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (master)

Change abandoned by Igor Kalnitsky (<email address hidden>) on branch: master
Review: https://review.openstack.org/268976
Reason: I abandon that patch since we're ready to remove it yet. We should support oldier releases.

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
Alexandr Kostrikov (akostrikov-mirantis) wrote :
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.