missing ntpdate package not installed in ansible deployment on ubuntu

Bug #1781092 reported by wenqing liang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
High
Leela Bharath Kumar Kassetti
Trunk
Fix Committed
High
Leela Bharath Kumar Kassetti

Bug Description

Missing ntpdate package was not installed in ansible deployment of ocata-5.0-136 on ubuntu 16.04.3 which failed.

Tags: ansible
Revision history for this message
Leela Bharath Kumar Kassetti (klbharat) wrote :

Installing ntp package also installs ntpdate in CentOS. This doesn't happen in case of Ubuntu.
We have to explicitly install ntpdate package

Revision history for this message
Leela Bharath Kumar Kassetti (klbharat) wrote :

In Ubuntu, ntp service is automatically started when ntp package is installed. This doesn't happen in case of CentOS

Revision history for this message
Ignatious Johnson Christopher (ijohnson-x) wrote : Change in REST API payload

Hi Team,

Go-api backend is changed as per the attached spec. the changes are available from build #139(has other bugs).build #141 will be usable.
So parent information is mandatory in the REST payload.

Backend infers parent information from parent_type/parent_uuid or parent_type/fq_name,
So all the resources should have either parent_type/parent_uuid or parent_type/fq_name in the /sync api payload.

Please modify your tools to send this information.

Parent of the resources used,
contrail-cluster -> global-system-config (earlier it was project) https://github.com/Juniper/contrail/blob/master/schemas/fabric/contrail_cluster.yml#L9
node -> global-system-config (earlier it was contrail-cluster) https://github.com/Juniper/contrail/blob/master/schemas/fabric/node.yml#L6

Let me know if you have queries,

Thanks,
Ignatious

Revision history for this message
Prachi Yadav (pryadav) wrote :

Hi,

Thanks Ignatious, I was in the process of updating the payloads and this documentation will be helpful.

Thanks,
Prachi

From: Ignatious Johnson <email address hidden>
Date: Thursday, July 12, 2018 at 4:16 PM
To: Soumil Kulkarni <email address hidden>, Leela Bharath Kumar Kassetti <email address hidden>, Prachi Yadav <email address hidden>, Prasanna Mucharikar <email address hidden>, Ram Yadav <email address hidden>
Cc: Ramprakash Ram Mohan <email address hidden>
Subject: Change in REST API payload

Hi Team,

Go-api backend is changed as per the attached spec. the changes are available from build #139(has other bugs).build #141 will be usable.
So parent information is mandatory in the REST payload.

Backend infers parent information from parent_type/parent_uuid or parent_type/fq_name,
So all the resources should have either parent_type/parent_uuid or parent_type/fq_name in the /sync api payload.

Please modify your tools to send this information.

Parent of the resources used,
contrail-cluster -> global-system-config (earlier it was project) https://github.com/Juniper/contrail/blob/master/schemas/fabric/contrail_cluster.yml#L9
node -> global-system-config (earlier it was contrail-cluster) https://github.com/Juniper/contrail/blob/master/schemas/fabric/node.yml#L6

Let me know if you have queries,

Thanks,
Ignatious

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

Review in progress for https://review.opencontrail.org/44578
Submitter: Leela Bharath Kumar Kassetti (<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/44579
Submitter: Leela Bharath Kumar Kassetti (<email address hidden>)

Revision history for this message
Ignatious Johnson Christopher (ijohnson-x) wrote : Re: Change in REST API payload

+ Nitish

From: Ignatious Johnson <<email address hidden><mailto:<email address hidden>>>
Date: Thursday, July 12, 2018 at 4:16 PM
To: Soumil Kulkarni <<email address hidden><mailto:<email address hidden>>>, Leela Bharath Kumar Kassetti <<email address hidden><mailto:<email address hidden>>>, Prachi Yadav <<email address hidden><mailto:<email address hidden>>>, Prasanna Mucharikar <<email address hidden><mailto:<email address hidden>>>, Ram Yadav <<email address hidden><mailto:<email address hidden>>>
Cc: Ramprakash Ram Mohan <<email address hidden><mailto:<email address hidden>>>
Subject: Change in REST API payload

Hi Team,

Go-api backend is changed as per the attached spec. the changes are available from build #139(has other bugs).build #141 will be usable.
So parent information is mandatory in the REST payload.

Backend infers parent information from parent_type/parent_uuid or parent_type/fq_name,
So all the resources should have either parent_type/parent_uuid or parent_type/fq_name in the /sync api payload.

Please modify your tools to send this information.

Parent of the resources used,
contrail-cluster -> global-system-config (earlier it was project) https://github.com/Juniper/contrail/blob/master/schemas/fabric/contrail_cluster.yml#L9
node -> global-system-config (earlier it was contrail-cluster) https://github.com/Juniper/contrail/blob/master/schemas/fabric/node.yml#L6

Let me know if you have queries,

Thanks,
Ignatious

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

Review in progress for https://review.opencontrail.org/44578
Submitter: Leela Bharath Kumar Kassetti (<email address hidden>)

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

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

commit 98bf9c28b701a7abe5916b354892fd64e6827abf
Author: Leela Bharath Kumar Kassetti <email address hidden>
Date: Thu Jul 12 16:06:11 2018 -0700

Fix ntp time sync issue on Ubuntu 16.04

Note:
- ntpdate package needs to be installed explicity on Ubuntu. In CentOS,
this it is automatically pulled when ntp package is installed.
- stop ntp service after installation. In Ubuntu, ntp service is automatically
started after installation.

patch 2: combined ntpdate and ntp package installation code

Change-Id: Ieaccbb969cd2b8ad6e930a620947a192be5aa619
Closes-Bug: #1781092

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

Review in progress for https://review.opencontrail.org/44579
Submitter: Leela Bharath Kumar Kassetti (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/44579
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/877905636c438fdfa79111f69b3cc57836871b6b
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 877905636c438fdfa79111f69b3cc57836871b6b
Author: Leela Bharath Kumar Kassetti <email address hidden>
Date: Thu Jul 12 16:06:11 2018 -0700

Fix ntp time sync issue on Ubuntu 16.04

Note:
- ntpdate package needs to be installed explicity on Ubuntu. In CentOS,
this it is automatically pulled when ntp package is installed.
- stop ntp service after installation. In Ubuntu, ntp service is automatically
started after installation.

patch 2: combined ntpdate and ntp package installation code

Change-Id: Ieaccbb969cd2b8ad6e930a620947a192be5aa619
Closes-Bug: #1781092
(cherry picked from commit 98bf9c28b701a7abe5916b354892fd64e6827abf)

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.