gating is not enabled on non-master branches

Bug #1409762 reported by Darren Birkett on 2015-01-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-ansible
Medium
Jesse Pretorius
Icehouse
High
Jesse Pretorius
Juno
High
Jesse Pretorius

Bug Description

The lint and AIO checks are only currently enabled for reviews on the master branch. We need to (re)enable these for the release branches (juno and icehouse)

Reviewed: https://review.openstack.org/146489
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=c1c22b48a19cad2f7a1f7dcd7792b13101e8d44e
Submitter: Jenkins
Branch: juno

commit c1c22b48a19cad2f7a1f7dcd7792b13101e8d44e
Author: Darren Birkett <email address hidden>
Date: Mon Jan 12 15:10:48 2015 +0000

    sync gate scripts with master

    There has been work going on to get the AIO gate working in the master
    branch. In order to enable the AIO for reviews against the juno branch, we
    need to pull these changes in from master, prior to enabling the AIO gate
    check for the juno branch.

    Change-Id: Ia0e2d226e193804c89e656bc57938f64f88c39cc
    Closes-Bug: 1409762

Fix to openstack-infra proposed: https://review.openstack.org/148211

Changed in openstack-ansible:
importance: Undecided → Medium
assignee: nobody → Jesse Pretorius (jesse-pretorius)
status: New → In Progress
Changed in openstack-ansible:
status: In Progress → Fix Committed

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/148958
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/148897
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/148945
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/148947
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/148959
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/148898
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/148905
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/148937
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/148994
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/148661
Reason: This patch can't be added individually in this branch - it needs to be squashed with other patches which combined together get gating working.

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/149230
Reason: I made a mistake with the commit message.

Reviewed: https://review.openstack.org/149209
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=e01165bcc729ba151473dfe3199f913906e8d4b9
Submitter: Jenkins
Branch: juno

commit e01165bcc729ba151473dfe3199f913906e8d4b9
Author: Jesse Pretorius <email address hidden>
Date: Mon Jan 19 17:47:38 2015 +0000

    Squash all patches required to make gating work

    This is a merge of the following patches:
    62d2525 Fixed generator type assumption error
    1339640 Allow swift_vars to not be specified for swift_hosts
    735eb68 AIO to use https to access the frozen repo
    033df65 use archive.mariadb.org for to install mariadb package
    8b9cf6e Use reliable static mirror for mariadb apt repository
    b30f9d1 Add debootstrap package to LXC package list
    5c464e1 Add swift_all inventory to swift-common play
    a88bcb7 Add swift_all inventory to swift account/container/object plays

    Change-Id: I630e38b35a00fe547ec3aaa68604a5902be4f83e
    Closes-Bug: #1399387
    Closes-Bug: #1406323
    Closes-Bug: #1399387
    Closes-Bug: #1402582
    Closes-Bug: #1402594
    Closes-Bug: #1402617
    Closes-Bug: #1409762
    Closes-Bug: #1410743
    Closes-Bug: #1411331
    Closes-Bug: #1412521

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/149213
Reason: This patch was an investigation to figure out what needed to go into the branch in order to get the gate operational. Now that we know, we can cherry-pick the patches individually.

Reviewed: https://review.openstack.org/148994
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=8ef95408b0103687c55328c95b0f9571169962f5
Submitter: Jenkins
Branch: icehouse

commit 8ef95408b0103687c55328c95b0f9571169962f5
Author: Jesse Pretorius <email address hidden>
Date: Wed Jan 21 17:23:27 2015 +0000

    Remove empty playbooks/infrastructure/repo-install.yml

    Not only is the file empty, but it causes the lint test to fail as a
    result of being empty.

    Change-Id: I3392cb0bc341f773fb1f5995daf5749c4822fd9c
    Related-Bug: #1409762

Reviewed: https://review.openstack.org/148959
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=12b56a6d8bc0db643d1fa68daf01a046d8e48977
Submitter: Jenkins
Branch: icehouse

commit 12b56a6d8bc0db643d1fa68daf01a046d8e48977
Author: Kevin Carter <email address hidden>
Date: Tue Nov 18 17:27:50 2014 -0600

    Fixed generator type assumption error

    When adding the swift config pieces the dynamic inventory has attempts
    to add extra arguments to host _meta inventory which assumes the option
    is in a dictionary format. To address the issue an if block has been
    added which which checks the object type and proceeds accordingly.

    Closes-Bug: #1399387
    Related-Bug: #1409762
    Change-Id: I0203dc525ca9e8f962ac683ca07b5cb3bea41589
    (cherry picked from commit 1710987f0a1d7e170e1d8e441f65742c645fa398)

Reviewed: https://review.openstack.org/148958
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=40183d68cf10895e568056cc35a92119dd66c376
Submitter: Jenkins
Branch: juno

commit 40183d68cf10895e568056cc35a92119dd66c376
Author: Kevin Carter <email address hidden>
Date: Tue Nov 18 17:27:50 2014 -0600

    Fixed generator type assumption error

    When adding the swift config pieces the dynamic inventory has attempts
    to add extra arguments to host _meta inventory which assumes the option
    is in a dictionary format. To address the issue an if block has been
    added which which checks the object type and proceeds accordingly.

    Closes-Bug: #1399387
    Related-Bug: #1409762
    Change-Id: I0203dc525ca9e8f962ac683ca07b5cb3bea41589
    (cherry picked from commit 1710987f0a1d7e170e1d8e441f65742c645fa398)

