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

Drop core/v1alpha1 API version #7965

Merged
merged 21 commits into from
May 29, 2023
Merged

Conversation

ary1992
Copy link
Contributor

@ary1992 ary1992 commented May 22, 2023

How to categorize this PR?

/area dev-productivity
/kind enhancement

What this PR does / why we need it:
Drop the core/v1alpha1 API version.

Which issue(s) this PR fixes:
Fixes #7358

Special notes for your reviewer:

Release note:

The `core/v1alpha1` API version is dropped. Make sure that you don't use the `core/v1alpha1` API version in your machinery.
The `core/v1alpha1` API version is dropped. Before upgrading to this version, make sure that there are no resources in the etcd stored in the `core/v1alpha1` API version. Otherwise, the gardener-apiserver@v1.72.0 will fail to start.

@gardener-prow gardener-prow bot added area/dev-productivity Developer productivity related (how to improve development) kind/enhancement Enhancement, improvement, extension cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. labels May 22, 2023
@gardener-prow gardener-prow bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label May 22, 2023
@ary1992
Copy link
Contributor Author

ary1992 commented May 22, 2023

/retest

@gardener-prow gardener-prow bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 22, 2023
@shafeeqes
Copy link
Contributor

/assign

Copy link
Contributor

@shafeeqes shafeeqes left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

  • example/81-exposureclass.yaml and docs/usage/exposureclasses.md still uses v1alpha1.
  • .golangci.yaml file needs to be adapted.
  • Also, gardenlet is stuck at
{"level":"info","ts":"2023-05-23T03:41:50.875Z","msg":"Waiting until the namespace has been cleaned up and deleted in the Seed cluster","controller":"shoot","object":{"name":"local","namespace":"garden-local"},"namespace":"garden-local","name":"local","reconcileID":"c3691032-a9ae-4d35-8498-9fe7a3812f4d","operation":"delete","namespaceName":"shoot--local--local"}

during deletion.
Checking the shoot namespace,

  - lastTransitionTime: "2023-05-23T03:40:20Z"
    message: 'Discovery failed for some groups, 1 failing: unable to retrieve the
      complete list of server APIs: core.gardener.cloud/v1alpha1: the server is currently
      unable to handle the request'
    reason: DiscoveryFailed
    status: "True"
    type: NamespaceDeletionDiscoveryFailure

@gardener-prow gardener-prow bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 23, 2023
@ary1992
Copy link
Contributor Author

ary1992 commented May 23, 2023

/retest

@ary1992
Copy link
Contributor Author

ary1992 commented May 25, 2023

/retest

Copy link
Contributor

@shafeeqes shafeeqes left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label May 26, 2023
@gardener-prow
Copy link
Contributor

gardener-prow bot commented May 26, 2023

LGTM label has been added.

Git tree hash: fff2099004c592cb4d23dfba7b89a84f25eb45c9

@shafeeqes
Copy link
Contributor

/approve

@gardener-prow
Copy link
Contributor

gardener-prow bot commented May 29, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: shafeeqes

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

@gardener-prow gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 29, 2023
@gardener-prow gardener-prow bot merged commit 70e54b2 into gardener:master May 29, 2023
16 checks passed
andrerun pushed a commit to andrerun/gardener that referenced this pull request Jul 6, 2023
* BackupBucket

* BackupEntry

* ControllerDeployment

* ControllerInstallation

* ControllerRegistration

* ExposureClass

* SecretBinding

* Quota

* ShootState

* Shoot

* Project

* Seed

* CloudProfile

* Drop pkg/apis/core/v1aplha1 folder

* Adapt Bastion

* Remove v1alpha1 occurances

* Update update-codegen.sh

* make generate

* Update Skaffold

* Delete apiservice-v1alpha1-core-gardener-cloud.yaml

* Address PR review feedback
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. area/dev-productivity Developer productivity related (how to improve development) cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop core/v1alpha1 API version
2 participants