Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Kubernetes deployment example #23

Merged
merged 2 commits into from
Jan 1, 2019
Merged

Add Kubernetes deployment example #23

merged 2 commits into from
Jan 1, 2019

Conversation

razius
Copy link
Contributor

@razius razius commented Jan 1, 2019

No description provided.

@adamdecaf
Copy link
Collaborator

Could 76eb0b4 be rebased out of this PR? I'm not sure why it was included.

There's a build error with that commit anyway: https://travis-ci.org/shift/domain_exporter/jobs/474077816#L583

apiVersion: v1
kind: ConfigMap
metadata:
name: config
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd name this domain-exporter instead of "config" to not overlap with another ConfigMap.

@razius
Copy link
Contributor Author

razius commented Jan 1, 2019

@adamdecaf done. Do you want a separate PR with the fix for the build error?

@adamdecaf
Copy link
Collaborator

Yea please! That error looks to be related to updated dependencies?

@razius
Copy link
Contributor Author

razius commented Jan 1, 2019

Yes it is. Seems related to this issue: prometheus/common#136. I'll read a bit more and send a PR.

@adamdecaf adamdecaf merged commit d01e339 into shift:master Jan 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants