Skip to content

Stabilize integration tests#135

Merged
rathishcholarajan merged 4 commits into
Qiskit:mainfrom
daka1510:integration-tests
Jan 31, 2022
Merged

Stabilize integration tests#135
rathishcholarajan merged 4 commits into
Qiskit:mainfrom
daka1510:integration-tests

Conversation

@daka1510
Copy link
Copy Markdown
Contributor

@daka1510 daka1510 commented Jan 31, 2022

Scope

I checked the integration test results after #114 was integrated.

Problems

  1. in-progress matrix jobs are cancelled if any test fails
    • e.g. if integration tests against cloud-production fail, the integration tests against the other environments are cancelled
  2. Run integration test - legacy staging fails
Traceback (most recent call last):
  File "/home/runner/work/qiskit-ibm-runtime/qiskit-ibm-runtime/test/utils/decorators.py", line 100, in _wrapper
    raise Exception("Configuration Issue")
Exception: Configuration Issue
  1. Run integration test - cloud production fails
qiskit_ibm_runtime.api.exceptions.RequestsApiError: "HTTPSConnectionPool(host='us-east.quantum-computing.us-east.quantum-computing.cloud.ibm.com', port=443): Max retries exceeded with url: /devices (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f45f9c9be20>: Failed to establish a new connection: [Errno -2] Name or service not known'))"
  1. Status Badge is broken
    image

Related Solutions

@daka1510 daka1510 self-assigned this Jan 31, 2022
@rathishcholarajan
Copy link
Copy Markdown
Member

I've added legacy-staging and cloud-staging envs. Update URL in cloud-production.

Comment thread .github/workflows/integration-tests.yml
@rathishcholarajan rathishcholarajan merged commit cc006c0 into Qiskit:main Jan 31, 2022
Copy link
Copy Markdown
Member

@rathishcholarajan rathishcholarajan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks.

@daka1510 daka1510 deleted the integration-tests branch February 1, 2022 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants