Commit Graph

493 Commits (1d8960798b035bb1fdfe0707cd4de0be7823330b)

Author SHA1 Message Date
Tim Hockin 3586986416 Switch to k8s.gcr.io vanity domain
This is the 2nd attempt.  The previous was reverted while we figured out
the regional mirrors (oops).

New plan: k8s.gcr.io is a read-only facade that auto-detects your source
region (us, eu, or asia for now) and pulls from the closest.  To publish
an image, push k8s-staging.gcr.io and it will be synced to the regionals
automatically (similar to today).  For now the staging is an alias to
gcr.io/google_containers (the legacy URL).

When we move off of google-owned projects (working on it), then we just
do a one-time sync, and change the google-internal config, and nobody
outside should notice.

We can, in parallel, change the auto-sync into a manual sync - send a PR
to "promote" something from staging, and a bot activates it.  Nice and
visible, easy to keep track of.
2018-02-07 21:14:19 -08:00
Kubernetes Submit Queue f2362ed6e5
Merge pull request #59396 from kwmonroe/bug/opts-before-login
Automatic merge from submit-queue (batch tested with PRs 59441, 58264, 59287, 59396, 59439). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Ensure docker daemon opts are in effect before docker login

**What this PR does / why we need it**:
Make sure the kubernetes-worker charm is using the configured `docker-opts` daemon options prior to running the `docker-logins` handler.
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/layer-docker/issues/112
**Special notes for your reviewer**:

**Release note**:

```release-note
Fixed charm issue where docker login would run prior to daemon options being set. 
```
2018-02-06 20:27:42 -08:00
Kevin W Monroe fa440c8dd2 split docker-logins logic into 2 handlers
Previous commit relied on non-obvious reactive behavior, with the
'docker.restart' flag being exploited to re-execute docker_logins_changed
in a single hook execution.

Split this into 2 handlers:
- when docker-logins config changes, set one flag to handle the login
and another to request a daemon restart (if needed).
- run docker login after any potential daemon restarts have occurred.
2018-02-06 11:29:42 -06:00
Kevin W Monroe 4af548bb5a Ensure daemon opts are in effect before docker login 2018-02-06 05:16:20 +00:00
halfcrazy 9b64650311 fix typo in cluster 2018-02-03 11:08:06 +08:00
Kubernetes Submit Queue ea501413ce
Merge pull request #59219 from hyperbolic2346/mwilson/get-node-name-timeout-bump
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

bumping timeouts for apiserver communication.

**What this PR does / why we need it**:
Our CI is failing due to timeouts. The times we set node names intersects with apiserver restarts, which can sometimes get caught up for longer than one minute. This is a temporary fix until we can decide on the best long-term solution.
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #

**Special notes for your reviewer**:

**Release note**:

```release-note
Upped the timeout for apiserver communication in the juju kubernetes-worker charm.
```
2018-02-02 10:25:36 -08:00
Kubernetes Submit Queue 4b8309e315
Merge pull request #59064 from hyperbolic2346/mwilson/upgrade-creds-fix
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Fixing upgrade charm failing if upgrading from an old enough charm

**What this PR does / why we need it**:
Charms before ~Nov 2017 can not upgrade to charms made after that time due to a race with a missing credential dictionary. This change forces the dictionary to be created before we attempt to use it.
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/474
**Special notes for your reviewer**:

**Release note**:

```release-note
Fixed issue with charm upgrades resulting in an error state.
```
2018-02-02 04:34:28 -08:00
Mike Wilson af71c9670f bumping timeouts for apiserver communication. 2018-02-01 15:55:44 -05:00
Kubernetes Submit Queue 57bd4f8344
Merge pull request #59116 from hyperbolic2346/mwilson/priv-typo-fix
Automatic merge from submit-queue (batch tested with PRs 57683, 59116, 58728, 59140, 58976). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Fixing issue with capitalization in allow-privileged

**What this PR does / why we need it**:
Config option in kubernetes-master allow-privileged would do odd things if you used True instead of true.
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/374
**Special notes for your reviewer**:

**Release note**:

```release-note
Fixed issue with kubernetes-worker option allow-privileged not properly handling the value True with a capital T.
```
2018-02-01 07:04:37 -08:00
Mike Wilson 33dac2d0f3 Adding lower() to kubernetes master's usage of allow-privileged. 2018-01-31 12:20:37 -05:00
Mike Wilson ec0bedbf43 Fixing issue with capitalization causing odd behaviors for allow-privileged configuration option. 2018-01-31 10:26:23 -05:00
Mike Wilson 17f8f18f6f Fixing upgrade charm failing if upgrading from an old enough charm(pre Nov 2017). 2018-01-30 14:52:57 -05:00
Mike Wilson e87b537654 Forcing get_node_name to continue searching for a node name if the returned list of nodes doesn't include this one.
There was a race condition where the kubelet was restarting and we were querying the api server for this node.
2018-01-29 20:27:35 -05:00
Kubernetes Submit Queue 46cd37480e
Merge pull request #58843 from hyperbolic2346/mwilson/spaces-fixes
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Fixing spaces issue found with tests.

Had some missing parameters for some functions.



**What this PR does / why we need it**:
Fixing charms when setting extra sans on load balancer or master
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/477
**Special notes for your reviewer**:

**Release note**:

```release-note
Fixing extra_sans option on master and load balancer.
```
2018-01-26 07:40:06 -08:00
Mike Wilson 5acf24db79 Fixing spaces issue found with tests. Had some missing parameters for some functions. 2018-01-25 21:20:35 -05:00
Kubernetes Submit Queue cd2692c49d
Merge pull request #58830 from wwwtyro/rye/storage-backend-config
Automatic merge from submit-queue (batch tested with PRs 53778, 58378, 58830). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Add storage-backend configuration option to kubernetes-master charm.

**What this PR does / why we need it**: Add storage-backend configuration option to kubernetes-master charm.

**Release note**:
```release-note
Add storage-backend configuration option to kubernetes-master charm.
```
2018-01-25 17:09:36 -08:00
Rye Terrell 6db03cf471 Add storage-backend configuration option to kubernetes-master charm. 2018-01-25 15:30:41 -06:00
Kubernetes Submit Queue 2e4947ec3c
Merge pull request #58756 from wwwtyro/rye/check-config-created
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Ensure config has been created before attempting to launch ingress.

**What this PR does / why we need it**: Ensure config has been created before attempting to launch ingress.

**Release note**:
```release-note
Ensure config has been created before attempting to launch ingress.
```
2018-01-24 15:06:11 -08:00
Rye Terrell 154caf6a8c Ensure config has been created before attempting to launch ingress. 2018-01-24 11:06:37 -06:00
Kubernetes Submit Queue 36ea7614e9
Merge pull request #58708 from hyperbolic2346/mwilson/kubeapi-network-spaces
Automatic merge from submit-queue (batch tested with PRs 54242, 58522, 58704, 58708, 58712). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Adding network spaces support for kubeapi-load-balancer.

**What this PR does / why we need it**:
Adding support for juju network spaces to kubeapi-load-balancer charm
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/406
**Special notes for your reviewer**:

**Release note**:

```release-note
Added support for network spaces in the kubeapi-load-balancer charm
```
2018-01-23 15:29:46 -08:00
Kubernetes Submit Queue 28adfecd9b
Merge pull request #58704 from hyperbolic2346/mwilson/kubernetes-master-spaces
Automatic merge from submit-queue (batch tested with PRs 54242, 58522, 58704, 58708, 58712). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Adding network spaces support for kubernetes-master.

**What this PR does / why we need it**:
Adding support for juju network spaces to kubernetes-master charm
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/406
**Special notes for your reviewer**:

**Release note**:

```release-note
Added support for network spaces in the kubernetes-master charm
```
2018-01-23 15:29:44 -08:00
Mike Wilson aef2aecd70 Adding network spaces support for kubeapi-load-balancer. 2018-01-23 13:33:14 -05:00
Mike Wilson 2afc76fadd Adding network spaces support for kubernetes-master. 2018-01-23 11:17:05 -05:00
Kubernetes Submit Queue ecb3d7bb6d
Merge pull request #58523 from hyperbolic2346/mwilson/network-spaces
Automatic merge from submit-queue (batch tested with PRs 58438, 58523, 58513, 57948). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

network spaces support for kubernetes-worker charm

**What this PR does / why we need it**:
Adds support for network spaces in the kubernetes-worker charm.
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/406
**Special notes for your reviewer**:

**Release note**:

```release-note
Added support for network spaces in the kubernetes-worker charm
```
2018-01-23 03:53:28 -08:00
Mike Wilson a22358e525 Fixing some flake8 issues 2018-01-22 14:54:58 -05:00
Mike Wilson b04a88cc9e Adding support for changing default backend and nginx container images 2018-01-19 16:40:57 -05:00
Mike Wilson 531c9a81c1 Changing where the charm gets network addresses in order to support network spaces. 2018-01-19 10:00:46 -05:00
George Kraft f56463bf32 Fix flake8 lint error in kubernetes-master charm 2018-01-18 13:53:13 -06:00
Kubernetes Submit Queue ee43f7c34e
Merge pull request #57246 from juju-solutions/feature/etcd3
Automatic merge from submit-queue (batch tested with PRs 49874, 56919, 57246). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Enable support for etcd3

**What this PR does / why we need it**: With this PR juju deployments of kubernetes can use both etcd2 and 3.

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:

**Special notes for your reviewer**: Lets not merge this immediately. Lets wait for next year. 

**Release note**:

```NONE

```
2018-01-10 08:15:46 -08:00
Konstantinos Tsakalozos e3cafd8303 Enable support for etcd3 2018-01-08 14:44:56 +02:00
Rye Terrell 9aa4fc0d3b Add proxy_read_timeout flag to kubeapi_load_balancer charm. 2018-01-05 21:20:31 -06:00
Kubernetes Submit Queue 7fb2d5432d
Merge pull request #57530 from hyperbolic2346/mwilson/nginx-daemonset
Automatic merge from submit-queue (batch tested with PRs 57492, 57530). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Changing ingress from manually scaling rc to a daemon set.

**What this PR does / why we need it**:
Quality of life fix. We used to set an RC to run the nginx ingress and then scale it in the charm as workers were added. Now we just use a daemon set to make sure it is running everywhere. Using application-specific names and roles so multiple applications can have different ingress settings.

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/367

**Special notes for your reviewer**:

**Release note**:

```release-note
CDK nginx ingress is now handled via a daemon set.
```
2018-01-05 08:14:40 -08:00
Mike Wilson 4aef85aab4 Removing duplicate import 2018-01-05 09:41:23 -05:00
Mike Wilson 806759cfc9 Changed return of empty string to raise an exception as it should have been from the beginning. 2018-01-04 17:45:40 -05:00
Tim Hockin e9dd8a68f6 Revert k8s.gcr.io vanity domain
This reverts commit eba5b6092a.

Fixes https://github.com/kubernetes/kubernetes/issues/57526
2017-12-22 14:36:16 -08:00
Mike Wilson 1b814c43ad Changing ingress from manually scaling rc to a daemon set. 2017-12-21 18:31:08 -05:00
Konstantinos Tsakalozos bd8eaa7f18 Even better 2017-12-20 17:25:54 +02:00
Konstantinos Tsakalozos c9688155f6 Much better 2017-12-20 17:14:00 +02:00
Konstantinos Tsakalozos d3ca7a0eb8 reason key should exist 2017-12-20 16:56:18 +02:00
Konstantinos Tsakalozos b2a6eb99c1 Evicted pods should respawn 2017-12-20 16:30:07 +02:00
Kubernetes Submit Queue 4db2245a9f
Merge pull request #57403 from wwwtyro/rye/deprecated-dns-ip-handling
Automatic merge from submit-queue (batch tested with PRs 57400, 57403, 57303). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Use old dns-ip mechanism with older cdk-addons.

**What this PR does / why we need it**: Use old dns-ip mechanism with older cdk-addons.

**Release note**:
```release-note
Use old dns-ip mechanism with older cdk-addons.
```
2017-12-19 11:27:35 -08:00
Kubernetes Submit Queue 866d1bed6b
Merge pull request #57400 from juju-solutions/feature/prep-1.9-release
Automatic merge from submit-queue (batch tested with PRs 57400, 57403, 57303). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Pointing juju charms to 1.9

**What this PR does / why we need it**: Point juju charms to 1.9 k8s release

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #

**Special notes for your reviewer**:

**Release note**:

```NONE

```
2017-12-19 11:27:32 -08:00
Rye Terrell d8e3861991 Use old dns-ip mechanism with older cdk-addons. 2017-12-19 07:48:16 -06:00
Konstantinos Tsakalozos f2da078168 Pointing juju charms to 1.9 2017-12-19 14:48:05 +02:00
Konstantinos Tsakalozos f4e8385a3b Minor lint fix 2017-12-19 09:13:28 +02:00
Kubernetes Submit Queue 4c36d487f0
Merge pull request #57337 from wwwtyro/rye/wait-kube-dns
Automatic merge from submit-queue (batch tested with PRs 55751, 57337, 56406, 56864, 57347). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Wait for kubedns to be ready when collecting the cluster IP.

**What this PR does / why we need it**: Wait for kubedns to be ready when collecting the cluster IP.

**Release note**:
```release-note
Wait for kubedns to be ready when collecting the cluster IP.
```
2017-12-18 18:50:40 -08:00
Kubernetes Submit Queue e5abffca6f
Merge pull request #54174 from thockin/gcr-vanity
Automatic merge from submit-queue (batch tested with PRs 54379, 56593, 56685, 54174, 57309). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Use k8s.gcr.io vanity domain for container images

Related issue: https://github.com/kubernetes/release/issues/281

```release-note
Use "k8s.gcr.io" for container images rather than "gcr.io/google_containers".  This is just a redirect, for now, so should not impact anyone materially.  

Documentation and tools should all convert to the new name. Users should take note of this in case they see this new name in the system.
```
2017-12-18 14:45:45 -08:00
Tim Hockin eba5b6092a Use k8s.gcr.io vanity domain for container images 2017-12-18 09:18:34 -08:00
Rye Terrell 23c81bb3d4 wait for kubedns to be ready 2017-12-18 09:28:03 -06:00
Konstantinos Tsakalozos adf3c21495 Fix LB lint errors 2017-12-18 10:56:34 +02:00
Kubernetes Submit Queue cab439b20f
Merge pull request #57195 from marcoceppi/s390x-fixes
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Use an s390x default-http-backend

This needs to be refactored to eventually say all non x86 architectures place a -arch() in the image name to support ppc64el, arm, etc.

Most all gcr.io/google_containers have -arch() image names.


**What this PR does / why we need it**:

Adds s390x images for when deploying to z system mainframes for default-http-backend image

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/455

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
2017-12-17 00:54:22 -08:00
Kubernetes Submit Queue 3492467178
Merge pull request #57113 from wwwtyro/rye/optional-kube-dns
Automatic merge from submit-queue (batch tested with PRs 56676, 57050, 54881, 56822, 57113). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

make kube-dns addon optional

**What this PR does / why we need it**: Makes the kube-dns addon optional so that users can deploy their own DNS solution.

**Release note**:
```release-note
Makes the kube-dns addon optional so that users can deploy their own DNS solution.
```
2017-12-16 19:19:49 -08:00
Kubernetes Submit Queue cb127830ff
Merge pull request #56979 from hyperbolic2346/mwilson/apilb-log-rotate
Automatic merge from submit-queue (batch tested with PRs 56858, 57040, 56979, 57051, 57052). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

enabling log rotate for api logs in nginx

**What this PR does / why we need it**:
Prevent the load balancer from filling the disk with logs from the api server
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/449
**Special notes for your reviewer**:

**Release note**:

```release-note
Enabled log rotation for load balancer's api logs to prevent running out of disk space.
```
2017-12-16 16:34:44 -08:00
Kubernetes Submit Queue 7a3fa9d58e
Merge pull request #56633 from juju-solutions/bug/restarts
Automatic merge from submit-queue (batch tested with PRs 56337, 56546, 56550, 56633, 56635). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Remove do-restart states

**What this PR does / why we need it**: Simplify juju reactive part of the master.

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/448

**Special notes for your reviewer**:

**Release note**:

```NONE```
2017-12-16 01:53:52 -08:00
Kubernetes Submit Queue 4af5dd1bff
Merge pull request #56490 from juju-solutions/bug/upgrade-masters
Automatic merge from submit-queue (batch tested with PRs 56413, 56322, 56490, 56460, 56487). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Fix status messaging when juju masters upgrade

**What this PR does / why we need it**: When upgrading masters we want to not skip the message prompting the admin to trigger an upgrade action.

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:


**Special notes for your reviewer**:

**Release note**:

```
NONE
```
2017-12-15 16:43:53 -08:00
Kubernetes Submit Queue 05acd736c1
Merge pull request #56217 from Cynerva/gkk/docker-logins
Automatic merge from submit-queue (batch tested with PRs 56217, 56268, 56263, 56328, 56200). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Add docker-logins config to kubernetes-worker charm

**What this PR does / why we need it**:

This adds a `docker-logins` config option to the kubernetes-worker charm, which allows cluster operators to authenticate against docker registries so kubelet can pull containers from them.

**Release note**:

```release-note
Added docker-logins config to kubernetes-worker charm
```
2017-12-14 23:37:39 -08:00
Marco Ceppi 6592e44d18 Use an s390x default-http-backend
This needs to be refactored to eventually say all non x86 architectures place a -arch() in the image name to support ppc64el, arm, etc.

