Warning about dropped context argument in distributed mode

Bug #1412929 reported by Andrew Lazarev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Medium
Andrey Pavlov

Bug Description

Steps to reproduce:
1. Run sahara-api and sahara-engine
2. Start cluster

Observed warning line in the logs:
2015-01-20 11:14:29.332 24197 WARNING sahara.context [-] Arguments dropped when creating context: {u'instance_uuid': None}

Changed in sahara:
assignee: nobody → Andrew Lazarev (alazarev)
importance: Undecided → Low
milestone: none → kilo-2
status: New → Confirmed
importance: Low → Medium
assignee: Andrew Lazarev (alazarev) → nobody
tags: removed: low-hanging-fruit
Revision history for this message
Andrew Lazarev (alazarev) wrote :

Oh,

class Context(context.RequestContext):
    def to_dict(self):
        return {
            ...
            'instance_uuid': self.resource_uuid
        }

I believe it is very bad idea to rename fields in such manner.

Changed in sahara:
assignee: nobody → Andrey Pavlov (apavlov-n)
Changed in sahara:
milestone: kilo-2 → kilo-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sahara (master)

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

Changed in sahara:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara (master)

Reviewed: https://review.openstack.org/152494
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=42225e34edbd71da6a18054d7f7f88711cc8f36b
Submitter: Jenkins
Branch: master

commit 42225e34edbd71da6a18054d7f7f88711cc8f36b
Author: Andrey Pavlov <email address hidden>
Date: Tue Feb 3 14:26:11 2015 +0300

    Using oslo context as context-storage for logs

    After release of oslo_log there is no need to store
    context in local.store

    Change-Id: Iea95c04fb29a12bebe9123f0be6868f3327b0c72
    Closes-bug: #1412929

Changed in sahara:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in sahara:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in sahara:
milestone: kilo-3 → 2015.1.0
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.