openshift-docs icon indicating copy to clipboard operation
openshift-docs copied to clipboard

[SRVLOGIC-469]: Deploying workflows chapter does not use kn worklow plugin

Open domhanak opened this issue 11 months ago • 16 comments
trafficstars

Now it uses the kn workflow plugin

Version(s):

serverless-docs-1.34 serverless-docs-1.35

Issue:

SRVLOGIC-269

Link to docs preview:

Preview

QE review:

  • [ ] QE has approved this change.

Additional information:

@kaldesai @wmedvede @ricardozanini PTAL

domhanak avatar Dec 06 '24 13:12 domhanak

Hi @domhanak. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

openshift-ci[bot] avatar Dec 06 '24 14:12 openshift-ci[bot]

Need to also fix the devmode and the other referenced guide. Ty for review, so far will ping you when I push the changes

domhanak avatar Dec 11 '24 14:12 domhanak

/ok-to-test

kaldesai avatar Jan 09 '25 06:01 kaldesai

🤖 Thu Jan 16 15:24:14 - Prow CI generated the docs preview:

https://85916--ocpdocs-pr.netlify.app/openshift-serverless/latest/serverless-logic/serverless-logic-getting-started/serverless-logic-deploying-workflows.html

ocpdocs-previewbot avatar Jan 09 '25 06:01 ocpdocs-previewbot

@domhanak Once you have drafted the content, you can assign this PR for the peer review process by adding a /label peer-review-needed label to it.

Note: Make sure you have squashed down the commit to only 1.

The OpenShift peer review squad will then peer review this PR, and you can proceed with the merge review process.

kaldesai avatar Jan 15 '25 06:01 kaldesai

@domhanak You want me to put this on the peer review as well?

kaldesai avatar Jan 16 '25 10:01 kaldesai

@kaldesai not yet I still feel like some parts can be enhanced. I ll ping you :)

domhanak avatar Jan 16 '25 14:01 domhanak

@domhanak: all tests passed!

Full PR test history. Your PR dashboard.

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.

openshift-ci[bot] avatar Jan 16 '25 15:01 openshift-ci[bot]

@domhanak Do you want me to put this on the peer review as well?

kaldesai avatar Jan 20 '25 06:01 kaldesai

@kaldesai if you think it is ok please do

domhanak avatar Jan 21 '25 16:01 domhanak

/label peer-review-needed

kaldesai avatar Jan 22 '25 11:01 kaldesai

@adellape You made me think hard about the guide and how it looks as whole. I am trying to improve it

domhanak avatar Feb 12 '25 13:02 domhanak

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

openshift-bot avatar May 14 '25 01:05 openshift-bot

Issue is stale as I do not have the bandwith for this. Either pick this up @kaldesai or close please.

domhanak avatar May 29 '25 07:05 domhanak

Stale issues rot after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity. Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle rotten /remove-lifecycle stale

openshift-bot avatar Jun 28 '25 08:06 openshift-bot

PR needs rebase.

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.

openshift-merge-robot avatar Jun 28 '25 08:06 openshift-merge-robot

Rotten issues close after 30d of inactivity.

Reopen the issue by commenting /reopen. Mark the issue as fresh by commenting /remove-lifecycle rotten. Exclude this issue from closing again by commenting /lifecycle frozen.

/close

openshift-bot avatar Jul 29 '25 00:07 openshift-bot

@openshift-bot: Closed this PR.

In response to this:

Rotten issues close after 30d of inactivity.

Reopen the issue by commenting /reopen. Mark the issue as fresh by commenting /remove-lifecycle rotten. Exclude this issue from closing again by commenting /lifecycle frozen.

/close

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.

openshift-ci[bot] avatar Jul 29 '25 00:07 openshift-ci[bot]