question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Update the `/api` endpoints to `/elyra`

See original GitHub issue

/api is used as the starting path for Elyra endpoints. e.g., /api/metadata/code-snippets.

while /api is convenient and also easy to remember, after some discussion and playing around with the swagger docs i have a couple comments:

  1. /api is very common. there is the possibility that some extension (or Jupyter functionality) comes along which also uses the /api path. this clash could cause issues.

  2. /api is not clearly identifiable as an Elyra specific endpoint. someone could easily assume that /api/metadata/runtimes is a Jupyter endpoint and have no way of immediately realizing it is strictly for Elyra.

with that being said, i would think Elyra would want its own/unique path to clearly distinguish its endpoints. e.g., /elyra.

the possibility of some other 3rd party component or a Jupyter update being introduced that used that same /elyra path is much more unlikely. at the same time, the Elyra API endpoints would become clearly recognizable:

  • /elyra/metadata/code-snippets
  • /elyra/metadata/runtimes
  • /elyra/schema
  • etc.

it doesn’t necessarily have to be /elyra (although i think that works well) but something more distinguishable and less common than /api should be considered.

i would expect changing/updating the endpoint path is a manageable task but then again i don’t know how set in stone the Elyra APIs are to properly speak on how much work would actually be involved. so i leave this here for discussion and thoughts from others?

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Reactions:5
  • Comments:8 (8 by maintainers)

github_iconTop GitHub Comments

2reactions
kjdoylecommented, May 9, 2020

I agree with this as well. We always try to differentiate the different APIs with a different prefix.

2reactions
kevin-batescommented, May 8, 2020

I agree. In fact, all of the notebook endpoints begin with /api. I think this change makes sense.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Runtime Configuration — Elyra 3.14.0.dev0 documentation
The KubeFlow Pipelines API endpoint you want to utilize. This setting is required. Example: https://kubernetes-service.domain.com/pipeline. Public Kubeflow ...
Read more >
elyra-ai/community - Gitter
... I am trying to run elyra pipelines on kubeflow in ibm cloud , but i have runtime error I think it is...
Read more >
Pipelines API Reference - Kubeflow
Reference documentation for the Kubeflow Pipelines API. ... Update the default pipeline version of a specific pipeline.
Read more >
Design a visual workflow with Charmed Kubeflow and Elyra
In this tutorial, we will examine Elyra, a visual workflow design ... In the section “Kubeflow Pipelines”, add your Kubeflow endpoint.
Read more >
JupyterLab/Elyra: pipeline run on Kubeflow Pipelines fails with ...
8/site-packages/kfp_server_api/api/pipeline_upload_service_api.py", line 83, in upload_pipeline return self.upload_pipeline_with_http_info( ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found