Juniper Openstack

draft firewall objects been pushed to agent

Bug #1771691 reported by Senthilnathan Murugappan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
Critical
Pramodh D'Souza
Juniper Openstack r5.0.1
Trunk
Fix Committed
Critical
Pramodh D'Souza
Juniper Openstack r5.1.0

Bug Description

Draft objects should be filtered by the control-node and shouldnt be pushed to the agent.
FYI, the original commit to ignore the drafts was made via https://review.opencontrail.org/#/c/41044/ which seems to be broken now.

msenthil-mbp:~ msenthil$ curl "http://10.87.74.130:8085/Snh_ShowIFMapAgentReq?table_name=&node_sub_string=draft&link_type_sub_string=&link_node_sub_string=" | xmllint --format -
<?xml version="1.0"?>
<?xml-stylesheet type="text/xsl" href="/universal_parse.xsl"?>
<ShowIFMapAgentResp type="sandesh">
  <table_data type="list" identifier="1">
    <list type="string" size="15">
      <element>
  type:address-group
   name:draft-policy-management:ctest-TestFirewallDraft_1-78880126-98706359

   draft-mode-state:created

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:11176500770961444195

     uuid-lslong:9430037126770745513
     Uuid : 9b1ae83c-4d09-4d63-82de-38cde7ea60a9

    enable:true

    created:2018-05-16T22:07:21

    last-modified:2018-05-16T22:07:21

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-98706359

Adjacencies:
 tag label=ag
 firewall-rule default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-33536440
 firewall-rule default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300
</element>
      <element>
  type:application-policy-set
   name:default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431

   draft-mode-state:created

   all-applications:false

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:12749880629021392902

     uuid-lslong:9557930504603351915
     Uuid : b0f0ace1-5d74-4006-84a4-972762c0bb6b

    enable:true

    created:2018-05-16T22:07:27

    last-modified:2018-05-16T22:07:27

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-50275431

Adjacencies:
 application-policy-set-firewall-policy attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)
 application-policy-set-firewall-policy attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431,draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508)
 tag default-domain:ctest-TestFirewallDraft_1-78880126:application=hr
 policy-management default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management
</element>
      <element>
  type:application-policy-set-firewall-policy
   name:attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)

   value
    sequence:20

Adjacencies:
 application-policy-set default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431
 firewall-policy default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397
</element>
      <element>
  type:application-policy-set-firewall-policy
   name:attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431,draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508)

   value
    sequence:30

Adjacencies:
 application-policy-set default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431
 firewall-policy draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508
</element>
      <element>
  type:firewall-policy
   name:default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397

   draft-mode-state:created

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:11976039415087449057

     uuid-lslong:13513037206489513293
     Uuid : a6337065-705d-43e1-bb87-f3be8171bd4d

    enable:true

    created:2018-05-16T22:07:26

    last-modified:2018-05-16T22:07:26

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-77233397

Adjacencies:
 application-policy-set-firewall-policy attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)
 firewall-policy-firewall-rule attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-33536440,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)
 firewall-policy-firewall-rule attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)
</element>
      <element>
  type:firewall-policy
   name:draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508

   draft-mode-state:created

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:14148363083224206658

     uuid-lslong:9245034414098185333
     Uuid : c4591543-50d4-4d42-804c-f5d13c957c75

    enable:true

    created:2018-05-16T22:07:26

    last-modified:2018-05-16T22:07:26

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-43770508

Adjacencies:
 application-policy-set-firewall-policy attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431,draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508)
 firewall-policy-firewall-rule attr(draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508,draft-policy-management:ctest-TestFirewallDraft_1-78880126-67254036)
</element>
      <element>
  type:firewall-policy-firewall-rule
   name:attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)

   value
    sequence:30

Adjacencies:
 firewall-policy default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397
 firewall-rule default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300
</element>
      <element>
  type:firewall-policy-firewall-rule
   name:attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-33536440,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)

   value
    sequence:40

Adjacencies:
 firewall-policy default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397
 firewall-rule default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-33536440
</element>
      <element>
  type:firewall-policy-firewall-rule
   name:attr(draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508,draft-policy-management:ctest-TestFirewallDraft_1-78880126-67254036)

   value
    sequence:20

Adjacencies:
 firewall-policy draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508
 firewall-rule draft-policy-management:ctest-TestFirewallDraft_1-78880126-67254036
</element>
      <element>
  type:firewall-rule
   name:default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300

   draft-mode-state:created

   action-list
    simple-action:pass

    mirror-to
     udp-port:0

     juniper-header:true

     static-nh-header
      vni:0

     nic-assisted-mirroring:false

     nic-assisted-mirroring-vlan:0

    log:false

    alert:false

   endpoint-1
    subnet
     ip-prefix-len:0

    tags:tier=web

    tag-ids:131334

    any:false

   endpoint-2
    subnet
     ip-prefix-len:0

    address-group:draft-policy-management:ctest-TestFirewallDraft_1-78880126-98706359

    any:false

   match-tags
    tag-list:deployment

   match-tag-types
    tag-type:3

   direction:&lt;&gt;

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:18406882394772030381

     uuid-lslong:12242370262651952067
     Uuid : ff726203-a2c7-43ad-a9e5-a2e41a2cdbc3

    enable:true

    created:2018-05-16T22:07:24

    last-modified:2018-05-16T22:07:24

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-02797300

Adjacencies:
 firewall-policy-firewall-rule attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)
 address-group draft-policy-management:ctest-TestFirewallDraft_1-78880126-98706359
 service-group default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-36316900
 tag default-domain:ctest-TestFirewallDraft_1-78880126:tier=web
</element>
      <element>
  type:firewall-rule
   name:default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-33536440

   draft-mode-state:created

   action-list
    simple-action:pass

    mirror-to
     udp-port:0

     juniper-header:true

     static-nh-header
      vni:0

     nic-assisted-mirroring:false

     nic-assisted-mirroring-vlan:0

    log:false

    alert:false

   service
    protocol:udp

    protocol-id:17

    src-ports
     start-port:0

     end-port:65535

    dst-ports
     start-port:8000

     end-port:8010

   endpoint-1
    subnet
     ip-prefix-len:0

    address-group:draft-policy-management:ctest-TestFirewallDraft_1-78880126-98706359

    any:false

   endpoint-2
    subnet
     ip-prefix-len:0

    tags:tier=db

    tag-ids:131338

    any:false

   match-tags
    tag-list:deployment

   match-tag-types
    tag-type:3

   direction:&lt;&gt;

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:1943059211804230369

     uuid-lslong:12472942233209114554
     Uuid : 1af7220e-f124-42e1-ad18-cae06a658fba

    enable:true

    created:2018-05-16T22:07:25

    last-modified:2018-05-16T22:07:25

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-33536440

Adjacencies:
 firewall-policy-firewall-rule attr(default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-33536440,default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-77233397)
 address-group draft-policy-management:ctest-TestFirewallDraft_1-78880126-98706359
 tag default-domain:ctest-TestFirewallDraft_1-78880126:tier=db
</element>
      <element>
  type:firewall-rule
   name:draft-policy-management:ctest-TestFirewallDraft_1-78880126-67254036

   draft-mode-state:created

   action-list
    simple-action:pass

    mirror-to
     udp-port:0

     juniper-header:true

     static-nh-header
      vni:0

     nic-assisted-mirroring:false

     nic-assisted-mirroring-vlan:0

    log:false

    alert:false

   endpoint-1
    subnet
     ip-prefix-len:0

    tags:global:site=blr

    tag-ids:262301

    any:false

   endpoint-2
    subnet
     ip-prefix-len:0

    tags:global:site=blr

    tag-ids:262301

    any:false

   match-tags
    tag-list:deployment

   match-tag-types
    tag-type:3

   direction:&lt;&gt;

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:6732227750186929838

     uuid-lslong:10323686590570670784
     Uuid : 5d6dad78-4ceb-4eae-8f45-1a41c36daac0

    enable:true

    created:2018-05-16T22:07:23

    last-modified:2018-05-16T22:07:23

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-67254036

Adjacencies:
 firewall-policy-firewall-rule attr(draft-policy-management:ctest-TestFirewallDraft_1-78880126-43770508,draft-policy-management:ctest-TestFirewallDraft_1-78880126-67254036)
 service-group draft-policy-management:ctest-TestFirewallDraft_1-78880126-09264388
 tag site=blr
</element>
      <element>
  type:policy-management
   name:default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:16747854295773236446

     uuid-lslong:13252669935028993859
     Uuid : e86c54bf-8190-4cde-b7ea-f1171ae6ff43

    enable:true

    created:2018-05-16T22:07:19

    last-modified:2018-05-16T22:07:19

    user-visible:true

   display-name:draft-policy-management

Adjacencies:
 application-policy-set default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-50275431
</element>
      <element>
  type:service-group
   name:default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-36316900

   draft-mode-state:created

   service-group-firewall-service-list
    firewall-service
     protocol:tcp

     protocol-id:6

     src-ports
      start-port:0

      end-port:65535

     dst-ports
      start-port:8000

      end-port:8010

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:4832821388003658840

     uuid-lslong:12794218310811680993
     Uuid : 4311a172-6ede-4c58-b18e-31c45aba80e1

    enable:true

    created:2018-05-16T22:07:22

    last-modified:2018-05-16T22:07:22

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-36316900

