Skip to content

Tnc7.0 release 4.22 - #971

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-kni:tnc7.0-release-4.22from
a1bokhari:tnc7.0-release-4.22
Sep 22, 2026
Merged

Tnc7.0 release 4.22#971
openshift-merge-bot[bot] merged 1 commit into
openshift-kni:tnc7.0-release-4.22from
a1bokhari:tnc7.0-release-4.22

Conversation

@a1bokhari

@a1bokhari a1bokhari commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: bcff7219-78c1-4b51-b8f5-44feb4be34a8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 16, 2026
snecklifter
snecklifter suggested changes Sep 17, 2026
View reviewed changes
Comment thread tnc-reference/tnc-mgmt-ztp/install/mgmt1.yaml Outdated
clusterImageSetNameRef: "openshift-4.18-disconnected"
sshPublicKey: "ssh-rsa AAAA...."
name: "assisted-deployment-pull-secret"
clusterImageSetNameRef: "openshift-4.18-disconnected"

@snecklifter snecklifter Sep 17, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@a1bokhari I think this should be 4.22?

@a1bokhari a1bokhari Sep 17, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Changed. Thanks1

@openshift-ci

openshift-ci Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

@snecklifter: changing LGTM is restricted to collaborators

Details

In response to this:

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.

snecklifter
snecklifter suggested changes Sep 17, 2026
View reviewed changes
Comment thread tnc-reference/tnc-cwl/configuration/core-baseline.yaml Outdated
namespace: openshift-logging
# Update with specific target version
name: cluster-logging.v6.2.11
name: cluster-logging.v6.5.1

@snecklifter snecklifter Sep 17, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I believe this needs to be 6.6 for OpenShift 4.22

@a1bokhari a1bokhari Sep 17, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Updated

@openshift-ci

openshift-ci Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

@snecklifter: changing LGTM is restricted to collaborators

Details

In response to this:

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.

snecklifter
snecklifter suggested changes Sep 18, 2026
View reviewed changes
Comment thread tnc-reference/tnc-cwl/configuration/core-baseline.yaml Outdated
namespace: openshift-logging
# Update with specific target version
name: cluster-logging.v6.2.11
name: cluster-logging.v6.6.0

@snecklifter snecklifter Sep 18, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nit: Should probably be cluster-logging.v6.6.1 really

@openshift-ci

openshift-ci Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

@snecklifter: changing LGTM is restricted to collaborators

Details

In response to this:

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.

clindo84
clindo84 reviewed Sep 21, 2026
View reviewed changes
Comment thread tnc-reference/tnc-cwl/configuration/upgrade-422.adoc Outdated

@clindo84 clindo84 Sep 21, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Requesting to delete this one since the upgrade guide can be referenced in a different location.

@a1bokhari a1bokhari Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Deleted

clindo84
clindo84 reviewed Sep 21, 2026
View reviewed changes
Comment thread tnc-reference/tnc-mgmt-ztp/configuration/upgrade-mgmt-422.adoc Outdated

@clindo84 clindo84 Sep 21, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Requesting to delete this one since the upgrade guide can be referenced in a different location.

@a1bokhari a1bokhari Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Deleted

@clindo84
clindo84 self-requested a review September 22, 2026 14:01
clindo84
clindo84 requested changes Sep 22, 2026
View reviewed changes
Comment thread tnc-reference/tnc-cwl/install/ns/cw-test1.yaml Outdated

@clindo84 clindo84 Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should we remove this test namespace CR?

@a1bokhari a1bokhari Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Removed

Ali Bokhari
removee 4.20 submodules
9af6db2 
Pick up 4.22 RDS submodules

TNC 7.0

Change cluster image to 4.22 - per snecklifter review comment

update openshift-logging and loki to 6.6

Fix MCP pausing for MGMT cluster upgrade

changing logging z-stream to 6.6.1

remove upgrade docs. Content moved to TNC 7.0 upgrade guide

remove test ns
clindo84
clindo84 approved these changes Sep 22, 2026
View reviewed changes

@clindo84 clindo84 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving.

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Sep 22, 2026
@openshift-ci

openshift-ci Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: a1bokhari, clindo84

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit c2a5b92 into openshift-kni:tnc7.0-release-4.22 Sep 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@clindo84 clindo84 clindo84 approved these changes
@sollinsnaya sollinsnaya Awaiting requested review from sollinsnaya sollinsnaya is a code owner
@Ladanow Ladanow Awaiting requested review from Ladanow Ladanow is a code owner
@git-shassan git-shassan Awaiting requested review from git-shassan git-shassan is a code owner
@imiller0 imiller0 Awaiting requested review from imiller0 imiller0 is a code owner
+1 more reviewer
@snecklifter snecklifter snecklifter requested changes
Reviewers whose approvals may not affect merge requirements

Assignees

@clindo84 clindo84

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@a1bokhari @snecklifter @clindo84