Most all gcr.io/google_containers have -arch() image names.
2017-12-14 13:28:41 +00:00
Rye Terrell 6559e4dede make kube-dns addon optional 2017-12-12 14:33:38 -06:00
Mike Wilson 57059ea442 apilb template writes a log file to /var/log/nginx.*.log and not into the nginx directory where the log rotation is setup. Adding a log rotation file for these logs. 2017-12-11 12:33:58 -05:00
Konstantinos Tsakalozos 0f591aeabc Fix flake8 error 2017-12-01 16:28:32 +02:00
Konstantinos Tsakalozos 85ac2dc9fa Remove do-restart states 2017-11-30 16:50:40 +02:00
Konstantinos Tsakalozos f0ace95218 Handling the case of an upgrade from a non-rolling master with resource change 2017-11-29 09:56:42 +02:00
Konstantinos Tsakalozos 61d9848438 Improve handling of snap resources 2017-11-28 20:47:19 +02:00
Konstantinos Tsakalozos 1550df99eb The change in channels will be caught config change after the upgrade. 2017-11-28 12:24:40 +02:00
Konstantinos Tsakalozos 280b833f39 Fix master upgrade cornercases 2017-11-27 21:53:22 +02:00
Konstantinos Tsakalozos a6ee55c4a7 Lowecasing the hostname on the known nodes entry 2017-11-22 16:55:39 +02:00
Konstantinos Tsakalozos 99a25fed19 Lowercase hostnames when used as node names in k8s 2017-11-22 09:45:23 +02:00
George Kraft ca9076cf0f Add docker-logins config to kubernetes-worker 2017-11-20 12:52:34 -06:00
Mike Wilson c8f6eaccba Master now supports delayed upgrades. It will wait until specifically told to upgrade with an action unless the configuration option require-manual-upgrade is false. 2017-11-18 01:04:13 -05:00
George Kraft 9036d07cb8 Add extra-args configs to kubernetes-worker charm 2017-11-08 12:49:37 -06:00
George Kraft 18a42bee37 Add extra-args configs for scheduler and controller-manager 2017-11-06 15:56:43 -06:00
Konstantinos Tsakalozos dce7a87d26 Removing flush_auth_for_departed 2017-11-03 23:24:57 +02:00
Konstantinos Tsakalozos b134ce7621 New master takes over 2017-11-03 20:47:43 +02:00
Mike Wilson 21c8253dfe Changing the way we clear the certificate written flag to use a helper function in the tls layer. 2017-11-01 17:53:29 -04:00
Mike Wilson b5b53bd273 Fixing indent 2017-11-01 13:24:55 -04:00
Mike Wilson b77b2b91c7 Merge branch 'master' into lb 2017-11-01 12:00:28 -04:00
Mike Wilson b25805f01c Adding extra_sans option to load balancer to allow custom SAN entries on the certificate
Adding support for restarting nginx on the load balancer
Added better support for knowing when certificates are written. This helps the master restart the apiserver appropriately.
2017-11-01 11:56:58 -04:00
George Kraft 47f44f58a5 Fix hook failure in kubernetes-worker charm due to iptables conflict 2017-10-31 17:05:53 -05:00
George Kraft 421379889d Fix iptables FORWARD policy for Docker 1.13 in kubernetes-worker charm 2017-10-27 11:04:56 -05:00
Konstantinos Tsakalozos 42702518a1 Address lint errors 2017-10-25 10:34:20 +03:00
Kubernetes Submit Queue df9e32b219 Merge pull request #53820 from juju-solutions/feature/rbac
Automatic merge from submit-queue (batch tested with PRs 53820, 53971). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Add support for RBAC support to Kubernetes via Juju

**What this PR does / why we need it**: This PR add RBAC to the Juju deployment of Kubernetes

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: 

**Special notes for your reviewer**:

**Release note**:

```Canonical Distribution of Kubernetes offers configurable RBAC 
```
2017-10-24 09:32:15 -07:00
Mike Wilson 16036c3fd4 Change config.changed to config.changed.extra_sans so we only try to update certificates when the SAN entries change 2017-10-20 10:38:26 -04:00
Mike Wilson d0a88264d8 Fixing spacing issue 2017-10-19 19:39:09 -04:00
Mike Wilson 73e8af34af Adding config option to add additional SANs to the master's certificate.
Regenerate certificate if data on certificate changes. This includes IP address and SANs.
Restart API server after updating certificate.
2017-10-19 09:44:58 -04:00
Kubernetes Submit Queue bd388e0d82 Merge pull request #51310 from xiangpengzhao/sc-eg
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Replace storage-class annotations with field in examples

**What this PR does / why we need it**:
storage class is already GA. Replace annotations with field `StorageClassName` in examples.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #51435 (update: thanks @gyliu513 for the issue)
ref: https://github.com/kubernetes/kubernetes/pull/50654#discussion_r134954171

**Special notes for your reviewer**:
We may also want to remove the beta annotations in 1.8 since the field will have already been in two releases. If @kubernetes/sig-storage-api-reviews confirm this, I'd like to help remove it.

/cc @liggitt @jsafrane @msau42 

**Release note**:

```release-note
NONE
```
2017-10-18 20:31:15 -07:00
Konstantinos Tsakalozos 9a28e9b125 Addressing review comments 2017-10-17 22:31:22 +03:00
Kubernetes Submit Queue 1d8f1e268f Merge pull request #47699 from supereagle/fix-typos
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

fix typos: remove duplicated word in comments

**What this PR does / why we need it**: Remove the duplicated word `the` in comments

**Which issue this PR fixes** : fixes #

**Special notes for your reviewer**:

```release-note
NONE
```
2017-10-17 02:35:52 -07:00
Konstantinos Tsakalozos 95fec2dc3f Work on upgrade path 2017-10-11 17:13:19 +03:00
shun-miyoshi-com 4820a6eadd fix kubemark, juju, and libvirt-coreos README.md (from minion to node) 2017-10-10 06:45:15 +00:00
Konstantinos Tsakalozos 50354896b6 Fix trimmed files comming from leadership 2017-10-09 11:05:42 +03:00
Konstantinos Tsakalozos b8a4fa6d81 Move ingress to kube-system. Rename enable-rbac to authorization-mode. 2017-10-09 11:05:42 +03:00
Konstantinos Tsakalozos eb698629df Use the updated (RBAC enabled) cdk-addons 2017-10-09 11:04:47 +03:00
Konstantinos Tsakalozos 1c7f4cf363 Fix ingress and microbot 2017-10-09 11:04:47 +03:00
Konstantinos Tsakalozos 6e4814e948 Update e2e to use new control interface 2017-10-09 11:03:52 +03:00
Konstantinos Tsakalozos deb68518a2 Use new kube-control interface 2017-10-09 11:03:52 +03:00
Konstantinos Tsakalozos 67e2f2881b Update worker actions to use client creds 2017-10-09 11:03:52 +03:00
Konstantinos Tsakalozos ca7f8973f7 RBAC work on PoC 2017-10-09 11:03:52 +03:00
George Kraft 80f06ffee7 Bump default snap channel to 1.8/stable in juju charms 2017-10-04 13:26:14 -05:00
Kubernetes Submit Queue 2fec911a53 Merge pull request #53386 from wwwtyro/rye/fail-swap-on
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

default fail-swap-on to false for kubelet on kubernetes-worker charm

**What this PR does / why we need it**: default fail-swap-on to false for kubelet on kubernetes-worker charm

**Release note**:

```release-note
default fail-swap-on to false for kubelet on kubernetes-worker charm
```
2017-10-03 10:50:15 -07:00
Rye Terrell 4b839465f8 default fail-swap-on to false for kubelet on kubernetes-worker charm 2017-10-03 09:58:42 -05:00
Konstantinos Tsakalozos b853e29d4a Fix failing import in juju master namespace actions. 2017-09-30 07:05:33 +03:00
xiangpengzhao 4bc05f4fc2 Remove storage-class annotations in examples 2017-09-29 10:09:30 +08:00
George Kraft afd01971ea Fix lint error on kubernetes-worker 2017-09-28 08:54:06 -05:00
Kubernetes Submit Queue f25bb13a82 Merge pull request #52663 from wwwtyro/rye/s390x-ingress-controller-support
Automatic merge from submit-queue (batch tested with PRs 50988, 50509, 52660, 52663, 52250). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

s390x ingress controller support

**What this PR does / why we need it**: Adds support for an s390x ingress image to the juju kubernetes-worker charm.

**Release note**:
```
Adds support for an s390x ingress image to the juju kubernetes-worker charm.
```
2017-09-27 05:32:27 -07:00
Konstantinos Tsakalozos 90e15b4bea Fix oversized comment line, lint error 2017-09-25 16:33:19 +03:00
Kubernetes Submit Queue e9dbad33ae Merge pull request #52537 from juju-solutions/feature/s390x
Automatic merge from submit-queue (batch tested with PRs 52355, 52537, 52551, 52403, 50673). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>..

Add s390x to juju kubernetes

**What this PR does / why we need it**: With this PR we add support for s390x to juju kubernetes worker

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: 

**Special notes for your reviewer**:

**Release note**:

```Kubernetes deployments to s390x via Juju 
```
2017-09-23 15:26:54 -07:00
Kubernetes Submit Queue c6ef395023 Merge pull request #52445 from Cynerva/gkk/cdk-service-kicker
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>..

Fix kubernetes charms not restarting services properly after host reboot on LXD

**What this PR does / why we need it**:

This fixes an issue when running the Kubernetes charms on LXD where the services don't restart properly after a reboot of the host machine.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/357

**Special notes for your reviewer**:

See https://github.com/juju-solutions/layer-cdk-service-kicker

**Release note**:

```release-note
Fix kubernetes charms not restarting services properly after host reboot on LXD
```
2017-09-23 14:30:08 -07:00
Kubernetes Submit Queue 4adddfea59 Merge pull request #47912 from jacekn/runtime-config
Automatic merge from submit-queue (batch tested with PRs 51438, 52182, 51607, 47912, 51595). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>..

Add api-extra-args support to the kubernetes-master juju layer

**What this PR does / why we need it**: It adds api-extra-args config option to the kubernetes-master juju layer

**Which issue this PR fixes**: fixes #46778

**Special notes for your reviewer**:


```release-note
Add api-extra-args support to the kubernetes-master juju layer
```
2017-09-23 08:09:51 -07:00
Rye Terrell ab9f95aa8e s390x ingress controller support 2017-09-18 11:57:29 -05:00
supereagle 87c29a08e1 fix typos: remove duplicated word in comments 2017-09-16 14:38:10 +08:00
Konstantinos Tsakalozos b6f399f4fa Add s390x to juju kubernetes 2017-09-15 15:40:21 +03:00
George Kraft 783e87409c Add cdk-service-kicker to kubernetes charms 2017-09-13 14:38:03 -05:00
Kubernetes Submit Queue 4a6bbb9f50 Merge pull request #50961 from wackxu/bugfix
Automatic merge from submit-queue

fix some bad URL in the /cluster/uju/layers/kubernetes-e2e/README.md

**What this PR does / why we need it**:

There are some bad URL when I read the file and I have fix it.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

**Special notes for your reviewer**:

**Release note**:

```
NONE
```
2017-08-30 23:26:07 -07:00
Jacek Nykis d6f5acc41a Rebase runtime-config branch on top of master 2017-08-29 15:06:53 +01:00
Kubernetes Submit Queue 684066c5b5 Merge pull request #51196 from juju-solutions/bug/dead-code
Automatic merge from submit-queue (batch tested with PRs 47115, 51196, 51204, 51208, 51206)

Removing push_api_data on kube-api.connected seems to be dead code

**What this PR does / why we need it**: Removing dead code is always good :)

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

**Special notes for your reviewer**: There is no kube-api relation. This method was replace probably at some point by push_service_data firing when kube-api-endpoint.available

**Release note**:

```
```
2017-08-24 07:20:06 -07:00
George Kraft f44ed68e12 Revert default service-cidr config on kubernetes-master charm 2017-08-23 10:36:05 -05:00
Konstantinos Tsakalozos 61ed067e63 Removing push_api_data on kube-api.connected seems to be dead code 2017-08-23 17:23:24 +03:00
Konstantinos Tsakalozos f573fc5bce Fix README registry error 2017-08-22 13:27:06 +03:00
xswack 92da45c326 fix bad url in the README file 2017-08-21 11:52:32 +08:00
xswack b66a89311a fix bad url 2017-08-19 18:59:26 +08:00
George Kraft 6a354c6d29 Fix kubernetes-worker charm hook failure when applying labels 2017-08-14 13:38:07 -05:00
Kubernetes Submit Queue a9a9241db2 Merge pull request #49717 from wwwtyro/rye/block-on-failed-services
Automatic merge from submit-queue (batch tested with PRs 49712, 49694, 49714, 49670, 49717)

set juju master charm state to blocked if the services appear to be failing

**What this PR does / why we need it**: set the juju master charm state to blocked if the services appear to be failing

**Release note**:
```release-note
set the juju master charm state to blocked if the services appear to be failing
```
2017-07-28 08:15:09 -07:00
Rye Terrell 809a0abb31 set k8s master charm state to blocked if the services appear to be failing 2017-07-27 08:13:15 -05:00
Konstantinos Tsakalozos 86a036feb1 Adding old juju maintainers 2017-07-27 15:42:36 +03:00
Konstantinos Tsakalozos 68b42c3b88 Update maintainers for Juju charm layers 2017-07-25 13:09:09 +03:00
Kubernetes Submit Queue 9712a80505 Merge pull request #49394 from wwwtyro/rye/rbd-stats-improvement
Automatic merge from submit-queue (batch tested with PRs 49326, 49394, 49346, 49379, 49399)

more robust stat handling from ceph df output in the kubernetes-master charm create-rbd-pv action

**What this PR does / why we need it**: more robust stat handling from ceph df output in the kubernetes-master charm create-rbd-pv action

**Release note**:
```release-note
more robust stat handling from ceph df output in the kubernetes-master charm create-rbd-pv action
```
2017-07-21 23:23:32 -07:00
Kubernetes Submit Queue b9e549e7cf Merge pull request #49371 from juju-solutions/bug/pv-metric
Automatic merge from submit-queue (batch tested with PRs 49420, 49296, 49299, 49371, 46514)

Fix: PV metric is not namespaced

**What this PR does / why we need it**: The PV metric of juju deployments is not namespaced. This PR fixes this bug. 

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/348

**Special notes for your reviewer**:

**Release note**:

```NONE
```
2017-07-21 22:23:21 -07:00
Kubernetes Submit Queue 0af4306695 Merge pull request #49296 from juju-solutions/feature/report-failing-services
Automatic merge from submit-queue (batch tested with PRs 49420, 49296, 49299, 49371, 46514)

Update status to show failing services.

**What this PR does / why we need it**: Report on charm status any services that are not running.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/341

**Special notes for your reviewer**:

**Release note**:

```Report failing services in Juju deployed clusters.
```
2017-07-21 22:23:17 -07:00
Rye Terrell 1c21e8ab31 more robust stat handling from ceph df output 2017-07-21 11:46:32 -05:00
Konstantinos Tsakalozos 0faf5bf4b0 Fix: PV metric is not namespaced 2017-07-21 13:55:17 +03:00
Konstantinos Tsakalozos 53f00722e2 Do not try to restart services 2017-07-21 12:39:42 +03:00
Kubernetes Submit Queue 0e1744bb92 Merge pull request #49278 from juju-solutions/bug/master-msg
Automatic merge from submit-queue (batch tested with PRs 49330, 49252, 49262, 49278, 49334)

Simplify master-worker relation missing message

**What this PR does / why we need it**: Simplify messaging of missing relation in Juju deployments

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/309

**Special notes for your reviewer**:

**Release note**:

```NONE
```
2017-07-20 21:59:34 -07:00
Konstantinos Tsakalozos 685dff99ab Update status to show failing services. 2017-07-20 14:57:02 +03:00
Konstantinos Tsakalozos d5a71eb2d1 Simplify master-worker relation missing message 2017-07-20 11:47:50 +03:00
jianglingxia 5de3464755 fix the typo of Kubernetes Worker 2017-07-20 14:08:12 +08:00
Konstantinos Tsakalozos 280ea7f485 Set default CIDR to /16 2017-07-19 09:34:23 +03:00
Kubernetes Submit Queue 013d37d881 Merge pull request #48874 from juju-solutions/feature/sync-channels
Automatic merge from submit-queue (batch tested with PRs 49019, 48919, 49040, 49018, 48874)

