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

Update Arbitrary Properties example on the Dependency Resolution page #271

Open
nschambu opened this issue Feb 6, 2023 · 0 comments
Open

Comments

@nschambu
Copy link

nschambu commented Feb 6, 2023

Issue: The example in the Arbitrary Properties section of the Dependency Resolution page is incorrect.
Page in question: https://olm.operatorframework.io/docs/concepts/olm-architecture/dependency-resolution/#arbitrary-properties

Details

The Arbitrary Properties section of the Dependency Resolution page includes an additional "property" field that is unnecessary.

Current example:

properties:
  - property:
      type: sushi
      value: salmon
  - property:
      type: soup
      value: miso
  - property:
      type: olm.gvk
      value:
        group: olm.coreos.io
        version: v1alpha1
        kind: bento

Proposed update:

properties:
  - type: sushi
    value: salmon
  - type: soup
    value: miso
  - type: olm.gvk
    value:
      group: olm.coreos.io
      version: v1alpha1
      kind: bento

My dependency resolution was failing when using the current example, but was successful when I updated it to the proposed update.

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

1 participant