summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Add WEBROOT to ngdetails navigationTatiana Ovchinnikova2021-01-043-2/+23
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | Default index URL for angular details page doesn't consider WEBROOT while building navigations. That is why refreshing details page or opening it in a new tab works perfectly when WEBROOT is "/" and fails for apache deployment where WEBROOT is "/dashboard/". This patch adds WEBROOT to index URL and builds a proper navigation. Closes-Bug: #1902821 Change-Id: Id609e5499cc8f53dbae36febc945568e45d291f3 (cherry picked from commit f7442851c4b85a6a091635f0930c07dbd43cf8fb)
* | Merge "Consume horizon tempest test from tempest master" into stable/victoriaZuul2021-02-011-4/+1
|\ \
| * | Consume horizon tempest test from tempest masterAkihiro Motoki2021-01-261-4/+1
| |/ | | | | | | | | | | | | | | | | | | | | As disscussed in Wallaby PTG[1], QA and Horizon team decided to move the horizon dashboard test from tempest-horizon to Tempest. As next step, we can remove the tempest-horizon plugin which will ease the maintaince of horizon tempest test. Depends-On: https://review.opendev.org/c/openstack/tempest/+/771727 Change-Id: I0dc58db29825446381c319ec904e8bf225b210e1 (cherry picked from commit 15cb035efb746a7e0747e0e3e3bcf6ed40ccaea1)
* | Fix full text search at containers pageTatiana Ovchinnikova2021-01-141-67/+73
|/ | | | | | | | | | | Containers page uses the standard Magic Search filtering, however its full text search option doesn't work. This patch makes the filtering work properly with full text search. Closes-Bug: #1902610 Change-Id: Ie784d187c89eba1a2673ecafd72e903115a264d3 (cherry picked from commit 40acf5b75c1a889e767cec32fff24456e3e245d0)
* Imported Translations from ZanataOpenStack Proposal Bot2020-12-2515-74/+35
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I1dc2e7ef286231c7dc8d9acb4cf83b8de58e950b
* Update lower constraints and change browser to Firefox to unblock gateIvan Kolodyazhny2020-12-164-7/+7
| | | | | | | | | | | | | | | | | | | | | Pip 20.x requires ensure_str method from six which was introduced in v.1.12.0 and wrapt v.1.11 is required by astroid v.2.4.0 with the pylint dependency. So this patch update six and wrapt version in lower-constraints.txt to fix the gate. It also updates the browser to Firefox in karma.conf.js and adds 'karma-firefox-launchers' in package.json because nodejs-job fail with chromium-browser due to not able to access some URL. It also removes 'karma-chrome-launcher' enteries from all files as it is no longer used. So let's switch to Firefox browser to unblock the gate then we can investigate later how to fix it with chromium-browser. Depends-On: https://review.opendev.org/c/zuul/zuul-jobs/+/766744 Co-Authored-By: Vishal Manchanda <manchandavishal143@gmail.com> Change-Id: I13e68d89f55407b1b3c64219969912700feef500 (cherry picked from commit 6f0200032d5fb26784c0ce1428ae0cad84ef058f)
* Merge "Fix create/update_port with python3" into stable/victoriaZuul2020-12-032-10/+17
|\
| * Fix create/update_port with python3Akihiro Motoki2020-11-252-10/+17
| | | | | | | | | | | | | | | | | | unescape_port_kwargs() in api/neutron.py changes a key of dict during the iteration and it is not allowed in python3. Change-Id: Ifc1b9a0191aacd32e8c8ecc082b6313f04003f63 Closes-Bug: #1900851 (cherry picked from commit f3927ffb3d5e8803115a436f27d15cf52f61d2e0)
* | Fix AttributeError in the project instance detail viewAkihiro Motoki2020-11-231-1/+1
|/ | | | | | | | | | The previous code does not handle a case where admin-specific attributes do not exist. This leads to AttributeError in the project instance detail page as these attributes are not included in the response from nova API. Change-Id: I4c2942960259782d47c6777394dec6888959e26c Closes-Bug: #1905024 (cherry picked from commit a20a275ef5cae1b38ead1c3191dd96a042eb2070)
* Imported Translations from ZanataOpenStack Proposal Bot2020-11-073-4/+135
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I2504f03270904a650752452bcf9fad3388e96f90
* Added validation for csrf_failure GET argumentIvan Kolodyazhny2020-10-163-7/+59
| | | | | | | | | During csrf_failure argument validation horizon drops unknown messages so nobody can't inject any message to login view. Change-Id: I78a7592562a6249629f4d236ca59eb83d9094123 Closes-Bug: #1898465 (cherry picked from commit 8a963626e12ee25cf2f9ab29c172b16f5bbce4c9)
* Imported Translations from ZanataOpenStack Proposal Bot2020-10-142-5/+29
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Ied2fa36d0079af7b09f15baaf70e92ed7ef39b95
* users: Correct lock_password status visibility (Yes/No) in user details18.6.1Walter Doekes2020-10-072-2/+2
| | | | | | | | | Before this change, it would always say No, regardless of actual setting. Change-Id: Ib30d818ecae7b8edd33e94ad2c3c0e2ad8e01822 Closes-Bug: #1897269 (cherry picked from commit 612942bdf7b3d31349aeb3c408df6d5ae7446cba)
* Merge "Imported Translations from Zanata" into stable/victoriaZuul2020-10-051-2/+40
|\
| * Imported Translations from ZanataOpenStack Proposal Bot2020-10-051-2/+40
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: If95f2d1ff8c53d3891c6a9b2b95d84383f315271
* | Merge "Update TOX_CONSTRAINTS_FILE for stable/victoria" into stable/victoriaZuul2020-10-051-2/+2
|\ \ | |/ |/|
| * Update TOX_CONSTRAINTS_FILE for stable/victoriaOpenStack Release Bot2020-09-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Update the URL to the upper-constraints file to point to the redirect rule on releases.openstack.org so that anyone working on this branch will switch to the correct upper-constraints list automatically when the requirements repository branches. Until the requirements repository has as stable/victoria branch, tests will continue to use the upper-constraints list on master. Change-Id: I569023014dc1a4bb9f400f0fce4d17cc31dce77f
* | Imported Translations from ZanataOpenStack Proposal Bot2020-10-042-5/+77
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I826bcce346bea790dfe9499dc713c20079171cc5
* | Imported Translations from ZanataOpenStack Proposal Bot2020-10-019-28651/+95
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I3467bde8d565e80e670ff1a1fcad4c2eaddb5221
* | Relnote on disabling stripping leading/trailing spaces in passwordAkihiro Motoki2020-09-291-0/+14
|/ | | | | | | | | | commit 6a07f5a5b4e2c0921967b8c47c8c9cfcd9a45b90 changed the password form to avoid dropping leading/trailing spaces in passwords. It is worth having a release note. Change-Id: I9009d616975acc1c84d0a150119167d424eb5dfb Related-Bug: #1861224 (cherry picked from commit 37ea4159f002213c5e378a7d925f51dc1c152f53)
* Update .gitreview for stable/victoriaOpenStack Release Bot2020-09-251-0/+1
| | | | Change-Id: If4399524c7e7687b146d324198aeb38e1660d6c2
* Merge "Add Unit Test for cinder-user-facing messages"18.6.0Zuul2020-09-253-1/+95
|\
| * Add Unit Test for cinder-user-facing messagesmanchandavishal2020-09-253-1/+95
| | | | | | | | | | | | | | | | | | | | | | | | This patch is a follow-up patch of 64db185c7942721d32308245b1527f2367dceb21 which adds unit tests for the change. Add message_level to cinder message test data. Add third cinder message to make tests more robust. Co-Authored-By: Akihiro Motoki <amotoki@gmail.com> Change-Id: I6be88fbacf998a2ba36fea5a776cd4215cdaba0f
* | Imported Translations from ZanataOpenStack Proposal Bot2020-09-253-8/+87
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I6b7ab556695f411616e0a20c801e2ffbfc674b60
* | Merge "Show all os-extended-server-attributes"Zuul2020-09-242-6/+68
|\ \
| * | Show all os-extended-server-attributesIvan Kolodyazhny2020-09-242-6/+68
| |/ | | | | | | | | | | | | | | | | | | Patch I0cfe9090e8263f983fa5f42f42616a26407be47a adds hypervisor hostname to the instance details view. This patch adds the rest of instance attributes allowed by 'os_compute_api:os-extended-server-attributes' policy. Change-Id: Id39ee14e3054422a96248f8cdd0a5bf07c27f2fc Closes-Bug: #1874273
* | Merge "Fix: Page crashes on instance confirm resize"Zuul2020-09-241-4/+16
|\ \
| * | Fix: Page crashes on instance confirm resizepedh2020-09-241-4/+16
| | | | | | | | | | | | | | | | | | | | | Add error handling to instance confirm/revert resize methods. Change-Id: I128049091f38e8db3c1524a5c4cb932f3e809714 Closes-Bug: #1882918
* | | Merge "Make identity tests work"Zuul2020-09-243-9/+18
|\ \ \
| * | | Make identity tests workTatiana Ovchinnikova2020-09-093-9/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Identity JS tests have no output expectations but Jasmine Spec Runner marks them 'passed' only adding "SPEC HAS NO EXPECTATIONS" into the tests names. This patch adds callbacks to make tests asynchronous and fix it. Partial-Bug: #1894127 Change-Id: I6c10c49b6a6aaee2fded8c3ef02f858208bc31b4
* | | | Merge "Set unique identifier for Django messages"Zuul2020-09-247-18/+38
|\ \ \ \
| * | | | Set unique identifier for Django messagesIvan Kolodyazhny2020-09-237-18/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Messages details template should have unique IDs to get collapse/expand working. Change-Id: Ia9ff81694fc7a84417ea2c998721475332a7e772 Closes: Bug #1895570
* | | | | Merge "Clarify the plan of OPENSTACK_NOVA_EXTENSIONS_BLACKLIST deprecation"Zuul2020-09-211-0/+22
|\ \ \ \ \
| * | | | | Clarify the plan of OPENSTACK_NOVA_EXTENSIONS_BLACKLIST deprecationAkihiro Motoki2020-09-101-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit clarify the plan on the removal of OPENSTACK_NOVA_EXTENSIONS_BLACKLIST setting. We received feedbacks that SimpleTenantUsage nova API feature can affect the nova API performance in larger deployments like [1]. Stopping the usage of SimpleTenantUsage in horizon is a good workaround and it was the main motivation we introduced this setting. Considering the situation that nova dropped the extension mechanism, I believe the right way is to introduce a new setting which controls the usage of SimpleTenantUsage rather than keeping the blacklist setting. The current deprecation notice does not explain the migration path. This commit clarifies the current plan. [1] https://review.opendev.org/#/c/743600/ Change-Id: I10fdbf23b56745b1be6497532530dd394c62ff29
* | | | | | Merge "Add allow_delete_snapshot_before_volume config option"Zuul2020-09-183-13/+55
|\ \ \ \ \ \
| * | | | | | Add allow_delete_snapshot_before_volume config optionOleksii Petrenko2020-09-173-13/+55
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds allow_delete_snapshot_before_volume config option, that controls whether to test only situation where first test deletes volume and then snapshot or also second situation, where first test deletes snapshot and then volume. Second situation is impossible with Ceph [1]. [1] https://docs.openstack.org/cinder/ussuri/drivers.html#rbddriver Co-Authored-By: Ivan Kolodyazhny <e0ne@e0ne.info> Change-Id: Ib422331892f077d78e3f2efbdd88abafc4c52b9a
* | | | | | Merge "Fix open redirect"Zuul2020-09-183-2/+42
|\ \ \ \ \ \
| * | | | | | Fix open redirectRadomir Dopieralski2020-09-153-2/+42
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure the "next" URL is in the same origin as Horizon before redirecting to it. Change-Id: I06b2bfc8e3638591615547780c3fa34b0abe19f6 Closes-bug: #1865026
* | | | | | Imported Translations from ZanataOpenStack Proposal Bot2020-09-174-10/+74
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I694ab7b6c1e0468604fee52b03ce95756102669c
* | | | | Merge "Refactor error messages."Zuul2020-09-165-5/+17
|\ \ \ \ \
| * | | | | Refactor error messages.manchandavishal2020-09-105-5/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we didn't pass the redirect parameter in exceptions.handle() then '_alert_message.html' template is used to displays the error messages. If details are empty in a case but it still shows the 'Details' href which is wrong and this patch adds a condition in that template to fix that issue. This patch also fix review comment of change-id 5081359295e5311ca725e80efa0bc4b2ec811a0e and define a constant for '\u2026' and use it in all files. Note: '\u2026' is used for unicode glyph representing '...'. Closes-Bug: #1893305 Partially-Implements blueprint refactor-error-messages Change-Id: If4409a1245c983fb51712c6c817c11e884916694 Co-Author: Ivan Kolodyazhny <e0ne@e0ne.info>
* | | | | | Merge "Make getPorts and setBreadcrumb tests work"Zuul2020-09-142-1/+4
|\ \ \ \ \ \
| * | | | | | Make getPorts and setBreadcrumb tests workTatiana Ovchinnikova2020-09-112-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds expectations to the tests and make them work. Partial-Bug: #1894127 Change-Id: Ic22033343c344a42148b0ec5a31bccb1e08478c6
* | | | | | | Merge "doc: Update default value of REST_API_REQUIRED_SETTINGS"Zuul2020-09-141-3/+4
|\ \ \ \ \ \ \
| * | | | | | | doc: Update default value of REST_API_REQUIRED_SETTINGSPierre Riteau2020-09-091-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9b4367c54f070711344c24339bf769b5e4f034e5
* | | | | | | | Merge "Fix missing comment in swift storage policy code"Zuul2020-09-141-0/+1
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix missing comment in swift storage policy codeTobias Urdin2020-09-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3a3cccc5c26d6fb6991dd22e5bbe0cd255cb5790
* | | | | | | | | Merge "Fix ironic-ui cross job"Zuul2020-09-142-0/+7
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix ironic-ui cross jobAkihiro Motoki2020-09-102-0/+7
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ironic-ui npm job uses phantomjs which requires libfontconfig1. libfontconfig1 is installed via bindep.txt in ironic-ui [1], but in case of horizon it is not a direct dependency of horizon. To handle the situation, this commit adds a playbook to install libfontconfig1 and use it in horizon-cross-ironic-ui-npm job only instead of adding libfontconfig1 to horizon bindep.txt. [1] https://review.opendev.org/#/c/744708/ Change-Id: I525ab42d1d5a16b982f6afdfae0b6544a6c7a057
* | | | | | | | | Merge "test: use safer way to override settings"Zuul2020-09-142-18/+20
|\ \ \ \ \ \ \ \ \