oam-dev / spec

Open Application Model (OAM).
https://oam.dev
Other
3.04k stars 246 forks source link

The case in Parameter fieldPaths may be settings.containers[0].image? #430

Closed jchalex closed 3 years ago

jchalex commented 3 years ago

The case in Parameter fieldPaths may be settings.containers[0].image, not spec.containers[0].image?

jchalex commented 3 years ago

In 3.component.md, parameter fieldPaths definition above Examples.

resouer commented 3 years ago

@jchalex It seems to be a bug. Not sure where's the inconsistency you noticed? Could you please paste a link?

jchalex commented 3 years ago

@jchalex It seems to be a bug. Not sure where's the inconsistency you noticed? Could you please paste a link?

https://github.com/oam-dev/spec/blob/master/3.component.md

image

resouer commented 3 years ago

@jchalex All bugs are fixed now! Thank you for reporting. We also did a round of clean up of the spec to reflect recent updates, for example, parameters are moved to WorkloadDefinitions now.