| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
| |
bandit 1.7.5 introduced the new lint rule with which current heat code
is not compliant. This pins bandit so that any new lint rule does not
affect pep8 job in stable branches.
Change-Id: Ie552e6398c10797e4df4afe757760c48ca7f80af
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
python-novaclient 18.3.0 replaced usage of kwargs by explicit arguments
and that broke some assertions of arguments.
story: 2010638
task: 47613
Change-Id: I039d1e672611b7b523ce61df072c60d161851eed
(cherry picked from commit 513dfd116e5177cb43a0fb78471ad1992f16210d)
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The new release of bandit just came out which introduced the new rule,
with which unfortunately heat is not compliant now.
This disables the rule temporally so that we can avoid adapting our
stable branches to new lint rules by backporting this.
Change-Id: I4a27049d40760ad8da6fda24e1195e26ef9c6c17
(cherry picked from commit 4fcbe943385d01b2e9c34db2a7c5738edb4c169d)
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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/2023.1 branch, tests will
continue to use the upper-constraints list on master.
Change-Id: I253d07729777dac493eace5f23eee7b30a378dd5
|
| |/
|
|
| |
Change-Id: I47e53e344572a9a67ce1f324694779fb28a2e94c
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We don't map the CircularDependencyException to
a 400 Bad Request in the fault middleware so users
can never get an understand of what is wrong in
their template. They are today instead present
with a 500 internal server error without details
which doesn't help them.
Change-Id: Idc8713b4c29e2534dd7059e4566ac3b777f418e5
|
| |\ \
| |/ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Moves the exception from inside the engine code
to the common code so that we can use it in the
API fault middleware.
Change-Id: I017b95153c358829501f6a5740918cdb005fb32f
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
story: 2010578
task: 47331
Change-Id: I3706d662ec56e3548eb991556be7bd0f82f49e05
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
depending on compute API version, flavor in the instance representation
may not have "id" field, but "original_name" instead.
Story: 2009753
Task: 44211
Change-Id: Ieeae2f36f5e59e01f57a2f8e66a983ec3d99ec63
|
| |\ \ \
| |/ / |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This test is very frequently failing now. The test case creates a stack
with an instance but the stack can't be deleted within timeout because
of the known libvirt bug in Ubuntu Jammy. Because the release deadline
is already over, we'll disable this test so that we can merge some
changes now. We'll revisit this (and the other test cases we disabled)
later.
Related-Bug: #1998274
Story: 2010631
Task: 47589
Change-Id: I87c4b1e7a911fd78e327393b1af200667e89c999
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
A unit test for verifying that hidden parameters should
be masked in the get stack environment API
Story: 2010555
Task: 47229
Change-Id: I44ef26849ebb450a5643907a0ef47166cfd9b1cb
|
| |\ \ \ \
| |_|_|/
|/| | | |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
The previous content is not very accurate because the fix is made in
API layer instead of the CLI command.
Change-Id: I0abbd535aacc03446ada0fa806dfdfdaa4522afe
|
| | |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Heat upgrade script set the env var TOX_CONSTRAINTS_FILE/UPPER_CONSTRAINTS_FILE
which are used to use the constraints during Tempest virtual env
installation. Those env var are set to non-master constraint when
we need to use non-master constraints but when we need to use the
master constraints we do not set/reset them point to master constraints.
This create the issue when this grenade script install the tempest with
stable constraints but with master Tempest and as there is mismatch of
constraints now with fasteners of stable branches. Below is failure
of heat greande job on stable/yoga
- https://zuul.opendev.org/t/openstack/build/3aaec4d59bb84068bb4d4428ea747cbd/log/controller/logs/grenade.sh_log.txt#3245
Similar way, role should set stable constraints only for the stable EM branch
not for all otherwise it fail when constraints/requirements are bumped.
- https://zuul.opendev.org/t/openstack/build/74f86b8097f44c35acaffdcfe41d9693
We should set/reset those constraint env var to master constraints if configuration
tell to use the master constraints.
Closes-Bug: #2003993
Change-Id: I024cd134577338fc1075e7742df7f006dc914646
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Version 5.5.0 of python-barbicanclient adds microversion support,
which causes it to call keystoneauth1.discover.get_version_data
to get the microversion information from the server. Adding
additional mock.patch for keystoneauth1.discover fixes the
breaking unit tests.
This is a gate fix for the python-barbicanclient upper-constraints bump[1].
[1] https://review.opendev.org/c/openstack/requirements/+/873906
Co-Authored-By: Grzegorz Grasza <xek@redhat.com>
Change-Id: Ic1c26d255ff0690e327b3cb6c622d3617e02df5f
|
| |
|
|
|
|
|
|
|
|
| |
Fix an assert statement
in heat/tests/aws/test_eip.py.
Change-Id: I3ae3e8d3132f8e100067fb86460b7c3f3ae35d07
Story: 2010265
Task: 46164
Signed-off-by: Takashi Natsume <takanattie@gmail.com>
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Using the healthcheck middleware as a filter is deprecated and
the middleware should be used as an application[1].
[1] 6feaa13610c450c8486f969703768db5319b4846
This change updates definition and usage of the healthcheck middleware
accordingly to avoid the following deprecation warning.
DeprecationWarning: Using function/method 'Healthcheck.factory()' is
deprecated: The healthcheck middleware must now be configured as
an application, not as a filter.
This also refactors composite definitions based on flavor by the new
pipeline factory.
Story: 2009071
Task: 42881
Change-Id: I75386dc4a7dc14b3c753dfff01f147ef8233bf94
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Related-Bug: #1999665
Story: 2010484
Task: 47052
Change-Id: Ifc51ff6a4deab05002ccded59383416f9a586aa0
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use the new API that is consistent with
the existing API instead of instantiating the client
class directly.
This was introduced in release 14.1.0 here [1] and
added into oslo.messaging here [2]
[1] https://review.opendev.org/c/openstack/requirements/+/869340
[2] https://review.opendev.org/c/openstack/oslo.messaging/+/862419
Change-Id: Ia48de36e3cb97378785bb4946b1a387c0da1ff81
|
| |\ \ \ \ |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This fixes the issue when a nested stack is serialized in
oslo.messaging and deserialization fails because we pass
a kwarg as an arg when we generate the exception remotely.
Story: #2010115
Task: #45695
Change-Id: Id75398d2ed2d4fab467df51057c4167cd77bb32f
|
| |\ \ \ \ \
| |_|/ / /
|/| | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This has been deprecated since SQLAlchemy 1.4.
Change-Id: Id3edcc188bb44d9752fe1bd7625308c5abed4d9c
|
| |\ \ \ \ \ |
|
| | | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: Ie12c2df93f846629a7c8b7c6415cebc31eaba1bc
Story: #2010485
Task: 47053
|
| |\ \ \ \ \
| |_|_|/ /
|/| | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is an automatically generated patch to ensure unit testing
is in place for all the of the tested runtimes for antelope. Also,
updating the template name to generic one.
See also the PTI in governance [1].
[1]: https://governance.openstack.org/tc/reference/project-testing-interface.html
Change-Id: I1df5bb05ea5ba40bea88ae0fe4cd8e8221429579
|
| | |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This test case is frequently failing because of a known libvirt issue
in Ubuntu Jammy. We already disabled one functional test case, but will
disable this test case as well, to reduce failure rate of CI runs and
unblock gate.
The existing skip for a different test case[1] is re-implemented using
the proper configuration knob.
This also fixes tox.ini to adapt to new tox 4.0.
- Update how passenv is defined because space-separated list is no
longer allowed. Also the values are not case sensitive.
- skipdist=True breaks installation so is removed.
[1] https://review.opendev.org/c/openstack/heat/+/866545
Co-Authored-By: Rabi Mishra <ramishra@redhat.com>
Related-Bug: #1998274
Story: 2010487
Task: 47056
Change-Id: I915dc83ccde6b6b8497642857292f9974fd84e98
|
| |\ \ \ \ |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Currently this test consistently fails because detaching an interface
does not progress, which is likely to be a libvirt issue.
This change disables the test case to temporally unblock CI.
Related-Bug: #1998274
Story: 2010458
Task: 46978
Change-Id: I7d9a1b2239a8406d12f3482062f4efd1d3e55d78
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
If the auth_encryption_key changes We can possibly
ignore these errors when deleting stacks.
Task: 42056
Change-Id: I326e415db194a5b9c67acd038d7d2d993293ecb3
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We only need the resource types for the check. Let's
avoid loading all resources.
Change-Id: Iaf5650b2ae972064292cfe0d17f98b5635c51bad
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html
Change-Id: I5db9cc7f9a13db05a7d0ab0320e53bfdf4b419d9
|
| |\ \ \ \ \ |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
During volume detachment an instance can be in error state or locked.
Nova returns error 409.
Instance can be unlocked or fixed a bit later, but heat doesn't retry to detach volume.
The patch makes heat to retry detachments untill stack update timeout.
story: 2010355
Change-Id: I9a08b938401cd71eddd0eb80782d10392f92bf45
|
| |\ \ \ \ \ \ |
|
| | |/ / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Templates can be complex and contain multiple instances of
map_merges. Debugging this can be complex, so this change
aims to improve the mapping and help users determine where
the problem has occured.
With this change, instead of just saying an error occured,
we will now return the object that the error occurred with
and a note about it's type. This will allow for quick
identification of what the problem was.
Change-Id: I1c4da88029bbf1a3cfb58f5e21da28ec2912f924
|
| | |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
stestr has removed whitelist / blacklist. This change updates the
devstack upgrade tests to ensure we use the new include-list instead.
https://github.com/mtreinish/stestr/commit/9ffeb470fb4eaed484152d35411668db19a64ace
Change-Id: Ia0df9b3468fee9382c42c8bd6a35b76ed7f2b4e5
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This change fixes the one remaining usage without the flag which was
missed by the previous attempt[1].
[1] 14d2d859ef4eb3d21bf6e419a77c2f3185d5cf77
Change-Id: Ic0b34b11d669851e47cae670f5299f934f396257
|
| |\ \ \ \ |
|
