summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fix typo in publicize_image policy name" into stable/ocataocata-eolstable/ocataZuul2020-04-242-1/+12
|\
| * Fix typo in publicize_image policy nameIvan Kolodyazhny2020-01-172-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | This patch is not cherry-picked from stable/train because if was fixed in a scope of a new feature implementation with https://review.opendev.org/#/c/602468/ commit. Change-Id: I519e15afc975e6da2afb9c72a05448541572bd10 Closes-Bug: 1859041 (cherry picked from commit fe61f2358a6e16ea462630747180b83337eb5b55) (cherry picked from commit ed23eb60d4674e7d50f9f13cc926d3e3bb4f1121) (cherry picked from commit e3f40725a4f16140f1c423bc9456b670b9bb76b9) (cherry picked from commit 7fbe9ab5cee0765f9913f765125a5423c35a8a1b)
* | Ensure tox registers failures properlyadrian-turjak2020-01-101-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Tox runs tests via unit_tests.sh which when not running a test subset will mean tox only checks for the final exit code. This means horizon test failues will not be seen by tox unless openstack_dashboard tests also fail. This change forces an exit code which !=0 if either test run fails. Change-Id: I8fbcb834d9817e7eea1b2c39fc4caab6004b981b (cherry picked from commit 280848c55ffeb8cfe948acd5f2ca234d2c1264f5)
* | operation_log: Log mask_fields only when exists in requestAkihiro Motoki2020-01-101-2/+2
|/ | | | | | | | | Previously mask_fields are always output even when they are not included in a user request. It is confusing. Change-Id: Ia207bfbc61246bc722a2ced544a8955f52025c0c Closes-Bug: #1675191 (cherry picked from commit d04f164ab2d2472e181226201a68feaaf1454450)
* Use python 2.7 as the default interpreter in toxIvan Kolodyazhny2020-01-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | Infra uses Python 3.5 of tox jobs. The pep8 job and py27djNNN jobs are now run with python3.5 while they are expected to run with python2.7 as of the release. The pep8 job fails on Python2-specific code in the openstack_dashboard/dashboards/admin/images/views.py module with 'long' type cast. This patch enforces us to use Python 2.7 for pep8 job. This probably could be fixed in the infra side but as a short term solution we can cope with this situation in the horizon repo side. [stable/ocata only] karma.conf.js used in the nodejs-npm-run-test job assumes python 2.7 is used when searching JS libraries, so we need to ensure python 2.7 is used for "npm" tox env. This change fixes the issue too. Co-Authored-By: Akihiro Motoki <amotoki@gmail.com> Change-Id: I25b0b00eb3d575f8a8c1705f987cfcc0336eccfb (cherry picked from commit f294fa93c24fd0ae64bc431a04c2385690b59084)
* Merge "Fix Angular errors in openstack_auth" into stable/ocataZuul2019-04-235-13/+38
|\
| * Fix Angular errors in openstack_authShu Muto2019-01-155-13/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Angular errors on login page are caused by loading ngdetails view. Although registration of URL for ngdetails is executed on top level of Horizon, ngdetails view is not needed to be loaded in login page, i.e. openstack_auth side. To fix this issue, this patch moves registration of URL for ngdetails into openstack_dashboard side. Conflicts: openstack_dashboard/test/urls.py openstack_dashboard/urls.py The above conflicts are caused by bp/django2-support which landed in Rocky, particully by https://review.openstack.org/#/c/527323/12. Another problem: ngdetails url doesn't have project prefix after Ocata release, pls see: https://github.com/openstack/horizon/blob/stable/ocata/ \ openstack_dashboard/static/app/core/images/images.service.js#L59 https://github.com/openstack/horizon/blob/stable/pike/ \ openstack_dashboard/static/app/core/images/images.service.js#L69 So the following simple changes need to be made in urls.py in addition to the primitive backport patches. -ngdetails_url = url(r'^ngdetails/', +ngdetails_url = url(r'^project/ngdetails/', Change-Id: Ib039417b4e666c2341f17ac05fd7723bc758816c Closes-Bug: #1754133 Closes-Bug: #1753557 (cherry picked from commit f494c6f2d475bfc936b78fda7ce11ed4d46c7224) (cherry picked from commit 6f6f46dc6d2f38719b114a80edabe2eb01b1ab06) Signed-off-by: Zhang Hua <joshua.zhang@canonical.com>
* | OpenDev Migration PatchOpenDev Sysadmins2019-04-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* | Merge "Enable to refresh ngdetails view" into stable/ocataZuul2019-03-205-0/+48
|\ \ | |/
| * Enable to refresh ngdetails viewShu Muto2019-01-155-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For now, refreshing ngdetails view by browser using F5 key or reload button, it causes 404 error from django. To fix this issue, this patch adds the url for '/ngdetails' into django. To reproduce navigations, each request for views.py requires 'dashboard' and 'panel' object. Otherwise django side returns 500 error. However the URL for request of details page (i.e. /ngdetails/) can not provide informations to set the proper navigation. So this patch set them to default dashboard and default panel. This patch focuses to prevent 500 error. So the issue for settings of proper navigation will be fixed by subsequent patch. Change-Id: Ib9f1fe48b3cdecff5ad56e68a5ba58a41cb35f38 Closes-Bug: #1681627 (cherry picked from commit 077163a03c9aea08efd56251e49d69eb1cc4d093) (cherry picked from commit a0fc40b4e85d3090e657cca9824de719b33a7ba2) Signed-off-by: Zhang Hua <joshua.zhang@canonical.com>
* | NaNJSONEncoder should be used in api "cinder/tenantabsolutelimits"zhangdebo19872019-03-062-3/+6
|/ | | | | | | | | | | In api "cinder/tenantabsolutelimits", result maybe contains Infinity values, but Infinity values are not supported by JSON standard, so NaNJSONEncoder is needed here. Change-Id: I750637e3214ad46a8b29e1ad565870cdcb827fe1 Closes-bug: #1714417 (cherry picked from commit de37fc1024a2e02cb9ec0f29a6dbd218474072df) (cherry picked from commit 2cda101cf9cea1684fa6be2b1929ac758708d0cb)
* do not flush selection on "old" boot sourcekairat_kushaev2018-08-312-2/+19
| | | | | | | | | | | | | | | | | This fixes an issue introduced by watchers: when page initialized images watchers called before select boot source watcher.it leads to sutiation when we specify selection from user input and flush it on updateBootSource. as a result page come to inconsistent state described in the bug. Perhaps, this requires big refactoring but in the current patch we do not flush selection if boot type was not changed. it allows to use image, snapshot, volume from context as ctrl.selection. Change-Id: I4b23071a2a9e2c02cc1f46713165e5cb5155894f Closes-Bug: #1710890 (cherry picked from commit 48862eb9b7f5789dee3362d9103416628c5d3e76)
* import zuul job settings from project-configNguyen Hai2018-08-211-0/+9
| | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I66a3fab4c6efd6f5ca2b23e7189fe8a37f5815e7 Story: #2002586 Task: #24299
* Fix logo in IE11/Edge browsersIvan Kolodyazhny2018-06-143-3/+3
| | | | | | | | | | IE11/Edge browsers doesn't scale SVG images correct if weight and height is specified directly in SVG. Also these browsers doesn't work well with max-height style. Change-Id: I3fe643dd5e0d47cb0e5a658c84d23a8750c217e7 Closes-Bug: #1715674 (cherry picked from commit 502cffb1f8d2f476cff682e6aba15f71ee49c3ef)
* operation_log: Mask more password fields by defaultAkihiro Motoki2018-02-171-1/+3
| | | | | | Change-Id: I69283a2b692d1fca93aad1d5ed26a29de4e0e4a9 Closes-Bug: #1744609 (cherry picked from commit 1941d34e5cecf33090e73665034a8196b220e690)
* Fix collectstatic in case of theme customizationRadomir Dopieralski2018-02-111-3/+5
| | | | | | | | | When doing theme customization with templates collectstatic failes with a traceback Change-Id: I0fdb80ddde1f73b657d445c75b55636d6f2e8d88 Closes-bug: #1744239 (cherry picked from commit dc0bce63a03e37e9f3f181930ba6d6b7c8b1346e)
* Merge "Redirect VM console dynamically" into stable/ocataZuul2018-02-015-30/+33
|\
| * Redirect VM console dynamicallyAkihiro Motoki2017-12-195-30/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously when a user accesses the network topology panel horizon resolves VM console URL when returning server data. This requires one API call to nova per VM, and this causes a scaling issue. Actually there is no need to resolves VM console URL when the network topology. A console URL corresponding to a requested VM can be resolved when a user actually accesses a console. This reduces the number of API calls drastically and addresses the scaling issue reported in the bug report. Change-Id: Icc667449e2988e6227012a6e899835a97ce0d738 Closes-Bug: #1723142 (cherry picked from commit 974f0418efdf171132747ded4a556df7b7af03d1)
* | Security group quota error handlingSimon Collins2018-01-311-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | When adding a new rule to a secruity group that would exceede the quota, the error message displays a quota error, rather than rule already exists error. It does this my first checking for an "OverQuotaClient" error, then if that except doesn't trigger it checks for any other conflict Change-Id: I8eaa0f00b25c8c3b75fef2bf46979f1f248c6896 Closes-Bug: #1699724 (cherry picked from commit 3929a8b7cd893f0b4c0ff6c20d5d1fd9077b4dd2)
* | Merge "Set PROJECT_DOMAIN_NAME in generated v3 openrc" into stable/ocataZuul2018-01-312-0/+7
|\ \
| * | Set PROJECT_DOMAIN_NAME in generated v3 openrcSam Morrison2018-01-152-0/+7
| |/ | | | | | | | | | | Change-Id: I97435d2137b5bd74cd9f8ebfb927e4e28a0dc00a Closes-bug: 1715525 (cherry picked from commit 2d2a562194d2d5792b91c156879c9494184cb3d2)
* | Zuul: Remove project nameJames E. Blair2018-01-261-1/+0
| | | | | | | | | | | | | | Zuul no longer requires the project-name for in-repo configuration. Omitting it makes forking or renaming projects easier. Change-Id: I6b56fc54600ec29d88fba2586ce817e7036467f6
* | Imported Translations from ZanataOpenStack Proposal Bot2018-01-201-3/+7
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I30295807bde731836df59b7c222f760dae00842d
* | Imported Translations from ZanataOpenStack Proposal Bot2018-01-192-9/+45
|/ | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: If749b92725925b7e228edd0aefa0b9f8590f5d86
* Merge "Fetch Cinder availability zones list for volume creation" into ↵Zuul2017-11-296-11/+76
|\ | | | | | | stable/ocata
| * Fetch Cinder availability zones list for volume creationIvan Kolodyazhny2017-11-026-11/+76
| | | | | | | | | | | | | | | | | | We have to fetch and show Cinder availability zones list during volume creation and volume creation from image. Change-Id: I1c8746870d94c183f5ef510c1ce09b3fc9c84220 Closes-Bug: #1721286 (cherry picked from commit b9972f73b008c48618909240665e80550e7fc3ed)
* | Imported Translations from ZanataOpenStack Proposal Bot2017-11-211-5/+6
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I23c81bac0970d9fea66a6b3d262bc4c6247190f1
* | Move django variants UT and selenium-headless to in-repo jobsAkihiro Motoki2017-11-121-0/+40
|/ | | | | | | | Note that horizon-openstack-tox-py27dj18 is unnecessary for Ocata because django 1.8 is the default Django version in Ocata. Change-Id: Ic040bb1bf3e6c177dd88f2304491b7e839555e76 (cherry picked from commit ee3e6127a14032c7d2b8f23971cdaedb8cc51fe0)
* Imported Translations from Zanataocata-em11.0.4OpenStack Proposal Bot2017-08-251-4/+14
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I6af59b662a85e2bee4f80d0dc4198eb2e458171e
* Imported Translations from ZanataOpenStack Proposal Bot2017-08-241-4/+68
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Ib82eb211f152d0f0cb10e7827948a9f73b3cbf9b
* Merge "Use request.COOKIES to activate the timezone" into stable/ocataJenkins2017-07-281-1/+1
|\
| * Use request.COOKIES to activate the timezonebhavani.cr2017-07-111-1/+1
| | | | | | | | | | | | | | | | | | | | This patch fixes the inconsistency in the instance creation time. Change-Id: Ic4afd9a03f0278883e0d00a2e075acaa4b4f8dc1 Co-Authored-By: bhavani <bhavani.r@nectechnologies.in> Closes-Bug: #1418031 (cherry picked from commit 29a3768379c5804891c1cd679b2bed3556588740)
* | Merge "Allow dropdowns to be in front of div.modal-footer" into stable/ocataJenkins2017-07-281-3/+0
|\ \
| * | Allow dropdowns to be in front of div.modal-footerEddie Ramirez2017-07-271-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes some oddities with the CSS causing select menus to be hidden in some circumstances. The side effect of this is that when the modal body is tall, the window now scrolls instead of the body. I'm not sure if this is actually bad, but its different. Change-Id: If890ada4c1c450962789db23e9b4cfc50fe77900 Closes-Bug: 1603496 (cherry picked from commit f40e9d438eb759d43e76e837d042740d89d622d8)
* | | Merge "Remove the extra Networks url in ports details page's breadcrumb" ↵Jenkins2017-07-271-1/+0
|\ \ \ | |/ / |/| | | | | into stable/ocata
| * | Remove the extra Networks url in ports details page's breadcrumbzhurong2017-07-101-1/+0
| |/ | | | | | | | | | | | | | | There are two Networks url in ports details page's breadcrumb, This patch remove the extra one. Change-Id: I8ce37da8f999eb18e925eecda22664490db2381a (cherry picked from commit c6f060a12a6d4952fef85ec58294c0014b10b6d1)
* | Imported Translations from ZanataOpenStack Proposal Bot2017-07-243-14/+55
| | | | | | | | | | | | | | For more information about this automatic import see: http://docs.openstack.org/developer/i18n/reviewing-translation-import.html Change-Id: I7b946684748f3a457620a1a337893829be23d4fe
* | Imported Translations from ZanataOpenStack Proposal Bot2017-07-201-14/+549
| | | | | | | | | | | | | | For more information about this automatic import see: http://docs.openstack.org/developer/i18n/reviewing-translation-import.html Change-Id: I87dab6d078434c1aa1e6adb174cce2fcff779f9d
* | Merge "Attribute "isServer" of images visibility filter should be true." ↵Jenkins2017-07-191-1/+1
|\ \ | | | | | | | | | into stable/ocata
| * | Attribute "isServer" of images visibility filter should be true.zhangdebo2017-07-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I found an interesting bug in images panel. In images panel, set filter to be "visibility: public", I got one image listed in the table, then I changed language from "en" to "zh_CN" and filter images again, I got none. In the tab of browser developer tool, I saw a correct response with one image data, but it was not appeared in the table. Later I found the attribute "isServer" of visibility filter is false, that caused the above bug. Change-Id: I130d07727fe74ce65aba72d3e63fdfe0c8721795 Closes-Bug: #1691683 (cherry picked from commit 3724066224adfe2a82f43bfdc255144b402c7236)
* | | Merge "There is problem in UpdateRow of Horizon page." into stable/ocataJenkins2017-07-191-1/+3
|\ \ \
| * | | There is problem in UpdateRow of Horizon page.Keiichi Hikita2017-07-151-1/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we do some resource action(e.g. create/update/delete ...), UpdateRow will be started. And then resource status will be updated automatically in front-end. But in case removing resource, number of resource that displayed on both above and bottom of table will not be updated correctly. I fixed update_footer_count function in horizon.tables.js. This function only updates number of resource in footer field. (tfoot span.table_count) But we also need to update header field. (thead span.table_count) So I add header update logic into this file. Change-Id: Idc4330fe779b62607ada39a9db3637de82d014f7 Closes-Bug: #1694207 (cherry picked from commit 5b2fe409edb6ffc3b03f28eab71aa6cb141a33d6)
* | | Merge "Add correct policy for some actions" into stable/ocataJenkins2017-07-191-1/+9
|\ \ \
| * | | Add correct policy for some actionsqiaomin2017-07-191-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch correct the wrong policy for add allowed address pair and add policy for delete allowed address pair. Closes-Bug: #1702804 Change-Id: Idb47a924fbf659459c6537f32dc2ad378e436255 (cherry picked from commit 95c0109f2a2dc318c27c35f92f3f948cdab7709c)
* | | | Merge "Only show the image visibility option if it's allowed" into stable/ocataJenkins2017-07-195-5/+27
|\ \ \ \ | |/ / / |/| | |
| * | | Only show the image visibility option if it's allowedYing Zuo2017-07-065-5/+27
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | The visibility field on create image modal and edit image modal should be shown if it's allowed based on the glance policy. Change-Id: I36f1b1c6e93c5ce06873f6a5f6106d8759fac6d0 Closes-bug: #1698490 (cherry picked from commit 932511031f6246af1b996e26e500fdd5d848462e)
* | | Merge "Fix an incorrect policy rule in Admin > Instances" into stable/ocataJenkins2017-07-181-1/+1
|\ \ \
| * | | Fix an incorrect policy rule in Admin > InstancesRob Cresswell2017-07-131-1/+1
| | |/ | |/| | | | | | | | | | | | | Change-Id: I765ae0c36d19c88138fbea9545a2ca4791377ffb Closes-Bug: #1703066 (cherry picked from commit 69c81b16e07b67ba0a0bc8e1f55049e7987c5b8c)
* | | Imported Translations from ZanataOpenStack Proposal Bot2017-07-142-7/+37
|/ / | | | | | | | | | | | | For more information about this automatic import see: http://docs.openstack.org/developer/i18n/reviewing-translation-import.html Change-Id: Ibee0129a9ae1152cb6b6db206dbb031e85c7646e
* | Imported Translations from ZanataOpenStack Proposal Bot2017-07-121-4/+4
|/ | | | | | | For more information about this automatic import see: http://docs.openstack.org/developer/i18n/reviewing-translation-import.html Change-Id: I35f93bd26cb0d7581a5f5ad2873db3ceb4c497db