Commit Graph

485 Commits

Author SHA1 Message Date
Justin Santa Barbara a8ee484b11 Merge pull request #109 from justinsb/fix_107
Delete cluster deletes keypairs
2016-07-09 22:29:43 -04:00
Justin Santa Barbara 524fb645d0 Delete cluster deletes keypairs
Again we delete by name, which is why we name them by the very unlikely
to conflict name "kubernetes.<clustername>"

Issue #107
2016-07-09 22:26:12 -04:00
Justin Santa Barbara 5b8b4d4da3 Detect & delete new ASG launch configs
We now output a ClusterName property into the launchconfig, even though
we don't technically need it.  But it allows us to more easily detect
the cluster, and it generally seems like a good idea.

Also rename to 'autoscaling-config' and clean up the cluster name
detection logic.

Fix #96
2016-07-09 22:07:24 -04:00
Justin Santa Barbara b98422fb3a Merge pull request #101 from justinsb/eventual_consistency_igw_tag
Tolerate eventual consistency error tagging gateways
2016-07-09 22:03:40 -04:00
Justin Santa Barbara 2632a46561 Merge pull request #106 from justinsb/fix_97
Clean up IAM roles as part of delete
2016-07-09 21:33:16 -04:00
Justin Santa Barbara d705a1001c Clean up IAM roles
We match by name; this seems acceptable given that we make the cluster
name a FQDN.

Issue #97
2016-07-09 15:33:57 -04:00
Justin Santa Barbara 885b728eb9 Merge pull request #105 from justinsb/fix_80
Move delete output to table helper
2016-07-09 14:53:40 -04:00
Justin Santa Barbara 952e74714f Sort table output
Currently sort order is locked to table order, but this could be changed
if needed.

Fix #80
2016-07-09 14:49:54 -04:00
Justin Santa Barbara e28c770c18 Move delete output to table helper 2016-07-09 13:57:02 -04:00
Justin Santa Barbara 555c637d93 Merge pull request #94 from justinsb/fix_66
Support `kops get/edit instancegroup`
2016-07-09 13:50:04 -04:00
Justin Santa Barbara fdfe710c92 Merge pull request #104 from justinsb/fix_35
More tweaks to the IAM async creation tolerance
2016-07-09 13:49:15 -04:00
Justin Santa Barbara 359fb8234a Merge pull request #102 from justinsb/fix_74
Delete Route53 host records on cluster-down
2016-07-09 02:46:53 -04:00
Justin Santa Barbara 0a8b94e748 More tweaks to the IAM async creation tolerance
Issue #35
2016-07-09 02:45:30 -04:00
Justin Santa Barbara c058a0c2a8 Tolerate shutting-down state on instance during delete 2016-07-09 02:41:56 -04:00
Justin Santa Barbara 5c65600229 Delete Route53 host records on cluster-down
Initial implementation: we match by name, for example; when we have a
hosted zone id we should plumb it through.

Does implement bulk deletion, because on Route53 we have to delete
multiple resources at once.

Issue #74
2016-07-09 02:41:25 -04:00
Justin Santa Barbara 935b08fba2 Tolerate eventual consistency error tagging gateways 2016-07-09 01:46:32 -04:00
Justin Santa Barbara 7ee06887f0 Merge pull request #100 from justinsb/fix_model
Fix model: numbers must be quoted
2016-07-09 01:42:17 -04:00
Justin Santa Barbara 126c508426 Fix model: numbers must be quoted 2016-07-09 01:41:04 -04:00
Justin Santa Barbara 6c6410dc81 Merge pull request #87 from justinsb/fix_35_part2
Report less-scary message when IAM instance profile is not ready yet
2016-07-09 01:25:28 -04:00
Justin Santa Barbara 13e514aeac Merge pull request #93 from justinsb/fix_24
Allow configurable RootDeviceSize & RootDeviceType
2016-07-09 01:25:20 -04:00
Justin Santa Barbara 5edef741b9 Merge pull request #99 from justinsb/node_role_name
Change node role tag to match master pattern
2016-07-09 01:25:09 -04:00
Justin Santa Barbara b42765816e Change node role tag to match master pattern
It's not currently used, and we hadn't updated it to match the better
pattern.

k8s.io/role=master can only be in one role
k8s.io/role/master=1 allows for multiple roles
2016-07-08 22:02:32 -04:00
Justin Santa Barbara 992a3e7d7f Support `kops get/edit instancegroup`
This lets us start to manage instancegroups through the CLI

Issue #66
2016-07-08 01:21:15 -04:00
Justin Santa Barbara 13b8e81bd6 Allow configurable RootDeviceSize & RootDeviceType
This allows for a larger EBS root volume (and we now default to 20GB,
just like kube-up did).

We remove the BlockDeviceMappings support because it wasn't used and
made things a lot more complicated.  We always map the ephemeral
devices.

