Using a scsi decorated image with:
hw_disk_bus='scsi'
hw_scsi_model='virtio-scsi'

This solve the case where an instance is
launched with 'image' selected as boot
source with a new volume. This will result
in /dev/vda instead of /dev/sda as it
should.

Not specifying device name in
setFinalSpecBootImageToVolume() leaves the
decision to nova to determine it.

Example:
-------
VM boot volume without scsi meta data decoration:
Attached To
0a0cd660-7ce3-4033-9983-2e1099edc5f0 on /dev/vda

VM boot volume with scsi meta data decoration:
Attached To
91f50dbc-8bdf-4293-84ea-fc5df27b5ee4 on /dev/sda
--------

Note: This commit doesn't address cases for
where instances are launched using existing
volume and snapshot, this will involve more
work to migrate the code from BDMv1 to BDMv2.

Closes-Bug #1560965

Change-Id: I9d114c2c2e6736a8f1a8092afa568f930b656f09
(cherry picked from commit 79a4c3bd)

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I922e35dc6524e71bc0036d4d042e30fa16deb001

Previously d3 version of the network topology view handles ports with
device_owner 'compute:nova' and 'compute:None' specially. This leads
to the situtaion that neutron ports with non-default AZ are not shown
properly. There is no reason to handle neutron ports with the default
AZ differently. What we would like to do is just to classify neutron
ports attached to nova servers.

This commit changes the logic to check device_owner of ports have
a prefix 'compute:'. Note that we also need to check if device_owner
is a string before checking the prefix.

Change-Id: I472bae9af3d14d8a31efcae8a7610b84c4f09c3d
Closes-Bug: #1820260
(cherry picked from commit ffbe0e2f)

That function is buggy and can't handle the Chinese language codes.

Closes-Bug: #1818639
Change-Id: I6ea03949f3ad6db82f5965de3fa028a84edf6cf3
(cherry picked from commit 6372a689)

To verify resource usages when updating project quotas,
project_id query parameter needs to be supported in
tenant_absolute_limits in the cinder API.

This is supported in 3.39 or later in the cinder API.
API vesions shipped with released versions (pike, queens and
rocky) are selected as verified versions.

Change-Id: If5fc190988cf173387da2b66be23db9134310692
Closes-Bug: #1810309
(cherry picked from commit If5fc190988cf173387da2b66be23db9134310692)

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I176ec7706a04201b0ab9ac4a1b8e239a7fd8b876

The link in the consistency group table for "detail" is a mismatch
for the corresponding value in the urls.py file, causing cg snapshots
clicks to not load details for the user.

This bug was introduced in https://review.openstack.org/#/c/429506
where the link in tables.py was changed from:
"horizon:project:volumes:cg_snapshots:cg_snapshot_detail"
to
"horizon:project:cg_snapshots:detail"

the removal of the volumes section of the link was correct for that
change, but changing "cg_snapshot_detail" to "detail" broke the
details link

Change-Id: I7ad8673bad353a234596df16c8e64eec93733934
Closes-Bug: #1813990
(cherry picked from commit 468184bd)

This patch rollbacks changes in APITestCase introduced in the
0d163613 commit to unblock Horizon
plugins gates.

Conflicts:
        openstack_dashboard/test/helpers.py

Note(elod.illes): the above mentioned change in APITestCase was
backported to queens, so this rollback should be backported, too.

Change-Id: I6b2be31568e4aa4273990f9cfa76ca139620c84b
Closes-bug: #1806053
Closes-bug: #1805240
(cherry picked from commit de69edd0)
(cherry picked from commit 85fef71c)

- Removes an inappropriate condition for the device deletion button

Change-Id: I0a4c7dd5782524a875bf208d4ea63ac6df4a62b4
Closes-Bug: 1781911
(cherry picked from commit a42a56f4)

When uploading an image, if we do not set a minimum ram and disk size we
try to send "None" to glance.
We should instead send "0" and glance will calculate the size from the
provided image.

Change-Id: Ia1aeabcdcf021943ebb7a4e35939c383ac820b19
Closes-Bug: #1809121
(cherry picked from commit 65b21e7f)

The "request" attribute is not available in
openstack_auth.backend.KeystoneBackend.get_user when session data is restored
and it's the first request to happen after a server restart.

As stated by the function document, the "request" attribute needs to be
monkey-patched by openstack_auth.utils.patch_middleware_get_user
for this function to work properly.

This should happen in openstack_auth.urls at import time. But there is nowhere
in Horizon where this module is imported at startup. It's only introspected
by openstack_dashboard.urls due to AUTHENTICATION_URLS setting.

Without this monkey-patching, the whole authentication mechanism falls back
to "AnonymousUser" and you will get redirected to the login page due
to horizon.exceptions.NotAuthenticated being raised by
horizon.decorators.require_auth as request.user.is_authenticated will be False.

But if a user requests a page under auth/, it will have the side-effect of
monkey-patching django.contrib.auth.middleware as expected. This means that
once this request is completed, all following requests to pages other than
the ones under auth/ will have there sessions properly restored and
you will be properly authenticated.

Therefore this change introduces a dummy middleware which sole purpose is
to perform this monkey-patching as early as possible.

There is also some cleanup to get rid of the previous attempts at
monkeypatching.

Closes-bug: #1764622

[backport specific notice]
Queens horizon supports Django 1.8 and 1.11.
The key point is Django 2.0 is not supported.
Django 2.0 dropped the legacy Django middleware interface and
the new interface is not supported in Django 1.8.
Thus, this backport changes OpenstackAuthMonkeyPatchMiddleware in
openstack_auth/middlware.py to use the legacy middleware interface.

Conflicts:
	openstack_dashboard/settings.py
	openstack_dashboard/test/helpers.py

Change-Id: Ib9912090a87b716e7f5710f6f360b0df168ec2e3
(cherry picked from commit 0d163613)
(cherry picked from commit 8851866a)

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I0f2e0df82a81ec9e8fc345c1c33fdc815498790e

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I15c0be4bac815a3353d36ecddbb8c2114aa59ebb

* Fix release version name(latest->queens)
* Fix list number(trivial)

Change-Id: I72fc09451b44c9cb18b5eb1f72007242197ce36e

If enable_backup is False in OPENSTACK_CINDER_FEATURES
then we should not display the backups panel.

Change-Id: I276eebf0f11406bf354f5d8bbecef7b244d6d340
Closes-Bug: #1778771
(cherry picked from commit 6c2225ba)

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I82ec39c41f52d098a52e413c4f8d41c2c60673bf

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I7e00099dd5fed372bde770c424ee776fa1da9b85

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I2f9883c0e376c677815691652ca646fb492ad0ac

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: I5d2ade7b7b4a3ce2250e893b941ad81746c00299
Story: #2002586
Task: #24299

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I316b8a77d9168bd41b02d4dbcfd3e5a7e41e2d97

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I7c9a77a91a53a1130423937abc1f9d45a48361d2

Horizon don't have related change which align with the
change in https://review.openstack.org/#/c/330285

.
With this fix, horizon can work as expected

Change-Id: I5940c662a0bec2beaf4863e07f7244311ba51212
Closes-Bug: #1770326
Signed-off-by: GUO Larry <guo1017138@163.com>
(cherry picked from commit 4776428c)

Just like the flavor sort issue, this is another regression issue
when we shift from the legacy launch instance wizard. By default,
the image list should be ordered by name.

Change-Id: Id619ad531f7e91326694ee90c112d6a7db0d14ad
(cherry picked from commit 71c48804)

openstack-dashboard.conf in openstack-dashboard package
for RHEL/CentOS does not contain WSGIApplicationGroup configuration
which is required to make the dashboard work correctly.

Change-Id: Ic62fa29015ec153eada9d5e1ba5c06dab3a5eeca
Closes-Bug: #1741354
(cherry picked from commit a180bafb)

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: Ib7325b6d68921ed6ade53055e6e45be87d8add10

Bug #1641383 was not fixed for REST API call.
The two implementation of create_image_metadata are merged
into api/glance.py.

There were minor differences in field naming and handling
(min_ vs minimum_, is_public vs public vs visibility).
This commit normalizes them to simplify the logic.

Closes-Bug: #1757136
Change-Id: I41e9d94345151bc4473d704cdc040ed77870ce45
(cherry picked from commit e3626260)

The current REST API tests have a lot of rooms to be improved like:
- mocking the whole module of the API wrapper

---
[Note on a backport to queens]

This is a preparation of a backport proposed in [1].
mock usage of api.rest.test_glance was improved in Rocky and
the test backported in [1] assumes the improved version of the test.
This is a test improvement which does not touch the main code,
so it is a safe backport.

The signature of create_mocks was changed in [2], so the arguments
of create_mocks in this backport are adjusted to Queens version.

[1] https://review.openstack.org/#/c/566267/
[2] https://review.openstack.org/#/c/545472/

Change-Id: Id573d97c467fe8457f60baff089d60c18345395e
Partial-Bug: #1752917
(cherry picked from commit dfa68461)

Merge "Replace all mentions of get_identity_providers with get_identity_provider" into stable/queens

For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I0447c1e746a442bca22ac9274dc6f1e1430a799d

There was a typo in keystone's policy files, and it has been fixed in
Keystone already, we should also fix it to match.

Change-Id: I41e4381765f3bfc5988ca235e6cbeb6d1ba62fc2
Closes-bug: #1703369
(cherry picked from commit 93bb5718)