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

OPA should expose version information via the REST API #277

Closed
tsandall opened this issue Feb 22, 2017 · 4 comments
Closed

OPA should expose version information via the REST API #277

tsandall opened this issue Feb 22, 2017 · 4 comments

Comments

@tsandall
Copy link
Member

It would be nice if OPA's version information could be retrieved through the API.

With the new /data/system namespace we could put it under /data/system/version for instance.

If we do this we should fix the version information shown in the REPL to go under /data/system/version as well.

@BenderScript
Copy link
Contributor

Hi,

returning the version information for a GET request to /data/system/version is straightforward even if there is nothing really inmem under data.system. All we need is a new route.

But if we want to actually have the version under data.system, we need to insert this information when the system starts. Basically we need to create a new document during initialization. Maybe something inside opa/cmd/run.go inside init() function? Is there a place where the system itself we can create documents before repl is up?

@tsandall
Copy link
Member Author

tsandall commented Oct 7, 2018

I think that adding a route with be fine. Putting it into storage seemed like the path of least resistance at the time, however, we'd need to rethink that given that the bundle plugin would overwrite the data on each activation.

@kichristensen
Copy link
Contributor

Isn't this closed by #986?

@tsandall
Copy link
Member Author

Fixed by #986

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

No branches or pull requests

3 participants