Extension should define urls in urls.py file

[Sylwester Brzeczkowski]

To reduce the mess in main extension file, we should move urls definitions (patterns + class) to urls.py

To do this we should introduce new method in BaseExtension class which will look for url patterns in 'urls.py' file and set the 'url' class variable.

The example of the mess: https://review.openstack.org/#/c/301945/16/nailgun/nailgun/extensions/network_manager/extension.py

[Evgeniy L (rustyrobot)]

>> will look for url patterns in 'urls.py' file and set the 'url' class variable.

I would not implement this kind of magic and would leave it up to the developer, to import module and explicitly assign it to extension class urls parameter.

But I would be interested to see what other engineers think about it.

[Ryan Moe (rmoe)]

I think importing the patterns from another file and assigning them explicitly is a good solution. Another thing to consider would be unifying the pattern format. Currently the format used in the extensions is a little more verbose than that used in core.