summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Disable form auto-complete on login form" into stable/wallabystable/wallabyZuul2023-05-121-2/+2
|\
| * Disable form auto-complete on login formDavid Hill2023-05-051-2/+2
| | | | | | | | | | | | | | | | Disable form auto-complete on login form with autocomplete="off" for the email/password field. Change-Id: Ie550eae36ece160d19186a404c0e688f2f6e785b (cherry picked from commit e4a13dbcde0df213f9b6e7d72f45c6610192204a)
* | Remove Description from Edit Image MetadataTatiana Ovchinnikova2023-05-051-0/+6
|/ | | | | | | | | | | | | | | Image description can be edited through Edit Image -> Image Detail, however it also appears as a property in Edit Image -> Metadata when the description is not empty. It's confusing to have it in two sections of one dialog form, and moreover, the Image Detail field always overwrites the Metadata field. This patch removes Description from Edit Metadata. Closes-Bug: #2017698 Change-Id: I041948b4b71f00894f03cc30cfa4592df4a100c3 (cherry picked from commit 283b38e4f10b14c61dff42936c80ae3987f017cf)
* Merge "Fix instance error with attribute cells." into stable/wallabyZuul2023-04-261-0/+10
|\
| * Fix instance error with attribute cells.Jan Jasek2023-04-241-0/+10
| | | | | | | | | | | | | | | | Selenium read data faster than the data are loaded. Added wait until element is visible. Change-Id: I2975ea5836a898d113a2baaab6a8f9bf8af8d742 (cherry picked from commit b14958df8bbf93c5c8623b6e367dfba607d795d1)
* | Fix delete image tests.Jan Jasek2023-04-252-0/+18
|/ | | | | | | | | | | | Add wait until checkbox is in active state. Fix edit_image_description_and_name test. Add wait until the name text box is filled by default name (is not empty). In the other case selenium write new name into text box faster than default name is written there so default name then rewrite the new name. Change-Id: I56803e19dd80180cc432ca872a28a7ee7f41290d (cherry picked from commit 01c4ab920d82ed7e84dab84d9c925cd476902f1e)
* Merge "Add setup and teardown to the volumes pagination tests" into ↵Zuul2023-03-041-0/+9
|\ | | | | | | stable/wallaby
| * Add setup and teardown to the volumes pagination testsRadomir Dopieralski2023-02-171-0/+9
| | | | | | | | | | | | | | For now just delete the old volumes at the start Change-Id: I013da554f58e1638178d3655459c9742fd7f85ba (cherry picked from commit 81edf08676f2651f4343498c23d93b000fca7127)
* | Integration test navigation machinery for Angular pagesTatiana Ovchinnikova2023-02-1610-236/+483
|/ | | | | | | | | | There's an implementation of navigation modules for pages written in Angular. Legacy tests for images page are rewritten to work with the new navigation. Change-Id: Ib8a40f0adb0a46c9d646c6796a4242988eefa6e3 (cherry picked from commit 4743c6c748be79c3a80a350d05aeed7bbe63c577)
* Speed up integration testsRadomir Dopieralski2023-02-081-6/+2
| | | | | | | | | | | | | | | We have a place in the form-handling code where we look for the blue form submit button, and if it's not found, we look for the red form submit button. It looks correct at first glance, however it takes 10 seconds to throw NoSuchElementException, because there is a delay that makes sure that the page loads fully before raising that exception. Replacing this with a selector that finds both the blue and the red buttons in one query speeds the whole integration tests suite by half an hour, because there is no more waiting on every delete confirmation dialog. Change-Id: Ie256118c0a84a8868393018b25b0aa049582a17d (cherry picked from commit 2e6eca294718deb7e82f50c071d332377c35085f)
* Merge "Get ports directly instead of via loop" into stable/wallabyZuul2023-02-012-39/+30
|\
| * Get ports directly instead of via loopNicolas Bock2022-12-072-39/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to get a list of available ports, the current implementation of the form code in `project/instances/attach_interface` loops through the available networks in the project and requests a list of ports for each via the Neutron API. This implementation is O(N) in time with a large prefactor (the time for a Neutron API call) where N is the number of networks. Instead of calling the Neutron API for each network this change uses one call to the Neutron API to get the list of ports on all networks and filters this list via a list comprehension. While this implementation is still O(N) the prefactor is significantly smaller. Includes some unit test fixes to workaround the fact that patch 6ac31e0 is not backported beyond Y. Closes-Bug: #1943639 Change-Id: I8fd32c3aad22d8ef7f57201f5144f6b2e357ef10 Signed-off-by: Nicolas Bock <nicolas.bock@canonical.com> (cherry picked from commit 9f5d659d160d24d359992dda22fecb0db6b6f805) (cherry picked from commit 1561fcce55371c6b2d93a31b83ca533dea82f746)
* | Pin tox<4 for stable branches testingmanchandavishal2023-01-131-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | horizon npm jobs start failing due to some recent changes in tox4. As discussed on open-discuss ML, it is decided to keep testing the stable branch with the tox version that they release with and not with the tox 4. This patch set the 'ensure_tox_version' to use 4, which will tell zuul to install tox<4 if it's not present. For more info. please refer [1]. [1] https://lists.openstack.org/pipermail/openstack-discuss/2023-January/031668.html Change-Id: I3652d87d817030bcd3855d26bfa41b8b7061d08e (cherry picked from commit 4cc580d94b1e579c307583d2f8eb2a5cb699b14f)
* | Fix flavor id auto generationTatiana Ovchinnikova2022-12-081-2/+2
|/ | | | | | | | | | | Currently the flavor creation form always check for the uniqueness of the UUID field, even when it is set to "auto". That means that if we create a flavor with UUID value of "auto", the check will fail. This patch disable the check when UUID is automatically generated. Change-Id: Ie31307d67cf3857e3bb80c124e92c81e0c6a2982 (cherry picked from commit af7e6c4f75299b9e47db059dc7ddbaf2e0b1a3fc)
* Merge "Fix preselection for launch Instance dialog" into stable/wallabyZuul2022-12-011-2/+11
|\
| * Fix preselection for launch Instance dialogTatiana Ovchinnikova2022-11-251-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | Launch instance from a volumes/volume snapshot/instance snapshot should open a dialog with preselected boot source. However the dialog shows Image as selected boot source in any case, discarding the preselection. This patch fixes it, explicitly changing the boot source according to the preselection. Change-Id: I1a5a46d7be12ed179474a6a74647d0872b27b492 (cherry picked from commit 452c37dd369d2fa4c0dac9da1aba53bf1da89a30)
* | Make readonly metadata properties unable to editTatiana Ovchinnikova2022-11-252-0/+10
|/ | | | | | | | | | | | | | Image metadata properties os_hash_algo and os_hash_value are readonly, and attempts to edit them fail with unclear message. This patch makes these fields readonly in update metadata form. Also, if the existing metadata property is null, Horizon should consider it optional too, in addition to empty field fix: https://review.opendev.org/c/openstack/horizon/+/812009 Change-Id: I892465ca4688fce9f7123682d02f11c92c7d2c5c (cherry picked from commit 892080ec0ff6174d5fd5de221ec4802bd21371a6)
* Fix app cred create without project_id for domain adminsRodrigo Barbieri2022-11-042-3/+58
| | | | | | | | | | | | | | Users with domain admin role that are not cloud admins are not able to get scoped context and create an application credential with project_id, so this change forces the scoped context in that particular case. Closes-bug: #1827120 Change-Id: I076a97a6f943ab74a2db8bc5179a7db194009db4 (cherry picked from commit 6eeaf9852478e25ff77c21117664ac126c5357a4) (cherry picked from commit 778a52e66aeab883b31db729e04944eeecfec6a2) (cherry picked from commit 13e821a079134a458b24593d9593e0e5318e6cd6) (cherry picked from commit 084ee8aaea703d99720c0cdc2751b6479dab3b2f)
* Fix success_url parameter issue for Edit Snapshotwallaby-em19.4.0manchandavishal2022-10-291-11/+1
| | | | | | | | | | | | | | The "success_url" param is used when updating the project snapshot [1] and it lacks sanitizing the input URL that allows an attacker to redirect the user to another website. This patch update 'Updateview' class to not use the "sucess_url" method. Closes-bug: #1982676 [1] https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/snapshots/views.py#L109 Change-Id: Ied142440965b1a722e7a4dd1be3b1be3b3e1644b (cherry picked from commit 79d139594290779b2f74ca894332aa7f2f7e4735)
* Merge "Fix flavor specification at instance overview page" into stable/wallabyZuul2022-10-282-5/+30
|\
| * Fix flavor specification at instance overview pageTatiana Ovchinnikova2022-10-062-5/+30
| | | | | | | | | | | | | | | | | | | | The details page uses flavor id to resolve instance flavor, relying on legacy Nova API v2.46. This patch adds the current Nova API option, resolving instance flavor by flavor name, so flavor specification at instance overview page is displayed properly. Change-Id: I1fe45063c9d1cdd8682998329d81f843d30f80b3 (cherry picked from commit d0e9e976f2769b289196d481b0928893b702b011)
* | Merge "Modified the image page to use CSS class" into stable/wallabyZuul2022-10-271-2/+1
|\ \
| * | Modified the image page to use CSS classWei Lingfei2022-09-291-2/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | Pages can not display normal when language is Simplified Chinese(zh-cn), a row in the table can display only one Chinese character. Remove "width: 100%" rule from the "word-wrap" class. The page should now display normally. Partial-Bug: #1936830 Change-Id: If72d49c78a9f83687785a1225151d819b85e88d2 (cherry picked from commit e9ca0016234632f6ad0fbd766c5829c587b6a951)
* | Merge "Don't try and update port security if its not changing" into ↵Zuul2022-10-273-4/+22
|\ \ | | | | | | | | | stable/wallaby
| * | Don't try and update port security if its not changingSam Morrison2022-09-243-4/+22
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default policy in neutron doesn't allow port security to change if network not owned by the user. To allow users to update other attributes of a port don't send port_security_enabled attribute to neutron unless it changes. If user tries to change port security on a port in a network not owned by them it will still error as it does now. Partial-Bug: #1841050 Change-Id: I301336103cabc3f1cab3ee72d7743385ff1a10d6 (cherry picked from commit d059b0bc40f04befc60657cd6504cc5711934b90)
* | zuul: Declare queue at top levelTakashi Kajinami2022-10-171-1/+1
|/ | | | | | | | | | | | | | Zuul deprecated declaring shared queues at a pipeline level with release 4.1.0[1]. This updates the job definition to use the top level declaration instead. See [2] for details. [1] https://zuul-ci.org/docs/zuul/latest/releasenotes.html#relnotes-4-1-0-deprecation-notes [2] http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028603.html Change-Id: Ie63dd3b161cb8fd9be89002027d699ce2c4a67a5 (cherry picked from commit 4212950a4a6b036f563a236cb005604e1c096f04)
* Imported Translations from Zanata19.3.0OpenStack Proposal Bot2022-06-294-8/+49
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I14ef48f665550971e45c1212f53bd69320384ba9
* Merge "Add default_availability_zone for VM creation" into stable/wallabyZuul2022-06-285-5/+107
|\
| * Add default_availability_zone for VM creationDavid Hill2021-06-185-5/+107
| | | | | | | | | | | | | | | | | | | | Add the LAUNCH_INSTANCE_DEFAULTS.default_availability_zone settings in VM creation dashboard in order to let administrator select a default availability zone instead of a random one when many are available. Co-Authored-By: Akihiro Motoki <amotoki@gmail.com> Change-Id: I297ff7f3c8e725c24c7f7687786c784f61f5d4e2 (cherry picked from commit fc8603a499fb3302aeb2ebef714420e0fdcbdfef)
* | Make existing empty metadata properties optionalTatiana Ovchinnikova2022-06-142-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | Editing of snapshot image or updating its metadata from Horizon is not allowed until existing properties with empty values are set or removed from metadata tree. They are allowed to be None, therefore Horizon should consider them optional too. Closes-Bug: #1945706 Change-Id: Idfec71d209430c7d069133a40ca1a7d340163783 (cherry picked from commit 4e0f5a5a85467ca83743e6e8126cd570f329afb9)
* | Merge "Add horizontal scrollbar to role dropdown" into stable/wallabyZuul2022-05-251-0/+2
|\ \
| * | Add horizontal scrollbar to role dropdownTatiana Ovchinnikova2021-09-211-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Manage members dialog has a dropdown which lists all the role names. If a role name is long enough, it pushes all the roles to the left and hides them partially. This patch adds horizontal scrollbar for such cases. Closes-Bug: #1926511 Change-Id: I1bbf3b23717f9924974920d6569e018b94e3dbe8 (cherry picked from commit 9b28ab402802e9848f5824b04d85cf4292f738d3)
* | | Merge "Fix Project Limits for resize instance dialog" into stable/wallabyZuul2022-04-191-20/+20
|\ \ \
| * | | Fix Project Limits for resize instance dialogTatiana Ovchinnikova2022-02-281-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resize instance dialog should use proper calls for quota and usage. This patch fixes that. Change-Id: Ied6fd4ad69b38a721b23700c98344e0edf14771b (cherry picked from commit e2bb7d833526e3ea6117cb2a5d58cbf90548c6da)
* | | | Change to a proper policy for Resume operationKeigo Noha2022-03-252-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, ToggleSuspend class checked 'os-rescue' policy for resuming an instance. To align to resume operation, this fix changes to 'os-suspend-server:resume'. Closes-Bug: #1963652 Change-Id: If6386ecdb81fb1f0d88dab447ee81c251b9857b7 (cherry picked from commit 29da801528a357449264e546182bd629f4654275) (cherry picked from commit db40c8ceb6c403bc94ac839f65c00805a5e4ff66)
* | | | Fix for "Resize instance" buttonThomas Goirand2022-03-095-22/+103
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, "Resize instance" widget is not working because it relies on legacy Nova API v2.46, obsoleted in Pike release. Proposed patch make Horizon use current Nova API (>=2.47). Note: It also cherry-picks a7956cd004 from the master branch which avoids the extra call of flavor_get in resize server form. Closes-Bug: #1940834 Co-Authored-By: Akihiro Motoki <amotoki@gmail.com> Change-Id: Id2f38acfc27cdf93cc4341422873e512aaff716a (cherry picked from commit 4f4e8800904dc98a696c5aebe3ffcc947e2deabc)
* | | Imported Translations from ZanataOpenStack Proposal Bot2021-11-211-8/+63
| | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I97bd92b2953fd9bf166845e650f060718a9143ef
* | | Merge "Escape unicode characters when setting logout_reason cookie" into ↵Zuul2021-10-184-11/+21
|\ \ \ | | | | | | | | | | | | stable/wallaby
| * | | Escape unicode characters when setting logout_reason cookieRadomir Dopieralski2021-09-224-11/+21
| |/ / | | | | | | | | | | | | | | | Change-Id: Ic61a3958461a4a939acc40d1039881e2d4c3a1cd Closes-bug: #1894801 (cherry picked from commit e68e23937341d03be7475d71903f31a61403c1e2)
* | | Imported Translations from ZanataOpenStack Proposal Bot2021-10-022-6/+28
| | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I57240345acec490fd48c8308085fea44136a6183
* | | Dropping lower constraints testing from stable branchesmanchandavishal2021-09-223-164/+0
|/ / | | | | | | | | | | | | | | | | | | | | This topic was discussed on the ML and QA team proposed to test lower constraints [1]. As decided in horizon weekly meeting in the past[2], stable branches will drop the lower-constraints job from CI. [1]http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019390.html [2]https://meetings.opendev.org/irclogs/%23openstack-meeting-alt/%23openstack-meeting-alt.2021-03-24.log.html#t2021-03-24T15:12:58 Change-Id: I3f4367817cdf04eacacba8b2ba2bbc06b09d3132
* | Merge "Fix Unable to use multiattach volume as boot for new server" into ↵Zuul2021-07-293-4/+5
|\ \ | | | | | | | | | stable/wallaby
| * | Fix Unable to use multiattach volume as boot for new servermanchandavishal2021-07-283-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we try to create a new server from a bootable volume that supports multiattach, it will fail to create with an error message that ``multiattach volumes are only supported starting with compute API version 2.60``. This patch fixes the issue. Closes-Bug: #1931440 Change-Id: Ic8330b947b1a733f70c3bdad8b3493f20a2f26fb (cherry picked from commit 64fe0abb653950c85d455dedd09ef42856c6b07b)
* | | Change with_data=False for swift_get_containerTobias Urdin2021-07-192-2/+3
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function is only used in the Container get method which retrieves details about a container. It's not used for listing data in the container even though with_data was True doing a GET instead of a HEAD on the container. This indirectly also fixes the Ceph bug [1] in the Swift API implementation that doesn't expose metadata correctly for a GET but does for a HEAD. [1] https://tracker.ceph.com/issues/47861 Change-Id: Iba4215b2e3a414d1794faa05be31bf69e0e48bec (cherry picked from commit c87f0e7a90d05f9355bd58165c25561dedae36d5)
* | Imported Translations from ZanataOpenStack Proposal Bot2021-06-252-10/+7
|/ | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I82e2fc10ee4cdd580c91bccfd7365ce9724e105e
* doc: Update our IRC server to OFTCAkihiro Motoki2021-06-041-1/+1
| | | | | Change-Id: I985ea0b19a96e5e3160b824efda20659d855c2dc (cherry picked from commit cdb08a1ad6b01693d30de9c27b809652af77ab27)
* Imported Translations from ZanataOpenStack Proposal Bot2021-05-256-83/+703
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Ia3bd62bc51c9f7ffb874adcd68be0aab9fb026ed
* Imported Translations from ZanataOpenStack Proposal Bot2021-04-205-55/+13
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I7930404ebd56bfdd1b2c949d857be0d740517bb5
* Imported Translations from ZanataOpenStack Proposal Bot2021-04-1516-28981/+487
| | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I869cc5fbb278e39ea4f2777a9660ddac60e9496a
* Merge "Use policy_rules for user role assignment and group tabs" into ↵19.2.0Zuul2021-04-091-8/+2
|\ | | | | | | stable/wallaby
View Lorry Controller Status