Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove dependency on novaclient list_extensions API
The novaclient list_extensions API binding was removed in the 16.0.0 release [1]. The ability to enable/disable extensions in nova has been deprecated since Liberty [2] and was removed in Newton [3]. For horizon this only matters for the OPENSTACK_NOVA_EXTENSIONS_BLACKLIST config setting and some javascript code used to compile panels based on enabled extensions. In order to work with novaclient 16.0.0+, this change removes the list_extensions usage since all extensions are enabled in nova and thus for horizon a nova extension is only not supported if it's in the configured blacklist. To continue supporting the javascript code which uses the getExtensions function, the extension names are hard-coded. Note that the method meant to test that code, _test_extension_list, was wrong but never ran because of the underscore prefix on the method name. That is fixed here. [1] https://review.opendev.org/686516/ [2] https://review.opendev.org/214592/ [3] https://review.opendev.org/351362/ Change-Id: Iebb1e78c718b931d632445e4de6d7a29ccb92be2 Closes-Bug: #1847959
- Loading branch information
Showing
5 changed files
with
124 additions
and
26 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters