Commit Graph

781 Commits

Author SHA1 Message Date
Ghanshyam Mann a1119eb2d8 Retire Tripleo: remove repo content
TripleO project is retiring
- https://review.opendev.org/c/openstack/governance/+/905145

this commit remove the content of this project repo

Change-Id: I5080dd23acbf6beca58e70e6ae1f1bc3d1843161
2024-02-24 11:45:33 -08:00
Sandeep Yadav 74eec6791c Disable auto discovery
setuptools 61.0 brings breaking changes which are not backwork
compatible, details in related bug and [1].

Users that don't set ``packages``, ``py_modules``, or `configuration`
are still likely to observe the auto-discovery behavior, which may
halt the build if the project contains multiple directories and/or
multiple Python files directly under the project root.

To disable auto discovery, one can do below in setup.py

~~~
setuptools.setup(..,packages=[],..)
~~~

or

~~~
setuptools.setup(..,py_modules=[],..)
~~~

[1] https://github.com/pypa/setuptools/issues/3197

Releated-Bug: #1966382
Change-Id: I0c9e9833bc30726d575f2560bd0a252417168f8a
2022-03-28 15:35:50 +05:30
maaoyu 73af2b0f64 setup.cfg: Replace dashes with underscores
Setuptools v54.1.0 introduces a warning that the use of dash-separated
options in 'setup.cfg' will not be supported in a future version [1].
Get ahead of the issue by replacing the dashes with underscores. Without
this, we see 'UserWarning' messages like the following on new enough
versions of setuptools:

  UserWarning: Usage of dash-separated 'description-file' will not be
  supported in future versions. Please use the underscore name
  'description_file' instead

[1] https://github.com/pypa/setuptools/commit/a2e9ae4cb

Change-Id: I553278217772a4f87d053b2632baf485cac5bfad
2021-04-27 16:24:35 +08:00
Zuul c873c8e18a Merge "Force to use markdown to prevent pypi issue" 2019-09-03 20:28:46 +00:00
Zuul ac9e92bec7 Merge "Update the outdated URL" 2019-09-03 20:28:16 +00:00
Keigo Noha 928e74d86e Modify grep rule to identify 'Clone Set' resources
Current filter rule gets wrong results in transient condition.
This commit changes the grep filter rule to increase the robustness of
the result.

Closes-Bug: #1839910

Change-Id: I92aba5bc7eb9e8a90d6a2c321b3a2a20414f1779
2019-08-13 12:59:23 +09:00
Keigo Noha 8bb060b864 Replace '-' with '_' in fence_compute options.
auth-url, tenant-name and read-only are defined in fence_compute as
auth_url, tenant_name and read_only. This commit replaces '-' with '_'
in those 3 options.

Closes-Bug: #1839063
Change-Id: I509ee3f405ed5adfbbb0b678650303a6b4829fcd
2019-08-06 16:38:46 +09:00
Hervé Beraud 61fccde07a Force to use markdown to prevent pypi issue
Recently the pypi team have reintroduce some deactivated tests
on package deployment and especially tests concerning long_description
syntaxe.

If projects don't specify that they use the markdown format the
package deployment will fail if something went wrong in the markdown format.

By example if a project use inline literal this can be an issue due to
the fact that if `long_description_content_type` is missing [1]. The
default format will be `text/x-rst` where code block is different from
markdown.

These changes force to use the markdown format to avoid this kind of issue
on project who use `README` at markdown format.

[1] http://lists.openstack.org/pipermail/openstack-discuss/2019-July/007459.html

For more details:
https://github.com/pypa/warehouse/issues/5890#issuecomment-494868157

Change-Id: I0f1b5c2f911b6a421525075a8c52cf27da38bf39
2019-07-01 20:46:41 +02:00
Zuul 4e34c3d820 Merge "Port rally plugins to use proper modules" 2019-06-18 10:23:12 +00:00
Luca Miccini af8cd25a2d Restore rsync functionality for jenkins
be6f1527c8 broke rsync when run via jenkins.
This commit partially reverts the change checking if running via zuul,
otherwise it restores the old syntax.

Change-Id: I1d66adf0cfe58570fa3fa581e768ed65f7076e9a
2019-05-22 09:36:52 +02:00
OpenDev Sysadmins 256a057941 OpenDev Migration Patch
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.
2019-04-19 19:29:42 +00:00
Luca Miccini be6f1527c8 Switch to {{ undercloud_user }} for rsync
{{ ansible_working_dir }} doesn't seem to expand to a valid path when
the playbook is ran by zuul.
Switching to {{ undercloud_user }} seems to work.

Change-Id: I0d89e379eae257289dc2fe560ccdaa14bdea9047
2019-04-17 06:14:20 +00:00
Zuul ebb6db142c Merge "Fix the License mismatch in README" 2018-11-23 15:13:26 +00:00
Raoul Scarazzini 5937e9c06a Fix the License mismatch in README
There was a mismatch between the indication of the used license in the
README (originally GPL) and the License file produced (Apache). Now both
refers to Apache, like it should always have been.

Change-Id: Ibd2d26d96a85f45a589b2bc1836c99eafac4454f
2018-11-22 17:34:13 +01:00
Raoul Scarazzini e758983ed4 Remove trailing white spaces
This commit removes various trailing white spaces.

Change-Id: I2e57535a1234dbf8904bd178c57186e50c4f75eb
2018-11-02 18:18:00 +01:00
Raoul Scarazzini 845cee0483 Fix get instance IP after heat test
There's an error in the way the IP of the instance that gets created by
the heat template is read. Since we're now redirecting the output to a
file, we need tee to let ansible still read from the standard output.

Change-Id: I796295879fe1d5693fcd349052f6d83611028a21
2018-11-02 18:01:44 +01:00
Zuul bdbc6a5b9b Merge "Fix STONITH readme removing release from examples" 2018-10-26 15:03:06 +00:00
Raoul Scarazzini 71c55e034f Fix STONITH readme removing release from examples
This commit removes the release parameter from the exaple commands
related to STONITH, the parameter is not requested, not needed and may
confuse users.

Change-Id: I664c8d4a2ae9b33f814bde083de0828115b1a73f
2018-10-25 11:24:30 +02:00
Raoul Scarazzini 5e1a38b875 Add heat results inside log files
This commit simplifies the way logs are generated, removing the split
between stdout and stderr and adding also the logs for the heat actions,
so that analyzing failures should become easier.

Change-Id: Ic1d150d5d67a1c22ce8a3bb6951265a8c9b94f27
2018-10-24 20:00:32 +02:00
Raoul Scarazzini 2e2cff1969 Add tags to validate HA playbook
This commit add tags inside the validate ha playbook, so that a user
will be able to execute atomically the HA validation tests when the
playbook is included in other playbooks (like it happens in
qucikstart).

Change-Id: I8f1e78e41368863aecb6962c16bba2240c3dc05c
2018-10-05 10:23:01 +02:00
melissaml 22b0aa55c7 Update the outdated URL
Change-Id: Iabf990335e3691175be5e15a7ac6f2650acbfc25
2018-09-23 16:39:46 +08:00
Zuul 8870baa85e Merge "Apply/Delete latency during HA validation" 2018-08-01 14:27:11 +00:00
Raoul Scarazzini 534970198b Apply/Delete latency during HA validation
This commit adds the latency management into the HA validation process.
If a user wants to go trough the HA validation with a specific latency
applied on all the nodes he will pass the "latency_ms" variable with a
value greater than zero and a "latency_eth_interface" variable with the
value related to the device chose for the test.

Change-Id: I3d0f9cc3301ba6be893919b58473f3f4135ea789
2018-07-30 16:57:43 +02:00
Raoul Scarazzini 9107b7d59c Fix hard-coded domain in instance HA role
This commit sets a default value of "localdomain" for the domain
parameter related to the overcloud.
This is used inside the role while configuring nova-compute-checkevacuate
resource and the fence-nova STONITH device. In this way, the user will
be able to override the default by passing the overcloud_domain
parameter in the command line.

Fixes-Bug: #1782752
Change-Id: Ib76523a08e43b8d9f604e9fb2544017c9800d8a9
2018-07-20 14:57:19 +02:00
Zuul fef8389b5f Merge "Fix custom STONITH compute names in IHA playbook" 2018-07-13 16:30:44 +00:00
Raoul Scarazzini 183b16c516 Fix custom STONITH compute names in IHA playbook
Today when STONITH for the compute nodes is already configured in the
TripleO deployment and the name of these STONITH resources is not
"ipmilan-<hostname>" the task will likely fail.
This commit fixes the way STONITH levels are managed, making the task to
not rely anymore on the fact that a user will configure also STONITH
with the instance HA playbooks, but reading dynamically the name of the
resources.
It also takes care about it while removing the IHA steps, preserving the
idempotent principle.

Closes-Bug: #1780434

Change-Id: I2f8351cf62cddd420c257110d60442ca4243a43d
2018-07-10 18:21:51 +02:00
cchen666 42ad0c4260 Fix iptables insert command from -A to -I
Change-Id: I24eb771eb1126e2b248c345da2f977ce1f83d8a1
2018-06-29 16:06:29 +08:00
Andrey Kurilin bda4071ae6 Port rally plugins to use proper modules
Sometime ago, all OpenStack related stuff moved from the main repository
to the separate one - https://github.com/openstack/rally-openstack .
All further development and maintance will be done only in a new
repository. As for first major release, the structure is not changed, so
only imports should be changed.

Also, this patch fixes deprecated usage of validators.

Change-Id: Ibd884b4686477ca990271868b0555fbea80e07f1
2018-06-13 09:10:07 +03:00
Raoul Scarazzini dd51dee43e Split heat stack creation in modules
This commit splits the heat stack validation in three phases: the
creation, the check and the deletion. In this way it will be more usable
for different purposes.

Change-Id: I7010b27b2762dce67702db0561380442d6ac2575
2018-06-12 18:32:06 +02:00
Raoul Scarazzini 5ec66ffeed Fix Pacemaker NG test C inside the main task
This commit fixes the C test of the Pacemaker list so that is correctly
called during the test execution.
Today it is never executed and instead we execute the B test two times.

Change-Id: Id5ffe3ed531071ac3cb3dd077e8df62d1f8b92e4
2018-05-30 18:47:06 +02:00
Raoul Scarazzini 20507623d0 Fix the last old repository references
This patch fixes the latest references that are around and refer to the
old repository name (tripleo-quickstart-utils), changing it into the new
tripleo-ha-utils.

Change-Id: I426387486b832314882a3094966e4f2806218331
2018-05-23 17:12:48 +02:00
Zuul 23b0d18a1f Merge "Fix the way iptables rules are managed" 2018-05-23 10:27:50 +00:00
pkomarov d78e5a9dbb switch to use the upstream utils repo
Change-Id: I9ea3746384d4db09abafb156afa056e9eb0c3502
2018-05-23 10:26:36 +03:00
Raoul Scarazzini 20bbd0f456 Fix the way iptables rules are managed
Today we use iptables-save to store the configuration related to the
firewall (we enable INPUT traffic for pacemaker remote on all the hosts)
but this method is not reliable since Neutron put and remove rules by
itself, and by using iptables-save we also store them.
This patch treat the single rule in both apply and undo playbooks so
that we don't touch anything that neutron might touch.

Change-Id: I6293d7a065f8e531de6218f272a8b08844b3eb42
2018-05-22 16:42:27 +02:00
Raoul Scarazzini 3367205138 Replace any redhat/quickstart previous reference
Since we are now on the new repository, we want to use it as a reference
instead of the original one, "redhat-openstack/tripleo-quickstart-utils":
this commit replaces any reference with "openstack/tripleo-ha-utils".

Change-Id: Ibc1c62e498c2fc9b84d88e245df080cf8c9471cc
2018-05-21 18:58:32 +02:00
Raoul Scarazzini e3aca95dca Make instance HA uninstallation idempotent
This commit pairs with the one related to the pre checks [1] and make
the instance HA uninstallation idempotent, so that it will be possible
to run even if something was interrupted before.

[1] https://review.openstack.org/#/c/567892/

Change-Id: I82c8bd4a41bce267fdad21c259c197e3edfe7497
2018-05-17 18:03:06 +02:00
Raoul Scarazzini 21510c1d2e Add pre checks before installing instance HA
This commit is the first of two that will try to make the instance HA
procedure idempotent. This first one adds pre checks that make the
procedure fail if there are already IHA resources or STONITH resources
inside the cluster.

Change-Id: Ib481f7be119ffd301e53d952c413f1ba0e05de84
2018-05-17 13:40:48 +02:00
pkomarov 29ebf802e1 Some fixes for known issues :
1) This change add the "user_domain_name" and "project_domain_name"
to the clouds.yaml parameters

2) This fixes quotes around file dest error:
"Could not replace file:...  /home/stack/clouds.yaml:
[Errno 2] No such file or directory "

Change-Id: I5f9573c65b4e773e6bed75753f7c221badee16da
2018-05-07 11:03:06 +00:00
Raoul Scarazzini b432c230bd Fix .gitreview file to point new gerrit
This commit fixes the .gitreview file which was pointing to the old
gerrit server. Now we're part of github.com/openstack and we should
point there.

Change-Id: Ie915264c112572b3f99e1b0a7102249c2753130b
2018-05-03 14:34:05 +02:00
Raoul Scarazzini 00f82288b1 Add overcloudrc.v3 support in IHA
This commit adds the support to the stack rc file related to the version
3 of keystone. It will check for an overcloudrc.v3 file in the
working_dir of the deployment (typically /home/stack) and if it is not
able to find it it will use overclourc as usual.
The command for creating the nova-evacuate resource was changed
accordingly, adding --force to support liberty/osp8 and mitaka/osp9
because in these releases we can't provide the new requested fields
project_domain and user_domain.
The way overcloudrc was treated before has been cleaned. No need to copy
the file on the controller node. The name of the file is calculated
dynamically as before, also for v3.

