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

add OAM policy design #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ryanzhang-oss
Copy link
Collaborator

Add an initial OAM Policy design

Copy link
Member

@wonderflow wonderflow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not very sure whether it's enough, hope other guy could give more scenarios.

containers:
- name: mysql
image: mysql:latest
policies
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
policies
policies:

containers:
- name: wordpress
image: wordpress:latest
policies
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
policies
policies:

database component in a master only mode. Therefore, an application operator cannot apply a
`ManualScalerTrait` to the `example-db` component. In the same time, let's say that the
`example-web` component developer wants to expose the component to the internet. This
means that an application operator needs to know that it should apply an `IngressTrait' to
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
means that an application operator needs to know that it should apply an `IngressTrait' to
means that an application operator needs to know that it should apply an `IngressTrait` to

image: mysql:latest
policies
- type: maxInstancePolicy
value: 1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

- type: replicaPolicy
   value: singleton

@hongchaodeng
Copy link
Member

I think this proposal is missing the problem background and use cases. Would like to see them added first.

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

Successfully merging this pull request may close these issues.

3 participants