Fuel for OpenStack

allow to specify path where to put packages built by perestroika scripts

Bug #1511757 reported by Aleksandra Fedorova
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Dmitry Burmistrov
Fuel for OpenStack 9.0

Bug Description

Currently perestroika scripts use internal logic to determine type of the request and the path for the repository where it publishes a package

see https://github.com/openstack/fuel-mirror/blob/master/perestroika/build-functions.sh#L60-L83

In this snippet it checks status of a change in Gerrit and then defines parameters for the publisher.

In Packaging CI status of the change and path for the request is defined by Zuul and Jenkins job it uses. For example to implement gating properly we need a following action:

- get change request and build package from it and put to repository with the name ".../$JOB_NAME/$BUILD_NUMBER"

This action mustn't be affected by status of a change request or any other parameter outside of the current Jenkins environment.

Dmitry Burmistrov (dburmistrov)
Changed in fuel:
assignee: Fuel build team (fuel-build) → Dmitry Burmistrov (dburmistrov)
status: New → Triaged
Aleksandra Fedorova (bookwar)
summary: - allow to specify path where to put packages built bu perestroika scripts
+ allow to specify path where to put packages built by perestroika scripts
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-mirror (master)

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

Changed in fuel:
status: Triaged → In Progress
Roman Vyalov (r0mikiam)
tags: added: area-build
Revision history for this message
Roman Vyalov (r0mikiam) wrote :

ci-team please review https://review.openstack.org/240822

Changed in fuel:
assignee: Dmitry Burmistrov (dburmistrov) → Fuel CI (fuel-ci)
status: In Progress → Confirmed
Roman Vyalov (r0mikiam)
tags: added: on-review
Changed in fuel:
assignee: Fuel CI (fuel-ci) → Dmitry Burmistrov (dburmistrov)
Roman Vyalov (r0mikiam)
Changed in fuel:
status: Confirmed → In Progress
Roman Vyalov (r0mikiam)
tags: added: move-to-9-0
removed: on-review
Roman Vyalov (r0mikiam)
Changed in fuel:
status: In Progress → Triaged
Roman Vyalov (r0mikiam)
Changed in fuel:
importance: High → Medium
Roman Vyalov (r0mikiam)
Changed in fuel:
milestone: 8.0 → 9.0
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

After discussing with ci-team I found that this case totally break current packages grouping workflow.
Need approve from mos-packaging team to disable group feature

Changed in fuel:
assignee: Dmitry Burmistrov (dburmistrov) → Igor Yozhikov (iyozhikov)
Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

Dima, in this bug we need a _possibility_ in perestroika scripts to publish package to the path we specify. This shouldn't affect the current packaging workflow.

I see it this way: if I specify explicitly the output path for example via CUSTOM_OUTPUT_REPO_PATH variable, then repo should be created there. If variable is not specified - then usual rules for grouping, gating and so on should apply.

Changed in fuel:
assignee: Igor Yozhikov (iyozhikov) → Dmitry Burmistrov (dburmistrov)
status: Triaged → New
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Change abandoned by Dmitry Burmistrov (<email address hidden>) on branch: master
Review: https://review.openstack.org/240822

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-mirror (master)

Reviewed: https://review.openstack.org/290648
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=4af7fc2e9cccbb99ffd3651462bf2c4f1a5ef9c5
Submitter: Jenkins
Branch: master

commit 4af7fc2e9cccbb99ffd3651462bf2c4f1a5ef9c5
Author: Dmitry Burmistrov <email address hidden>
Date: Wed Mar 9 19:38:06 2016 +0300

    [build] Do not include CR repositories at build stage

    Change-Id: I4beb448d5b3073033cfa1df6580680cb9153fc66
    Partial-Bug: #1511757

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/290649
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=36fd222432445620888068feed7fdee3b9f96926
Submitter: Jenkins
Branch: master

