Rename "devstackgatenativel3rc" and the associated gate job to a more appropriate one

Bug #1606942 reported by Numan Siddique
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-ovn
Fix Released
Undecided
Brad Behle

Bug Description

In order to use and test native OVN DHCP in networking-ovn CI, "devstackgatenativel3rc" is being used (The patch is not merged yet). It will be good if this file and the gate job is appropriately renamed.

Revision history for this message
Russell Bryant (russellb) wrote :

What name do you have in mind?

Revision history for this message
Numan Siddique (numansiddique) wrote :

I don't have any thing in specific. I raised the bug, so that we don't loose track of it :)

Revision history for this message
Richard Theis (rtheis) wrote :

How about changing "nativel3" to "nativeservices" for the parts and gate job?

Revision history for this message
Numan Siddique (numansiddique) wrote :

@Richard - That seems fine to me

Brad Behle (behle)
Changed in networking-ovn:
assignee: nobody → Brad Behle (behle)
Revision history for this message
Brad Behle (behle) wrote :

I'm planning on creating three patches for this:

1) networking-ovn: Duplicate the devstack/devstackgatenativel3rc file but call the new file devstack/devstackgatenativeservicesrc (duplicate it so the gate job isn't broken), and update the file comments with the new job name

2) project-config:
    jenkins/jobs/networking-ovn.yaml - update the job name and the reference to the duplicated file in patch one (making this patch dependent on patch 1)
    jenkins/jobs/projects.yaml - update the job name
    zuul/layout.yaml - update the job name

3) networking-ovn: Remove the devstack/devstackgatenativel3rc file since after patch 2 merges it isn't being used (making this patch dependent on patch 2)

Let me know if anyone sees a problem with this

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to networking-ovn (master)

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

Changed in networking-ovn:
status: New → In Progress
Revision history for this message
Richard Theis (rtheis) wrote :

Hi Brad, your plan sounds good to me.

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

Reviewed: https://review.openstack.org/352664
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=21fbaf3e730808354f7b380fb69a588005d9efdd
Submitter: Jenkins
Branch: master

commit 21fbaf3e730808354f7b380fb69a588005d9efdd
Author: Brad Behle <email address hidden>
Date: Mon Aug 8 20:40:12 2016 -0500

    Rename native-l3 rc file to nativeservices

    Renaming native-l3 rc file to nativeservices since the job is now
    also being used for OVN native dhcp, and will soon be used for
    other native OVN services as well. This patch just creates a new
    file with the new name, a future patch will delete the old file
    once no jobs use it.

    Change-Id: Ibf6a772d06f9b0a4e4f279f15fcfbca7c6c6a186
    Partial-Bug: #1606942

Revision history for this message
Richard Theis (rtheis) wrote :

The infra patch has been merged (https://review.openstack.org/#/c/352676/) along with the final networking-ovn patch to remove the unused file (https://review.openstack.org/#/c/361388/).

Changed in networking-ovn:
status: In Progress → Fix Released
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.