vrouter-agent config can be outdated on Windows

Bug #1806677 reported by Michał Cłapiński
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
In Progress
Undecided
Michał Cłapiński
OpenContrail
In Progress
Undecided
Michał Cłapiński

Bug Description

If vhost interface is recreated it can have a different ifname. We create agent config only once during deployment and that will cause agent to crash.

Changed in opencontrail:
milestone: none → r5.1.0
assignee: nobody → Michał Cłapiński (michal.clapinski)
status: New → In Progress
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/48006
Submitter: Micha? C?api?ski (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/48007
Submitter: Micha? C?api?ski (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/48032
Submitter: Micha? C?api?ski (<email address hidden>)

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

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

commit c2dd5fcd4d09a11828a8dcddc1e6c67452ecf520
Author: Michal Clapinski <email address hidden>
Date: Tue Dec 4 12:49:21 2018 +0100

Add entrypoint to vrouter-agent service on Windows

Introduce entrypoint for agent similar in design to that from
microservice deployment. For now it will only start agent,
actual features will be added in following changes.

Change-Id: I1fff384efe5636882680c0eb2dea0c5583fe3122
Partial-Bug: #1806677

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

Reviewed: https://review.opencontrail.org/48007
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/02ddf36be2b22cc60fd35117a5f38b0c9e593550
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 02ddf36be2b22cc60fd35117a5f38b0c9e593550
Author: Michał Cłapiński <email address hidden>
Date: Tue Dec 4 14:05:52 2018 +0100

Use agent's entrypoint on Windows

Start using agent's entrypoint to achieve similar behaviour
to Linux's microservices.

Change-Id: If99760bf0270be73a16197b62075bab7f1d0a023
Partial-Bug: #1806677
Depends-On: I1fff384efe5636882680c0eb2dea0c5583fe3122

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

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

commit 90e1e43e8fecba1f0b42c0a2b96499e51d6469f6
Author: Michal Clapinski <email address hidden>
Date: Wed Dec 5 13:51:01 2018 +0100

Make agent's entrypoint update agent's config on Windows

In future we will generate the whole config from scratch
as on Linux, but for now we only update the vhost's ifname.
It's the only field that can change upon restart.

Change-Id: Iec93bf6985295f9791b5a718c7b76cccaee7dacc
Depends-On: If99760bf0270be73a16197b62075bab7f1d0a023
Partial-Bug: #1806677

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :
Download full text (5.3 KiB)

Reviewed: https://review.opencontrail.org/49385
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/6d56f894b1547baddfc84059e0f6ab3cd34dec8e
Submitter: Vinay Vithal Mahuli (<email address hidden>)
Branch: R6.0-WIP

commit 6d56f894b1547baddfc84059e0f6ab3cd34dec8e
Author: Michael Henkel <email address hidden>
Date: Mon Dec 3 02:49:13 2018 +0100

Apply commits from master onto R6.0-WIP

ansible-deployer use correct InternalIP for kubelet
Closes-Bug: 1806299

Use agent's entrypoint on Windows
Start using agent's entrypoint to achieve similar behaviour
to Linux's microservices.
Partial-Bug: #1806677

Starting docker service before pull
Closes-Bug: #1807716

Closes-Bug: #1808241 - Protect against reprovision failures
VROUTER_NODES_LIST is not needed - causes reprovision un-necessarily
Also, Openstack might be in half-configured state if provision fails first time due to intermittent timeout
Here re-provision shouldn't fail due to missing endpoints from first run.

Support stats collector destination in ansible
Stat messages are written to socket file which will
be read by stats collector. The socket file to be
written to is specified in instances.yaml. The folder
holfing the socket file will be mounted by all the
containers. The user specified STATS_COLLECTOR_DESTINATION_PATH
in the instances.yaml.
Partial-Bug: 1794107

add ssl mounts for rabbitmq and cassandra
closes-jira-bug: JCB-218684

change configuration description for additional rabbitmq options
partial-jira-bug: JCB-218684

Handle domainsuffix for Windows nodes
1. Expose provider_config to install container roles
2. Use proper FQDN in POST to config node
Closes-Bug: #1810173

Add domainsuffix to Windows examples
Closes-Bug: #1810173

Change default openstack version from ocata to queens
Closes Jira Bug: JCB-218736

Use kolla playbook to generate self-signed certs
If generate_self_signed_certs is set to yes under kolla_config->kolla_globals
section of the instance.yml file, then generate self-signed certs to be used for
the openstack haproxy service. Note that this will be used only when
kolla_enable_tls_external is set to true.
If using already generated certs, then give the path to the certificates using
the kolla_external_fqdn_cert variable
Partial Jira Bug: JCB-218738

Making kafka/alarm-gen/snmp-collector/topology/zookeeper/cassandra/query-engine installation optional.
Three new roles are introduced.
analytics_alarm - If added, alarm-gen, kafka will be installed
analytics_snmp - If added, snmp-collector and topology will be installed
If we need to enable analytics_alarm role, then analytics_database_kafka role also needs to
be added in any of the deploying node in cluster.
All components will use config_database zookeeper except kafka.
Partial-Bug: #1784493

add dnsmasq to config-api pod
next commits will move device manager and dnsmasq to separate pod
partial-jira-bug: JCB-218801

add capabilities required for dnsmasq
partial-jira-bug: JCB-218801

Fix extracting hostname from Windows
Use `hostname` bash command instead of ansible_hostname
Closes Jira Bug: TFB-1370

Initial commit for Openstack Rocky support
Partial-Jira-Bug: JCB-218848

fix issues preventing to use...

Read more...

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :
Download full text (37.3 KiB)

Reviewed: https://review.opencontrail.org/49393
Committed: http://github.com/Juniper/contrail-controller/commit/77df3b58265b3fab414dfbc00e1ff39d19f0a99c
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R6.0-WIP

commit 77df3b58265b3fab414dfbc00e1ff39d19f0a99c
Author: Shivayogi Ugaji <email address hidden>
Date: Mon Nov 5 22:07:18 2018 -0800

Apply commits from master onto R6.0-WIP

db_resync_done lock is used to indicate the amqp thread to wait for resync to
complete. In this case, when we call SchemaTransformer.destroy_instance()
due to Casandra connection failure, this lock remains locked blocking
destroy_instance. destroy_instance calls _vnc_subscribe_callback to drain the
amqp queue which waits infinitely for db_resync_done lock to be released.
This fix releases db_resync_done lock so that destroy_instance doesnt get
blocked.
Closes-Bug: #1801474

[DM] Hitless image upgrade implementation
Closes-Bug: #1799322

Provisioner for the devicemanager node.
usage:
from /opt/contrail/utils
python provision_devicemgr_node.py --host_name aio --host_ip 10.87.82.2
--oper add --admin_user admin --admin_password contrail123 --admin_tenant_name
admin --openstack_ip 10.87.82.2 --api_server_ip 10.87.82.2
Closes-Bug: #1805303

CFM: Changes for onboarding L3PNF
- Add new platform SRX240
- Add L3PNF subnet is schema
- Add new namespace, VN and IPAM for L3PNF during brownfield onboarding
Closes-Bug: 1800701

Add entrypoint to vrouter-agent service on Windows
Introduce entrypoint for agent similar in design to that from
microservice deployment. For now it will only start agent,
actual features will be added in following changes.
Partial-Bug: #1806677

Check build dependencies for tbb, SimpleAmqpClient and rabbitmq
Closes-Bug: #1806719

Make agent's entrypoint update agent's config on Windows
In future we will generate the whole config from scratch
as on Linux, but for now we only update the vhost's ifname.
It's the only field that can change upon restart.
Partial-Bug: #1806677

bgp-peer selection support for bgpaas
1. Listener BgpRouterConfig is added for BgpRouter and ControlNodeZone
2. BgpRouterConfig builds BgpRouterTree and ControlNodeZoneTree
from IFMapNode
3. BGPaaS gets BgpRouter for configured ControlNodeZone from
BgpRouterConfig and Updates bgp-peer-ip and bgp-peer-port in
the flow.
4. Step 3 is followed for xmpp based peer-selection also.
5. BGPaaS sandesh is updated with primary_control_node_zone,
secondary_control_node_zone, bgp_peer_ip and bgp_peer_port
Partial-bug: #1775872

[DM] Inside-outside workflow - lag/mH
1. Change the exisiting business logic to adhere to the new data model for lag/mH workflow
2. Multi-vlan support
Partial-Bug: #1799329

Rework nodemgr before fixing ntp issue
- move windows/linux code to separate classes instead of same condition through the code
- simplify main.py
- remove copy duplication
Closes-Bug: 1800704

[fabric] Added playbook retry support to job manager
1) When playbook return retry_devices in the output, job manager will retry the playbooks against those devices
2) remove obsolete playbooks from 5.0
3) remove obsolete ansible roles from 5.0
4) added a warning log on missing loopback interface when ...

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.