Skip to content

Conversation

eromanova97
Copy link
Contributor

@eromanova97 eromanova97 commented Oct 29, 2024

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 29, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 29, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:
*

QE review:

  • QE has approved this change.

Additional information:

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 Oct 29, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 30, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@eromanova97
Copy link
Contributor Author

Hello @pratap0007 @ppitonak could you please review? Thank you!

@eromanova97 eromanova97 force-pushed the RHDEVDOCS-6210 branch 2 times, most recently from 83cd01e to 26574a2 Compare November 6, 2024 08:26
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 11, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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-robot
Copy link

openshift-ci-robot commented Nov 12, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@eromanova97 eromanova97 force-pushed the RHDEVDOCS-6210 branch 2 times, most recently from c8dff57 to ceed60e Compare November 12, 2024 13:19
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 12, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@savitaashture
Copy link

@mramendi @eromanova97 should we add some example as part of this PR to show how a PipelineRun looks with ClusterTasks and before upgrading to 1.17 what steps user has to follow to use tasks using cluster Resolver

@mramendi
Copy link
Contributor

@mramendi @eromanova97 should we add some example as part of this PR to show how a PipelineRun looks with ClusterTasks and before upgrading to 1.17 what steps user has to follow to use tasks using cluster Resolver

Sounds good to me. If you can provide such examples, then we can find a way to add them into the doc. However, we do not have the resources to craft these examples on our own.

@eromanova97 eromanova97 force-pushed the RHDEVDOCS-6210 branch 2 times, most recently from 50a6441 to ef08ef9 Compare November 22, 2024 08:55
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 10, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@eromanova97
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Dec 10, 2024
@adellape adellape self-assigned this Dec 10, 2024
@adellape adellape added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Dec 10, 2024
@openshift-ci openshift-ci bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Dec 11, 2024
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Dec 11, 2024
@eromanova97 eromanova97 force-pushed the RHDEVDOCS-6210 branch 2 times, most recently from fcef987 to 6c2bfb3 Compare December 11, 2024 10:01
@openshift-ci openshift-ci bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Dec 11, 2024
@eromanova97 eromanova97 changed the base branch from pipelines-docs-main to pipelines-docs-1.17 December 11, 2024 10:04
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 11, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@eromanova97 eromanova97 changed the base branch from pipelines-docs-1.17 to pipelines-docs-main December 11, 2024 10:05
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 11, 2024

@eromanova97: This pull request references RHDEVDOCS-6210 which is a valid jira issue.

In response to this:

Version(s): pipelines-docs-1.17

Issue: RHDEVDOCS-6210

Links to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Dec 11, 2024
@eromanova97
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Dec 11, 2024
@eromanova97
Copy link
Contributor Author

To the merge reviewer: I had some rebase issues which messed with the size labels a little bit before I got it fixed, so sorry for that 😅

@ShaunaDiaz ShaunaDiaz added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Dec 11, 2024
Copy link
Contributor

@ShaunaDiaz ShaunaDiaz left a comment

Choose a reason for hiding this comment

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

Some thoughts for the future. LGTM

An {pipelines-shortname} installation includes a set of standard tasks that you can use in your pipelines. These tasks are located in the {pipelines-shortname} installation namespace, which is normally the `openshift-pipelines` namespace. You can use the cluster resolver to access the tasks.

`ClusterTask` functionality is deprecated since {pipelines-shortname} 1.10 and is planned for removal in a future release. If your pipelines use `ClusterTasks`, you can re-create them with the tasks that are available from the {pipelines-shortname} installation namespace by using the cluster resolver. However, certain changes are made in these tasks compared to the existing `ClusterTasks`.
Until version 1.16, {pipelines-shortname} included `ClusterTask` functionality. Versions 1.17 and later no longer include this functionality. If your pipelines use `ClusterTask` references, you can re-create them with the tasks that are available from the {pipelines-shortname} installation namespace by using the cluster resolver. However, certain changes are made in these tasks compared to the previously existing `ClusterTask` definitions.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Until version 1.16, {pipelines-shortname} included `ClusterTask` functionality. Versions 1.17 and later no longer include this functionality. If your pipelines use `ClusterTask` references, you can re-create them with the tasks that are available from the {pipelines-shortname} installation namespace by using the cluster resolver. However, certain changes are made in these tasks compared to the previously existing `ClusterTask` definitions.
Until version 1.16, {pipelines-shortname} included `ClusterTask` functionality. Versions 1.17 and later do not include this functionality. If your pipelines use `ClusterTask` references, you can re-create them with the tasks that are available from the {pipelines-shortname} installation namespace by using the cluster resolver. However, certain changes are made in these tasks compared to the previously existing `ClusterTask` definitions.

I would simplify this at some point for minimalism. It sounds like 1.17 and later used to also have this function, but it was removed ("no longer"). If they never had it, I would just say they don't. Not required for merge, just a thought.


Non-versioned and versioned tasks and step actions have the same metadata, behavior, and specifications, including `params`, `workspaces`, and `steps`. However, they behave differently when you disable them or upgrade the Operator.

Before adopting non-versioned or versioned tasks and step actions as a standard in production environments, cluster administrators might consider their advantages and disadvantages.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Before adopting non-versioned or versioned tasks and step actions as a standard in production environments, cluster administrators might consider their advantages and disadvantages.
Before adopting non-versioned or versioned tasks and step actions as a standard in production environments, cluster administrators should consider their advantages and disadvantages.

Hmm, is it advisable to consider the differences? "Might" doesn't sound like advice, per se; seems very very optional. What's the happy path to direct customers to do?

Non-versioned and versioned tasks and step actions have the same metadata, behavior, and specifications, including `params`, `workspaces`, and `steps`. However, they behave differently when you disable them or upgrade the Operator.

Before adopting non-versioned or versioned tasks and step actions as a standard in production environments, cluster administrators might consider their advantages and disadvantages.

Copy link
Contributor

Choose a reason for hiding this comment

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

IDK what the latest guidance is on TOC level, but it might (ha) be nice to have L3 headings above these table sections for scannability. I am not certain where we are landing with the depth of the TOCs on docs.redhat, though, so take my thought with a grain of salt.

@ShaunaDiaz ShaunaDiaz merged commit 7042f82 into openshift:pipelines-docs-main Dec 11, 2024
2 checks passed
@ShaunaDiaz
Copy link
Contributor

/cherrypick pipelines-docs-1.17

@openshift-cherrypick-robot

@ShaunaDiaz: new pull request created: #86099

In response to this:

/cherrypick pipelines-docs-1.17

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.

@ShaunaDiaz ShaunaDiaz removed the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Dec 11, 2024
@eromanova97 eromanova97 deleted the RHDEVDOCS-6210 branch December 11, 2024 14:13
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. peer-review-done Signifies that the peer review team has reviewed this PR pipelines 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.

10 participants