commit 36fd222432445620888068feed7fdee3b9f96926
Author: Dmitry Burmistrov <email address hidden>
Date: Wed Mar 9 19:50:45 2016 +0300

    [publisher] Implement custom repo path

       Use CUSTOM_REPO_ID variable to override default path for CR
       repositories

    Change-Id: I222be69d28a2c1c43a1ef1e063696bf25ad15c10
    Closes-Bug: #1511757

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Reviewed: https://review.openstack.org/310744
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=a79627345afe38c5cfe1aec35ba710e24aa23d0e
Submitter: Jenkins
Branch: stable/8.0

commit a79627345afe38c5cfe1aec35ba710e24aa23d0e
Author: Dmitry Burmistrov <email address hidden>
Date: Wed Mar 9 19:38:06 2016 +0300

    [build] Do not include CR repositories at build stage

    Partial-Bug: #1575171

    Change-Id: I4beb448d5b3073033cfa1df6580680cb9153fc66
    Partial-Bug: #1511757
    (cherry picked from commit 4af7fc2e9cccbb99ffd3651462bf2c4f1a5ef9c5)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/310741
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=33f71a0e94c420e648d44f79acae504cbe9302eb
Submitter: Jenkins
Branch: stable/8.0

commit 33f71a0e94c420e648d44f79acae504cbe9302eb
Author: Dmitry Burmistrov <email address hidden>
Date: Wed Mar 9 19:50:45 2016 +0300

    [publisher] Implement custom repo path

       Use CUSTOM_REPO_ID variable to override default path for CR
       repositories

    Partial-Bug: #1575171

    Change-Id: I222be69d28a2c1c43a1ef1e063696bf25ad15c10
    Closes-Bug: #1511757
    (cherry picked from commit 36fd222432445620888068feed7fdee3b9f96926)

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

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

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

Reviewed: https://review.openstack.org/322952
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=24236dc0700aaf8db4e30f53d49068e478d1bdf9
Submitter: Jenkins
Branch: master

commit 24236dc0700aaf8db4e30f53d49068e478d1bdf9
Author: Dmitry Burmistrov <email address hidden>
Date: Mon May 30 21:00:45 2016 +0300

    [builder] Implement custom repo path

       Use CUSTOM_REPO_ID variable to override default path for CR
       repositories (missed builder part)

    Change-Id: Ia1acd83afcec456b5bec25dc6300daecc20c728c
    Related-Bug: #1511757

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

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/335917

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-mirror (stable/mitaka)

Reviewed: https://review.openstack.org/335917
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=5393be0fab3b8916b448479cbdc52cd9a5b5cbe0
Submitter: Jenkins
Branch: stable/mitaka

commit 5393be0fab3b8916b448479cbdc52cd9a5b5cbe0
Author: Dmitry Burmistrov <email address hidden>
Date: Mon May 30 21:00:45 2016 +0300

    [builder] Implement custom repo path

       Use CUSTOM_REPO_ID variable to override default path for CR
       repositories (missed builder part)

    Change-Id: Ia1acd83afcec456b5bec25dc6300daecc20c728c
    Related-Bug: #1511757
    (cherry picked from commit 24236dc0700aaf8db4e30f53d49068e478d1bdf9)

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-mirror (stable/8.0)

Reviewed: https://review.openstack.org/337700
Committed: https://git.openstack.org/cgit/openstack/fuel-mirror/commit/?id=952988707336ccc3af2537c08b66165693d8cee0
Submitter: Jenkins
Branch: stable/8.0

commit 952988707336ccc3af2537c08b66165693d8cee0
Author: Dmitry Burmistrov <email address hidden>
Date: Mon May 30 21:00:45 2016 +0300

    [builder] Implement custom repo path

       Use CUSTOM_REPO_ID variable to override default path for CR
       repositories (missed builder part)

    Change-Id: Ia1acd83afcec456b5bec25dc6300daecc20c728c
    Related-Bug: #1511757
    (cherry picked from commit 24236dc0700aaf8db4e30f53d49068e478d1bdf9)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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