Deployment log is broken

Bug #1570932 reported by Serg Melikyan on 2016-04-15
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Status tracked in 10.0.x
10.0.x
High
Stan Lagun
8.0.x
High
Stan Lagun
9.x
High
Stan Lagun

Bug Description

Upstream bug: https://bugs.launchpad.net/murano/+bug/1570933

Steps to Reproduce
1. Deploy Apache application

Expected results:
1. Open Environment -> Last Deployment Log
2. Log Contains:
Deployment scheduled
Creating VM for Apache Server.
Instance is created. Deploying Apache
Apache is installed.
Apache is available at ...
Deployment finished

Actual Result
1. Open Environment -> Last Deployment Log
2. Log Contains:
Deployment scheduled
Deployment finished

Additional information:

murano-engine.log contains following exception:

2016-04-15 00:16:45.528 909 DEBUG murano.common.server [-] Got report from orchestration engine:
{u'environment_id': u'199e907a12984d3b80bd68f6d7405dc9', u'level': u'info', u'text': u'Creating VM for Apache Server.', u'details': None, u'timestamp': u'2016-04-15T00:16:45Z', u'id': u'e93b3ca7-48e5-4c6c-92d0-2c32489479a5'} report_notification /usr/lib/python2.7/dist-packages/murano/common/server.py:175
2016-04-15 00:16:45.564 909 ERROR oslo_messaging.notify.dispatcher [-] Exception during message handling
2016-04-15 00:16:45.564 909 ERROR oslo_messaging.notify.dispatcher Traceback (most recent call last):
016-04-15 00:16:45.564 909 ERROR oslo_messaging.notify.dispatcher DBDataError: (_mysql_exceptions.OperationalError) (1292, "Incorrect datetime value: '2016-04-15 00:16:45+00:00' for column 'created' at row 1") [SQL: u'INSERT INTO status (created, updated, id, entity_id, entity, task_id, text, level, details) VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s)'] [parameters: (datetime.datetime(2016, 4, 15, 0, 16, 45, tzinfo=<iso8601.iso8601.Utc object at 0x7fcf5e567550>), datetime.datetime(2016, 4, 15, 0, 16, 45, 550656), '17e2d9415e844ecf820fc27bb3bd8b4a', 'e93b3ca7-48e5-4c6c-92d0-2c32489479a5', None, '82a12c20c76e47bb89930db9d6bf9051', 'Creating VM for MySql', 'info', None)]

summary: - Deployment log is not available in Murano
+ Deployment log is broken
description: updated
Changed in mos:
assignee: MOS Murano (mos-murano) → Stan Lagun (slagun)

Related fix proposed to branch: 9.0/mitaka
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/20342

Reviewed: https://review.fuel-infra.org/20342
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: ad7be0f55ec7244387b7fb26cc5ed974d0ce2a37
Author: Stan Lagun <email address hidden>
Date: Thu May 5 06:07:21 2016

Increase status report messages time resolution

We use DATETIME SQL type for created/updated columns
everywhere through the database. In SQL standard
DATETIME defaults to DATETIME(6) which is 6 digit
fraction second part. However MySQL defaults it
to DATETIME(0) for backward compatibility. In result
status messages timestamps get rounded to the
second boundary and if there are several status
messages were generated within a second
after table sort they might be shown in a different
order.

This commit adds MySQL migration that increases
resolution for the status table and adds subseconds
to the generated status repor timestamps.

Change-Id: I0ced904063679a81503c67d426d0161204b4831b
Related-Bug: #1570932

Serg Melikyan (smelikyan) wrote :

Fixed in upstream for 10.0 by: https://review.openstack.org/307391

Changed in mos:
status: Confirmed → Fix Committed

Related fix proposed to branch: 9.0/mitaka
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/20862

Dina Belova (dbelova) on 2016-05-25
tags: added: area-murano
Alex Kholkin (akholkin) wrote :

Verified on #390 ISO
9.0-mos.all

Reviewed: https://review.fuel-infra.org/20862
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 7eee6392e14b07de03a34a0db7959134971ccbc9
Author: Stan Lagun <email address hidden>
Date: Wed May 18 17:48:33 2016

Fixes downgrade step for DB migration #14

Migration #14 increases resolution for timestamp
columns in "status" table. However not all of them
were reverted for downgrade path.

Change-Id: Id9144fab69f8f44a2fe5ae06aa98e7bab0e71796
Related-Bug: #1570932

Related fix proposed to branch: mcp/newton
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/33515

Related fix proposed to branch: mcp/newton
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/33589

Related fix proposed to branch: 11.0/ocata
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/33896

Related fix proposed to branch: 11.0/ocata
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/34055

Related fix proposed to branch: mcp/ocata
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/34576

Related fix proposed to branch: mcp/ocata
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/34765

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34576
Reason: We don't need this patch anymore since upstream Murano already contains the fix.

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34055
Reason: 11.0/ocata is obsolete. We use mcp/ocata instead.

Change abandoned by Roman Podoliaka <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/33896
Reason: we do not need 11.0/ocata anymore - use mcp/ocata instead

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34765
Reason: The fix has been upstreamed and is not required for mcp/ocata.

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/newton
Review: https://review.fuel-infra.org/33589
Reason: The patch has been upstreamed and is not required for mcp/newton.

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/newton
Review: https://review.fuel-infra.org/33515
Reason: The patch has been upstreamed and is not required for mcp/newton.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers