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

Remove v1alpha4 CRDs, types and associated code #453

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

thunderboltsid
Copy link
Contributor

@thunderboltsid thunderboltsid commented Jul 4, 2024

v1alpha4 has been deprecated and unserved in the past releases. This removes v1alpha4 and all supporting test code for v1alpha4. We are not aware of anyone using v1alpha4 APIs.

@thunderboltsid thunderboltsid changed the title Remove v1alpha4 API and associated code Remove v1alpha4 CRDs, types and associated code Jul 4, 2024
@thunderboltsid thunderboltsid force-pushed the issue/remove-v1alpha4 branch from 3d2b6f7 to 52fe69a Compare July 4, 2024 18:28
@thunderboltsid thunderboltsid requested a review from adiantum July 4, 2024 18:28
Copy link

codecov bot commented Jul 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 31.69%. Comparing base (7a88f97) to head (1b6fee9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #453      +/-   ##
==========================================
- Coverage   31.74%   31.69%   -0.05%     
==========================================
  Files          14       14              
  Lines        1367     1366       -1     
==========================================
- Hits          434      433       -1     
  Misses        933      933              

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

@thunderboltsid thunderboltsid requested a review from jimmidyson July 4, 2024 19:04
@thunderboltsid
Copy link
Contributor Author

/retest

@thunderboltsid thunderboltsid requested a review from dkoshkin July 4, 2024 19:07
@nutanix-cn-prow-bot nutanix-cn-prow-bot bot requested a review from tuxtof July 5, 2024 00:25
@thunderboltsid thunderboltsid force-pushed the issue/remove-v1alpha4 branch from 52fe69a to f3403cd Compare July 5, 2024 07:38
@thunderboltsid
Copy link
Contributor Author

/retest

@nutanix-cn-prow-bot
Copy link

@thunderboltsid: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-capx-controller-upgrade f3403cd link true /test e2e-capx-controller-upgrade
ci/prow/e2e-k8s-upgrade f3403cd link true /test e2e-k8s-upgrade
ci/prow/e2e-capx-clusterclass f3403cd link true /test e2e-capx-clusterclass
ci/prow/e2e-capx-scaling f3403cd link true /test e2e-capx-scaling
ci/prow/e2e-nutanix-features f3403cd link true /test e2e-nutanix-features
ci/prow/e2e-k8s-conformance f3403cd link true /test e2e-k8s-conformance

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/test-infra repository. I understand the commands that are listed here.

@thunderboltsid
Copy link
Contributor Author

/retest

@thunderboltsid thunderboltsid removed the request for review from tuxtof July 5, 2024 10:52
@thunderboltsid
Copy link
Contributor Author

thunderboltsid commented Jul 5, 2024

/hold putting on hold until GitHub Actions are fixed

@thunderboltsid thunderboltsid force-pushed the issue/remove-v1alpha4 branch from 6703a1c to f8dca6c Compare July 9, 2024 11:23
Remove all supporting test code for v1alpha4
@thunderboltsid thunderboltsid merged commit 85e5d47 into main Jul 10, 2024
10 of 11 checks passed
@thunderboltsid thunderboltsid deleted the issue/remove-v1alpha4 branch July 10, 2024 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants