Skip to content

Release note addition 4.14: Add S2I ga release note item#67006

Merged
bscott-rh merged 1 commit intoopenshift:enterprise-4.14from
nebelsprachen:5614-S2I-build
Oct 27, 2023
Merged

Release note addition 4.14: Add S2I ga release note item#67006
bscott-rh merged 1 commit intoopenshift:enterprise-4.14from
nebelsprachen:5614-S2I-build

Conversation

@nebelsprachen
Copy link

@nebelsprachen nebelsprachen commented Oct 27, 2023 •
edited by bscott-rh
Loading

@nebelsprachen nebelsprachen changed the title Add S2I ga release note item Release note addition 4.14: Add S2I ga release note item Oct 27, 2023
@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 27, 2023
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 27, 2023 •
edited
Loading

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

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

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 27, 2023
@nebelsprachen
Copy link
Author

nebelsprachen commented Oct 27, 2023

/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 Oct 27, 2023
@nebelsprachen
Copy link
Author

nebelsprachen commented Oct 27, 2023

/label enterprise-4.14

@openshift-ci
Copy link

openshift-ci bot commented Oct 27, 2023

@joaedwar: The label(s) /label enterprise-4.14 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?

Details

In response to this:

/label enterprise-4.14

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.

@mjpytlak mjpytlak added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Oct 27, 2023
mjpytlak
mjpytlak reviewed Oct 27, 2023
View reviewed changes
Copy link
Contributor

@mjpytlak mjpytlak left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@joaedwar Content looks good, but have a few questions. Please also include a link to the preview build.

Also looks like you still need QE approval.

release_notes/ocp-4-14-release-notes.adoc
=== Builds

The Source-to-Image (S2I) tool is now generally available in {product-title} {product-version}. With this update, you can use the S2I tool to build container images from source code, and transform application code into ready-to-deploy container images. This feature enhances the platform's ability to support reproducible containerized application development.

Copy link
Contributor

@mjpytlak mjpytlak Oct 27, 2023

Choose a reason for hiding this comment

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

Do the entries for S2I and BuildCSIVolume each require there own heading? Asking, as these seem to be two distinct features being released.

release_notes/ocp-4-14-release-notes.adoc Outdated

For more information, see link:https://access.redhat.com/documentation/en-us/red_hat_software_collections/2/html/using_red_hat_software_collections_container_images/sti#doc-wrapper[Using Source-to-Image (S2I) tool].

BuildCSIVolume is now generally available in {product-title} {product-version}.
Copy link
Contributor

@mjpytlak mjpytlak Oct 27, 2023

Choose a reason for hiding this comment

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

Should BuildCSIVolume be Build CSI Volumes? As that is how you have referenced the feature in the Tech Preview table.

@mjpytlak mjpytlak 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 Oct 27, 2023
@jitendar-singh
Copy link

jitendar-singh commented Oct 27, 2023

/label qe-approved

@openshift-ci openshift-ci bot added the qe-approved Signifies that QE has signed off on this PR label Oct 27, 2023
@nebelsprachen
Copy link
Author

nebelsprachen commented Oct 27, 2023

/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 Oct 27, 2023
@bscott-rh bscott-rh 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 Oct 27, 2023
Add S2I release
2a18b9c 
rebase and move builds location

minor move for terminal

fix error

update link

add to table

BuildCSIVolume is available

confirmation

bulleted

copy edit
@bscott-rh bscott-rh merged commit 73daa9c into openshift:enterprise-4.14 Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mjpytlak mjpytlak mjpytlak left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

branch/enterprise-4.14 merge-review-in-progress Signifies that the merge review team is reviewing this PR peer-review-done Signifies that the peer review team has reviewed this PR qe-approved Signifies that QE has signed off on this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@nebelsprachen @ocpdocs-previewbot @jitendar-singh @mjpytlak @Preeticp @bscott-rh