Adjacencies:
 firewall-rule default-domain:ctest-TestFirewallDraft_1-78880126:draft-policy-management:ctest-TestFirewallDraft_1-78880126-02797300
</element>
      <element>
  type:service-group
   name:draft-policy-management:ctest-TestFirewallDraft_1-78880126-09264388

   draft-mode-state:created

   service-group-firewall-service-list
    firewall-service
     protocol:icmp

     protocol-id:1

     src-ports
      start-port:0

      end-port:65535

     dst-ports
      start-port:0

      end-port:65535

   id-perms
    permissions
     owner:contrail-api

     owner-access:7

     group:admin

     group-access:7

     other-access:7

    uuid
     uuid-mslong:3221455382348189250

     uuid-lslong:12866248318197379142
     Uuid : 2cb4e89a-2ca2-4e42-b28e-18ad561f3046

    enable:true

    created:2018-05-16T22:07:23

    last-modified:2018-05-16T22:07:23

    user-visible:true

   display-name:ctest-TestFirewallDraft_1-78880126-09264388

Adjacencies:
 firewall-rule draft-policy-management:ctest-TestFirewallDraft_1-78880126-67254036
</element>
    </list>
  </table_data>
  <more type="bool" identifier="0">false</more>
</ShowIFMapAgentResp>

Senthilnathan Murugappan (msenthil)
tags: added: sanityblocker
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/43143
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43144
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/43156
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43157
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/43144
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/43156
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/43156
Committed: http://github.com/Juniper/contrail-controller/commit/fca19fb5d6cbaa9edfc224c70630bd746e988d27
Submitter: Rudra Rugge (<email address hidden>)
Branch: R5.0

commit fca19fb5d6cbaa9edfc224c70630bd746e988d27
Author: Pramodh D'Souza <email address hidden>
Date: Thu May 17 15:39:24 2018 -0700

Fix UT for Draft Mode Objects

Correctled property "draft-mode-object" to "draft_mode_object" as
it would appear when read from Cassandra.

Closes-Bug: 1771691
Depends-On: I9637d6dd133330ff1f26aac74e1bda977c4dfb78
Change-Id: I426a138b043ed07e3ee9978229b681ac89611bcb

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43144
Committed: http://github.com/Juniper/contrail-controller/commit/3d54d4f9f42a0159228f4b42b02602c787471a31
Submitter: Rudra Rugge (<email address hidden>)
Branch: master

commit 3d54d4f9f42a0159228f4b42b02602c787471a31
Author: Pramodh D'Souza <email address hidden>
Date: Thu May 17 15:39:24 2018 -0700

Fix UT for Draft Mode Objects

Correctled property "draft-mode-object" to "draft_mode_object" as
it would appear when read from Cassandra.

Closes-Bug: 1771691
Depends-On: I9637d6dd133330ff1f26aac74e1bda977c4dfb78
Change-Id: I426a138b043ed07e3ee9978229b681ac89611bcb

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43143
Committed: http://github.com/Juniper/contrail-common/commit/c67c0628dd99235ff6878cdd015e8f8b4b6a9f0c
Submitter: Rudra Rugge (<email address hidden>)
Branch: master

commit c67c0628dd99235ff6878cdd015e8f8b4b6a9f0c
Author: Pramodh D'Souza <email address hidden>
Date: Thu May 17 15:31:05 2018 -0700

Fix check for Draft Objects

The draft Mode object was being checked for "draft-mode-object" property
whereas it should have been checked for "draft_mode_object", the format
it comes in from the Cassandra DB. With this fix Draft Objects will be
ignored properly.

Change-Id: I9637d6dd133330ff1f26aac74e1bda977c4dfb78
Closes-Bug: 1771691

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43157
Committed: http://github.com/Juniper/contrail-common/commit/15b1ba74b3f7dd286d51ad19c402fdf4acfc766a
Submitter: Rudra Rugge (<email address hidden>)
Branch: R5.0

commit 15b1ba74b3f7dd286d51ad19c402fdf4acfc766a
Author: Pramodh D'Souza <email address hidden>
Date: Thu May 17 15:31:05 2018 -0700

Fix check for Draft Objects

The draft Mode object was being checked for "draft-mode-object" property
whereas it should have been checked for "draft_mode_object", the format
it comes in from the Cassandra DB. With this fix Draft Objects will be
ignored properly.

Change-Id: I9637d6dd133330ff1f26aac74e1bda977c4dfb78
Closes-Bug: 1771691

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/43550
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43551
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43552
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43550
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43551
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43552
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/43605
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43606
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43607
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/43605
Committed: http://github.com/Juniper/contrail-controller/commit/05e60b5b9ade65d9a41bf01c37e53d9f443a0b14
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 05e60b5b9ade65d9a41bf01c37e53d9f443a0b14
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 15:52:24 2018 -0700

