Skip to content
This repository has been archived by the owner on Mar 29, 2023. It is now read-only.

Use Stackdriver Kubernetes Engine Monitoring as default instead of Legacy Stackdriver #42

Merged
merged 1 commit into from
Jun 3, 2019

Conversation

rileykarson
Copy link
Contributor

Fixes #41

This was released as GA on May 20th. It will become the default logging / monitoring service for GKE beginning in 1.14.X series releases, which will go GA ~3 months after 1.13.X (May 13th).

As I understand it, this isn't a breaking change for Kubernetes, but is for Stackdriver as the cluster will output different fields. We should probably bump our minor version based on that, or hold this is a default until later.

@autero1
Copy link
Contributor

autero1 commented Jun 3, 2019

We should probably bump our minor version based on that

+1

Copy link
Contributor

@autero1 autero1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I re-ran the tests and they passed. I would set monitoring.googleapis.com/kubernetes as default value and bump the minor version.

Copy link
Contributor

@robmorgan robmorgan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

description = "The logging service that the cluster should write logs to. Available options include logging.googleapis.com, logging.googleapis.com/kubernetes (beta), and none"
default = "logging.googleapis.com"
description = "The logging service that the cluster should write logs to. Available options include logging.googleapis.com/kubernetes, logging.googleapis.com (legacy), and none"
default = "logging.googleapis.com/kubernetes"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@rileykarson rileykarson merged commit d49470a into master Jun 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stackdriver support
3 participants