Enhancements: Updated Plugin mechanism for kubectl

Created on 26 Jun 2018  Â·  23Comments  Â·  Source: kubernetes/enhancements

Feature Description

  • One-line feature description (can be used as a release note): kubectl should support extensions adding new commands as well as overriding specific subcommands (at any depth).
  • Primary contact (assignee): juanvallejo, soltysh
  • Responsible SIGs: sig-cli
  • Design proposal link (community repo): https://github.com/kubernetes/community/pull/2437
  • Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred: pwittrock, soltysh
  • Approver (likely from SIG/area to which feature belongs): pwittrock, soltysh
  • Feature target (which target equals to which milestone): 1.12

    • Alpha release target (x.y) 1.12

    • Beta release target (x.y) 1.13

    • Stable release target (x.y) 1.14

kinfeature sicli stagstable trackeno

Most helpful comment

@spiffxp

what work remains to be done to promote this to beta? are there issues or PR's we could link to track?

All of the outstanding work has been completed. We'll just need to update docs and tags labeling this feature as beta

All 23 comments

/sig cli

cc @soltysh

@juanvallejo @soltysh --

It looks like this feature is currently in the Kubernetes 1.12 Milestone.

If that is still accurate, please ensure that this issue is up-to-date with ALL of the following information:

  • One-line feature description (can be used as a release note):
  • Primary contact (assignee):
  • Responsible SIGs:
  • Design proposal link (community repo):
  • Link to e2e and/or unit tests:
  • Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred:
  • Approver (likely from SIG/area to which feature belongs):
  • Feature target (which target equals to which milestone):

    • Alpha release target (x.y)

    • Beta release target (x.y)

    • Stable release target (x.y)

Set the following:

  • Description
  • Assignee(s)
  • Labels:

    • stage/{alpha,beta,stable}

    • sig/*

    • kind/feature

Once this feature is appropriately updated, please explicitly ping @justaugustus, @kacole2, @robertsandoval, @rajendar38 to note that it is ready to be included in the Features Tracking Spreadsheet for Kubernetes 1.12.


Please note that the Features Freeze is July 31st, after which any incomplete Feature issues will require an Exception request to be accepted into the milestone.

In addition, please be aware of the following relevant deadlines:

  • Docs deadline (open placeholder PRs): 8/21
  • Test case freeze: 8/28

Please make sure all PRs for features have relevant release notes included as well.

Happy shipping!

/stage beta

/remove-stage beta
/stage alpha

Hey there! @juanvallejo I'm the wrangler for the Docs this release. Is there any chance I could have you open up a docs PR against the release-1.12 branch as a placeholder? That gives us more confidence in the feature shipping in this release and gives me something to work with when we start doing reviews/edits. Thanks! If this feature does not require docs, could you please update the features tracking spreadsheet to reflect it?

@zparnold Sure, will open PR updating cli docs against 1.12

Thank you!

On Tue, Aug 21, 2018 at 10:21 AM Juan Vallejo notifications@github.com
wrote:

@zparnold https://github.com/zparnold Sure, will open PR updating cli
docs against 1.12

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/features/issues/579#issuecomment-414713171,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE81SHkRIyMSnCxaExJUlYuVrivhQu0xks5uTCVxgaJpZM4U4GJX
.

@kacole2 this will be promoted to beta in 1.13, I haven't seen it in the tracking spreadsheet.
/milestone v1.13

@soltysh this has been added to the tracking sheet. thanks!

@soltysh what work remains to be done to promote this to beta? are there issues or PR's we could link to track?

@spiffxp

what work remains to be done to promote this to beta? are there issues or PR's we could link to track?

All of the outstanding work has been completed. We'll just need to update docs and tags labeling this feature as beta

@juanvallejo @soltysh Hello - I’m the enhancement’s lead for 1.14 and I’m checking in on this issue to see what work (if any) is being planned for the 1.14 release. Enhancements freeze is Jan 29th and I want to remind that all enhancements must have a KEP - it looks like this is the KEP for this enhancement https://github.com/kubernetes/enhancements/blob/master/keps/sig-cli/0024-kubectl-plugins.md - but let me know if that is not correct

@claurence that KEP is the right one. The scope of work will be to promote this feature to GA in 1.14, based on initial comment.
/milestone v1.14

@juanvallejo @soltysh Hello - looking at the KEP it is marked as "provisional" - what more work is needed for it to be "implementable"?

Additionally are there any open PRs in k/k that we should track for the 1.14 release?

@juanvallejo @soltysh Hello - looking at the KEP it is marked as "provisional" - what more work is needed for it to be "implementable"?

I'll update the KEP accordingly.

Additionally are there any open PRs in k/k that we should track for the 1.14 release?

Only documentation fixes stating the mechanism is GA are required at this point in time.

I'll open both PRs tomorrow.

Heya, I'm working on tracking docs for the v1.14 launch. Has a docs PR been filed for this already? (It's not due until March 1st, just being proactive)

Any plans to support shadowing for CRDs? For example, I want to provide a custom view for kubectl describe <MyCRD> name .

Hello @jaredbhatti @soltysh, 1.14 enhancement shadow here. Code Freeze is March 7th and all PRs must be merged by then to your issue to make the 1.14 release. What open K/K PRs do you still have that need to merge? Thanks

Any plans to support shadowing for CRDs? For example, I want to provide a custom view for kubectl describe <MyCRD> name .

That's still being discussed, but we're quite hesitant since that might introduce hard to debug behavior. For kubectl describe itself we're planning to have a server-side version of it, similarly to kubectl get.

@lachie83 @jaredbhatti @claurence sorry for the delay, but here it goes, the updated KEP (to implemented) is https://github.com/kubernetes/enhancements/pull/856 and doc update is here: https://github.com/kubernetes/website/pull/12847

/remove-stage beta
/stage stable

Now that 1.14 is out the door I think we can close this issue since we've reached its final state.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

wojtek-t picture wojtek-t  Â·  12Comments

xing-yang picture xing-yang  Â·  13Comments

andrewsykim picture andrewsykim  Â·  12Comments

justaugustus picture justaugustus  Â·  3Comments

msau42 picture msau42  Â·  13Comments