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

Ensure celery-strategy is returning correct job-id #246

Open
quaat opened this issue Feb 24, 2023 · 3 comments · Fixed by EMMC-ASBL/oteapi-core#240
Open

Ensure celery-strategy is returning correct job-id #246

quaat opened this issue Feb 24, 2023 · 3 comments · Fixed by EMMC-ASBL/oteapi-core#240
Assignees
Labels
enhancement New feature or request

Comments

@quaat
Copy link
Contributor

quaat commented Feb 24, 2023

This issue goes beyond the oteapi-core. Ensure celery-strategy is returning correct job-id when a remote job is triggered. From the oteapi-service, ensure the status of the job can be correctly introspected. At the termination of the remote celery-worker a return value (model) is submitted back to the broker service (Redis), ensure this data can be fetched from the oteapi-services

@quaat quaat assigned quaat and CasperWA and unassigned quaat Feb 24, 2023
@CasperWA
Copy link
Contributor

CasperWA commented Mar 1, 2023

I'll just try to summarize the OP:

  • Ensure celery-strategy is returning correct job-id when a remote job is triggered.
  • Ensure the status of the job can be correctly introspected from the OTEAPI Services.
  • Ensure remote return value (model) data can be fetched from the OTEAPI Services.

@CasperWA
Copy link
Contributor

CasperWA commented Mar 1, 2023

Need to discuss this before I'll be able to move on. The whole surrounding design on how to use this strategy is a haze to me.

@CasperWA
Copy link
Contributor

CasperWA commented Mar 2, 2023

Exactly what is the "correct" job-id when a remote job is triggered? Is it the "local" Celery server's job ID or is it the job ID of a remotely running Celery server?

@CasperWA CasperWA transferred this issue from EMMC-ASBL/oteapi-core Mar 7, 2023
@CasperWA CasperWA reopened this Mar 7, 2023
@CasperWA CasperWA added the enhancement New feature or request label Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants