-
Notifications
You must be signed in to change notification settings - Fork 12
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
Updated docs to support resource classes in Score spec #63
Conversation
`type`: specifies the resource type. | ||
|
||
- **Type**: string. | ||
- **Constraints**: alphanumeric string. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alphanumeric with dashes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for class below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume the same is true for resource-name
and annotations
? https://docs.score.dev/docs/reference/score-spec-reference/#resources
- **Type**: string. | ||
- **Constraints**: alphanumeric string. | ||
|
||
`class`: a specialisation of the resource type. For example, a workload that needs an externally accessible resource might set the class to external while the workload that requires an encrypted resource might have a class of sensitive. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggest replace resource
with storage bucket
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Added support for resource classes as described in score-spec/spec#65