Drop qiskit-ibmq-provider requirement and integration tests

[coruscating]

Closes #1056. qiskit-ibmq-provider is deprecated and no longer required. Gadi has already set up the cron integration tests in qiskit-ibm-experiment, so the tests can be removed from this repository.

[wshanks]

Looks good!

cron-staging.yml is pretty close to main.yml now. It makes me wonder if we should do something like suggested here to merge the two together by putting a if: ${{ github.event_name != 'schedule' }} on the test running step.

[coruscating]

Thanks, I'll make a separate PR for merging the workflow files.

[mtreinish]

Do you want to add an optional dependency (via setuptools extras) because there is code in the tree which optionally will use the ibmq provider still: https://github.com/Qiskit/qiskit-experiments/blob/main/qiskit_experiments/database_service/utils.py#L31-L34

[coruscating]

Actually, I think those exceptions are in the experiments service now, so I've gotten rid of the optional IBMQ dependency. Thanks for catching that.

[wshanks]

Actually, I think those exceptions are in the experiments service now, so I've gotten rid of the optional IBMQ dependency. Thanks for catching that.

Nice to see that gone. Ideally there would be some way to catch an exception without needing to assume a specific experiment service implementation. I suppose any alternative service could make qiskit-experiments a dependency and use ExperimentEntryNotFound. qiskit-ibm-experiment could not do that because we wanted it split out as a dependency of qiskit-experiments.