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

add engagement model to app file #518

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

maorfr
Copy link
Contributor

@maorfr maorfr commented Sep 21, 2023

supporting the proposal in https://gitlab.cee.redhat.com/app-sre/contract/-/merge_requests/109

adding engagementModel to app files will help determine the engagement level with each different service.

this attribute can be added to either child apps or parent apps, illustrating the engagement level per component if needed.

starting out with this attribute as optional so we don't create a breaking change that requires every instance to add this attribute to all app files. we can make it mandatory down the line, or not. perhaps the default is Standard, unless otherwise specified.

schemas/app-sre/app-1.yml Outdated Show resolved Hide resolved
@rporres
Copy link
Contributor

rporres commented Oct 6, 2023

Do we need this if the status of the onboarding is different? The related contract MR mentions a different onboarding status.

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

Successfully merging this pull request may close these issues.

3 participants