Note: this commit solves also what was proposed here [1] when dealing
with overcloudrc files in which there are both OS_TENANT_NAME and
OS_PROJECT_NAME variables, taking the latter.
The stonith creation python script has been modified to support also
Ocata, for which we still have OS_TENANT_NAME but we don't support
positional arguments anymore.

Change-Id: I1a143b67850c67d6d3207c06abcf60d32f2456ff
2018-04-06 10:19:17 -04:00
Raoul Scarazzini 8fc618c3d3 Fix STONITH script creation for Liberty/Mitaka
Without using positional arguments inside the connection function in the
template for the stonith script creation the backward compatibility is
broken. This commit fix this by using again positional arguments and it
will fix Liberty/OSP8 and Mitaka/OSP9 deployments.

Change-Id: I49920dd23e372e034706d49a15d6f91a0a248bd8
2018-03-29 10:40:01 -04:00
Raoul Scarazzini a05aa6acbe Merge "This section creates the clouds.yaml file from undercloud/overcloud credentials . This is for use by pythonsdk api for OSP connection and querrying. This will allow us to run the same rally/sts tests on the OSP deployment whether it was deployed by Infrared or Quickstart." 2018-03-22 07:14:23 -04:00
Raoul Scarazzini c6fc656592 Increase wait timeout for instances to come up
There were some problems with the previous one minute startup, sometimes
we caught failures just for a few seconds. Increasing to five minutes
gives us a reasonable amount of time to determine whether or not are
there problems.

Change-Id: Ibdcec51347e1a816b019638a2b9a69e390f15eac
2018-03-21 12:40:30 -04:00
Raoul Scarazzini f52f30947d Add Queens/RHOS13 and Rocky release coverage
This commit creates all the necessary var files to cover the new
OpenStack releases, starting from Queens (linked by RHOS-13) and adding
also Rocky which is now pointed by the Master link.

Change-Id: Iec39b969e166aee51c836ec7d76eba14af42b53c
2018-03-02 12:24:55 -05:00
pkomarov ed71ae7e7c This section creates the clouds.yaml file from
undercloud/overcloud credentials .
This is for use by pythonsdk api for OSP
connection and querrying.
This will allow us to run the same rally/sts tests on
the OSP deployment whether it was deployed by
Infrared or Quickstart.

Change-Id: I82e3fd04282506f706f91bfd399795f92612cf36
2018-02-26 13:20:52 +02:00
Raoul Scarazzini f41bd7f805 Create dedicated ha-test-suite task file
This commit creates a dedicated task file related to the ha-test-suite
command invocation, because of this it will be easier to declare in the
main task a test like this:

- name: HA test - Master/Slave core resource stop and start
  vars:
    ha_test_name: "test_master-slave"
    ha_recovery_name: "recovery_master-slave"
    include: ha-test-suite.yml
  when: test_ha_master_slave|bool

In case the test does not require a recovery, then the variable shuld be
declared as "".

Change-Id: Idd83393e212eed0658898ffd07e7d7316ab061b4
2018-02-13 09:24:21 -05:00
pkomarov 73640870f9 fix indent in lines 46,69
Change-Id: Ia7c02101dea703f6e5b678fb2e3ca4579d80d5a5
2018-02-08 13:38:52 +02:00
Raoul Scarazzini 54bec14470 Use Heat template to verify tests
This commit introduces the usage of heat to verify each HA test made.
The test template involves:

- Orchestration (heat);
- Volumes (cinder);
- Images (glance);
- Network, public and private (neutron);
- Server (nova);

Summary of what this commit does:

- Get rid of the workarounds;
- Get rid of undercloud options and its ha-test-suite local copy (now
  useless);
- Get rid of the old environment file needed to spawn instances;
- Get rid of instance test from ha-test-suite;
- Add Heat template verification method;

Change-Id: I2dd9d67f494717654e39c60ac5fb067afb9e1835
2018-02-02 04:51:39 -05:00
Raoul Scarazzini d9c6c5e7fa Merge " Changes to be committed: modified: infrared/infrared_instance-ha_plugin_main.yml new file: infrared/templates/ansible_hosts.yml.j2" 2018-01-23 12:23:39 -05:00
pkomarov f1bf629e19 Changes to be committed:
modified:   infrared/infrared_instance-ha_plugin_main.yml
	new file:   infrared/templates/ansible_hosts.yml.j2

manual  override because of https://github.com/ansible/ansible/issues/26336
This executes all from the undercloud itself

Change-Id: I1be692d268cd9701558b5cdd47176ccc2a6e1144
2018-01-21 07:00:33 -04:00