yahoo-eng-team team mailing list archive

[OpenStack Infra <1776504@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/609839
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=184c84ae76b1e4dc6b9907caf0cfa0d65bf8ecc5
Submitter: Zuul
Branch:    master

commit 184c84ae76b1e4dc6b9907caf0cfa0d65bf8ecc5
Author: Morgan Fainberg <morgan.fainberg@xxxxxxxxx>
Date:   Thu Oct 11 15:16:02 2018 -0700

    Remove pre-flask legacy code
    
    This removes common.controller, common.extension, common.router, and
    common.wsgi. Relevant code from common.wsgi (used by AuthContext) was
    moved into keystone.server.flask.request_processing.middleware.auth_context.
    
    keystone.api.discovery now uses keystone.flask.base_url
    
    test_middleware and test_exception were modified to reflect the changes
    to the remaining code from keystone.common.wsgi
    
    keystone.common.authorization only holds a couple constants for auth
    work now.
    
    Routes is removed from requirements.txt
    
    Release-Note for migration to flask added.
    
    Change-Id: I81563b6a49c8f12ecade058a9483f3b6f070dc72
    Closes-Bug: #1776504


** Changed in: keystone
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Identity (keystone).
https://bugs.launchpad.net/bugs/1776504

Title:
  flaskification

Status in OpenStack Identity (keystone):
  Fix Released

Bug description:
  Moving keystone to Flask away from it's home-grown WSGI framework is a
  long-term plan. The major reasons for this is to ensure we have an
  easy way for folks start contributing.

  This will include a number of improvements including:

  * moving to flask-restful for API definitions

  * all routable paths will be owned by the base prefix (e.g.
  keystone.api.user will own everything under /user/)

  * Paste Deploy removed

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1776504/+subscriptions