Test-infra: Migrate away from deprecated `--build-cluster` flag and format.

Created on 17 Oct 2019  路  18Comments  路  Source: kubernetes/test-infra

What should be cleaned up or changed:

Migrate away from and deprecate --build-cluster flag and format.

  • [x] Migrate K8s Prow deployments to use --kubeconfig (#16906)
  • [x] Deprecation and removal of mkbuild-cluster

    • [x] Create an announcement and set removal date, #15285.

    • [x] Replace mkbuild-cluster references in getting_started_deploy.md with the equivalent gencred command(s), #15285.

    • [x] Remove mkbuild-cluster source code (ref), to be completed May 2020, #15287.

    • [x] Remove mkbuild-cluster label (ref), to be completed May 2020, #15287.

/assign

help wanted kincleanup kinoncall-hotlist

All 18 comments

/help

@clarketm:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/help

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.

Migrate K8s Prow deployments to use --kubeconfig

Doesn't seem like contributors can handle this? Also you've already assigned yourself...?

/unassign

/assign

/kind oncall-hotlist

The only thing left is on-call process for k8s Prow:

  1. create a kubeconfig file
  2. save it in a secret, and
  3. define it (using --kubeconfig) on the k8s resources that require it (e.g. plank, sinker, deck).

Would be great to make some progress on here, happy to help out during my oncall shift.

/assign @fejta @clarketm

Do we have a tool to create the kubeconfig file? (convert the existing one?)

Do we have a tool to create the kubeconfig file? (convert the existing one?)

Yes, that is gencred. Its usage has also been added to the Prow getting started guide https://github.com/kubernetes/test-infra/pull/15285.

/unassign @clarketm
I'll give it a go then, thanks!

Still pending

Migrate K8s Prow deployments to use --kubeconfig

This should now only be a single command followed by a PR to update object config yaml. Can someone with k8s-prow cluster access give this a try before May 2020?


Note: The build-cluster format is planned for removal in May 2020 (ref) (https://github.com/kubernetes/test-infra/pull/15287).


Update
Done via #16906

/unassign @stevekuznetsov @fejta
/assign @clarketm
/close

@fejta: Closing this issue.

In response to this:

/unassign @stevekuznetsov @fejta
/assign @clarketm
/close

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.

/reopen

@clarketm: Reopened this issue.

In response to this:

/reopen

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.

/close

@clarketm: Closing this issue.

In response to this:

/close

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.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

cjwagner picture cjwagner  路  3Comments

lavalamp picture lavalamp  路  3Comments

fen4o picture fen4o  路  4Comments

spiffxp picture spiffxp  路  3Comments

BenTheElder picture BenTheElder  路  4Comments