vmware-archive / kops

Kubernetes Operations (kops) - Production Grade K8s Installation, Upgrades, and Management
Apache License 2.0
3 stars 3 forks source link

Fail kops commands gracefully for vsphere #48

Closed prashima closed 7 years ago

prashima commented 7 years ago

Fail gracefully for vsphere cloud provider, while executing following kops commands-

  1. create instancegroup
  2. delete instancegroup
  3. replace
  4. rolling-update
  5. toolbox dump
  6. toolbox convert-imported
  7. update cluster (filing a separate issue)
  8. upgrade cluster

'edit' and 'update' commands for federation are yet to be investigated. They might also make to this list.

prashima commented 7 years ago

One more addition-

  1. import cluster
prashima commented 7 years ago

Some more commands made to this list:

  1. delete instancegroup
  2. delete -f FILENAME
  3. edit cluster
  4. edit ig
  5. edit federation
  6. update federation
prashima commented 7 years ago

For following commands no cloud provider is getting queried. So it's not possible to decide if a command is meant for vSphere. Additional parameters will have to be added to identify vSphere cloud provider.

prashima commented 7 years ago

https://github.com/vmware/kops/commit/ed42a932d175cd235a013a846735200a5ae0a7df fixes this issue, by supplying correct error message.

Commands discussed in comment above this one are yet to be fixed

prashima commented 7 years ago

Reduced priority, since the fix is available for common used commands. The remaining 4 commands don't get used very frequently. So a mention in doc will also do for now, in my opinion.

prashima commented 7 years ago

Filed https://github.com/kubernetes/kops/issues/2421 for 'import cluster' command.

For now, following commands will have to be documented as unsupported. I don't see any other way around to put a check for vSphere cloud provider, just for the duration for which they won't be supported.

Closing this issue. Will file a separate issue for documentation.