Skip to content

Materials for Grafanacon 2019#866

Closed
bwplotka wants to merge 1 commit into
masterfrom
grafanacon-la-2019
Closed

Materials for Grafanacon 2019#866
bwplotka wants to merge 1 commit into
masterfrom
grafanacon-la-2019

Conversation

@bwplotka

Copy link
Copy Markdown
Member

Signed-off-by: Bartek Plotka bwplotka@gmail.com

@bwplotka
bwplotka force-pushed the grafanacon-la-2019 branch 5 times, most recently from 1103f1d to 7d0cdbb Compare February 25, 2019 23:39
Signed-off-by: Bartek Plotka <bwplotka@gmail.com>

@domgreen domgreen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM - just nits but nothing blocking

mountPath: /var/prometheus
- name: thanos
image: improbable/thanos:v0.3.1
image: improbable/thanos:v0.3.2-rc.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

any way we can template this or get it from the VERSION file?

# pipeline is functional.
# expr: vector(1)
# labels:
# severity: none

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

delete rather than comment

@bwplotka

bwplotka commented Oct 3, 2019

Copy link
Copy Markdown
Member Author

Not actionable anymore.

@bwplotka bwplotka closed this Oct 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants