Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RHDEVDOCS 5854 Release Notes for Pipelines 1.13.1 #69377

Merged
merged 1 commit into from
Dec 15, 2023
Merged

RHDEVDOCS 5854 Release Notes for Pipelines 1.13.1 #69377

merged 1 commit into from
Dec 15, 2023

Conversation

mramendi
Copy link
Contributor

@mramendi mramendi commented Dec 14, 2023 •
edited
Loading

Version(s):

please cp to pipelines-docs-1.13

Issue:

RHDEVDOCS 5854

Link to docs preview:

https://69377--docspreview.netlify.app/openshift-pipelines/latest/about/op-release-notes#release-notes-1-13-1_op-release-notes

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 14, 2023
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Dec 14, 2023 •
edited
Loading

🤖 Updated build preview is available at:
https://69377--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/36812

vdemeester
vdemeester reviewed Dec 14, 2023
View reviewed changes
modules/op-release-notes-1-13.adoc Outdated Show resolved Hide resolved
modules/op-release-notes-1-13.adoc Outdated Show resolved Hide resolved
modules/op-release-notes-1-13.adoc Outdated Show resolved Hide resolved
@mramendi
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 15, 2023
jkandasa
jkandasa approved these changes Dec 15, 2023
View reviewed changes
Copy link

@jkandasa jkandasa left a comment

Choose a reason for hiding this comment

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

LGTM

@sheriff-rh sheriff-rh added peer-review-in-progress Signifies that the peer review team is reviewing this PR pipelines and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Dec 15, 2023
sheriff-rh
sheriff-rh reviewed Dec 15, 2023
View reviewed changes
Copy link
Contributor

@sheriff-rh sheriff-rh left a comment

Choose a reason for hiding this comment

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

The content looks good - please squash to one commit prior to merge. Thanks!

@sheriff-rh sheriff-rh added this to the Continuous Release milestone Dec 15, 2023
@sheriff-rh sheriff-rh added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Dec 15, 2023
@mramendi mramendi force-pushed the pipelines-1.13.1-RN branch from b683302 to c34597f Compare December 15, 2023 15:10
@mramendi
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 15, 2023
@mramendi
Copy link
Contributor Author

/label hold

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Dec 15, 2023

@mramendi: The label(s) /label hold cannot be applied. These labels are supported: acknowledge-critical-fixes-only, platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, px-approved, docs-approved, qe-approved, downstream-change-needed, rebase/manual, approved, backport-risk-assessed, bugzilla/valid-bug, cherry-pick-approved, cloud-experts, cnv, dev-tools, distributed-tracing, ims, jira/valid-bug, merge-review-in-progress, merge-review-needed, mtc, multi-arch, oadp, peer-review-done, peer-review-in-progress, peer-review-needed, rhacs, rhv, sd-docs, serverless, service-mesh, sme-review-done, sme-review-needed, staff-eng-approved, telco. Is this label configured under labels -> additional_labels or labels -> restricted_labels in plugin.yaml?

In response to this:

/label hold

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.

@mramendi
Copy link
Contributor Author

/remove-label merge-review-needed

@openshift-ci openshift-ci bot removed the merge-review-needed Signifies that the merge review team needs to review this PR label Dec 15, 2023
@mramendi
Copy link
Contributor Author

/label merge-review-needed

@mramendi
Copy link
Contributor Author

Engineering approved merging at this point

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Dec 15, 2023
@sheriff-rh sheriff-rh merged commit ac90442 into openshift:pipelines-docs Dec 15, 2023
1 check passed
@sheriff-rh
Copy link
Contributor

/cherrypick pipelines-docs-1.13

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Dec 15, 2023
Merged
@openshift-cherrypick-robot
Copy link

@sheriff-rh: new pull request created: #69488

In response to this:

/cherrypick pipelines-docs-1.13

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.

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

@sheriff-rh sheriff-rh sheriff-rh left review comments

@concaf concaf concaf approved these changes

@jkandasa jkandasa jkandasa approved these changes

@ppitonak ppitonak ppitonak approved these changes

@vdemeester vdemeester vdemeester approved these changes

Assignees
No one assigned
Labels
merge-review-needed Signifies that the merge review team needs to review this PR peer-review-done Signifies that the peer review team has reviewed this PR pipelines size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
Continuous Release
Development

Successfully merging this pull request may close these issues.
8 participants
@mramendi @ocpdocs-previewbot @sheriff-rh @openshift-cherrypick-robot @vdemeester @ppitonak @jkandasa @concaf