diff options
author | Matt Clay <mclay@redhat.com> | 2020-08-24 15:17:03 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-08-24 15:17:03 -0700 |
commit | d099591964d6fedc174eb1d3fc1bbee8d2ba0f16 (patch) | |
tree | 97fa469de3ba30aa586261d9bba1be4e99e8ee0f /test/lib/ansible_test/_internal/core_ci.py | |
parent | a6537b59abc9c69777130d0af3d34fac50535a20 (diff) | |
download | ansible-d099591964d6fedc174eb1d3fc1bbee8d2ba0f16.tar.gz |
Update ansible-test remote endpoint handling. (#71413)
* Request ansible-core-ci resources by provider.
* Remove obsolete us-east-2 CI endpoint.
* Add new --remote-endpoint option.
* Add warning for --remote-aws-region option.
* Update service endpoints.
* Allow non-standard remote stages.
* Add changelog fragment.
Diffstat (limited to 'test/lib/ansible_test/_internal/core_ci.py')
-rw-r--r-- | test/lib/ansible_test/_internal/core_ci.py | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/test/lib/ansible_test/_internal/core_ci.py b/test/lib/ansible_test/_internal/core_ci.py index 24cf7c4280..b1e9dd9f2a 100644 --- a/test/lib/ansible_test/_internal/core_ci.py +++ b/test/lib/ansible_test/_internal/core_ci.py @@ -50,8 +50,7 @@ from .data import ( ) AWS_ENDPOINTS = { - 'us-east-1': 'https://14blg63h2i.execute-api.us-east-1.amazonaws.com', - 'us-east-2': 'https://g5xynwbk96.execute-api.us-east-2.amazonaws.com', + 'us-east-1': 'https://ansible-core-ci.testing.ansible.com', } @@ -87,8 +86,6 @@ class AnsibleCoreCI: else: self.name = '%s-%s' % (self.platform, self.version) - self.resource = 'jobs' - # Assign each supported platform to one provider. # This is used to determine the provider from the platform when no provider is specified. providers = dict( @@ -157,10 +154,8 @@ class AnsibleCoreCI: self.path = os.path.expanduser('~/.ansible/test/instances/%s-%s-%s' % (self.name, self.provider, self.stage)) if self.provider in ('aws', 'azure', 'ibmps', 'ibmvpc'): - if self.provider != 'aws': - self.resource = self.provider - if args.remote_aws_region: + display.warning('The --remote-aws-region option is obsolete and will be removed in a future version of ansible-test.') # permit command-line override of region selection region = args.remote_aws_region # use a dedicated CI key when overriding the region selection @@ -169,7 +164,12 @@ class AnsibleCoreCI: region = 'us-east-1' self.path = "%s-%s" % (self.path, region) - self.endpoints = (AWS_ENDPOINTS[region],) + + if self.args.remote_endpoint: + self.endpoints = (self.args.remote_endpoint,) + else: + self.endpoints = (AWS_ENDPOINTS[region],) + self.ssh_key = SshKey(args) if self.platform == 'windows': @@ -183,8 +183,11 @@ class AnsibleCoreCI: # 90 seconds self.retries = 7 elif self.provider == 'parallels': - self.endpoints = self._get_parallels_endpoints() - self.max_threshold = 6 + if self.args.remote_endpoint: + self.endpoints = (self.args.remote_endpoint,) + else: + self.endpoints = self._get_parallels_endpoints() + self.max_threshold = 6 self.ssh_key = SshKey(args) self.port = None @@ -236,7 +239,7 @@ class AnsibleCoreCI: sleep = 3 for _iteration in range(1, 10): - response = client.get('https://s3.amazonaws.com/ansible-ci-files/ansible-test/parallels-endpoints.txt') + response = client.get('https://ansible-ci-files.s3.amazonaws.com/ansible-test/parallels-endpoints.txt') if response.status_code == 200: endpoints = tuple(response.response.splitlines()) @@ -370,7 +373,7 @@ class AnsibleCoreCI: @property def _uri(self): - return '%s/%s/%s/%s' % (self.endpoint, self.stage, self.resource, self.instance_id) + return '%s/%s/%s/%s' % (self.endpoint, self.stage, self.provider, self.instance_id) def _start(self, auth): """Start instance.""" |