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

Set OwnerRef on TrustBundle configmaps and add selfhosted E2E test #438

Merged
merged 1 commit into from
May 30, 2024

Conversation

thunderboltsid
Copy link
Contributor

@thunderboltsid thunderboltsid commented May 28, 2024

This PR adds a E2E test for clusterctl move and also adds a method to reconcile trust bundle refs in NutanixmachineReconciler to set ownerReference on trust bundle configmaps so those configmaps get moved when clusterctl move operation happens.

@thunderboltsid
Copy link
Contributor Author

/retest

Copy link

codecov bot commented May 28, 2024

Codecov Report

Attention: Patch coverage is 89.28571% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 29.65%. Comparing base (0fdeb83) to head (0100072).

Files Patch % Lines
controllers/nutanixcluster_controller.go 87.50% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #438      +/-   ##
==========================================
+ Coverage   28.37%   29.65%   +1.28%     
==========================================
  Files          14       14              
  Lines        1304     1332      +28     
==========================================
+ Hits          370      395      +25     
- Misses        934      937       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Makefile Outdated Show resolved Hide resolved
@thunderboltsid thunderboltsid force-pushed the jira/krbn-8210 branch 2 times, most recently from 773cbaf to 9e3a2fb Compare May 29, 2024 13:14
@thunderboltsid thunderboltsid changed the title Add Clusterctl Move E2E test Add reconciliation logic for trust bundle configmaps and clusterctl move E2E test May 29, 2024
@thunderboltsid thunderboltsid changed the title Add reconciliation logic for trust bundle configmaps and clusterctl move E2E test Set OwnerRef on TrustBundle configmaps and add selhosted E2E test May 29, 2024
@thunderboltsid thunderboltsid changed the title Set OwnerRef on TrustBundle configmaps and add selhosted E2E test Set OwnerRef on TrustBundle configmaps and add selfhosted E2E test May 29, 2024
Copy link
Contributor

@adiantum adiantum left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@thunderboltsid
Copy link
Contributor Author

/retest

@thunderboltsid
Copy link
Contributor Author

/retest

Makefile Show resolved Hide resolved
@thunderboltsid thunderboltsid force-pushed the jira/krbn-8210 branch 3 times, most recently from 1302d8d to 85b3f8d Compare May 29, 2024 20:51
@thunderboltsid
Copy link
Contributor Author

/retest

Copy link
Contributor

@deepakm-ntnx deepakm-ntnx left a comment

Choose a reason for hiding this comment

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

we need to fix the codecov errors

Also set ownerReference on trust bundle configmaps so CMs get moved
when clusterctl move happens.
@thunderboltsid
Copy link
Contributor Author

/retest

Copy link
Contributor

@adiantum adiantum left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@nutanix-cn-prow-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adiantum, deepakm-ntnx, thunderboltsid

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:
  • OWNERS [adiantum,deepakm-ntnx,thunderboltsid]

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

@thunderboltsid thunderboltsid merged commit 37d75eb into main May 30, 2024
12 of 13 checks passed
@thunderboltsid thunderboltsid deleted the jira/krbn-8210 branch May 30, 2024 10:33
thunderboltsid added a commit that referenced this pull request May 30, 2024
Also set ownerReference on trust bundle configmaps so CMs get moved
when clusterctl move happens.
thunderboltsid added a commit that referenced this pull request May 30, 2024
Also set ownerReference on trust bundle configmaps so CMs get moved
when clusterctl move happens.
@thunderboltsid thunderboltsid added the bug Something isn't working label May 30, 2024
thunderboltsid added a commit that referenced this pull request May 31, 2024
Also set ownerReference on trust bundle configmaps so CMs get moved
when clusterctl move happens.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved bug Something isn't working lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants