Error: Unknown argument error in WSME

Bug #1310674 reported by Devdatta Kulkarni
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Solum
Fix Released
High
Arati Mahimane

Bug Description

See also (parent issue): https://bugs.launchpad.net/solum/+bug/1309583

When we do a git push, the git webhook sends a POST to the trigger with a payload.

However, some of the payload fields cannot be recognized by wsme, which results in a Client side Unknown argument error.

To resolve this issue, we added the ignore_extra_args=True in the wsme decorator for the TriggerController’s post method.

However, this resulted in another error in wsme - ‘kw’ referenced before assignment.

This revealed a bug in wsme (the ‘kw’ variable is returned before it is assigned) The bug can be fixed by initializing the variable before it is returned.

This bug ticket is to track the repair of the defect in WSME.

Revision history for this message
Adrian Otto (aotto) wrote :

I do not understand. We opened a bug to track a bug?

Revision history for this message
Adrian Otto (aotto) wrote :

I edited the bug description to resolve the ambiguity.

description: updated
Adrian Otto (aotto)
Changed in solum:
importance: Undecided → High
milestone: none → 2014.1.2
Adrian Otto (aotto)
summary: - Fix a bug identified in wsme
+ Error: Unknown argument error in WSME
Changed in solum:
status: New → Triaged
Adrian Otto (aotto)
tags: added: git-trigger-wsme-bug
Revision history for this message
Arati Mahimane (arati-mahimane) wrote :

Added a bug in wsme: https://bugs.launchpad.net/wsme/+bug/1311742

Submitted a patch to fix the bug: https://review.openstack.org/#/c/89883/

Changed in solum:
assignee: nobody → Arati Mahimane (arati-mahimane)
Revision history for this message
Adrian Otto (aotto) wrote :

I updated the WSME bug as affecting both WSME and Solum, so it will also show up in our bug list.

Changed in solum:
status: Triaged → In Progress
Adrian Otto (aotto)
tags: added: errors
Revision history for this message
Arati Mahimane (arati-mahimane) wrote :

The fix got merged in wsme - https://review.openstack.org/#/c/89883/

Changed in solum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
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.