Skip to content

multiple clusters in patch-k8s-resource - #695

Merged
ydayagi merged 1 commit into
mainfrom
flpath2424
Sep 25, 2025
Merged

multiple clusters in patch-k8s-resource#695
ydayagi merged 1 commit into
mainfrom
flpath2424

Conversation

@ydayagi

@ydayagi ydayagi commented Jun 19, 2025

Copy link
Copy Markdown
Collaborator

@ydayagi
ydayagi marked this pull request as draft June 19, 2025 15:42
@ydayagi
ydayagi marked this pull request as ready for review September 21, 2025 09:29
Comment thread workflows/patch-k8s-resource/README.md Outdated
- name: patch
operation: "specs/patch-k8s-resource.yaml#patchResource"
metadata:
configKey: ".clusterName"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so the configKey metadata will be used to map the openapi spec URL to the right cluster, it's using the following pattern: quarkus.rest-client."<normalised_spec_file>.<configKey>".url am I right?

Is this documented anywhere in sonataflow/openapi generator docs?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to document so users know how to provide it

@gabriel-farache gabriel-farache left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, just a typo and a question about doc with configKey metdata mapping the cluster URL

@ydayagi
ydayagi merged commit c4aaf0c into main Sep 25, 2025
1 check passed
@ydayagi
ydayagi deleted the flpath2424 branch September 25, 2025 05:44
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