Closed Bug 1007501 Opened 10 years ago Closed 7 years ago

Initial (v0) Deploy of Slaveloan tool

Categories

(Release Engineering :: General, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Callek, Assigned: Callek)

References

Details

(Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/1949] )

Attachments

(1 file)

To get slaveloan tool able to be iterated on by many humans, we need an initial skeleton deployed to do so.

This bug will handle that skeleton
Feedback welcome
Comment on attachment 8419184 [details] [diff] [review]
[relengapi] WIP 1

"Human" and "human" should be plural in the model.

Also, 'import *' is frowned upon by tools like pyflakes (and, I think, pep8) since it introduces arbitrary symbols into the namespace without any syntactic indication of *which* symbols.  Please change the * in __init__.py to list the model classes explicitly.

The POST to /admin/ should be a REST-style POST, rather than a browser form submission.  The differences are minor, but involve sending a JSON request in the POST body, rather than form-encoded, and flagging it with @apimethod.  You'll need to do a little jQuery work on the browser side to make that happen.  You can look at the tokenauth blueprint for an example of that.

Finally, please delete the dangling 'slaveloan/src' symlink
Attachment #8419184 - Flags: feedback+
No longer blocks: byebyebuildduty
Depends on: 1075080
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/1949]
Component: Tools → General
What was worked on in this bug is now dead/unmaintained code, I'm calling this wontfix.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: