Skip to content

Define the structure of istio/api repo #3

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

Closed
wora opened this issue Dec 17, 2016 · 1 comment
Closed

Define the structure of istio/api repo #3

wora opened this issue Dec 17, 2016 · 1 comment
Assignees

Comments

@wora
Copy link
Contributor

wora commented Dec 17, 2016

github.com/googleapis/googleapis has a well defined structure. We should do the same for istio/api repo. For example, we can structure it like:

  • istio/mixer/v1
  • istio/config/v1
  • istio/attribute/v1
  • istio/adapter/authentication/v1

When we add more components, it is easy to put them into the right place. We also want a clear versioning story, such as no breaking change within a major version, no dependency between two major versions of the same component, all new features must default to off, how to tag Alpha/Beta/GA of individual features.

When many people contribute to the system, we need more guidelines to scale the development. Such rules are simple to define and relatively easy to follow.

@geeknoid geeknoid self-assigned this Dec 17, 2016
@douglas-reid
Copy link
Contributor

I like some of these suggestions. The more I look at it, I don't like having 'api' in the path beyond istio/api.

incfly pushed a commit to incfly/api that referenced this issue Jun 13, 2018
Add contrib/endpoints prefix to all includes.
incfly pushed a commit to incfly/api that referenced this issue Jun 13, 2018
* Add client source files.

* Remove servicecontrol build target.

* Add options as private variable.

* Add bazel build in travis.yml.
nacx pushed a commit to nacx/api that referenced this issue Apr 15, 2020
myidpt pushed a commit that referenced this issue May 11, 2021
istio-testing pushed a commit that referenced this issue May 11, 2021
howardjohn pushed a commit that referenced this issue Sep 7, 2022
add dataplane label to api
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants