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

Create go/v4 plugin with alpha1 level to address changes which are breaking ones for go/v3 #2547

Closed
camilamacedo86 opened this issue Mar 18, 2022 · 10 comments · Fixed by #2552
Assignees
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/feature Categorizes issue or PR as related to a new feature. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. triage/accepted Indicates an issue or PR is ready to be actively worked on. triage/blocked
Milestone

Comments

@camilamacedo86
Copy link
Member

camilamacedo86 commented Mar 18, 2022

What do you want to happen?

The goal of this task is to:

  • Create go/v4 plugin with alpha1 level to address changes which are breaking ones for go/v3

Also, you can check the policy: https://github.com/kubernetes-sigs/kubebuilder/blob/master/VERSIONING.md#introducing-changes-to-plugins.

What is required to:

Screenshot 2022-03-18 at 15 45 03

For go/v4 we do not need the sample v1beta1 since we will remove its support. See: #2531

Extra Labels

No response

@camilamacedo86 camilamacedo86 added the kind/feature Categorizes issue or PR as related to a new feature. label Mar 18, 2022
@camilamacedo86 camilamacedo86 added this to the go/v4 milestone Mar 18, 2022
@camilamacedo86 camilamacedo86 added the good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. label Mar 18, 2022
@Ankit152
Copy link
Contributor

Hey @camilamacedo86 🙂
I would like to work on this issue. Can you assign me this one?

@camilamacedo86
Copy link
Member Author

that is great 👍

/assign @Ankit152

^ done

@Ankit152
Copy link
Contributor

Thank you very much 🙂

@Ankit152
Copy link
Contributor

Hey @NikhilSharmaWe 🙂
Thank you for your interest on this but this issue is assigned to me.

@NikhilSharmaWe
Copy link
Member

NikhilSharmaWe commented Mar 20, 2022

@Ankit152 Actually I was interested in working on #2531 but we should add new go/v4 plugin before working on that. So I started working on it before you were assigned. Apologies if you were also working on it.

You should join this thread https://kubernetes.slack.com/archives/CAR30FCJZ/p1647679086601079 and we can distribute our work further .

@Ankit152
Copy link
Contributor

@NikhilSharmaWe I totally understand but before moving forward with my assigned issue you could have asked me regarding this.

@Ankit152
Copy link
Contributor

Hey @camilamacedo86 🙂
I think this issue should be assigned to @NikhilSharmaWe as he has done quite a lot of progress with his PR.

@Ankit152 Ankit152 removed their assignment Mar 20, 2022
@camilamacedo86
Copy link
Member Author

/assign @NikhilSharmaWe

@camilamacedo86
Copy link
Member Author

We discussed this issue in the triage and accepted it but we forgot to add the label.
Adding now

@camilamacedo86 camilamacedo86 added the triage/accepted Indicates an issue or PR is ready to be actively worked on. label Mar 31, 2022
@camilamacedo86
Copy link
Member Author

camilamacedo86 commented Apr 22, 2022

Based on the latest discussions on the Kubebuider and C+R meetings and the latest finds:

a) To address the kustomize requirements we do not need a new base golang to go v4 alpha plugin. It needs to be done in the kustomize one. See: #2583

b) Then, shows that we have not enough requirements yet to justify a new base go v4 plugin. The only things that we could address on this one are:

So that, at this moment it would add an extra cost for us to keep maintaining the projects which are not justified by the value that it brings.

c) The only thing that we might want to still add currently would be a new go/v4-alpha plugin to help users begin to use the kustomize/v2-alpha after the PR #2583 get merged.

That would mean

we only change the cli to have:

	gov4Bundle, _ := plugin.NewBundle(golang.DefaultNameQualifier, plugin.Version{Number: 4, Stage: stage.Alpha},
		kustomizecommonv2.Plugin{}, // new alpha plugin
		golangv3.Plugin{}, // current language based plugin 
	)

And the test data scaffolds for this one.
PS.: Looking in the PR: #2547 that would mean not add all files under: https://github.com/kubernetes-sigs/kubebuilder/tree/master/pkg/plugins/golang/v4

However, IMO would be nice if we tried to do the above ^ change and discuss it with the other maintainers as well.

In this way, @NikhilSharmaWe I would like to recommend:
a) Let's wait for the PR #2583 to get merged
b) Then, I'd suggest we change your PR to #2547 as described above so that, we can easily show the results and bring this topic for others be able to opnieted about its pros and cons.

c/c @varshaprasad96 @rashmigottipati @ryantking

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. kind/feature Categorizes issue or PR as related to a new feature. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. triage/accepted Indicates an issue or PR is ready to be actively worked on. triage/blocked
Projects
None yet
3 participants