Set default snap channel on charms to 1.7 stable

**What this PR does / why we need it**: This PR sets the default snap channel on charms to 1.7/stable. 
This addresses problems where the the user might want to deploy the charm and get the same kubernetes version found on the bundles.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/305

**Special notes for your reviewer**:

**Release note**:

```Set default snap channel on charms to 1.7/stable
```
2017-07-17 21:25:13 -07:00
Rye Terrell 7c06da5b0e prevent unsetting of nonexistent previous port in kubeapi-load-balancer charm 2017-07-17 12:22:54 -05:00
Rye Terrell ece0461cce use port configuration 2017-07-14 13:31:47 -05:00
Konstantinos Tsakalozos ca0f042854 Set default snap channel on charms to 1.7 stable 2017-07-13 16:24:48 +03:00
Kubernetes Submit Queue 40a21312d1 Merge pull request #48144 from juju-solutions/bug/worker-termination
Automatic merge from submit-queue (batch tested with PRs 48399, 48450, 48144)

Skip errors when unregistering juju kubernetes-workers

**What this PR does / why we need it**: When removing a kubernetes node from using Juju and for some reason kubernetes master fails we should not error the node, instead we should proceed with the removal of the node and the master will recognise that node as unavailable because it will fail heartbeats.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/300

**Special notes for your reviewer**:

**Release note**:

```
Clean decommission of Juju kubernetes worker units 
```
2017-07-05 08:58:39 -07:00
Kubernetes Submit Queue 7b13208d61 Merge pull request #48450 from wwwtyro/rye/lxc-disable-conntrack-max
Automatic merge from submit-queue (batch tested with PRs 48399, 48450, 48144)

configure kube-proxy to run with unset conntrack param when in lxc

**What this PR does / why we need it**: Configures the Juju Charm code to run kube-proxy with `conntrack-max-per-core` set to `0` when in an lxc as a workaround for issues when mounting `/sys/module/nf_conntrack/parameters/hashsize`

**Release note**:

```release-note
Configures the Juju Charm code to run kube-proxy with conntrack-max-per-core set to 0 when in an lxc as a workaround for issues when mounting /sys/module/nf_conntrack/parameters/hashsize
```
2017-07-05 08:58:37 -07:00
Konstantinos Tsakalozos 90a57931af Skip errors when unregistering juju kubernetes-workers 2017-07-05 16:00:37 +03:00
Kubernetes Submit Queue 3823270b9e Merge pull request #48446 from Cynerva/gkk/stop-snaps
Automatic merge from submit-queue (batch tested with PRs 47043, 48448, 47515, 48446)

Fix charms leaving services running after remove-unit

**What this PR does / why we need it**:

This fixes a case where removed charm units can sometimes leave behind running services that interfere with the rest of the cluster.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

**Special notes for your reviewer**:

**Release note**:

```release-note
Fix charms leaving services running after remove-unit
```
2017-07-04 09:12:57 -07:00
Rye Terrell 05fbc7a7f8 configure kube-proxy to run with unset conntrack param when in lxc 2017-07-03 21:58:54 -05:00
Kubernetes Submit Queue e1d9ab205a Merge pull request #48440 from Cynerva/gkk/snap-upgrades-restart-services
Automatic merge from submit-queue (batch tested with PRs 48439, 48440, 48394)

Fix kubernetes charms not restarting services after snap upgrades

**What this PR does / why we need it**:

This fixes a problem where the Kubernetes charms don't restart services after upgrading snaps. This can cause certain fixes not to be picked up (for example https://github.com/juju-solutions/release/pull/10)

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

**Special notes for your reviewer**:

**Release note**:

```release-note
Fixed kubernetes charms not restarting services after snap upgrades
```
2017-07-03 13:05:28 -07:00
Kubernetes Submit Queue 937369bc21 Merge pull request #48439 from juju-solutions/bug/namespaces-path
Automatic merge from submit-queue (batch tested with PRs 48439, 48440, 48394)

Fix: namespace-create have kubectl in path

**What this PR does / why we need it**: In juju deployed clusters namespace-create action is failing

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/326

**Special notes for your reviewer**:

**Release note**:

```Fix: namespace-create action on Juju deployed clusters
```
2017-07-03 13:05:27 -07:00
George Kraft f0e08818d8 Fix charms leaving services running after unit removal 2017-07-03 14:55:07 -05:00
Konstantinos Tsakalozos cea934bcbc Fix: namespace-create have kubectl in path 2017-07-03 18:22:20 +03:00
George Kraft c21b305fe9 Fix kubernetes charms not restarting services after snap upgrades 2017-07-03 09:47:08 -05:00