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.

Sign up for GitHub

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

Jump to bottom

Influxdb migrated to PetSet and PersistentVolumes. #28109

Merged
merged 1 commit into from
Jun 29, 2016
Merged

Influxdb migrated to PetSet and PersistentVolumes. #28109

merged 1 commit into from
Jun 29, 2016
+90 −83

Conversation

jszczepkowski
Copy link
Contributor 

Influxdb migrated to PetSet and PersistentVolumes.

Analytics

Influxdb migrated to PetSet and PersistentVolumes.

@k8s-github-robot k8s-github-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. release-note-label-needed labels Jun 27, 2016
@jszczepkowski jszczepkowski force-pushed the influx-ps branch 2 times, most recently from af7db11 to 759c161 Compare June 27, 2016 11:33
@k8s-bot
Copy link

k8s-bot commented Jun 27, 2016

GCE e2e build/test failed for commit e82a7f6e004f7d969ad60e08bac90b3170bc012d.

Please reference the list of currently known flakes when examining this failure. If you request a re-test, you must reference the issue describing the flake.

@jszczepkowski jszczepkowski assigned piosz and unassigned roberthbailey Jun 27, 2016
@jszczepkowski
Copy link
Contributor Author

CC @mwielgus

@jszczepkowski jszczepkowski added release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-label-needed labels Jun 27, 2016
@k8s-bot
Copy link

k8s-bot commented Jun 27, 2016

GCE e2e build/test failed for commit af7db11403c6cf2415895a845d6553e9ca08c26b.

Please reference the list of currently known flakes when examining this failure. If you request a re-test, you must reference the issue describing the flake.

@k8s-bot
Copy link

k8s-bot commented Jun 27, 2016

GCE e2e build/test failed for commit 759c161ee3e08923b098dcfdd5e95afdfa461c6e.

Please reference the list of currently known flakes when examining this failure. If you request a re-test, you must reference the issue describing the flake.

@jszczepkowski
Influxdb migrated to PetSet and PersistentVolumes.
d00cdf7 
Influxdb migrated to PetSet and PersistentVolumes.
@jszczepkowski
Copy link
Contributor Author

Resolves kubernetes-retired/heapster#1197.

@k8s-bot
Copy link

k8s-bot commented Jun 27, 2016

GCE e2e build/test passed for commit d00cdf7.

piosz
piosz reviewed Jun 27, 2016
View reviewed changes
cluster/addons/cluster-monitoring/influxdb/influxdb-claim.yaml
- ReadWriteOnce
resources:
requests:
storage: 10Gi
Copy link
Member

Choose a reason for hiding this comment

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

@mwielgus do you think this value is reasonable?

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe :)

@piosz
Copy link
Member

piosz commented Jun 27, 2016

LGTM

@piosz piosz added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 27, 2016
@k8s-bot
Copy link

k8s-bot commented Jun 29, 2016

GCE e2e build/test passed for commit d00cdf7.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit d4e6064 into kubernetes:master Jun 29, 2016
@jszczepkowski jszczepkowski mentioned this pull request Aug 4, 2016
Merged
@piosz piosz mentioned this pull request Jan 12, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@piosz piosz

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@jszczepkowski @k8s-bot @piosz @k8s-github-robot @mwielgus @googlebot @roberthbailey