add-logs-to-workflow-resource

Bug #1498147 reported by Devdatta Kulkarni
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Solum
Fix Released
High
Vijendar Komalla

Bug Description

We want to move the logs resource from the assembly to workflow resource.

i.e., currently we have:
/v1/assemblies/<assembly-id>/logs

we want to change this to:
/v1/workflows/<workflow-id>/logs

The main changes are in following locations:

1) Move the userlog resources from AssemblyController to WorkflowController

2) Modify TenantLogger to use Workflow instead of Assembly. Specifically,
solum/solum/uploaders/tenant_logger currently takes in assembly as input. We can keep that, but in line 53 where we are calling uploadr we should pass the workflow.id corresponding to this assembly.

3) Modify SwiftUpload to use resource.id instead of resource.uuid. This is because the workflow object uses 'id' and not 'uuid'.

4) Modify solum/uploaders/common.py line 78 to use 'id' instead of 'uuid'.

Tags: solum-api
Changed in solum:
assignee: nobody → Vijendar Komalla (vijendar-komalla)
importance: Undecided → High
description: updated
Changed in solum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to solum (master)

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

tags: added: solum-api
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to solum (master)

Reviewed: https://review.openstack.org/237088
Committed: https://git.openstack.org/cgit/openstack/solum/commit/?id=6ac24ace3bbb43c24cafe9162ac1b5ea623ccf99
Submitter: Jenkins
Branch: master

commit 6ac24ace3bbb43c24cafe9162ac1b5ea623ccf99
Author: Vijendar Komalla <email address hidden>
Date: Mon Oct 19 11:49:57 2015 -0500

    Add logs to workflow resource

    This change is to move the logs from assembly resource to
    workflow resource.

    Change-Id: I501fab0985be48ac261feeb81b9eb453cd73debe
    Closes-Bug: #1498147

Changed in solum:
status: In Progress → Fix Committed
Changed in solum:
milestone: none → 2015.1.2
Changed in solum:
milestone: 2015.1.2 → 4.0.0
Changed in solum:
status: Fix Committed → 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.