Skip to content

PR for SRVLOGIC-976: Add a section about explaining the workflow id naming and versioning recommended practices and limitations in the OSL product docs#112874

Open
kaldesai wants to merge 1 commit into
openshift:serverless-docs-mainfrom
kaldesai:SRVLOGIC-976-workflow-idnameversioning
Open

PR for SRVLOGIC-976: Add a section about explaining the workflow id naming and versioning recommended practices and limitations in the OSL product docs#112874
kaldesai wants to merge 1 commit into
openshift:serverless-docs-mainfrom
kaldesai:SRVLOGIC-976-workflow-idnameversioning

Conversation

@kaldesai

@kaldesai kaldesai commented Jun 8, 2026

Copy link
Copy Markdown

You can cherry-pick for the following branches:

  • serverless-docs-1.38
  • serverless-docs-1.37

Tracking JIRA: https://redhat.atlassian.net/browse/SRVLOGIC-976

Changes:
Added a "Managing Workflow identifiers and versioning" section under the Getting started directory.

Reviews:

  • SME has approved this change
  • QE has approved this change
  • Peer has approved this change

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 8, 2026
@openshift-ci-robot

openshift-ci-robot commented Jun 8, 2026

Copy link
Copy Markdown

@kaldesai: This pull request references SRVLOGIC-976 which is a valid jira issue.

Details

In response to this:

You can cherry-pick for the following branches:

  • serverless-docs-1.38
  • serverless-docs-1.37

Tracking JIRA: https://redhat.atlassian.net/browse/SRVLOGIC-976

Changes:

Reviews:

  • SME has approved this change
  • QE has approved this change
  • Peer has approved this change

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 8, 2026
@ocpdocs-previewbot

ocpdocs-previewbot commented Jun 8, 2026

Copy link
Copy Markdown

@kaldesai

kaldesai commented Jun 8, 2026

Copy link
Copy Markdown
Author

/test validate-asciidoc

@kaldesai

kaldesai commented Jun 8, 2026

Copy link
Copy Markdown
Author

/retest

1 similar comment
@kaldesai

kaldesai commented Jun 8, 2026

Copy link
Copy Markdown
Author

/retest

@openshift-ci

openshift-ci Bot commented Jun 8, 2026

Copy link
Copy Markdown

@kaldesai: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@kaldesai

kaldesai commented Jun 8, 2026

Copy link
Copy Markdown
Author

/ serverless

@wmedvede wmedvede left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi @kaldesai , some minor nitpicks from my side.
Many thanks!

|Represents the workflow `id` from YAML

|`sonataflow.org/name`
|Optional human-readable name annotation

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think that here we can also clarify that this value must match the configured value in the YAML if it was configured. Since we would like "name" field in the YAML and the SonataFlow name annotation be the same value.

|Optional human-readable name annotation

|`sonataflow.org/description`
|Optional description annotation

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For the description same concept as for the name.

|Optional description annotation

|`sonataflow.org/version`
|Must match version from workflow YAML

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good clarification here, it must match the value from the yaml.


This approach ensures clarity and avoids conflicts when managing multiple workflow iterations.

The following example displays Version 1.0 workflow:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for adding these examples showing to move from version 1.0 to 2.0.
Do you think we can the corresponding:

apiVersion: sonataflow.org/v1alpha08
kind: SonataFlow
metadata:
name: hello-world

too?

Not sure if it could derive in something too long 🤔 , but could be helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants