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

Invalid hierarchy of x-kuadrant field #142

Closed
leathersole opened this issue Oct 23, 2024 · 1 comment
Closed

Invalid hierarchy of x-kuadrant field #142

leathersole opened this issue Oct 23, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@leathersole
Copy link

leathersole commented Oct 23, 2024

In this OpenAPI example and another example , first x-kuadrant fields must be root level. These files are referred from this document.

Current:

---
openapi: "3.1.0"
info:
  title: "Pet Store API"
  version: "1.0.0"
  x-kuadrant:
    route:
      name: "toystore"
      namespace: $devNS
      hostnames:
        - toystore.$rootDomain
      parentRefs:
        - name: $gatewayName
          namespace: $gatewayNS

Must be:

---
openapi: "3.1.0"
info:
  title: "Pet Store API"
  version: "1.0.0"
x-kuadrant:
  route:
    name: "toystore"
    namespace: $devNS
    hostnames:
      - toystore.$rootDomain
    parentRefs:
      - name: $gatewayName
        namespace: $gatewayNS
@david-martin
Copy link
Member

Thanks for reporting this @leathersole
The files are sourced from the kuadrant-operator repo, so I've opened a fix PR for that.

Note that the docs page that references these examples has changed since v0.11.0 and no longer references them or the use of kaudrantctl.
That could change in future, but as of now, there is no further action on the docs site/repo, so I'll close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

3 participants