Reviewed: https://review.openstack.org/148937
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=919deb837468e9047cd30deb0c7dd692950a92e7
Submitter: Jenkins
Branch: icehouse

commit 919deb837468e9047cd30deb0c7dd692950a92e7
Author: Jesse Pretorius <email address hidden>
Date: Wed Jan 21 14:13:08 2015 +0000

    Sync gate scripts with master

    There has been work going on to get the AIO gate working in the master
    branch. In order to enable the AIO for reviews against the icehouse
    branch we need to pull these changes in from master.

    The differences between this commit and master's version are:
    1) cloudserver-aio.sh uses the icehouse branch (instead of master)
    2) os-ansible-aio-check.sh has swift deployment disabled

    Change-Id: If015428ee83be8d32766507765e1c9e6fe0879d6
    Closes-Bug: #1409762

Reviewed: https://review.openstack.org/148898
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=23124164037b187fd1be671bcf548d27071b1be2
Submitter: Jenkins
Branch: icehouse

commit 23124164037b187fd1be671bcf548d27071b1be2
Author: Darren Birkett <email address hidden>
Date: Wed Jan 14 11:14:48 2015 +0000

    use archive.mariadb.org for to install mariadb package

    the current source repo for mariadb expires packages after 6 months.
    The official archive keeps all versions, so we switch to using that.

    Additionally, this commit removes unused references to this repo and key
    from vars/repo_packages/all_common.yml

    Closes-Bug: #1410743
    Related-Bug: #1409762
    Change-Id: Ic5de676422a6bde5d35ff5b1f6bf85037dcbcafc

Reviewed: https://review.openstack.org/148905
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=4d7096d6434d35936c620579cfee2b88632afa04
Submitter: Jenkins
Branch: icehouse

commit 4d7096d6434d35936c620579cfee2b88632afa04
Author: Jesse Pretorius <email address hidden>
Date: Mon Jan 19 17:47:38 2015 +0000

    Use reliable static mirror for mariadb apt repository

    This patch enables the use of the rackspace mirror of the mariadb
    archive. This is necessary as archive.mariadb.org is not always
    responding appropriately to apt requests.

    Change-Id: I9005c6f03fd88345028183f38c682c7734562b24
    Closes-Bug: #1412521
    Related-Bug: #1409762

Reviewed: https://review.openstack.org/148661
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=56f4ba03bd8196e9d504b0518f24a3f311036260
Submitter: Jenkins
Branch: icehouse

commit 56f4ba03bd8196e9d504b0518f24a3f311036260
Author: Ian Cordasco <email address hidden>
Date: Thu Jan 15 11:28:33 2015 -0600

    Use HTTPS when installing from mirror.rackspace.com

    SecurityImpact

    Closes-Bug: #1411331
    Related-Bug: #1409762

    Conflicts:
     requirements.txt
     rpc_deployment/inventory/group_vars/all.yml
     scripts/os-ansible-aio-check.sh

    Change-Id: I78716b737b37feab8b21d6e765a349e4e6724bf6
    (cherry picked from commit 28b6c2da9c4138b8f05b673174c7560d6c8efe07)

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/149230
Reason: mistaken commit

Reviewed: https://review.openstack.org/148945
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=1ae0fe55236e2df0ae0a16b9e705e065d9adb0b7
Submitter: Jenkins
Branch: juno

commit 1ae0fe55236e2df0ae0a16b9e705e065d9adb0b7
Author: Miguel Alex Cantu <email address hidden>
Date: Mon Dec 29 13:11:13 2014 -0600

    Adds conditional for swift-all plays

    This check to see if the DEPLOY_SWIFT variable is set to yes before
    running the swift-all plays.

    Change-Id: Ia65449d987cccac8a3086005a36eb24ef52fc603
    Closes-Bug: #1406323
    Related-Bug: #1409762
    (cherry picked from commit f45d9e6a463e88ba2535a06cd9380ac99dee4f77)

Reviewed: https://review.openstack.org/148897
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=0f8ef2ef646bb72dc7ab7155cb9e28eb327730f2
Submitter: Jenkins
Branch: juno

commit 0f8ef2ef646bb72dc7ab7155cb9e28eb327730f2
Author: Darren Birkett <email address hidden>
Date: Wed Jan 14 11:14:48 2015 +0000

    use archive.mariadb.org for to install mariadb package

    the current source repo for mariadb expires packages after 6 months.
    The official archive keeps all versions, so we switch to using that.

    Additionally, this commit removes unused references to this repo and key
    from vars/repo_packages/all_common.yml

    Conflicts:
            rpc_deployment/inventory/group_vars/all.yml

    Closes-Bug: #1410743
    Related-Bug: #1409762
    Change-Id: Ic5de676422a6bde5d35ff5b1f6bf85037dcbcafc
    (cherry picked from commit 1bc96069c2f28bf0ff3e1183b43dd3acaef206f8)

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: juno
Review: https://review.openstack.org/149230
Reason: never mind - borked

Reviewed: https://review.openstack.org/148947
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=c1dd592ad6fbb62bd37d239419626035102c470d
Submitter: Jenkins
Branch: juno

commit c1dd592ad6fbb62bd37d239419626035102c470d
Author: Jesse Pretorius <email address hidden>
Date: Wed Jan 21 14:46:37 2015 +0000

    Ensure that https is used to access the frozen repo for the AIO check

    Change-Id: Ib358f1d7f15dfec0fda626a898a197e99adb2be1
    Related-Bug: #1409762
    Related-Bug: #1411331

Changed in openstack-ansible:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers