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

Migration to networking.k8s.io/v1beta1 #417

Merged
merged 1 commit into from
Apr 9, 2021
Merged

Conversation

kuritka
Copy link
Collaborator

@kuritka kuritka commented Mar 30, 2021

closes #401

k0da
k0da previously approved these changes Mar 30, 2021
Copy link
Collaborator

@k0da k0da left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

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

How the k8gb upgrade will look like with the CRD change? Can we please test permutation?

@kuritka kuritka force-pushed the bump-to-networking-v1beta1 branch from 665cacc to 0e067ff Compare April 6, 2021 07:27
@kuritka
Copy link
Collaborator Author

kuritka commented Apr 6, 2021

@ytsarev, necessary CRD change was implemented by #425. Would be happy to implement permutation testing in scope of #349 .

@kuritka kuritka requested review from ytsarev and k0da April 6, 2021 09:43
k0da
k0da previously approved these changes Apr 6, 2021
@ytsarev
Copy link
Member

ytsarev commented Apr 7, 2021

@kuritka it's an api group change, if the automation is not yet there we need to at least test the upgrade from previous version manually

@kuritka kuritka force-pushed the bump-to-networking-v1beta1 branch from 0e067ff to 3949923 Compare April 9, 2021 11:00
 - migration gslb_types and upstream_types to `networking/v1beta1`
 - migration yamls in terratest, generates warnings see: [terratest #840](gruntwork-io/terratest#840 (comment))
 - migration apiGroup in roles
 - **CRD change**

closes #401

Signed-off-by: kuritka <[email protected]>
@kuritka kuritka force-pushed the bump-to-networking-v1beta1 branch from 3949923 to 74c3397 Compare April 9, 2021 11:01
@kuritka
Copy link
Collaborator Author

kuritka commented Apr 9, 2021

@kuritka it's an api group change, if the automation is not yet there we need to at least test the upgrade from previous version manually

@ytsarev , upgrade test + local playground test passed

@kuritka kuritka merged commit 8b48d7d into master Apr 9, 2021
@kuritka kuritka deleted the bump-to-networking-v1beta1 branch April 9, 2021 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

automate ingress validation annotation
3 participants