Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Add Upstream API to regular API #2210

Closed
squaremo opened this issue Jul 1, 2019 · 0 comments · Fixed by #2211
Closed

Add Upstream API to regular API #2210

squaremo opened this issue Jul 1, 2019 · 0 comments · Fixed by #2211

Comments

@squaremo
Copy link
Member

squaremo commented Jul 1, 2019

At present, there are a small number of methods in the Upstream API that are not in the flux API. These are methods that can be called via RPC from an upstream (i.e., Weave Cloud) connection, but not via HTTP (e.g., not by fluxctl).

In the course of separating the Upstream part of the daemon out into weaveworks/flux-adapter, these will have to move into the public API, since flux-adapter runs in a sidecar, and uses HTTP (via http/client) rather than having in-process access to the daemon.

I believe the upstream methods can just be added to the v11 Server interface, and the Upstream interface abandoned from now on.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant