Skip to content

Conversation

sjhala-ccs
Copy link
Contributor

@sjhala-ccs sjhala-ccs commented Oct 8, 2020

First draft of the 2.5 release notes

Please note that the virt-2-4-release-notes.adoc file did not have any content so I had rewrite it using git checkout enterprise-4.5 virt-2-4-release-notes.adoc before cloning it.

Also added a missing module virt-supported-cluster-version.adoc

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 8, 2020
@sjhala-ccs sjhala-ccs force-pushed the cnv-2.5-release-notes branch 2 times, most recently from c1d1718 to 165f4c2 Compare October 8, 2020 14:43
@openshift-docs-preview-bot

The preview will be available shortly at:

Copy link
Contributor

Choose a reason for hiding this comment

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

Which reminds me, we should add a dummy 2.5 file in the master branch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

PR to add the dummy 2.5 file: #26245

Copy link
Contributor

Choose a reason for hiding this comment

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

We can uncomment this include now.
It might also be worthwhile removing the first sentence in L9 since it's redundant (and this module uses version attribute so won't run into maintenance issues) but let's see how it looks for now. It might not be a bad thing to repeat.

@sjhala-ccs sjhala-ccs force-pushed the cnv-2.5-release-notes branch from 165f4c2 to e7c2044 Compare October 8, 2020 17:56
@sjhala-ccs sjhala-ccs force-pushed the cnv-2.5-release-notes branch from e7c2044 to 39352a3 Compare October 8, 2020 18:09
@aburdenthehand aburdenthehand added the CNV Label for all CNV PRs label Oct 9, 2020
@aburdenthehand aburdenthehand added this to the Future Release milestone Oct 9, 2020
@aburdenthehand
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 9, 2020
@aburdenthehand aburdenthehand merged commit 76e1f8f into openshift:enterprise-4.6 Oct 9, 2020
@sjhala-ccs sjhala-ccs deleted the cnv-2.5-release-notes branch October 9, 2020 14:23
Red Hat {VirtProductName} {VirtVersion} is supported for use on {product-title} 4.6 clusters. {VirtProductName} enables you to bring traditional virtual machines (VMs) into {product-title} where they run alongside containers, and are managed as native Kubernetes objects.

//Branding-Logo
{VirtProductName} is represented by the image:Operator_Icon-OpenShift_Virtualization-5.png[{VirtProductName}] logo.
Copy link
Contributor

Choose a reason for hiding this comment

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

@sjhala-ccs @aburdenthehand - this image doesn't exist. Can you add this image or remove this line?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CNV Label for all CNV PRs lgtm Indicates that a PR is ready to be merged. 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.

5 participants