cyberark / conjur-openapi-spec

OpenAPI v3 specification for Conjur / DAP v10+
https://conjur.org
Apache License 2.0
11 stars 2 forks source link

The OpenAPI spec has a versioning scheme that is also clear about the Conjur version #89

Closed izgeri closed 3 years ago

izgeri commented 3 years ago

In our design document, we proposed that the link between the Conjur server and the OpenAPI spec can be indicated by two files in this project - VERSION (for the spec / API version) and SERVER_VERSION (for the Conjur OSS release that this spec is compatible with).

In this issue, we will add these version files to the spec project and add documentation to this project and to the Conjur repo that gives guidance on how to maintain the spec when making changes to Conjur, and how to change the version when releasing the spec.

In general, we expect the initial workflow for making changes to the API to be:

To release the spec, the workflow will be:

AC:

izgeri commented 3 years ago

Note: we may not need a standalone VERSION file, since the conjur-openapi.yml file has a version field.

We may also want to refer users to find the correct server version in the description field of the spec YAML.

One last open question: do we want to archive specs for older API versions? We could do that the way that GitHub does here.

telday commented 3 years ago

@izgeri Isn't archiving old versions of the spec done automatically whenever we release the project in the form of the attached zip/tarball files on each release? Or did we want something that just included the spec and not the entire repo? If that is the goal we could also just zip the OpenAPI files and attach that to each release as well.

izgeri commented 3 years ago

@telday I think this came up because we wanted an easy reference to the spec for a specific Conjur version - you can see how we archive suite releases here

I'm not sure there's a clean way to archive the old YAMLs in this repo, so I'm fine with having the spec attached to a gh release. I can see the benefit of having a smaller ZIP with just the spec files, fwiw.

izgeri commented 3 years ago

@telday can we close this? if the last item is outstanding, let's file a new issue for it to be addressed at a future date

telday commented 3 years ago

Can definitely close. The last item should be good, as long as we don't have anymore issues with the conjur:edge image.