Issue #24
2016-07-08 01:11:14 -04:00
Justin Santa Barbara 6264714e14 Merge pull request #86 from justinsb/docs_4
First pass at docs describing how kops boots k8s
2016-07-07 21:02:38 -04:00
Justin Santa Barbara 9d1770ad31 Merge pull request #89 from justinsb/format_expose_secret
When exposing secrets, output a blank line afterwards
2016-07-07 15:28:40 -04:00
Justin Santa Barbara 29dbbe3af3 When exposing secrets, output a blank line afterwards
Otherwise it becomes hard to copy and paste
2016-07-07 11:52:52 -04:00
Justin Santa Barbara d1ef7580a9 Report less-scary message when IAM instance profile is not ready yet
IAM instance profile creation is very async, and this causes dependent
resources to fail.  That's fine - we have good retry logic - but we
should output a less frightening error message.

Issue #35
2016-07-07 10:22:44 -04:00
Justin Santa Barbara 7bb75d2e43 First pass at docs describing how kops boots k8s 2016-07-07 10:06:04 -04:00
Justin Santa Barbara b7fa910cb8 Merge pull request #83 from justinsb/log_more_clusters_cannot_span_regions
More info on "clusters cannot span multiple regions"
2016-07-07 09:51:18 -04:00
Justin Santa Barbara 4bf51a825b More info on "clusters cannot span multiple regions" 2016-07-07 09:49:23 -04:00
Justin Santa Barbara 901b82e78a Merge pull request #82 from justinsb/fix_73
Avoid a race between etcd start and volume mount
2016-07-07 02:22:29 -04:00
Justin Santa Barbara f82a407e73 Avoid a race between etcd start and volume mount
If the instance restarted but lost the volume mount, there might be a
short or indefinite delay before protokube can mount the volume again.
But the etcd manifest would probably still be in
/etc/kubernetes/manifests from the previous run.

To ensure that kubelet doesn't run etcd until the volume is actually
mounted, we use a symlink to a directory on the volume itself.  Thus
kubelet can't start etcd until we put the volume there.  We can also
delete the symlink before mounting, so we have full control.

Issue #73
2016-07-07 02:21:14 -04:00
Justin Santa Barbara 8f12f3e4fe Merge pull request #79 from justinsb/instancetypes_g2_i2_x1
More instance type support: g2, i2, x1 families
2016-07-07 00:37:55 -04:00
Justin Santa Barbara a6ba62abe9 More instance type support: g2, i2, x1 families 2016-07-06 23:27:08 -04:00
Justin Santa Barbara f89c80c52c Merge pull request #78 from justinsb/fix_restart_1
Make sure that we enable systemd services
2016-07-06 23:25:07 -04:00
Justin Santa Barbara df4dadff98 Merge pull request #77 from justinsb/fix_76
Add support for r3 family
2016-07-06 23:24:59 -04:00
Justin Santa Barbara 09dd3002ff Make sure that we enable systemd services
We were previously just starting them, which works fine for the new
machine and machine termination case, but doesn't work when doing a
machine restart (e.g. init 6)

Issue #70
2016-07-06 23:04:51 -04:00
Justin Santa Barbara 40cbdc4d00 Add support for r3 family
Fix #76
2016-07-06 22:02:56 -04:00
Justin Santa Barbara fecce6cbfe Merge pull request #62 from justinsb/docs_2
docs: suggest more strongly to export KOPS_STATE_STORE
2016-07-06 21:58:58 -04:00
Justin Santa Barbara fcd644441e Merge pull request #61 from justinsb/typo_1
Fix typo in error message
2016-07-06 16:42:47 -04:00
Justin Santa Barbara 957d30a157 Merge pull request #68 from justinsb/fix_elb_pagesize
Fetch ELB tags in chunks of 20
2016-07-06 13:47:30 -04:00
Justin Santa Barbara 274a8eff3c Fetch ELB tags in chunks of 20
ELB DescribeTags has a limit of 20 ELBs / call.  So we paginate the
DescribeLoadBalancers call with page size = 20 also.
2016-07-06 13:46:16 -04:00
Justin Santa Barbara b032be9a79 Merge pull request #65 from justinsb/issue_63
Add tests for SSH key fingerprint
2016-07-06 11:19:48 -04:00
Justin Santa Barbara 7347f1815a Parse SSH public keys with 'odd' whitespace
e.g. a key with \r instead of spaces

Fix #63
2016-07-06 11:18:09 -04:00
Justin Santa Barbara a91cc6e9a4 Add tests for SSH key fingerprint
Issue #63
2016-07-06 10:47:20 -04:00
Justin Santa Barbara 8b204f6d82 Merge pull request #64 from justinsb/issue_63
Quote public key in error message
2016-07-06 10:23:22 -04:00
Justin Santa Barbara 4dc869756f Quote public key in error message
Just in case it contains invalid characters

Issue #63
2016-07-06 10:22:09 -04:00
Justin Santa Barbara 48cabafd7c docs: suggest more strongly to export KOPS_STATE_STORE
Otherwise it's not entirely clear that you really want to do this, and
that it's not just for the formatting of the examples!
2016-07-06 10:16:20 -04:00
Justin Santa Barbara b6035bffaf Fix typo in error message
Fingerpring -> Fingerprint
2016-07-06 10:04:13 -04:00