Skip to content

Conversation

VaniHaripriya
Copy link
Collaborator

@VaniHaripriya VaniHaripriya commented May 24, 2023

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)
  • [KOGITO-8443] Added note about simplified format for Knative service discovery(#326)
  • Updated with steps to overcome an issue on macOS (#274)
  • [KOGITO-8818] Rearrange guides to add operator and quarkus cloud (#313)
  • [KOGITO-8820] Fix references in timeout guide and create a new section for job service(#320)
  • Updated versions in antora.yml file
  • Updated Release notes

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)

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/test-infra repository.

@netlify
Copy link

netlify bot commented May 24, 2023

Deploy Preview for jazzy-shortbread-5f62b7 ready!

Name Link
🔨 Latest commit b62e60c
🔍 Latest deploy log https://app.netlify.com/sites/jazzy-shortbread-5f62b7/deploys/648c8bb4481b090008f69d5b
😎 Deploy Preview https://deploy-preview-94--jazzy-shortbread-5f62b7.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)

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/test-infra repository.

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)
  • [KOGITO-8443] Added note about simplified format for Knative service discovery(#326)
  • Updated with steps to overcome an issue on macOS (#274)

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/test-infra repository.

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)
  • [KOGITO-8443] Added note about simplified format for Knative service discovery(#326)
  • Updated with steps to overcome an issue on macOS (#274)
  • [KOGITO-8818] Rearrange guides to add operator and quarkus cloud (#313)

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/test-infra repository.

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)
  • [KOGITO-8443] Added note about simplified format for Knative service discovery(#326)
  • Updated with steps to overcome an issue on macOS (#274)
  • [KOGITO-8818] Rearrange guides to add operator and quarkus cloud (#313)
  • [KOGITO-8820] Fix references in timeout guide and create a new section for job service(#320)

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/test-infra repository.

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)
  • [KOGITO-8443] Added note about simplified format for Knative service discovery(#326)
  • Updated with steps to overcome an issue on macOS (#274)
  • [KOGITO-8818] Rearrange guides to add operator and quarkus cloud (#313)
  • [KOGITO-8820] Fix references in timeout guide and create a new section for job service(#320)
  • Updated versions in antora.yml file

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/test-infra repository.

@openshift-ci-robot
Copy link

@VaniHaripriya: This pull request references SRVLOGIC-138 which is a valid jira issue.

In response to this:

Jira : SRVLOGIC-138

Following changes have been added:

  • [KOGITO-8676] OnOverflow management (#315)
  • [KOGITO-8409]Added info about CloudEvents to Knative functions (#312)
  • [KOGITO-8908] Documenting sysout arguments and log level (#321)
  • [KOGITO-8443] Added note about simplified format for Knative service discovery(#326)
  • Updated with steps to overcome an issue on macOS (#274)
  • [KOGITO-8818] Rearrange guides to add operator and quarkus cloud (#313)
  • [KOGITO-8820] Fix references in timeout guide and create a new section for job service(#320)
  • Updated versions in antora.yml file
  • Updated Release notes

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/test-infra repository.

Copy link
Collaborator

@radtriste radtriste left a comment

Choose a reason for hiding this comment

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

Also serverless-logic-web-tools-enable-kogito-swf-visualization.adoc still contain some {context} strings

Copy link
Collaborator

Choose a reason for hiding this comment

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


== Sending as CloudEvent

Knative functions support https://github.com/knative/func/blob/main/docs/function-templates/quarkus.md#invocation-parameters[CloudEvent as the message protocol]. {context} can create and post CloudEvent messages in `functionRef`. For more information see xref:core/custom-functions-support.adoc#sending-cloudevents[]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Knative functions support https://github.com/knative/func/blob/main/docs/function-templates/quarkus.md#invocation-parameters[CloudEvent as the message protocol]. {context} can create and post CloudEvent messages in `functionRef`. For more information see xref:core/custom-functions-support.adoc#sending-cloudevents[]
Knative functions support https://github.com/knative/func/blob/main/docs/function-templates/quarkus.md#invocation-parameters[CloudEvent as the message protocol]. {product_name} can create and post CloudEvent messages in `functionRef`. For more information see xref:core/custom-functions-support.adoc#sending-cloudevents[]

Copy link
Collaborator

Choose a reason for hiding this comment

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

Check for {context} in that file ;)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thank you Tristan, replaced all the {context} to {product_name}.

@VaniHaripriya VaniHaripriya requested a review from radtriste June 2, 2023 15:35
Co-authored-by: Marián Macik <macik.marian@gmail.com>
antora.yml Outdated
operator_version: v1.38.0
kogito_devservices_imagename: registry.redhat.io/openshift-serverless-1-tech-preview/logic-data-index-ephemeral-rhel8:1.24.0-11
kogito_examples_repository_url: 'https://github.com/kiegroup/kogito-examples'
kogito_sw_examples_url: https://github.com/kiegroup/kogito-examples/tree/1.32.x/serverless-workflow-examples
Copy link
Contributor

Choose a reason for hiding this comment

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

This probably needs updating to 1.38 as well @VaniHaripriya ?

antora.yml Outdated
docker_compose_min_version: 1.27.2
operator_version: v1.35.0
operator_version: v1.38.0
kogito_devservices_imagename: registry.redhat.io/openshift-serverless-1-tech-preview/logic-data-index-ephemeral-rhel8:1.24.0-11
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks outdated, but I don't think it's used anymore (as this should be set by default), so maybe best to remove?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@krisv Thank you Kris, I will remove them.

@VaniHaripriya VaniHaripriya requested a review from MarianMacik June 9, 2023 14:31
@radtriste
Copy link
Collaborator

Is that on purpose that the Cloud part is not included into the Serverless Logic but rather a totally different menu on the left panel ?
image

Co-authored-by: Marián Macik <macik.marian@gmail.com>
@VaniHaripriya
Copy link
Collaborator Author

Is that on purpose that the Cloud part is not included into the Serverless Logic but rather a totally different menu on the left panel ? image

Thank you Tristan. Updating the nav doc with correct fromat.

Copy link
Collaborator

@radtriste radtriste left a comment

Choose a reason for hiding this comment

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

one minor comment

@ricardozanini
Copy link
Member

@krisv wanna take a final look?

@radtriste radtriste requested a review from MarianMacik June 23, 2023 06:46
@openshift-ci
Copy link

openshift-ci bot commented Jun 23, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: krisv, MarianMacik, radtriste, ricardozanini, VaniHaripriya

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@radtriste radtriste merged commit 83de874 into openshift-knative:master Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants