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.

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

CNF-14537: Ensure Cluster deploy and config status check on input issues during resource preparation #205

Merged

Conversation

Missxiaoguo
Copy link
Collaborator

@Missxiaoguo Missxiaoguo commented Sep 16, 2024

Currently, invalid configuration input causes resource preparation to fail, stopping the controller reconciliation. However, if Cluster deployment resources (NodePool, ClusterInstance, or Policies) have already been created and are in progress, subsequent invalid configuration changes should not prevent the controller from continuing to track the current Cluster deployment status.

This update ensures that the controller checks the Cluster deploy and config status when an input issue occurs during resource preparation before stopping reconciliation.

Unittests are added for ClusterRequest controller reconciliation to verify desired conditions.

@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Sep 16, 2024

@Missxiaoguo: This pull request references CNF-14537 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.

In response to this:

…resource preparation

Currently, invalid configuration input causes resource preparation to fail, stopping the controller reconciliation. However, if Cluster deployment resources (NodePool, ClusterInstance, or Policies) have already been created and are in progress, subsequent invalid configuration changes should not prevent the controller from continuing to track the current Cluster deployment status.

This update ensures that the controller checks the Cluster deploy and config status when an input issue occurs during resource preparation before stopping reconciliation.

Unittests are added for ClusterRequest controller reconciliation to verify desired conditions.

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 openshift-eng/jira-lifecycle-plugin repository.

@Missxiaoguo Missxiaoguo changed the title /CNF-14537: Ensure Cluster deploy and config status check on input issues during … CNF-14537: Ensure Cluster deploy and config status check on input issues during … Sep 16, 2024
@Missxiaoguo Missxiaoguo removed the request for review from pixelsoccupied September 16, 2024 20:29
t.object.Status.ClusterInstanceRef.Name = clusterInstance.GetName()
}

// Continune checking the existing ClusterInstance provision status
Copy link
Collaborator

Choose a reason for hiding this comment

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

Typo Continue

…resource preparation

Currently, invalid configuration input causes resource preparation to fail, stopping the
controller reconciliation. However, if Cluster deployment resources (NodePool, ClusterInstance,
or Policies) have already been created and are in progress, subsequent invalid configuration
changes should not prevent the controller from continuing to track the current Cluster deployment status.

This update ensures that the controller checks the Cluster deploy and config status
when an input issue occurs during resource preparation before stopping reconciliation.

Unittests are added for ClusterRequest controller reconciliation to verify desired conditions.
@bartwensley
Copy link
Collaborator

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 17, 2024
@Missxiaoguo Missxiaoguo changed the title CNF-14537: Ensure Cluster deploy and config status check on input issues during … CNF-14537: Ensure Cluster deploy and config status check on input issues during resource preparation Sep 17, 2024
@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Sep 17, 2024

@Missxiaoguo: This pull request references CNF-14537 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.

In response to this:

Currently, invalid configuration input causes resource preparation to fail, stopping the controller reconciliation. However, if Cluster deployment resources (NodePool, ClusterInstance, or Policies) have already been created and are in progress, subsequent invalid configuration changes should not prevent the controller from continuing to track the current Cluster deployment status.

This update ensures that the controller checks the Cluster deploy and config status when an input issue occurs during resource preparation before stopping reconciliation.

Unittests are added for ClusterRequest controller reconciliation to verify desired conditions.

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 openshift-eng/jira-lifecycle-plugin repository.

@irinamihai
Copy link
Collaborator

/approve

Copy link

openshift-ci bot commented Sep 17, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: irinamihai

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

The pull request process is described here

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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 17, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit ad16503 into openshift-kni:main Sep 17, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

4 participants