Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use LocalProvider for test #1212

Merged
merged 4 commits into from
Feb 7, 2024
Merged

Conversation

akihikokuroda
Copy link
Collaborator

Summary

fix #1141

Details and comments

@akihikokuroda akihikokuroda marked this pull request as draft February 6, 2024 19:10
Signed-off-by: Akihiko Kuroda <[email protected]>
Copy link
Collaborator

@psschwei psschwei left a comment

Choose a reason for hiding this comment

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

Thanks Aki!

Would it be possible to make this a separate test? I'd lean towards keeping the docker compose test for testing gateway / scheduler until we can get an e2e test on kubernetes working (I have some ideas for how we can do that which I'll try to tackle new week).

Copy link
Collaborator

@psschwei psschwei left a comment

Choose a reason for hiding this comment

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

LGTM!

@akihikokuroda akihikokuroda merged commit 4afc634 into Qiskit:main Feb 7, 2024
14 checks passed
david-alber pushed a commit to david-alber/quantum-serverless that referenced this pull request Feb 13, 2024
* use LocalProvider for notebook test

Signed-off-by: Akihiko Kuroda <[email protected]>
Tansito added a commit that referenced this pull request Feb 27, 2024
* adding api documentation (swagger & redoc) to gateway

* allow unauthenticated users to read RuntimeJobViewSet

* allow swagger docu if user is not yet set

* only show api/v1/ patterns, add token authorization, rm django login

* drop jupyter from main helm readme (#1208)

Signed-off-by: Paul S. Schweigert <[email protected]>

* CatalogEntry API (#1204)

* add ModelView API for CatalogEntry

* additional apis for catalogentry

---------

Signed-off-by: Akihiko Kuroda <[email protected]>

* Repository removal (#1207)

* Removed main components from repository project

* Updated chart lock

* Remove repository references from the client

* add catalog status (#1211)

Signed-off-by: Akihiko Kuroda <[email protected]>

* Use LocalProvider for test (#1212)

* use LocalProvider for notebook test

Signed-off-by: Akihiko Kuroda <[email protected]>

* Documentation update (#1214)

* Update contribution guidelines

* Update contributing doc with supported python versions

* Remove Python 3.7 badge from README files

* Upgrade min python version in setup.py

* Test notebooks against k8s qs deployment (#1216)

* test notebooks against k8s qs deployment

Signed-off-by: Paul S. Schweigert <[email protected]>

* update notebook location

Signed-off-by: Paul S. Schweigert <[email protected]>

* use action for kind creation

Signed-off-by: Paul S. Schweigert <[email protected]>

* test all notebooks

Signed-off-by: Paul S. Schweigert <[email protected]>

* drop sleep

Signed-off-by: Paul S. Schweigert <[email protected]>

This was redundant, as the python setup in the next few steps takes
waaaay longer than 60s

---------

Signed-off-by: Paul S. Schweigert <[email protected]>

* Take out unnecessary Job.save() (#1217)

* take out unnecessary Job.save()

Signed-off-by: Akihiko Kuroda <[email protected]>

* Update Helm release kuberay-operator to v1 (#1121)

* Update Helm release kuberay-operator to v1

* Update api version

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: David <[email protected]>

* Fix link for Qiskit deprecation policy (#1218)

* Remove default login view (#1215)

* remove default login view

Signed-off-by: Akihiko Kuroda <[email protected]>

* adding api documentation (swagger & redoc) to gateway

---------

Signed-off-by: Paul S. Schweigert <[email protected]>
Signed-off-by: Akihiko Kuroda <[email protected]>
Co-authored-by: Paul Schweigert <[email protected]>
Co-authored-by: Akihiko (Aki) Kuroda <[email protected]>
Co-authored-by: David <[email protected]>
Co-authored-by: Karla Spuldaro <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Eric Arellano <[email protected]>
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.

Run tests using LocalProvider
2 participants