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

Update ocp-4-15-release-notes, since there is no plan to remove icsp in future release.

Open bshaw7 opened this issue 1 year ago • 15 comments

SInce there is no plan to remove ICSP object hence we need to update the release notes accordingly.

From this: Using an ICSP object to configure repository mirroring is a deprecated feature. Deprecated functionality is still included in OpenShift Container Platform and continues to be supported; however, it will be removed in a future release of this product and is not recommended for new deployments.

To this: Using an ICSP object to configure repository mirroring is a deprecated feature. Deprecated functionality is still included in OpenShift Container Platform and continues to be supported; however, It might be removed in a future release of this product. Because it is deprecated functionality, avoid using it for new deployments.

Version(s): 4.13, 4.14, 4.15

Issue: https://issues.redhat.com/browse/OCPBUGS-36519

Link to docs preview: https://docs.redhat.com/en/documentation/openshift_container_platform/4.15/html/release_notes/ocp-4-15-release-notes#ocp-4-15-nodes-icsp-idms-compatibility

OpenShift Container Platform 4.15 release notes > 1.3. New features and enhancements > 1.3.17. Nodes > 1.3.17.3. ICSP, IDMS, and ITMS are now compatible

QE review:

bshaw7 avatar Jul 03 '24 17:07 bshaw7

/label peer-review-needed

bshaw7 avatar Jul 03 '24 17:07 bshaw7

🤖 Thu Jul 04 12:28:57 - Prow CI generated the docs preview:

https://78465--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-15-release-notes.html

ocpdocs-previewbot avatar Jul 03 '24 17:07 ocpdocs-previewbot

@mburke5678, will you please get this issue validated and through CM?

kalexand-rh avatar Jul 03 '24 18:07 kalexand-rh

Hello @gauravsingh85 , Can you help with the validation on this PR to @mburke5678 and @kalexand-rh

bshaw7 avatar Jul 03 '24 19:07 bshaw7

The future is long, we should not commit to this.

zaneb avatar Jul 03 '24 21:07 zaneb

hey @zaneb now the statement is - "it may be removed in a future release of this product and is not recommended for new deployments" Can you review once.

bshaw7 avatar Jul 04 '24 04:07 bshaw7

I'm OK with this. Note that we explicitly document this as a Tier 1 API, meaning that we have committed not to removing it within a major release. So if you want to be more specific you can say that.

zaneb avatar Jul 04 '24 04:07 zaneb

@bshaw7 Are you sure you want this PR to be CP-ed only in 4.13, 4.14, and 4.15? Not to later versions? Please clarify. TY!

Srivaralakshmi avatar Jul 04 '24 10:07 Srivaralakshmi

@Srivaralakshmi Yes only for 4.13, 4.14, and 4.15. It's not mention in 4.16 release notes.

bshaw7 avatar Jul 04 '24 10:07 bshaw7

@Srivaralakshmi we should go with below only. Instead of saying 'Do not use'

Using an ICSP object to configure repository mirroring is a deprecated feature. Deprecated functionality is still included in {product-title} and continues to be supported. However, it might be removed in a future release of this product. It is not recommended to use deprecated functionality for new deployments.

bshaw7 avatar Jul 04 '24 11:07 bshaw7

@Srivaralakshmi updated as per suggestion.

bshaw7 avatar Jul 04 '24 12:07 bshaw7

/label merge-review-needed

bshaw7 avatar Jul 04 '24 12:07 bshaw7

@bshaw7: 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 Jul 04 '24 12:07 openshift-ci[bot]

@gauravsingh85 @nee1esh @control-d @rphillips @arnaik-rh PTAL

mburke5678 avatar Jul 04 '24 13:07 mburke5678

Looks like we need a few more acks before we can merge. I see @mburke5678 has tagged them. I am removing the merge review needed label for now.

opayne1 avatar Jul 05 '24 11:07 opayne1

LGTM

gauravsingh85 avatar Jul 05 '24 17:07 gauravsingh85

Adding @sunilcio . Please review.

arnaik-rh avatar Jul 08 '24 09:07 arnaik-rh

LGTM

nee1esh avatar Jul 08 '24 14:07 nee1esh

LGTM

control-d avatar Jul 08 '24 18:07 control-d

/lgtm

sunilcio avatar Jul 09 '24 04:07 sunilcio

/label merge-review-needed

bshaw7 avatar Jul 09 '24 04:07 bshaw7

/cherrypick enterprise-4.14

sheriff-rh avatar Jul 09 '24 12:07 sheriff-rh

/cherrypick enterprise-4.13

sheriff-rh avatar Jul 09 '24 12:07 sheriff-rh

@sheriff-rh: #78465 failed to apply on top of branch "enterprise-4.14":

Applying: Update ocp-4-15-release-notes.adoc
Using index info to reconstruct a base tree...
A	release_notes/ocp-4-15-release-notes.adoc
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): release_notes/ocp-4-15-release-notes.adoc deleted in HEAD and modified in Update ocp-4-15-release-notes.adoc. Version Update ocp-4-15-release-notes.adoc of release_notes/ocp-4-15-release-notes.adoc left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Update ocp-4-15-release-notes.adoc
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick 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-sigs/prow repository.

@sheriff-rh: #78465 failed to apply on top of branch "enterprise-4.13":

Applying: Update ocp-4-15-release-notes.adoc
Using index info to reconstruct a base tree...
A	release_notes/ocp-4-15-release-notes.adoc
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): release_notes/ocp-4-15-release-notes.adoc deleted in HEAD and modified in Update ocp-4-15-release-notes.adoc. Version Update ocp-4-15-release-notes.adoc of release_notes/ocp-4-15-release-notes.adoc left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Update ocp-4-15-release-notes.adoc
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick enterprise-4.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-sigs/prow repository.

Well that was silly of me to try and cherrypick - @bshaw7 you will have to manually modify 4.14 and 4.13's release notes with this text. Be sure to reference this PR in the top comment so it can be tracked.

sheriff-rh avatar Jul 09 '24 12:07 sheriff-rh

@sheriff-rh created for 4.14 and 4.13 https://github.com/openshift/openshift-docs/pull/78638 https://github.com/openshift/openshift-docs/pull/78640

bshaw7 avatar Jul 09 '24 14:07 bshaw7