Disable Draft Mode test temporarily

Temporarily disabling to a circular dependency with config client mgr
in a different repo.
Closes-Bug: 1771691

Change-Id: I65a23bdfd9c77e1e0533aebe31a1b00b9129d8be

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43550
Committed: http://github.com/Juniper/contrail-controller/commit/144f520f428b6d6c2e5e2f0577c84c1187654c90
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 144f520f428b6d6c2e5e2f0577c84c1187654c90
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 15:52:24 2018 -0700

Disable Draft Mode test temporarily

Temporarily disabling to a circular dependency with config client mgr
in a different repo.
Closes-Bug: 1771691

Change-Id: I65a23bdfd9c77e1e0533aebe31a1b00b9129d8be

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43606
Committed: http://github.com/Juniper/contrail-common/commit/4a27b4668d2d5ad5fa13cd857f30cc2d825cc770
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 4a27b4668d2d5ad5fa13cd857f30cc2d825cc770
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 15:58:57 2018 -0700

Check for correct draft mode string

Check for "prop:draft_mode_state" not "draft_mode_state"
Closes-Bug: 1771691
Depends-On: I65a23bdfd9c77e1e0533aebe31a1b00b9129d8be

Change-Id: Icc4038c54a23e296d0c03c73551b45d5398d01d8

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43551
Committed: http://github.com/Juniper/contrail-common/commit/594bbd5efd3310c1fb367b194cc656bb9c932911
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 594bbd5efd3310c1fb367b194cc656bb9c932911
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 15:58:57 2018 -0700

Check for correct draft mode string

Check for "prop:draft_mode_state" not "draft_mode_state"
Closes-Bug: 1771691
Depends-On: I65a23bdfd9c77e1e0533aebe31a1b00b9129d8be

Change-Id: Icc4038c54a23e296d0c03c73551b45d5398d01d8

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43607
Committed: http://github.com/Juniper/contrail-controller/commit/96bea37bddfae2f905fd33ffb618734db4afc5e0
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 96bea37bddfae2f905fd33ffb618734db4afc5e0
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 16:08:34 2018 -0700

Enable Draft Object test

Re-enable Draft Object test.
Closes-Bug: 1771691
Depends-On: Icc4038c54a23e296d0c03c73551b45d5398d01d8

Change-Id: I4b4d0827cf7620ba6c4d0e5a62e3af1f30cfc7c2

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/43552
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43811
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43552
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/43813
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43815
Submitter: Pramodh D'Souza (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/43811
Committed: http://github.com/Juniper/contrail-common/commit/673bc57014385eb11597984bf30d041e3e70a493
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 673bc57014385eb11597984bf30d041e3e70a493
Author: Pramodh D'Souza <email address hidden>
Date: Wed Jun 13 16:35:33 2018 -0700

Move draft mode prop to ignore list

Ignore draft_mode_state property.
Closes-Bug: 1771691

Change-Id: I3f499a9b073f62af0f87b9ad239bb669b12cd3d1

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43813
Committed: http://github.com/Juniper/contrail-common/commit/1a3f8dd3fdf4feabeb57d79da4ce19a86291d432
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 1a3f8dd3fdf4feabeb57d79da4ce19a86291d432
Author: Pramodh D'Souza <email address hidden>
Date: Wed Jun 13 16:35:33 2018 -0700

Move draft mode prop to ignore list

Ignore draft_mode_state property.
Closes-Bug: 1771691

Change-Id: I3f499a9b073f62af0f87b9ad239bb669b12cd3d1

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43552
Committed: http://github.com/Juniper/contrail-controller/commit/0461ba2124c69b06d60150a4a562dfd529668f49
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 0461ba2124c69b06d60150a4a562dfd529668f49
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 16:08:34 2018 -0700

Enable Draft Object test

Re-enable Draft Object test.
Closes-Bug: 1771691
Depends-On: I3f499a9b073f62af0f87b9ad239bb669b12cd3d1
Change-Id: I4b4d0827cf7620ba6c4d0e5a62e3af1f30cfc7c2

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/43815
Committed: http://github.com/Juniper/contrail-controller/commit/a23a50ca8d27e92ddd7a28efaa173956bc9e35df
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit a23a50ca8d27e92ddd7a28efaa173956bc9e35df
Author: Pramodh D'Souza <email address hidden>
Date: Tue Jun 5 16:08:34 2018 -0700

Re-Enable Draft Object test

Re-enable Draft Object test.
Closes-Bug: 1771691
Depends-On: I3f499a9b073f62af0f87b9ad239bb669b12cd3d1

Change-Id: Idef64315cdab2c949692141ec20fc51df3479b8d

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.