Skip to content

fix k8s storage urls#838

Merged
robskillington merged 1 commit into
m3db:masterfrom
poblahblahblah:fix-kube-storage-url
Aug 10, 2018
Merged

fix k8s storage urls#838
robskillington merged 1 commit into
m3db:masterfrom
poblahblahblah:fix-kube-storage-url

Conversation

@poblahblahblah

Copy link
Copy Markdown
Contributor

This fixes the k8s storage urls in the docs.

@CLAassistant

CLAassistant commented Aug 9, 2018

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@codecov

codecov Bot commented Aug 9, 2018

Copy link
Copy Markdown

Codecov Report

Merging #838 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #838      +/-   ##
==========================================
+ Coverage   78.44%   78.46%   +0.01%     
==========================================
  Files         369      369              
  Lines       31938    31938              
==========================================
+ Hits        25055    25060       +5     
+ Misses       5185     5181       -4     
+ Partials     1698     1697       -1
Flag Coverage Δ
#dbnode 81.48% <ø> (+0.01%) ⬆️
#m3ninx 72.7% <ø> (ø) ⬆️
#query 64.57% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9eb3c55...21e46d5. Read the comment docs.

@robskillington

robskillington commented Aug 10, 2018

Copy link
Copy Markdown
Collaborator

Thanks a lot for catching this, hopefully at some point we can find a way of writing an integration test to avoid this guide regressing in the future. Appreciated!

@robskillington robskillington left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@robskillington robskillington merged commit 8694e3e into m3db:master Aug 10, 2018
@poblahblahblah poblahblahblah deleted the fix-kube-storage-url branch August 10, 2018 15:05
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.

3 participants